Skip to content

chore(skill-harness): sync vendored validator bundle (zod 4.6.6) - #55

Open
oschaaf wants to merge 1 commit into
mainfrom
chore/validator-bundle-sha
Open

oschaaf wants to merge 1 commit into
mainfrom
chore/validator-bundle-sha

Conversation

@oschaaf

@oschaaf oschaaf commented Oct 24, 2026

Copy link
Copy Markdown
Member

Re-vendors the gateway config validator bundle that skill-harness uses, rebuilt against zod 4.6.6, and updates EXPECTED_VALIDATOR_BUNDLE_SHA256 to match.

zod 4.6.6 fixes GHSA-mrhx-c7xc-m3w5 (memory exhaustion when validating oversized constrained collections).

  • vendor/config-validator.bundle.mjs is copied verbatim from the upstream build; its sha256 matches the upstream published digest exactly. The type shim (.d.mts) is unchanged.
  • The previous pin had already drifted from upstream, so this also brings the vendored copy back in step.
  • Harness-only change, so no plugin.json version bump.

Checks: pnpm test (181/181), biome:check, typecheck, and the schema-digest --check (which asserts the bundle matches the pinned sha).

@oschaaf
oschaaf requested a review from nicobatalla October 24, 2026 17:16
Re-vendor the gateway config validator bundle built against zod 4.6.6, which fixes GHSA-mrhx-c7xc-m3w5, and update EXPECTED_VALIDATOR_BUNDLE_SHA256 to match. The bundle was copied verbatim from the upstream build; the type shim is unchanged. Harness-only change, so no plugin.json version bump.

@pareilly pareilly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but needs a rebase

@oschaaf
oschaaf force-pushed the chore/validator-bundle-sha branch from b5aaeff to 8b3d4e7 Compare October 27, 2026 06:43
@oschaaf

oschaaf commented Oct 27, 2026

Copy link
Copy Markdown
Member Author

Rebased onto main after #51. #51 vendored the bundle from before the zod 4.6.6 bump (sha 3bd9f63a…), so this keeps the current build (9dba8b63…), which matches upstream again. #51's Vault-reference and schema changes are untouched and pass; harness tests 183/183.

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