Skip to content

docs: fix single-step tutorial response output - #2702

Merged
ananthsub merged 1 commit into
mainfrom
ananthsub/fix-single-step-tutorial-output
Aug 26, 2026
Merged

docs: fix single-step tutorial response output#2702
ananthsub merged 1 commit into
mainfrom
ananthsub/fix-single-step-tutorial-output

Conversation

@ananthsub

@ananthsub ananthsub commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add the required type: "message" discriminator to the tutorial's assistant output fixture.
  • Mark the message as completed so the copied verifier test matches the current Responses schema.

Test plan

  • Replayed the tutorial scaffold, app, config, data, and tests
  • Ran gym env test --resources-server my_weather_tool.
  • Ran cd fern && npm run check.

Closes #2695.

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added the sla:review-overdue Review response is over the one-business-day SLA label Aug 25, 2026
@ananthsub
ananthsub requested a review from init-nikhil August 25, 2026 20:17
@ananthsub
ananthsub force-pushed the ananthsub/fix-single-step-tutorial-output branch from a817d98 to 6cca686 Compare August 25, 2026 20:17
@copy-pr-bot

copy-pr-bot Bot commented Aug 25, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ananthsub

Copy link
Copy Markdown
Contributor Author

/ok to test 6cca686

Add the required Responses message discriminator and completion status so the documented verifier test validates against the current schema.

Signed-off-by: Ananth Subramaniam <ansubramania@nvidia.com>
@ananthsub
ananthsub force-pushed the ananthsub/fix-single-step-tutorial-output branch from 6cca686 to a6ba967 Compare August 26, 2026 08:42
@ananthsub
ananthsub enabled auto-merge (squash) August 26, 2026 08:42
@ananthsub

Copy link
Copy Markdown
Contributor Author

/ok to test a6ba967

@ananthsub
ananthsub merged commit 501a752 into main Aug 26, 2026
19 checks passed
@ananthsub
ananthsub deleted the ananthsub/fix-single-step-tutorial-output branch August 26, 2026 09:37
ananthsub added a commit that referenced this pull request Aug 26, 2026
## Summary
- Update the generated file tree to include the package initializers and
verifier fixture cases.
- Preserve the scaffolded stateless verifier, verifier mixin, and
`VERIFIER_FIXTURE` while adding the tutorial's weather tool behavior.
- Document fixture cases for the reward contract and retain a focused
unit test for the weather tool.
- Retain the Responses message discriminator and completion status
correction from #2702.

## Test plan
- [x] `cd fern && npm run check`
- [x] Replay the documented app, verifier cases, and tests against the
current scaffold

Closes #2696.

---------

Signed-off-by: Ananth Subramaniam <ansubramania@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sla:review-overdue Review response is over the one-business-day SLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[VDR][v0.6.0] TUT-63304da8 · Single-step tutorial's own unit test fails: output items now require a type discriminator

2 participants