@lowdefy/api 0.0.0-experimental-20260420104522 → 0.0.0-experimental-20260420135540

package/dist/routes/agent/callAgent.js

@@ -13,6 +13,7 @@
   See the License for the specific language governing permissions and
   limitations under the License.
 */ import { serializer, type } from '@lowdefy/helpers';
+import { createPhaseLogger } from '@lowdefy/ai-utils';
 import createEvaluateOperators from '../../context/createEvaluateOperators.js';
 import authorizeApiEndpoint from '../endpoints/authorizeApiEndpoint.js';
 import getEndpointConfig from '../endpoints/getEndpointConfig.js';
@@ -21,18 +22,30 @@ import getAgentConfig from './getAgentConfig.js';
 import getAgentResolver from './getAgentResolver.js';
 import getConnectionConfig from '../request/getConnectionConfig.js';
 import getConnection from '../request/getConnection.js';
-async function callAgent(context, { agentId, pageId, messages, conversationId, urlQuery, sharedState }) {
+async function callAgent(context, { agentId, pageId, messages, conversationId, urlQuery, sharedState, phaseLogger: ingressPhaseLogger, turnId }) {
     const { logger } = context;
     context.pageId = pageId;
     context.evaluateOperators = createEvaluateOperators(context);
+    // Use the ingress-provided phase logger when present so turnId/turnStart are
+    // continuous with the HTTP-ingress logs. Fall back to a fresh logger so
+    // internal callers of callAgent still get structured output.
+    const phaseLogger = ingressPhaseLogger ?? createPhaseLogger({
+        logger,
+        agentId,
+        pageId,
+        conversationId: conversationId ?? undefined,
+        turnId
+    });
+    context.phaseLogger = phaseLogger;
     logger.debug({
         event: 'debug_agent',
         agentId,
         pageId
     });
-    const agentConfig = await getAgentConfig(context, {
-        agentId
-    });
+    phaseLogger.phase('handler.entry');
+    const agentConfig = await phaseLogger.time('config.agent.load', ()=>getAgentConfig(context, {
+            agentId
+        }));
     const agentContext = {
         conversationId: conversationId ?? undefined,
         pageId,
@@ -41,75 +54,88 @@ async function callAgent(context, { agentId, pageId, messages, conversationId, u
         userId: context.user?.sub ?? context.user?.id ?? null
     };
     // Evaluate operators in agent properties (e.g. _user, _secret, _payload)
-    agentConfig.properties = context.evaluateOperators({
-        input: agentConfig.properties ?? {},
-        location: agentConfig.agentId,
-        payload: agentContext,
-        steps: {}
-    });
+    agentConfig.properties = await phaseLogger.time('operators.agent.properties', ()=>context.evaluateOperators({
+            input: agentConfig.properties ?? {},
+            location: agentConfig.agentId,
+            payload: agentContext,
+            steps: {}
+        }));
     // Load connection config from build artifacts using agent's connectionId
-    const connectionConfig = await getConnectionConfig(context, {
-        requestConfig: {
-            connectionId: agentConfig.connectionId,
-            requestId: agentConfig.agentId,
-            '~k': agentConfig['~k']
-        }
-    });
+    const connectionConfig = await phaseLogger.time('config.connection.load', ()=>getConnectionConfig(context, {
+            requestConfig: {
+                connectionId: agentConfig.connectionId,
+                requestId: agentConfig.agentId,
+                '~k': agentConfig['~k']
+            }
+        }));
     // Get connection plugin from registry
     const connection = getConnection(context, {
         connectionConfig
     });
     // Evaluate operators in connection properties
-    const connectionProperties = context.evaluateOperators({
-        input: connectionConfig.properties || {},
-        location: connectionConfig.connectionId,
-        payload: {},
-        steps: {}
-    });
+    const connectionProperties = await phaseLogger.time('operators.connection.properties', ()=>context.evaluateOperators({
+            input: connectionConfig.properties || {},
+            location: connectionConfig.connectionId,
+            payload: {},
+            steps: {}
+        }));
     // Create connection instance (e.g., Anthropic provider)
-    const connectionInstance = connection.create({
-        connection: connectionProperties
-    });
+    const connectionInstance = await phaseLogger.time('connection.instantiate', ()=>Promise.resolve(connection.create({
+            connection: connectionProperties
+        })));
     // Get agent type from plugin registry
     const agentType = getAgentResolver(context, {
         agentConfig
     });
+    phaseLogger.phase('resolver.lookup.done', {
+        type: agentConfig.type
+    });
     // Build resolver context with callEndpoint that allows InternalApi endpoints
     const resolverContext = {
         agentContext,
+        phaseLogger,
         evaluateOperators: (input)=>context.evaluateOperators({
                 input,
                 location: agentConfig.agentId,
                 payload: agentContext,
                 steps: {}
             }),
-        callEndpoint: async (endpointId, { payload, abortSignal })=>{
-            const endpointConfig = await getEndpointConfig(context, {
-                endpointId
-            });
-            authorizeApiEndpoint(context, {
-                endpointConfig
+        callEndpoint: async (endpointId, { payload, abortSignal, kind = 'tool' } = {})=>{
+            const prefix = kind === 'hook' ? 'hook.endpoint' : 'tool.endpoint';
+            const callLog = phaseLogger.child({
+                endpointId,
+                kind
             });
-            const routineContext = {
-                steps: {},
-                payload: payload ?? {},
-                arrayIndices: [],
-                items: {},
-                endpointDepth: 0
-            };
-            const { error, response, status } = await runRoutine(context, routineContext, {
-                routine: endpointConfig.routine
+            return callLog.time(`${prefix}.exec`, async ()=>{
+                const endpointConfig = await callLog.time(`${prefix}.config.load`, ()=>getEndpointConfig(context, {
+                        endpointId
+                    }));
+                authorizeApiEndpoint(context, {
+                    endpointConfig
+                });
+                const routineContext = {
+                    steps: {},
+                    payload: payload ?? {},
+                    arrayIndices: [],
+                    items: {},
+                    endpointDepth: 0
+                };
+                const { error, response, status } = await runRoutine(context, routineContext, {
+                    routine: endpointConfig.routine
+                });
+                const success = ![
+                    'error',
+                    'reject'
+                ].includes(status);
+                return {
+                    error: serializer.serialize(error),
+                    response: serializer.serialize(response),
+                    status: success ? 'success' : status,
+                    success
+                };
+            }, {
+                hasPayload: payload != null
             });
-            const success = ![
-                'error',
-                'reject'
-            ].includes(status);
-            return {
-                error: serializer.serialize(error),
-                response: serializer.serialize(response),
-                status: success ? 'success' : status,
-                success
-            };
         },
         getEndpointConfig: async ({ endpointId })=>{
             return getEndpointConfig(context, {
@@ -181,28 +207,39 @@ async function callAgent(context, { agentId, pageId, messages, conversationId, u
     // Agent-level overrides (like confirm) may still contain operators —
     // handleAgentChat evaluates those via its existing evaluateOperators call.
     const resolvedMcp = [];
-    for (const mcpSource of agentConfig.mcp ?? []){
+    const mcpSources = agentConfig.mcp ?? [];
+    phaseLogger.phase('mcp.sources.resolve.start', {
+        sourceCount: mcpSources.length
+    });
+    for(let i = 0; i < mcpSources.length; i += 1){
+        const mcpSource = mcpSources[i];
         if (!type.isNone(mcpSource.connectionId)) {
-            const mcpConnConfig = await getConnectionConfig(context, {
-                requestConfig: {
-                    connectionId: mcpSource.connectionId,
-                    requestId: agentConfig.agentId,
-                    '~k': agentConfig['~k']
-                }
+            const mcpLog = phaseLogger.child({
+                mcpIndex: i,
+                connectionId: mcpSource.connectionId
             });
-            const mcpConnection = getConnection(context, {
-                connectionConfig: mcpConnConfig
+            // eslint-disable-next-line no-await-in-loop
+            const mcpConfig = await mcpLog.time('mcp.source.resolve', async ()=>{
+                const mcpConnConfig = await getConnectionConfig(context, {
+                    requestConfig: {
+                        connectionId: mcpSource.connectionId,
+                        requestId: agentConfig.agentId,
+                        '~k': agentConfig['~k']
+                    }
+                });
+                const mcpConnection = getConnection(context, {
+                    connectionConfig: mcpConnConfig
+                });
+                const mcpConnProps = context.evaluateOperators({
+                    input: mcpConnConfig.properties || {},
+                    location: mcpConnConfig.connectionId,
+                    payload: {},
+                    steps: {}
+                });
+                return mcpConnection.create({
+                    connection: mcpConnProps
+                });
             });
-            const mcpConnProps = context.evaluateOperators({
-                input: mcpConnConfig.properties || {},
-                location: mcpConnConfig.connectionId,
-                payload: {},
-                steps: {}
-            });
-            const mcpConfig = mcpConnection.create({
-                connection: mcpConnProps
-            });
-            // Merge: connection properties as base, agent-level overrides on top
             const { connectionId: _, ...overrides } = mcpSource;
             resolvedMcp.push({
                 ...mcpConfig,
@@ -213,6 +250,12 @@ async function callAgent(context, { agentId, pageId, messages, conversationId, u
         }
     }
     agentConfig.mcp = resolvedMcp;
+    phaseLogger.phase('mcp.sources.resolve.done', {
+        resolvedCount: resolvedMcp.length
+    });
+    phaseLogger.phase('resolver.dispatch.start', {
+        type: agentConfig.type
+    });
     // Call the agent resolver
     const { response } = await agentType.resolver({
         connection: connectionInstance,
@@ -222,6 +265,7 @@ async function callAgent(context, { agentId, pageId, messages, conversationId, u
         },
         context: resolverContext
     });
+    phaseLogger.phase('resolver.dispatch.returned');
     return {
         response
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lowdefy/api",
-  "version": "0.0.0-experimental-20260420104522",
+  "version": "0.0.0-experimental-20260420135540",
   "license": "Apache-2.0",
   "description": "",
   "homepage": "https://lowdefy.com",
@@ -34,13 +34,14 @@
     "dist/*"
   ],
   "dependencies": {
-    "@lowdefy/ajv": "0.0.0-experimental-20260420104522",
-    "@lowdefy/errors": "0.0.0-experimental-20260420104522",
-    "@lowdefy/helpers": "0.0.0-experimental-20260420104522",
-    "@lowdefy/node-utils": "0.0.0-experimental-20260420104522",
-    "@lowdefy/nunjucks": "0.0.0-experimental-20260420104522",
-    "@lowdefy/operators": "0.0.0-experimental-20260420104522",
-    "@lowdefy/operators-js": "0.0.0-experimental-20260420104522"
+    "@lowdefy/ai-utils": "0.0.0-experimental-20260420135540",
+    "@lowdefy/ajv": "0.0.0-experimental-20260420135540",
+    "@lowdefy/errors": "0.0.0-experimental-20260420135540",
+    "@lowdefy/helpers": "0.0.0-experimental-20260420135540",
+    "@lowdefy/node-utils": "0.0.0-experimental-20260420135540",
+    "@lowdefy/nunjucks": "0.0.0-experimental-20260420135540",
+    "@lowdefy/operators": "0.0.0-experimental-20260420135540",
+    "@lowdefy/operators-js": "0.0.0-experimental-20260420135540"
   },
   "devDependencies": {
     "@jest/globals": "28.1.3",