Skip to content

Conversation

@CatsMiaow
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Docs
  • Other... Please describe:

What is the current behavior?

Because @as-integrations/express5 isn’t listed as a dependency of the @nestjs/apollo package, you’ll get an error unless you install it separately.

https://github.com/nestjs/graphql/blob/89d2d29baaed140bd82e9e32cd91957396461444/packages/apollo/package.json#L40-L55

[Nest] 40751  - 10/27/2025, 2:45:53 PM   ERROR [PackageLoader] The "@as-integrations/express5" package is missing. Please, make sure to install it to take advantage of GraphQLModule.

What is the new behavior?

Add @as-integrations/express5 installation to the For Express and Apollo (default) guide.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Or would it be correct to add it as a dependency of the @nestjs/apollo package instead?

Copilot AI review requested due to automatic review settings October 27, 2025 05:54
Copy link

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

This PR updates the GraphQL installation documentation to include the @as-integrations/express5 package, which is required but not listed as a dependency of @nestjs/apollo. Without this package, users encounter a runtime error when attempting to use the GraphQL module with Express.

Key Changes:

  • Added @as-integrations/express5 to the Express/Apollo installation command

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

@kamilmysliwiec kamilmysliwiec merged commit 5f68563 into nestjs:master Oct 27, 2025
@kamilmysliwiec
Copy link
Member

lgtm

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