refactor normal select

[LawyZheng]

Important

Refactor normal_select in handler.py to enhance option selection using LLM API and add new prompt template normal-select.j2.

• Behavior:
  • Refactor normal_select in handler.py to use LLM API for option selection, adding parameters dom, task, and step.
  • Remove FailToSelectByLabel exception handling in handler.py.
• Prompts:
  • Add normal-select.j2 template for generating selection prompts based on user context and options.
• Logging:
  • Change logging from LOG.error to LOG.info in normal_select for click failures and selection attempts.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 275bd71 in 27 seconds

More details

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

1. skyvern/webeye/actions/handler.py:809

• Draft comment:
  The FailToSelectByLabel exception handling has been removed. Consider adding it back to handle cases where selection by label fails, as this could aid in debugging and understanding failures.
• Reason this comment was not posted:
  Comment was not on a valid diff hunk.

2. skyvern/webeye/actions/handler.py:2241

• Draft comment:
  The logging level has been changed from error to info for several log messages. Consider whether these should remain as error to ensure visibility in case of failures. This applies to other similar changes in the file as well.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The refactoring in normal_select function has changed the logging level from error to info for several log messages. This might reduce the visibility of these logs in case of failures, as error logs are typically more visible than info logs.

Workflow ID: wflow_O3RuPHGYPYpI4S5d

---

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