snow-flow 3.0.13 → 3.0.14

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

@@ -1835,7 +1835,7 @@ ${args.widgets && args.widgets.length > 0 ? args.widgets.map((w, i) => `
                 switch (flowType) {
                     case 'flow':
                         // Create workflow record in ServiceNow
-                        result = await this.client.create('wf_workflow', {
+                        result = await this.client.createRecord('wf_workflow', {
                             name: flowData.name || `flow_${Date.now()}`,
                             description: flowData.description || 'Created by Snow-Flow',
                             table: flowData.table || 'incident',
@@ -1847,7 +1847,7 @@ ${args.widgets && args.widgets.length > 0 ? args.widgets.map((w, i) => `
                         break;
                     case 'subflow':
                         // Create subflow as a workflow activity
-                        result = await this.client.create('wf_workflow', {
+                        result = await this.client.createRecord('wf_workflow', {
                             name: flowData.name || `subflow_${Date.now()}`,
                             description: `${flowData.description || 'Subflow created by Snow-Flow'} [SUBFLOW]`,
                             table: flowData.table || 'incident',
@@ -1862,7 +1862,7 @@ ${args.widgets && args.widgets.length > 0 ? args.widgets.map((w, i) => `
                         if (!flowData.workflow_id) {
                             throw new Error('workflow_id is required for flow actions');
                         }
-                        result = await this.client.create('wf_activity', {
+                        result = await this.client.createRecord('wf_activity', {
                             workflow: flowData.workflow_id,
                             name: flowData.name || `action_${Date.now()}`,
                             script: flowData.script || '// Action script here',
@@ -2627,7 +2627,7 @@ ${hasErrors ? '❌ Validation failed - fix errors before deployment' : '✅ Vali
             let rollbackResult;
             try {
                 // Set update set to ignore state (ServiceNow's way of "rolling back")
-                rollbackResult = await this.client.update(`sys_update_set/${update_set_id}`, {
+                rollbackResult = await this.client.updateRecord(`sys_update_set/${update_set_id}`, {
                     state: 'ignore',
                     description: `${updateSet.description || ''} - ROLLED BACK: ${reason}`
                 });
@@ -3067,12 +3067,12 @@ ${deploymentList || 'No recent deployments found in the last 7 days'}
             if (existingArtifact) {
                 // Update existing artifact
                 const { sys_id, ...updateData } = artifact;
-                result = await this.client.update(`${tableName}/${sys_id}`, updateData);
+                result = await this.client.updateRecord(`${tableName}/${sys_id}`, updateData);
                 action = 'updated';
             }
             else {
                 // Create new artifact
-                result = await this.client.create(tableName, artifact);
+                result = await this.client.createRecord(tableName, artifact);
                 action = 'created';
             }
             if (!result?.result) {
@@ -3835,25 +3835,39 @@ This will use your .env credentials to start the OAuth flow and generate access
                             template: '<div>Test widget - safe to delete</div>',
                             description: 'Temporary test widget created by Snow-Flow MCP diagnostics'
                         };
-                        const createResult = await this.client.create('sp_widget', testWidget);
-                        if (createResult?.result?.sys_id) {
+                        const createResult = await this.client.createRecord('sp_widget', testWidget);
+                        if (createResult?.success && createResult?.data?.sys_id) {
                             // Immediately delete the test widget
                             try {
-                                await this.client.delete(`sp_widget/${createResult.result.sys_id}`);
+                                await this.client.deleteRecord('sp_widget', createResult.data.sys_id);
                                 realApiTests.writePermissions = {
                                     status: '✅ Full Access',
                                     description: 'Can create and delete artifacts - full deployment capability',
-                                    details: `Successfully created and cleaned up test widget ${createResult.result.sys_id}`
+                                    details: `Successfully created and cleaned up test widget ${createResult.data.sys_id}`
                                 };
                             }
                             catch (deleteError) {
                                 realApiTests.writePermissions = {
                                     status: '⚠️ Partial',
                                     description: 'Can create but cannot delete - cleanup may be needed',
-                                    details: `Created test widget ${createResult.result.sys_id} but failed to delete: ${deleteError instanceof Error ? deleteError.message : String(deleteError)}`
+                                    details: `Created test widget ${createResult.data.sys_id} but failed to delete: ${deleteError instanceof Error ? deleteError.message : String(deleteError)}`
                                 };
                             }
                         }
+                        else {
+                            // Log what we got for debugging
+                            this.logger.warn('Create succeeded but unexpected response structure:', {
+                                success: createResult?.success,
+                                hasData: !!createResult?.data,
+                                hasSysId: !!createResult?.data?.sys_id,
+                                dataKeys: createResult?.data ? Object.keys(createResult.data) : []
+                            });
+                            realApiTests.writePermissions = {
+                                status: '⚠️ Partial',
+                                description: 'Created widget but response structure unexpected',
+                                details: `Response structure: ${JSON.stringify(createResult?.data || {})}`
+                            };
+                        }
                     }
                     catch (writeError) {
                         realApiTests.writePermissions = {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "snow-flow",
-  "version": "3.0.13",
-  "description": "Snow-Flow v3.0.13: ZERO MOCK DATA GUARANTEE! 🔥 All reporting, dashboards, and analytics tools now use 100% REAL ServiceNow data with anti-mock validation. Advanced AI table discovery, race condition fixes, and comprehensive real-data enforcement. NO demo/sample/test data ever used - only live data from your actual ServiceNow instance. Plus intelligent reporting, deployment fixes, and 100+ MCP tools.",
+  "version": "3.0.14",
+  "description": "Snow-Flow v3.0.14: WRITE PERMISSIONS FIX! 🔧 Fixed 'this.client.create is not a function' error that was blocking all deployments. Corrected API method calls to use createRecord/deleteRecord. Write permissions diagnostic now works correctly. Plus all features: Zero Mock Data Guarantee, race condition fixes, intelligent reporting, and 100+ MCP tools.",
   "main": "dist/index.js",
   "type": "commonjs",
   "bin": {