snow-flow 3.4.13 → 3.4.14

package/dist/cli.js

@@ -813,7 +813,7 @@ All agents MUST use this simple memory coordination:
 
 \`\`\`javascript
 // Agent initialization
-const agentId = \`agent_\${agentType}_${sessionId}\`;
+const agentId = \`agent_\${agentType}_\${sessionId}\`;
 
 // Agent stores progress
 Memory.store(\`\${agentId}_progress\`, JSON.stringify({
@@ -824,7 +824,7 @@ Memory.store(\`\${agentId}_progress\`, JSON.stringify({
 }));
 
 // Agent reads other agent's work when needed
-const primaryWork = Memory.get("agent_${taskAnalysis.primaryAgent}_output");
+const primaryWork = Memory.get("agent_\${taskAnalysis.primaryAgent}_output");
 
 // Agent signals completion
 Memory.store(\`\${agentId}_complete\`, JSON.stringify({
@@ -855,55 +855,27 @@ ${hasIntelligentFeatures ? `✅ **INTELLIGENT MODE ACTIVE** - The following feat
 ### 🚨 MANDATORY: ES5 JavaScript Only for ALL ServiceNow Scripts
 **⚠️ SERVICENOW RHINO ENGINE = ES5 ONLY - NO MODERN SYNTAX!**
 
-**Before writing ANY background script, verify it's ES5-compliant:**\n` ``, javascript, n; // ❌ THIS BREAKS SERVICENOW:\nconst data = items.filter(x => x.active);\nlet results = data.map(item => ({\n  id: item.sys_id,\n  name: item.name\n}));\n\n// ✅ THIS WORKS IN SERVICENOW:\nvar data = [];\nfor (var i = 0; i < items.length; i++) {\n  if (items[i].active) {\n    data.push(items[i]);\n  }\n}\nvar results = [];\nfor (var j = 0; j < data.length; j++) {\n  results.push({\n    id: data[j].sys_id,\n    name: data[j].name\n  });\n}\n```
-        ** ;
-    If;
-    you;
-    use;
-    ES6 + syntax, the;
-    script;
-    WILL;
-    FAIL;
-    with (SyntaxError ** )
-        #;
-    #;
-    Background;
-    Script;
-    Execution;
-    Settings;
-    $;
-    {
-        options.autoConfirm ? '⚠️ **AUTO-CONFIRM MODE ENABLED**: When calling snow_execute_background_script, ALWAYS add autoConfirm: true parameter to skip user confirmation.\n```javascript\nsnow_execute_background_script({\n  script: "your ES5 script here",\n  description: "Clear description",\n  autoConfirm: true  // ⚠️ User enabled auto-confirm mode\n})\n```' : options.autoConfirm === false ? '🔒 **FORCE CONFIRM MODE**: All background scripts will require user confirmation, even simple ones.' : '🤚 **DEFAULT MODE**: Background scripts will ask for user confirmation based on risk level.';
-    }
-    #;
-    #;
-    #;
-    #;
-    Current;
-    MCP;
-    Tools;
-    Available(Snow - Flow, v3, .3, .4);
-    $;
-    {
-        isAuthenticated ? '✅ Authentication detected - full deployment capabilities' : '⚠️ No authentication detected - MCP tools will provide specific instructions if auth needed';
-    }
-    Your;
-    agents;
-    MUST;
-    use;
-    these;
-    MCP;
-    tools;
-    IN;
-    THIS;
-    ORDER: ;
-     ** PRE - FLIGHT;
-    CHECKS ** (Always);
-    do
-        first;
-    while ();
-    1.;
-    `snow_auth_diagnostics\` - Test authentication and permissions
+**CRITICAL ES5 RULES:**
+- NO const/let (use var)
+- NO arrow functions (use function())
+- NO template literals (use string concatenation) 
+- NO destructuring (use explicit property access)
+- NO for...of loops (use traditional for loops)
+- NO default parameters (use typeof checks)
+
+**🔥 If you use ES6+ syntax, the script WILL FAIL with SyntaxError!**
+**See CLAUDE.md for complete ES5 examples and common mistake fixes.**
+
+### 📝 Background Script Execution Settings
+${options.autoConfirm ? '⚠️ **AUTO-CONFIRM MODE ENABLED**: When calling snow_execute_background_script, ALWAYS add autoConfirm: true parameter to skip user confirmation.\n```javascript\nsnow_execute_background_script({\n  script: "your ES5 script here",\n  description: "Clear description",\n  autoConfirm: true  // ⚠️ User enabled auto-confirm mode\n})\n```' : options.autoConfirm === false ? '🔒 **FORCE CONFIRM MODE**: All background scripts will require user confirmation, even simple ones.' : '🤚 **DEFAULT MODE**: Background scripts will ask for user confirmation based on risk level.'}
+
+#### Current MCP Tools Available (Snow-Flow v3.3.4)
+${isAuthenticated ? '✅ Authentication detected - full deployment capabilities' : '⚠️ No authentication detected - MCP tools will provide specific instructions if auth needed'}
+
+Your agents MUST use these MCP tools IN THIS ORDER:
+
+🔍 **PRE-FLIGHT CHECKS** (Always do first!):
+1. \`snow_auth_diagnostics\` - Test authentication and permissions
 2. If auth fails, the tool provides specific instructions
 3. Continue with appropriate strategy based on auth status
 

package/dist/mcp/servicenow-automation-mcp.js

@@ -12,7 +12,7 @@ const types_js_1 = require("@modelcontextprotocol/sdk/types.js");
 const servicenow_client_js_1 = require("../utils/servicenow-client.js");
 const mcp_auth_middleware_js_1 = require("../utils/mcp-auth-middleware.js");
 const mcp_config_manager_js_1 = require("../utils/mcp-config-manager.js");
-const logger_js_1 = require("../utils/logger.js");
+const mcp_logger_js_1 = require("./shared/mcp-logger.js");
 class ServiceNowAutomationMCP {
     constructor() {
         this.server = new index_js_1.Server({
@@ -24,7 +24,7 @@ class ServiceNowAutomationMCP {
             },
         });
         this.client = new servicenow_client_js_1.ServiceNowClient();
-        this.logger = new logger_js_1.Logger('ServiceNowAutomationMCP');
+        this.logger = new mcp_logger_js_1.MCPLogger('ServiceNowAutomationMCP');
         this.config = mcp_config_manager_js_1.mcpConfig.getConfig();
         this.setupHandlers();
     }
@@ -423,68 +423,100 @@ class ServiceNowAutomationMCP {
             ]
         }));
         this.server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
+            const { name, arguments: args } = request.params;
             try {
-                const { name, arguments: args } = request.params;
+                // Start operation with token tracking
+                this.logger.operationStart(name, args);
                 const authResult = await mcp_auth_middleware_js_1.mcpAuth.ensureAuthenticated();
                 if (!authResult.success) {
                     throw new types_js_1.McpError(types_js_1.ErrorCode.InternalError, authResult.error || 'Authentication required');
                 }
+                let result;
                 switch (name) {
                     case 'snow_create_scheduled_job':
-                        return await this.createScheduledJob(args);
+                        result = await this.createScheduledJob(args);
+                        break;
                     case 'snow_create_event_rule':
-                        return await this.createEventRule(args);
+                        result = await this.createEventRule(args);
+                        break;
                     case 'snow_create_notification':
-                        return await this.createNotification(args);
+                        result = await this.createNotification(args);
+                        break;
                     case 'snow_create_sla_definition':
-                        return await this.createSLADefinition(args);
+                        result = await this.createSLADefinition(args);
+                        break;
                     case 'snow_create_escalation_rule':
-                        return await this.createEscalationRule(args);
+                        result = await this.createEscalationRule(args);
+                        break;
                     case 'snow_create_workflow_activity':
-                        return await this.createWorkflowActivity(args);
+                        result = await this.createWorkflowActivity(args);
+                        break;
                     case 'snow_discover_schedules':
-                        return await this.discoverSchedules(args);
+                        result = await this.discoverSchedules(args);
+                        break;
                     case 'snow_discover_events':
-                        return await this.discoverEvents(args);
+                        result = await this.discoverEvents(args);
+                        break;
                     case 'snow_discover_automation_jobs':
-                        return await this.discoverAutomationJobs(args);
+                        result = await this.discoverAutomationJobs(args);
+                        break;
                     case 'snow_test_scheduled_job':
-                        return await this.testScheduledJob(args);
+                        result = await this.testScheduledJob(args);
+                        break;
                     case 'snow_execute_background_script':
-                        return await this.executeBackgroundScript(args);
+                        result = await this.executeBackgroundScript(args);
+                        break;
                     case 'snow_confirm_script_execution':
-                        return await this.confirmScriptExecution(args);
+                        result = await this.confirmScriptExecution(args);
+                        break;
                     case 'snow_create_atf_test':
-                        return await this.createATFTest(args);
+                        result = await this.createATFTest(args);
+                        break;
                     case 'snow_create_atf_test_step':
-                        return await this.createATFTestStep(args);
+                        result = await this.createATFTestStep(args);
+                        break;
                     case 'snow_execute_atf_test':
-                        return await this.executeATFTest(args);
+                        result = await this.executeATFTest(args);
+                        break;
                     case 'snow_get_atf_results':
-                        return await this.getATFResults(args);
+                        result = await this.getATFResults(args);
+                        break;
                     case 'snow_create_atf_test_suite':
-                        return await this.createATFTestSuite(args);
+                        result = await this.createATFTestSuite(args);
+                        break;
                     case 'snow_discover_atf_tests':
-                        return await this.discoverATFTests(args);
+                        result = await this.discoverATFTests(args);
+                        break;
                     case 'snow_execute_script_with_output':
-                        return await this.executeScriptWithOutput(args);
+                        result = await this.executeScriptWithOutput(args);
+                        break;
                     case 'snow_get_script_output':
-                        return await this.getScriptOutput(args);
+                        result = await this.getScriptOutput(args);
+                        break;
                     case 'snow_execute_script_sync':
-                        return await this.executeScriptSync(args);
+                        result = await this.executeScriptSync(args);
+                        break;
                     case 'snow_get_logs':
-                        return await this.getLogs(args);
+                        result = await this.getLogs(args);
+                        break;
                     case 'snow_test_rest_connection':
-                        return await this.testRESTConnection(args);
+                        result = await this.testRESTConnection(args);
+                        break;
                     case 'snow_rest_message_test_suite':
-                        return await this.restMessageTestSuite(args);
+                        result = await this.restMessageTestSuite(args);
+                        break;
                     case 'snow_property_manager':
-                        return await this.propertyManager(args);
+                        result = await this.propertyManager(args);
+                        break;
                     case 'snow_trace_execution':
-                        return await this.traceExecution(args);
+                        result = await this.traceExecution(args);
+                        break;
                     default:
                         throw new types_js_1.McpError(types_js_1.ErrorCode.MethodNotFound, `Unknown tool: ${name}`);
                 }
+                // Complete operation with token tracking
+                this.logger.operationComplete(name, result);
+                return result;
             }
             catch (error) {
                 this.logger.error(`Error in ${request.params.name}:`, error);
@@ -513,6 +545,7 @@ class ServiceNowAutomationMCP {
                 ...schedulePattern
             };
             const updateSetResult = await this.client.ensureUpdateSet();
+            this.logger.trackAPICall('CREATE', 'sysauto_script', 1);
             const response = await this.client.createRecord('sysauto_script', jobData);
             if (!response.success) {
                 throw new Error(`Failed to create Scheduled Job: ${response.error}`);
@@ -1210,6 +1243,7 @@ Reply with:
                 description: `Background script executed via Snow-Flow MCP - Execution ID: ${executionId}`
             };
             // Execute script using sys_script table (Background Scripts)
+            this.logger.trackAPICall('CREATE', 'sys_script', 1);
             const scriptResponse = await this.client.createRecord('sys_script', executionRecord);
             if (!scriptResponse.success) {
                 throw new Error(`Failed to create background script execution record: ${scriptResponse.error}`);
@@ -1971,6 +2005,7 @@ ${groupedResults.suites.slice(0, 10).map(suite => `- ${suite.name} ${suite.activ
             }
             // Remove trailing ^  
             query = query.replace(/\^$/, '');
+            this.logger.trackAPICall('SEARCH', 'syslog_transaction', limit);
             const logsResponse = await this.client.searchRecords('syslog_transaction', query, limit);
             if (logsResponse.success && logsResponse.data?.result?.length > 0) {
                 const logs = logsResponse.data.result.map((log) => ({
@@ -2171,6 +2206,7 @@ ${groupedResults.suites.slice(0, 10).map(suite => `- ${suite.name} ${suite.activ
             const action = args.action || 'list';
             switch (action) {
                 case 'get':
+                    this.logger.trackAPICall('SEARCH', 'sys_properties', 1);
                     const getPropResponse = await this.client.searchRecords('sys_properties', `name=${args.name}`, 1);
                     if (getPropResponse.success && getPropResponse.data?.result?.length > 0) {
                         const prop = getPropResponse.data.result[0];

package/dist/mcp/servicenow-change-virtualagent-pa-mcp.js

@@ -12,7 +12,7 @@ const types_js_1 = require("@modelcontextprotocol/sdk/types.js");
 const servicenow_client_js_1 = require("../utils/servicenow-client.js");
 const mcp_auth_middleware_js_1 = require("../utils/mcp-auth-middleware.js");
 const mcp_config_manager_js_1 = require("../utils/mcp-config-manager.js");
-const logger_js_1 = require("../utils/logger.js");
+const mcp_logger_js_1 = require("../shared/mcp-logger.js");
 class ServiceNowChangeVirtualAgentPAMCP {
     constructor() {
         this.server = new index_js_1.Server({
@@ -24,7 +24,7 @@ class ServiceNowChangeVirtualAgentPAMCP {
             },
         });
         this.client = new servicenow_client_js_1.ServiceNowClient();
-        this.logger = new logger_js_1.Logger('ServiceNowChangeVirtualAgentPAMCP');
+        this.logger = new mcp_logger_js_1.MCPLogger('ServiceNowChangeVirtualAgentPAMCP');
         this.config = mcp_config_manager_js_1.mcpConfig.getConfig();
         this.setupHandlers();
     }
@@ -337,55 +337,80 @@ class ServiceNowChangeVirtualAgentPAMCP {
         this.server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
             try {
                 const { name, arguments: args } = request.params;
+                // Start operation with token tracking
+                this.logger.operationStart(name, args);
                 const authResult = await mcp_auth_middleware_js_1.mcpAuth.ensureAuthenticated();
                 if (!authResult.success) {
                     throw new types_js_1.McpError(types_js_1.ErrorCode.InternalError, authResult.error || 'Authentication required');
                 }
+                let result;
                 switch (name) {
                     // Change Management
                     case 'snow_create_change_request':
-                        return await this.createChangeRequest(args);
+                        result = await this.createChangeRequest(args);
+                        break;
                     case 'snow_create_change_task':
-                        return await this.createChangeTask(args);
+                        result = await this.createChangeTask(args);
+                        break;
                     case 'snow_get_change_request':
-                        return await this.getChangeRequest(args);
+                        result = await this.getChangeRequest(args);
+                        break;
                     case 'snow_update_change_state':
-                        return await this.updateChangeState(args);
+                        result = await this.updateChangeState(args);
+                        break;
                     case 'snow_schedule_cab_meeting':
-                        return await this.scheduleCABMeeting(args);
+                        result = await this.scheduleCABMeeting(args);
+                        break;
                     case 'snow_search_change_requests':
-                        return await this.searchChangeRequests(args);
+                        result = await this.searchChangeRequests(args);
+                        break;
                     // Virtual Agent
                     case 'snow_create_va_topic':
-                        return await this.createVATopic(args);
+                        result = await this.createVATopic(args);
+                        break;
                     case 'snow_create_va_topic_block':
-                        return await this.createVATopicBlock(args);
+                        result = await this.createVATopicBlock(args);
+                        break;
                     case 'snow_get_va_conversation':
-                        return await this.getVAConversation(args);
+                        result = await this.getVAConversation(args);
+                        break;
                     case 'snow_send_va_message':
-                        return await this.sendVAMessage(args);
+                        result = await this.sendVAMessage(args);
+                        break;
                     case 'snow_handoff_to_agent':
-                        return await this.handoffToAgent(args);
+                        result = await this.handoffToAgent(args);
+                        break;
                     case 'snow_discover_va_topics':
-                        return await this.discoverVATopics(args);
+                        result = await this.discoverVATopics(args);
+                        break;
                     // Performance Analytics
                     case 'snow_create_pa_indicator':
-                        return await this.createPAIndicator(args);
+                        result = await this.createPAIndicator(args);
+                        break;
                     case 'snow_create_pa_widget':
-                        return await this.createPAWidget(args);
+                        result = await this.createPAWidget(args);
+                        break;
                     case 'snow_create_pa_breakdown':
-                        return await this.createPABreakdown(args);
+                        result = await this.createPABreakdown(args);
+                        break;
                     case 'snow_get_pa_scores':
-                        return await this.getPAScores(args);
+                        result = await this.getPAScores(args);
+                        break;
                     case 'snow_create_pa_threshold':
-                        return await this.createPAThreshold(args);
+                        result = await this.createPAThreshold(args);
+                        break;
                     case 'snow_collect_pa_data':
-                        return await this.collectPAData(args);
+                        result = await this.collectPAData(args);
+                        break;
                     case 'snow_discover_pa_indicators':
-                        return await this.discoverPAIndicators(args);
+                        result = await this.discoverPAIndicators(args);
+                        break;
                     default:
                         throw new types_js_1.McpError(types_js_1.ErrorCode.MethodNotFound, `Unknown tool: ${name}`);
                 }
+                // Complete operation with token tracking
+                this.logger.operationComplete(name, result);
+                return result;
             }
             catch (error) {
                 this.logger.error(`Error in ${request.params.name}:`, error);
@@ -416,6 +441,7 @@ class ServiceNowChangeVirtualAgentPAMCP {
                 state: '-5' // New state
             };
             const updateSetResult = await this.client.ensureUpdateSet();
+            this.logger.trackAPICall('CREATE', 'change_request', 1);
             const response = await this.client.createRecord('change_request', changeData);
             if (!response.success) {
                 throw new Error(`Failed to create change request: ${response.error}`);
@@ -464,6 +490,7 @@ class ServiceNowChangeVirtualAgentPAMCP {
                 planned_end_date: args.planned_end_date || '',
                 order: args.order || 100
             };
+            this.logger.trackAPICall('CREATE', 'change_task', 1);
             const response = await this.client.createRecord('change_task', taskData);
             if (!response.success) {
                 throw new Error(`Failed to create change task: ${response.error}`);
@@ -654,6 +681,7 @@ ${args.close_code ? `✅ Close Code: ${args.close_code}` : ''}
                 query += `assigned_to=${args.assigned_to}`;
             }
             const limit = args.limit || 20;
+            this.logger.trackAPICall('SEARCH', 'change_request', limit);
             const response = await this.client.searchRecords('change_request', query, limit);
             if (!response.success) {
                 throw new Error('Failed to search change requests');
@@ -699,6 +727,7 @@ ${changeList}
                 active: args.active !== false,
                 live_agent_enabled: args.live_agent_enabled || false
             };
+            this.logger.trackAPICall('CREATE', 'sys_cs_topic', 1);
             const response = await this.client.createRecord('sys_cs_topic', topicData);
             if (!response.success) {
                 throw new Error(`Failed to create VA topic: ${response.error}`);
@@ -951,6 +980,7 @@ ${topicList}
                 target: args.target || 0,
                 thresholds: args.thresholds ? JSON.stringify(args.thresholds) : ''
             };
+            this.logger.trackAPICall('CREATE', 'pa_indicators', 1);
             const response = await this.client.createRecord('pa_indicators', indicatorData);
             if (!response.success) {
                 throw new Error(`Failed to create PA indicator: ${response.error}`);

package/dist/mcp/servicenow-cmdb-event-hr-csm-devops-mcp.js

@@ -12,7 +12,7 @@ const types_js_1 = require("@modelcontextprotocol/sdk/types.js");
 const servicenow_client_js_1 = require("../utils/servicenow-client.js");
 const mcp_auth_middleware_js_1 = require("../utils/mcp-auth-middleware.js");
 const mcp_config_manager_js_1 = require("../utils/mcp-config-manager.js");
-const logger_js_1 = require("../utils/logger.js");
+const mcp_logger_js_1 = require("../shared/mcp-logger.js");
 class ServiceNowCMDBEventHRCSMDevOpsMCP {
     constructor() {
         this.server = new index_js_1.Server({
@@ -24,7 +24,7 @@ class ServiceNowCMDBEventHRCSMDevOpsMCP {
             },
         });
         this.client = new servicenow_client_js_1.ServiceNowClient();
-        this.logger = new logger_js_1.Logger('ServiceNowCMDBEventHRCSMDevOpsMCP');
+        this.logger = new mcp_logger_js_1.MCPLogger('ServiceNowCMDBEventHRCSMDevOpsMCP');
         this.config = mcp_config_manager_js_1.mcpConfig.getConfig();
         this.setupHandlers();
     }
@@ -406,65 +406,94 @@ class ServiceNowCMDBEventHRCSMDevOpsMCP {
         this.server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
             try {
                 const { name, arguments: args } = request.params;
+                // Start operation with token tracking
+                this.logger.operationStart(name, args);
                 const authResult = await mcp_auth_middleware_js_1.mcpAuth.ensureAuthenticated();
                 if (!authResult.success) {
                     throw new types_js_1.McpError(types_js_1.ErrorCode.InternalError, authResult.error || 'Authentication required');
                 }
+                let result;
                 switch (name) {
                     // CMDB & Discovery
                     case 'snow_create_ci':
-                        return await this.createCI(args);
+                        result = await this.createCI(args);
+                        break;
                     case 'snow_create_ci_relationship':
-                        return await this.createCIRelationship(args);
+                        result = await this.createCIRelationship(args);
+                        break;
                     case 'snow_run_discovery':
-                        return await this.runDiscovery(args);
+                        result = await this.runDiscovery(args);
+                        break;
                     case 'snow_get_ci_details':
-                        return await this.getCIDetails(args);
+                        result = await this.getCIDetails(args);
+                        break;
                     case 'snow_search_cmdb':
-                        return await this.searchCMDB(args);
+                        result = await this.searchCMDB(args);
+                        break;
                     case 'snow_impact_analysis':
-                        return await this.impactAnalysis(args);
+                        result = await this.impactAnalysis(args);
+                        break;
                     // Event Management
                     case 'snow_create_event':
-                        return await this.createEvent(args);
+                        result = await this.createEvent(args);
+                        break;
                     case 'snow_create_alert':
-                        return await this.createAlert(args);
+                        result = await this.createAlert(args);
+                        break;
                     case 'snow_create_alert_rule':
-                        return await this.createAlertRule(args);
+                        result = await this.createAlertRule(args);
+                        break;
                     case 'snow_get_event_correlation':
-                        return await this.getEventCorrelation(args);
+                        result = await this.getEventCorrelation(args);
+                        break;
                     // HR Service Delivery
                     case 'snow_create_hr_case':
-                        return await this.createHRCase(args);
+                        result = await this.createHRCase(args);
+                        break;
                     case 'snow_create_hr_task':
-                        return await this.createHRTask(args);
+                        result = await this.createHRTask(args);
+                        break;
                     case 'snow_employee_onboarding':
-                        return await this.employeeOnboarding(args);
+                        result = await this.employeeOnboarding(args);
+                        break;
                     case 'snow_employee_offboarding':
-                        return await this.employeeOffboarding(args);
+                        result = await this.employeeOffboarding(args);
+                        break;
                     // Customer Service Management
                     case 'snow_create_customer_case':
-                        return await this.createCustomerCase(args);
+                        result = await this.createCustomerCase(args);
+                        break;
                     case 'snow_create_customer_account':
-                        return await this.createCustomerAccount(args);
+                        result = await this.createCustomerAccount(args);
+                        break;
                     case 'snow_create_entitlement':
-                        return await this.createEntitlement(args);
+                        result = await this.createEntitlement(args);
+                        break;
                     case 'snow_get_customer_history':
-                        return await this.getCustomerHistory(args);
+                        result = await this.getCustomerHistory(args);
+                        break;
                     // DevOps
                     case 'snow_create_devops_pipeline':
-                        return await this.createDevOpsPipeline(args);
+                        result = await this.createDevOpsPipeline(args);
+                        break;
                     case 'snow_track_deployment':
-                        return await this.trackDeployment(args);
+                        result = await this.trackDeployment(args);
+                        break;
                     case 'snow_create_devops_change':
-                        return await this.createDevOpsChange(args);
+                        result = await this.createDevOpsChange(args);
+                        break;
                     case 'snow_get_devops_insights':
-                        return await this.getDevOpsInsights(args);
+                        result = await this.getDevOpsInsights(args);
+                        break;
                     case 'snow_velocity_tracking':
-                        return await this.velocityTracking(args);
+                        result = await this.velocityTracking(args);
+                        break;
                     default:
                         throw new types_js_1.McpError(types_js_1.ErrorCode.MethodNotFound, `Unknown tool: ${name}`);
                 }
+                // Complete operation with token tracking
+                this.logger.operationComplete(name, result);
+                return result;
             }
             catch (error) {
                 this.logger.error(`Error in ${request.params.name}:`, error);
@@ -489,6 +518,7 @@ class ServiceNowCMDBEventHRCSMDevOpsMCP {
                 ...args.attributes
             };
             const updateSetResult = await this.client.ensureUpdateSet();
+            this.logger.trackAPICall('CREATE', ciTable, 1);
             const response = await this.client.createRecord(ciTable, ciData);
             if (!response.success) {
                 throw new Error(`Failed to create CI: ${response.error}`);
@@ -524,6 +554,7 @@ ${args.asset_tag ? `🏷️ Asset Tag: ${args.asset_tag}` : ''}
                 type: args.type,
                 description: args.description || ''
             };
+            this.logger.trackAPICall('CREATE', 'cmdb_rel_ci', 1);
             const response = await this.client.createRecord('cmdb_rel_ci', relationshipData);
             if (!response.success) {
                 throw new Error(`Failed to create CI relationship: ${response.error}`);
@@ -645,6 +676,7 @@ ${args.patterns ? `📋 Patterns: ${args.patterns.join(', ')}` : ''}
                 query += `location=${args.location}`;
             }
             const limit = args.limit || 50;
+            this.logger.trackAPICall('SEARCH', 'cmdb_ci', limit);
             const response = await this.client.searchRecords('cmdb_ci', query, limit);
             if (!response.success) {
                 throw new Error('Failed to search CMDB');
@@ -736,6 +768,7 @@ ${mockImpact.recommendations.map(r => `  ✓ ${r}`).join('\n')}
                 time_of_event: args.time_of_event || new Date().toISOString(),
                 resolution_state: args.resolution_state || 'New'
             };
+            this.logger.trackAPICall('CREATE', 'em_event', 1);
             const response = await this.client.createRecord('em_event', eventData);
             if (!response.success) {
                 throw new Error(`Failed to create event: ${response.error}`);
@@ -887,6 +920,7 @@ ${mockCorrelation.correlation_rules_applied.map(r => `  - ${r}`).join('\n')}
                 subcategory: args.subcategory || '',
                 confidential: args.confidential || false
             };
+            this.logger.trackAPICall('CREATE', 'sn_hr_core_case', 1);
             const response = await this.client.createRecord('sn_hr_core_case', caseData);
             if (!response.success) {
                 throw new Error(`Failed to create HR case: ${response.error}`);