Skip to content

chore(voucher): upgrade @apollo/server v4→v5, @as-integrations/next v3→v4#508

Open
k9ert wants to merge 2 commits intomainfrom
kn/upgrade-apollo-server-voucher
Open

chore(voucher): upgrade @apollo/server v4→v5, @as-integrations/next v3→v4#508
k9ert wants to merge 2 commits intomainfrom
kn/upgrade-apollo-server-voucher

Conversation

@k9ert
Copy link
Copy Markdown

@k9ert k9ert commented Mar 25, 2026

Summary

Upgrades Apollo Server and its Next.js integration in apps/voucher:

  • @apollo/server: ^4.7.1^5.0.0 (resolved: 5.5.0)
  • @as-integrations/next: ^3.1.0^4.0.0 (resolved: 4.1.0)

Motivation: Apollo Server v4 is End of Life. Tracked in blinkbitcoin/blink-wip#635 (parent: blinkbitcoin/blink-wip#631).

What changed

Only apps/voucher/package.json and pnpm-lock.yaml. No source code changes required — the voucher app's usage of ApolloServer and startServerAndCreateNextHandler is fully compatible with v5.

Breaking changes reviewed (Apollo Server v4 → v5)

Full migration guide: https://www.apollographql.com/docs/apollo-server/migration/

Breaking change Impact on voucher Action
Node.js ≥ v20 required Already on v20+ None
graphql.js ≥ v16.11.0 required Currently ^16.6.0 in package.json — resolved version already satisfies this None
Express integration extracted to @as-integrations/express4 Voucher uses Next.js integration, not Express N/A
status400ForVariableCoercionErrors now defaults to true Variable coercion errors now return HTTP 400 instead of 200. This is correct behavior per the GraphQL spec None (improvement)
startStandaloneServer no longer uses Express Not used in voucher N/A
precomputedNonce option removed from landing page plugins Not used N/A
Compiled targeting ES2023 Fine for Node.js v20+ None
formatError callback Unchanged API. Voucher's formatError in app/api/graphql/route.ts works as-is None
Constructor API (new ApolloServer({...})) Unchanged. typeDefs, resolvers, formatError all work identically None
Native fetch instead of node-fetch for built-in plugins No HTTP proxy usage in voucher N/A

Breaking changes reviewed (@as-integrations/next v3 → v4)

Release notes: https://github.com/apollo-server-integrations/apollo-server-integration-next/releases/tag/v4.0.0

Breaking change Impact on voucher Action
Requires @apollo/server ≥ v5 Bumped in this PR Done
Drops Node.js 18 support Already on v20+ None
startServerAndCreateNextHandler API Unchanged between v3 and v4 None

Verification

  • pnpm install --filter voucher — clean install ✅
  • tsc --noEmit — no Apollo-related type errors ✅ (pre-existing test type issues unrelated to this change)

Files

  • apps/voucher/package.json — version bumps
  • pnpm-lock.yaml — lockfile update

Closes blinkbitcoin/blink-wip#635

🤖 Generated with Claude Code

…3→v4

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Upgrades Apollo Server and the Next.js Apollo integration used by the apps/voucher application to supported (non‑EOL) major versions, updating the workspace lockfile accordingly.

Changes:

  • Bump @apollo/server from v4 to v5 in apps/voucher.
  • Bump @as-integrations/next from v3 to v4 in apps/voucher.
  • Regenerate pnpm-lock.yaml to reflect the new dependency graph.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
apps/voucher/package.json Updates Apollo Server and Next.js integration dependency ranges for the voucher app.
pnpm-lock.yaml Updates resolved versions and transitive dependencies for the new Apollo packages.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread apps/voucher/package.json
…rver v5

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants