make MissingValueForParameter a 400 error

[wintonzheng]

Important

MissingValueForParameter now returns a 400 status code for missing parameter values in workflow runs.

• Behavior:
  • MissingValueForParameter in exceptions.py now returns a 400 status code for missing parameter values in workflow runs.
• Misc:
  • Added status_code=status.HTTP_400_BAD_REQUEST to MissingValueForParameter constructor.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to b1282ff in 24 seconds

More details

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

1. skyvern/exceptions.py:156

• Draft comment:
  Setting status_code to HTTP_400_BAD_REQUEST is correct and consistent with our convention. LGTM.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50%
  None

2. skyvern/exceptions.py:156

• Draft comment:
  Explicitly passing status_code clarifies that this exception returns HTTP 400. Since HTTP_400_BAD_REQUEST is the default in SkyvernHTTPException, this may be redundant, but it's clear and consistent with the intent.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50%
  None

Workflow ID: wflow_TtgfCwRI9lkWKoX0

---

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