snow-flow 3.4.13 → 3.4.14

package/dist/mcp/servicenow-system-properties-mcp.js

@@ -11,9 +11,8 @@ const index_js_1 = require("@modelcontextprotocol/sdk/server/index.js");
 const stdio_js_1 = require("@modelcontextprotocol/sdk/server/stdio.js");
 const types_js_1 = require("@modelcontextprotocol/sdk/types.js");
 const servicenow_client_js_1 = require("../utils/servicenow-client.js");
-const logger_js_1 = require("../utils/logger.js");
+const mcp_logger_js_1 = require("./shared/mcp-logger.js");
 const snow_oauth_js_1 = require("../utils/snow-oauth.js");
-const logger = new logger_js_1.Logger('ServiceNowSystemProperties');
 /**
  * ServiceNow System Properties MCP Server
  * Manages system properties through official ServiceNow REST APIs
@@ -31,6 +30,7 @@ class ServiceNowSystemPropertiesMCP {
         });
         this.client = new servicenow_client_js_1.ServiceNowClient();
         this.oauth = new snow_oauth_js_1.SnowOAuth();
+        this.logger = new mcp_logger_js_1.MCPLogger('ServiceNowSystemProperties');
         this.setupHandlers();
         this.setupTools();
     }
@@ -319,42 +319,60 @@ class ServiceNowSystemPropertiesMCP {
         this.server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
             const { name, arguments: args } = request.params;
             try {
+                // Start operation with token tracking
+                this.logger.operationStart(name, args);
                 // Ensure authentication
                 const isAuthenticated = await this.oauth.isAuthenticated();
                 if (!isAuthenticated) {
                     throw new types_js_1.McpError(types_js_1.ErrorCode.InvalidRequest, 'Not authenticated. Please run "snow-flow auth login" first.');
                 }
+                let result;
                 switch (name) {
                     case 'snow_property_get':
-                        return await this.getProperty(args);
+                        result = await this.getProperty(args);
+                        break;
                     case 'snow_property_set':
-                        return await this.setProperty(args);
+                        result = await this.setProperty(args);
+                        break;
                     case 'snow_property_list':
-                        return await this.listProperties(args);
+                        result = await this.listProperties(args);
+                        break;
                     case 'snow_property_delete':
-                        return await this.deleteProperty(args);
+                        result = await this.deleteProperty(args);
+                        break;
                     case 'snow_property_search':
-                        return await this.searchProperties(args);
+                        result = await this.searchProperties(args);
+                        break;
                     case 'snow_property_bulk_get':
-                        return await this.bulkGetProperties(args);
+                        result = await this.bulkGetProperties(args);
+                        break;
                     case 'snow_property_bulk_set':
-                        return await this.bulkSetProperties(args);
+                        result = await this.bulkSetProperties(args);
+                        break;
                     case 'snow_property_export':
-                        return await this.exportProperties(args);
+                        result = await this.exportProperties(args);
+                        break;
                     case 'snow_property_import':
-                        return await this.importProperties(args);
+                        result = await this.importProperties(args);
+                        break;
                     case 'snow_property_validate':
-                        return await this.validateProperty(args);
+                        result = await this.validateProperty(args);
+                        break;
                     case 'snow_property_categories':
-                        return await this.getCategories(args);
+                        result = await this.getCategories(args);
+                        break;
                     case 'snow_property_history':
-                        return await this.getPropertyHistory(args);
+                        result = await this.getPropertyHistory(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) {
-                logger.error(`Tool execution failed: ${name}`, error);
+                this.logger.error(`Tool execution failed: ${name}`, error);
                 throw new types_js_1.McpError(types_js_1.ErrorCode.InternalError, error instanceof Error ? error.message : String(error));
             }
         });
@@ -364,8 +382,9 @@ class ServiceNowSystemPropertiesMCP {
      */
     async getProperty(args) {
         const { name, include_metadata = false } = args;
-        logger.info(`Getting property: ${name}`);
+        this.logger.info(`Getting property: ${name}`);
         try {
+            this.logger.trackAPICall('SEARCH', 'sys_properties', 1);
             const response = await this.client.searchRecords('sys_properties', `name=${name}`, 1);
             if (!response.success || !response.data?.result?.length) {
                 return {
@@ -406,7 +425,7 @@ class ServiceNowSystemPropertiesMCP {
             }
         }
         catch (error) {
-            logger.error('Failed to get property:', error);
+            this.logger.error('Failed to get property:', error);
             throw error;
         }
     }
@@ -415,14 +434,16 @@ class ServiceNowSystemPropertiesMCP {
      */
     async setProperty(args) {
         const { name, value, description, type = 'string', choices, is_private = false, suffix } = args;
-        logger.info(`Setting property: ${name} = ${value}`);
+        this.logger.info(`Setting property: ${name} = ${value}`);
         try {
             // Check if property exists
+            this.logger.trackAPICall('SEARCH', 'sys_properties', 1);
             const existing = await this.client.searchRecords('sys_properties', `name=${name}`, 1);
             let result;
             if (existing.success && existing.data?.result?.length > 0) {
                 // Update existing property
                 const sys_id = existing.data.result[0].sys_id;
+                this.logger.trackAPICall('UPDATE', 'sys_properties', 1);
                 result = await this.client.updateRecord('sys_properties', sys_id, {
                     value,
                     ...(description && { description }),
@@ -431,10 +452,11 @@ class ServiceNowSystemPropertiesMCP {
                     ...(suffix && { suffix }),
                     is_private: is_private ? 'true' : 'false'
                 });
-                logger.info(`Updated property: ${name}`);
+                this.logger.info(`Updated property: ${name}`);
             }
             else {
                 // Create new property
+                this.logger.trackAPICall('CREATE', 'sys_properties', 1);
                 result = await this.client.createRecord('sys_properties', {
                     name,
                     value,
@@ -444,7 +466,7 @@ class ServiceNowSystemPropertiesMCP {
                     is_private: is_private ? 'true' : 'false',
                     suffix: suffix || 'global'
                 });
-                logger.info(`Created new property: ${name}`);
+                this.logger.info(`Created new property: ${name}`);
             }
             if (!result.success) {
                 throw new Error(`Failed to set property: ${result.error}`);
@@ -468,7 +490,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
             };
         }
         catch (error) {
-            logger.error('Failed to set property:', error);
+            this.logger.error('Failed to set property:', error);
             throw error;
         }
     }
@@ -477,7 +499,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
      */
     async listProperties(args) {
         const { pattern, category, is_private, limit = 100, include_values = true } = args;
-        logger.info('Listing properties', { pattern, category, limit });
+        this.logger.info('Listing properties', { pattern, category, limit });
         try {
             let query = '';
             const conditions = [];
@@ -535,7 +557,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
             };
         }
         catch (error) {
-            logger.error('Failed to list properties:', error);
+            this.logger.error('Failed to list properties:', error);
             throw error;
         }
     }
@@ -556,7 +578,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
                     }]
             };
         }
-        logger.info(`Deleting property: ${name}`);
+        this.logger.info(`Deleting property: ${name}`);
         try {
             // Find the property
             const response = await this.client.searchRecords('sys_properties', `name=${name}`, 1);
@@ -585,7 +607,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
             };
         }
         catch (error) {
-            logger.error('Failed to delete property:', error);
+            this.logger.error('Failed to delete property:', error);
             throw error;
         }
     }
@@ -594,7 +616,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
      */
     async searchProperties(args) {
         const { search_term, search_in = 'all', limit = 50 } = args;
-        logger.info(`Searching properties for: ${search_term}`);
+        this.logger.info(`Searching properties for: ${search_term}`);
         try {
             let query = '';
             switch (search_in) {
@@ -642,7 +664,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
             };
         }
         catch (error) {
-            logger.error('Search failed:', error);
+            this.logger.error('Search failed:', error);
             throw error;
         }
     }
@@ -651,7 +673,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
      */
     async bulkGetProperties(args) {
         const { names, include_metadata = false } = args;
-        logger.info(`Bulk getting ${names.length} properties`);
+        this.logger.info(`Bulk getting ${names.length} properties`);
         const results = {};
         const errors = [];
         for (const name of names) {
@@ -673,7 +695,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
                 }
             }
             catch (error) {
-                logger.error(`Failed to get property ${name}:`, error);
+                this.logger.error(`Failed to get property ${name}:`, error);
                 results[name] = null;
                 errors.push(name);
             }
@@ -702,7 +724,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
      */
     async bulkSetProperties(args) {
         const { properties } = args;
-        logger.info(`Bulk setting ${properties.length} properties`);
+        this.logger.info(`Bulk setting ${properties.length} properties`);
         const results = {
             created: [],
             updated: [],
@@ -730,6 +752,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
                 }
                 else {
                     // Create
+                    this.logger.trackAPICall('CREATE', 'sys_properties', 1);
                     result = await this.client.createRecord('sys_properties', {
                         name: prop.name,
                         value: prop.value,
@@ -747,7 +770,7 @@ ${choices ? `**Choices:** ${choices}` : ''}
                 this.propertyCache.delete(prop.name);
             }
             catch (error) {
-                logger.error(`Failed to set property ${prop.name}:`, error);
+                this.logger.error(`Failed to set property ${prop.name}:`, error);
                 results.failed.push(`${prop.name}: ${error}`);
             }
         }
@@ -773,7 +796,7 @@ Total processed: ${properties.length}`
      */
     async exportProperties(args) {
         const { pattern, include_system = false, include_private = false } = args;
-        logger.info('Exporting properties', { pattern, include_system, include_private });
+        this.logger.info('Exporting properties', { pattern, include_system, include_private });
         try {
             let query = '';
             const conditions = [];
@@ -823,7 +846,7 @@ ${JSON.stringify(exportData, null, 2)}
             };
         }
         catch (error) {
-            logger.error('Export failed:', error);
+            this.logger.error('Export failed:', error);
             throw error;
         }
     }
@@ -832,7 +855,7 @@ ${JSON.stringify(exportData, null, 2)}
      */
     async importProperties(args) {
         const { properties, overwrite = false, dry_run = false } = args;
-        logger.info('Importing properties', { count: Object.keys(properties).length, overwrite, dry_run });
+        this.logger.info('Importing properties', { count: Object.keys(properties).length, overwrite, dry_run });
         const results = {
             would_create: [],
             would_update: [],
@@ -884,6 +907,7 @@ ${JSON.stringify(exportData, null, 2)}
                 }
                 else {
                     // Create
+                    this.logger.trackAPICall('CREATE', 'sys_properties', 1);
                     const result = await this.client.createRecord('sys_properties', {
                         name,
                         value: propertyData.value,
@@ -904,7 +928,7 @@ ${JSON.stringify(exportData, null, 2)}
                 this.propertyCache.delete(name);
             }
             catch (error) {
-                logger.error(`Failed to import property ${name}:`, error);
+                this.logger.error(`Failed to import property ${name}:`, error);
                 results.failed.push(`${name}: ${error}`);
             }
         }
@@ -946,7 +970,7 @@ Total processed: ${Object.keys(properties).length}`
      */
     async validateProperty(args) {
         const { name, value } = args;
-        logger.info(`Validating property: ${name} = ${value}`);
+        this.logger.info(`Validating property: ${name} = ${value}`);
         try {
             // Get property metadata
             const response = await this.client.searchRecords('sys_properties', `name=${name}`, 1);
@@ -1026,7 +1050,7 @@ ${validationResults.join('\n')}
             };
         }
         catch (error) {
-            logger.error('Validation failed:', error);
+            this.logger.error('Validation failed:', error);
             throw error;
         }
     }
@@ -1035,7 +1059,7 @@ ${validationResults.join('\n')}
      */
     async getCategories(args) {
         const { include_counts = true } = args;
-        logger.info('Getting property categories');
+        this.logger.info('Getting property categories');
         try {
             // Get distinct suffixes (categories)
             const response = await this.client.searchRecords('sys_properties', '', 1000);
@@ -1067,7 +1091,7 @@ ${validationResults.join('\n')}
             };
         }
         catch (error) {
-            logger.error('Failed to get categories:', error);
+            this.logger.error('Failed to get categories:', error);
             throw error;
         }
     }
@@ -1076,7 +1100,7 @@ ${validationResults.join('\n')}
      */
     async getPropertyHistory(args) {
         const { name, limit = 10 } = args;
-        logger.info(`Getting history for property: ${name}`);
+        this.logger.info(`Getting history for property: ${name}`);
         try {
             // First, get the property to get its sys_id
             const propResponse = await this.client.searchRecords('sys_properties', `name=${name}`, 1);
@@ -1116,7 +1140,7 @@ ${validationResults.join('\n')}
             };
         }
         catch (error) {
-            logger.error('Failed to get history:', error);
+            this.logger.error('Failed to get history:', error);
             // Audit might not be available
             return {
                 content: [{
@@ -1131,7 +1155,7 @@ Note: Audit history requires sys_audit to be enabled for sys_properties table.`
     async start() {
         const transport = new stdio_js_1.StdioServerTransport();
         await this.server.connect(transport);
-        logger.info('ServiceNow System Properties MCP Server started');
+        this.logger.info('ServiceNow System Properties MCP Server started');
     }
 }
 exports.ServiceNowSystemPropertiesMCP = ServiceNowSystemPropertiesMCP;

package/dist/mcp/servicenow-update-set-mcp.js

@@ -10,7 +10,7 @@ const stdio_js_1 = require("@modelcontextprotocol/sdk/server/stdio.js");
 const types_js_1 = require("@modelcontextprotocol/sdk/types.js");
 const servicenow_client_js_1 = require("../utils/servicenow-client.js");
 const snow_oauth_js_1 = require("../utils/snow-oauth.js");
-const logger_js_1 = require("../utils/logger.js");
+const mcp_logger_js_1 = require("../shared/mcp-logger.js");
 const fs_1 = require("fs");
 const path_1 = require("path");
 class ServiceNowUpdateSetMCP {
@@ -26,7 +26,7 @@ class ServiceNowUpdateSetMCP {
         });
         this.client = new servicenow_client_js_1.ServiceNowClient();
         this.oauth = new snow_oauth_js_1.ServiceNowOAuth();
-        this.logger = new logger_js_1.Logger('ServiceNowUpdateSetMCP');
+        this.logger = new mcp_logger_js_1.MCPLogger('ServiceNowUpdateSetMCP');
         this.sessionsPath = (0, path_1.join)(process.cwd(), 'memory', 'update-set-sessions');
         // Debug: Test credentials on startup
         this.testCredentials();
@@ -227,33 +227,48 @@ class ServiceNowUpdateSetMCP {
         this.server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
             const { name, arguments: args } = request.params;
             try {
+                // Start operation with token tracking
+                this.logger.operationStart(name, args);
                 // Check authentication for all operations
                 const isAuthenticated = await this.oauth.isAuthenticated();
                 if (!isAuthenticated) {
                     throw new types_js_1.McpError(types_js_1.ErrorCode.InvalidRequest, 'Not authenticated. Run "snow-flow auth login" first.');
                 }
+                let result;
                 switch (name) {
                     case 'snow_update_set_create':
-                        return await this.createUpdateSet(args);
+                        result = await this.createUpdateSet(args);
+                        break;
                     case 'snow_update_set_switch':
-                        return await this.switchUpdateSet(args);
+                        result = await this.switchUpdateSet(args);
+                        break;
                     case 'snow_update_set_current':
-                        return await this.getCurrentUpdateSet();
+                        result = await this.getCurrentUpdateSet();
+                        break;
                     case 'snow_update_set_list':
-                        return await this.listUpdateSets(args);
+                        result = await this.listUpdateSets(args);
+                        break;
                     case 'snow_update_set_complete':
-                        return await this.completeUpdateSet(args);
+                        result = await this.completeUpdateSet(args);
+                        break;
                     case 'snow_update_set_add_artifact':
-                        return await this.addArtifactToSession(args);
+                        result = await this.addArtifactToSession(args);
+                        break;
                     case 'snow_update_set_preview':
-                        return await this.previewUpdateSet(args);
+                        result = await this.previewUpdateSet(args);
+                        break;
                     case 'snow_update_set_export':
-                        return await this.exportUpdateSet(args);
+                        result = await this.exportUpdateSet(args);
+                        break;
                     case 'snow_ensure_active_update_set':
-                        return await this.ensureActiveUpdateSet(args);
+                        result = await this.ensureActiveUpdateSet(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) {
                 if (error instanceof types_js_1.McpError)
@@ -267,6 +282,7 @@ class ServiceNowUpdateSetMCP {
         try {
             this.logger.info('Creating new Update Set', args);
             // Create Update Set in ServiceNow
+            this.logger.trackAPICall('CREATE', 'sys_update_set', 1);
             const response = await this.client.createUpdateSet({
                 name: args.name,
                 description: args.description,
@@ -284,6 +300,7 @@ class ServiceNowUpdateSetMCP {
             const autoSwitch = args.auto_switch !== false;
             let switchedToUpdateSet = false;
             if (autoSwitch) {
+                this.logger.trackAPICall('UPDATE', 'sys_update_set', 1);
                 await this.client.setCurrentUpdateSet(response.data.sys_id);
                 switchedToUpdateSet = true;
                 // Create local session
@@ -339,6 +356,7 @@ Use \`snow_update_set_switch\` to activate this Update Set before making changes
         try {
             this.logger.info('Switching to Update Set', { update_set_id: args.update_set_id });
             // Set as current in ServiceNow
+            this.logger.trackAPICall('UPDATE', 'sys_update_set', 1);
             await this.client.setCurrentUpdateSet(args.update_set_id);
             // Load or create session
             const sessionFile = (0, path_1.join)(this.sessionsPath, `${args.update_set_id}.json`);
@@ -348,6 +366,7 @@ Use \`snow_update_set_switch\` to activate this Update Set before making changes
             }
             catch {
                 // Create new session for existing Update Set
+                this.logger.trackAPICall('GET', 'sys_update_set', 1);
                 const updateSet = await this.client.getUpdateSet(args.update_set_id);
                 this.currentSession = {
                     update_set_id: args.update_set_id,
@@ -382,6 +401,7 @@ All subsequent changes will be tracked in this Update Set.`
     async getCurrentUpdateSet() {
         if (!this.currentSession) {
             // Try to get from ServiceNow
+            this.logger.trackAPICall('GET', 'sys_update_set', 1);
             const current = await this.client.getCurrentUpdateSet();
             if (current.success && current.data) {
                 return {
@@ -423,6 +443,7 @@ ${this.currentSession.artifacts.length > 0
     }
     async listUpdateSets(args) {
         try {
+            this.logger.trackAPICall('SEARCH', 'sys_update_set', args.limit || 10);
             const response = await this.client.listUpdateSets({
                 state: args.state,
                 limit: args.limit || 10
@@ -462,6 +483,7 @@ ${updateSets.map((us) => `
                 throw new Error('No Update Set specified and no active session');
             }
             // Mark as complete in ServiceNow
+            this.logger.trackAPICall('UPDATE', 'sys_update_set', 1);
             const response = await this.client.completeUpdateSet(updateSetId, args.notes);
             if (!response.success) {
                 throw new Error(response.error || 'Failed to complete Update Set');
@@ -559,6 +581,7 @@ ${autoCreatedNotice}
                 throw new Error('No Update Set specified and no active session');
             }
             // Get Update Set details and changes
+            this.logger.trackAPICall('GET', 'sys_update_set_preview', 1);
             const response = await this.client.previewUpdateSet(updateSetId);
             if (!response.success) {
                 throw new Error(response.error || 'Failed to preview Update Set');
@@ -601,6 +624,7 @@ ${changes.length > 20 ? `\n... and ${changes.length - 20} more changes` : ''}
                 throw new Error('Update Set ID is required for export');
             }
             // Export Update Set as XML
+            this.logger.trackAPICall('GET', 'sys_update_set_export', 1);
             const response = await this.client.exportUpdateSet(updateSetId);
             if (!response.success) {
                 throw new Error(response.error || 'Failed to export Update Set');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "snow-flow",
-  "version": "3.4.13",
+  "version": "3.4.14",
   "description": "ServiceNow development framework with MCP server integration. Executes background scripts using ES5 JavaScript only (ServiceNow Rhino engine requirement). Provides 12 MCP servers for ServiceNow operations including widget deployment with coherence validation, table operations, script execution, and system property management.",
   "main": "dist/index.js",
   "type": "commonjs",