-
Notifications
You must be signed in to change notification settings - Fork 944
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update user goal complete prompt to have more chain of thoughts #990
Conversation
<!-- ELLIPSIS_HIDDEN --> > [!IMPORTANT] > Enhance user goal completion and action extraction prompts by updating JSON structure and descriptions in `check-user-goal.j2` and `extract-action.j2`. > > - **Behavior**: > - Updated `check-user-goal.j2` to include a `state` field in JSON output, describing page state and user progress. > - Enhanced `reasoning` field in `check-user-goal.j2` for precise explanations using site information. > - Modified `extract-action.j2` to clarify use of user details for filling values and ensuring valid JSON output. > - **Examples**: > - Updated examples in `check-user-goal.j2` to reflect new `state` and `reasoning` fields. > - **Misc**: > - Minor text adjustments in `extract-action.j2` for clarity. > > <sup>This description was created by </sup>[<img alt="Ellipsis" src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=Skyvern-AI%2Fskyvern-cloud&utm_source=github&utm_medium=referral)<sup> for 72b81f02deede3de5420d6c5ce1dad0d37a43fcc. It will automatically update as commits are pushed.</sup> <!-- ELLIPSIS_HIDDEN -->
…src/' <!-- ELLIPSIS_HIDDEN --> > [!IMPORTANT] > Enhance user goal completion and action extraction prompts by updating JSON structure and descriptions in `check-user-goal.j2` and `extract-action.j2`. > > - **Behavior**: > - Updated `check-user-goal.j2` to include a `state` field in JSON output, describing page state and user progress. > - Enhanced `reasoning` field in `check-user-goal.j2` for precise explanations using site information. > - Modified `extract-action.j2` to clarify use of user details for filling values and ensuring valid JSON output. > - **Examples**: > - Updated examples in `check-user-goal.j2` to reflect new `state` and `reasoning` fields. > - **Misc**: > - Minor text adjustments in `extract-action.j2` for clarity. > > <sup>This description was created by </sup>[<img alt="Ellipsis" src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=Skyvern-AI%2Fskyvern-cloud&utm_source=github&utm_medium=referral)<sup> for 72b81f02deede3de5420d6c5ce1dad0d37a43fcc. It will automatically update as commits are pushed.</sup> <!-- ELLIPSIS_HIDDEN -->
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Incremental review on 9827e4c in 8 seconds
More details
- Looked at
42
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. skyvern-frontend/src/routes/workflows/editor/nodes/TaskNode/TaskNodeDisplayModeSwitch.tsx:4
- Draft comment:
Consider adding PropTypes validation to ensure the correct types are passed as props. - Reason this comment was not posted:
Confidence changes required:50%
The code is missing PropTypes validation, which is a best practice for React components to ensure the correct types are passed as props.
Workflow ID: wflow_RvgLcA2IShjBfFYl
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Reviewed everything up to 9827e4c in 19 seconds
More details
- Looked at
98
lines of code in3
files - Skipped
0
files when reviewing. - Skipped posting
3
drafted comments based on config settings.
1. skyvern/forge/prompts/skyvern/check-user-goal.j2:1
- Draft comment:
Typo: Replace 'Your are' with 'You are'.
You are here to help the user determine if the user has completed their goal on the web. Use the content of the elements parsed from the page, the user goal and user details to determine whether the user goal has been completed or not.
- Reason this comment was not posted:
Confidence changes required:10%
The word 'Your' is misspelled as 'Your are' instead of 'You are'.
2. skyvern/forge/prompts/skyvern/check-user-goal.j2:6
- Draft comment:
Typo: Replace 'accoriding' with 'according'.
"state": str, // Describe the state of the page according to the user goal. Be precise and talk about what progress the user made, and what the next step is to move towards the user goal if any.
- Reason this comment was not posted:
Confidence changes required:10%
The word 'according' is misspelled as 'accoriding'.
3. skyvern/forge/prompts/skyvern/check-user-goal.j2:8
- Draft comment:
JSON does not support comments. Consider removing comments or documenting them outside the JSON block. This issue is also present on lines 6 and 7. - Reason this comment was not posted:
Confidence changes required:50%
The comment format in JSON is incorrect as JSON does not support comments. This is a common issue in multiple places.
Workflow ID: wflow_kmJ3Homy7uGFrr2c
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
…skNodeDisplayModeSwitch.tsx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Incremental review on e11dbcb in 8 seconds
More details
- Looked at
40
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_rGRtMMZKyy1cagHG
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
Important
Enhance user goal completion prompts with detailed state and reasoning fields, and add a frontend component for display mode switching.
check-user-goal.j2
to includestate
field in JSON output, describing page state and user progress.reasoning
field incheck-user-goal.j2
for precise explanations using site information.extract-action.j2
to clarify use of user details for filling values and ensuring valid JSON output.check-user-goal.j2
to reflect newstate
andreasoning
fields.TaskNodeDisplayModeSwitch
component inTaskNodeDisplayModeSwitch.tsx
for switching between basic and advanced display modes.This description was created by
for e11dbcb. It will automatically update as commits are pushed.