Skip to content

fix(runtime): validate body-backed required inputs#73

Merged
samzong merged 1 commit into
mainfrom
fix/runtime-body-input-validation
Jun 26, 2026
Merged

fix(runtime): validate body-backed required inputs#73
samzong merged 1 commit into
mainfrom
fix/runtime-body-input-validation

Conversation

@samzong

@samzong samzong commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

Verification

  • make check
  • go test -count=1 ./pkg/runtime -run "TestBuild_RequiredVariableCanComeFromBodyInput|TestBuildBodyFromSet_ArrayIndex|TestBuildEnvelopeBody|TestBuild_VariableFlagsMergeIntoEnvelope"

Compatibility

  • Generated CLI body behavior changes only: body-backed required variables can be provided through merged body input, and missing required body fields are still rejected before sending the request.
  • Catalog schema and generated output are unchanged.

Checklist

  • Tests or focused verification cover the changed surface.
  • User-facing behavior changes are covered by existing body flag help; no docs update needed for this bug fix.
  • Generated output under internal/generated/, .cache/, and ad-hoc skills/<cli-name>/ directories is not committed.
  • Commits are signed off when this is ready to merge.

Signed-off-by: samzong <samzong.lu@gmail.com>
@samzong samzong merged commit dad8002 into main Jun 26, 2026
3 checks passed
@samzong samzong deleted the fix/runtime-body-input-validation branch June 26, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(runtime): support explicit empty array body inputs bug(runtime): validate required fields after merging file and set bodies

1 participant