Skip to content
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

Merged
merged 3 commits into from
Oct 17, 2024

Conversation

wintonzheng
Copy link
Contributor

@wintonzheng wintonzheng commented Oct 17, 2024

Important

Enhance user goal completion prompts with detailed state and reasoning fields, and add a frontend component for display mode switching.

  • Behavior:
    • Updated check-user-goal.j2 to include 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.
  • Frontend:
    • Added TaskNodeDisplayModeSwitch component in TaskNodeDisplayModeSwitch.tsx for switching between basic and advanced display modes.

This description was created by Ellipsis for e11dbcb. It will automatically update as commits are pushed.

<!-- 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 -->
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a 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 in 1 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.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a 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 in 3 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.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a 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 in 1 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.

@wintonzheng wintonzheng merged commit e56aae4 into main Oct 17, 2024
2 checks passed
@wintonzheng wintonzheng deleted the shu/update_user_goal_completion_prompt branch October 17, 2024 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant