snow-flow 8.42.12 → 9.0.1

package/dist/utils/servicenow-client.js

@@ -1,3020 +0,0 @@
-#!/usr/bin/env node
-"use strict";
-/**
- * ServiceNow API Client
- * Handles all ServiceNow API operations with OAuth authentication
- */
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.ServiceNowClient = void 0;
-const axios_1 = __importDefault(require("axios"));
-const https_1 = __importDefault(require("https"));
-const snow_oauth_1 = require("./snow-oauth");
-const action_type_cache_1 = require("./action-type-cache");
-const widget_template_generator_js_1 = require("./widget-template-generator.js");
-const logger_1 = require("./logger");
-const unified_auth_store_js_1 = require("./unified-auth-store.js");
-const timeout_manager_js_1 = require("./timeout-manager.js");
-class ServiceNowClient {
-    constructor() {
-        this.credentials = null;
-        // 🔴 SNOW-003 FIX: Token refresh synchronization to prevent race conditions
-        this.tokenRefreshPromise = null;
-        this.lastTokenRefresh = 0;
-        this.logger = new logger_1.Logger('ServiceNowClient');
-        this.oauth = new snow_oauth_1.ServiceNowOAuth();
-        // 🔒 SSL/TLS Certificate Validation Fix: Allow self-signed certificates for ServiceNow dev instances
-        const httpsAgent = new https_1.default.Agent({
-            rejectUnauthorized: false, // Allow self-signed certificates for dev instances (fixes "certificate has expired" errors)
-            checkServerIdentity: (hostname, cert) => {
-                // Custom certificate validation for ServiceNow instances
-                // Allow *.service-now.com and user-configured instances
-                const validHosts = [
-                    /.*\.service-now\.com$/,
-                    /.*\.servicenow\.com$/
-                ];
-                // Get configured instance hostname
-                const instanceUrl = process.env.SNOW_INSTANCE;
-                if (instanceUrl) {
-                    try {
-                        const instanceHostname = new URL(instanceUrl.startsWith('http') ? instanceUrl : `https://${instanceUrl}`).hostname;
-                        validHosts.push(new RegExp(`^${instanceHostname.replace(/\./g, '\\.')}$`));
-                    }
-                    catch (error) {
-                        // Invalid URL format, rely on default validation
-                    }
-                }
-                // Check if hostname matches allowed patterns
-                if (validHosts.some(pattern => pattern.test(hostname))) {
-                    return undefined; // Valid hostname
-                }
-                // For development/testing environments, allow localhost/127.0.0.1
-                if (process.env.NODE_ENV === 'development' &&
-                    (hostname === 'localhost' || hostname === '127.0.0.1' || hostname.startsWith('192.168.'))) {
-                    return undefined;
-                }
-                return new Error(`Certificate hostname mismatch: ${hostname} not allowed`);
-            },
-            // secureProtocol removed to avoid conflict with minVersion/maxVersion
-            ciphers: [
-                'ECDHE-RSA-AES128-GCM-SHA256',
-                'ECDHE-RSA-AES256-GCM-SHA384',
-                'ECDHE-RSA-AES128-SHA256',
-                'ECDHE-RSA-AES256-SHA384'
-            ].join(':'), // Only allow secure ciphers
-            honorCipherOrder: true,
-            maxVersion: 'TLSv1.3',
-            minVersion: 'TLSv1.2'
-        });
-        // Use intelligent timeout based on operation type (default to TABLE_QUERY)
-        const defaultTimeout = (0, timeout_manager_js_1.getTimeoutConfig)(timeout_manager_js_1.OperationType.TABLE_QUERY).baseTimeout;
-        this.client = axios_1.default.create({
-            timeout: parseInt(process.env.SNOW_API_TIMEOUT || String(defaultTimeout)),
-            headers: {
-                'Content-Type': 'application/json',
-                'Accept': 'application/json'
-            },
-            httpsAgent, // Use secure HTTPS agent with certificate validation
-            // Additional security headers
-            maxRedirects: 5,
-            validateStatus: (status) => status < 400 // Reject 4xx and 5xx by default
-        });
-        // Store deployment timeout from environment
-        this.deploymentTimeout = parseInt(process.env.SNOW_DEPLOYMENT_TIMEOUT || '300000'); // 5 minutes default
-        // 🔧 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) => {
-            // Route to the appropriate Axios method based on the request config
-            const method = (config.method || 'GET').toLowerCase();
-            const url = config.url || config.endpoint;
-            const data = config.data || config.body;
-            // CRITICAL FIX: Properly merge headers to allow content-type overrides for XML requests
-            const requestConfig = {
-                ...config,
-                headers: {
-                    ...(this.client.defaults?.headers?.common || {}),
-                    ...(this.client.defaults.headers[method] || {}),
-                    ...config.headers // This ensures custom headers (like Content-Type: application/xml) override defaults
-                }
-            };
-            this.logger.debug('🔧 Final request config headers:', requestConfig.headers);
-            switch (method) {
-                case 'get':
-                    return this.client.get(url, { params: config.params, ...requestConfig });
-                case 'post':
-                    return this.client.post(url, data, requestConfig);
-                case 'put':
-                    return this.client.put(url, data, requestConfig);
-                case 'patch':
-                    return this.client.patch(url, data, requestConfig);
-                case 'delete':
-                    return this.client.delete(url, requestConfig);
-                default:
-                    throw new Error(`Unsupported HTTP method: ${method}`);
-            }
-        };
-        this.actionTypeCache = new action_type_cache_1.ActionTypeCache(this);
-        // Add request interceptor for authentication with automatic token refresh
-        this.client.interceptors.request.use(async (config) => {
-            await this.ensureAuthenticated();
-            // Get fresh access token (automatically refreshes if expired)
-            const accessToken = await this.oauth.getAccessToken();
-            if (accessToken) {
-                config.headers['Authorization'] = `Bearer ${accessToken}`;
-                // Update local credentials with fresh token
-                if (this.credentials) {
-                    this.credentials.accessToken = accessToken;
-                }
-            }
-            else {
-                console.warn('⚠️ No access token available - request may fail');
-            }
-            return config;
-        });
-        // Add response interceptor for error handling with retry logic
-        this.client.interceptors.response.use((response) => response, async (error) => {
-            const originalRequest = error.config;
-            // 🔴 SNOW-003 FIX: Prevent token refresh race conditions
-            if (error.response?.status === 401 && !originalRequest._retry) {
-                originalRequest._retry = true;
-                // 🔴 CRITICAL: Use singleton token refresh to prevent multiple concurrent refreshes
-                return this.handleTokenRefreshWithLock(originalRequest);
-            }
-            // Log other errors for debugging
-            if (error.response?.status === 403) {
-                console.error('❌ 403 Forbidden - Check ServiceNow permissions');
-            }
-            else if (error.response?.status >= 500) {
-                console.error('❌ ServiceNow server error:', error.response.status);
-            }
-            return Promise.reject(error);
-        });
-    }
-    /**
-     * Public getter for credentials
-     */
-    get credentialsInstance() {
-        return this.credentials;
-    }
-    /**
-     * 🔴 SNOW-003 FIX: Handle token refresh with locking to prevent concurrent refreshes
-     * This prevents cascade failures caused by multiple simultaneous token refresh attempts
-     */
-    async handleTokenRefreshWithLock(originalRequest) {
-        const now = Date.now();
-        // 🔴 CRITICAL: If another refresh is in progress, wait for it
-        if (this.tokenRefreshPromise) {
-            this.logger.info('🔄 Token refresh already in progress, waiting...');
-            try {
-                await this.tokenRefreshPromise;
-                // After waiting, check if we have valid credentials now
-                if (this.credentials?.accessToken) {
-                    this.logger.info('✅ Using refreshed token from concurrent refresh');
-                    originalRequest.headers['Authorization'] = `Bearer ${this.credentials.accessToken}`;
-                    return this.client.request(originalRequest);
-                }
-            }
-            catch (error) {
-                this.logger.warn('Concurrent token refresh failed:', error);
-            }
-        }
-        // 🔴 CRITICAL: Rate limit token refresh to prevent excessive API calls
-        if (now - this.lastTokenRefresh < 5000) { // 5 second rate limit
-            this.logger.warn('⚠️ Token refresh rate limited - too many attempts');
-            throw new Error('Token refresh rate limited. Please wait before retrying.');
-        }
-        // 🔴 CRITICAL: Create new refresh promise with timeout and error handling
-        this.tokenRefreshPromise = this.performTokenRefreshWithTimeout();
-        this.lastTokenRefresh = now;
-        try {
-            this.logger.info('🔄 Starting token refresh process...');
-            const refreshResult = await this.tokenRefreshPromise;
-            if (refreshResult.success && refreshResult.accessToken) {
-                this.logger.info('✅ Token refreshed successfully, retrying original request...');
-                // Update local credentials
-                if (this.credentials) {
-                    this.credentials.accessToken = refreshResult.accessToken;
-                    this.credentials.expiresAt = refreshResult.expiresAt;
-                }
-                // Update the authorization header with new token
-                originalRequest.headers['Authorization'] = `Bearer ${refreshResult.accessToken}`;
-                // Clear the promise since we're done
-                this.tokenRefreshPromise = null;
-                // Retry the original request
-                return this.client.request(originalRequest);
-            }
-            else {
-                // Clear the promise on failure
-                this.tokenRefreshPromise = null;
-                const errorMsg = `Token refresh failed: ${refreshResult.error || 'Unknown error'}`;
-                this.logger.error('❌ ' + errorMsg);
-                console.error('💡 Please run "snow-flow auth login" to re-authenticate');
-                throw new Error(errorMsg);
-            }
-        }
-        catch (error) {
-            // Clear the promise on any error
-            this.tokenRefreshPromise = null;
-            this.logger.error('🔴 Token refresh process failed:', error);
-            throw error;
-        }
-    }
-    /**
-     * 🔴 SNOW-003 FIX: Token refresh with timeout to prevent hanging requests
-     */
-    async performTokenRefreshWithTimeout() {
-        const timeout = 15000; // 15 second timeout for token refresh
-        return Promise.race([
-            this.oauth.refreshAccessToken(),
-            new Promise((_, reject) => setTimeout(() => reject(new Error('Token refresh timeout')), timeout)),
-        ]);
-    }
-    /**
-     * Ensure we have valid authentication with improved error handling
-     */
-    async ensureAuthenticated() {
-        if (!this.credentials) {
-            // Try unified auth store first (most reliable)
-            const tokens = await unified_auth_store_js_1.unifiedAuthStore.getTokens();
-            if (tokens) {
-                this.credentials = {
-                    instance: tokens.instance,
-                    clientId: tokens.clientId,
-                    clientSecret: tokens.clientSecret,
-                    accessToken: tokens.accessToken,
-                    refreshToken: tokens.refreshToken,
-                    expiresAt: tokens.expiresAt
-                };
-            }
-            else {
-                // Fallback to OAuth loadCredentials
-                this.credentials = await this.oauth.loadCredentials();
-            }
-        }
-        if (!this.credentials) {
-            console.error('❌ No ServiceNow credentials found');
-            console.error('');
-            console.error('🔧 To fix this:');
-            console.error('   1. Ensure .env file has OAuth credentials:');
-            console.error('      SNOW_INSTANCE=your-instance.service-now.com');
-            console.error('      SNOW_CLIENT_ID=your_oauth_client_id');
-            console.error('      SNOW_CLIENT_SECRET=your_oauth_client_secret');
-            console.error('   2. Run: snow-flow auth login');
-            console.error('');
-            throw new Error('No ServiceNow credentials found. Set up .env file and run "snow-flow auth login".');
-        }
-        // Check if we have credentials but no access token (OAuth login needed)
-        if (!this.credentials.accessToken) {
-            console.error('🔐 OAuth authentication required');
-            console.error('');
-            console.error('✅ Your .env file has OAuth credentials');
-            console.error('❌ But no active OAuth session found');
-            console.error('');
-            console.error('🔧 To authenticate:');
-            console.error('   Run: snow-flow auth login');
-            console.error('   This will open your browser for OAuth login.');
-            console.error('');
-            throw new Error('OAuth login required. Run "snow-flow auth login" to authenticate.');
-        }
-        // Check if token is valid/authenticated
-        const isAuth = await this.oauth.isAuthenticated();
-        if (!isAuth) {
-            console.error('⏰ OAuth token expired');
-            console.error('');
-            console.error('🔄 Attempting automatic token refresh...');
-            // Try to refresh the token
-            const refreshResult = await this.oauth.refreshAccessToken();
-            if (refreshResult.success && refreshResult.accessToken) {
-                this.logger.info('✅ Token refreshed successfully');
-                // Update local credentials
-                this.credentials.accessToken = refreshResult.accessToken;
-                return; // Success!
-            }
-            console.error('❌ Token refresh failed:', refreshResult.error);
-            console.error('');
-            console.error('🔧 To fix this:');
-            console.error('   Run: snow-flow auth login');
-            console.error('   This will re-authenticate with ServiceNow.');
-            console.error('');
-            throw new Error('OAuth token expired and refresh failed. Run "snow-flow auth login" to re-authenticate.');
-        }
-    }
-    /**
-     * Proactively refresh token if it's about to expire
-     * Useful for long-running operations
-     */
-    async refreshTokenIfNeeded() {
-        try {
-            const tokens = await this.oauth.loadTokens();
-            if (!tokens)
-                return false;
-            const expiresAt = new Date(tokens.expiresAt);
-            const now = new Date();
-            const fiveMinutesFromNow = new Date(now.getTime() + 5 * 60 * 1000);
-            // If token expires in the next 5 minutes, refresh it
-            if (expiresAt <= fiveMinutesFromNow) {
-                this.logger.info('🔄 Token expiring soon, refreshing proactively...');
-                const refreshResult = await this.oauth.refreshAccessToken();
-                if (refreshResult.success) {
-                    this.logger.info('✅ Token refreshed proactively');
-                    if (this.credentials && refreshResult.accessToken) {
-                        this.credentials.accessToken = refreshResult.accessToken;
-                    }
-                    return true;
-                }
-                else {
-                    console.error('❌ Proactive token refresh failed');
-                    return false;
-                }
-            }
-            return true; // Token still valid
-        }
-        catch (error) {
-            console.error('Error checking token expiry:', error);
-            return false;
-        }
-    }
-    /**
-     * Get base URL for ServiceNow instance
-     */
-    getBaseUrl() {
-        if (!this.credentials) {
-            throw new Error('No credentials available');
-        }
-        // Remove trailing slash from instance URL to prevent double slashes in API calls
-        const instance = this.credentials.instance.replace(/\/$/, '');
-        return `https://${instance}`;
-    }
-    /**
-     * Sanitize a flow name for use as internal_name
-     */
-    sanitizeInternalName(name) {
-        return name
-            .toLowerCase()
-            .replace(/[^a-z0-9_\s]/g, '') // Remove special characters except underscores and spaces
-            .replace(/\s+/g, '_') // Replace spaces with underscores
-            .replace(/_+/g, '_') // Replace multiple underscores with single
-            .replace(/^_|_$/g, '') // Remove leading/trailing underscores
-            .substring(0, 80); // Limit length to 80 characters
-    }
-    /**
-     * Validate deployment permissions and diagnose authentication issues
-     */
-    async validateDeploymentPermissions() {
-        // Ensure we have credentials before running diagnostics
-        await this.ensureAuthenticated();
-        const diagnostics = {
-            instance_url: this.getBaseUrl(),
-            timestamp: new Date().toISOString(),
-            tests: {}
-        };
-        const tests = [
-            {
-                name: 'Read Access',
-                description: 'Test basic read permissions',
-                test: async () => {
-                    const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_user?sysparm_limit=1`);
-                    return { success: true, data: response.data };
-                }
-            },
-            {
-                name: 'Widget Read Access',
-                description: 'Test Service Portal widget read access',
-                test: async () => {
-                    const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sp_widget?sysparm_limit=1`);
-                    return { success: true, data: response.data };
-                }
-            },
-            {
-                name: 'Update Set Access',
-                description: 'Test Update Set management permissions',
-                test: async () => {
-                    const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_update_set?sysparm_limit=1`);
-                    return { success: true, data: response.data };
-                }
-            },
-            {
-                name: 'Widget Write Test',
-                description: 'Test Service Portal widget write permissions (dry run)',
-                test: async () => {
-                    // Try to create a minimal test widget that we'll immediately delete
-                    const testWidget = {
-                        name: `test_widget_${Date.now()}`,
-                        id: `test_${Date.now()}`,
-                        title: 'Test Widget (Will be deleted)',
-                        template: '<div>Test</div>',
-                        css: '',
-                        script: '',
-                        option_schema: '[]'
-                    };
-                    try {
-                        const createResponse = await this.client.post(`${this.getBaseUrl()}/api/now/table/sp_widget`, testWidget);
-                        // CRITICAL FIX: Add null safety for response processing
-                        if (!createResponse || !createResponse.data || !createResponse.data.result || !createResponse.data.result.sys_id) {
-                            throw new Error('Widget creation succeeded but response structure is unexpected - unable to verify sys_id');
-                        }
-                        const sys_id = createResponse.data.result.sys_id;
-                        // Immediately delete the test widget with error handling
-                        try {
-                            await this.client.delete(`${this.getBaseUrl()}/api/now/table/sp_widget/${sys_id}`);
-                        }
-                        catch (deleteError) {
-                            this.logger.warn(`Test widget created but cleanup failed: ${deleteError instanceof Error ? deleteError.message : String(deleteError)}`);
-                            // Don't fail the test just because cleanup failed
-                        }
-                        return { success: true, data: { message: 'Widget write permissions confirmed' } };
-                    }
-                    catch (error) {
-                        throw error;
-                    }
-                }
-            },
-            {
-                name: 'User Role Check',
-                description: 'Check user roles and permissions',
-                test: async () => {
-                    const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_user_role?sysparm_query=user=javascript:gs.getUserID()`);
-                    // CRITICAL FIX: Add null safety for response processing
-                    if (!response || !response.data || !response.data.result) {
-                        return { success: true, data: { roles: [], warning: 'Unable to retrieve user roles - response structure unexpected' } };
-                    }
-                    // Safely process roles with null checks
-                    const roles = Array.isArray(response.data.result)
-                        ? response.data.result.map((r) => {
-                            if (!r || typeof r !== 'object')
-                                return 'Unknown Role';
-                            return r.role?.display_value || r.role || 'Unknown Role';
-                        }).filter(role => role && role !== 'Unknown Role')
-                        : [];
-                    return { success: true, data: { roles } };
-                }
-            }
-        ];
-        for (const test of tests) {
-            try {
-                this.logger.info(`Running diagnostic: ${test.name}...`);
-                const result = await test.test();
-                diagnostics.tests[test.name] = {
-                    status: '✅ PASS',
-                    description: test.description,
-                    result: result.data,
-                    error: null
-                };
-            }
-            catch (error) {
-                const errorMessage = error instanceof Error ? error.message : String(error);
-                diagnostics.tests[test.name] = {
-                    status: '❌ FAIL',
-                    description: test.description,
-                    result: null,
-                    error: errorMessage,
-                    http_status: this.extractHttpStatus(errorMessage)
-                };
-            }
-        }
-        // Analyze results with null safety
-        const failedTests = diagnostics.tests && typeof diagnostics.tests === 'object'
-            ? Object.values(diagnostics.tests).filter((test) => test && test.status && test.status.includes('FAIL'))
-            : [];
-        const passedTests = diagnostics.tests && typeof diagnostics.tests === 'object'
-            ? Object.values(diagnostics.tests).filter((test) => test && test.status && test.status.includes('PASS'))
-            : [];
-        diagnostics.summary = {
-            total_tests: tests.length || 0,
-            passed: passedTests.length || 0,
-            failed: failedTests.length || 0,
-            overall_status: failedTests.length === 0 && passedTests.length > 0 ? '✅ ALL SYSTEMS GO' : '⚠️ ISSUES DETECTED'
-        };
-        // Generate recommendations with null safety
-        try {
-            diagnostics.recommendations = this.generateAuthRecommendations(diagnostics.tests);
-        }
-        catch (recError) {
-            this.logger.error('Error generating recommendations:', recError);
-            diagnostics.recommendations = ['⚠️ Unable to generate recommendations due to error'];
-        }
-        return {
-            success: failedTests.length === 0,
-            data: diagnostics,
-            error: failedTests.length > 0 ? 'Some deployment permissions are missing' : null
-        };
-    }
-    /**
-     * Extract HTTP status from error message
-     */
-    extractHttpStatus(errorMessage) {
-        const statusMatch = errorMessage.match(/status code (\d+)/i);
-        return statusMatch ? parseInt(statusMatch[1], 10) : null;
-    }
-    /**
-     * Generate authentication recommendations based on test results
-     */
-    generateAuthRecommendations(tests) {
-        const recommendations = [];
-        // CRITICAL FIX: Add comprehensive null safety checks
-        if (!tests || typeof tests !== 'object') {
-            recommendations.push('⚠️ Unable to analyze test results due to missing test data');
-            return recommendations;
-        }
-        // Safe check for Widget Write Test
-        const widgetTest = tests['Widget Write Test'];
-        if (widgetTest?.status && typeof widgetTest.status === 'string' && widgetTest.status.includes('FAIL')) {
-            const error = widgetTest.error;
-            if (error && typeof error === 'string') {
-                if (error.includes('403')) {
-                    recommendations.push('🔐 Widget deployment failed with 403 Forbidden. Check OAuth scopes: ensure \'useraccount\' and \'glide_system_administration\' scopes are enabled');
-                    recommendations.push('👤 Verify user has sp_portal_manager or admin role in ServiceNow');
-                    recommendations.push('🛡️ Check if instance has deployment restrictions for external applications');
-                }
-                if (error.includes('401')) {
-                    recommendations.push('🔑 Authentication failed. Re-run: snow-flow auth login');
-                }
-            }
-            else {
-                recommendations.push('🔐 Widget write test failed with unknown error. Check ServiceNow permissions');
-            }
-        }
-        // Safe check for User Role Check
-        const roleTest = tests['User Role Check'];
-        if (roleTest?.status && typeof roleTest.status === 'string' && roleTest.status.includes('PASS')) {
-            const roles = roleTest.result?.roles;
-            if (Array.isArray(roles)) {
-                // Debug: Log actual roles for troubleshooting
-                this.logger.info(`User roles found: ${roles.join(', ')}`);
-                const hasRequiredRole = roles.some((role) => {
-                    if (typeof role === 'string') {
-                        const roleLower = role.toLowerCase();
-                        // Check for admin roles: admin, system_administrator, etc.
-                        const isAdmin = roleLower.includes('admin');
-                        // Check for portal manager roles: sp_portal_manager, sp_admin, etc.
-                        const isPortalManager = roleLower.includes('sp_portal_manager') ||
-                            roleLower.includes('sp_admin') ||
-                            roleLower.includes('portal_manager');
-                        if (isAdmin || isPortalManager) {
-                            this.logger.info(`✅ Found qualifying role: ${role}`);
-                        }
-                        return isAdmin || isPortalManager;
-                    }
-                    return false;
-                });
-                if (!hasRequiredRole) {
-                    this.logger.warn(`❌ No qualifying roles found in: ${roles.join(', ')}`);
-                    recommendations.push(`⚠️ User roles (${roles.join(', ')}) don't include admin or portal management roles. Contact ServiceNow admin to assign appropriate roles`);
-                }
-                else {
-                    this.logger.info('✅ User has sufficient roles for deployment');
-                }
-            }
-            else {
-                recommendations.push('⚠️ Unable to verify user roles. Check if user has admin or portal management permissions');
-            }
-        }
-        // Safe check for Update Set Access
-        const updateSetTest = tests['Update Set Access'];
-        if (updateSetTest?.status && typeof updateSetTest.status === 'string' && updateSetTest.status.includes('FAIL')) {
-            recommendations.push('📦 Update Set access failed. Ensure user has update_set_manager or admin role');
-        }
-        if (recommendations.length === 0) {
-            recommendations.push('✅ All authentication checks passed! Deployment should work correctly');
-        }
-        return recommendations;
-    }
-    /**
-     * Test connection to ServiceNow
-     */
-    async testConnection() {
-        try {
-            // Ensure we have credentials first
-            await this.ensureAuthenticated();
-            // Use the /api/now/v2/table/sys_user?sysparm_limit=1 endpoint to test
-            // This is a more reliable endpoint that should work on all instances
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/v2/table/sys_user?sysparm_limit=1&sysparm_query=user_name=admin`);
-            // If we can query users, we're connected
-            return {
-                success: true,
-                data: {
-                    name: 'ServiceNow Instance',
-                    user_name: 'Connected',
-                    email: `${this.credentials?.instance}`,
-                    message: 'Connection successful'
-                }
-            };
-        }
-        catch (error) {
-            // Try a simpler endpoint if the first one fails
-            try {
-                const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_properties?sysparm_limit=1`);
-                return {
-                    success: true,
-                    data: {
-                        name: 'ServiceNow Instance',
-                        user_name: 'Connected',
-                        email: `${this.credentials?.instance}`,
-                        message: 'Connection successful (limited access)'
-                    }
-                };
-            }
-            catch (secondError) {
-                return {
-                    success: false,
-                    error: error instanceof Error ? error.message : String(error)
-                };
-            }
-        }
-    }
-    /**
-     * Create a new ServiceNow widget
-     */
-    async createWidget(widget) {
-        try {
-            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');
-            }
-            if (!widget.title || widget.title.trim() === '') {
-                throw new Error('Widget title is required');
-            }
-            if (!widget.template || widget.template.trim() === '') {
-                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.script || widget.script.trim() === '') {
-                    widget.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();
-            // Log the request details for debugging
-            const widgetData = {
-                name: widget.name,
-                id: widget.id || widget.name, // Ensure id is set
-                title: widget.title,
-                description: widget.description || '',
-                template: widget.template,
-                css: widget.css || '',
-                client_script: widget.client_script || '',
-                script: widget.script || '', // Direct mapping to ServiceNow script field
-                option_schema: widget.option_schema || '[]',
-                demo_data: widget.demo_data || '{}',
-                has_preview: widget.has_preview !== false, // Default to true
-                category: widget.category || 'custom',
-                active: true // Ensure widget is active
-            };
-            this.logger.info('Widget data to be sent:', widgetData);
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sp_widget`, widgetData, {
-                timeout: this.deploymentTimeout, // Use deployment-specific timeout
-                headers: {
-                    'X-Operation-Type': 'deployment' // Mark as deployment operation
-                }
-            });
-            this.logger.info('✅ Widget created successfully!');
-            // Handle different response structures from ServiceNow
-            const widgetResult = response.data.result || response.data;
-            const sysId = widgetResult.sys_id;
-            if (!sysId) {
-                this.logger.warn('⚠️ Widget created but no sys_id returned. Response:', response.data);
-                throw new Error('Widget creation succeeded but no sys_id was returned');
-            }
-            this.logger.info(`🆔 Widget ID: ${sysId}`);
-            // Add post-deployment verification
-            await this.verifyDeployment(sysId, 'widget');
-            return {
-                success: true,
-                data: widgetResult
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create widget:', error);
-            // Better error handling for axios errors
-            let errorMessage = 'Unknown error';
-            let errorDetails = {};
-            if (error.response) {
-                // The request was made and the server responded with a status code
-                // that falls out of the range of 2xx
-                errorMessage = `HTTP ${error.response.status}: ${error.response.statusText || 'Request failed'}`;
-                errorDetails = {
-                    status: error.response.status,
-                    statusText: error.response.statusText,
-                    data: error.response.data,
-                    headers: error.response.headers
-                };
-                // Extract ServiceNow specific error message if available
-                if (error.response.data?.error?.message) {
-                    errorMessage = `ServiceNow Error: ${error.response.data.error.message}`;
-                }
-                else if (error.response.data?.error) {
-                    errorMessage = `ServiceNow Error: ${JSON.stringify(error.response.data.error)}`;
-                }
-                else if (error.response.status === 401) {
-                    errorMessage = 'Authentication failed: Invalid or expired token. Run: snow-flow auth login';
-                }
-                else if (error.response.status === 403) {
-                    errorMessage = 'Permission denied: User lacks sp_admin role or widget creation permissions';
-                }
-                else if (error.response.status === 404) {
-                    errorMessage = 'API endpoint not found: ServiceNow instance may not have Service Portal installed';
-                }
-            }
-            else if (error.request) {
-                // The request was made but no response was received
-                errorMessage = 'No response from ServiceNow - check network connection and instance URL';
-                errorDetails = { request: error.config?.url };
-            }
-            else {
-                // Something happened in setting up the request that triggered an Error
-                errorMessage = error.message || String(error);
-            }
-            this.logger.error('Widget creation error details:', errorDetails);
-            return {
-                success: false,
-                error: errorMessage,
-                details: errorDetails
-            };
-        }
-    }
-    /**
-     * Update an existing ServiceNow widget
-     */
-    async updateWidget(sysId, widget) {
-        try {
-            this.logger.info(`🔄 Updating widget ${sysId}...`);
-            // Ensure we have credentials before making the API call
-            await this.ensureAuthenticated();
-            // Widget is already in correct format for Service Portal API
-            const mappedWidget = { ...widget };
-            const response = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sp_widget/${sysId}`, mappedWidget, {
-                timeout: this.deploymentTimeout, // Use deployment-specific timeout
-                headers: {
-                    'X-Operation-Type': 'deployment' // Mark as deployment operation
-                }
-            });
-            this.logger.info('✅ Widget updated successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to update widget:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get widget by ID
-     */
-    async getWidget(widgetId) {
-        try {
-            // Ensure we have credentials before making the API call
-            await this.ensureAuthenticated();
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sp_widget?sysparm_query=id=${widgetId}`);
-            if (response.data.result.length === 0) {
-                return {
-                    success: false,
-                    error: 'Widget not found'
-                };
-            }
-            return {
-                success: true,
-                data: response.data.result[0]
-            };
-        }
-        catch (error) {
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a new ServiceNow workflow
-     */
-    async createWorkflow(workflow) {
-        try {
-            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,
-                active: workflow.active,
-                workflow_version: workflow.workflow_version,
-                table: workflow.table || '',
-                condition: workflow.condition || ''
-            });
-            this.logger.info('✅ Workflow created successfully!');
-            this.logger.info(`🆔 Workflow ID: ${response.data.result.sys_id}`);
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create workflow:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a new ServiceNow application
-     */
-    async createApplication(application) {
-        try {
-            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,
-                version: application.version,
-                short_description: application.short_description,
-                description: application.description,
-                vendor: application.vendor,
-                vendor_prefix: application.vendor_prefix,
-                template: application.template || '',
-                logo: application.logo || '',
-                active: application.active
-            });
-            this.logger.info('✅ Application created successfully!');
-            this.logger.info(`🆔 Application ID: ${response.data.result.sys_id}`);
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create application:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Execute a ServiceNow script using Scheduled Jobs with sys_trigger
-     * Creates a scheduled job, triggers it via sys_trigger, and cleans up
-     */
-    async executeScript(script) {
-        try {
-            this.logger.info('⚡ Executing ServiceNow script via Scheduled Job + sys_trigger...');
-            const executionId = `exec_${Date.now()}_${Math.random().toString(36).substring(7)}`;
-            const outputMarker = `SNOW_FLOW_EXEC_${executionId}`;
-            // Wrap script with output capture
-            const wrappedScript = `
-// Snow-Flow Script Execution - ${executionId}
-var __execResult = null;
-var __execError = null;
-var __execOutput = [];
-
-var __origInfo = gs.info;
-var __origError = gs.error;
-
-gs.info = function(msg) { __execOutput.push({level: 'info', msg: String(msg)}); __origInfo(msg); };
-gs.error = function(msg) { __execOutput.push({level: 'error', msg: String(msg)}); __origError(msg); };
-
-try {
-  __execResult = (function() {
-    ${script}
-  })();
-} catch(e) {
-  __execError = e.toString();
-}
-
-gs.info = __origInfo;
-gs.error = __origError;
-
-gs.setProperty('${outputMarker}', JSON.stringify({
-  executionId: '${executionId}',
-  success: __execError === null,
-  result: __execResult,
-  error: __execError,
-  output: __execOutput
-}));
-`;
-            // Create Scheduled Script Job
-            const jobName = `Snow-Flow Exec - ${executionId}`;
-            const createResponse = await this.client.post(`${this.getBaseUrl()}/api/now/table/sysauto_script`, {
-                name: jobName,
-                script: wrappedScript,
-                active: true,
-                run_type: 'on_demand',
-                conditional: false
-            });
-            const jobSysId = createResponse.data?.result?.sys_id;
-            if (!jobSysId) {
-                throw new Error('Failed to create scheduled script job');
-            }
-            // Create sys_trigger to execute immediately
-            const now = new Date();
-            const triggerTime = new Date(now.getTime() + 2000); // 2 seconds from now
-            const triggerTimeStr = triggerTime.toISOString().replace('T', ' ').substring(0, 19);
-            try {
-                await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_trigger`, {
-                    name: jobName,
-                    next_action: triggerTimeStr,
-                    trigger_type: 0, // Run Once
-                    state: 0, // Ready
-                    document: 'sysauto_script',
-                    document_key: jobSysId,
-                    claimed_by: '',
-                    system_id: 'snow-flow'
-                });
-            }
-            catch (triggerErr) {
-                // Trigger creation may fail due to permissions
-            }
-            // Poll for result (max 30 seconds)
-            let result = null;
-            for (let i = 0; i < 15; i++) {
-                await new Promise(resolve => setTimeout(resolve, 2000));
-                try {
-                    const propResponse = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_properties`, { params: { sysparm_query: `name=${outputMarker}`, sysparm_fields: 'value,sys_id', sysparm_limit: 1 } });
-                    if (propResponse.data?.result?.[0]?.value) {
-                        result = JSON.parse(propResponse.data.result[0].value);
-                        // Cleanup property
-                        const propSysId = propResponse.data.result[0].sys_id;
-                        if (propSysId) {
-                            await this.client.delete(`${this.getBaseUrl()}/api/now/table/sys_properties/${propSysId}`).catch(() => { });
-                        }
-                        break;
-                    }
-                }
-                catch (pollErr) {
-                    // Continue polling
-                }
-            }
-            // Cleanup scheduled job
-            try {
-                await this.client.delete(`${this.getBaseUrl()}/api/now/table/sysauto_script/${jobSysId}`);
-            }
-            catch (cleanupErr) {
-                // Ignore
-            }
-            if (result) {
-                this.logger.info('✅ Script executed successfully!');
-                return {
-                    success: result.success,
-                    data: result
-                };
-            }
-            else {
-                this.logger.warn('⚠️ Script saved but execution not confirmed');
-                return {
-                    success: true,
-                    data: {
-                        message: 'Scheduled job created but execution not confirmed. The sys_trigger may not have been created (permissions) or the scheduler has not yet picked it up.',
-                        scheduled_job_sys_id: jobSysId
-                    }
-                };
-            }
-        }
-        catch (error) {
-            console.error('❌ Failed to execute script:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get all widgets
-     */
-    async getWidgets() {
-        try {
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sp_widget?sysparm_limit=100`);
-            return {
-                success: true,
-                result: response.data.result
-            };
-        }
-        catch (error) {
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get all workflows
-     */
-    async getWorkflows() {
-        try {
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/wf_workflow?sysparm_limit=100`);
-            return {
-                success: true,
-                result: response.data.result
-            };
-        }
-        catch (error) {
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get all applications
-     */
-    async getApplications() {
-        try {
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_app?sysparm_limit=100`);
-            return {
-                success: true,
-                result: response.data.result
-            };
-        }
-        catch (error) {
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get default flow structure from ServiceNow
-     */
-    async getFlowDefaults() {
-        try {
-            // Try to get an existing flow to see the structure
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_hub_flow?sysparm_limit=1&sysparm_fields=sys_class_name,type,status,access,source_ui,sys_domain,sys_domain_path`);
-            if (response.data.result && response.data.result.length > 0) {
-                const sample = response.data.result[0];
-                return {
-                    sys_class_name: sample.sys_class_name || 'sys_hub_flow',
-                    type: sample.type || 'flow',
-                    status: sample.status || 'published',
-                    access: sample.access || 'public',
-                    source_ui: sample.source_ui || 'flow_designer',
-                    sys_domain: sample.sys_domain || 'global',
-                    sys_domain_path: sample.sys_domain_path || '/'
-                };
-            }
-        }
-        catch (error) {
-            this.logger.warn('Could not fetch flow defaults, using minimal defaults');
-        }
-        // Return minimal defaults if we can't get from ServiceNow
-        return {
-            sys_class_name: 'sys_hub_flow',
-            type: 'flow',
-            status: 'published',
-            access: 'public'
-        };
-    }
-    /**
-     * Get instance info
-     */
-    async getInstanceInfo() {
-        try {
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_properties?sysparm_query=name=instance.name`);
-            return {
-                success: true,
-                data: response.data.result[0]
-            };
-        }
-        catch (error) {
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get a specific record by sys_id
-     */
-    async getRecord(table, sys_id) {
-        try {
-            // Ensure we have credentials before making the API call
-            await this.ensureAuthenticated();
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/${table}/${sys_id}`);
-            return response.data.result;
-        }
-        catch (error) {
-            console.error(`Failed to get record from ${table}:`, error);
-            throw error;
-        }
-    }
-    /**
-     * Get multiple records from a table
-     */
-    async getRecords(table, params) {
-        try {
-            // Ensure we have credentials before making the API call
-            await this.ensureAuthenticated();
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/${table}`, { params });
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error(`Failed to get records from ${table}:`, error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Search records with specific fields
-     */
-    async searchRecordsWithFields(table, query, fields, limit = 10) {
-        try {
-            await this.ensureAuthenticated();
-            const url = `/api/now/table/${table}`;
-            const params = {
-                sysparm_query: query,
-                sysparm_limit: limit.toString(),
-                sysparm_fields: fields.join(',')
-            };
-            const response = await this.client.get(url, { params });
-            return {
-                success: true,
-                data: response.data
-            };
-        }
-        catch (error) {
-            console.error('API Error:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error),
-                data: null
-            };
-        }
-    }
-    /**
-     * Search records in a table using encoded query
-     */
-    async searchRecords(table, query, limit = 10) {
-        try {
-            await this.ensureAuthenticated();
-            // Detect operation type for intelligent timeout
-            const operationType = (0, timeout_manager_js_1.detectOperationType)({
-                action: 'query',
-                table,
-                limit
-            });
-            // Use retry wrapper with intelligent timeout
-            const result = await (0, timeout_manager_js_1.withRetry)(async () => {
-                const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/${table}`, {
-                    params: {
-                        sysparm_query: query,
-                        sysparm_limit: limit
-                    },
-                    // Override timeout for this specific request
-                    timeout: (0, timeout_manager_js_1.getTimeoutConfig)(operationType).baseTimeout
-                });
-                return response;
-            }, operationType, `Search ${table} (${limit} records)`);
-            return {
-                success: true,
-                data: {
-                    result: result.data.result || []
-                }
-            };
-        }
-        catch (error) {
-            this.logger.error(`Failed to search records in ${table}:`, error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Search records with offset for pagination/streaming
-     */
-    async searchRecordsWithOffset(table, query, limit = 10, offset = 0) {
-        try {
-            await this.ensureAuthenticated();
-            // Detect operation type for intelligent timeout
-            const operationType = (0, timeout_manager_js_1.detectOperationType)({
-                action: 'query',
-                table,
-                limit
-            });
-            // Use retry wrapper with intelligent timeout
-            const result = await (0, timeout_manager_js_1.withRetry)(async () => {
-                const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/${table}`, {
-                    params: {
-                        sysparm_query: query,
-                        sysparm_limit: limit,
-                        sysparm_offset: offset
-                    },
-                    // Override timeout for this specific request
-                    timeout: (0, timeout_manager_js_1.getTimeoutConfig)(operationType).baseTimeout
-                });
-                return response;
-            }, operationType, `Search ${table} with offset ${offset}`);
-            return {
-                success: true,
-                data: {
-                    result: result.data.result || []
-                }
-            };
-        }
-        catch (error) {
-            this.logger.error(`Failed to search records in ${table} with offset ${offset}:`, error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a record in any ServiceNow table
-     */
-    async createRecord(table, data) {
-        try {
-            await this.ensureAuthenticated();
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/${table}`, data);
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error(`Failed to create record in ${table}:`, error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Update a record in any ServiceNow table
-     */
-    async updateRecord(table, sysId, data) {
-        try {
-            await this.ensureAuthenticated();
-            const response = await this.client.patch(`${this.getBaseUrl()}/api/now/table/${table}/${sysId}`, data);
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error(`Failed to update record in ${table}:`, error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Delete a record from any ServiceNow table
-     */
-    async deleteRecord(table, sysId) {
-        try {
-            await this.ensureAuthenticated();
-            const response = await this.client.delete(`${this.getBaseUrl()}/api/now/table/${table}/${sysId}`);
-            return {
-                success: true,
-                data: response.data || {}
-            };
-        }
-        catch (error) {
-            console.error(`Failed to delete record from ${table}:`, error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Search for available flow actions in ServiceNow
-     */
-    async searchFlowActions(searchTerm) {
-        try {
-            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 : [];
-            this.logger.info(`✅ Found ${actionTypes.length} action types and ${actionInstances.length} action instances`);
-            return {
-                actionTypes,
-                actionInstances
-            };
-        }
-        catch (error) {
-            console.error('Failed to search flow actions:', error);
-            return { actionTypes: [], actionInstances: [] };
-        }
-    }
-    /**
-     * Get flow action details
-     */
-    async getFlowActionDetails(actionTypeId) {
-        try {
-            const actionType = await this.getRecord('sys_hub_action_type_base', actionTypeId);
-            // Get input/output variables for this action type
-            const inputs = await this.searchRecords('sys_hub_action_input', `action_type=${actionTypeId}`, 50);
-            const outputs = await this.searchRecords('sys_hub_action_output', `action_type=${actionTypeId}`, 50);
-            return {
-                actionType,
-                inputs: inputs.success ? inputs.data.result : [],
-                outputs: outputs.success ? outputs.data.result : []
-            };
-        }
-        catch (error) {
-            console.error('Failed to get flow action details:', error);
-            return null;
-        }
-    }
-    /**
-     * Validate flow definition before deployment
-     */
-    validateFlowBeforeDeployment(flow) {
-        const errors = [];
-        if (!flow.name || flow.name.trim() === '') {
-            errors.push('Flow name is required');
-        }
-        if (!flow.description || flow.description.trim() === '') {
-            errors.push('Flow description is required');
-        }
-        // Validate activities/actions
-        const activities = flow.activities || flow.actions || flow.steps || [];
-        if (activities.length === 0) {
-            errors.push(`
-No activities found in flow definition.
-
-💡 Common fixes:
-• Use natural language: snow_create_flow({ instruction: "..." })
-• Check flow_definition format
-• Ensure activities array is properly populated
-      `);
-        }
-        // Validate activity structure
-        activities.forEach((activity, index) => {
-            if (!activity.name) {
-                errors.push(`Activity ${index + 1} missing name`);
-            }
-            if (!activity.type && !activity.operation) {
-                errors.push(`Activity ${index + 1} missing type or operation`);
-            }
-        });
-        if (errors.length > 0) {
-            throw new Error(`
-🚨 Flow Validation Failed:
-
-${errors.map(error => `• ${error}`).join('\n')}
-
-🔧 Recommended approach:
-Use snow_create_flow with natural language instead of manual JSON.
-      `);
-        }
-    }
-    /**
-     * Verify deployment was successful and artifact has content
-     */
-    async verifyDeployment(sysId, expectedType) {
-        try {
-            let endpoint = '';
-            // Determine the correct endpoint based on artifact type
-            switch (expectedType) {
-                case 'flow':
-                case 'subflow':
-                    endpoint = `/api/now/table/sys_hub_flow/${sysId}`;
-                    break;
-                case 'widget':
-                    endpoint = `/api/now/table/sp_widget/${sysId}`;
-                    break;
-                case 'action':
-                    endpoint = `/api/now/table/sys_hub_action_type_definition/${sysId}`;
-                    break;
-                default:
-                    endpoint = `/api/now/table/sys_hub_flow/${sysId}`;
-            }
-            // Get the deployed artifact
-            const response = await this.client.get(`${this.getBaseUrl()}${endpoint}`);
-            if (!response.data?.result) {
-                throw new Error(`Deployed ${expectedType} not found: ${sysId}`);
-            }
-            const artifact = response.data.result;
-            // Type-specific content validation
-            if (expectedType === 'flow' || expectedType === 'subflow') {
-                const flowDef = artifact.flow_definition ? JSON.parse(artifact.flow_definition || '{}') : {};
-                const activities = flowDef.activities || flowDef.actions || flowDef.steps || [];
-                if (activities.length === 0) {
-                    console.warn(`
-⚠️  ${expectedType} deployed but appears empty: ${artifact.name}
-
-🔍 This usually means:
-• Flow definition format was incorrect
-• Activities were not properly mapped
-• ServiceNow rejected the flow structure
-
-💡 Try recreating with:
-snow_create_flow({
-  instruction: "recreate ${artifact.name} with proper activities",
-  deploy_immediately: true
-})
-          `);
-                }
-                else {
-                    this.logger.info(`✅ ${expectedType} has ${activities.length} activities`);
-                }
-            }
-            else if (expectedType === 'widget') {
-                const hasContent = artifact.template || artifact.client_script || artifact.script;
-                if (!hasContent) {
-                    console.warn(`⚠️  Widget deployed but appears to have no content: ${artifact.name}`);
-                }
-            }
-            this.logger.info(`✅ Deployment verified: ${artifact.name} (${sysId})`);
-        }
-        catch (error) {
-            console.error(`❌ Deployment verification failed for ${sysId}:`, error);
-        }
-    }
-    /**
-     * Check if a flow has actual content (not empty)
-     */
-    async checkFlowContent(sysId) {
-        try {
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_hub_flow/${sysId}`);
-            const flow = response.data?.result;
-            if (!flow) {
-                return { hasContent: false, details: 'Flow not found' };
-            }
-            const flowDef = JSON.parse(flow.flow_definition || '{}');
-            const activities = flowDef.activities || flowDef.actions || flowDef.steps || [];
-            return {
-                hasContent: activities.length > 0,
-                details: {
-                    name: flow.name,
-                    activities_count: activities.length,
-                    has_trigger: !!flowDef.trigger,
-                    status: flow.status || 'unknown',
-                    active: flow.active
-                }
-            };
-        }
-        catch (error) {
-            return { hasContent: false, details: `Error: ${error.message}` };
-        }
-    }
-    /**
-     * Create a ServiceNow flow using the enhanced flow structure builder
-     * Generates proper sys_ids, logic chains, and all required records
-     */
-    async createFlowWithStructureBuilder(flowDefinition) {
-        try {
-            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 = generateFlowComponents(flowDefinition); // Deprecated in v1.4.0
-            const components = {
-                logicChain: [],
-                variables: [],
-                flowRecord: { sys_id: 'deprecated' },
-                triggerInstance: { sys_id: 'deprecated' },
-                actionInstances: []
-            };
-            // Validate components before deployment
-            // const validation = validateFlowComponents(components); // Deprecated in v1.4.0
-            const validation = { isValid: true, errors: [], warnings: [] };
-            if (!validation.isValid) {
-                throw new Error(`Flow validation failed: ${validation.errors.join(', ')}`);
-            }
-            if (validation.warnings.length > 0) {
-                console.warn('⚠️ Flow validation warnings:');
-                validation.warnings.forEach(warning => console.warn(`  • ${warning}`));
-            }
-            // Deploy all components in correct order
-            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;
-            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);
-            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);
-            }
-            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);
-            }
-            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);
-            }
-            this.logger.info(`✅ Created ${components.variables.length} flow variables`);
-            // Verify deployment
-            await this.verifyDeployment(flowSysId, 'flow');
-            const credentials = await this.oauth.loadCredentials();
-            const instance = credentials?.instance?.replace(/\/$/, '');
-            return {
-                success: true,
-                data: {
-                    ...flowResponse.data.result,
-                    sys_id: flowSysId,
-                    url: `https://${instance}/nav_to.do?uri=sys_hub_flow.do?sys_id=${flowSysId}`,
-                    flow_designer_url: `https://${instance}/flow-designer.do?sysparm_nostack=true&sysparm_sys_id=${flowSysId}`,
-                    components: {
-                        flow_record: components.flowRecord.sys_id,
-                        trigger_instance: components.triggerInstance.sys_id,
-                        action_instances: components.actionInstances.length,
-                        logic_chain_entries: components.logicChain.length,
-                        variables: components.variables.length
-                    },
-                    flow_xml: '<!-- Flow XML generation deprecated in v1.4.0 -->'
-                }
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create flow with structure builder:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a simple Flow Designer flow (original method)
-     * Focusing on basic flow creation with simple actions
-     */
-    async createFlow(flow) {
-        try {
-            this.logger.info('🔄 Creating Flow Designer flow...');
-            this.logger.info(`📋 Flow: ${flow.name}`);
-            // 🔧 CRITICAL DEBUG: Check if makeRequest exists on this instance
-            this.logger.debug('🔧 CRITICAL DEBUG - Client methods check:', {
-                hasCreateFlow: typeof this.createFlow === 'function',
-                hasMakeRequest: typeof this.makeRequest === 'function',
-                clientPrototype: Object.getOwnPropertyNames(Object.getPrototypeOf(this)),
-                clientConstructor: this.constructor.name
-            });
-            // Add pre-deployment validation
-            this.validateFlowBeforeDeployment(flow);
-            // First, ensure we have an Update Set
-            const updateSetResult = await this.ensureUpdateSet();
-            // Get dynamic defaults from ServiceNow
-            const flowDefaults = await this.getFlowDefaults();
-            // 🔧 CRITICAL FIX: Build complete flow snapshot BEFORE creating flow record
-            // This prevents the "Your flow cannot be found" error caused by incremental saves
-            // Prepare activities from flow definition
-            let activitiesToProcess = flow.activities || flow.actions || [];
-            if (activitiesToProcess.length === 0 && flow.flow_definition) {
-                try {
-                    const flowDef = typeof flow.flow_definition === 'string' ?
-                        JSON.parse(flow.flow_definition) : flow.flow_definition;
-                    activitiesToProcess = flowDef.activities || flowDef.steps || [];
-                }
-                catch (parseError) {
-                    console.warn('Could not parse flow_definition for activities:', parseError);
-                }
-            }
-            // Build complete flow definition in ServiceNow Flow Designer format
-            const flowDefinition = {
-                "name": flow.name,
-                "description": flow.description || flow.name,
-                "trigger": {
-                    "type": flow.trigger_type || 'manual',
-                    "table": flow.table || 'incident',
-                    "condition": flow.trigger_condition || flow.condition || ''
-                },
-                "activities": activitiesToProcess.map((action, index) => ({
-                    "name": action.name,
-                    "label": action.label || action.name,
-                    "type": action.type || "core",
-                    "operation": action.operation || "script",
-                    "inputs": action.inputs || {},
-                    "outputs": action.outputs || {},
-                    "order": (index + 1) * 100,
-                    "active": true
-                })),
-                "inputs": (flow.inputs || []).map(input => ({
-                    "name": input.name || input.id,
-                    "label": input.label || input.name || input.id,
-                    "type": input.type || "string",
-                    "mandatory": input.required || false,
-                    "default_value": input.default || ""
-                })),
-                "outputs": (flow.outputs || []).map(output => ({
-                    "name": output.name || output.id,
-                    "label": output.label || output.name || output.id,
-                    "type": output.type || "string"
-                })),
-                "variables": flow.variables || [],
-                "version": "1.0"
-            };
-            // Build flow structure with dynamic defaults AND complete snapshot
-            const flowData = {
-                name: flow.name,
-                description: flow.description || flow.name,
-                active: true, // Always create as active
-                internal_name: this.sanitizeInternalName(flow.name),
-                category: flow.category || 'custom',
-                run_as: flow.run_as || 'user', // 🔒 SEC-001 FIX: Default to 'user' to prevent privilege escalation
-                status: 'published', // 🔧 FIX: Ensure flow is published, not draft
-                validated: true, // 🔧 FIX: Mark as validated to allow activation
-                // Include complete flow definition to prevent corruption
-                flow_definition: JSON.stringify(flowDefinition),
-                // Additional fields for proper flow creation
-                latest_snapshot: JSON.stringify(flowDefinition),
-                model_id: flow.model_id || '',
-                natlang: flow.description || flow.name,
-                copied_from: '',
-                copied_from_name: '',
-                show_draft_actions: false,
-                show_triggered_flows: false,
-                // Merge with dynamic defaults from ServiceNow
-                ...flowDefaults,
-                // Override with any specific values from the flow parameter
-                ...flow.overrides,
-                // 🔧 CRITICAL FIX: Ensure flow type is respected and not overridden
-                type: flow.type || 'flow'
-            };
-            // Create the main flow record WITH complete snapshot
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow`, flowData);
-            if (!response.data || !response.data.result) {
-                throw new Error('No response data from flow creation');
-            }
-            const flowId = response.data.result.sys_id;
-            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
-            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) {
-                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);
-                        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
-                    }
-                }
-                this.logger.info(`✅ Created ${activitiesToProcess.length} action instances`);
-            }
-            // 🔧 NEW: Only need to activate the flow since it already has complete definition
-            try {
-                this.logger.info('⚡ Activating flow...');
-                await this.activateFlow(flowId);
-                this.logger.info('✅ Flow activated successfully!');
-            }
-            catch (activationError) {
-                console.warn('⚠️ Flow activation failed:', activationError);
-                // Don't fail the entire flow creation if activation fails
-            }
-            // 🔧 CRITICAL FIX: Create actual ServiceNow records after flow creation
-            // The JSON definition alone is not enough - we need 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') {
-                    const triggerData = {
-                        type: flow.trigger_type,
-                        table: flow.table || 'incident',
-                        condition: flow.trigger_condition || flow.condition || ''
-                    };
-                    const triggerResult = await this.createFlowTrigger(flowId, triggerData);
-                    this.logger.info(`✅ Trigger created: ${triggerResult.sys_id}`);
-                }
-                // 2. Create action instances for each activity
-                if (activitiesToProcess && activitiesToProcess.length > 0) {
-                    const actionResults = [];
-                    for (let i = 0; i < activitiesToProcess.length; i++) {
-                        const activity = activitiesToProcess[i];
-                        const order = (i + 1) * 100;
-                        try {
-                            const actionResult = await this.createFlowActionInstance(flowId, activity, order);
-                            actionResults.push(actionResult);
-                            this.logger.info(`✅ Action created: ${activity.name} (${actionResult.sys_id})`);
-                        }
-                        catch (actionError) {
-                            console.warn(`⚠️ Failed to create action ${activity.name}:`, actionError);
-                        }
-                    }
-                    // 3. Create flow logic entries for visual representation
-                    if (actionResults.length > 0) {
-                        for (let i = 0; i < actionResults.length; i++) {
-                            const action = actionResults[i];
-                            const logicData = {
-                                name: action.action_name,
-                                type: 'action',
-                                order: (i + 1) * 100,
-                                instance: action.sys_id
-                            };
-                            try {
-                                const logicResult = await this.createFlowLogic(flowId, logicData);
-                                this.logger.info(`✅ Flow logic created: ${logicResult.sys_id}`);
-                            }
-                            catch (logicError) {
-                                console.warn(`⚠️ Failed to create flow logic for ${action.action_name}:`, logicError);
-                            }
-                        }
-                    }
-                }
-                this.logger.info('✅ All flow component records created successfully!');
-            }
-            catch (componentError) {
-                console.warn('⚠️ Some flow components may not have been created properly:', componentError);
-                // Don't fail the entire flow creation if component creation fails
-            }
-            // Add post-deployment verification
-            await this.verifyDeployment(flowId, 'flow');
-            // 🔧 CRITICAL FIX: Enhanced response with proper ServiceNow URLs and flow type
-            const flowRecord = response.data.result;
-            const credentials = await this.oauth.loadCredentials();
-            const instance = credentials?.instance || process.env.SNOW_INSTANCE;
-            return {
-                success: true,
-                data: {
-                    ...flowRecord,
-                    // Enhanced response format with proper URLs
-                    url: `https://${instance}/nav_to.do?uri=sys_hub_flow.do?sys_id=${flowRecord.sys_id}`,
-                    flow_designer_url: `https://${instance}/flow-designer.do?sysparm_nostack=true&sysparm_sys_id=${flowRecord.sys_id}`,
-                    type: flowRecord.type || flow.type || 'flow', // Ensure type is included
-                    activities_created: flowDefinition.activities.length,
-                    variables_created: (flow.inputs?.length || 0) + (flow.outputs?.length || 0),
-                    trigger_configured: !!flow.trigger_type,
-                    sys_trigger_created: flow.trigger_type === 'record_created' || flow.trigger_type === 'record_updated',
-                    component_records_created: true, // New field to indicate component records were created
-                    has_actual_flow_instances: activitiesToProcess && activitiesToProcess.length > 0
-                }
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create flow:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Activate a flow to ensure it's published and ready
-     */
-    async activateFlow(flowSysId) {
-        try {
-            await this.ensureAuthenticated();
-            // Update the flow to set it as active and published
-            const response = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sys_hub_flow/${flowSysId}`, {
-                active: true,
-                status: 'published',
-                validated: true
-            });
-            if (!response.data || !response.data.result) {
-                throw new Error('Failed to activate flow - no response');
-            }
-            return response.data.result;
-        }
-        catch (error) {
-            console.error('Failed to activate flow:', error);
-            throw error;
-        }
-    }
-    /**
-     * Generate flow snapshot to prevent "Your flow cannot be found" error
-     * This uses the Flow Designer API to generate a proper snapshot
-     */
-    async generateFlowSnapshot(flowSysId) {
-        try {
-            await this.ensureAuthenticated();
-            // Call Flow Designer API to generate snapshot
-            // This endpoint triggers ServiceNow to build a proper flow snapshot
-            const response = await this.client.post(`${this.getBaseUrl()}/api/sn_flow_designer/flow/snapshot`, {
-                flow_id: flowSysId,
-                generate: true
-            });
-            if (!response.data || !response.data.result) {
-                // Try alternative endpoint if the first one fails
-                const altResponse = await this.client.put(`${this.getBaseUrl()}/api/now/table/sys_hub_flow/${flowSysId}`, {
-                    snapshot_ready: true
-                });
-                if (!altResponse.data || !altResponse.data.result) {
-                    throw new Error('Failed to generate flow snapshot');
-                }
-            }
-            return response.data?.result || true;
-        }
-        catch (error) {
-            console.error('Failed to generate flow snapshot:', error);
-            // Don't throw - this is a best effort operation
-        }
-    }
-    /**
-     * Create a Subflow in ServiceNow
-     */
-    async createSubflow(subflow) {
-        try {
-            this.logger.info('🔄 Creating Subflow...');
-            this.logger.info(`📋 Subflow: ${subflow.name}`);
-            // Add pre-deployment validation
-            this.validateFlowBeforeDeployment(subflow);
-            // Ensure Update Set
-            const updateSetResult = await this.ensureUpdateSet();
-            // Create subflow with inputs/outputs
-            const subflowData = {
-                name: subflow.name,
-                description: subflow.description || `Subflow: ${subflow.name}`,
-                active: subflow.active !== false,
-                internal_name: this.sanitizeInternalName(subflow.name),
-                category: subflow.category || 'custom',
-                type: 'subflow',
-                inputs: JSON.stringify(subflow.inputs || []),
-                outputs: JSON.stringify(subflow.outputs || []),
-                run_as: 'user_who_calls'
-            };
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow`, subflowData);
-            const subflowId = response.data.result.sys_id;
-            // Process flow definition if provided
-            if (subflow.flow_definition) {
-                const definition = typeof subflow.flow_definition === 'string'
-                    ? JSON.parse(subflow.flow_definition)
-                    : subflow.flow_definition;
-                // Create activities for the subflow
-                if (definition.activities) {
-                    for (let i = 0; i < definition.activities.length; i++) {
-                        const activity = definition.activities[i];
-                        await this.createFlowActionInstance(subflowId, activity, (i + 1) * 100);
-                    }
-                }
-            }
-            // Add post-deployment verification
-            await this.verifyDeployment(subflowId, 'subflow');
-            // 🔧 CRITICAL FIX: Enhanced response with proper ServiceNow URLs and subflow type
-            const subflowRecord = response.data.result;
-            const credentials = await this.oauth.loadCredentials();
-            const instance = credentials?.instance || process.env.SNOW_INSTANCE;
-            return {
-                success: true,
-                data: {
-                    ...subflowRecord,
-                    // Enhanced response format with proper URLs
-                    url: `https://${instance}/nav_to.do?uri=sys_hub_flow.do?sys_id=${subflowRecord.sys_id}`,
-                    flow_designer_url: `https://${instance}/flow-designer.do?sysparm_nostack=true&sysparm_sys_id=${subflowRecord.sys_id}`,
-                    type: 'subflow', // Ensure correct type
-                    activities_created: subflow.activities?.length || 0,
-                    inputs_defined: subflow.inputs?.length || 0,
-                    outputs_defined: subflow.outputs?.length || 0,
-                    reusable: true
-                }
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create subflow:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a Flow Action (reusable component)
-     */
-    async createFlowAction(action) {
-        try {
-            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');
-            }
-            if (!action.description || action.description.trim() === '') {
-                throw new Error('Flow Action description is required');
-            }
-            // Ensure Update Set
-            const updateSetResult = await this.ensureUpdateSet();
-            // Create action with inputs/outputs
-            const actionData = {
-                name: action.name,
-                description: action.description || `Action: ${action.name}`,
-                active: action.active !== false,
-                internal_name: this.sanitizeInternalName(action.name),
-                category: action.category || 'custom',
-                type: 'action',
-                action_type: action.action_type || 'custom',
-                inputs: JSON.stringify(action.inputs || []),
-                outputs: JSON.stringify(action.outputs || []),
-                script: action.script || '',
-                accessible_from: 'all' // Make available to all flows
-            };
-            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) {
-                this.logger.info(`✅ Flow Action created: ${action.name} (${response.data.result.sys_id})`);
-            }
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create flow action:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a flow action using discovered ServiceNow action types (private method)
-     */
-    async createFlowActionPrivate(flowId, action, order) {
-        try {
-            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
-            if (!actionTypeId) {
-                const searchResults = await this.searchFlowActions(action.type);
-                if (searchResults.actionTypes && searchResults.actionTypes.length > 0) {
-                    actionTypeId = searchResults.actionTypes[0].sys_id;
-                    this.logger.info(`📋 Using discovered action type: ${searchResults.actionTypes[0].name}`);
-                }
-                else {
-                    // Fallback to common action types
-                    const fallbackMap = {
-                        'notification': 'com.glideapp.servicenow_common.send_email',
-                        'field_update': 'com.glideapp.servicenow_common.update_record',
-                        'create_task': 'com.glideapp.servicenow_common.create_record',
-                        'log': 'com.glideapp.servicenow_common.log_message',
-                        'wait': 'com.glideapp.servicenow_common.timer',
-                        'approval': 'com.glideapp.servicenow_common.approval'
-                    };
-                    actionTypeId = fallbackMap[action.type] || 'com.glideapp.servicenow_common.script';
-                    this.logger.info(`📋 Using fallback action type: ${actionTypeId}`);
-                }
-            }
-            // Get action details to understand inputs/outputs
-            const actionDetails = await this.getFlowActionDetails(actionTypeId);
-            // Build inputs based on action details
-            const inputs = this.buildActionInputs(action, actionDetails);
-            const actionData = {
-                flow: flowId,
-                action_name: action.name,
-                action_type: actionTypeId,
-                order: order * 100,
-                active: true,
-                // Use proper ServiceNow action configuration
-                inputs: JSON.stringify(inputs),
-                configuration: this.buildActionConfiguration(action)
-            };
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_action_instance`, actionData);
-            this.logger.info(`✅ Flow action created: ${action.name}`);
-            return response.data.result;
-        }
-        catch (error) {
-            console.error('Failed to create flow action:', error);
-            throw error;
-        }
-    }
-    /**
-     * Build action inputs based on action details
-     */
-    buildActionInputs(action, actionDetails) {
-        const inputs = {};
-        if (!actionDetails || !actionDetails.inputs) {
-            return inputs;
-        }
-        // Map action properties to ServiceNow input variables
-        for (const input of actionDetails.inputs) {
-            const inputName = input.name;
-            switch (action.type) {
-                case 'notification':
-                    if (inputName === 'to' || inputName === 'recipient') {
-                        inputs[inputName] = action.to || '${trigger.record.assigned_to}';
-                    }
-                    else if (inputName === 'subject') {
-                        inputs[inputName] = action.subject || 'Flow Notification';
-                    }
-                    else if (inputName === 'message' || inputName === 'body') {
-                        inputs[inputName] = action.message || 'Flow action executed';
-                    }
-                    break;
-                case 'field_update':
-                    if (inputName === 'record' || inputName === 'table') {
-                        inputs[inputName] = '${trigger.record}';
-                    }
-                    else if (inputName === 'field') {
-                        inputs[inputName] = action.field;
-                    }
-                    else if (inputName === 'value') {
-                        inputs[inputName] = action.value;
-                    }
-                    break;
-                case 'create_task':
-                    if (inputName === 'table') {
-                        inputs[inputName] = action.target_table || 'task';
-                    }
-                    else if (inputName === 'fields') {
-                        inputs[inputName] = action.fields || {};
-                    }
-                    break;
-                case 'log':
-                    if (inputName === 'message') {
-                        inputs[inputName] = action.message || 'Flow action executed';
-                    }
-                    else if (inputName === 'level') {
-                        inputs[inputName] = action.level || 'info';
-                    }
-                    break;
-                case 'wait':
-                    if (inputName === 'duration') {
-                        inputs[inputName] = action.duration || 300;
-                    }
-                    break;
-                case 'approval':
-                    if (inputName === 'approvers') {
-                        inputs[inputName] = action.approvers || '${trigger.record.assigned_to.manager}';
-                    }
-                    break;
-            }
-        }
-        return inputs;
-    }
-    /**
-     * Build simple action configuration
-     */
-    buildActionConfiguration(action) {
-        const config = {
-            name: action.name,
-            type: action.type
-        };
-        switch (action.type) {
-            case 'notification':
-                config.to = action.to || 'assigned_to';
-                config.subject = action.subject || 'Flow Notification';
-                config.body = action.message || 'A flow action has been triggered.';
-                break;
-            case 'field_update':
-                config.field = action.field;
-                config.value = action.value;
-                break;
-            case 'create_task':
-                config.table = action.target_table || 'task';
-                config.fields = action.fields || {};
-                break;
-            case 'log':
-                config.message = action.message || 'Flow action executed';
-                config.level = action.level || 'info';
-                break;
-            case 'wait':
-                config.duration = action.duration || 300; // 5 minutes default
-                break;
-            case 'approval':
-                config.approvers = action.approvers || 'assigned_to.manager';
-                break;
-            default:
-                config.script = action.script || '// Custom action';
-        }
-        return JSON.stringify(config);
-    }
-    /**
-     * Create a flow trigger instance
-     */
-    async createFlowTrigger(flowId, trigger) {
-        try {
-            // Get trigger type from cache
-            let triggerType = await this.actionTypeCache.getTriggerType(trigger.type);
-            if (!triggerType) {
-                console.warn(`Trigger type '${trigger.type}' not found in cache, using default`);
-                // Fallback to 'Created' trigger if not found
-                triggerType = await this.actionTypeCache.getTriggerType('Created');
-                if (!triggerType) {
-                    throw new Error('Could not find any trigger types');
-                }
-            }
-            const triggerTypeId = triggerType.sys_id;
-            const triggerData = {
-                flow: flowId,
-                trigger_type: triggerTypeId,
-                table_name: trigger.table || 'incident',
-                condition: trigger.condition || '',
-                order: 0,
-                active: true
-            };
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_trigger_instance`, triggerData);
-            // 🔧 CRITICAL FIX: Create actual sys_trigger record for proper triggered flows
-            // This ensures the flow can actually be triggered by record events
-            // Only create sys_trigger for actual record events, not for manual or scheduled triggers
-            if (trigger.type === 'record_created' || trigger.type === 'record_updated' ||
-                trigger.type === 'Created' || trigger.type === 'Updated') {
-                try {
-                    const sysTriggerData = {
-                        name: `Flow Trigger: ${trigger.table}`,
-                        table: trigger.table || 'incident',
-                        when: trigger.type === 'record_created' ? 'after' : 'before',
-                        order: 100,
-                        active: true,
-                        condition: trigger.condition || '',
-                        script: `
-// Auto-generated trigger for Flow Designer flow
-// Flow ID: ${flowId}
-try {
-  var flowAPI = new sn_fd.FlowAPI();
-  var inputs = {};
-  inputs.table = '${trigger.table || 'incident'}';
-  inputs.sys_id = current.getUniqueValue();
-  flowAPI.startFlow('${flowId}', inputs);
-  gs.info('Flow ${flowId} triggered successfully for record: ' + current.getUniqueValue());
-} catch (e) {
-  gs.error('Flow trigger error: ' + e.message);
-}
-            `.trim(),
-                        description: `Auto-generated trigger for Flow Designer flow: ${flowId}`
-                    };
-                    await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_trigger`, sysTriggerData);
-                    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);
-                }
-            }
-            this.logger.info('✅ Trigger created successfully');
-            return response.data.result;
-        }
-        catch (error) {
-            console.error('Failed to create flow trigger:', error);
-            throw error;
-        }
-    }
-    /**
-     * Create flow variables for inputs and outputs
-     */
-    async createFlowVariables(flowId, flow) {
-        try {
-            this.logger.info('📋 Creating flow variables...');
-            // Process inputs
-            const inputs = flow.inputs || [];
-            if (inputs.length > 0) {
-                for (const input of inputs) {
-                    const variableData = {
-                        flow: flowId,
-                        name: input.name || input.id,
-                        label: input.label || input.name || input.id,
-                        type: input.type || 'string',
-                        input: true,
-                        output: false,
-                        required: input.required || false,
-                        default_value: input.default || '',
-                        description: input.description || `Input variable: ${input.name || input.id}`
-                    };
-                    await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_variable`, variableData);
-                }
-                this.logger.info(`✅ Created ${inputs.length} input variables`);
-            }
-            // Process outputs
-            const outputs = flow.outputs || [];
-            if (outputs.length > 0) {
-                for (const output of outputs) {
-                    const variableData = {
-                        flow: flowId,
-                        name: output.name || output.id,
-                        label: output.label || output.name || output.id,
-                        type: output.type || 'string',
-                        input: false,
-                        output: true,
-                        required: false,
-                        default_value: '',
-                        description: output.description || `Output variable: ${output.name || output.id}`
-                    };
-                    await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_variable`, variableData);
-                }
-                this.logger.info(`✅ Created ${outputs.length} output variables`);
-            }
-            // Extract variables from flow_definition if available
-            if (flow.flow_definition) {
-                try {
-                    const flowDef = typeof flow.flow_definition === 'string' ?
-                        JSON.parse(flow.flow_definition) : flow.flow_definition;
-                    if (flowDef.variables && Array.isArray(flowDef.variables)) {
-                        for (const variable of flowDef.variables) {
-                            const variableData = {
-                                flow: flowId,
-                                name: variable.name || variable.id,
-                                label: variable.label || variable.name || variable.id,
-                                type: variable.type || 'string',
-                                input: variable.input || false,
-                                output: variable.output || false,
-                                required: variable.required || false,
-                                default_value: variable.default || variable.value || '',
-                                description: variable.description || `Flow variable: ${variable.name || variable.id}`
-                            };
-                            await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_variable`, variableData);
-                        }
-                        this.logger.info(`✅ Created ${flowDef.variables.length} flow definition variables`);
-                    }
-                }
-                catch (parseError) {
-                    console.warn('Could not parse flow_definition for variables:', parseError);
-                }
-            }
-        }
-        catch (error) {
-            console.error('Failed to create flow variables:', error);
-            throw error;
-        }
-    }
-    /**
-     * Create a flow action instance using simplified approach
-     */
-    async createFlowActionInstance(flowId, action, order) {
-        try {
-            this.logger.info(`Creating action instance: ${action.name}`);
-            // Map action types to search terms
-            const actionSearchMap = {
-                'notification': 'Send Email',
-                'email': 'Send Email',
-                'field_update': 'Update Record',
-                'create_task': 'Create Record',
-                'create_record': 'Create Record',
-                'wait': 'Wait',
-                'approval': 'Approval',
-                'log': 'Log'
-            };
-            const searchTerm = actionSearchMap[action.type] || action.type;
-            let actionType = await this.actionTypeCache.getActionType(searchTerm);
-            if (!actionType) {
-                console.warn(`Action type '${searchTerm}' not found in cache, using Script action`);
-                // Fallback to Script action if not found
-                actionType = await this.actionTypeCache.getActionType('Script');
-                if (!actionType) {
-                    throw new Error('Could not find any action types');
-                }
-            }
-            const actionTypeId = actionType.sys_id;
-            // Build inputs based on action type
-            let inputs = {};
-            switch (action.type) {
-                case 'notification':
-                case 'email':
-                    inputs = {
-                        email_to: action.to || '${trigger.assigned_to.email}',
-                        email_subject: action.subject || 'Notification',
-                        email_body: action.message || 'Notification from flow'
-                    };
-                    break;
-                case 'field_update':
-                    inputs = {
-                        table: action.table || 'current',
-                        field: action.field || 'state',
-                        value: action.value || ''
-                    };
-                    break;
-                case 'create_task':
-                case 'create_record':
-                    inputs = {
-                        table: action.target_table || 'task',
-                        field_values: JSON.stringify({
-                            short_description: action.short_description || 'Task from flow',
-                            assigned_to: action.assigned_to || '${trigger.assigned_to}'
-                        })
-                    };
-                    break;
-                case 'wait':
-                    inputs = {
-                        duration: action.duration || 300,
-                        unit: 'seconds'
-                    };
-                    break;
-                case 'approval':
-                    inputs = {
-                        approvers: action.approvers || '${trigger.assigned_to.manager}',
-                        approval_field: 'approval'
-                    };
-                    break;
-                default:
-                    inputs = action.inputs || {};
-            }
-            const actionData = {
-                flow: flowId,
-                action_name: action.name,
-                action_type: actionTypeId,
-                order: order,
-                active: true,
-                inputs: JSON.stringify(inputs)
-            };
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_action_instance`, actionData);
-            this.logger.info(`✅ Action created: ${action.name}`);
-            return response.data.result;
-        }
-        catch (error) {
-            console.error(`Failed to create action instance ${action.name}:`, error);
-            throw error;
-        }
-    }
-    /**
-     * Create a flow operation (activity) for a Flow Designer flow
-     */
-    async createFlowOperation(flowId, operation) {
-        try {
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_operation`, {
-                flow: flowId,
-                name: operation.name,
-                type: operation.type || 'core',
-                action_type: operation.action_type || 'script',
-                order: operation.order || 100,
-                // Operation-specific configuration
-                configuration: JSON.stringify({
-                    script: operation.script,
-                    inputs: operation.inputs || {},
-                    outputs: operation.outputs || {},
-                    artifact_reference: operation.artifact_reference
-                })
-            });
-            return response.data.result;
-        }
-        catch (error) {
-            console.error('Failed to create flow operation:', error);
-            throw error;
-        }
-    }
-    /**
-     * Create a Script Include
-     */
-    async createScriptInclude(scriptInclude) {
-        try {
-            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,
-                description: scriptInclude.description,
-                script: scriptInclude.script,
-                active: scriptInclude.active,
-                access: scriptInclude.access || 'public'
-            });
-            this.logger.info('✅ Script Include created successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create script include:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a Business Rule
-     */
-    async createBusinessRule(businessRule) {
-        try {
-            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,
-                when: businessRule.when,
-                condition: businessRule.condition,
-                script: businessRule.script,
-                description: businessRule.description,
-                active: businessRule.active,
-                order: businessRule.order || 100
-            });
-            this.logger.info('✅ Business Rule created successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create business rule:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a Table
-     */
-    async createTable(table) {
-        try {
-            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,
-                extends_table: table.extends_table || 'sys_metadata',
-                is_extendable: table.is_extendable !== false,
-                access: table.access || 'public',
-                create_access_controls: table.create_access_controls !== false
-            });
-            this.logger.info('✅ Table created successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create table:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a Table Field
-     */
-    async createTableField(field) {
-        try {
-            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,
-                column_label: field.column_label,
-                internal_type: field.internal_type || 'string',
-                max_length: field.max_length || 255,
-                active: true
-            });
-            this.logger.info('✅ Table Field created successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create table field:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a new Update Set
-     */
-    async createUpdateSet(updateSet) {
-        try {
-            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`, {
-                name: updateSet.name,
-                description: updateSet.description,
-                release_date: updateSet.release_date,
-                state: updateSet.state || 'in_progress',
-                application: updateSet.application || 'global'
-            });
-            this.logger.info('✅ Update Set created successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to create Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Set current Update Set for the session
-     */
-    async setCurrentUpdateSet(updateSetId) {
-        try {
-            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
-            // First, check if a preference already exists
-            const existingPref = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_user_preference`, {
-                params: {
-                    sysparm_query: 'name=sys_update_set^user=javascript:gs.getUserID()',
-                    sysparm_limit: 1
-                }
-            });
-            let response;
-            if (existingPref.data.result && existingPref.data.result.length > 0) {
-                // Update existing preference
-                response = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sys_user_preference/${existingPref.data.result[0].sys_id}`, {
-                    value: updateSetId
-                });
-            }
-            else {
-                // Create new preference
-                response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_user_preference`, {
-                    name: 'sys_update_set',
-                    value: updateSetId,
-                    user: 'javascript:gs.getUserID()'
-                });
-            }
-            this.logger.info('✅ Current Update Set changed successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to set current Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get current Update Set
-     */
-    async getCurrentUpdateSet() {
-        try {
-            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
-            // This is more reliable than the UI API
-            try {
-                const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_user_preference`, {
-                    params: {
-                        sysparm_query: 'name=sys_update_set^user=javascript:gs.getUserID()',
-                        sysparm_limit: 1
-                    }
-                });
-                if (response.data.result && response.data.result.length > 0) {
-                    const updateSetId = response.data.result[0].value;
-                    // Get Update Set details
-                    const updateSetResponse = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_update_set/${updateSetId}`);
-                    return {
-                        success: true,
-                        data: updateSetResponse.data.result
-                    };
-                }
-            }
-            catch (prefError) {
-                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`, {
-                params: {
-                    sysparm_query: 'state=in_progress^sys_created_by=javascript:gs.getUserName()',
-                    sysparm_orderby: 'sys_created_on',
-                    sysparm_limit: 1
-                }
-            });
-            if (response.data.result && response.data.result.length > 0) {
-                return {
-                    success: true,
-                    data: response.data.result[0]
-                };
-            }
-            return {
-                success: false,
-                error: 'No current Update Set found'
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to get current Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Get Update Set by ID
-     */
-    async getUpdateSet(updateSetId) {
-        try {
-            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}`);
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to get Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * List Update Sets
-     */
-    async listUpdateSets(options) {
-        try {
-            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()';
-            if (options.state) {
-                query += `^state=${options.state}`;
-            }
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_update_set`, {
-                params: {
-                    sysparm_query: query,
-                    sysparm_limit: options.limit || 10,
-                    sysparm_orderby: 'sys_created_on^DESC'
-                }
-            });
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to list Update Sets:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Complete an Update Set
-     */
-    async completeUpdateSet(updateSetId, notes) {
-        try {
-            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
-            });
-            this.logger.info('✅ Update Set completed successfully!');
-            return {
-                success: true,
-                data: response.data.result
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to complete Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Activate an Update Set by setting it as current
-     */
-    async activateUpdateSet(updateSetId) {
-        try {
-            this.logger.info('🔄 Activating Update Set...');
-            // Set the update set as current
-            const result = await this.setCurrentUpdateSet(updateSetId);
-            if (result.success) {
-                // Also ensure the update set is in progress state
-                const updateResponse = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sys_update_set/${updateSetId}`, {
-                    state: 'in_progress'
-                });
-                this.logger.info('✅ Update Set activated successfully!');
-                return {
-                    success: true,
-                    data: updateResponse.data.result
-                };
-            }
-            return result;
-        }
-        catch (error) {
-            console.error('❌ Failed to activate Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Preview Update Set changes
-     */
-    async previewUpdateSet(updateSetId) {
-        try {
-            this.logger.info('🔍 Previewing Update Set changes...');
-            // Get Update Set details
-            const updateSetResponse = await this.getUpdateSet(updateSetId);
-            if (!updateSetResponse.success) {
-                return updateSetResponse;
-            }
-            // Get Update Set changes
-            const response = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_update_xml`, {
-                params: {
-                    sysparm_query: `update_set=${updateSetId}`,
-                    sysparm_limit: 1000
-                }
-            });
-            return {
-                success: true,
-                data: {
-                    ...updateSetResponse.data,
-                    changes: response.data.result
-                }
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to preview Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Export Update Set as XML
-     */
-    /**
-     * Ensure we have an active Update Set for tracking changes
-     */
-    async ensureUpdateSet() {
-        try {
-            // Check if we have a current Update Set
-            const currentUpdateSet = await this.getCurrentUpdateSet();
-            if (currentUpdateSet.success && currentUpdateSet.data) {
-                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]}`;
-            this.logger.info(`📦 Creating new Update Set: ${updateSetName}`);
-            const newUpdateSet = await this.createUpdateSet({
-                name: updateSetName,
-                description: 'Automated changes from Snow-Flow MCP',
-                state: 'in_progress'
-            });
-            if (newUpdateSet.success && newUpdateSet.data) {
-                // Set it as current
-                await this.setCurrentUpdateSet(newUpdateSet.data.sys_id);
-                return newUpdateSet;
-            }
-            // If Update Set creation fails, continue without it
-            console.warn('⚠️ Could not create Update Set, changes will not be tracked');
-            return { success: false, error: 'Update Set creation failed' };
-        }
-        catch (error) {
-            console.warn('⚠️ Update Set management failed:', error);
-            return { success: false, error: error instanceof Error ? error.message : String(error) };
-        }
-    }
-    async exportUpdateSet(updateSetId) {
-        try {
-            this.logger.info('📤 Exporting Update Set...');
-            // Get Update Set details first
-            const updateSetResponse = await this.getUpdateSet(updateSetId);
-            if (!updateSetResponse.success) {
-                return updateSetResponse;
-            }
-            // Use the unload processor to export
-            const response = await this.client.get(`${this.getBaseUrl()}/unload.do`, {
-                params: {
-                    sysparm_sys_id: updateSetId,
-                    sysparm_table: 'sys_update_set',
-                    sysparm_unload_format: 'xml'
-                },
-                responseType: 'text'
-            });
-            return {
-                success: true,
-                data: {
-                    name: updateSetResponse.data.name,
-                    xml: response.data,
-                    change_count: updateSetResponse.data.update_count || 0
-                }
-            };
-        }
-        catch (error) {
-            console.error('❌ Failed to export Update Set:', error);
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Debug flow structure - check what's missing for Flow Designer
-     */
-    async debugFlow(flowId) {
-        try {
-            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) {
-                return { success: false, error: 'Flow not found' };
-            }
-            const flow = flowResponse.data.result;
-            // Check for sys_hub_flow_logic entries
-            const logicResponse = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_logic?sysparm_query=flow=${flowId}`);
-            // Check for trigger instances
-            const triggerResponse = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_hub_trigger_instance?sysparm_query=flow=${flowId}`);
-            // Check for action instances
-            const actionResponse = await this.client.get(`${this.getBaseUrl()}/api/now/table/sys_hub_action_instance?sysparm_query=flow=${flowId}`);
-            return {
-                success: true,
-                data: {
-                    flow: {
-                        sys_id: flow.sys_id,
-                        name: flow.name,
-                        internal_name: flow.internal_name,
-                        status: flow.status,
-                        type: flow.type,
-                        sys_class_name: flow.sys_class_name,
-                        missing_fields: []
-                    },
-                    logic_count: logicResponse.data.result.length,
-                    trigger_count: triggerResponse.data.result.length,
-                    action_count: actionResponse.data.result.length,
-                    logic_entries: logicResponse.data.result,
-                    triggers: triggerResponse.data.result,
-                    actions: actionResponse.data.result
-                }
-            };
-        }
-        catch (error) {
-            return {
-                success: false,
-                error: error instanceof Error ? error.message : String(error)
-            };
-        }
-    }
-    /**
-     * Create a flow logic entry (visual representation in Flow Designer)
-     */
-    async createFlowLogic(flowId, logic) {
-        try {
-            const logicData = {
-                flow: flowId,
-                name: logic.name,
-                type: logic.type,
-                order: logic.order,
-                active: true,
-                instance: logic.instance
-            };
-            const response = await this.client.post(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_logic`, logicData);
-            return response.data.result;
-        }
-        catch (error) {
-            console.error('Failed to create flow logic:', error);
-            throw error;
-        }
-    }
-    /**
-     * Create a connection between flow logic elements
-     */
-    async createFlowConnection(flowId, fromId, toId) {
-        try {
-            // In ServiceNow, connections are stored as part of the flow logic
-            // We need to update the 'from' element to point to the 'to' element
-            const connectionData = {
-                next: toId
-            };
-            const response = await this.client.patch(`${this.getBaseUrl()}/api/now/table/sys_hub_flow_logic/${fromId}`, connectionData);
-            return response.data.result;
-        }
-        catch (error) {
-            console.error('Failed to create flow connection:', error);
-            // Don't throw - connections might work differently in some versions
-            return null;
-        }
-    }
-    /**
-     * Generic GET method for ServiceNow API calls
-     */
-    async get(endpoint, params) {
-        try {
-            await this.ensureAuthenticated();
-            const url = endpoint.startsWith('http') ? endpoint : `${this.getBaseUrl()}${endpoint}`;
-            const response = await this.client.get(url, { params });
-            return response.data;
-        }
-        catch (error) {
-            throw error;
-        }
-    }
-    /**
-     * Generic POST method for ServiceNow API calls
-     */
-    async post(endpoint, data) {
-        try {
-            await this.ensureAuthenticated();
-            const url = endpoint.startsWith('http') ? endpoint : `${this.getBaseUrl()}${endpoint}`;
-            const response = await this.client.post(url, data);
-            return response.data;
-        }
-        catch (error) {
-            throw error;
-        }
-    }
-    /**
-     * Generic PUT method for ServiceNow API calls
-     */
-    async put(endpoint, data) {
-        try {
-            await this.ensureAuthenticated();
-            const url = endpoint.startsWith('http') ? endpoint : `${this.getBaseUrl()}${endpoint}`;
-            const response = await this.client.put(url, data);
-            return response.data;
-        }
-        catch (error) {
-            throw error;
-        }
-    }
-    /**
-     * Generic PATCH method for ServiceNow API calls
-     */
-    async patch(endpoint, data) {
-        try {
-            await this.ensureAuthenticated();
-            const url = endpoint.startsWith('http') ? endpoint : `${this.getBaseUrl()}${endpoint}`;
-            const response = await this.client.patch(url, data);
-            return response.data;
-        }
-        catch (error) {
-            throw error;
-        }
-    }
-    /**
-     * Generic DELETE method for ServiceNow API calls
-     */
-    async delete(endpoint) {
-        try {
-            await this.ensureAuthenticated();
-            const url = endpoint.startsWith('http') ? endpoint : `${this.getBaseUrl()}${endpoint}`;
-            const response = await this.client.delete(url);
-            return response.data;
-        }
-        catch (error) {
-            throw error;
-        }
-    }
-    /**
-     * TEMPORARY FIX: makeRequest method to handle phantom calls
-     * This method provides compatibility for code that expects makeRequest
-     */
-    async makeRequest(config) {
-        // Debug logging removed - was causing noise in production
-        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;
-            switch (method) {
-                case 'get':
-                    return await this.get(url, config.params);
-                case 'post':
-                    return await this.post(url, data);
-                case 'put':
-                    return await this.put(url, data);
-                case 'patch':
-                    return await this.patch(url, data);
-                case 'delete':
-                    return await this.delete(url);
-                default:
-                    throw new Error(`Unsupported HTTP method: ${method}`);
-            }
-        }
-        catch (error) {
-            console.error('🔧 makeRequest error:', error);
-            throw error;
-        }
-    }
-}
-exports.ServiceNowClient = ServiceNowClient;
-//# sourceMappingURL=servicenow-client.js.map