Use popover in add parameter experience

[wintonzheng]

Important

Refactor parameter addition UI in node components by integrating a popover-based interface for a consistent user experience.

• UI Enhancement:
  • Integrate Popover for parameter addition in WorkflowBlockInput and WorkflowBlockInputTextarea.
  • Replace onIconClick with PopoverTrigger in WorkflowBlockInput and WorkflowBlockInputTextarea.
• Node Components:
  • Update ActionNode, ExtractionNode, FileDownloadNode to use Popover for parameter selection.
  • Remove parametersPanelField state and related logic from node components.
• Parameter Selection:
  • Modify WorkflowBlockParameterSelect to remove onClose prop and adjust layout for popover usage.

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

[ellipsis-dev]

👍 Looks good to me! Incremental review on f58fa79 in 23 seconds

More details

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

1. skyvern-frontend/src/routes/workflows/editor/nodes/WorkflowBlockParameterSelect.tsx:80

• Draft comment:
  The ?. operator is unnecessary here since onAdd is a required prop and will always be defined. Remove the ?. for clarity.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The onAdd function is called with a parameter key, but the ?. operator is unnecessary since onAdd is a required prop and will always be defined. This is a minor issue but should be corrected for clarity and consistency.

Workflow ID: wflow_W0CINBsQY5WL5oHD

---

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 f58fa79 in 28 seconds

More details

• Looked at 993 lines of code in 10 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. skyvern-frontend/src/components/WorkflowBlockInput.tsx:35

• Draft comment:
  Ensure props.value is up-to-date before using it in the onAdd callback to avoid potential issues with stale values.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The onAdd function is called with a parameter key, but the props.value is not updated before using it in the onAdd callback. This could lead to incorrect values being used if props.value is not up-to-date.

2. skyvern-frontend/src/components/WorkflowBlockInputTextarea.tsx:38

• Draft comment:
  Ensure props.value is up-to-date before using it in the onAdd callback to avoid potential issues with stale values.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The onAdd function is called with a parameter key, but the props.value is not updated before using it in the onAdd callback. This could lead to incorrect values being used if props.value is not up-to-date.

Workflow ID: wflow_yDKBGbKeHff5peD9

---

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