@graphql-tools/delegate 12.0.14 → 12.0.16

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @graphql-tools/delegate
 
+## 12.0.16
+### Patch Changes
+
+
+
+- [#2320](https://github.com/graphql-hive/gateway/pull/2320) [`deafdc0`](https://github.com/graphql-hive/gateway/commit/deafdc00e388ed195c2cdf77e98cd19e7d496d48) Thanks [@copilot-swe-agent](https://github.com/apps/copilot-swe-agent)! - Fix `@stream` directive not working for federated subgraphs with non-null scalar list return types.
+  
+  Two bugs were addressed in `delegateRequest`:
+  
+  1. `isListType` was called without first unwrapping `NonNull`, so a field typed `[String!]!` (a `NonNull(List(...))`) was not recognised as a list. The fix wraps the check with `getNullableType()`.
+  
+  2. Deduplication of already-pushed stream items used a `WeakSet`, which throws a `TypeError` for primitive values (strings, numbers, etc.). This was replaced with an integer index counter that works for both object and primitive list items.
+
+## 12.0.15
+### Patch Changes
+
+
+
+- [#2306](https://github.com/graphql-hive/gateway/pull/2306) [`7171b46`](https://github.com/graphql-hive/gateway/commit/7171b46546a8e60776f70d18d14dd6fa19d5649f) Thanks [@copilot-swe-agent](https://github.com/apps/copilot-swe-agent)! - Fix incorrect error path when delegating a single type to a list type in a subschema.
+  
+  When using schema stitching with type merging (e.g., a supergraph `book` field delegating to a subschema `books` list field via batch delegation), errors from the subschema now correctly report the path as seen in the supergraph (e.g., `['book', 'title']`) instead of including an unexpected array index (e.g., `['book', 0, 'title']`).
+  
+  Also ensures `onLocatedError` is passed through in all branches of `getDelegationContext`.
+
 ## 12.0.14
 ### Patch Changes
 

package/dist/index.cjs

@@ -2647,9 +2647,9 @@ function delegateRequest(options) {
     () => getExecutor(delegationContext)(processedRequest),
     function handleExecutorResult(executorResult) {
       if (utils.isAsyncIterable(executorResult)) {
-        if (delegationContext.operation === "query" && graphql.isListType(delegationContext.returnType)) {
+        if (delegationContext.operation === "query" && graphql.isListType(graphql.getNullableType(delegationContext.returnType))) {
           return new repeater.Repeater(async (push, stop) => {
-            const pushed = /* @__PURE__ */ new WeakSet();
+            let pushedCount = 0;
             let stopped = false;
             stop.finally(() => {
               stopped = true;
@@ -2681,15 +2681,12 @@ function delegateRequest(options) {
                 }
                 const transformedResult = await transformer.transformResult(result);
                 if (Array.isArray(transformedResult)) {
-                  for (const individualResult$ of transformedResult) {
+                  for (let i = pushedCount; i < transformedResult.length; i++) {
                     if (stopped) {
                       break;
                     }
-                    const individualResult = await individualResult$;
-                    if (!pushed.has(individualResult)) {
-                      pushed.add(individualResult);
-                      await push(individualResult);
-                    }
+                    await push(await transformedResult[i]);
+                    pushedCount = i + 1;
                   }
                 } else {
                   await push(await transformedResult);
@@ -2765,7 +2762,9 @@ function getDelegationContext({
     ),
     transforms,
     transformedSchema: transformedSchema ?? subschemaOrSubschemaConfig,
-    skipTypeMerging
+    skipTypeMerging,
+    onLocatedError,
+    args
   };
 }
 function validateRequest(delegationContext, document) {

package/dist/index.js

@@ -2647,9 +2647,9 @@ function delegateRequest(options) {
     () => getExecutor(delegationContext)(processedRequest),
     function handleExecutorResult(executorResult) {
       if (isAsyncIterable(executorResult)) {
-        if (delegationContext.operation === "query" && isListType(delegationContext.returnType)) {
+        if (delegationContext.operation === "query" && isListType(getNullableType(delegationContext.returnType))) {
           return new Repeater(async (push, stop) => {
-            const pushed = /* @__PURE__ */ new WeakSet();
+            let pushedCount = 0;
             let stopped = false;
             stop.finally(() => {
               stopped = true;
@@ -2681,15 +2681,12 @@ function delegateRequest(options) {
                 }
                 const transformedResult = await transformer.transformResult(result);
                 if (Array.isArray(transformedResult)) {
-                  for (const individualResult$ of transformedResult) {
+                  for (let i = pushedCount; i < transformedResult.length; i++) {
                     if (stopped) {
                       break;
                     }
-                    const individualResult = await individualResult$;
-                    if (!pushed.has(individualResult)) {
-                      pushed.add(individualResult);
-                      await push(individualResult);
-                    }
+                    await push(await transformedResult[i]);
+                    pushedCount = i + 1;
                   }
                 } else {
                   await push(await transformedResult);
@@ -2765,7 +2762,9 @@ function getDelegationContext({
     ),
     transforms,
     transformedSchema: transformedSchema ?? subschemaOrSubschemaConfig,
-    skipTypeMerging
+    skipTypeMerging,
+    onLocatedError,
+    args
   };
 }
 function validateRequest(delegationContext, document) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphql-tools/delegate",
-  "version": "12.0.14",
+  "version": "12.0.16",
   "type": "module",
   "description": "A set of utils for faster development of GraphQL tools",
   "repository": {