@robota-sdk/agent-core 3.0.0-beta.2 → 3.0.0-beta.21

package/dist/services/tool-execution-service.js

@@ -8,7 +8,7 @@ export const TOOL_EVENTS = {
     CALL_START: 'call_start',
     CALL_COMPLETE: 'call_complete',
     CALL_ERROR: 'call_error',
-    CALL_RESPONSE_READY: 'call_response_ready'
+    CALL_RESPONSE_READY: 'call_response_ready',
 };
 export const TOOL_EVENT_PREFIX = 'tool';
 /**
@@ -22,6 +22,22 @@ export class ToolExecutionService {
         this.tools = tools;
         this.logger = logger;
     }
+    requireExecutionRequestFields(request) {
+        if (!request.executionId) {
+            throw new ValidationError('[STRICT-POLICY][EMITTER-CONTRACT] Tool execution request missing executionId');
+        }
+        if (!request.ownerType) {
+            throw new ValidationError(`[STRICT-POLICY][EMITTER-CONTRACT] Tool execution request missing ownerType: executionId=${request.executionId}`);
+        }
+        if (!request.ownerId) {
+            throw new ValidationError(`[STRICT-POLICY][EMITTER-CONTRACT] Tool execution request missing ownerId: executionId=${request.executionId}`);
+        }
+        return {
+            executionId: request.executionId,
+            ownerType: request.ownerType,
+            ownerId: request.ownerId,
+        };
+    }
     /**
      * Execute a single tool
      * @param toolName - Name of the tool to execute
@@ -40,7 +56,7 @@ export class ToolExecutionService {
                 const startEvent = {
                     timestamp: new Date(),
                     toolName,
-                    parameters
+                    parameters,
                 };
                 eventService.emit(TOOL_EVENTS.CALL_START, startEvent);
             }
@@ -52,7 +68,7 @@ export class ToolExecutionService {
                 ...restContext,
                 toolName,
                 parameters,
-                executionId: context.executionId
+                executionId: context.executionId,
             };
             // Execute the tool with full context
             // Context already contains all necessary information including tool call ID
@@ -62,7 +78,7 @@ export class ToolExecutionService {
                 const completeEvent = {
                     timestamp: new Date(),
                     toolName,
-                    result: result
+                    result: result,
                 };
                 eventService.emit(TOOL_EVENTS.CALL_COMPLETE, completeEvent);
                 eventService.emit(TOOL_EVENTS.CALL_RESPONSE_READY, completeEvent);
@@ -71,7 +87,7 @@ export class ToolExecutionService {
                 success: true,
                 result: result,
                 toolName,
-                executionId: executionContext.executionId
+                executionId: executionContext.executionId,
             };
         }
         catch (error) {
@@ -82,7 +98,7 @@ export class ToolExecutionService {
                 const errorEvent = {
                     timestamp: new Date(),
                     toolName,
-                    error: toolError.message
+                    error: toolError.message,
                 };
                 eventService.emit(TOOL_EVENTS.CALL_ERROR, errorEvent);
             }
@@ -90,7 +106,7 @@ export class ToolExecutionService {
                 success: false,
                 error: toolError.message,
                 toolName,
-                executionId: context?.executionId
+                executionId: context?.executionId,
             };
         }
     }
@@ -101,15 +117,24 @@ export class ToolExecutionService {
      * @returns Array of tool execution requests
      */
     createExecutionRequestsWithContext(toolCalls, context) {
-        return toolCalls.map(toolCall => ({
-            toolName: toolCall.function.name,
-            parameters: JSON.parse(toolCall.function.arguments),
-            executionId: toolCall.id,
-            ownerType: 'tool',
-            ownerId: toolCall.id,
-            ownerPath: [...context.ownerPathBase, { type: 'tool', id: toolCall.id }],
-            metadata: context.metadataFactory ? context.metadataFactory(toolCall) : undefined
-        }));
+        return toolCalls.map((toolCall) => {
+            let parsedParameters;
+            try {
+                parsedParameters = JSON.parse(toolCall.function.arguments);
+            }
+            catch {
+                throw new ValidationError(`Failed to parse arguments for tool "${toolCall.function.name}" (call ${toolCall.id}): invalid JSON`);
+            }
+            return {
+                toolName: toolCall.function.name,
+                parameters: parsedParameters,
+                executionId: toolCall.id,
+                ownerType: 'tool',
+                ownerId: toolCall.id,
+                ownerPath: [...context.ownerPathBase, { type: 'tool', id: toolCall.id }],
+                metadata: context.metadataFactory ? context.metadataFactory(toolCall) : undefined,
+            };
+        });
     }
     /**
      * Execute tools from batch context (for ExecutionService compatibility)
@@ -121,48 +146,68 @@ export class ToolExecutionService {
         const results = [];
         const errors = [];
         if (batchContext.mode === 'parallel') {
-            const promises = batchContext.requests.map(request => this.executeTool(request.toolName, request.parameters, {
-                toolName: request.toolName,
-                parameters: request.parameters,
-                executionId: request.executionId,
-                ownerType: request.ownerType || 'tool',
-                ownerId: request.ownerId || request.executionId,
-                ownerPath: request.ownerPath,
-                metadata: request.metadata,
-                eventService: request.eventService,
-                baseEventService: request.baseEventService
-            }).catch(error => ({
-                toolName: request.toolName || 'unknown',
-                result: null,
-                success: false,
-                error: error instanceof Error ? error.message : String(error),
-                executionId: request.executionId || 'unknown'
-            })));
-            const allResults = await Promise.all(promises);
+            const promises = batchContext.requests.map((request) => (() => {
+                const required = this.requireExecutionRequestFields(request);
+                return this.executeTool(request.toolName, request.parameters, {
+                    toolName: request.toolName,
+                    parameters: request.parameters,
+                    executionId: required.executionId,
+                    ownerType: required.ownerType,
+                    ownerId: required.ownerId,
+                    ownerPath: request.ownerPath,
+                    metadata: request.metadata,
+                    eventService: request.eventService,
+                    baseEventService: request.baseEventService,
+                });
+            })());
+            const allResults = await Promise.allSettled(promises);
             // Preserve a result entry for every request (SSOT for toolCallId → result mapping).
             // ExecutionService depends on `executionId` to add tool messages in the original call order.
-            allResults.forEach(result => {
-                results.push(result);
-                if (!result.success) {
-                    const err = new Error(`Tool execution failed: toolName=${String(result.toolName)} executionId=${String(result.executionId)} error=${String(result.error || 'Unknown error')}`);
-                    errors.push(err);
+            allResults.forEach((settledResult, index) => {
+                const request = batchContext.requests[index];
+                if (!request) {
+                    return;
                 }
+                if (settledResult.status === 'fulfilled') {
+                    const result = settledResult.value;
+                    results.push(result);
+                    if (!result.success) {
+                        const err = new Error(`Tool execution failed: toolName=${String(result.toolName)} executionId=${String(result.executionId)} error=${String(result.error || 'Unknown error')}`);
+                        errors.push(err);
+                    }
+                    return;
+                }
+                const err = settledResult.reason instanceof Error
+                    ? settledResult.reason
+                    : new Error(String(settledResult.reason));
+                errors.push(err);
+                results.push({
+                    toolName: request.toolName,
+                    result: null,
+                    success: false,
+                    error: err.message,
+                    executionId: request.executionId,
+                });
             });
+            if (errors.length > 0 && !batchContext.continueOnError) {
+                throw errors[0];
+            }
         }
         else {
             // Sequential execution
             for (const request of batchContext.requests) {
                 try {
+                    const required = this.requireExecutionRequestFields(request);
                     const result = await this.executeTool(request.toolName, request.parameters, {
                         toolName: request.toolName,
                         parameters: request.parameters,
-                        executionId: request.executionId,
-                        ownerType: request.ownerType || 'tool',
-                        ownerId: request.ownerId || request.executionId,
+                        executionId: required.executionId,
+                        ownerType: required.ownerType,
+                        ownerId: required.ownerId,
                         ownerPath: request.ownerPath,
                         metadata: request.metadata,
                         eventService: request.eventService,
-                        baseEventService: request.baseEventService
+                        baseEventService: request.baseEventService,
                     });
                     results.push(result);
                     if (!result.success) {
@@ -173,9 +218,7 @@ export class ToolExecutionService {
                     }
                 }
                 catch (error) {
-                    const err = error instanceof Error
-                        ? error
-                        : new Error(String(error));
+                    const err = error instanceof Error ? error : new Error(String(error));
                     errors.push(err);
                     if (!batchContext.continueOnError) {
                         break;

package/dist/services/tool-execution-service.js.map

@@ -1 +1 @@
-{"version":3,"file":"tool-execution-service.js","sourceRoot":"","sources":["../../src/services/tool-execution-service.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,YAAY,EAAgB,MAAM,iBAAiB,CAAC;AAC7D,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAC;AAElD;;;GAGG;AACH,MAAM,CAAC,MAAM,WAAW,GAAG;IACvB,UAAU,EAAE,YAAY;IACxB,aAAa,EAAE,eAAe;IAC9B,UAAU,EAAE,YAAY;IACxB,mBAAmB,EAAE,qBAAqB;CACpC,CAAC;AAEX,MAAM,CAAC,MAAM,iBAAiB,GAAG,MAAe,CAAC;AAWjD;;;GAGG;AACH,MAAM,OAAO,oBAAoB;IACrB,KAAK,CAAe;IACpB,MAAM,CAAU;IAExB,YAAY,KAAmB,EAAE,SAAkB,YAAY;QAC3D,IAAI,CAAC,KAAK,GAAG,KAAK,CAAC;QACnB,IAAI,CAAC,MAAM,GAAG,MAAM,CAAC;IACzB,CAAC;IAED;;;;;;OAMG;IACH,KAAK,CAAC,WAAW,CACb,QAAgB,EAChB,UAA2B,EAC3B,OAA+B;QAE/B,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,mBAAmB,QAAQ,EAAE,CAAC,CAAC;QAEjD,IAAI,CAAC;YACD,IAAI,CAAC,OAAO,EAAE,WAAW,EAAE,CAAC;gBACxB,MAAM,IAAI,eAAe,CAAC,gFAAgF,CAAC,CAAC;YAChH,CAAC;YAED,MAAM,YAAY,GAAG,OAAO,CAAC,YAAY,CAAC;YAC1C,IAAI,YAAY,EAAE,CAAC;gBACf,MAAM,UAAU,GAAmB;oBAC/B,SAAS,EAAE,IAAI,IAAI,EAAE;oBACrB,QAAQ;oBACR,UAAU;iBACb,CAAC;gBACF,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;YAC1D,CAAC;YAED,wEAAwE;YACxE,MAAM,EACF,QAAQ,EAAE,SAAS,EACnB,UAAU,EAAE,WAAW,EACvB,GAAG,WAAW,EACjB,GAAG,OAAO,CAAC;YACZ,KAAK,SAAS,CAAC;YACf,KAAK,WAAW,CAAC;YAEjB,MAAM,gBAAgB,GAA0B;gBAC5C,GAAG,WAAW;gBACd,QAAQ;gBACR,UAAU;gBACV,WAAW,EAAE,OAAO,CAAC,WAAW;aACnC,CAAC;YAEF,qCAAqC;YACrC,4EAA4E;YAC5E,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,QAAQ,EAAE,UAAU,EAAE,gBAAgB,CAAC,CAAC;YAEpF,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,6BAA6B,QAAQ,EAAE,CAAC,CAAC;YAE3D,IAAI,YAAY,EAAE,CAAC;gBACf,MAAM,aAAa,GAAmB;oBAClC,SAAS,EAAE,IAAI,IAAI,EAAE;oBACrB,QAAQ;oBACR,MAAM,EAAE,MAAM;iBACjB,CAAC;gBACF,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,aAAa,EAAE,aAAa,CAAC,CAAC;gBAC5D,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,mBAAmB,EAAE,aAAa,CAAC,CAAC;YACtE,CAAC;YAED,OAAO;gBACH,OAAO,EAAE,IAAI;gBACb,MAAM,EAAE,MAAM;gBACd,QAAQ;gBACR,WAAW,EAAE,gBAAgB,CAAC,WAAY;aAC7C,CAAC;QACN,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACb,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,0BAA0B,QAAQ,EAAE,CAAC,CAAC;YAExD,MAAM,SAAS,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;YAE5E,MAAM,YAAY,GAAG,OAAO,EAAE,YAAY,CAAC;YAC3C,IAAI,YAAY,IAAI,OAAO,EAAE,WAAW,EAAE,CAAC;gBACvC,MAAM,UAAU,GAAmB;oBAC/B,SAAS,EAAE,IAAI,IAAI,EAAE;oBACrB,QAAQ;oBACR,KAAK,EAAE,SAAS,CAAC,OAAO;iBAC3B,CAAC;gBACF,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;YAC1D,CAAC;YAED,OAAO;gBACH,OAAO,EAAE,KAAK;gBACd,KAAK,EAAE,SAAS,CAAC,OAAO;gBACxB,QAAQ;gBACR,WAAW,EAAE,OAAO,EAAE,WAAW;aACpC,CAAC;QACN,CAAC;IACL,CAAC;IAED;;;;;OAKG;IACH,kCAAkC,CAC9B,SAA+E,EAC/E,OAGC;QAED,OAAO,SAAS,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC;YAC9B,QAAQ,EAAE,QAAQ,CAAC,QAAQ,CAAC,IAAI;YAChC,UAAU,EAAE,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,SAAS,CAAC;YACnD,WAAW,EAAE,QAAQ,CAAC,EAAE;YACxB,SAAS,EAAE,MAAM;YACjB,OAAO,EAAE,QAAQ,CAAC,EAAE;YACpB,SAAS,EAAE,CAAC,GAAG,OAAO,CAAC,aAAa,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,QAAQ,CAAC,EAAE,EAAE,CAAC;YACxE,QAAQ,EAAE,OAAO,CAAC,eAAe,CAAC,CAAC,CAAC,OAAO,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,SAAS;SACpF,CAAC,CAAC,CAAC;IACR,CAAC;IAED;;;;OAIG;IACH,KAAK,CAAC,YAAY,CAAC,YAAwC;QACvD,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,aAAa,YAAY,CAAC,QAAQ,CAAC,MAAM,aAAa,YAAY,CAAC,IAAI,OAAO,CAAC,CAAC;QAElG,MAAM,OAAO,GAA2B,EAAE,CAAC;QAC3C,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,IAAI,YAAY,CAAC,IAAI,KAAK,UAAU,EAAE,CAAC;YACnC,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CACjD,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,UAAU,EAAE;gBACnD,QAAQ,EAAE,OAAO,CAAC,QAAQ;gBAC1B,UAAU,EAAE,OAAO,CAAC,UAAU;gBAC9B,WAAW,EAAE,OAAO,CAAC,WAAW;gBAChC,SAAS,EAAE,OAAO,CAAC,SAAS,IAAI,MAAM;gBACtC,OAAO,EAAE,OAAO,CAAC,OAAO,IAAI,OAAO,CAAC,WAAW;gBAC/C,SAAS,EAAE,OAAO,CAAC,SAAS;gBAC5B,QAAQ,EAAE,OAAO,CAAC,QAAQ;gBAC1B,YAAY,EAAE,OAAO,CAAC,YAAY;gBAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;aAC7C,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;gBACf,QAAQ,EAAE,OAAO,CAAC,QAAQ,IAAI,SAAS;gBACvC,MAAM,EAAE,IAAI;gBACZ,OAAO,EAAE,KAAK;gBACd,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;gBAC7D,WAAW,EAAE,OAAO,CAAC,WAAW,IAAI,SAAS;aAChD,CAAC,CAAC,CACN,CAAC;YACF,MAAM,UAAU,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;YAE/C,oFAAoF;YACpF,6FAA6F;YAC7F,UAAU,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE;gBACxB,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;gBACrB,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;oBAClB,MAAM,GAAG,GAAG,IAAI,KAAK,CACjB,mCAAmC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,UAAU,MAAM,CAAC,MAAM,CAAC,KAAK,IAAI,eAAe,CAAC,EAAE,CAC1J,CAAC;oBACF,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;gBACrB,CAAC;YACL,CAAC,CAAC,CAAC;QACP,CAAC;aAAM,CAAC;YACJ,uBAAuB;YACvB,KAAK,MAAM,OAAO,IAAI,YAAY,CAAC,QAAQ,EAAE,CAAC;gBAC1C,IAAI,CAAC;oBACD,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,UAAU,EAAE;wBACxE,QAAQ,EAAE,OAAO,CAAC,QAAQ;wBAC1B,UAAU,EAAE,OAAO,CAAC,UAAU;wBAC9B,WAAW,EAAE,OAAO,CAAC,WAAW;wBAChC,SAAS,EAAE,OAAO,CAAC,SAAS,IAAI,MAAM;wBACtC,OAAO,EAAE,OAAO,CAAC,OAAO,IAAI,OAAO,CAAC,WAAW;wBAC/C,SAAS,EAAE,OAAO,CAAC,SAAS;wBAC5B,QAAQ,EAAE,OAAO,CAAC,QAAQ;wBAC1B,YAAY,EAAE,OAAO,CAAC,YAAY;wBAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;qBAC7C,CAAC,CAAC;oBACH,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;oBACrB,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;wBAClB,MAAM,CAAC,IAAI,CACP,IAAI,KAAK,CACL,mCAAmC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,UAAU,MAAM,CAAC,MAAM,CAAC,KAAK,IAAI,eAAe,CAAC,EAAE,CAC1J,CACJ,CAAC;oBACN,CAAC;oBAED,IAAI,CAAC,MAAM,CAAC,OAAO,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE,CAAC;wBACnD,MAAM;oBACV,CAAC;gBACL,CAAC;gBAAC,OAAO,KAAK,EAAE,CAAC;oBACb,MAAM,GAAG,GAAG,KAAK,YAAY,KAAK;wBAC9B,CAAC,CAAC,KAAK;wBACP,CAAC,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;oBAC/B,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;oBAEjB,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE,CAAC;wBAChC,MAAM;oBACV,CAAC;gBACL,CAAC;YACL,CAAC;QACL,CAAC;QAED,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,CAAC;IAC/B,CAAC;CACJ"}
+{"version":3,"file":"tool-execution-service.js","sourceRoot":"","sources":["../../src/services/tool-execution-service.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,YAAY,EAAgB,MAAM,iBAAiB,CAAC;AAC7D,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAC;AAElD;;;GAGG;AACH,MAAM,CAAC,MAAM,WAAW,GAAG;IACzB,UAAU,EAAE,YAAY;IACxB,aAAa,EAAE,eAAe;IAC9B,UAAU,EAAE,YAAY;IACxB,mBAAmB,EAAE,qBAAqB;CAClC,CAAC;AAEX,MAAM,CAAC,MAAM,iBAAiB,GAAG,MAAe,CAAC;AAiBjD;;;GAGG;AACH,MAAM,OAAO,oBAAoB;IACvB,KAAK,CAAe;IACpB,MAAM,CAAU;IAExB,YAAY,KAAmB,EAAE,SAAkB,YAAY;QAC7D,IAAI,CAAC,KAAK,GAAG,KAAK,CAAC;QACnB,IAAI,CAAC,MAAM,GAAG,MAAM,CAAC;IACvB,CAAC;IAEO,6BAA6B,CACnC,OAA8B;QAE9B,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC;YACzB,MAAM,IAAI,eAAe,CACvB,8EAA8E,CAC/E,CAAC;QACJ,CAAC;QACD,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,CAAC;YACvB,MAAM,IAAI,eAAe,CACvB,2FAA2F,OAAO,CAAC,WAAW,EAAE,CACjH,CAAC;QACJ,CAAC;QACD,IAAI,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;YACrB,MAAM,IAAI,eAAe,CACvB,yFAAyF,OAAO,CAAC,WAAW,EAAE,CAC/G,CAAC;QACJ,CAAC;QACD,OAAO;YACL,WAAW,EAAE,OAAO,CAAC,WAAW;YAChC,SAAS,EAAE,OAAO,CAAC,SAAS;YAC5B,OAAO,EAAE,OAAO,CAAC,OAAO;SACzB,CAAC;IACJ,CAAC;IAED;;;;;;OAMG;IACH,KAAK,CAAC,WAAW,CACf,QAAgB,EAChB,UAA2B,EAC3B,OAA+B;QAE/B,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,mBAAmB,QAAQ,EAAE,CAAC,CAAC;QAEjD,IAAI,CAAC;YACH,IAAI,CAAC,OAAO,EAAE,WAAW,EAAE,CAAC;gBAC1B,MAAM,IAAI,eAAe,CACvB,gFAAgF,CACjF,CAAC;YACJ,CAAC;YAED,MAAM,YAAY,GAAG,OAAO,CAAC,YAAY,CAAC;YAC1C,IAAI,YAAY,EAAE,CAAC;gBACjB,MAAM,UAAU,GAAmB;oBACjC,SAAS,EAAE,IAAI,IAAI,EAAE;oBACrB,QAAQ;oBACR,UAAU;iBACX,CAAC;gBACF,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;YACxD,CAAC;YAED,wEAAwE;YACxE,MAAM,EAAE,QAAQ,EAAE,SAAS,EAAE,UAAU,EAAE,WAAW,EAAE,GAAG,WAAW,EAAE,GAAG,OAAO,CAAC;YACjF,KAAK,SAAS,CAAC;YACf,KAAK,WAAW,CAAC;YAEjB,MAAM,gBAAgB,GAA0B;gBAC9C,GAAG,WAAW;gBACd,QAAQ;gBACR,UAAU;gBACV,WAAW,EAAE,OAAO,CAAC,WAAW;aACjC,CAAC;YAEF,qCAAqC;YACrC,4EAA4E;YAC5E,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,QAAQ,EAAE,UAAU,EAAE,gBAAgB,CAAC,CAAC;YAEpF,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,6BAA6B,QAAQ,EAAE,CAAC,CAAC;YAE3D,IAAI,YAAY,EAAE,CAAC;gBACjB,MAAM,aAAa,GAAmB;oBACpC,SAAS,EAAE,IAAI,IAAI,EAAE;oBACrB,QAAQ;oBACR,MAAM,EAAE,MAAM;iBACf,CAAC;gBACF,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,aAAa,EAAE,aAAa,CAAC,CAAC;gBAC5D,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,mBAAmB,EAAE,aAAa,CAAC,CAAC;YACpE,CAAC;YAED,OAAO;gBACL,OAAO,EAAE,IAAI;gBACb,MAAM,EAAE,MAAM;gBACd,QAAQ;gBACR,WAAW,EAAE,gBAAgB,CAAC,WAAY;aAC3C,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,0BAA0B,QAAQ,EAAE,CAAC,CAAC;YAExD,MAAM,SAAS,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;YAE5E,MAAM,YAAY,GAAG,OAAO,EAAE,YAAY,CAAC;YAC3C,IAAI,YAAY,IAAI,OAAO,EAAE,WAAW,EAAE,CAAC;gBACzC,MAAM,UAAU,GAAmB;oBACjC,SAAS,EAAE,IAAI,IAAI,EAAE;oBACrB,QAAQ;oBACR,KAAK,EAAE,SAAS,CAAC,OAAO;iBACzB,CAAC;gBACF,YAAY,CAAC,IAAI,CAAC,WAAW,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;YACxD,CAAC;YAED,OAAO;gBACL,OAAO,EAAE,KAAK;gBACd,KAAK,EAAE,SAAS,CAAC,OAAO;gBACxB,QAAQ;gBACR,WAAW,EAAE,OAAO,EAAE,WAAW;aAClC,CAAC;QACJ,CAAC;IACH,CAAC;IAED;;;;;OAKG;IACH,kCAAkC,CAChC,SAA+E,EAC/E,OAMC;QAED,OAAO,SAAS,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE;YAChC,IAAI,gBAAiC,CAAC;YACtC,IAAI,CAAC;gBACH,gBAAgB,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,SAAS,CAAoB,CAAC;YAChF,CAAC;YAAC,MAAM,CAAC;gBACP,MAAM,IAAI,eAAe,CACvB,uCAAuC,QAAQ,CAAC,QAAQ,CAAC,IAAI,WAAW,QAAQ,CAAC,EAAE,iBAAiB,CACrG,CAAC;YACJ,CAAC;YACD,OAAO;gBACL,QAAQ,EAAE,QAAQ,CAAC,QAAQ,CAAC,IAAI;gBAChC,UAAU,EAAE,gBAAgB;gBAC5B,WAAW,EAAE,QAAQ,CAAC,EAAE;gBACxB,SAAS,EAAE,MAAM;gBACjB,OAAO,EAAE,QAAQ,CAAC,EAAE;gBACpB,SAAS,EAAE,CAAC,GAAG,OAAO,CAAC,aAAa,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,QAAQ,CAAC,EAAE,EAAE,CAAC;gBACxE,QAAQ,EAAE,OAAO,CAAC,eAAe,CAAC,CAAC,CAAC,OAAO,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,SAAS;aAClF,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC;IAED;;;;OAIG;IACH,KAAK,CAAC,YAAY,CAChB,YAAwC;QAExC,IAAI,CAAC,MAAM,CAAC,KAAK,CACf,aAAa,YAAY,CAAC,QAAQ,CAAC,MAAM,aAAa,YAAY,CAAC,IAAI,OAAO,CAC/E,CAAC;QAEF,MAAM,OAAO,GAA2B,EAAE,CAAC;QAC3C,MAAM,MAAM,GAAY,EAAE,CAAC;QAE3B,IAAI,YAAY,CAAC,IAAI,KAAK,UAAU,EAAE,CAAC;YACrC,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CACrD,CAAC,GAAG,EAAE;gBACJ,MAAM,QAAQ,GAAG,IAAI,CAAC,6BAA6B,CAAC,OAAO,CAAC,CAAC;gBAC7D,OAAO,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,UAAU,EAAE;oBAC5D,QAAQ,EAAE,OAAO,CAAC,QAAQ;oBAC1B,UAAU,EAAE,OAAO,CAAC,UAAU;oBAC9B,WAAW,EAAE,QAAQ,CAAC,WAAW;oBACjC,SAAS,EAAE,QAAQ,CAAC,SAAS;oBAC7B,OAAO,EAAE,QAAQ,CAAC,OAAO;oBACzB,SAAS,EAAE,OAAO,CAAC,SAAS;oBAC5B,QAAQ,EAAE,OAAO,CAAC,QAAQ;oBAC1B,YAAY,EAAE,OAAO,CAAC,YAAY;oBAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;iBAC3C,CAAC,CAAC;YACL,CAAC,CAAC,EAAE,CACL,CAAC;YACF,MAAM,UAAU,GAAG,MAAM,OAAO,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC;YAEtD,oFAAoF;YACpF,6FAA6F;YAC7F,UAAU,CAAC,OAAO,CAAC,CAAC,aAAa,EAAE,KAAK,EAAE,EAAE;gBAC1C,MAAM,OAAO,GAAG,YAAY,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;gBAC7C,IAAI,CAAC,OAAO,EAAE,CAAC;oBACb,OAAO;gBACT,CAAC;gBACD,IAAI,aAAa,CAAC,MAAM,KAAK,WAAW,EAAE,CAAC;oBACzC,MAAM,MAAM,GAAG,aAAa,CAAC,KAAK,CAAC;oBACnC,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;oBACrB,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;wBACpB,MAAM,GAAG,GAAG,IAAI,KAAK,CACnB,mCAAmC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,UAAU,MAAM,CAAC,MAAM,CAAC,KAAK,IAAI,eAAe,CAAC,EAAE,CACxJ,CAAC;wBACF,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;oBACnB,CAAC;oBACD,OAAO;gBACT,CAAC;gBACD,MAAM,GAAG,GACP,aAAa,CAAC,MAAM,YAAY,KAAK;oBACnC,CAAC,CAAC,aAAa,CAAC,MAAM;oBACtB,CAAC,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,aAAa,CAAC,MAAM,CAAC,CAAC,CAAC;gBAC9C,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;gBACjB,OAAO,CAAC,IAAI,CAAC;oBACX,QAAQ,EAAE,OAAO,CAAC,QAAQ;oBAC1B,MAAM,EAAE,IAAI;oBACZ,OAAO,EAAE,KAAK;oBACd,KAAK,EAAE,GAAG,CAAC,OAAO;oBAClB,WAAW,EAAE,OAAO,CAAC,WAAW;iBACjC,CAAC,CAAC;YACL,CAAC,CAAC,CAAC;YACH,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE,CAAC;gBACvD,MAAM,MAAM,CAAC,CAAC,CAAC,CAAC;YAClB,CAAC;QACH,CAAC;aAAM,CAAC;YACN,uBAAuB;YACvB,KAAK,MAAM,OAAO,IAAI,YAAY,CAAC,QAAQ,EAAE,CAAC;gBAC5C,IAAI,CAAC;oBACH,MAAM,QAAQ,GAAG,IAAI,CAAC,6BAA6B,CAAC,OAAO,CAAC,CAAC;oBAC7D,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,UAAU,EAAE;wBAC1E,QAAQ,EAAE,OAAO,CAAC,QAAQ;wBAC1B,UAAU,EAAE,OAAO,CAAC,UAAU;wBAC9B,WAAW,EAAE,QAAQ,CAAC,WAAW;wBACjC,SAAS,EAAE,QAAQ,CAAC,SAAS;wBAC7B,OAAO,EAAE,QAAQ,CAAC,OAAO;wBACzB,SAAS,EAAE,OAAO,CAAC,SAAS;wBAC5B,QAAQ,EAAE,OAAO,CAAC,QAAQ;wBAC1B,YAAY,EAAE,OAAO,CAAC,YAAY;wBAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;qBAC3C,CAAC,CAAC;oBACH,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;oBACrB,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;wBACpB,MAAM,CAAC,IAAI,CACT,IAAI,KAAK,CACP,mCAAmC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,UAAU,MAAM,CAAC,MAAM,CAAC,KAAK,IAAI,eAAe,CAAC,EAAE,CACxJ,CACF,CAAC;oBACJ,CAAC;oBAED,IAAI,CAAC,MAAM,CAAC,OAAO,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE,CAAC;wBACrD,MAAM;oBACR,CAAC;gBACH,CAAC;gBAAC,OAAO,KAAK,EAAE,CAAC;oBACf,MAAM,GAAG,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;oBACtE,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;oBAEjB,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE,CAAC;wBAClC,MAAM;oBACR,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,CAAC;IAC7B,CAAC;CACF"}

package/dist/services/tool-execution-service.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=tool-execution-service.test.d.ts.map

package/dist/services/tool-execution-service.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"tool-execution-service.test.d.ts","sourceRoot":"","sources":["../../src/services/tool-execution-service.test.ts"],"names":[],"mappings":""}

package/dist/services/tool-execution-service.test.js

@@ -0,0 +1,358 @@
+import { describe, it, expect, vi } from 'vitest';
+// Mock logger before importing ToolExecutionService
+vi.mock('../utils/logger', () => ({
+    Logger: vi.fn().mockImplementation(() => ({
+        debug: vi.fn(),
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+    })),
+    createLogger: vi.fn().mockReturnValue({
+        debug: vi.fn(),
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+        isDebugEnabled: vi.fn().mockReturnValue(false),
+        setLevel: vi.fn(),
+        getLevel: vi.fn().mockReturnValue('warn'),
+    }),
+    SilentLogger: {
+        debug: vi.fn(),
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+    },
+}));
+import { ToolExecutionService, TOOL_EVENTS, TOOL_EVENT_PREFIX } from './tool-execution-service';
+/**
+ * Create a mock IToolManager.
+ */
+function createMockToolManager(overrides = {}) {
+    return {
+        addTool: vi.fn(),
+        removeTool: vi.fn(),
+        getTool: vi.fn(),
+        getToolSchema: vi.fn(),
+        getTools: vi.fn().mockReturnValue([]),
+        executeTool: vi.fn().mockResolvedValue('tool result'),
+        hasTool: vi.fn().mockReturnValue(true),
+        setAllowedTools: vi.fn(),
+        getAllowedTools: vi.fn(),
+        ...overrides,
+    };
+}
+/**
+ * Create a mock IEventService.
+ */
+function createMockEventService() {
+    const emitCalls = [];
+    return {
+        emitCalls,
+        emit: vi.fn((eventType, data) => {
+            emitCalls.push({ eventType, data });
+        }),
+        subscribe: vi.fn(),
+        unsubscribe: vi.fn(),
+    };
+}
+/**
+ * Create a minimal IToolExecutionContext.
+ */
+function createToolContext(overrides = {}) {
+    return {
+        toolName: 'test-tool',
+        parameters: { key: 'value' },
+        executionId: 'tool_call_1',
+        ownerType: 'tool',
+        ownerId: 'tool_call_1',
+        ...overrides,
+    };
+}
+/**
+ * Create a mock logger.
+ */
+function createMockLogger() {
+    return {
+        debug: vi.fn(),
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+        log: vi.fn(),
+    };
+}
+describe('ToolExecutionService', () => {
+    describe('TOOL_EVENTS and TOOL_EVENT_PREFIX', () => {
+        it('should export correct event constants', () => {
+            expect(TOOL_EVENT_PREFIX).toBe('tool');
+            expect(TOOL_EVENTS.CALL_START).toBe('call_start');
+            expect(TOOL_EVENTS.CALL_COMPLETE).toBe('call_complete');
+            expect(TOOL_EVENTS.CALL_ERROR).toBe('call_error');
+            expect(TOOL_EVENTS.CALL_RESPONSE_READY).toBe('call_response_ready');
+        });
+    });
+    // ----------------------------------------------------------------
+    // executeTool
+    // ----------------------------------------------------------------
+    describe('executeTool', () => {
+        it('should execute a tool and return a success result', async () => {
+            const tools = createMockToolManager({
+                executeTool: vi.fn().mockResolvedValue('success data'),
+            });
+            const logger = createMockLogger();
+            const service = new ToolExecutionService(tools, logger);
+            const result = await service.executeTool('my-tool', { foo: 'bar' }, createToolContext());
+            expect(result.success).toBe(true);
+            expect(result.result).toBe('success data');
+            expect(result.toolName).toBe('my-tool');
+            expect(result.executionId).toBe('tool_call_1');
+            expect(tools.executeTool).toHaveBeenCalledWith('my-tool', { foo: 'bar' }, expect.objectContaining({
+                toolName: 'my-tool',
+                parameters: { foo: 'bar' },
+                executionId: 'tool_call_1',
+            }));
+        });
+        it('should throw ValidationError when executionId is missing', async () => {
+            const tools = createMockToolManager();
+            const service = new ToolExecutionService(tools);
+            const result = await service.executeTool('my-tool', {}, createToolContext({ executionId: undefined }));
+            expect(result.success).toBe(false);
+            expect(result.error).toContain('executionId');
+        });
+        it('should return failure result when tool execution throws', async () => {
+            const tools = createMockToolManager({
+                executeTool: vi.fn().mockRejectedValue(new Error('Tool exploded')),
+            });
+            const logger = createMockLogger();
+            const service = new ToolExecutionService(tools, logger);
+            const result = await service.executeTool('failing-tool', {}, createToolContext());
+            expect(result.success).toBe(false);
+            expect(result.error).toBe('Tool exploded');
+            expect(result.toolName).toBe('failing-tool');
+        });
+        it('should emit start and complete events when eventService is provided', async () => {
+            const tools = createMockToolManager({
+                executeTool: vi.fn().mockResolvedValue('result'),
+            });
+            const eventService = createMockEventService();
+            const service = new ToolExecutionService(tools, createMockLogger());
+            await service.executeTool('my-tool', { a: '1' }, createToolContext({ eventService }));
+            expect(eventService.emit).toHaveBeenCalledWith(TOOL_EVENTS.CALL_START, expect.objectContaining({ toolName: 'my-tool' }));
+            expect(eventService.emit).toHaveBeenCalledWith(TOOL_EVENTS.CALL_COMPLETE, expect.objectContaining({ toolName: 'my-tool' }));
+            expect(eventService.emit).toHaveBeenCalledWith(TOOL_EVENTS.CALL_RESPONSE_READY, expect.objectContaining({ toolName: 'my-tool' }));
+        });
+        it('should emit error event when tool fails and eventService is provided', async () => {
+            const tools = createMockToolManager({
+                executeTool: vi.fn().mockRejectedValue(new Error('Boom')),
+            });
+            const eventService = createMockEventService();
+            const service = new ToolExecutionService(tools, createMockLogger());
+            await service.executeTool('my-tool', {}, createToolContext({ eventService }));
+            expect(eventService.emit).toHaveBeenCalledWith(TOOL_EVENTS.CALL_ERROR, expect.objectContaining({ toolName: 'my-tool', error: 'Boom' }));
+        });
+        it('should handle non-Error thrown values', async () => {
+            const tools = createMockToolManager({
+                executeTool: vi.fn().mockRejectedValue('string error'),
+            });
+            const service = new ToolExecutionService(tools, createMockLogger());
+            const result = await service.executeTool('my-tool', {}, createToolContext());
+            expect(result.success).toBe(false);
+            expect(result.error).toBe('string error');
+        });
+    });
+    // ----------------------------------------------------------------
+    // createExecutionRequestsWithContext
+    // ----------------------------------------------------------------
+    describe('createExecutionRequestsWithContext', () => {
+        it('should create execution requests from tool calls', () => {
+            const tools = createMockToolManager();
+            const service = new ToolExecutionService(tools);
+            const toolCalls = [
+                { id: 'call_1', function: { name: 'tool_a', arguments: '{"x": 1}' } },
+                { id: 'call_2', function: { name: 'tool_b', arguments: '{"y": "hello"}' } },
+            ];
+            const requests = service.createExecutionRequestsWithContext(toolCalls, {
+                ownerPathBase: [{ type: 'execution', id: 'exec_1' }],
+            });
+            expect(requests).toHaveLength(2);
+            expect(requests[0]?.toolName).toBe('tool_a');
+            expect(requests[0]?.parameters).toEqual({ x: 1 });
+            expect(requests[0]?.executionId).toBe('call_1');
+            expect(requests[0]?.ownerType).toBe('tool');
+            expect(requests[0]?.ownerId).toBe('call_1');
+            expect(requests[0]?.ownerPath).toEqual([
+                { type: 'execution', id: 'exec_1' },
+                { type: 'tool', id: 'call_1' },
+            ]);
+            expect(requests[1]?.toolName).toBe('tool_b');
+            expect(requests[1]?.parameters).toEqual({ y: 'hello' });
+        });
+        it('should apply metadata factory when provided', () => {
+            const tools = createMockToolManager();
+            const service = new ToolExecutionService(tools);
+            const toolCalls = [{ id: 'call_1', function: { name: 'tool_a', arguments: '{}' } }];
+            const requests = service.createExecutionRequestsWithContext(toolCalls, {
+                ownerPathBase: [],
+                metadataFactory: (tc) => ({ toolCallId: tc.id }),
+            });
+            expect(requests[0]?.metadata).toEqual({ toolCallId: 'call_1' });
+        });
+    });
+    // ----------------------------------------------------------------
+    // executeTools (batch)
+    // ----------------------------------------------------------------
+    describe('executeTools', () => {
+        function createRequest(overrides = {}) {
+            return {
+                toolName: 'batch-tool',
+                parameters: {},
+                executionId: 'batch_exec_1',
+                ownerType: 'tool',
+                ownerId: 'batch_exec_1',
+                ...overrides,
+            };
+        }
+        describe('parallel mode', () => {
+            it('should execute all tools in parallel and return results', async () => {
+                const tools = createMockToolManager({
+                    executeTool: vi.fn().mockResolvedValue('parallel result'),
+                });
+                const service = new ToolExecutionService(tools, createMockLogger());
+                const { results, errors } = await service.executeTools({
+                    requests: [
+                        createRequest({ executionId: 'p1', ownerId: 'p1' }),
+                        createRequest({ executionId: 'p2', ownerId: 'p2' }),
+                    ],
+                    mode: 'parallel',
+                    continueOnError: true,
+                });
+                expect(results).toHaveLength(2);
+                expect(results[0]?.success).toBe(true);
+                expect(results[1]?.success).toBe(true);
+                expect(errors).toHaveLength(0);
+            });
+            it('should throw on first error when continueOnError is false', async () => {
+                const tools = createMockToolManager({
+                    executeTool: vi.fn().mockResolvedValueOnce('ok').mockRejectedValueOnce(new Error('fail')),
+                });
+                const service = new ToolExecutionService(tools, createMockLogger());
+                // When parallel execution fails and continueOnError is false,
+                // the method should throw
+                // But note: executeTool catches errors internally and returns {success: false}
+                // So the error path through Promise.allSettled would be from requireExecutionRequestFields
+                // or from the tool execution producing a non-success result.
+                // In this case, the tool execution returns { success: false }, which pushes to errors array.
+                // Since continueOnError is false, the first error is thrown.
+                await expect(service.executeTools({
+                    requests: [
+                        createRequest({ executionId: 'p1', ownerId: 'p1' }),
+                        createRequest({ executionId: 'p2', ownerId: 'p2' }),
+                    ],
+                    mode: 'parallel',
+                    continueOnError: false,
+                })).rejects.toThrow();
+            });
+            it('should validate required fields and throw for missing executionId', async () => {
+                const tools = createMockToolManager();
+                const service = new ToolExecutionService(tools, createMockLogger());
+                await expect(service.executeTools({
+                    requests: [createRequest({ executionId: undefined })],
+                    mode: 'parallel',
+                })).rejects.toThrow('executionId');
+            });
+            it('should validate required fields and throw for missing ownerType', async () => {
+                const tools = createMockToolManager();
+                const service = new ToolExecutionService(tools, createMockLogger());
+                await expect(service.executeTools({
+                    requests: [createRequest({ ownerType: undefined })],
+                    mode: 'parallel',
+                })).rejects.toThrow('ownerType');
+            });
+            it('should validate required fields and throw for missing ownerId', async () => {
+                const tools = createMockToolManager();
+                const service = new ToolExecutionService(tools, createMockLogger());
+                await expect(service.executeTools({
+                    requests: [createRequest({ ownerId: undefined })],
+                    mode: 'parallel',
+                })).rejects.toThrow('ownerId');
+            });
+        });
+        describe('sequential mode', () => {
+            it('should execute tools sequentially and return results', async () => {
+                const executionOrder = [];
+                const tools = createMockToolManager({
+                    executeTool: vi.fn().mockImplementation(async (name) => {
+                        executionOrder.push(name);
+                        return `result_${name}`;
+                    }),
+                });
+                const service = new ToolExecutionService(tools, createMockLogger());
+                const { results, errors } = await service.executeTools({
+                    requests: [
+                        createRequest({ toolName: 'tool_a', executionId: 's1', ownerId: 's1' }),
+                        createRequest({ toolName: 'tool_b', executionId: 's2', ownerId: 's2' }),
+                    ],
+                    mode: 'sequential',
+                });
+                expect(results).toHaveLength(2);
+                expect(errors).toHaveLength(0);
+                expect(executionOrder).toEqual(['tool_a', 'tool_b']);
+            });
+            it('should stop on error when continueOnError is false', async () => {
+                const tools = createMockToolManager({
+                    executeTool: vi
+                        .fn()
+                        .mockRejectedValueOnce(new Error('first fail'))
+                        .mockResolvedValueOnce('second ok'),
+                });
+                const service = new ToolExecutionService(tools, createMockLogger());
+                const { results, errors } = await service.executeTools({
+                    requests: [
+                        createRequest({ executionId: 's1', ownerId: 's1' }),
+                        createRequest({ executionId: 's2', ownerId: 's2' }),
+                    ],
+                    mode: 'sequential',
+                    continueOnError: false,
+                });
+                // First tool fails (executeTool catches error and returns {success: false})
+                // which triggers errors.push and then break (continueOnError=false)
+                expect(results).toHaveLength(1);
+                expect(results[0]?.success).toBe(false);
+                expect(errors).toHaveLength(1);
+            });
+            it('should continue on error when continueOnError is true', async () => {
+                const tools = createMockToolManager({
+                    executeTool: vi
+                        .fn()
+                        .mockRejectedValueOnce(new Error('first fail'))
+                        .mockResolvedValueOnce('second ok'),
+                });
+                const service = new ToolExecutionService(tools, createMockLogger());
+                const { results, errors } = await service.executeTools({
+                    requests: [
+                        createRequest({ executionId: 's1', ownerId: 's1' }),
+                        createRequest({ executionId: 's2', ownerId: 's2' }),
+                    ],
+                    mode: 'sequential',
+                    continueOnError: true,
+                });
+                // First returns { success: false }, second returns { success: true }
+                expect(results).toHaveLength(2);
+                expect(errors).toHaveLength(1);
+            });
+            it('should validate required fields for sequential execution', async () => {
+                const tools = createMockToolManager();
+                const service = new ToolExecutionService(tools, createMockLogger());
+                // Missing executionId should cause requireExecutionRequestFields to throw
+                // In sequential mode, it catches the error and adds to errors array, then breaks
+                const { results, errors } = await service.executeTools({
+                    requests: [createRequest({ executionId: undefined })],
+                    mode: 'sequential',
+                    continueOnError: false,
+                });
+                expect(errors).toHaveLength(1);
+                expect(errors[0]?.message).toContain('executionId');
+            });
+        });
+    });
+});
+//# sourceMappingURL=tool-execution-service.test.js.map