fix extraction block bug

[LawyZheng]

Important

Fixes logic in agent_step() to correctly handle ValidationBlock tasks without a navigation goal.

• Bug Fix:
  • Fixes logic in agent_step() in agent.py to correctly handle tasks of type ValidationBlock.
  • Ensures CompleteAction is not created for ValidationBlock tasks without a navigation goal.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to f06c1e3 in 12 seconds

More details

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

1. skyvern/forge/agent.py:635

• Draft comment:
  The condition if not task.navigation_goal and not isinstance(task_block, ValidationBlock): is correct, but it can be simplified for better readability. Consider using a single line for the condition check.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The change in the condition is correct, but the logic can be simplified for better readability.

Workflow ID: wflow_BU7MjNKujfViao62

---

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