@envelop/sentry 3.7.1-alpha-20220824165935-4720f031 → 3.8.0

package/README.md

@@ -30,17 +30,12 @@ yarn add @sentry/node @sentry/tracing @envelop/sentry
 ## Usage Example
 
 ```ts
-import { parse, validate, execute, subscribe } from 'graphql'
 import { envelop } from '@envelop/core'
 import { useSentry } from '@envelop/sentry'
 // do this only once in you entry file.
 import '@sentry/tracing'
 
 const getEnveloped = envelop({
-  parse,
-  validate,
-  execute,
-  subscribe,
   plugins: [
     // ... other plugins ...
     useSentry({

package/cjs/index.js

@@ -6,7 +6,7 @@ const core_1 = require("@envelop/core");
 const Sentry = tslib_1.__importStar(require("@sentry/node"));
 const graphql_1 = require("graphql");
 function defaultSkipError(error) {
-    return error instanceof Error;
+    return error instanceof core_1.EnvelopError;
 }
 exports.defaultSkipError = defaultSkipError;
 const sentryTracingSymbol = Symbol('sentryTracing');
@@ -78,9 +78,7 @@ const useSentry = (options = {}) => {
             if (skipOperation(args)) {
                 return;
             }
-            const rootOperation = args.document.definitions.find(
-            // @ts-expect-error TODO: not sure how we will make it dev friendly
-            o => o.kind === graphql_1.Kind.OPERATION_DEFINITION);
+            const rootOperation = args.document.definitions.find(o => o.kind === graphql_1.Kind.OPERATION_DEFINITION);
             const operationType = rootOperation.operation;
             const document = (0, graphql_1.print)(args.document);
             const opName = args.operationName || ((_a = rootOperation.name) === null || _a === void 0 ? void 0 : _a.value) || 'Anonymous Operation';
@@ -175,7 +173,7 @@ const useSentry = (options = {}) => {
                                     }
                                     // Map index values in list to $index for better grouping of events.
                                     const errorPathWithIndex = ((_b = err.path) !== null && _b !== void 0 ? _b : [])
-                                        .map((v) => (typeof v === 'number' ? '$index' : v))
+                                        .map(v => (typeof v === 'number' ? '$index' : v))
                                         .join(' > ');
                                     const eventId = Sentry.captureException(err, {
                                         fingerprint: ['graphql', errorPathWithIndex, opName, operationType],

package/esm/index.js

@@ -1,8 +1,8 @@
-import { handleStreamOrSingleExecutionResult, } from '@envelop/core';
+import { EnvelopError, handleStreamOrSingleExecutionResult, } from '@envelop/core';
 import * as Sentry from '@sentry/node';
 import { GraphQLError, Kind, print, responsePathAsArray } from 'graphql';
 export function defaultSkipError(error) {
-    return error instanceof Error;
+    return error instanceof EnvelopError;
 }
 const sentryTracingSymbol = Symbol('sentryTracing');
 export const useSentry = (options = {}) => {
@@ -73,9 +73,7 @@ export const useSentry = (options = {}) => {
             if (skipOperation(args)) {
                 return;
             }
-            const rootOperation = args.document.definitions.find(
-            // @ts-expect-error TODO: not sure how we will make it dev friendly
-            o => o.kind === Kind.OPERATION_DEFINITION);
+            const rootOperation = args.document.definitions.find(o => o.kind === Kind.OPERATION_DEFINITION);
             const operationType = rootOperation.operation;
             const document = print(args.document);
             const opName = args.operationName || ((_a = rootOperation.name) === null || _a === void 0 ? void 0 : _a.value) || 'Anonymous Operation';
@@ -170,7 +168,7 @@ export const useSentry = (options = {}) => {
                                     }
                                     // Map index values in list to $index for better grouping of events.
                                     const errorPathWithIndex = ((_b = err.path) !== null && _b !== void 0 ? _b : [])
-                                        .map((v) => (typeof v === 'number' ? '$index' : v))
+                                        .map(v => (typeof v === 'number' ? '$index' : v))
                                         .join(' > ');
                                     const eventId = Sentry.captureException(err, {
                                         fingerprint: ['graphql', errorPathWithIndex, opName, operationType],

package/package.json

@@ -1,13 +1,15 @@
 {
   "name": "@envelop/sentry",
-  "version": "3.7.1-alpha-20220824165935-4720f031",
+  "version": "3.8.0",
   "sideEffects": false,
   "peerDependencies": {
-    "@envelop/core": "2.5.1-alpha-20220824165935-4720f031",
+    "@envelop/core": "^2.6.0",
     "@sentry/node": "^6 || ^7",
     "graphql": "^14.0.0 || ^15.0.0 || ^16.0.0"
   },
-  "dependencies": {},
+  "dependencies": {
+    "tslib": "^2.4.0"
+  },
   "repository": {
     "type": "git",
     "url": "https://github.com/n1ru4l/envelop.git",

package/typings/index.d.cts

@@ -73,7 +73,7 @@ export declare type SentryPluginOptions = {
     skip?: (args: ExecutionArgs) => boolean;
     /**
      * Indicates whether or not to skip Sentry exception reporting for a given error.
-     * By default, this plugin skips all `Error` errors and does not report it to Sentry.
+     * By default, this plugin skips all `EnvelopError` errors and does not report it to Sentry.
      */
     skipError?: (args: Error) => boolean;
 };

package/typings/index.d.ts

@@ -73,7 +73,7 @@ export declare type SentryPluginOptions = {
     skip?: (args: ExecutionArgs) => boolean;
     /**
      * Indicates whether or not to skip Sentry exception reporting for a given error.
-     * By default, this plugin skips all `Error` errors and does not report it to Sentry.
+     * By default, this plugin skips all `EnvelopError` errors and does not report it to Sentry.
      */
     skipError?: (args: Error) => boolean;
 };