Skip to content

fix(solicitation-create): unwrap data: {...} envelope in Step 6 code example#413

Merged
jjackson merged 1 commit into
mainfrom
emdash/solicitation-code-example-flat
May 22, 2026
Merged

fix(solicitation-create): unwrap data: {...} envelope in Step 6 code example#413
jjackson merged 1 commit into
mainfrom
emdash/solicitation-code-example-flat

Conversation

@jjackson
Copy link
Copy Markdown
Owner

Tiny follow-up to PR #411. The verification re-run of /ace:step solicitation-create (malaria-itn-app/20260521-1400 — solicitation 3141, successful clean execution) flagged that the Step 6 code-block example still showed the wrapped data: {...} shape even though #411 corrected the surrounding prose to flat fields. A literal reader who copy-pasted the code block would have produced a payload labs now rejects under additionalProperties: false. Code block fixed to match the prose + live tools/list contract.

Also tightened the criterion shape in the example to lead with id (canonical labs field order) and the question shape to lead with id, text, type (the three required fields per live schema).

🤖 Generated with Claude Code

…example

The verification re-run of /ace:step solicitation-create
(malaria-itn-app/20260521-1400 — solicitation 3141) flagged that
the Step 6 code-block example still showed the wrapped data: {...}
shape even though PR #411 corrected the surrounding prose to flat
fields.

A literal reader who copy-pasted the code block would have ended
up with a payload labs rejects with additionalProperties:false.
Fixed the code block to match the prose + live tools/list contract.

Also tightened the criterion shape in the example to lead with
`id` (canonical labs field order) and the question shape to lead
with `id, text, type` (the three required fields per live schema).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@jjackson jjackson enabled auto-merge May 22, 2026 09:48
@jjackson jjackson merged commit 62b3067 into main May 22, 2026
2 checks passed
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.

1 participant