Skip to content

Constrain TResponse.json to JSON values - #445

Merged
mismosmi merged 3 commits into
mainfrom
issue-426
Aug 14, 2026
Merged

Constrain TResponse.json to JSON values#445
mismosmi merged 3 commits into
mainfrom
issue-426

Conversation

@ghost

@ghost ghost commented Aug 14, 2026

Copy link
Copy Markdown

Summary

  • constrain TResponse.json to recursively JSON-compatible values
  • add compile-time coverage for accepted and rejected response values
  • update form-data mocks to avoid returning potentially non-JSON File values
  • add a changeset for the affected packages

Verification

  • git diff --check passes
  • Local TypeScript/Vitest binaries were unavailable in the sandbox; CI should run the package checks

Resolves #426

🤖 Open this thread in ColorCode

@ghost ghost added the ColorCode label Aug 14, 2026
@changeset-bot

changeset-bot Bot commented Aug 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ea94c64

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@toapi/common Patch
@toapi/server Patch
@toapi/client Patch
@toapi/worker Patch
@toapi/react Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ghost

ghost commented Aug 14, 2026

Copy link
Copy Markdown
Author

✅ Review passed

Every acceptance criterion is met and every review comment is resolved.
ColorCode has nothing outstanding on this pull request.

@ghost
ghost marked this pull request as ready for review August 14, 2026 14:20
Comment thread .changeset/tresponse-json-constraint.md Outdated
@mismosmi
mismosmi enabled auto-merge August 14, 2026 14:28
@ghost

ghost commented Aug 14, 2026

Copy link
Copy Markdown
Author

✅ Review passed

Every acceptance criterion is met and every review comment is resolved.
ColorCode has nothing outstanding on this pull request.

@mismosmi
mismosmi merged commit 98834fd into main Aug 14, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

json-responses accept structured types but client returns strings

1 participant