refactor: tooltips and placeholders

[wintonzheng]

Important

Centralize tooltips and placeholders in helpContent.ts and update node components to use this centralized content.

• Refactor:
  • Move commonHelpTooltipContent and commonFieldPlaceholders from constants.ts to helpContent.ts.
  • Create baseHelpTooltipContent and basePlaceholderContent in helpContent.ts.
  • Define helpTooltips and placeholders in helpContent.ts for various node types.
• Node Components:
  • Update ActionNode.tsx, DownloadNode.tsx, ExtractionNode.tsx to use helpTooltips and placeholders from helpContent.ts.
  • Update FileDownloadNode.tsx, FileParserNode.tsx, LoginNode.tsx to use centralized content.
  • Update LoopNode.tsx, NavigationNode.tsx, SendEmailNode.tsx to use centralized content.
  • Update TaskNode.tsx, TextPromptNode.tsx, UploadNode.tsx to use centralized content.
  • Update ValidationNode.tsx, WaitNode.tsx to use centralized content.
• Types:
  • Remove helpTooltipContent and fieldPlaceholders from individual node types.ts files.

^{This description was created by for 9624c4c. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Incremental review on 0e1dcf2 in 25 seconds

More details

• Looked at 1507 lines of code in 26 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. skyvern-frontend/src/routes/workflows/editor/nodes/FileDownloadNode/FileDownloadNode.tsx:27

• Draft comment:
  Consider using the centralized helpTooltips and placeholders for consistency.

const urlTooltip = helpTooltips["download"]["url"];
const urlPlaceholder = placeholders["download"]["url"];

• Reason this comment was not posted:
  Confidence changes required: 50%
  The refactoring seems consistent across the files, with the centralized help content being used correctly. However, there is a minor inconsistency in the FileDownloadNode.tsx file where the urlTooltip and urlPlaceholder constants are defined but not used. Instead, the centralized helpTooltips and placeholders should be used for consistency.

Workflow ID: wflow_hf2KB1m0p2QDEJGe

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 0e1dcf2 in 38 seconds

More details

• Looked at 1524 lines of code in 27 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. skyvern-frontend/src/routes/workflows/editor/nodes/ActionNode/ActionNode.tsx:26

• Draft comment:
  Consider using helpTooltips["action"]["url"] instead of the hardcoded urlTooltip for consistency with the centralized content.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The refactoring of tooltips and placeholders is consistent across the codebase. However, there are some hardcoded tooltips and placeholders in the ActionNode.tsx file that should be replaced with the centralized content from helpContent.ts.

Workflow ID: wflow_k8kAsT6TPUxnyjIh

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.