@upyo/pool 0.5.0-dev.158 → 0.5.0-dev.168

package/README.md

@@ -97,7 +97,7 @@ const transport = new PoolTransport({
     { transport: backupTransport, priority: 50 },
     { transport: emergencyTransport, priority: 10 },
   ],
-  maxRetries: 3, // Try up to 3 transports
+  maxRetries: 2, // Retry up to 2 other transports
 });
 
 // Always tries primary first, only uses backup if primary fails
@@ -219,13 +219,13 @@ Configuration
 
 ### `PoolConfig`
 
-| Property            | Type                                                                        | Required | Default              | Description                                                       |
-| ------------------- | --------------------------------------------------------------------------- | -------- | -------------------- | ----------------------------------------------------------------- |
-| `strategy`          | `"round-robin" \| "weighted" \| "priority" \| "selector-based" \| Strategy` | Yes      |                      | The strategy for selecting transports                             |
-| `transports`        | `TransportEntry[]`                                                          | Yes      |                      | Array of transport configurations                                 |
-| `maxRetries`        | `number`                                                                    | No       | Number of transports | Maximum retry attempts on failure                                 |
-| `timeout`           | `number`                                                                    | No       |                      | Timeout in milliseconds for each send attempt                     |
-| `continueOnSuccess` | `boolean`                                                                   | No       | `false`              | Continue trying transports after success (selector strategy only) |
+| Property            | Type                                                                        | Required | Default                           | Description                                                       |
+| ------------------- | --------------------------------------------------------------------------- | -------- | --------------------------------- | ----------------------------------------------------------------- |
+| `strategy`          | `"round-robin" \| "weighted" \| "priority" \| "selector-based" \| Strategy` | Yes      |                                   | The strategy for selecting transports                             |
+| `transports`        | `TransportEntry[]`                                                          | Yes      |                                   | Array of transport configurations                                 |
+| `maxRetries`        | `number`                                                                    | No       | Enough to try each transport once | Maximum retry attempts after the initial attempt                  |
+| `timeout`           | `number`                                                                    | No       |                                   | Timeout in milliseconds for each send attempt                     |
+| `continueOnSuccess` | `boolean`                                                                   | No       | `false`                           | Continue trying transports after success (selector strategy only) |
 
 ### `TransportEntry`
 

package/dist/index.cjs

@@ -50,7 +50,7 @@ function createPoolConfig(config) {
 	return {
 		strategy: config.strategy,
 		transports: resolvedTransports,
-		maxRetries: config.maxRetries ?? enabledTransports.length,
+		maxRetries: config.maxRetries ?? enabledTransports.length - 1,
 		timeout: config.timeout,
 		continueOnSuccess: config.continueOnSuccess ?? false
 	};
@@ -328,7 +328,7 @@ var PoolTransport = class {
 		const attemptedIndices = /* @__PURE__ */ new Set();
 		const errorMessages = [];
 		const errors = [];
-		for (let attempt = 0; attempt < this.config.maxRetries; attempt++) {
+		for (let attempt = 0; attempt <= this.config.maxRetries; attempt++) {
 			if (options?.signal?.aborted) throw new DOMException("The operation was aborted.", "AbortError");
 			const selection = this.strategy.select(message, this.config.transports, attemptedIndices);
 			if (!selection) break;
@@ -340,7 +340,7 @@ var PoolTransport = class {
 				try {
 					receipt = await selection.entry.transport.send(message, sendOptions.options);
 				} catch (error) {
-					if (isAbortError(error) && sendOptions.abortedByCaller()) {
+					if (sendOptions.abortedByCaller() && isCallerAbort(error, options?.signal)) {
 						abortedByCaller = true;
 						throw error;
 					}
@@ -352,7 +352,7 @@ var PoolTransport = class {
 				errorMessages.push(...receipt.errorMessages);
 				errors.push(...getReceiptErrors(receipt, selection.entry.transport.id));
 			} catch (error) {
-				if (isAbortError(error) && abortedByCaller) throw error;
+				if (abortedByCaller && isCallerAbort(error, options?.signal)) throw error;
 				const thrownErrors = getThrownReceiptErrors(error, selection.entry.transport.id);
 				if (thrownErrors.length > 0) {
 					errorMessages.push(...thrownErrors.map((item) => item.message));
@@ -433,39 +433,76 @@ var PoolTransport = class {
 			abortedByCaller: () => options?.signal?.aborted ?? false,
 			cleanup: () => {}
 		};
-		const controller = new AbortController();
+		const timeoutController = new AbortController();
 		let abortSource;
 		const timeoutId = setTimeout(() => {
 			abortSource ??= "timeout";
-			controller.abort();
+			timeoutController.abort(createAbortError());
 		}, this.config.timeout);
-		let cleanup = () => clearTimeout(timeoutId);
+		let cleanupAbortSource = () => {};
+		let signal = timeoutController.signal;
+		let cleanupCombinedSignal = () => {};
 		if (options?.signal) {
-			const abort = () => {
+			const markCallerAbort = () => {
 				abortSource ??= "caller";
 				clearTimeout(timeoutId);
-				controller.abort();
-			};
-			if (options.signal.aborted) abort();
-			else options.signal.addEventListener("abort", abort, { once: true });
-			cleanup = () => {
-				clearTimeout(timeoutId);
-				options.signal?.removeEventListener("abort", abort);
 			};
+			if (options.signal.aborted) markCallerAbort();
+			else options.signal.addEventListener("abort", markCallerAbort, { once: true });
+			cleanupAbortSource = () => options.signal?.removeEventListener("abort", markCallerAbort);
+			const combinedSignal = combineSignals(timeoutController.signal, options.signal);
+			signal = combinedSignal.signal;
+			cleanupCombinedSignal = combinedSignal.cleanup;
 		}
-		controller.signal.addEventListener("abort", () => {
+		timeoutController.signal.addEventListener("abort", () => {
 			clearTimeout(timeoutId);
 		});
 		return {
 			options: {
 				...options,
-				signal: controller.signal
+				signal
 			},
 			abortedByCaller: () => abortSource === "caller",
-			cleanup
+			cleanup: () => {
+				clearTimeout(timeoutId);
+				cleanupAbortSource();
+				cleanupCombinedSignal();
+			}
 		};
 	}
 };
+function combineSignals(timeoutSignal, externalSignal) {
+	if (typeof AbortSignal.any === "function") return {
+		signal: AbortSignal.any([timeoutSignal, externalSignal]),
+		cleanup: () => {}
+	};
+	const controller = new AbortController();
+	const abort = (signal) => {
+		controller.abort(getAbortReason(signal));
+	};
+	const abortTimeout = () => abort(timeoutSignal);
+	const abortExternal = () => abort(externalSignal);
+	timeoutSignal.addEventListener("abort", abortTimeout, { once: true });
+	externalSignal.addEventListener("abort", abortExternal, { once: true });
+	if (timeoutSignal.aborted) abortTimeout();
+	else if (externalSignal.aborted) abortExternal();
+	return {
+		signal: controller.signal,
+		cleanup: () => {
+			timeoutSignal.removeEventListener("abort", abortTimeout);
+			externalSignal.removeEventListener("abort", abortExternal);
+		}
+	};
+}
+function getAbortReason(signal) {
+	return signal.reason ?? createAbortError();
+}
+function createAbortError() {
+	return new DOMException("The operation was aborted.", "AbortError");
+}
+function isCallerAbort(error, signal) {
+	return isAbortError(error) || error === signal?.reason;
+}
 function getReceiptErrors(receipt, provider) {
 	if (receipt.errors != null && receipt.errors.length > 0) return receipt.errors.map((error) => error.provider == null ? {
 		...error,

package/dist/index.d.cts

@@ -95,8 +95,9 @@ interface PoolConfig<TProviderId extends string = string> {
    */
   readonly transports: readonly TransportEntry<TProviderId>[];
   /**
-   * Maximum number of retry attempts when a transport fails.
-   * Set to 0 to disable retries. Defaults to the number of transports.
+   * Maximum number of retry attempts after the initial send attempt fails.
+   * Set to 0 to disable retries. Defaults to enough retries to try every
+   * enabled transport once.
    */
   readonly maxRetries?: number;
   /**

package/dist/index.d.ts

@@ -95,8 +95,9 @@ interface PoolConfig<TProviderId extends string = string> {
    */
   readonly transports: readonly TransportEntry<TProviderId>[];
   /**
-   * Maximum number of retry attempts when a transport fails.
-   * Set to 0 to disable retries. Defaults to the number of transports.
+   * Maximum number of retry attempts after the initial send attempt fails.
+   * Set to 0 to disable retries. Defaults to enough retries to try every
+   * enabled transport once.
    */
   readonly maxRetries?: number;
   /**

package/dist/index.js

@@ -27,7 +27,7 @@ function createPoolConfig(config) {
 	return {
 		strategy: config.strategy,
 		transports: resolvedTransports,
-		maxRetries: config.maxRetries ?? enabledTransports.length,
+		maxRetries: config.maxRetries ?? enabledTransports.length - 1,
 		timeout: config.timeout,
 		continueOnSuccess: config.continueOnSuccess ?? false
 	};
@@ -305,7 +305,7 @@ var PoolTransport = class {
 		const attemptedIndices = /* @__PURE__ */ new Set();
 		const errorMessages = [];
 		const errors = [];
-		for (let attempt = 0; attempt < this.config.maxRetries; attempt++) {
+		for (let attempt = 0; attempt <= this.config.maxRetries; attempt++) {
 			if (options?.signal?.aborted) throw new DOMException("The operation was aborted.", "AbortError");
 			const selection = this.strategy.select(message, this.config.transports, attemptedIndices);
 			if (!selection) break;
@@ -317,7 +317,7 @@ var PoolTransport = class {
 				try {
 					receipt = await selection.entry.transport.send(message, sendOptions.options);
 				} catch (error) {
-					if (isAbortError(error) && sendOptions.abortedByCaller()) {
+					if (sendOptions.abortedByCaller() && isCallerAbort(error, options?.signal)) {
 						abortedByCaller = true;
 						throw error;
 					}
@@ -329,7 +329,7 @@ var PoolTransport = class {
 				errorMessages.push(...receipt.errorMessages);
 				errors.push(...getReceiptErrors(receipt, selection.entry.transport.id));
 			} catch (error) {
-				if (isAbortError(error) && abortedByCaller) throw error;
+				if (abortedByCaller && isCallerAbort(error, options?.signal)) throw error;
 				const thrownErrors = getThrownReceiptErrors(error, selection.entry.transport.id);
 				if (thrownErrors.length > 0) {
 					errorMessages.push(...thrownErrors.map((item) => item.message));
@@ -410,39 +410,76 @@ var PoolTransport = class {
 			abortedByCaller: () => options?.signal?.aborted ?? false,
 			cleanup: () => {}
 		};
-		const controller = new AbortController();
+		const timeoutController = new AbortController();
 		let abortSource;
 		const timeoutId = setTimeout(() => {
 			abortSource ??= "timeout";
-			controller.abort();
+			timeoutController.abort(createAbortError());
 		}, this.config.timeout);
-		let cleanup = () => clearTimeout(timeoutId);
+		let cleanupAbortSource = () => {};
+		let signal = timeoutController.signal;
+		let cleanupCombinedSignal = () => {};
 		if (options?.signal) {
-			const abort = () => {
+			const markCallerAbort = () => {
 				abortSource ??= "caller";
 				clearTimeout(timeoutId);
-				controller.abort();
-			};
-			if (options.signal.aborted) abort();
-			else options.signal.addEventListener("abort", abort, { once: true });
-			cleanup = () => {
-				clearTimeout(timeoutId);
-				options.signal?.removeEventListener("abort", abort);
 			};
+			if (options.signal.aborted) markCallerAbort();
+			else options.signal.addEventListener("abort", markCallerAbort, { once: true });
+			cleanupAbortSource = () => options.signal?.removeEventListener("abort", markCallerAbort);
+			const combinedSignal = combineSignals(timeoutController.signal, options.signal);
+			signal = combinedSignal.signal;
+			cleanupCombinedSignal = combinedSignal.cleanup;
 		}
-		controller.signal.addEventListener("abort", () => {
+		timeoutController.signal.addEventListener("abort", () => {
 			clearTimeout(timeoutId);
 		});
 		return {
 			options: {
 				...options,
-				signal: controller.signal
+				signal
 			},
 			abortedByCaller: () => abortSource === "caller",
-			cleanup
+			cleanup: () => {
+				clearTimeout(timeoutId);
+				cleanupAbortSource();
+				cleanupCombinedSignal();
+			}
 		};
 	}
 };
+function combineSignals(timeoutSignal, externalSignal) {
+	if (typeof AbortSignal.any === "function") return {
+		signal: AbortSignal.any([timeoutSignal, externalSignal]),
+		cleanup: () => {}
+	};
+	const controller = new AbortController();
+	const abort = (signal) => {
+		controller.abort(getAbortReason(signal));
+	};
+	const abortTimeout = () => abort(timeoutSignal);
+	const abortExternal = () => abort(externalSignal);
+	timeoutSignal.addEventListener("abort", abortTimeout, { once: true });
+	externalSignal.addEventListener("abort", abortExternal, { once: true });
+	if (timeoutSignal.aborted) abortTimeout();
+	else if (externalSignal.aborted) abortExternal();
+	return {
+		signal: controller.signal,
+		cleanup: () => {
+			timeoutSignal.removeEventListener("abort", abortTimeout);
+			externalSignal.removeEventListener("abort", abortExternal);
+		}
+	};
+}
+function getAbortReason(signal) {
+	return signal.reason ?? createAbortError();
+}
+function createAbortError() {
+	return new DOMException("The operation was aborted.", "AbortError");
+}
+function isCallerAbort(error, signal) {
+	return isAbortError(error) || error === signal?.reason;
+}
 function getReceiptErrors(receipt, provider) {
 	if (receipt.errors != null && receipt.errors.length > 0) return receipt.errors.map((error) => error.provider == null ? {
 		...error,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@upyo/pool",
-  "version": "0.5.0-dev.158",
+  "version": "0.5.0-dev.168",
   "description": "Pool transport for Upyo email library—provides load balancing and failover for multiple email providers",
   "keywords": [
     "email",
@@ -56,12 +56,12 @@
   },
   "sideEffects": false,
   "peerDependencies": {
-    "@upyo/core": "0.5.0-dev.158+468b767c"
+    "@upyo/core": "0.5.0-dev.168+1e808a3a"
   },
   "devDependencies": {
     "tsdown": "^0.12.7",
     "typescript": "5.8.3",
-    "@upyo/mock": "0.5.0-dev.158+468b767c"
+    "@upyo/mock": "0.5.0-dev.168+1e808a3a"
   },
   "scripts": {
     "prepublish": "mise run --no-deps :build"