@context-action/core 0.7.3 → 0.7.4

package/dist/index.js

@@ -49,6 +49,16 @@ async function executeSequential(context, createController) {
 		const controller = createController(registration, i);
 		try {
 			if (context.aborted) break;
+			if (registration.config.condition) try {
+				const shouldExecute = registration.config.condition(context.payload);
+				if (!shouldExecute) {
+					i++;
+					continue;
+				}
+			} catch {
+				i++;
+				continue;
+			}
 			const result = registration.handler(context.payload, controller);
 			if (registration.config.blocking) {
 				const handlerResult = result instanceof Promise ? await result : result;
@@ -73,8 +83,20 @@ async function executeSequential(context, createController) {
 			if (context.terminated) break;
 			/** Handle jump to priority AFTER handler execution */
 			if (context.jumpToPriority !== void 0) {
-				const jumpIndex = context.handlers.findIndex((handler) => handler.config.priority === context.jumpToPriority);
-				if (jumpIndex !== -1) {
+				context.jumpCount = (context.jumpCount || 0) + 1;
+				if (context.jumpCount > (context.maxJumps || 10)) {
+					console.error(`[ActionRegister] ERROR: Maximum jump limit (${context.maxJumps || 10}) exceeded. Aborting to prevent infinite loop. Check your jumpToPriority logic and conditions.`);
+					context.aborted = true;
+					context.abortReason = `Maximum jump limit exceeded (${context.jumpCount} jumps)`;
+					context.jumpToPriority = void 0;
+					break;
+				}
+				const jumpIndex = context.handlers.findIndex((handler) => (handler.config.priority || 0) <= context.jumpToPriority);
+				if (jumpIndex !== -1 && jumpIndex !== i) {
+					if (jumpIndex < i) {
+						const targetHandler = context.handlers[jumpIndex];
+						if (targetHandler && !targetHandler.config.condition) console.warn(`[ActionRegister] WARNING: Backward jumpToPriority to handler '${targetHandler.config.id || "unnamed"}' without condition. This may cause infinite loops! Consider adding a condition to prevent re-execution. Jump count: ${context.jumpCount}/${context.maxJumps || 10}`);
+					}
 					i = jumpIndex;
 					context.jumpToPriority = void 0;
 					continue;
@@ -85,7 +107,9 @@ async function executeSequential(context, createController) {
 			} else i++;
 		} catch (error) {
 			const handlerError = handleExecutionError(error, registration);
-			throw handlerError.error;
+			errors.push(handlerError);
+			if (registration.config.blocking) throw handlerError.error;
+			i++;
 		}
 	}
 	if (nonBlockingPromises.length > 0) await Promise.allSettled(nonBlockingPromises);
@@ -125,6 +149,24 @@ async function executeParallel(context, createController) {
 	const handlerPromises = runnableHandlers.map(async (registration, _index) => {
 		const controller = createController(registration, _index);
 		try {
+			if (registration.config.condition) try {
+				const shouldExecute = registration.config.condition(context.payload);
+				if (!shouldExecute) return {
+					success: true,
+					handlerId: registration.id,
+					result: void 0,
+					terminated: false,
+					skipped: true
+				};
+			} catch {
+				return {
+					success: true,
+					handlerId: registration.id,
+					result: void 0,
+					terminated: false,
+					skipped: true
+				};
+			}
 			const result = registration.handler(context.payload, controller);
 			let handlerResult;
 			if (result instanceof Promise) {
@@ -199,6 +241,26 @@ async function executeRace(context, createController) {
 	const handlerPromises = runnableHandlers.map(async (registration, _index) => {
 		const controller = createController(registration, _index);
 		try {
+			if (registration.config.condition) try {
+				const shouldExecute = registration.config.condition(context.payload);
+				if (!shouldExecute) return {
+					success: true,
+					handlerId: registration.id,
+					registration,
+					result: void 0,
+					terminated: false,
+					skipped: true
+				};
+			} catch {
+				return {
+					success: true,
+					handlerId: registration.id,
+					registration,
+					result: void 0,
+					terminated: false,
+					skipped: true
+				};
+			}
 			const result = registration.handler(context.payload, controller);
 			let handlerResult;
 			if (result instanceof Promise) {
@@ -725,6 +787,7 @@ var ActionRegister = class {
 		this.executionMode = "sequential";
 		this.actionExecutionModes = /* @__PURE__ */ new Map();
 		this.unregisterFunctions = /* @__PURE__ */ new Map();
+		this.lastRegisteredTimestamps = /* @__PURE__ */ new Map();
 		this.filterCacheDisabled = true;
 		this.handlerIdCounter = 0;
 		this.controllerPool = [];
@@ -845,8 +908,9 @@ var ActionRegister = class {
 				once: config.once ?? false,
 				debounce: config.debounce ?? void 0,
 				throttle: config.throttle ?? void 0,
-				replaceExisting: config.replaceExisting ?? false,
-				cleanup: config.cleanup
+				replaceExisting: config.replaceExisting ?? true,
+				cleanup: config.cleanup,
+				condition: config.condition
 			},
 			id: handlerId
 		};
@@ -860,18 +924,16 @@ var ActionRegister = class {
 		if (existingIndex !== -1) {
 			const existing = pipeline[existingIndex];
 			const existingUnregister = this.unregisterFunctions.get(handlerId);
-			if (config.replaceExisting) {
-				if (existingUnregister) {
-					existingUnregister();
-					this.unregisterFunctions.delete(handlerId);
-				}
-				if (existing && typeof existing.cleanup === "function") try {
-					existing.cleanup();
+			if (registration.config.replaceExisting) {
+				if (existing && existing.config.cleanup && typeof existing.config.cleanup === "function") try {
+					existing.config.cleanup();
 				} catch (cleanupError) {
 					this.log(`Cleanup error for replaced handler: ${String(action)}`, cleanupError, "warn");
 				}
+				if (existingUnregister) this.unregisterFunctions.delete(handlerId);
 				pipeline[existingIndex] = registration;
 				pipeline.sort((a, b) => b.config.priority - a.config.priority);
+				this.lastRegisteredTimestamps.set(action, /* @__PURE__ */ new Date());
 				const newUnregister = this.createUnregisterFunction(action, handlerId, registration);
 				this.unregisterFunctions.set(handlerId, newUnregister);
 				this.log(`Handler replaced: ${String(action)}`, {
@@ -901,6 +963,7 @@ var ActionRegister = class {
 		}
 		pipeline.push(registration);
 		pipeline.sort((a, b) => b.config.priority - a.config.priority);
+		this.lastRegisteredTimestamps.set(action, /* @__PURE__ */ new Date());
 		const unregister = this.createUnregisterFunction(action, handlerId, registration);
 		this.unregisterFunctions.set(handlerId, unregister);
 		this.log(`Handler registered: ${String(action)}`, {
@@ -994,6 +1057,8 @@ var ActionRegister = class {
 			abortReason: void 0,
 			currentIndex: 0,
 			jumpToPriority: void 0,
+			jumpCount: 0,
+			maxJumps: 10,
 			executionMode: currentExecutionMode,
 			results: [],
 			terminated: false,
@@ -1085,6 +1150,8 @@ var ActionRegister = class {
 			abortReason: void 0,
 			currentIndex: 0,
 			jumpToPriority: void 0,
+			jumpCount: 0,
+			maxJumps: 10,
 			executionMode: currentExecutionMode,
 			results: [],
 			terminated: false,
@@ -1271,7 +1338,11 @@ var ActionRegister = class {
 			if (autoAbortController && autoAbortOptions?.allowHandlerAbort) autoAbortController.abort(reason);
 		};
 		controller.modifyPayload = (modifier) => {
-			context.payload = modifier(context.payload);
+			try {
+				context.payload = modifier(context.payload);
+			} catch (modificationError) {
+				this.log("Payload modification error", modificationError, "warn");
+			}
 		};
 		controller.getPayload = () => context.payload;
 		controller.jumpToPriority = (priority) => {
@@ -1320,9 +1391,10 @@ var ActionRegister = class {
 		return filtered;
 	}
 	processResults(context, resultOptions) {
-		if (!resultOptions || !resultOptions.collect) return void 0;
 		const results = context.results;
 		if (context.terminated && context.terminationResult !== void 0) return context.terminationResult;
+		if (!resultOptions) return results.length > 0 ? results[results.length - 1] : void 0;
+		if (!resultOptions.collect && !resultOptions.strategy) return void 0;
 		const limitedResults = resultOptions.maxResults ? results.slice(0, resultOptions.maxResults) : results;
 		if (limitedResults.length === 0) return void 0;
 		switch (resultOptions.strategy) {
@@ -1335,7 +1407,9 @@ var ActionRegister = class {
 			case "custom":
 				if (resultOptions.merger) return resultOptions.merger(limitedResults);
 				throw new Error("Custom result strategy requires a merger function");
-			default: return limitedResults;
+			default:
+				if (resultOptions.collect) return limitedResults;
+				return limitedResults[limitedResults.length - 1];
 		}
 	}
 	async executePipeline(context, autoAbortController, autoAbortOptions) {
@@ -1372,6 +1446,10 @@ var ActionRegister = class {
 				});
 			}
 		});
+		if (pipeline.length === 0) {
+			this.pipelines.delete(action);
+			this.lastRegisteredTimestamps.delete(action);
+		}
 	}
 	/**
 	* Get the number of registered handlers for an action
@@ -1425,6 +1503,7 @@ var ActionRegister = class {
 	*/
 	clearAction(action) {
 		this.pipelines.delete(action);
+		this.lastRegisteredTimestamps.delete(action);
 	}
 	/**
 	* Remove all handlers for all actions
@@ -1435,6 +1514,7 @@ var ActionRegister = class {
 	*/
 	clearAll() {
 		this.pipelines.clear();
+		this.lastRegisteredTimestamps.clear();
 	}
 	/**
 	* Get the name of this action register
@@ -1488,7 +1568,8 @@ var ActionRegister = class {
 			handlerCount: pipeline.length,
 			totalHandlers: pipeline.length,
 			handlersByPriority,
-			executionStats
+			executionStats,
+			lastRegistered: this.lastRegisteredTimestamps.get(action)
 		};
 	}
 	/**
@@ -1569,6 +1650,10 @@ var ActionRegister = class {
 			if (index !== -1) {
 				pipeline.splice(index, 1);
 				this.unregisterFunctions.delete(handlerId);
+				if (pipeline.length === 0) {
+					this.pipelines.delete(action);
+					this.lastRegisteredTimestamps.delete(action);
+				}
 				if (registration.config.cleanup && typeof registration.config.cleanup === "function") try {
 					registration.config.cleanup();
 				} catch (cleanupError) {
@@ -1576,7 +1661,8 @@ var ActionRegister = class {
 				}
 				this.log(`Handler unregistered: ${String(action)}`, {
 					handlerId,
-					remainingHandlers: pipeline.length
+					remainingHandlers: pipeline.length,
+					actionRemoved: pipeline.length === 0
 				});
 			}
 		};
@@ -1609,15 +1695,14 @@ var ActionRegister = class {
 	* @public
 	*/
 	destroy() {
-		this.unregisterFunctions.forEach((unregister) => {
-			try {
-				unregister();
-			} catch (error) {
-				this.log("Error during unregister in destroy", error, "error");
-			}
-		});
 		this.unregisterFunctions.clear();
+		for (const [action, pipeline] of this.pipelines.entries()) for (const registration of pipeline) if (registration.config.cleanup && typeof registration.config.cleanup === "function") try {
+			registration.config.cleanup();
+		} catch (cleanupError) {
+			this.log(`Cleanup error for handler during destroy: ${String(action)}`, cleanupError, "warn");
+		}
 		this.pipelines.clear();
+		this.lastRegisteredTimestamps.clear();
 		this.actionGuard.destroy();
 		this.dispatchQueue?.clear?.();
 		this.actionExecutionModes.clear();