Repository navigation
Upgrade Zod to fix collection validation advisory - #189
Merged
Merged
Conversation
sgasser
enabled auto-merge (squash)
November 8, 2026 09:31
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Zod 3.25.76 is affected by GHSA-mrhx-c7xc-m3w5, which can exhaust memory while validating oversized constrained collections. Upgrade to Zod 4.6.7 and preserve existing configuration defaults, Anthropic tool records, and validation-error handling through the required Zod 4 compatibility changes.
Added regression coverage for omitted configuration sections and masking API error paths. The upstream patch rejects oversized collections before parsing elements when a schema declares an upper bound; existing unbounded request collections retain their current behavior.
The Test & Lint job now runs in the same digest-pinned Bun image as the production Docker build. This removes the Bun release download step, which repeatedly failed with HTTP 504 before checks could run.
Validation:
bun test: 515 passed; one optional Postgres integration test skipped.linux/amd64;actionlint .github/workflows/ci.ymlalso passed.bun run typecheck,bun run typecheck:benchmarks,bun run check, andbun run build: passed.linux/arm64; 19 live checks passed with its real in-container GLiNER detector and local HTTP provider fixtures. Verified normal supervisor startup, nonroot execution, Docker health, installed Zod, masking/restoration, streaming, tool records, and validation errors. Hosted LLM inference was not exercised.Application changes reviewed with sol-loop: 1 round with GPT-6.1 Sol on high effort. Sol verdict: approved; no findings.