snow-flow 3.4.11 → 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({
@@ -852,6 +852,20 @@ ${hasIntelligentFeatures ? `✅ **INTELLIGENT MODE ACTIVE** - The following feat
 
 **CRITICAL RULE**: All agents MUST attempt to use ServiceNow MCP tools first, regardless of authentication status.
 
+### 🚨 MANDATORY: ES5 JavaScript Only for ALL ServiceNow Scripts
+**⚠️ SERVICENOW RHINO ENGINE = ES5 ONLY - NO MODERN SYNTAX!**
+
+**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.'}
 

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();
     }
@@ -38,7 +38,7 @@ class ServiceNowAutomationMCP {
                         type: 'object',
                         properties: {
                             name: { type: 'string', description: 'Scheduled Job name' },
-                            script: { type: 'string', description: 'JavaScript code to execute' },
+                            script: { type: 'string', description: '🚨 ES5 ONLY! JavaScript code to execute (no const/let/arrows/templates - Rhino engine)' },
                             description: { type: 'string', description: 'Job description' },
                             schedule: { type: 'string', description: 'Schedule pattern (daily, weekly, monthly, or cron)' },
                             active: { type: 'boolean', description: 'Job active status' },
@@ -56,8 +56,8 @@ class ServiceNowAutomationMCP {
                         properties: {
                             name: { type: 'string', description: 'Event Rule name' },
                             eventName: { type: 'string', description: 'Event name to listen for' },
-                            condition: { type: 'string', description: 'Event condition script' },
-                            script: { type: 'string', description: 'Action script to execute' },
+                            condition: { type: 'string', description: 'Event condition script (ES5 only!)' },
+                            script: { type: 'string', description: '🚨 ES5 ONLY! Action script to execute (no const/let/arrows/templates - Rhino engine)' },
                             description: { type: 'string', description: 'Rule description' },
                             active: { type: 'boolean', description: 'Rule active status' },
                             order: { type: 'number', description: 'Execution order' }
@@ -179,13 +179,13 @@ class ServiceNowAutomationMCP {
                 },
                 {
                     name: 'snow_execute_background_script',
-                    description: '🚨 REQUIRES USER CONFIRMATION (unless autoConfirm=true): Executes a JavaScript background script in ServiceNow. Script runs in server-side context with full API access. By default asks for user approval.',
+                    description: '🚨 REQUIRES USER CONFIRMATION (unless autoConfirm=true): Executes JavaScript background script in ServiceNow. ⚠️ CRITICAL: Script MUST be ES5-only (no const/let/arrow functions/template literals) - ES6+ will cause SyntaxError on Rhino engine!',
                     inputSchema: {
                         type: 'object',
                         properties: {
                             script: {
                                 type: 'string',
-                                description: 'JavaScript code to execute in background. Has access to GlideRecord, GlideAggregate, gs, etc.'
+                                description: '🚨 ES5 JAVASCRIPT ONLY! Use var (not const/let), function(){} (not arrows), string concatenation (not templates). Has access to GlideRecord, GlideAggregate, gs, etc.'
                             },
                             description: {
                                 type: 'string',
@@ -308,11 +308,11 @@ class ServiceNowAutomationMCP {
                 },
                 {
                     name: 'snow_execute_script_with_output',
-                    description: 'Executes a background script and retrieves the actual output. Waits for execution to complete and returns the results.',
+                    description: 'Executes a background script and retrieves the actual output. ⚠️ CRITICAL: Script MUST be ES5-only (Rhino engine) - no const/let/arrows/templates!',
                     inputSchema: {
                         type: 'object',
                         properties: {
-                            script: { type: 'string', description: 'JavaScript code to execute' },
+                            script: { type: 'string', description: '🚨 ES5 ONLY! Use var, function(){}, string+concatenation. JavaScript code to execute on ServiceNow server.' },
                             return_output: { type: 'boolean', description: 'Return script output', default: true },
                             max_wait: { type: 'number', description: 'Maximum wait time in milliseconds', default: 5000 },
                             capture_logs: { type: 'boolean', description: 'Capture system logs during execution', default: true }
@@ -333,11 +333,11 @@ class ServiceNowAutomationMCP {
                 },
                 {
                     name: 'snow_execute_script_sync',
-                    description: 'Synchronously executes a script and waits for the result. Returns output immediately.',
+                    description: 'Synchronously executes a script and waits for the result. ⚠️ CRITICAL: Script MUST be ES5-only (Rhino engine)!',
                     inputSchema: {
                         type: 'object',
                         properties: {
-                            script: { type: 'string', description: 'JavaScript code to execute' },
+                            script: { type: 'string', description: '🚨 ES5 ONLY! No const/let/arrows/templates. JavaScript code to execute synchronously.' },
                             timeout: { type: 'number', description: 'Timeout in milliseconds', default: 3000 },
                             capture_output: { type: 'boolean', description: 'Capture and return output', default: true }
                         },
@@ -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}`);