feat(opentelemetry): add schema coordinate to error events

.changeset/@graphql-hive_gateway-1636-dependencies.md

@@ -0,0 +1,7 @@
+---
+'@graphql-hive/gateway': patch
+---
+
+dependencies updates: 
+
+- Updated dependency [`graphql-yoga@^5.17.0` ↗︎](https://www.npmjs.com/package/graphql-yoga/v/5.17.0) (from `^5.16.2`, in `dependencies`)

.changeset/@graphql-hive_gateway-runtime-1636-dependencies.md

@@ -0,0 +1,7 @@
+---
+'@graphql-hive/gateway-runtime': patch
+---
+
+dependencies updates: 
+
+- Updated dependency [`graphql-yoga@^5.17.0` ↗︎](https://www.npmjs.com/package/graphql-yoga/v/5.17.0) (from `^5.16.2`, in `dependencies`)

.changeset/@graphql-hive_gateway-testing-1636-dependencies.md

@@ -0,0 +1,7 @@
+---
+'@graphql-hive/gateway-testing': patch
+---
+
+dependencies updates: 
+
+- Updated dependency [`graphql-yoga@^5.17.0` ↗︎](https://www.npmjs.com/package/graphql-yoga/v/5.17.0) (from `^5.16.2`, in `dependencies`)

.changeset/@graphql-hive_plugin-opentelemetry-1636-dependencies.md

@@ -0,0 +1,7 @@
+---
+'@graphql-hive/plugin-opentelemetry': patch
+---
+
+dependencies updates: 
+
+- Added dependency [`@graphql-tools/executor@^1.4.9` ↗︎](https://www.npmjs.com/package/@graphql-tools/executor/v/1.4.9) (to `dependencies`)

.changeset/@graphql-mesh_fusion-runtime-1636-dependencies.md

@@ -0,0 +1,7 @@
+---
+'@graphql-mesh/fusion-runtime': patch
+---
+
+dependencies updates: 
+
+- Updated dependency [`graphql-yoga@^5.17.0` ↗︎](https://www.npmjs.com/package/graphql-yoga/v/5.17.0) (from `^5.16.2`, in `dependencies`)

.changeset/little-ties-jog.md

@@ -0,0 +1,22 @@
+---
+'@graphql-hive/plugin-opentelemetry': minor
+'@graphql-hive/gateway-runtime': minor
+---
+
+Expose GraphQLError as OpenTelemetry Events.
+
+Errors contains in the result of a graphql operation are now reported as standalone OpenTelemetry
+Events (name `graphql.error`) instead of OpenTelemetry Exceptions.
+
+This is aligned with the guidance of the Graphql OpenTelemetry working group.
+
+It allows to add more graphql specific attributes to errors reported in a response:
+
+- `message`: The error message
+- `path`: The path in the operation document from which the error originated
+- `locations`: The list of related locations in the document source
+- `coordinate`: The schema coordinate of the resolver which is the source of the error
+
+This brings the experimental support of the `coordinate` error attribute in the Yoga Runtime. For
+security reason, this attribute is purposefully not serialized, to avoid leaking schema information
+to clients.

e2e/auto-type-merging/package.json

@@ -5,7 +5,7 @@
     "@graphql-mesh/compose-cli": "^1.5.3",
     "@omnigraph/openapi": "^0.109.23",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/aws-sigv4/package.json

@@ -6,7 +6,7 @@
     "aws4-express": "^0.12.0",
     "express": "^5.0.0",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/demand-control/package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "@apollo/subgraph": "^2.11.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/extra-fields/package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/graphos-polling/package.json

@@ -6,7 +6,7 @@
     "@apollo/subgraph": "^2.11.3",
     "fastify": "^5.6.2",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "pino-pretty": "^13.1.2"
   }
 }

e2e/hmac-auth-https/package.json

@@ -14,6 +14,6 @@
     "@graphql-mesh/hmac-upstream-signature": "workspace:^",
     "@graphql-mesh/plugin-jwt-auth": "workspace:^",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2"
+    "graphql-yoga": "^5.17.0"
   }
 }

e2e/interface-additional-resolvers/package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/opentelemetry/opentelemetry.e2e.ts

@@ -55,11 +55,17 @@ type JaegerTraceResource = {
   tags: JaegerTraceTag[];
 };
 
+type JaegerTraceLog = {
+  timestamp: number;
+  fields: JaegerTraceTag[];
+};
+
 type JaegerTraceSpan = {
   traceID: string;
   spanID: string;
   operationName: string;
   tags: Array<JaegerTraceTag>;
+  logs: Array<JaegerTraceLog>;
   references: Array<{ refType: string; spanID: string; traceID: string }>;
 };
 
@@ -402,6 +408,30 @@ describe('OpenTelemetry', () => {
           expect(relevantTrace?.spans).toContainEqual(
             expect.objectContaining({ operationName: 'POST /graphql' }),
           );
+
+          const operationSpan = relevantTrace!.spans.find(
+            (span) => span.operationName === 'graphql.operation',
+          );
+
+          expect(operationSpan?.logs).toContainEqual(
+            expect.objectContaining({
+              fields: expect.arrayContaining([
+                expect.objectContaining({
+                  key: 'event',
+                  value: 'graphql.error',
+                }),
+                expect.objectContaining({
+                  key: 'hive.graphql.error.locations',
+                  value: '["1:13"]',
+                }),
+                expect.objectContaining({
+                  key: 'hive.graphql.error.message',
+                  value: 'Syntax Error: Expected Name, found <EOF>.',
+                }),
+              ]),
+            }),
+          );
+
           expect(relevantTrace?.spans).toContainEqual(
             expect.objectContaining({
               operationName: 'graphql.parse',
@@ -416,7 +446,7 @@ describe('OpenTelemetry', () => {
                 }),
                 expect.objectContaining({
                   key: 'otel.status_description',
-                  value: 'Syntax Error: Expected Name, found <EOF>.',
+                  value: 'GraphQL Parse Error',
                 }),
                 expect.objectContaining({
                   key: 'hive.graphql.error.count',
@@ -477,6 +507,53 @@ describe('OpenTelemetry', () => {
           expect(relevantTrace?.spans).toContainEqual(
             expect.objectContaining({ operationName: 'POST /graphql' }),
           );
+          expect(relevantTrace?.spans).toContainEqual(
+            expect.objectContaining({
+              operationName: 'graphql.operation',
+              tags: expect.arrayContaining([
+                expect.objectContaining({
+                  key: 'otel.status_code',
+                  value: 'ERROR',
+                }),
+                expect.objectContaining({
+                  key: 'error',
+                  value: true,
+                }),
+                expect.objectContaining({
+                  key: 'otel.status_description',
+                  value: 'GraphQL Validation Error',
+                }),
+                expect.objectContaining({
+                  key: 'hive.graphql.error.count',
+                  value: 1,
+                }),
+              ]),
+            }),
+          );
+          const operationSpan = relevantTrace!.spans.find(
+            (span) => span.operationName === 'graphql.operation',
+          );
+
+          expect(operationSpan?.logs).toContainEqual(
+            expect.objectContaining({
+              fields: expect.arrayContaining([
+                expect.objectContaining({
+                  key: 'event',
+                  value: 'graphql.error',
+                }),
+                expect.objectContaining({
+                  key: 'hive.graphql.error.locations',
+                  value: '["1:9"]',
+                }),
+                expect.objectContaining({
+                  key: 'hive.graphql.error.message',
+                  value:
+                    'Cannot query field "nonExistentField" on type "Query".',
+                }),
+              ]),
+            }),
+          );
+
           expect(relevantTrace?.spans).toContainEqual(
             expect.objectContaining({ operationName: 'graphql.parse' }),
           );
@@ -494,8 +571,7 @@ describe('OpenTelemetry', () => {
                 }),
                 expect.objectContaining({
                   key: 'otel.status_description',
-                  value:
-                    'Cannot query field "nonExistentField" on type "Query".',
+                  value: 'GraphQL Validation Error',
                 }),
                 expect.objectContaining({
                   key: 'hive.graphql.error.count',

e2e/opentelemetry/package.json

@@ -6,7 +6,7 @@
     "@apollo/subgraph": "^2.11.3",
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/polling/package.json

@@ -5,6 +5,6 @@
     "@graphql-mesh/compose-cli": "^1.5.3",
     "express": "^5.0.0",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2"
+    "graphql-yoga": "^5.17.0"
   }
 }

e2e/retry-timeout/package.json

@@ -5,7 +5,7 @@
     "@apollo/subgraph": "^2.11.3",
     "@graphql-hive/gateway": "workspace:*",
     "graphql": "16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/subscriptions-with-transforms/package.json

@@ -5,7 +5,7 @@
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "16.12.0",
     "graphql-sse": "^2.6.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

e2e/type-merging-batching/package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1"
   }
 }

examples/extra-fields/package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1",
     "@graphql-hive/gateway": "^2.1.21"
   },

examples/hmac-auth-https/package.json

@@ -18,7 +18,7 @@
     "@graphql-mesh/hmac-upstream-signature": "^2.0.8",
     "@graphql-mesh/plugin-jwt-auth": "^2.0.9",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2"
+    "graphql-yoga": "^5.17.0"
   },
   "devDependencies": {
     "tsx": "^4.20.3"

examples/interface-additional-resolvers/package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1",
     "@graphql-hive/gateway": "^2.1.21"
   },

examples/subscriptions-with-transforms/package.json

@@ -5,7 +5,7 @@
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "16.12.0",
     "graphql-sse": "^2.6.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1",
     "@graphql-hive/gateway": "^2.1.21"
   },

examples/type-merging-batching/package.json

@@ -4,7 +4,7 @@
   "dependencies": {
     "@graphql-mesh/compose-cli": "^1.5.3",
     "graphql": "^16.12.0",
-    "graphql-yoga": "^5.16.2",
+    "graphql-yoga": "^5.17.0",
     "tslib": "^2.8.1",
     "@graphql-hive/gateway": "^2.1.21"
   },