snow-flow 1.3.0 → 1.3.2

package/dist/utils/servicenow-client.js

@@ -12,13 +12,17 @@ exports.ServiceNowClient = void 0;
 const axios_1 = __importDefault(require("axios"));
 const snow_oauth_1 = require("./snow-oauth");
 const action_type_cache_1 = require("./action-type-cache");
+const snow_flow_config_js_1 = require("../config/snow-flow-config.js");
+const widget_template_generator_js_1 = require("./widget-template-generator.js");
+const logger_1 = require("./logger");
 const flow_structure_builder_1 = require("./flow-structure-builder");
 class ServiceNowClient {
     constructor() {
         this.credentials = null;
+        this.logger = new logger_1.Logger('ServiceNowClient');
         this.oauth = new snow_oauth_1.ServiceNowOAuth();
         this.client = axios_1.default.create({
-            timeout: 30000,
+            timeout: snow_flow_config_js_1.snowFlowConfig.servicenow.timeout,
             headers: {
                 'Content-Type': 'application/json',
                 'Accept': 'application/json'
@@ -27,8 +31,8 @@ class ServiceNowClient {
         // 🔧 CRITICAL FIX: Add makeRequest method to Axios instance to fix phantom calls
         // Some code expects makeRequest to exist on this.client (the Axios instance)
         this.client.makeRequest = async (config) => {
-            console.log('🔧 AXIOS makeRequest called! Config:', config);
-            console.log('🔧 Routing to appropriate HTTP method...');
+            this.logger.debug('🔧 AXIOS makeRequest called! Config:', config);
+            this.logger.debug('🔧 Routing to appropriate HTTP method...');
             // Route to the appropriate Axios method based on the request config
             const method = (config.method || 'GET').toLowerCase();
             const url = config.url || config.endpoint;
@@ -72,10 +76,10 @@ class ServiceNowClient {
             // Check if this is a 401 error and we haven't already tried to refresh
             if (error.response?.status === 401 && !originalRequest._retry) {
                 originalRequest._retry = true;
-                console.log('🔄 Received 401 error, attempting token refresh...');
+                this.logger.info('🔄 Received 401 error, attempting token refresh...');
                 const refreshResult = await this.oauth.refreshAccessToken();
                 if (refreshResult.success && refreshResult.accessToken) {
-                    console.log('✅ Token refreshed successfully, retrying request...');
+                    this.logger.info('✅ Token refreshed successfully, retrying request...');
                     // Update local credentials
                     if (this.credentials) {
                         this.credentials.accessToken = refreshResult.accessToken;
@@ -87,7 +91,7 @@ class ServiceNowClient {
                 }
                 else {
                     console.error('❌ Token refresh failed:', refreshResult.error);
-                    console.log('💡 Please run "snow-flow auth login" to re-authenticate');
+                    this.logger.warn('💡 Please run "snow-flow auth login" to re-authenticate');
                 }
             }
             // Log other errors for debugging
@@ -147,7 +151,7 @@ class ServiceNowClient {
             // Try to refresh the token
             const refreshResult = await this.oauth.refreshAccessToken();
             if (refreshResult.success && refreshResult.accessToken) {
-                console.log('✅ Token refreshed successfully');
+                this.logger.info('✅ Token refreshed successfully');
                 // Update local credentials
                 this.credentials.accessToken = refreshResult.accessToken;
                 return; // Success!
@@ -175,10 +179,10 @@ class ServiceNowClient {
             const fiveMinutesFromNow = new Date(now.getTime() + 5 * 60 * 1000);
             // If token expires in the next 5 minutes, refresh it
             if (expiresAt <= fiveMinutesFromNow) {
-                console.log('🔄 Token expiring soon, refreshing proactively...');
+                this.logger.info('🔄 Token expiring soon, refreshing proactively...');
                 const refreshResult = await this.oauth.refreshAccessToken();
                 if (refreshResult.success) {
-                    console.log('✅ Token refreshed proactively');
+                    this.logger.info('✅ Token refreshed proactively');
                     if (this.credentials && refreshResult.accessToken) {
                         this.credentials.accessToken = refreshResult.accessToken;
                     }
@@ -290,7 +294,7 @@ class ServiceNowClient {
         ];
         for (const test of tests) {
             try {
-                console.log(`Running diagnostic: ${test.name}...`);
+                this.logger.info(`Running diagnostic: ${test.name}...`);
                 const result = await test.test();
                 diagnostics.tests[test.name] = {
                     status: '✅ PASS',
@@ -412,8 +416,8 @@ class ServiceNowClient {
      */
     async createWidget(widget) {
         try {
-            console.log('🎨 Creating ServiceNow widget...');
-            console.log(`📋 Widget Name: ${widget.name}`);
+            this.logger.info('🎨 Creating ServiceNow widget...');
+            this.logger.info(`📋 Widget Name: ${widget.name}`);
             // Add pre-deployment validation for widgets
             if (!widget.name || widget.name.trim() === '') {
                 throw new Error('Widget name is required');
@@ -422,7 +426,30 @@ class ServiceNowClient {
                 throw new Error('Widget title is required');
             }
             if (!widget.template || widget.template.trim() === '') {
-                console.warn('⚠️ Widget has no template content - this may result in an empty widget');
+                console.warn('⚠️ Widget has no template content - generating functional template automatically');
+                // Generate a functional template instead of deploying an empty widget
+                const generatedWidget = widget_template_generator_js_1.widgetTemplateGenerator.generateWidget({
+                    title: widget.title,
+                    instruction: widget.description || widget.name || 'auto-generated widget',
+                    type: 'info', // Default to info widget for auto-generated templates
+                    theme: 'default',
+                    responsive: true
+                });
+                // Apply generated components to the widget
+                widget.template = generatedWidget.template;
+                if (!widget.css || widget.css.trim() === '') {
+                    widget.css = generatedWidget.css;
+                }
+                if (!widget.client_script || widget.client_script.trim() === '') {
+                    widget.client_script = generatedWidget.clientScript;
+                }
+                if (!widget.server_script || widget.server_script.trim() === '') {
+                    widget.server_script = generatedWidget.serverScript;
+                }
+                if (!widget.option_schema || widget.option_schema.trim() === '' || widget.option_schema === '[]') {
+                    widget.option_schema = generatedWidget.optionSchema;
+                }
+                this.logger.info('✅ Generated functional widget template automatically');
             }
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
@@ -440,8 +467,8 @@ class ServiceNowClient {
                 has_preview: widget.has_preview || false,
                 category: widget.category || 'custom'
             });
-            console.log('✅ Widget created successfully!');
-            console.log(`🆔 Widget ID: ${response.data.result.sys_id}`);
+            this.logger.info('✅ Widget created successfully!');
+            this.logger.info(`🆔 Widget ID: ${response.data.result.sys_id}`);
             // Add post-deployment verification
             await this.verifyDeployment(response.data.result.sys_id, 'widget');
             return {
@@ -462,7 +489,7 @@ class ServiceNowClient {
      */
     async updateWidget(sysId, widget) {
         try {
-            console.log(`🔄 Updating widget ${sysId}...`);
+            this.logger.info(`🔄 Updating widget ${sysId}...`);
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
             // Map fields for Service Portal widget API
@@ -472,7 +499,7 @@ class ServiceNowClient {
                 delete mappedWidget.server_script;
             }
             const response = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sp_widget/${sysId}`, mappedWidget);
-            console.log('✅ Widget updated successfully!');
+            this.logger.info('✅ Widget updated successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -517,8 +544,8 @@ class ServiceNowClient {
      */
     async createWorkflow(workflow) {
         try {
-            console.log('🔄 Creating ServiceNow workflow...');
-            console.log(`📋 Workflow Name: ${workflow.name}`);
+            this.logger.info('🔄 Creating ServiceNow workflow...');
+            this.logger.info(`📋 Workflow Name: ${workflow.name}`);
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/wf_workflow`, {
                 name: workflow.name,
                 description: workflow.description,
@@ -527,8 +554,8 @@ class ServiceNowClient {
                 table: workflow.table || '',
                 condition: workflow.condition || ''
             });
-            console.log('✅ Workflow created successfully!');
-            console.log(`🆔 Workflow ID: ${response.data.result.sys_id}`);
+            this.logger.info('✅ Workflow created successfully!');
+            this.logger.info(`🆔 Workflow ID: ${response.data.result.sys_id}`);
             return {
                 success: true,
                 data: response.data.result
@@ -547,8 +574,8 @@ class ServiceNowClient {
      */
     async createApplication(application) {
         try {
-            console.log('🏗️ Creating ServiceNow application...');
-            console.log(`📋 Application Name: ${application.name}`);
+            this.logger.info('🏗️ Creating ServiceNow application...');
+            this.logger.info(`📋 Application Name: ${application.name}`);
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_app`, {
                 name: application.name,
                 scope: application.scope,
@@ -561,8 +588,8 @@ class ServiceNowClient {
                 logo: application.logo || '',
                 active: application.active
             });
-            console.log('✅ Application created successfully!');
-            console.log(`🆔 Application ID: ${response.data.result.sys_id}`);
+            this.logger.info('✅ Application created successfully!');
+            this.logger.info(`🆔 Application ID: ${response.data.result.sys_id}`);
             return {
                 success: true,
                 data: response.data.result
@@ -581,12 +608,12 @@ class ServiceNowClient {
      */
     async executeScript(script) {
         try {
-            console.log('⚡ Executing ServiceNow script...');
+            this.logger.info('⚡ Executing ServiceNow script...');
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_script_execution`, {
                 script: script,
                 type: 'server'
             });
-            console.log('✅ Script executed successfully!');
+            this.logger.info('✅ Script executed successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -675,7 +702,7 @@ class ServiceNowClient {
             }
         }
         catch (error) {
-            console.log('Could not fetch flow defaults, using minimal defaults');
+            this.logger.warn('Could not fetch flow defaults, using minimal defaults');
         }
         // Return minimal defaults if we can't get from ServiceNow
         return {
@@ -831,14 +858,14 @@ class ServiceNowClient {
      */
     async searchFlowActions(searchTerm) {
         try {
-            console.log(`🔍 Searching for flow actions: ${searchTerm}`);
+            this.logger.info(`🔍 Searching for flow actions: ${searchTerm}`);
             // Search in sys_hub_action_type_base for available action types
             const results = await this.searchRecords('sys_hub_action_type_base', `nameLIKE${searchTerm}^ORlabelLIKE${searchTerm}^ORdescriptionLIKE${searchTerm}`, 20);
             // Also search in sys_hub_action_instance for existing actions
             const instanceResults = await this.searchRecords('sys_hub_action_instance', `action_nameLIKE${searchTerm}^ORdescriptionLIKE${searchTerm}`, 10);
             const actionTypes = results.success ? results.data.result : [];
             const actionInstances = instanceResults.success ? instanceResults.data.result : [];
-            console.log(`✅ Found ${actionTypes.length} action types and ${actionInstances.length} action instances`);
+            this.logger.info(`✅ Found ${actionTypes.length} action types and ${actionInstances.length} action instances`);
             return {
                 actionTypes,
                 actionInstances
@@ -960,7 +987,7 @@ snow_create_flow({
           `);
                 }
                 else {
-                    console.log(`✅ ${expectedType} has ${activities.length} activities`);
+                    this.logger.info(`✅ ${expectedType} has ${activities.length} activities`);
                 }
             }
             else if (expectedType === 'widget') {
@@ -969,7 +996,7 @@ snow_create_flow({
                     console.warn(`⚠️  Widget deployed but appears to have no content: ${artifact.name}`);
                 }
             }
-            console.log(`✅ Deployment verified: ${artifact.name} (${sysId})`);
+            this.logger.info(`✅ Deployment verified: ${artifact.name} (${sysId})`);
         }
         catch (error) {
             console.error(`❌ Deployment verification failed for ${sysId}:`, error);
@@ -1008,8 +1035,8 @@ snow_create_flow({
      */
     async createFlowWithStructureBuilder(flowDefinition) {
         try {
-            console.log('🏗️ Creating flow with structure builder...');
-            console.log(`📋 Flow: ${flowDefinition.name}`);
+            this.logger.info('🏗️ Creating flow with structure builder...');
+            this.logger.info(`📋 Flow: ${flowDefinition.name}`);
             await this.ensureAuthenticated();
             // Generate all flow components with proper structure
             const components = (0, flow_structure_builder_1.generateFlowComponents)(flowDefinition);
@@ -1023,32 +1050,32 @@ snow_create_flow({
                 validation.warnings.forEach(warning => console.warn(`  • ${warning}`));
             }
             // Deploy all components in correct order
-            console.log('🚀 Deploying flow components...');
+            this.logger.info('🚀 Deploying flow components...');
             // 1. Create main flow record
             const flowResponse = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow`, components.flowRecord);
             if (!flowResponse.data?.result) {
                 throw new Error('Failed to create flow record');
             }
             const flowSysId = flowResponse.data.result.sys_id;
-            console.log(`✅ Flow record created: ${flowSysId}`);
+            this.logger.info(`✅ Flow record created: ${flowSysId}`);
             // 2. Create trigger instance
             await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_trigger_instance`, components.triggerInstance);
-            console.log(`✅ Trigger created: ${components.triggerInstance.sys_id}`);
+            this.logger.info(`✅ Trigger created: ${components.triggerInstance.sys_id}`);
             // 3. Create action instances
             for (const action of components.actionInstances) {
                 await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_action_instance`, action);
             }
-            console.log(`✅ Created ${components.actionInstances.length} action instances`);
+            this.logger.info(`✅ Created ${components.actionInstances.length} action instances`);
             // 4. Create logic chain (connections)
             for (const logic of components.logicChain) {
                 await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_logic`, logic);
             }
-            console.log(`✅ Created logic chain with ${components.logicChain.length} connections`);
+            this.logger.info(`✅ Created logic chain with ${components.logicChain.length} connections`);
             // 5. Create variables
             for (const variable of components.variables) {
                 await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_variable`, variable);
             }
-            console.log(`✅ Created ${components.variables.length} flow variables`);
+            this.logger.info(`✅ Created ${components.variables.length} flow variables`);
             // Verify deployment
             await this.verifyDeployment(flowSysId, 'flow');
             const credentials = await this.oauth.loadCredentials();
@@ -1085,10 +1112,10 @@ snow_create_flow({
      */
     async createFlow(flow) {
         try {
-            console.log('🔄 Creating Flow Designer flow...');
-            console.log(`📋 Flow: ${flow.name}`);
+            this.logger.info('🔄 Creating Flow Designer flow...');
+            this.logger.info(`📋 Flow: ${flow.name}`);
             // 🔧 CRITICAL DEBUG: Check if makeRequest exists on this instance
-            console.log('🔧 CRITICAL DEBUG - Client methods check:', {
+            this.logger.debug('🔧 CRITICAL DEBUG - Client methods check:', {
                 hasCreateFlow: typeof this.createFlow === 'function',
                 hasMakeRequest: typeof this.makeRequest === 'function',
                 clientPrototype: Object.getOwnPropertyNames(Object.getPrototypeOf(this)),
@@ -1181,37 +1208,37 @@ snow_create_flow({
                 throw new Error('No response data from flow creation');
             }
             const flowId = response.data.result.sys_id;
-            console.log('✅ Flow created successfully with complete snapshot!');
-            console.log(`🆔 Flow sys_id: ${flowId}`);
+            this.logger.info('✅ Flow created successfully with complete snapshot!');
+            this.logger.info(`🆔 Flow sys_id: ${flowId}`);
             // 🔧 CRITICAL FIX: Create action instances after flow creation
             // While the flow definition contains the structure, ServiceNow also needs 
             // actual sys_hub_action_instance records for proper execution
-            console.log('📋 Flow components included in definition:');
-            console.log(`- Trigger: ${flowDefinition.trigger.type}`);
-            console.log(`- Activities: ${flowDefinition.activities.length}`);
-            console.log(`- Inputs: ${flowDefinition.inputs.length}`);
-            console.log(`- Outputs: ${flowDefinition.outputs.length}`);
+            this.logger.info('📋 Flow components included in definition:');
+            this.logger.info(`- Trigger: ${flowDefinition.trigger.type}`);
+            this.logger.info(`- Activities: ${flowDefinition.activities.length}`);
+            this.logger.info(`- Inputs: ${flowDefinition.inputs.length}`);
+            this.logger.info(`- Outputs: ${flowDefinition.outputs.length}`);
             // Create action instances for each activity
             if (activitiesToProcess.length > 0) {
-                console.log('🔧 Creating action instances for activities...');
+                this.logger.info('🔧 Creating action instances for activities...');
                 for (let i = 0; i < activitiesToProcess.length; i++) {
                     const activity = activitiesToProcess[i];
                     try {
                         await this.createFlowActionInstance(flowId, activity, (i + 1) * 100);
-                        console.log(`✅ Action instance created: ${activity.name}`);
+                        this.logger.info(`✅ Action instance created: ${activity.name}`);
                     }
                     catch (activityError) {
                         console.warn(`⚠️ Failed to create action instance ${activity.name}:`, activityError);
                         // Continue with other activities even if one fails
                     }
                 }
-                console.log(`✅ Created ${activitiesToProcess.length} action instances`);
+                this.logger.info(`✅ Created ${activitiesToProcess.length} action instances`);
             }
             // 🔧 NEW: Only need to activate the flow since it already has complete definition
             try {
-                console.log('⚡ Activating flow...');
+                this.logger.info('⚡ Activating flow...');
                 await this.activateFlow(flowId);
-                console.log('✅ Flow activated successfully!');
+                this.logger.info('✅ Flow activated successfully!');
             }
             catch (activationError) {
                 console.warn('⚠️ Flow activation failed:', activationError);
@@ -1219,7 +1246,7 @@ snow_create_flow({
             }
             // 🔧 CRITICAL FIX: Create actual ServiceNow records after flow creation
             // The JSON definition alone is not enough - we need component records
-            console.log('🔧 Creating actual ServiceNow flow component records...');
+            this.logger.info('🔧 Creating actual ServiceNow flow component records...');
             try {
                 // 1. Create trigger instance if trigger is specified
                 if (flow.trigger_type && flow.trigger_type !== 'manual') {
@@ -1229,7 +1256,7 @@ snow_create_flow({
                         condition: flow.trigger_condition || flow.condition || ''
                     };
                     const triggerResult = await this.createFlowTrigger(flowId, triggerData);
-                    console.log(`✅ Trigger created: ${triggerResult.sys_id}`);
+                    this.logger.info(`✅ Trigger created: ${triggerResult.sys_id}`);
                 }
                 // 2. Create action instances for each activity
                 if (activitiesToProcess && activitiesToProcess.length > 0) {
@@ -1240,7 +1267,7 @@ snow_create_flow({
                         try {
                             const actionResult = await this.createFlowActionInstance(flowId, activity, order);
                             actionResults.push(actionResult);
-                            console.log(`✅ Action created: ${activity.name} (${actionResult.sys_id})`);
+                            this.logger.info(`✅ Action created: ${activity.name} (${actionResult.sys_id})`);
                         }
                         catch (actionError) {
                             console.warn(`⚠️ Failed to create action ${activity.name}:`, actionError);
@@ -1258,7 +1285,7 @@ snow_create_flow({
                             };
                             try {
                                 const logicResult = await this.createFlowLogic(flowId, logicData);
-                                console.log(`✅ Flow logic created: ${logicResult.sys_id}`);
+                                this.logger.info(`✅ Flow logic created: ${logicResult.sys_id}`);
                             }
                             catch (logicError) {
                                 console.warn(`⚠️ Failed to create flow logic for ${action.action_name}:`, logicError);
@@ -1266,7 +1293,7 @@ snow_create_flow({
                         }
                     }
                 }
-                console.log('✅ All flow component records created successfully!');
+                this.logger.info('✅ All flow component records created successfully!');
             }
             catch (componentError) {
                 console.warn('⚠️ Some flow components may not have been created properly:', componentError);
@@ -1359,8 +1386,8 @@ snow_create_flow({
      */
     async createSubflow(subflow) {
         try {
-            console.log('🔄 Creating Subflow...');
-            console.log(`📋 Subflow: ${subflow.name}`);
+            this.logger.info('🔄 Creating Subflow...');
+            this.logger.info(`📋 Subflow: ${subflow.name}`);
             // Add pre-deployment validation
             this.validateFlowBeforeDeployment(subflow);
             // Ensure Update Set
@@ -1426,8 +1453,8 @@ snow_create_flow({
      */
     async createFlowAction(action) {
         try {
-            console.log('⚡ Creating Flow Action...');
-            console.log(`📋 Action: ${action.name}`);
+            this.logger.info('⚡ Creating Flow Action...');
+            this.logger.info(`📋 Action: ${action.name}`);
             // Add pre-deployment validation (adapted for actions)
             if (!action.name || action.name.trim() === '') {
                 throw new Error('Flow Action name is required');
@@ -1454,7 +1481,7 @@ snow_create_flow({
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_action_type_definition`, actionData);
             // Add basic verification for action creation
             if (response.data.result?.sys_id) {
-                console.log(`✅ Flow Action created: ${action.name} (${response.data.result.sys_id})`);
+                this.logger.info(`✅ Flow Action created: ${action.name} (${response.data.result.sys_id})`);
             }
             return {
                 success: true,
@@ -1474,7 +1501,7 @@ snow_create_flow({
      */
     async createFlowActionPrivate(flowId, action, order) {
         try {
-            console.log(`Creating flow action: ${action.type} - ${action.name}`);
+            this.logger.info(`Creating flow action: ${action.type} - ${action.name}`);
             // If we have a discovered action type, use it
             let actionTypeId = action.action_type_id;
             // If no specific action type provided, search for one
@@ -1482,7 +1509,7 @@ snow_create_flow({
                 const searchResults = await this.searchFlowActions(action.type);
                 if (searchResults.actionTypes && searchResults.actionTypes.length > 0) {
                     actionTypeId = searchResults.actionTypes[0].sys_id;
-                    console.log(`📋 Using discovered action type: ${searchResults.actionTypes[0].name}`);
+                    this.logger.info(`📋 Using discovered action type: ${searchResults.actionTypes[0].name}`);
                 }
                 else {
                     // Fallback to common action types
@@ -1495,7 +1522,7 @@ snow_create_flow({
                         'approval': 'com.glideapp.servicenow_common.approval'
                     };
                     actionTypeId = fallbackMap[action.type] || 'com.glideapp.servicenow_common.script';
-                    console.log(`📋 Using fallback action type: ${actionTypeId}`);
+                    this.logger.info(`📋 Using fallback action type: ${actionTypeId}`);
                 }
             }
             // Get action details to understand inputs/outputs
@@ -1513,7 +1540,7 @@ snow_create_flow({
                 configuration: this.buildActionConfiguration(action)
             };
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_action_instance`, actionData);
-            console.log(`✅ Flow action created: ${action.name}`);
+            this.logger.info(`✅ Flow action created: ${action.name}`);
             return response.data.result;
         }
         catch (error) {
@@ -1677,13 +1704,13 @@ try {
                         description: `Auto-generated trigger for Flow Designer flow: ${flowId}`
                     };
                     await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_trigger`, sysTriggerData);
-                    console.log('✅ sys_trigger record created for proper flow execution');
+                    this.logger.info('✅ sys_trigger record created for proper flow execution');
                 }
                 catch (sysTriggerError) {
                     console.warn('Could not create sys_trigger record, flow may not trigger properly:', sysTriggerError);
                 }
             }
-            console.log('✅ Trigger created successfully');
+            this.logger.info('✅ Trigger created successfully');
             return response.data.result;
         }
         catch (error) {
@@ -1696,7 +1723,7 @@ try {
      */
     async createFlowVariables(flowId, flow) {
         try {
-            console.log('📋 Creating flow variables...');
+            this.logger.info('📋 Creating flow variables...');
             // Process inputs
             const inputs = flow.inputs || [];
             if (inputs.length > 0) {
@@ -1714,7 +1741,7 @@ try {
                     };
                     await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_variable`, variableData);
                 }
-                console.log(`✅ Created ${inputs.length} input variables`);
+                this.logger.info(`✅ Created ${inputs.length} input variables`);
             }
             // Process outputs
             const outputs = flow.outputs || [];
@@ -1733,7 +1760,7 @@ try {
                     };
                     await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_variable`, variableData);
                 }
-                console.log(`✅ Created ${outputs.length} output variables`);
+                this.logger.info(`✅ Created ${outputs.length} output variables`);
             }
             // Extract variables from flow_definition if available
             if (flow.flow_definition) {
@@ -1755,7 +1782,7 @@ try {
                             };
                             await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_variable`, variableData);
                         }
-                        console.log(`✅ Created ${flowDef.variables.length} flow definition variables`);
+                        this.logger.info(`✅ Created ${flowDef.variables.length} flow definition variables`);
                     }
                 }
                 catch (parseError) {
@@ -1773,7 +1800,7 @@ try {
      */
     async createFlowActionInstance(flowId, action, order) {
         try {
-            console.log(`Creating action instance: ${action.name}`);
+            this.logger.info(`Creating action instance: ${action.name}`);
             // Map action types to search terms
             const actionSearchMap = {
                 'notification': 'Send Email',
@@ -1848,7 +1875,7 @@ try {
                 inputs: JSON.stringify(inputs)
             };
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_action_instance`, actionData);
-            console.log(`✅ Action created: ${action.name}`);
+            this.logger.info(`✅ Action created: ${action.name}`);
             return response.data.result;
         }
         catch (error) {
@@ -1887,7 +1914,7 @@ try {
      */
     async createScriptInclude(scriptInclude) {
         try {
-            console.log('📝 Creating Script Include...');
+            this.logger.info('📝 Creating Script Include...');
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_script_include`, {
                 name: scriptInclude.name,
                 api_name: scriptInclude.api_name,
@@ -1896,7 +1923,7 @@ try {
                 active: scriptInclude.active,
                 access: scriptInclude.access || 'public'
             });
-            console.log('✅ Script Include created successfully!');
+            this.logger.info('✅ Script Include created successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -1915,7 +1942,7 @@ try {
      */
     async createBusinessRule(businessRule) {
         try {
-            console.log('📋 Creating Business Rule...');
+            this.logger.info('📋 Creating Business Rule...');
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_script`, {
                 name: businessRule.name,
                 collection: businessRule.table,
@@ -1926,7 +1953,7 @@ try {
                 active: businessRule.active,
                 order: businessRule.order || 100
             });
-            console.log('✅ Business Rule created successfully!');
+            this.logger.info('✅ Business Rule created successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -1945,7 +1972,7 @@ try {
      */
     async createTable(table) {
         try {
-            console.log('🗄️ Creating Table...');
+            this.logger.info('🗄️ Creating Table...');
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_db_object`, {
                 name: table.name,
                 label: table.label,
@@ -1954,7 +1981,7 @@ try {
                 access: table.access || 'public',
                 create_access_controls: table.create_access_controls !== false
             });
-            console.log('✅ Table created successfully!');
+            this.logger.info('✅ Table created successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -1973,7 +2000,7 @@ try {
      */
     async createTableField(field) {
         try {
-            console.log('📊 Creating Table Field...');
+            this.logger.info('📊 Creating Table Field...');
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_dictionary`, {
                 name: `${field.table}.${field.element}`,
                 element: field.element,
@@ -1982,7 +2009,7 @@ try {
                 max_length: field.max_length || 255,
                 active: true
             });
-            console.log('✅ Table Field created successfully!');
+            this.logger.info('✅ Table Field created successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -2001,7 +2028,7 @@ try {
      */
     async createUpdateSet(updateSet) {
         try {
-            console.log('📦 Creating Update Set...');
+            this.logger.info('📦 Creating Update Set...');
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
             const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_update_set`, {
@@ -2011,7 +2038,7 @@ try {
                 state: updateSet.state || 'in_progress',
                 application: updateSet.application || 'global'
             });
-            console.log('✅ Update Set created successfully!');
+            this.logger.info('✅ Update Set created successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -2030,7 +2057,7 @@ try {
      */
     async setCurrentUpdateSet(updateSetId) {
         try {
-            console.log('🔄 Setting current Update Set...');
+            this.logger.info('🔄 Setting current Update Set...');
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
             // Use the sys_user_preference table to set the current update set
@@ -2056,7 +2083,7 @@ try {
                     user: 'javascript:gs.getUserID()'
                 });
             }
-            console.log('✅ Current Update Set changed successfully!');
+            this.logger.info('✅ Current Update Set changed successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -2075,7 +2102,7 @@ try {
      */
     async getCurrentUpdateSet() {
         try {
-            console.log('📋 Getting current Update Set...');
+            this.logger.info('📋 Getting current Update Set...');
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
             // Try to get the current update set using sys_user_preference table
@@ -2098,7 +2125,7 @@ try {
                 }
             }
             catch (prefError) {
-                console.log('⚠️ User preference lookup failed, trying fallback...');
+                this.logger.info('⚠️ User preference lookup failed, trying fallback...');
             }
             // Fallback: Get the most recent in-progress update set for the current user
             const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_update_set`, {
@@ -2132,7 +2159,7 @@ try {
      */
     async getUpdateSet(updateSetId) {
         try {
-            console.log(`📋 Getting Update Set ${updateSetId}...`);
+            this.logger.info(`📋 Getting Update Set ${updateSetId}...`);
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
             const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_update_set/${updateSetId}`);
@@ -2154,7 +2181,7 @@ try {
      */
     async listUpdateSets(options) {
         try {
-            console.log('📋 Listing Update Sets...');
+            this.logger.info('📋 Listing Update Sets...');
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
             let query = 'sys_created_by=javascript:gs.getUserName()';
@@ -2186,14 +2213,14 @@ try {
      */
     async completeUpdateSet(updateSetId, notes) {
         try {
-            console.log('✅ Completing Update Set...');
+            this.logger.info('✅ Completing Update Set...');
             // Ensure we have credentials before making the API call
             await this.ensureAuthenticated();
             const response = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sys_update_set/${updateSetId}`, {
                 state: 'complete',
                 description: notes ? `${notes}\n\nCompleted: ${new Date().toISOString()}` : undefined
             });
-            console.log('✅ Update Set completed successfully!');
+            this.logger.info('✅ Update Set completed successfully!');
             return {
                 success: true,
                 data: response.data.result
@@ -2212,7 +2239,7 @@ try {
      */
     async activateUpdateSet(updateSetId) {
         try {
-            console.log('🔄 Activating Update Set...');
+            this.logger.info('🔄 Activating Update Set...');
             // Set the update set as current
             const result = await this.setCurrentUpdateSet(updateSetId);
             if (result.success) {
@@ -2220,7 +2247,7 @@ try {
                 const updateResponse = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sys_update_set/${updateSetId}`, {
                     state: 'in_progress'
                 });
-                console.log('✅ Update Set activated successfully!');
+                this.logger.info('✅ Update Set activated successfully!');
                 return {
                     success: true,
                     data: updateResponse.data.result
@@ -2241,7 +2268,7 @@ try {
      */
     async previewUpdateSet(updateSetId) {
         try {
-            console.log('🔍 Previewing Update Set changes...');
+            this.logger.info('🔍 Previewing Update Set changes...');
             // Get Update Set details
             const updateSetResponse = await this.getUpdateSet(updateSetId);
             if (!updateSetResponse.success) {
@@ -2281,12 +2308,12 @@ try {
             // Check if we have a current Update Set
             const currentUpdateSet = await this.getCurrentUpdateSet();
             if (currentUpdateSet.success && currentUpdateSet.data) {
-                console.log(`📦 Using existing Update Set: ${currentUpdateSet.data.name}`);
+                this.logger.info(`📦 Using existing Update Set: ${currentUpdateSet.data.name}`);
                 return currentUpdateSet;
             }
             // Create a new Update Set if none exists
             const updateSetName = `Snow-Flow Changes ${new Date().toISOString().split('T')[0]}`;
-            console.log(`📦 Creating new Update Set: ${updateSetName}`);
+            this.logger.info(`📦 Creating new Update Set: ${updateSetName}`);
             const newUpdateSet = await this.createUpdateSet({
                 name: updateSetName,
                 description: 'Automated changes from Snow-Flow MCP',
@@ -2308,7 +2335,7 @@ try {
     }
     async exportUpdateSet(updateSetId) {
         try {
-            console.log('📤 Exporting Update Set...');
+            this.logger.info('📤 Exporting Update Set...');
             // Get Update Set details first
             const updateSetResponse = await this.getUpdateSet(updateSetId);
             if (!updateSetResponse.success) {
@@ -2345,7 +2372,7 @@ try {
      */
     async debugFlow(flowId) {
         try {
-            console.log('🔍 Debugging flow structure...');
+            this.logger.info('🔍 Debugging flow structure...');
             // Get the flow record
             const flowResponse = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_hub_flow/${flowId}`);
             if (!flowResponse.data.result) {
@@ -2501,17 +2528,17 @@ try {
      * This method provides compatibility for code that expects makeRequest
      */
     async makeRequest(config) {
-        console.log('🔧 MAKEQUEST CALLED! Stack trace:', new Error().stack);
-        console.log('🔧 TEMP FIX: makeRequest called with config:', config);
-        console.log('🔧 This instance constructor:', this.constructor.name);
-        console.log('🔧 This instance methods:', Object.getOwnPropertyNames(Object.getPrototypeOf(this)));
+        this.logger.info('🔧 MAKEQUEST CALLED! Stack trace:', new Error().stack);
+        this.logger.info('🔧 TEMP FIX: makeRequest called with config:', config);
+        this.logger.info('🔧 This instance constructor:', this.constructor.name);
+        this.logger.info('🔧 This instance methods:', Object.getOwnPropertyNames(Object.getPrototypeOf(this)));
         try {
             await this.ensureAuthenticated();
             // Route the request to the appropriate HTTP method
             const method = (config.method || 'GET').toLowerCase();
             const url = config.url || config.endpoint;
             const data = config.data || config.body;
-            console.log(`🔧 Routing ${method.toUpperCase()} request to: ${url}`);
+            this.logger.info(`🔧 Routing ${method.toUpperCase()} request to: ${url}`);
             switch (method) {
                 case 'get':
                     return await this.get(url, config.params);