Error: Cannot find module 'graphql' (is graphql mandatory?) about nestjs-sentry HOT 17 CLOSED

Looks like the library is exporting everything in the index.ts file, therefore the import statements in the graphql.interceptor gets evaulated.

from nestjs-sentry.

I'll take a look at this. Thanks

from nestjs-sentry.

I failed to get this package to work out of the box, so I now use nest-raven.
Works like a charm.

from nestjs-sentry.

I have the same issue just use the SentryInterceptor, like that

 app.useGlobalInterceptors(new SentryInterceptor());

and my module:

SentryModule.forRootAsync({
    imports: [ConfigModule],
    useFactory: async (configService: ConfigType<typeof config>) => {
      return {
        dsn: configService.sentryKey,
        debug: true,
        environment: configService.env,
        enabled: configService.env === 'production',
        logLevel: LogLevel.Debug,
      };
    },
    inject: [config.KEY],
  }),

but show error because don't have @nestjs/graphql, I'm working w/ @ntegral/nestjs-sentry 2.0.6

from nestjs-sentry.

graphql is not a required module for this version of the npm package when using the SentryInterceptor?? I can not replicate your issue??

see the link to the sample application using the package.

from nestjs-sentry.

@module({
imports: [
ConfigModule.forRoot(),
SentryModule.forRootAsync({
imports: [ConfigModule],
useFactory: async (cfg:ConfigService) => ({
dsn: cfg.get('SENTRY_DSN'),
debug: true,
environment: 'dev',
release: null, // must create a release in sentry.io dashboard
logLevel: LogLevel.Debug //based on sentry.io loglevel //
}),
inject: [ConfigService],
}),
],
controllers: [AppController],
providers: [AppService],
})
export class AppModule {}

from nestjs-sentry.

Hello! Ran npm install and my integration stopped working

FAIL test/e2e/legacy/verify.e2e-spec.ts
  ● Test suite failed to run
    Cannot find module '@nestjs/graphql' from '../../node_modules/@ntegral/nestjs-sentry/dist/graphql.interceptor.js'
    Require stack:
      /home/runner/work/.../node_modules/@ntegral/nestjs-sentry/dist/graphql.interceptor.js
      /home/runner/work/.../node_modules/@ntegral/nestjs-sentry/dist/index.js
      /home/runner/work/../src/app.module.ts
      legacy/verify.e2e-spec.ts
      at Resolver.resolveModule (../../node_modules/jest-resolve/build/index.js:306:11)
      at Object.<anonymous> (../../../../../../../graphql.interceptor.ts:12:3)

  imports: [
    ConfigModule.forRoot({ cache: true }),
    SentryModule.forRootAsync({
      imports: [ConfigModule],
      useFactory: async (configService: ConfigService) => ({
        dsn: configService.get('SENTRY_URL'),
        environment: configService.get('ENVIRONMENT'),
      }),
      inject: [ConfigService],
    }),
    TokenModule,
  ],

from nestjs-sentry.

Looks like this is an issue with npm 7

from nestjs-sentry.

I've tested the module using npm 7.11.2. I have not gotten the error or been able to replicate it? Would you clone the sample nest application to see if you are able to replicate this error?

https://github.com/ntegral/nestjs-sentry-example

from nestjs-sentry.

@adamtester
I've tested the module using npm 7.11.2. I have not gotten the error or been able to replicate it? Would you clone the sample nest application to see if you are able to replicate this error?

https://github.com/ntegral/nestjs-sentry-example

from nestjs-sentry.

@ntegral yep I'll give it a go tomorrow and report back!

from nestjs-sentry.

@ntegral
Hi, I have the same issue.
I have cloned your example repo, just git clone, yarn, yarn start and.. Error: Cannot find module 'graphql'

node v12.21.0
yarn 1.22.10
OS: Ubuntu 20.04

from nestjs-sentry.

Could we at least re-open the issue? It's reproducible.

from nestjs-sentry.

Please update to the latest version of the module 2.0.7 @cerireyhan @vladi-strilets @adamtester

from nestjs-sentry.

If you are using the GraphQLInterceptor, you will have to install graphQL

from nestjs-sentry.

@ntegral I can confirm that after update to 2.0.07 it works

from nestjs-sentry.

@vladi-strilets Thanks for letting me know. With that confirmation, I'll go ahead and close this issue.

from nestjs-sentry.