do not take fullpage screenshot post CUA action

[wintonzheng]

Important

Modify record_artifacts_after_action in agent.py to disable full-page screenshots for RunEngine.openai_cua.

• Behavior:
  • Modify record_artifacts_after_action in agent.py to conditionally disable full-page screenshots for RunEngine.openai_cua.
  • Update calls to record_artifacts_after_action in agent_step() to pass engine parameter.
• Functions:
  • Add engine parameter to record_artifacts_after_action() to control screenshot behavior.
  • Update agent_step() to pass engine to record_artifacts_after_action() in three places.

^{This description was created by for cb5dcc5. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Caution

Changes requested ❌

Reviewed everything up to cb5dcc5 in 1 minute and 56 seconds. Click for details.

• Reviewed 57 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 4 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. skyvern/forge/agent.py:1455

• Draft comment:
  Good change: Using the engine parameter to conditionally disable full-page screenshots for openai_cua improves efficiency.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

2. skyvern/forge/agent.py:1444

• Draft comment:
  Ensure all calls to record_artifacts_after_action are updated with the new engine parameter. The changes seem consistent.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

3. skyvern/forge/agent.py:1455

• Draft comment:
  For clarity, consider using a ternary operator for setting fullpage_screenshot, e.g. 'fullpage_screenshot = False if engine == RunEngine.openai_cua else True'.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

4. skyvern/forge/agent.py:1293

• Draft comment:
  Typo: 'skyvern_repsonse_prompt' appears to be misspelled. It should likely be 'skyvern_response_prompt'. Please correct this for clarity.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

Workflow ID: wflow_DEBsrGQt3gB4tj8d

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[ellipsis-dev]

Since the new engine parameter controls whether a full-page screenshot is taken (with if engine == RunEngine.openai_cua), consider updating the function’s docstring/comments to document this behavior. Simplify the assignment by writing: fullpage_screenshot = (engine != RunEngine.openai_cua) for clarity.