@graphql-tools/executor 1.2.4 → 1.2.5-alpha-20240328134228-8452ac4e9c9e6970bfeb2ff7ab914352a0d7a646

package/cjs/execution/execute.js

@@ -830,15 +830,10 @@ function subscribe(args) {
     return mapSourceToResponse(exeContext, resultOrStream);
 }
 exports.subscribe = subscribe;
-function flattenIncrementalResults(incrementalResults, signal) {
+function flattenIncrementalResults(incrementalResults) {
     const subsequentIterator = incrementalResults.subsequentResults;
     let initialResultSent = false;
     let done = false;
-    signal?.addEventListener('abort', () => {
-        done = true;
-        // we catch here to avoid unhandled promise rejection
-        subsequentIterator.throw?.(signal?.reason).catch(() => undefined);
-    });
     return {
         [Symbol.asyncIterator]() {
             return this;
@@ -870,9 +865,9 @@ function flattenIncrementalResults(incrementalResults, signal) {
     };
 }
 exports.flattenIncrementalResults = flattenIncrementalResults;
-async function* ensureAsyncIterable(someExecutionResult, signal) {
+async function* ensureAsyncIterable(someExecutionResult) {
     if ('initialResult' in someExecutionResult) {
-        yield* flattenIncrementalResults(someExecutionResult, signal);
+        yield* flattenIncrementalResults(someExecutionResult);
     }
     else {
         yield someExecutionResult;
@@ -888,7 +883,7 @@ function mapSourceToResponse(exeContext, resultOrStream) {
     // the GraphQL specification. The `execute` function provides the
     // "ExecuteSubscriptionEvent" algorithm, as it is nearly identical to the
     // "ExecuteQuery" algorithm, for which `execute` is also used.
-    return (0, flattenAsyncIterable_js_1.flattenAsyncIterable)((0, utils_1.mapAsyncIterator)(resultOrStream[Symbol.asyncIterator](), async (payload) => ensureAsyncIterable(await executeImpl(buildPerEventExecutionContext(exeContext, payload)), exeContext.signal), (error) => {
+    return (0, flattenAsyncIterable_js_1.flattenAsyncIterable)((0, utils_1.mapAsyncIterator)(resultOrStream[Symbol.asyncIterator](), async (payload) => ensureAsyncIterable(await executeImpl(buildPerEventExecutionContext(exeContext, payload))), (error) => {
         const wrappedError = (0, utils_1.createGraphQLError)(error.message, {
             originalError: error,
             nodes: [exeContext.operation],
@@ -1185,16 +1180,20 @@ function getCompletedIncrementalResults(exeContext) {
 }
 function yieldSubsequentPayloads(exeContext) {
     let isDone = false;
-    exeContext.signal?.addEventListener('abort', () => {
-        for (const payload of exeContext.subsequentPayloads) {
-            payload.cancel(exeContext.signal?.reason);
-        }
+    const abortPromise = new Promise((_, reject) => {
+        exeContext.signal?.addEventListener('abort', () => {
+            isDone = true;
+            reject(exeContext.signal?.reason);
+        });
     });
     async function next() {
         if (isDone) {
             return { value: undefined, done: true };
         }
-        await Promise.race(Array.from(exeContext.subsequentPayloads).map(p => p.promise));
+        await Promise.race([
+            abortPromise,
+            ...Array.from(exeContext.subsequentPayloads).map(p => p.promise),
+        ]);
         if (isDone) {
             // a different call to next has exhausted all payloads
             return { value: undefined, done: true };
@@ -1249,11 +1248,10 @@ class DeferredFragmentRecord {
         this._exeContext.subsequentPayloads.add(this);
         this.isCompleted = false;
         this.data = null;
-        this.promise = new Promise((resolve, reject) => {
+        this.promise = new Promise(resolve => {
             this._resolve = MaybePromise => {
                 resolve(MaybePromise);
             };
-            this._reject = reject;
         }).then(data => {
             this.data = data;
             this.isCompleted = true;
@@ -1267,9 +1265,6 @@ class DeferredFragmentRecord {
         }
         this._resolve?.(data);
     }
-    cancel(reason) {
-        this._reject?.(reason);
-    }
 }
 class StreamRecord {
     constructor(opts) {
@@ -1284,11 +1279,10 @@ class StreamRecord {
         this._exeContext.subsequentPayloads.add(this);
         this.isCompleted = false;
         this.items = null;
-        this.promise = new Promise((resolve, reject) => {
+        this.promise = new Promise(resolve => {
             this._resolve = MaybePromise => {
                 resolve(MaybePromise);
             };
-            this._reject = reject;
         }).then(items => {
             this.items = items;
             this.isCompleted = true;
@@ -1305,9 +1299,6 @@ class StreamRecord {
     setIsCompletedIterator() {
         this.isCompletedIterator = true;
     }
-    cancel(reason) {
-        this._reject?.(reason);
-    }
 }
 function isStreamPayload(asyncPayload) {
     return asyncPayload.type === 'stream';

package/cjs/execution/normalizedExecutor.js

@@ -15,7 +15,7 @@ function normalizedExecutor(args) {
     return new value_or_promise_1.ValueOrPromise(() => (0, execute_js_1.execute)(args))
         .then((result) => {
         if ('initialResult' in result) {
-            return (0, execute_js_1.flattenIncrementalResults)(result, args.signal);
+            return (0, execute_js_1.flattenIncrementalResults)(result);
         }
         return result;
     })

package/esm/execution/execute.js

@@ -819,15 +819,10 @@ export function subscribe(args) {
     }
     return mapSourceToResponse(exeContext, resultOrStream);
 }
-export function flattenIncrementalResults(incrementalResults, signal) {
+export function flattenIncrementalResults(incrementalResults) {
     const subsequentIterator = incrementalResults.subsequentResults;
     let initialResultSent = false;
     let done = false;
-    signal?.addEventListener('abort', () => {
-        done = true;
-        // we catch here to avoid unhandled promise rejection
-        subsequentIterator.throw?.(signal?.reason).catch(() => undefined);
-    });
     return {
         [Symbol.asyncIterator]() {
             return this;
@@ -858,9 +853,9 @@ export function flattenIncrementalResults(incrementalResults, signal) {
         },
     };
 }
-async function* ensureAsyncIterable(someExecutionResult, signal) {
+async function* ensureAsyncIterable(someExecutionResult) {
     if ('initialResult' in someExecutionResult) {
-        yield* flattenIncrementalResults(someExecutionResult, signal);
+        yield* flattenIncrementalResults(someExecutionResult);
     }
     else {
         yield someExecutionResult;
@@ -876,7 +871,7 @@ function mapSourceToResponse(exeContext, resultOrStream) {
     // the GraphQL specification. The `execute` function provides the
     // "ExecuteSubscriptionEvent" algorithm, as it is nearly identical to the
     // "ExecuteQuery" algorithm, for which `execute` is also used.
-    return flattenAsyncIterable(mapAsyncIterator(resultOrStream[Symbol.asyncIterator](), async (payload) => ensureAsyncIterable(await executeImpl(buildPerEventExecutionContext(exeContext, payload)), exeContext.signal), (error) => {
+    return flattenAsyncIterable(mapAsyncIterator(resultOrStream[Symbol.asyncIterator](), async (payload) => ensureAsyncIterable(await executeImpl(buildPerEventExecutionContext(exeContext, payload))), (error) => {
         const wrappedError = createGraphQLError(error.message, {
             originalError: error,
             nodes: [exeContext.operation],
@@ -1173,16 +1168,20 @@ function getCompletedIncrementalResults(exeContext) {
 }
 function yieldSubsequentPayloads(exeContext) {
     let isDone = false;
-    exeContext.signal?.addEventListener('abort', () => {
-        for (const payload of exeContext.subsequentPayloads) {
-            payload.cancel(exeContext.signal?.reason);
-        }
+    const abortPromise = new Promise((_, reject) => {
+        exeContext.signal?.addEventListener('abort', () => {
+            isDone = true;
+            reject(exeContext.signal?.reason);
+        });
     });
     async function next() {
         if (isDone) {
             return { value: undefined, done: true };
         }
-        await Promise.race(Array.from(exeContext.subsequentPayloads).map(p => p.promise));
+        await Promise.race([
+            abortPromise,
+            ...Array.from(exeContext.subsequentPayloads).map(p => p.promise),
+        ]);
         if (isDone) {
             // a different call to next has exhausted all payloads
             return { value: undefined, done: true };
@@ -1237,11 +1236,10 @@ class DeferredFragmentRecord {
         this._exeContext.subsequentPayloads.add(this);
         this.isCompleted = false;
         this.data = null;
-        this.promise = new Promise((resolve, reject) => {
+        this.promise = new Promise(resolve => {
             this._resolve = MaybePromise => {
                 resolve(MaybePromise);
             };
-            this._reject = reject;
         }).then(data => {
             this.data = data;
             this.isCompleted = true;
@@ -1255,9 +1253,6 @@ class DeferredFragmentRecord {
         }
         this._resolve?.(data);
     }
-    cancel(reason) {
-        this._reject?.(reason);
-    }
 }
 class StreamRecord {
     constructor(opts) {
@@ -1272,11 +1267,10 @@ class StreamRecord {
         this._exeContext.subsequentPayloads.add(this);
         this.isCompleted = false;
         this.items = null;
-        this.promise = new Promise((resolve, reject) => {
+        this.promise = new Promise(resolve => {
             this._resolve = MaybePromise => {
                 resolve(MaybePromise);
             };
-            this._reject = reject;
         }).then(items => {
             this.items = items;
             this.isCompleted = true;
@@ -1293,9 +1287,6 @@ class StreamRecord {
     setIsCompletedIterator() {
         this.isCompletedIterator = true;
     }
-    cancel(reason) {
-        this._reject?.(reason);
-    }
 }
 function isStreamPayload(asyncPayload) {
     return asyncPayload.type === 'stream';

package/esm/execution/normalizedExecutor.js

@@ -12,7 +12,7 @@ export function normalizedExecutor(args) {
     return new ValueOrPromise(() => execute(args))
         .then((result) => {
         if ('initialResult' in result) {
-            return flattenIncrementalResults(result, args.signal);
+            return flattenIncrementalResults(result);
         }
         return result;
     })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphql-tools/executor",
-  "version": "1.2.4",
+  "version": "1.2.5-alpha-20240328134228-8452ac4e9c9e6970bfeb2ff7ab914352a0d7a646",
   "sideEffects": false,
   "peerDependencies": {
     "graphql": "^14.0.0 || ^15.0.0 || ^16.0.0 || ^17.0.0"

package/typings/execution/execute.d.cts

@@ -204,7 +204,7 @@ export declare const defaultFieldResolver: GraphQLFieldResolver<unknown, unknown
  * Accepts an object with named arguments.
  */
 export declare function subscribe<TData = any, TVariables = any, TContext = any>(args: ExecutionArgs<TData, TVariables, TContext>): MaybePromise<AsyncGenerator<SingularExecutionResult<TData> | InitialIncrementalExecutionResult<TData> | SubsequentIncrementalExecutionResult<TData>, void, void> | SingularExecutionResult<TData>>;
-export declare function flattenIncrementalResults<TData>(incrementalResults: IncrementalExecutionResults<TData>, signal?: AbortSignal): AsyncGenerator<SubsequentIncrementalExecutionResult<TData, Record<string, unknown>>, void, void>;
+export declare function flattenIncrementalResults<TData>(incrementalResults: IncrementalExecutionResults<TData>): AsyncGenerator<SubsequentIncrementalExecutionResult<TData, Record<string, unknown>>, void, void>;
 declare class DeferredFragmentRecord {
     type: 'defer';
     errors: Array<GraphQLError>;
@@ -216,7 +216,6 @@ declare class DeferredFragmentRecord {
     isCompleted: boolean;
     _exeContext: ExecutionContext;
     _resolve?: (arg: MaybePromise<Record<string, unknown> | null>) => void;
-    _reject?: (reason: unknown) => void;
     constructor(opts: {
         label: string | undefined;
         path: Path | undefined;
@@ -224,7 +223,6 @@ declare class DeferredFragmentRecord {
         exeContext: ExecutionContext;
     });
     addData(data: MaybePromise<Record<string, unknown> | null>): void;
-    cancel(reason: unknown): void;
 }
 declare class StreamRecord {
     type: 'stream';
@@ -239,7 +237,6 @@ declare class StreamRecord {
     isCompleted: boolean;
     _exeContext: ExecutionContext;
     _resolve?: (arg: MaybePromise<Array<unknown> | null>) => void;
-    _reject?: (reason: unknown) => void;
     constructor(opts: {
         label: string | undefined;
         path: Path | undefined;
@@ -249,7 +246,6 @@ declare class StreamRecord {
     });
     addItems(items: MaybePromise<Array<unknown> | null>): void;
     setIsCompletedIterator(): void;
-    cancel(reason: unknown): void;
 }
 type AsyncPayloadRecord = DeferredFragmentRecord | StreamRecord;
 /**

package/typings/execution/execute.d.ts

@@ -204,7 +204,7 @@ export declare const defaultFieldResolver: GraphQLFieldResolver<unknown, unknown
  * Accepts an object with named arguments.
  */
 export declare function subscribe<TData = any, TVariables = any, TContext = any>(args: ExecutionArgs<TData, TVariables, TContext>): MaybePromise<AsyncGenerator<SingularExecutionResult<TData> | InitialIncrementalExecutionResult<TData> | SubsequentIncrementalExecutionResult<TData>, void, void> | SingularExecutionResult<TData>>;
-export declare function flattenIncrementalResults<TData>(incrementalResults: IncrementalExecutionResults<TData>, signal?: AbortSignal): AsyncGenerator<SubsequentIncrementalExecutionResult<TData, Record<string, unknown>>, void, void>;
+export declare function flattenIncrementalResults<TData>(incrementalResults: IncrementalExecutionResults<TData>): AsyncGenerator<SubsequentIncrementalExecutionResult<TData, Record<string, unknown>>, void, void>;
 declare class DeferredFragmentRecord {
     type: 'defer';
     errors: Array<GraphQLError>;
@@ -216,7 +216,6 @@ declare class DeferredFragmentRecord {
     isCompleted: boolean;
     _exeContext: ExecutionContext;
     _resolve?: (arg: MaybePromise<Record<string, unknown> | null>) => void;
-    _reject?: (reason: unknown) => void;
     constructor(opts: {
         label: string | undefined;
         path: Path | undefined;
@@ -224,7 +223,6 @@ declare class DeferredFragmentRecord {
         exeContext: ExecutionContext;
     });
     addData(data: MaybePromise<Record<string, unknown> | null>): void;
-    cancel(reason: unknown): void;
 }
 declare class StreamRecord {
     type: 'stream';
@@ -239,7 +237,6 @@ declare class StreamRecord {
     isCompleted: boolean;
     _exeContext: ExecutionContext;
     _resolve?: (arg: MaybePromise<Array<unknown> | null>) => void;
-    _reject?: (reason: unknown) => void;
     constructor(opts: {
         label: string | undefined;
         path: Path | undefined;
@@ -249,7 +246,6 @@ declare class StreamRecord {
     });
     addItems(items: MaybePromise<Array<unknown> | null>): void;
     setIsCompletedIterator(): void;
-    cancel(reason: unknown): void;
 }
 type AsyncPayloadRecord = DeferredFragmentRecord | StreamRecord;
 /**