@graphql-tools/delegate 12.0.15 → 12.0.16-alpha-543a0b9a2221fdc5b35aebcc397631f5145aedc3

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @graphql-tools/delegate
 
+## 12.0.16-alpha-543a0b9a2221fdc5b35aebcc397631f5145aedc3
+### Patch Changes
+
+
+
+- [#2320](https://github.com/graphql-hive/gateway/pull/2320) [`61b9271`](https://github.com/graphql-hive/gateway/commit/61b9271be928551ecb321c60acf2516120d29908) 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
 

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);
-                    }
+                    pushedCount++;
+                    await push(await transformedResult[i]);
                   }
                 } else {
                   await push(await transformedResult);

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);
-                    }
+                    pushedCount++;
+                    await push(await transformedResult[i]);
                   }
                 } else {
                   await push(await transformedResult);

package/package.json

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