@whatwg-node/node-fetch 0.7.23-alpha-20250726084802-683ac7e676415a5d08903fcace71146b2a74a837 → 0.7.23-alpha-20250727153122-fb7fb010b88fa0b07d5665b008435071f7f23bc5

package/cjs/Body.js

@@ -36,11 +36,13 @@ class PonyfillBody {
         this.contentLength = contentLength;
         this.bodyType = bodyType;
         this._buffer = buffer;
+        this._signal = options.signal;
     }
     bodyType;
     _bodyFactory = () => null;
     _generatedBody = null;
     _buffer;
+    _signal;
     generateBody() {
         if (this._generatedBody?.readable?.destroyed && this._buffer) {
             this._generatedBody.readable = node_stream_1.Readable.from(this._buffer);
@@ -131,8 +133,7 @@ class PonyfillBody {
             return (0, utils_js_1.fakePromise)(this._chunks);
         }
         if (_body.readable.destroyed) {
-            this._chunks = [];
-            return (0, utils_js_1.fakePromise)(this._chunks);
+            return (0, utils_js_1.fakePromise)((this._chunks = []));
         }
         const chunks = [];
         return new Promise((resolve, reject) => {
@@ -219,6 +220,9 @@ class PonyfillBody {
                 limits: formDataLimits,
                 defCharset: 'utf-8',
             });
+            if (this._signal) {
+                (0, node_stream_1.addAbortSignal)(this._signal, bb);
+            }
             let completed = false;
             const complete = (err) => {
                 if (completed)

package/cjs/Request.js

@@ -76,7 +76,6 @@ class PonyfillRequest extends Body_js_1.PonyfillBody {
                 this.agent = requestInit.agent;
             }
         }
-        this._signal = requestInit?.signal || undefined;
     }
     headersSerializer;
     cache;
@@ -92,7 +91,6 @@ class PonyfillRequest extends Body_js_1.PonyfillBody {
     referrer;
     referrerPolicy;
     _url;
-    _signal;
     get signal() {
         this._signal ||= new AbortController().signal;
         return this._signal;

package/cjs/fetchNodeHttp.js

@@ -92,10 +92,21 @@ function fetchNodeHttp(fetchRequest) {
                         return;
                     }
                 }
-                outputStream ||= new node_stream_1.PassThrough({
+                outputStream ||= new node_stream_1.PassThrough();
+                (0, utils_js_1.pipeThrough)({
+                    src: nodeResponse,
+                    dest: outputStream,
                     signal,
+                    onError: e => {
+                        if (!nodeResponse.destroyed) {
+                            nodeResponse.destroy(e);
+                        }
+                        if (!outputStream.destroyed) {
+                            outputStream.destroy(e);
+                        }
+                        reject(e);
+                    },
                 });
-                nodeResponse.pipe(outputStream);
                 const statusCode = nodeResponse.statusCode || 200;
                 let statusText = nodeResponse.statusMessage || node_http_1.STATUS_CODES[statusCode];
                 if (statusText == null) {
@@ -106,6 +117,7 @@ function fetchNodeHttp(fetchRequest) {
                     statusText,
                     headers: nodeResponse.headers,
                     url: fetchRequest.url,
+                    signal,
                 });
                 resolve(ponyfillResponse);
             });

package/cjs/utils.js

@@ -7,6 +7,7 @@ exports.isArrayBufferView = isArrayBufferView;
 exports.isNodeReadable = isNodeReadable;
 exports.isIterable = isIterable;
 exports.shouldRedirect = shouldRedirect;
+exports.pipeThrough = pipeThrough;
 exports.endStream = endStream;
 exports.safeWrite = safeWrite;
 const node_events_1 = require("node:events");
@@ -48,6 +49,42 @@ function isIterable(value) {
 function shouldRedirect(status) {
     return status === 301 || status === 302 || status === 303 || status === 307 || status === 308;
 }
+function pipeThrough({ src, dest, signal, onError, }) {
+    if (onError) {
+        // listen for errors on the destination stream if necessary. if the readable
+        // stream (src) emits an error, the writable destination (dest) will be
+        // destroyed with that error (see below)
+        dest.once('error', onError);
+    }
+    src.once('error', (e) => {
+        // if the readable stream (src) emits an error during pipe, the writable
+        // destination (dest) is not closed automatically. that needs to be
+        // done manually. the readable stream is closed when error is emitted,
+        // so only the writable destination needs to be destroyed
+        dest.destroy(e);
+    });
+    if (signal) {
+        // this is faster than `import('node:signal').addAbortSignal(signal, src)`
+        const srcRef = new WeakRef(src);
+        const signalRef = new WeakRef(signal);
+        function cleanup() {
+            signalRef.deref()?.removeEventListener('abort', onAbort);
+            srcRef.deref()?.removeListener('end', cleanup);
+            srcRef.deref()?.removeListener('error', cleanup);
+            srcRef.deref()?.removeListener('close', cleanup);
+        }
+        function onAbort() {
+            srcRef.deref()?.destroy(new AbortError());
+            cleanup();
+        }
+        signal.addEventListener('abort', onAbort, { once: true });
+        // this is faster than `import('node:signal').finished(src, cleanup)`
+        src.once('end', cleanup);
+        src.once('error', cleanup);
+        src.once('close', cleanup);
+    }
+    src.pipe(dest, { end: true /* already default */ });
+}
 function endStream(stream) {
     // @ts-expect-error Avoid arguments adaptor trampoline https://v8.dev/blog/adaptor-frame
     return stream.end(null, null, null);
@@ -58,3 +95,10 @@ function safeWrite(chunk, stream) {
         return (0, node_events_1.once)(stream, 'drain');
     }
 }
+// https://github.com/nodejs/node/blob/f692878dec6354c0a82241f224906981861bc840/lib/internal/errors.js#L961-L973
+class AbortError extends Error {
+    constructor(message = 'The operation was aborted', options = undefined) {
+        super(message, options);
+        this.name = 'AbortError';
+    }
+}

package/esm/Body.js

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/ban-ts-comment */
 import { Buffer } from 'node:buffer';
-import { Readable } from 'node:stream';
+import { addAbortSignal, Readable } from 'node:stream';
 import { Busboy } from '@fastify/busboy';
 import { handleMaybePromise } from '@whatwg-node/promise-helpers';
 import { hasArrayBufferMethod, hasBufferMethod, hasBytesMethod, PonyfillBlob } from './Blob.js';
@@ -33,11 +33,13 @@ export class PonyfillBody {
         this.contentLength = contentLength;
         this.bodyType = bodyType;
         this._buffer = buffer;
+        this._signal = options.signal;
     }
     bodyType;
     _bodyFactory = () => null;
     _generatedBody = null;
     _buffer;
+    _signal;
     generateBody() {
         if (this._generatedBody?.readable?.destroyed && this._buffer) {
             this._generatedBody.readable = Readable.from(this._buffer);
@@ -128,8 +130,7 @@ export class PonyfillBody {
             return fakePromise(this._chunks);
         }
         if (_body.readable.destroyed) {
-            this._chunks = [];
-            return fakePromise(this._chunks);
+            return fakePromise((this._chunks = []));
         }
         const chunks = [];
         return new Promise((resolve, reject) => {
@@ -216,6 +217,9 @@ export class PonyfillBody {
                 limits: formDataLimits,
                 defCharset: 'utf-8',
             });
+            if (this._signal) {
+                addAbortSignal(this._signal, bb);
+            }
             let completed = false;
             const complete = (err) => {
                 if (completed)

package/esm/Request.js

@@ -73,7 +73,6 @@ export class PonyfillRequest extends PonyfillBody {
                 this.agent = requestInit.agent;
             }
         }
-        this._signal = requestInit?.signal || undefined;
     }
     headersSerializer;
     cache;
@@ -89,7 +88,6 @@ export class PonyfillRequest extends PonyfillBody {
     referrer;
     referrerPolicy;
     _url;
-    _signal;
     get signal() {
         this._signal ||= new AbortController().signal;
         return this._signal;

package/esm/fetchNodeHttp.js

@@ -6,7 +6,7 @@ import { handleMaybePromise } from '@whatwg-node/promise-helpers';
 import { PonyfillRequest } from './Request.js';
 import { PonyfillResponse } from './Response.js';
 import { PonyfillURL } from './URL.js';
-import { endStream, getHeadersObj, isNodeReadable, safeWrite, shouldRedirect } from './utils.js';
+import { endStream, getHeadersObj, isNodeReadable, pipeThrough, safeWrite, shouldRedirect, } from './utils.js';
 function getRequestFnForProtocol(url) {
     if (url.startsWith('http:')) {
         return httpRequest;
@@ -89,10 +89,21 @@ export function fetchNodeHttp(fetchRequest) {
                         return;
                     }
                 }
-                outputStream ||= new PassThrough({
+                outputStream ||= new PassThrough();
+                pipeThrough({
+                    src: nodeResponse,
+                    dest: outputStream,
                     signal,
+                    onError: e => {
+                        if (!nodeResponse.destroyed) {
+                            nodeResponse.destroy(e);
+                        }
+                        if (!outputStream.destroyed) {
+                            outputStream.destroy(e);
+                        }
+                        reject(e);
+                    },
                 });
-                nodeResponse.pipe(outputStream);
                 const statusCode = nodeResponse.statusCode || 200;
                 let statusText = nodeResponse.statusMessage || STATUS_CODES[statusCode];
                 if (statusText == null) {
@@ -103,6 +114,7 @@ export function fetchNodeHttp(fetchRequest) {
                     statusText,
                     headers: nodeResponse.headers,
                     url: fetchRequest.url,
+                    signal,
                 });
                 resolve(ponyfillResponse);
             });

package/esm/utils.js

@@ -36,6 +36,42 @@ export function isIterable(value) {
 export function shouldRedirect(status) {
     return status === 301 || status === 302 || status === 303 || status === 307 || status === 308;
 }
+export function pipeThrough({ src, dest, signal, onError, }) {
+    if (onError) {
+        // listen for errors on the destination stream if necessary. if the readable
+        // stream (src) emits an error, the writable destination (dest) will be
+        // destroyed with that error (see below)
+        dest.once('error', onError);
+    }
+    src.once('error', (e) => {
+        // if the readable stream (src) emits an error during pipe, the writable
+        // destination (dest) is not closed automatically. that needs to be
+        // done manually. the readable stream is closed when error is emitted,
+        // so only the writable destination needs to be destroyed
+        dest.destroy(e);
+    });
+    if (signal) {
+        // this is faster than `import('node:signal').addAbortSignal(signal, src)`
+        const srcRef = new WeakRef(src);
+        const signalRef = new WeakRef(signal);
+        function cleanup() {
+            signalRef.deref()?.removeEventListener('abort', onAbort);
+            srcRef.deref()?.removeListener('end', cleanup);
+            srcRef.deref()?.removeListener('error', cleanup);
+            srcRef.deref()?.removeListener('close', cleanup);
+        }
+        function onAbort() {
+            srcRef.deref()?.destroy(new AbortError());
+            cleanup();
+        }
+        signal.addEventListener('abort', onAbort, { once: true });
+        // this is faster than `import('node:signal').finished(src, cleanup)`
+        src.once('end', cleanup);
+        src.once('error', cleanup);
+        src.once('close', cleanup);
+    }
+    src.pipe(dest, { end: true /* already default */ });
+}
 export function endStream(stream) {
     // @ts-expect-error Avoid arguments adaptor trampoline https://v8.dev/blog/adaptor-frame
     return stream.end(null, null, null);
@@ -46,3 +82,10 @@ export function safeWrite(chunk, stream) {
         return once(stream, 'drain');
     }
 }
+// https://github.com/nodejs/node/blob/f692878dec6354c0a82241f224906981861bc840/lib/internal/errors.js#L961-L973
+class AbortError extends Error {
+    constructor(message = 'The operation was aborted', options = undefined) {
+        super(message, options);
+        this.name = 'AbortError';
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@whatwg-node/node-fetch",
-  "version": "0.7.23-alpha-20250726084802-683ac7e676415a5d08903fcace71146b2a74a837",
+  "version": "0.7.23-alpha-20250727153122-fb7fb010b88fa0b07d5665b008435071f7f23bc5",
   "description": "Fetch API implementation for Node",
   "sideEffects": false,
   "dependencies": {

package/typings/Body.d.cts

@@ -29,6 +29,7 @@ export declare class PonyfillBody<TJSON = any> implements Body {
     private _bodyFactory;
     private _generatedBody;
     private _buffer?;
+    _signal?: AbortSignal | undefined;
     private generateBody;
     protected handleContentLengthHeader(this: PonyfillBody & {
         headers: Headers;

package/typings/Body.d.ts

@@ -29,6 +29,7 @@ export declare class PonyfillBody<TJSON = any> implements Body {
     private _bodyFactory;
     private _generatedBody;
     private _buffer?;
+    _signal?: AbortSignal | undefined;
     private generateBody;
     protected handleContentLengthHeader(this: PonyfillBody & {
         headers: Headers;

package/typings/Request.d.cts

@@ -26,7 +26,6 @@ export declare class PonyfillRequest<TJSON = any> extends PonyfillBody<TJSON> im
     referrer: string;
     referrerPolicy: ReferrerPolicy;
     _url: string | undefined;
-    _signal: AbortSignal | undefined;
     get signal(): AbortSignal;
     get url(): string;
     _parsedUrl: URL | undefined;

package/typings/Request.d.ts

@@ -26,7 +26,6 @@ export declare class PonyfillRequest<TJSON = any> extends PonyfillBody<TJSON> im
     referrer: string;
     referrerPolicy: ReferrerPolicy;
     _url: string | undefined;
-    _signal: AbortSignal | undefined;
     get signal(): AbortSignal;
     get url(): string;
     _parsedUrl: URL | undefined;

package/typings/utils.d.cts

@@ -6,6 +6,12 @@ export declare function isArrayBufferView(obj: any): obj is ArrayBufferView;
 export declare function isNodeReadable(obj: any): obj is Readable;
 export declare function isIterable(value: any): value is Iterable<unknown>;
 export declare function shouldRedirect(status?: number): boolean;
+export declare function pipeThrough({ src, dest, signal, onError, }: {
+    src: Readable;
+    dest: Writable;
+    signal?: AbortSignal | undefined;
+    onError?: ((e: Error) => void) | undefined;
+}): void;
 export declare function endStream(stream: {
     end: () => void;
 }): void;

package/typings/utils.d.ts

@@ -6,6 +6,12 @@ export declare function isArrayBufferView(obj: any): obj is ArrayBufferView;
 export declare function isNodeReadable(obj: any): obj is Readable;
 export declare function isIterable(value: any): value is Iterable<unknown>;
 export declare function shouldRedirect(status?: number): boolean;
+export declare function pipeThrough({ src, dest, signal, onError, }: {
+    src: Readable;
+    dest: Writable;
+    signal?: AbortSignal | undefined;
+    onError?: ((e: Error) => void) | undefined;
+}): void;
 export declare function endStream(stream: {
     end: () => void;
 }): void;