@context-action/core 0.7.4 → 0.7.5

package/dist/index.js

@@ -50,8 +50,7 @@ async function executeSequential(context, createController) {
 		try {
 			if (context.aborted) break;
 			if (registration.config.condition) try {
-				const shouldExecute = registration.config.condition(context.payload);
-				if (!shouldExecute) {
+				if (!registration.config.condition(context.payload)) {
 					i++;
 					continue;
 				}
@@ -75,7 +74,6 @@ async function executeSequential(context, createController) {
 						timestamp: handlerError.timestamp,
 						severity: "non-blocking"
 					});
-					return void 0;
 				});
 				nonBlockingPromises.push(promiseWithErrorHandling);
 			} else if (result !== void 0 && !context.terminated) context.results.push(result);
@@ -113,15 +111,12 @@ async function executeSequential(context, createController) {
 		}
 	}
 	if (nonBlockingPromises.length > 0) await Promise.allSettled(nonBlockingPromises);
-	if (errors.length > 0) {
-		const handlerErrors = errors.map((err) => ({
-			handlerId: err.handlerId,
-			error: err.error,
-			timestamp: err.timestamp,
-			severity: "non-blocking"
-		}));
-		context.collectedErrors = handlerErrors;
-	}
+	if (errors.length > 0) context.collectedErrors = errors.map((err) => ({
+		handlerId: err.handlerId,
+		error: err.error,
+		timestamp: err.timestamp,
+		severity: "non-blocking"
+	}));
 }
 /**
 * Execute handlers in parallel mode (all at once)
@@ -150,8 +145,7 @@ async function executeParallel(context, createController) {
 		const controller = createController(registration, _index);
 		try {
 			if (registration.config.condition) try {
-				const shouldExecute = registration.config.condition(context.payload);
-				if (!shouldExecute) return {
+				if (!registration.config.condition(context.payload)) return {
 					success: true,
 					handlerId: registration.id,
 					result: void 0,
@@ -169,10 +163,8 @@ async function executeParallel(context, createController) {
 			}
 			const result = registration.handler(context.payload, controller);
 			let handlerResult;
-			if (result instanceof Promise) {
-				const resolved = await result;
-				handlerResult = resolved;
-			} else handlerResult = result;
+			if (result instanceof Promise) handlerResult = await result;
+			else handlerResult = result;
 			/** Collect result if handler returned something and pipeline wasn't terminated */
 			if (handlerResult !== void 0 && !context.terminated) context.results.push(handlerResult);
 			return {
@@ -195,22 +187,15 @@ async function executeParallel(context, createController) {
 	const results = await Promise.allSettled(handlerPromises);
 	/** Check for any rejected blocking handlers */
 	const failures = results.filter((result, index) => {
-		if (result.status === "rejected") {
-			const registration = runnableHandlers[index];
-			return registration?.config.blocking ?? false;
-		}
+		if (result.status === "rejected") return runnableHandlers[index]?.config.blocking ?? false;
 		return false;
 	});
-	if (failures.length > 0) {
-		const firstFailure = failures[0];
-		throw firstFailure.reason;
-	}
+	if (failures.length > 0) throw failures[0].reason;
 	/** Check if any handler terminated the pipeline */
 	const terminatedResults = results.filter((result) => result.status === "fulfilled" && result.value.terminated);
 	if (terminatedResults.length > 0) {
 		context.terminated = true;
-		const firstTerminated = terminatedResults[0];
-		context.terminationResult = firstTerminated.value.result;
+		context.terminationResult = terminatedResults[0].value.result;
 	}
 }
 /**
@@ -242,8 +227,7 @@ async function executeRace(context, createController) {
 		const controller = createController(registration, _index);
 		try {
 			if (registration.config.condition) try {
-				const shouldExecute = registration.config.condition(context.payload);
-				if (!shouldExecute) return {
+				if (!registration.config.condition(context.payload)) return {
 					success: true,
 					handlerId: registration.id,
 					registration,
@@ -263,10 +247,8 @@ async function executeRace(context, createController) {
 			}
 			const result = registration.handler(context.payload, controller);
 			let handlerResult;
-			if (result instanceof Promise) {
-				const resolved = await result;
-				handlerResult = resolved;
-			} else handlerResult = result;
+			if (result instanceof Promise) handlerResult = await result;
+			else handlerResult = result;
 			return {
 				success: true,
 				handlerId: registration.id,
@@ -689,8 +671,7 @@ var OperationQueue = class {
 	* 🆕 작업 완료 신호 - 대기 중인 프로세스들에게 알림
 	*/
 	notifyOperationComplete() {
-		const resolvers = this.pendingResolvers.splice(0);
-		resolvers.forEach((resolve) => resolve());
+		this.pendingResolvers.splice(0).forEach((resolve) => resolve());
 	}
 	/**
 	* 🆕 새로운 작업 추가 신호 - processQueue에서 호출
@@ -747,8 +728,7 @@ var OperationQueue = class {
 		});
 		this.queue = [];
 		this.processingPromise = null;
-		const resolvers = this.pendingResolvers.splice(0);
-		resolvers.forEach((resolve) => resolve());
+		this.pendingResolvers.splice(0).forEach((resolve) => resolve());
 	}
 	/**
 	* 큐 크기 조회
@@ -823,8 +803,7 @@ var ActionRegister = class {
 	*/
 	register(action, handler, config = {}) {
 		const handlerId = config.id || this.generateHandlerId(action);
-		const unregisterFn = this._performRegistrationSync(action, handler, config, handlerId);
-		return unregisterFn;
+		return this._performRegistrationSync(action, handler, config, handlerId);
 	}
 	/**
 	* 🆕 Unified logging method with cached debug mode check
@@ -1041,12 +1020,10 @@ var ActionRegister = class {
 			}
 		}
 		if (debounceMs !== void 0) {
-			const shouldProceed = await this.actionGuard.debounce(actionKey, debounceMs);
-			if (!shouldProceed) return;
+			if (!await this.actionGuard.debounce(actionKey, debounceMs)) return;
 		}
 		if (throttleMs !== void 0) {
-			const shouldProceed = this.actionGuard.throttle(actionKey, throttleMs);
-			if (!shouldProceed) return;
+			if (!this.actionGuard.throttle(actionKey, throttleMs)) return;
 		}
 		const currentExecutionMode = options?.executionMode || this.actionExecutionModes.get(action) || this.executionMode;
 		const context = {
@@ -1177,8 +1154,7 @@ var ActionRegister = class {
 		let errors = [];
 		try {
 			await this.executePipeline(context, autoAbortController, options?.autoAbort);
-			const contextWithErrors = context;
-			errors = contextWithErrors.collectedErrors || [];
+			errors = context.collectedErrors || [];
 			const executedCount = Math.min(context.currentIndex + (context.aborted ? 0 : 1), filteredHandlers.length);
 			for (let i = 0; i < executedCount; i++) {
 				const handler = filteredHandlers[i];
@@ -1187,8 +1163,7 @@ var ActionRegister = class {
 				if (handlerResult) handlerResult.executed = true;
 			}
 		} catch (error) {
-			const contextWithErrors = context;
-			errors = contextWithErrors.collectedErrors || [];
+			errors = context.collectedErrors || [];
 			executionError = error instanceof Error ? error : new Error(String(error));
 			errors.push({
 				handlerId: "pipeline",
@@ -1264,8 +1239,7 @@ var ActionRegister = class {
 			}
 		}
 		if (debounceMs !== void 0) {
-			const shouldProceed = await this.actionGuard.debounce(actionKey, debounceMs);
-			if (!shouldProceed) return {
+			if (!await this.actionGuard.debounce(actionKey, debounceMs)) return {
 				success: false,
 				aborted: true,
 				abortReason: "Debounced execution",
@@ -1287,8 +1261,7 @@ var ActionRegister = class {
 			};
 		}
 		if (throttleMs !== void 0) {
-			const shouldProceed = this.actionGuard.throttle(actionKey, throttleMs);
-			if (!shouldProceed) return {
+			if (!this.actionGuard.throttle(actionKey, throttleMs)) return {
 				success: false,
 				aborted: true,
 				abortReason: "Throttled execution",
@@ -1376,7 +1349,7 @@ var ActionRegister = class {
 		if (!filterOptions) return handlers;
 		const handlerIdSet = filterOptions.handlerIds ? new Set(filterOptions.handlerIds) : null;
 		const excludeIdSet = filterOptions.excludeHandlerIds ? new Set(filterOptions.excludeHandlerIds) : null;
-		const filtered = handlers.filter((registration) => {
+		return handlers.filter((registration) => {
 			const config = registration.config;
 			if (handlerIdSet && !handlerIdSet.has(config.id)) return false;
 			if (excludeIdSet && excludeIdSet.has(config.id)) return false;
@@ -1388,15 +1361,14 @@ var ActionRegister = class {
 			if (filterOptions.custom && !filterOptions.custom(config)) return false;
 			return true;
 		});
-		return filtered;
 	}
 	processResults(context, resultOptions) {
 		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;
+		if (!resultOptions.collect && !resultOptions.strategy) return;
 		const limitedResults = resultOptions.maxResults ? results.slice(0, resultOptions.maxResults) : results;
-		if (limitedResults.length === 0) return void 0;
+		if (limitedResults.length === 0) return;
 		switch (resultOptions.strategy) {
 			case "first": return limitedResults[0];
 			case "last": return limitedResults[limitedResults.length - 1];
@@ -1562,13 +1534,12 @@ var ActionRegister = class {
 			priority,
 			handlers: handlers.map((h) => ({ id: h.config.id }))
 		}));
-		const executionStats = void 0;
 		return {
 			action,
 			handlerCount: pipeline.length,
 			totalHandlers: pipeline.length,
 			handlersByPriority,
-			executionStats,
+			executionStats: void 0,
 			lastRegistered: this.lastRegisteredTimestamps.get(action)
 		};
 	}