Skip to content

fix: discriminatedUnion format handling#94

Open
wststone wants to merge 1 commit into
vercel-labs:mainfrom
wststone:fix/format-discriminated-union
Open

fix: discriminatedUnion format handling#94
wststone wants to merge 1 commit into
vercel-labs:mainfrom
wststone:fix/format-discriminated-union

Conversation

@wststone

Copy link
Copy Markdown
  • Update formatZodType and extractPropsFromSchema in
    catalog.ts to work with Zod v4's internal _def structure
  • discriminatedUnion now correctly resolves literal types
    (was showing type: undefined)
  • Add tests for discriminatedUnion and nativeEnum in prompt
    generation

@vercel

vercel Bot commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

@wststone is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

danielsimonjr added a commit to danielsimonjr/json-render that referenced this pull request Jun 30, 2026
…ndabot alerts)

5 vite alerts (vercel-labs#44/vercel-labs#45/vercel-labs#46/vercel-labs#94/vercel-labs#95, <=7.3.4) + 3 minimatch alerts (vercel-labs#23/vercel-labs#26/vercel-labs#28, <9.0.7) were all in pnpm-lock.yaml. The existing pnpm.overrides were recorded but never applied to the lockfile: vite enters only as vitest's peer (auto-installed peers bypass overrides under the repo's lowest-direct resolution), and the minimatch range-selector override missed the eslint-plugin-react>minimatch edge. Fix: add vite as a direct devDependency (^7.3.5 -> resolves 7.3.6) and add a deterministic nested override eslint-plugin-react>minimatch ^9.0.7 (-> 9.0.9); consolidated the redundant/overlapping vite+minimatch selectors. Targeted lockfile update (33/48 lines), vitest and all other deps unchanged. Verified: pnpm audit clean, type-check 5/5, turbo build 5/5, 166 tests pass.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HWrTgpQ6CZV7tbVCNib9Pi
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.

1 participant