@looopy-ai/core 2.1.18 → 2.1.20

package/dist/core/agent.js

@@ -227,7 +227,7 @@ export class Agent {
                             observer.next(event);
                         },
                         error: (error) => {
-                            logger.error({ error }, 'Turn execution failed');
+                            logger.error({ error: serializeError(error) }, 'Turn execution failed');
                             failAgentTurnSpan(turnSpan, error);
                             observer.error(error);
                         },
@@ -247,7 +247,7 @@ export class Agent {
                                 observer.complete();
                             }
                             catch (error) {
-                                logger.error({ error }, 'Failed to save turn results');
+                                logger.error({ error: serializeError(error) }, 'Failed to save turn results');
                                 failAgentTurnSpan(turnSpan, error);
                                 observer.error(error);
                             }
@@ -255,7 +255,7 @@ export class Agent {
                     });
                 }
                 catch (error) {
-                    logger.error({ error }, 'Failed to prepare turn');
+                    logger.error({ error: serializeError(error) }, 'Failed to prepare turn');
                     failAgentTurnSpan(turnSpan, error);
                     observer.error(error);
                 }
@@ -294,7 +294,7 @@ export class Agent {
             this.config.logger.info('Agent shutdown complete');
         }
         catch (error) {
-            this.config.logger.error({ error }, 'Failed to shutdown agent');
+            this.config.logger.error({ error: serializeError(error) }, 'Failed to shutdown agent');
             throw error;
         }
         finally {
@@ -317,7 +317,7 @@ export class Agent {
             this.logger.info('Agent data cleared');
         }
         catch (error) {
-            this.logger.error({ error }, 'Failed to clear agent');
+            this.logger.error({ error: serializeError(error) }, 'Failed to clear agent');
             throw error;
         }
     }
@@ -337,7 +337,7 @@ export class Agent {
             return;
         }
         void this.persistState().catch((error) => {
-            this.logger.error({ error }, 'Failed to persist agent state');
+            this.logger.error({ error: serializeError(error) }, 'Failed to persist agent state');
         });
     }
     async loadPersistedState() {

package/dist/core/iteration.js

@@ -1,7 +1,5 @@
 import { concat, defer, filter, map, mergeMap, shareReplay } from 'rxjs';
 import { startLLMCallSpan, startLoopIterationSpan } from '../observability/spans';
-import { toolErrorEvent } from '../tools/tool-result-events';
-import { safeValidateToolCall } from '../types/tools';
 import { getSystemPrompts } from '../utils/prompt';
 import { runToolCall } from './tools';
 export const runIteration = (context, config, history) => {
@@ -37,35 +35,7 @@ export const runIteration = (context, config, history) => {
             stream: true,
             sessionId: context.taskId,
         })
-            .pipe(finishLLMCallSpan, map((event) => {
-            if (event.kind === 'tool-call') {
-                const validation = safeValidateToolCall({
-                    id: event.toolCallId,
-                    type: 'function',
-                    function: {
-                        name: event.toolName,
-                        arguments: event.arguments,
-                    },
-                });
-                if (!validation.success) {
-                    const errorMessage = `Invalid tool call format: ${(validation.errors || []).map((e) => `${e.path.join('.')}: ${e.message}`).join(', ')}`;
-                    context.logger.error({
-                        toolCallId: event.toolCallId,
-                        toolName: event.toolName,
-                        errors: validation.errors,
-                    }, 'Invalid tool call from LLM - tool name must match ^[a-zA-Z0-9_-]+$');
-                    return toolErrorEvent({
-                        id: event.toolCallId,
-                        type: 'function',
-                        function: {
-                            name: event.toolName,
-                            arguments: event.arguments,
-                        },
-                    }, errorMessage);
-                }
-            }
-            return event;
-        }), map((event) => ({
+            .pipe(finishLLMCallSpan, map((event) => ({
             contextId: context.contextId,
             taskId: context.taskId,
             path: undefined,

package/dist/observability/spans/llm-call.js

@@ -28,13 +28,19 @@ export const startLLMCallSpan = (context, systemPrompts, messages, tools) => {
                 if (isChildTaskEvent(event))
                     return;
                 switch (event.kind) {
-                    case 'content-complete':
+                    case 'content-complete': {
                         if (event.content) {
                             span.setAttribute(SpanAttributes.GEN_AI_COMPLETION, event.content);
                         }
                         span.setAttribute(SpanAttributes.LLM_FINISH_REASON, event.finishReason || 'unknown');
+                        const toolCalls = event.toolCalls?.map((t) => t.function);
+                        if (toolCalls?.length) {
+                            context.logger.debug({ toolCalls }, 'Tools calls');
+                            span.setAttribute('llm.tool_calls', JSON.stringify(toolCalls.map((t) => t.name)));
+                        }
                         span.setStatus({ code: SpanStatusCode.OK });
                         break;
+                    }
                     case 'llm-usage':
                         span.setAttribute(SpanAttributes.GEN_AI_RESPONSE_MODEL, event.model);
                         span.setAttribute(SpanAttributes.GEN_AI_USAGE_PROMPT_TOKENS, event.prompt_tokens || 0);

package/dist/providers/litellm-provider.js

@@ -4,6 +4,7 @@ import { merge, Observable } from 'rxjs';
 import { concatWith, filter, map, mergeMap, shareReplay, tap } from 'rxjs/operators';
 import { getLogger } from '../core/logger';
 import { generateEventId } from '../events/utils';
+import { serializeError } from '../utils';
 import { aggregateChoice, aggregateLLMUsage, choices, getContent, splitInlineXml, usage, } from './chat-completions';
 const singleString = (input) => {
     if (!input)
@@ -87,7 +88,7 @@ export class LiteLLMProvider {
                 id: tc.id,
                 type: 'function',
                 function: {
-                    name: tc.function?.name,
+                    name: fixToolFunctionName(tc.function?.name),
                     arguments: (typeof tc.function?.arguments === 'string'
                         ? JSON.parse(tc.function?.arguments)
                         : tc.function?.arguments) || {},
@@ -110,7 +111,7 @@ export class LiteLLMProvider {
         const toolCalls$ = contentComplete$.pipe(filter((event) => event.finishReason === 'tool_calls'), mergeMap((event) => event.toolCalls?.map((tc) => ({
             kind: 'tool-call',
             toolCallId: tc.id,
-            toolName: tc.function.name,
+            toolName: fixToolFunctionName(tc.function.name),
             arguments: tc.function.arguments,
             timestamp: event.timestamp,
         })) || []));
@@ -127,7 +128,7 @@ export class LiteLLMProvider {
                 this.debugLogInitialized = true;
             }
             catch (error) {
-                this.logger.warn({ error }, 'Failed to initialize debug log file');
+                this.logger.warn({ error: serializeError(error) }, 'Failed to initialize debug log file');
                 return;
             }
         }
@@ -140,7 +141,7 @@ export class LiteLLMProvider {
             appendFileSync(this.config.debugLogPath, `${logEntry}\n`);
         }
         catch (error) {
-            this.logger.warn({ error }, 'Failed to write debug log');
+            this.logger.warn({ error: serializeError(error) }, 'Failed to write debug log');
         }
     }
     debugLog(eventType) {
@@ -190,7 +191,7 @@ export class LiteLLMProvider {
                             id: tc.id,
                             type: tc.type,
                             function: {
-                                name: tc.function.name,
+                                name: fixToolFunctionName(tc.function.name),
                                 arguments: typeof tc.function.arguments === 'string'
                                     ? tc.function.arguments
                                     : JSON.stringify(tc.function.arguments),
@@ -215,7 +216,7 @@ export class LiteLLMProvider {
                 litellmRequest.tools = request.tools.map((tool) => ({
                     type: 'function',
                     function: {
-                        name: getValidToolName(tool.id),
+                        name: fixToolFunctionName(tool.id),
                         description: tool.description,
                         parameters: tool.parameters,
                     },
@@ -377,7 +378,7 @@ export const LiteLLM = {
         });
     },
 };
-const getValidToolName = (toolId) => {
+const fixToolFunctionName = (toolId) => {
     const match = toolId.match(/^[a-zA-Z0-9_-]+/);
     return match ? match[0] : 'invalid_tool_name';
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@looopy-ai/core",
-  "version": "2.1.18",
+  "version": "2.1.20",
   "description": "RxJS-based AI agent framework",
   "repository": {
     "url": "https://github.com/looopy-ai/lib"