@upyo/retry 0.5.0-dev.195 → 0.5.0-dev.199

package/dist/index.cjs

@@ -113,6 +113,7 @@ var RetryTransport = class {
 	* @param transport The transport to wrap.
 	* @param config Retry configuration.
 	* @throws {RangeError} If retry configuration is invalid.
+	* @since 0.5.0
 	*/
 	constructor(transport, config = {}) {
 		this.wrappedTransport = transport;
@@ -126,25 +127,16 @@ var RetryTransport = class {
 	* @param options Optional transport options.
 	* @returns The final delivery receipt.
 	* @throws {DOMException} If the operation is aborted.
+	* @since 0.5.0
 	*/
 	async send(message, options) {
 		options?.signal?.throwIfAborted();
 		let lastThrownError;
 		for (let attempt = 1; attempt <= this.config.maxAttempts; attempt++) {
 			options?.signal?.throwIfAborted();
+			let receipt;
 			try {
-				const receipt = await this.wrappedTransport.send(message, options);
-				if (receipt.successful) return this.withSuccessMetadata(receipt, attempt);
-				const failedReceipt = this.withFailureMetadata(receipt, attempt);
-				if (attempt >= this.config.maxAttempts || !this.shouldRetryReceipt(failedReceipt)) return failedReceipt;
-				await this.waitBeforeRetry({
-					attempt,
-					nextAttempt: attempt + 1,
-					maxAttempts: this.config.maxAttempts,
-					delayMilliseconds: this.calculateDelay(attempt, failedReceipt),
-					receipt: failedReceipt,
-					reason: "retry"
-				}, options?.signal);
+				receipt = await this.wrappedTransport.send(message, options);
 			} catch (error) {
 				options?.signal?.throwIfAborted();
 				lastThrownError = error;
@@ -157,7 +149,20 @@ var RetryTransport = class {
 					error,
 					reason: "retry"
 				}, options?.signal);
+				continue;
 			}
+			options?.signal?.throwIfAborted();
+			if (receipt.successful) return this.withSuccessMetadata(receipt, attempt);
+			const failedReceipt = this.withFailureMetadata(receipt, attempt);
+			if (attempt >= this.config.maxAttempts || !this.shouldRetryReceipt(failedReceipt)) return failedReceipt;
+			await this.waitBeforeRetry({
+				attempt,
+				nextAttempt: attempt + 1,
+				maxAttempts: this.config.maxAttempts,
+				delayMilliseconds: this.calculateDelay(attempt, failedReceipt),
+				receipt: failedReceipt,
+				reason: "retry"
+			}, options?.signal);
 		}
 		return this.createThrownFailure(lastThrownError, this.config.maxAttempts);
 	}
@@ -170,6 +175,7 @@ var RetryTransport = class {
 	* @param options Optional transport options.
 	* @returns An async iterable of receipts.
 	* @throws {DOMException} If the operation is aborted.
+	* @since 0.5.0
 	*/
 	async *sendMany(messages, options) {
 		const iterator = toAsyncIterator(messages);
@@ -193,12 +199,16 @@ var RetryTransport = class {
 			if (inputDone) return;
 			combinedSignal.signal.throwIfAborted();
 			pullingInput = true;
-			let next;
+			let inputPull;
 			try {
-				next = await iterator.next();
-			} finally {
-				pullingInput = false;
+				inputPull = Promise.resolve(iterator.next());
+			} catch (error) {
+				inputPull = Promise.reject(error);
 			}
+			inputPull = inputPull.finally(() => {
+				pullingInput = false;
+			});
+			const next = await raceWithAbort(inputPull, combinedSignal.signal);
 			if (closed) return;
 			if (next.done) {
 				inputDone = true;
@@ -219,7 +229,7 @@ var RetryTransport = class {
 			inFlight.set(index, promise);
 		};
 		const startLaunch = () => {
-			if (launchPromise != null || inputDone || inFlight.size >= this.config.sendMany.maxConcurrent) return;
+			if (launchPromise != null || inputDone || hasLaunchError || inFlight.size >= this.config.sendMany.maxConcurrent) return;
 			launchPromise = launchNext().catch((error) => {
 				launchError = error;
 				hasLaunchError = true;
@@ -239,7 +249,7 @@ var RetryTransport = class {
 					nextYieldIndex++;
 					startLaunch();
 				}
-				if (hasLaunchError) throw launchError;
+				if (hasLaunchError && inFlight.size <= 0) throw launchError;
 				startLaunch();
 				if (inFlight.size <= 0 && launchPromise == null) break;
 				const launch = launchPromise?.then(() => ({ launched: true }));
@@ -255,13 +265,10 @@ var RetryTransport = class {
 			controller.abort(new DOMException("The operation was aborted.", "AbortError"));
 			launchPromise?.catch(() => {});
 			try {
-				const closeIterator = async () => {
-					if (!inputDone) await iterator.return?.();
-				};
-				if (pullingInput) launchPromise?.finally(() => {
-					closeIterator().catch(() => {});
-				});
-				else await closeIterator();
+				const closeIterator = () => inputDone ? void 0 : iterator.return?.();
+				const closePromise = closeIterator();
+				if (pullingInput) closePromise?.catch(() => {});
+				else await closePromise;
 			} finally {
 				combinedSignal.cleanup();
 			}
@@ -270,6 +277,8 @@ var RetryTransport = class {
 	/**
 	* Disposes the wrapped transport when it supports disposal.
 	*
+	* @throws {unknown} If the wrapped transport's async or sync disposal hook
+	*   throws or rejects.
 	* @since 0.5.0
 	*/
 	async [Symbol.asyncDispose]() {
@@ -324,7 +333,7 @@ var RetryTransport = class {
 	}
 	calculateDelay(attempt, failure) {
 		const retryAfterMilliseconds = getRetryAfterMilliseconds(failure);
-		const cappedRetryAfter = retryAfterMilliseconds == null ? void 0 : Math.min(retryAfterMilliseconds, this.config.backoff.maxDelayMilliseconds);
+		const cappedRetryAfter = retryAfterMilliseconds == null || !Number.isFinite(retryAfterMilliseconds) || retryAfterMilliseconds <= 0 ? void 0 : Math.min(retryAfterMilliseconds, this.config.backoff.maxDelayMilliseconds);
 		if (cappedRetryAfter != null) return cappedRetryAfter;
 		const computedDelay = Math.min(this.config.backoff.baseDelayMilliseconds * Math.pow(this.config.backoff.factor, attempt - 1), this.config.backoff.maxDelayMilliseconds);
 		if (this.config.jitter === false || this.config.jitter === "none") return computedDelay;
@@ -387,8 +396,40 @@ function getRetryAfterMilliseconds(failure) {
 	if (isReceiptError(failure)) return failure.retryAfterMilliseconds;
 	return failure.errors?.find((error) => error.retryAfterMilliseconds != null)?.retryAfterMilliseconds;
 }
-async function* toAsyncIterator(values) {
-	for await (const value of values) yield value;
+function toAsyncIterator(values) {
+	if (Symbol.asyncIterator in values) return values[Symbol.asyncIterator]();
+	const iterator = values[Symbol.iterator]();
+	return {
+		async next() {
+			return await Promise.resolve(iterator.next());
+		},
+		async return(value) {
+			return await Promise.resolve(iterator.return?.(value) ?? {
+				done: true,
+				value
+			});
+		}
+	};
+}
+function raceWithAbort(promise, signal) {
+	if (signal.aborted) return Promise.reject(signal.reason);
+	return new Promise((resolve, reject) => {
+		const abort = () => {
+			cleanup();
+			reject(signal.reason);
+		};
+		const cleanup = () => {
+			signal.removeEventListener("abort", abort);
+		};
+		signal.addEventListener("abort", abort, { once: true });
+		promise.then((value) => {
+			cleanup();
+			resolve(value);
+		}, (error) => {
+			cleanup();
+			reject(error);
+		});
+	});
 }
 function toReceiptError(value) {
 	return isReceiptError(value) ? value : void 0;

package/dist/index.d.cts

@@ -207,6 +207,7 @@ declare class RetryTransport<TProviderId extends string = string> implements Tra
    * @param transport The transport to wrap.
    * @param config Retry configuration.
    * @throws {RangeError} If retry configuration is invalid.
+   * @since 0.5.0
    */
   constructor(transport: Transport<TProviderId>, config?: RetryConfig<TProviderId>);
   /**
@@ -216,6 +217,7 @@ declare class RetryTransport<TProviderId extends string = string> implements Tra
    * @param options Optional transport options.
    * @returns The final delivery receipt.
    * @throws {DOMException} If the operation is aborted.
+   * @since 0.5.0
    */
   send(message: Message, options?: TransportOptions): Promise<Receipt<TProviderId>>;
   /**
@@ -227,11 +229,14 @@ declare class RetryTransport<TProviderId extends string = string> implements Tra
    * @param options Optional transport options.
    * @returns An async iterable of receipts.
    * @throws {DOMException} If the operation is aborted.
+   * @since 0.5.0
    */
   sendMany(messages: Iterable<Message> | AsyncIterable<Message>, options?: TransportOptions): AsyncIterable<Receipt<TProviderId>>;
   /**
    * Disposes the wrapped transport when it supports disposal.
    *
+   * @throws {unknown} If the wrapped transport's async or sync disposal hook
+   *   throws or rejects.
    * @since 0.5.0
    */
   [Symbol.asyncDispose](): Promise<void>;

package/dist/index.d.ts

@@ -207,6 +207,7 @@ declare class RetryTransport<TProviderId extends string = string> implements Tra
    * @param transport The transport to wrap.
    * @param config Retry configuration.
    * @throws {RangeError} If retry configuration is invalid.
+   * @since 0.5.0
    */
   constructor(transport: Transport<TProviderId>, config?: RetryConfig<TProviderId>);
   /**
@@ -216,6 +217,7 @@ declare class RetryTransport<TProviderId extends string = string> implements Tra
    * @param options Optional transport options.
    * @returns The final delivery receipt.
    * @throws {DOMException} If the operation is aborted.
+   * @since 0.5.0
    */
   send(message: Message, options?: TransportOptions): Promise<Receipt<TProviderId>>;
   /**
@@ -227,11 +229,14 @@ declare class RetryTransport<TProviderId extends string = string> implements Tra
    * @param options Optional transport options.
    * @returns An async iterable of receipts.
    * @throws {DOMException} If the operation is aborted.
+   * @since 0.5.0
    */
   sendMany(messages: Iterable<Message> | AsyncIterable<Message>, options?: TransportOptions): AsyncIterable<Receipt<TProviderId>>;
   /**
    * Disposes the wrapped transport when it supports disposal.
    *
+   * @throws {unknown} If the wrapped transport's async or sync disposal hook
+   *   throws or rejects.
    * @since 0.5.0
    */
   [Symbol.asyncDispose](): Promise<void>;

package/dist/index.js

@@ -90,6 +90,7 @@ var RetryTransport = class {
 	* @param transport The transport to wrap.
 	* @param config Retry configuration.
 	* @throws {RangeError} If retry configuration is invalid.
+	* @since 0.5.0
 	*/
 	constructor(transport, config = {}) {
 		this.wrappedTransport = transport;
@@ -103,25 +104,16 @@ var RetryTransport = class {
 	* @param options Optional transport options.
 	* @returns The final delivery receipt.
 	* @throws {DOMException} If the operation is aborted.
+	* @since 0.5.0
 	*/
 	async send(message, options) {
 		options?.signal?.throwIfAborted();
 		let lastThrownError;
 		for (let attempt = 1; attempt <= this.config.maxAttempts; attempt++) {
 			options?.signal?.throwIfAborted();
+			let receipt;
 			try {
-				const receipt = await this.wrappedTransport.send(message, options);
-				if (receipt.successful) return this.withSuccessMetadata(receipt, attempt);
-				const failedReceipt = this.withFailureMetadata(receipt, attempt);
-				if (attempt >= this.config.maxAttempts || !this.shouldRetryReceipt(failedReceipt)) return failedReceipt;
-				await this.waitBeforeRetry({
-					attempt,
-					nextAttempt: attempt + 1,
-					maxAttempts: this.config.maxAttempts,
-					delayMilliseconds: this.calculateDelay(attempt, failedReceipt),
-					receipt: failedReceipt,
-					reason: "retry"
-				}, options?.signal);
+				receipt = await this.wrappedTransport.send(message, options);
 			} catch (error) {
 				options?.signal?.throwIfAborted();
 				lastThrownError = error;
@@ -134,7 +126,20 @@ var RetryTransport = class {
 					error,
 					reason: "retry"
 				}, options?.signal);
+				continue;
 			}
+			options?.signal?.throwIfAborted();
+			if (receipt.successful) return this.withSuccessMetadata(receipt, attempt);
+			const failedReceipt = this.withFailureMetadata(receipt, attempt);
+			if (attempt >= this.config.maxAttempts || !this.shouldRetryReceipt(failedReceipt)) return failedReceipt;
+			await this.waitBeforeRetry({
+				attempt,
+				nextAttempt: attempt + 1,
+				maxAttempts: this.config.maxAttempts,
+				delayMilliseconds: this.calculateDelay(attempt, failedReceipt),
+				receipt: failedReceipt,
+				reason: "retry"
+			}, options?.signal);
 		}
 		return this.createThrownFailure(lastThrownError, this.config.maxAttempts);
 	}
@@ -147,6 +152,7 @@ var RetryTransport = class {
 	* @param options Optional transport options.
 	* @returns An async iterable of receipts.
 	* @throws {DOMException} If the operation is aborted.
+	* @since 0.5.0
 	*/
 	async *sendMany(messages, options) {
 		const iterator = toAsyncIterator(messages);
@@ -170,12 +176,16 @@ var RetryTransport = class {
 			if (inputDone) return;
 			combinedSignal.signal.throwIfAborted();
 			pullingInput = true;
-			let next;
+			let inputPull;
 			try {
-				next = await iterator.next();
-			} finally {
-				pullingInput = false;
+				inputPull = Promise.resolve(iterator.next());
+			} catch (error) {
+				inputPull = Promise.reject(error);
 			}
+			inputPull = inputPull.finally(() => {
+				pullingInput = false;
+			});
+			const next = await raceWithAbort(inputPull, combinedSignal.signal);
 			if (closed) return;
 			if (next.done) {
 				inputDone = true;
@@ -196,7 +206,7 @@ var RetryTransport = class {
 			inFlight.set(index, promise);
 		};
 		const startLaunch = () => {
-			if (launchPromise != null || inputDone || inFlight.size >= this.config.sendMany.maxConcurrent) return;
+			if (launchPromise != null || inputDone || hasLaunchError || inFlight.size >= this.config.sendMany.maxConcurrent) return;
 			launchPromise = launchNext().catch((error) => {
 				launchError = error;
 				hasLaunchError = true;
@@ -216,7 +226,7 @@ var RetryTransport = class {
 					nextYieldIndex++;
 					startLaunch();
 				}
-				if (hasLaunchError) throw launchError;
+				if (hasLaunchError && inFlight.size <= 0) throw launchError;
 				startLaunch();
 				if (inFlight.size <= 0 && launchPromise == null) break;
 				const launch = launchPromise?.then(() => ({ launched: true }));
@@ -232,13 +242,10 @@ var RetryTransport = class {
 			controller.abort(new DOMException("The operation was aborted.", "AbortError"));
 			launchPromise?.catch(() => {});
 			try {
-				const closeIterator = async () => {
-					if (!inputDone) await iterator.return?.();
-				};
-				if (pullingInput) launchPromise?.finally(() => {
-					closeIterator().catch(() => {});
-				});
-				else await closeIterator();
+				const closeIterator = () => inputDone ? void 0 : iterator.return?.();
+				const closePromise = closeIterator();
+				if (pullingInput) closePromise?.catch(() => {});
+				else await closePromise;
 			} finally {
 				combinedSignal.cleanup();
 			}
@@ -247,6 +254,8 @@ var RetryTransport = class {
 	/**
 	* Disposes the wrapped transport when it supports disposal.
 	*
+	* @throws {unknown} If the wrapped transport's async or sync disposal hook
+	*   throws or rejects.
 	* @since 0.5.0
 	*/
 	async [Symbol.asyncDispose]() {
@@ -301,7 +310,7 @@ var RetryTransport = class {
 	}
 	calculateDelay(attempt, failure) {
 		const retryAfterMilliseconds = getRetryAfterMilliseconds(failure);
-		const cappedRetryAfter = retryAfterMilliseconds == null ? void 0 : Math.min(retryAfterMilliseconds, this.config.backoff.maxDelayMilliseconds);
+		const cappedRetryAfter = retryAfterMilliseconds == null || !Number.isFinite(retryAfterMilliseconds) || retryAfterMilliseconds <= 0 ? void 0 : Math.min(retryAfterMilliseconds, this.config.backoff.maxDelayMilliseconds);
 		if (cappedRetryAfter != null) return cappedRetryAfter;
 		const computedDelay = Math.min(this.config.backoff.baseDelayMilliseconds * Math.pow(this.config.backoff.factor, attempt - 1), this.config.backoff.maxDelayMilliseconds);
 		if (this.config.jitter === false || this.config.jitter === "none") return computedDelay;
@@ -364,8 +373,40 @@ function getRetryAfterMilliseconds(failure) {
 	if (isReceiptError(failure)) return failure.retryAfterMilliseconds;
 	return failure.errors?.find((error) => error.retryAfterMilliseconds != null)?.retryAfterMilliseconds;
 }
-async function* toAsyncIterator(values) {
-	for await (const value of values) yield value;
+function toAsyncIterator(values) {
+	if (Symbol.asyncIterator in values) return values[Symbol.asyncIterator]();
+	const iterator = values[Symbol.iterator]();
+	return {
+		async next() {
+			return await Promise.resolve(iterator.next());
+		},
+		async return(value) {
+			return await Promise.resolve(iterator.return?.(value) ?? {
+				done: true,
+				value
+			});
+		}
+	};
+}
+function raceWithAbort(promise, signal) {
+	if (signal.aborted) return Promise.reject(signal.reason);
+	return new Promise((resolve, reject) => {
+		const abort = () => {
+			cleanup();
+			reject(signal.reason);
+		};
+		const cleanup = () => {
+			signal.removeEventListener("abort", abort);
+		};
+		signal.addEventListener("abort", abort, { once: true });
+		promise.then((value) => {
+			cleanup();
+			resolve(value);
+		}, (error) => {
+			cleanup();
+			reject(error);
+		});
+	});
 }
 function toReceiptError(value) {
 	return isReceiptError(value) ? value : void 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@upyo/retry",
-  "version": "0.5.0-dev.195",
+  "version": "0.5.0-dev.199",
   "description": "Retry and backoff decorator transport for Upyo email library",
   "keywords": [
     "email",
@@ -54,7 +54,7 @@
   },
   "sideEffects": false,
   "peerDependencies": {
-    "@upyo/core": "0.5.0-dev.195+5d1ba2de"
+    "@upyo/core": "0.5.0-dev.199+f10145fc"
   },
   "devDependencies": {
     "tsdown": "^0.12.7",