fix forloop output parameter issue

[LawyZheng]

Important

Fixes output parameter handling in execute_loop_helper in block.py by setting output_value to None if not present in workflow_run_context.

• Behavior:
  • Fixes issue in execute_loop_helper in block.py where output_value is now set to None if workflow_run_context does not have the output_parameter.key.
• Error Handling:
  • Adds error handling for missing output_parameter values in loop execution, preventing potential errors when values are not set.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 49e9ee0 in 10 seconds

More details

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

1. skyvern/forge/sdk/workflow/models/block.py:851

• Draft comment:
  Ensure that the get_value method is safe to call without checking has_value first, or else this pattern should be used consistently throughout the codebase.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The change in the PR ensures that if the workflow_run_context does not have a value for the output parameter key, None is returned instead of causing an error. This is a good practice to prevent potential runtime errors when accessing keys that might not exist.

Workflow ID: wflow_WPfELXG2MGPAylPN

---

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