Skip to content

Merge GraphQL and Vitest updates#1428

Merged
Code-Hex merged 4 commits intomainfrom
refactoring2
Apr 25, 2026
Merged

Merge GraphQL and Vitest updates#1428
Code-Hex merged 4 commits intomainfrom
refactoring2

Conversation

@Code-Hex
Copy link
Copy Markdown
Owner

What changed

This branch pulls in the Renovate updates from #1424 and #1426:

  • update graphql to 16.13.2
  • update vitest to 4.1.5
  • keep the lockfile in sync

It also includes the review cleanup from #1427 follow-up work:

  • move the shared Zod helpers to src/zod_shared.ts, since both zod and zodv4 use them
  • delete the unused test helper files instead of leaving half-migrated helpers around
  • fix the JSDoc lint warning in src/lazy.ts

Vitest 4 exposed a GraphQL module identity problem. The tests were building schemas through one graphql entry and @graphql-tools/* was checking them through another, which trips GraphQL-JS' realm check. The config now aliases graphql to the CJS entry and externalizes it for Vitest, so the test runner uses one GraphQL instance.

Checks

  • pnpm install --frozen-lockfile
  • pnpm type-check
  • pnpm test
  • pnpm lint

@Code-Hex Code-Hex marked this pull request as ready for review April 25, 2026 16:34
@Code-Hex Code-Hex changed the title [codex] Merge GraphQL and Vitest updates Merge GraphQL and Vitest updates Apr 25, 2026
@Code-Hex Code-Hex merged commit d487e0c into main Apr 25, 2026
2 checks passed
@Code-Hex Code-Hex deleted the refactoring2 branch April 25, 2026 16:34
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