Skip to content

Keep JSON response transformations consistent - #295

Merged
patchstackdave merged 2 commits into
mainfrom
codex/response-json-values
Sep 28, 2026
Merged

patchstackdave merged 2 commits into
mainfrom
codex/response-json-values

Conversation

@patchstackdave

@patchstackdave patchstackdave commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Keep numeric JSON values aligned between matching and response transformation, while preserving untouched large integer tokens and literal strings.

Validate whole-document JSON before tokenization. Check each text-span rewrite for unchanged keys, containers, and non-string tokens; string values may change. Explicit path transformations retain their existing replacement semantics. An incompatible rewrite uses the existing generic withheld response.

Synthetic tests cover numeric leaves, literal strings, custom masks, transformation ordering, object and array structure, dry-run behaviour, and the Node response path.

Validation: full suite (3,428 passed, 7 skipped), typecheck, build, and unchanged capability and rule contracts.

@coderbuds

coderbuds Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Comprehensive JSON transformation with robust structure and big-int handling.

🎯 Quality: 100% Elite · 📦 Size: Medium

🤖 Authorship: Probably agent-written — OpenAI Codex, going by its branch name. Whether a person read it is unknown; coding agents can call the report-ai-usage tool to say.

📈 This month: Your 147th PR — above team average · Averaging Excellent

See how your team is trending →

@patchstackdave

Copy link
Copy Markdown
Contributor Author

/review

@patchstackdave
patchstackdave merged commit f5c268b into main Sep 28, 2026
18 checks passed
@patchstackdave
patchstackdave deleted the codex/response-json-values branch September 28, 2026 14:00
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.

2 participants