Missing "driver" option. Nest thinks I'm using version 10 of @nestjs/graphql but I'm using 9 | NestJS | Page 1

dense plover Nov 15, 2023, 10:16 AM

#

I've updated prisma on my app and nothing else, when I run my API I get this error:

ERROR [GraphQLModule] Missing "driver" option. In the latest version of "@nestjs/graphql" package (v10) a new required configuration property called "driver" has been introduced. Check out the official documentation for more details on how to migrate (https://docs.nestjs.com/graphql/migration-guide).

Running yarn why @nestjs/graphql I get the following which clearly states I'm running version 9.

[1/4] Why do we have the module "@nestjs/graphql"...?
[2/4] Initialising dependency graph...
[3/4] Finding dependency...
[4/4] Calculating file sizes...
=> Found "@nestjs/graphql@9.2.7"
info Reasons this module exists
   - "_project_#common" depends on it
   - Hoisted from "_project_#common#@nestjs#graphql"
   - Hoisted from "_project_#api#@nestjs#graphql"
info Disk size without dependencies: "6.97MB"
info Disk size with unique dependencies: "15.83MB"
info Disk size with transitive dependencies: "17.02MB"
info Number of shared dependencies: 36
Done in 0.95s.

Any ideas what's going on?

sage axle Nov 15, 2023, 10:17 AM

#

Hey Allan. Long time no see. 🙂

What does your GraphQL module registration look like?

#

Oh wait. You are on version 9? 🤔

dense plover Nov 15, 2023, 10:18 AM

#

Oh hey Scott! 🙂

#

Yes, exactly... V9.

#

Was working before I updated my prisma deps 🙄

#

This is my config, it complains about the debug and cors flag too if I have them in place:

#

I did also update typescript from 4.7 to 4.8 and removed the @lament temple/cli package from another API I have in the same workspace.
That's the rundown 😅

#

(I booted it up after the TS upgrade and it was fine)

sage axle Nov 15, 2023, 10:22 AM

#

Can you still post your GraphQL module registration (forRoot)?

#

I believe the driver property was in v9 too.

dense plover Nov 15, 2023, 10:24 AM

#

where export class ConfigService implements GqlOptionsFactory {
And contains the section I pasted above

#

@Injectable()
export class ConfigService implements GqlOptionsFactory {
  public createGqlOptions (): GqlModuleOptions {
    return {
      autoSchemaFile: join(process.cwd(), 'graphql/schema.gql'),
      debug: !this.isLive,
      context: ({ req }) => ({ req }),
      cors: {
        origin: true,
        credentials: true
      }
    }
  }
}

sage axle Nov 15, 2023, 10:24 AM

#

Are you using Apollo?

dense plover Nov 15, 2023, 10:25 AM

#

Yes

sage axle Nov 15, 2023, 10:25 AM

#

Then for SnGs try adding driver: ApolloDriver, to your config. 🙂

dense plover Nov 15, 2023, 10:26 AM

#

But why am I needing to do this for v9?

sage axle Nov 15, 2023, 10:26 AM

#

You'll want to get to something like this:

import { Module } from '@nestjs/common';
import { GraphQLModule } from '@nestjs/graphql';
import { ApolloDriver, ApolloDriverConfig } from '@nestjs/apollo';

@Module({
 imports: [
   GraphQLModule.forRoot<ApolloDriverConfig>({
     driver: ApolloDriver,
   }),
 ],

dense plover Nov 15, 2023, 10:26 AM

#

This is a v10 c hange.

sage axle Nov 15, 2023, 10:26 AM

#

I'm looking at the v9 docs, and it's asking for this too. https://docs.nestjs.com/v9/graphql/quick-start

dense plover Nov 15, 2023, 10:26 AM

#

Oh!?

#

Did this come about between v9.1 and 9.2?

sage axle Nov 15, 2023, 10:27 AM

#

I have no idea tbh.

dense plover Nov 15, 2023, 10:27 AM

#

My colleague is using 9.1.2 and I'm on 9.2.7. Seems odd for a breaking change

#

Ah ok

sage axle Nov 15, 2023, 10:28 AM

#

Let me go check. It would interest me just from a knowledge standpoint.

dense plover Nov 15, 2023, 10:28 AM

#

Going to lock that dep down to 9.1 and see if it works too 🙂

#

Works!

#

sage axle Nov 15, 2023, 10:30 AM

#

Interesting. Ok. Then I don't have to go looking. Haha!

dense plover Nov 15, 2023, 10:30 AM

#

Nope - kinda breaks semantic versioning though TBH.

sage axle Nov 15, 2023, 10:31 AM

#

I think it's the GraphQL module version possibly. What version are you on?

#

Although, the Nest version should make that a difference. Hmm....

dense plover Nov 15, 2023, 10:32 AM

#

Even the docs suggest v9 -> v10 is when the driver change was introduced:

sage axle Nov 15, 2023, 10:32 AM

#

There hasn't been any uproar on this at all. First I'm hearing it.

dense plover Nov 15, 2023, 10:32 AM

#

How odd 😄

#

Always me ...

#

Want me to pop a GH issue on for it to be looked at or not bother? I don't mind too much now I know why it's happening but even the error in the console said it was a V10 change so very misleading:

sage axle Nov 15, 2023, 10:33 AM

#

Yeah, that's version 10 of the Gql module.

#

No need for a GH issue.

dense plover Nov 15, 2023, 10:34 AM

#

But it's not, its v9.2..

sage axle Nov 15, 2023, 10:35 AM

#

So, 9.2 of the Gql module or Nest core?

dense plover Nov 15, 2023, 10:35 AM

#

GQL

#

sage axle Nov 15, 2023, 10:35 AM

#

Oh. Oh duh. Ok.

dense plover Nov 15, 2023, 10:35 AM

#

Having the hat upgrades to 9.2 which then breaks

#

removing the hat to lock at 9.1 is fine.

#

Going to re-test just to make sure it wasn't a one off but even yarn why showed 9.2

sage axle Nov 15, 2023, 10:36 AM

#

Ok. I'm still going back to look at change logs.

dense plover Nov 15, 2023, 10:36 AM

#

Ok, back to 9.2 and:

#

And I've had to remove the debug, cors options from my config to get around other compile issues.

#

And to confirm its 9.2:

yarn why v1.22.19
[1/4] Why do we have the module "@nestjs/graphql"...?
[2/4] Initialising dependency graph...
[3/4] Finding dependency...
[4/4] Calculating file sizes...
=> Found "@nestjs/graphql@9.2.7"
info Reasons this module exists
   - "_project_#common" depends on it
   - Hoisted from "_project_#common#@nestjs#graphql"
   - Hoisted from "_project_#api#@nestjs#graphql"
info Disk size without dependencies: "6.97MB"
info Disk size with unique dependencies: "15.83MB"
info Disk size with transitive dependencies: "17.02MB"
info Number of shared dependencies: 36
Done in 1.01s.

sage axle Nov 15, 2023, 10:38 AM

#

Interesting, interesting. There is no released version of 9 higher then 9.1.2.

dense plover Nov 15, 2023, 10:38 AM

#

🤣

#

Tell NPM repo that 😄

sage axle Nov 15, 2023, 10:39 AM

#

That is from NPM.

dense plover Nov 15, 2023, 10:39 AM

#

Eh?!

sage axle Nov 15, 2023, 10:39 AM

#

https://www.npmjs.com/package/@nestjs/graphql?activeTab=versions

#

Same goes for Github.

dense plover Nov 15, 2023, 10:40 AM

#

Eh!?! Well it's upgrading me ... How the heck?

sage axle Nov 15, 2023, 10:40 AM

#

https://github.com/nestjs/graphql/releases?page=3

dense plover Nov 15, 2023, 10:40 AM

#

Issue with yarn, perhaps?

sage axle Nov 15, 2023, 10:40 AM

#

Yeah. Very strange.

dense plover Nov 15, 2023, 10:41 AM

#

Ok, well the fix is simple and given no one else is seeing it, I'll just lock down the version. Worth keeping in mind if others report an issue

sage axle Nov 15, 2023, 10:41 AM

#

I've dropped yarn many moons ago. Moved to pnpm. 🙂

sage axle Nov 15, 2023, 10:41 AM

#

dense plover Ok, well the fix is simple and given no one else is seeing it, I'll just lock do...

Yes. Very interesting issue indeed, but the root cause isn't from Nest.

dense plover Nov 15, 2023, 10:42 AM

#

So odd:

#

Either way, it's sorted with the locked version. Thanks for the brainstorm Scott.

sage axle Nov 15, 2023, 10:42 AM

#

Huh?

#

Very strange that is. Might want to have Yarn point to npm for packages.

#

I've seen yarn's registry holding crap before.

dense plover Nov 15, 2023, 10:47 AM

#

Ah, good point. I'll check on that.
Thanks mate.

sage axle Nov 15, 2023, 10:48 AM

#

Glad I could help a friend. 🙂

#

Doing yarn info <package> versions. I get this:

dense plover Nov 15, 2023, 10:52 AM

#

Ahhh, wasn't sure how to check the yarn registry.
How comes its soooo wrong?

sage axle Nov 15, 2023, 10:53 AM

#

Doing npm show <package> versions I get the same.

#

Very strange. 🤔

dense plover Nov 15, 2023, 10:53 AM

#

Haha, this is soo strange

sage axle Nov 15, 2023, 10:53 AM

#

Maybe @unborn rampart can make rhyme or reason of this.

#

I'd say, let's wait until he comes online.

dense plover Nov 15, 2023, 10:54 AM

#

I'm wondering if a version was released and then later retracted to make it a new major version change

#

No worries about waiting - I'm up and running with the locked version for now 🙂

sage axle Nov 15, 2023, 10:54 AM

#

Yeah, that could be. No, I mean me waiting to dig deeper. Haha.

dense plover Nov 15, 2023, 10:55 AM

#

Yeah, I got it 🙂

sage axle Nov 15, 2023, 10:55 AM

#

Maybe it was introduced, people got whacked out of shape and then went to major.

dense plover Nov 15, 2023, 10:55 AM

#

Yep

sage axle Nov 15, 2023, 10:55 AM

#

I don't recall any ruckus about this though.

#

And I'm a moderator here. So....

dense plover Nov 15, 2023, 10:55 AM

#

Haha yeah 😛 Always me, as I said ..

sage axle Nov 15, 2023, 10:55 AM

#

Though, it might have been when I stepped away from my project for a while.

#

It's 2 years old.

dense plover Nov 15, 2023, 10:56 AM

#

Yeah, I should really upgrade but we're just about to launch into Beta so don't want to rock the boat too much 😄

sage axle Nov 15, 2023, 10:56 AM

#

At any rate, hopefully Jay can shed some light on the situation.

unborn rampart Nov 15, 2023, 6:39 PM

#

9.2.0 was published but has since been deprecated as an "unstable version". Looks like this is when Kamil made the repo into a monorepo and started supporting mercurius. There some logged issues around it, but Kamil said he was publishing under @next which shouldn't usually be installed, but they were being used due to semver compliance.

sage axle Nov 15, 2023, 7:51 PM

#

Thanks for the clarification Jay.

#Missing "driver" option. Nest thinks I'm using version 10 of @nestjs/graphql but I'm using 9