n8n-mcp 2.21.1 → 2.22.5
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +1 -0
- package/data/nodes.db +0 -0
- package/dist/database/node-repository.d.ts +62 -0
- package/dist/database/node-repository.d.ts.map +1 -1
- package/dist/database/node-repository.js +276 -0
- package/dist/database/node-repository.js.map +1 -1
- package/dist/mcp/absolute-minimal.d.ts +4 -0
- package/dist/mcp/absolute-minimal.d.ts.map +1 -0
- package/dist/mcp/absolute-minimal.js +34 -0
- package/dist/mcp/absolute-minimal.js.map +1 -0
- package/dist/mcp/fixed-protocol.d.ts +17 -0
- package/dist/mcp/fixed-protocol.d.ts.map +1 -0
- package/dist/mcp/fixed-protocol.js +162 -0
- package/dist/mcp/fixed-protocol.js.map +1 -0
- package/dist/mcp/handlers-documentation.d.ts +18 -0
- package/dist/mcp/handlers-documentation.d.ts.map +1 -0
- package/dist/mcp/handlers-documentation.js +113 -0
- package/dist/mcp/handlers-documentation.js.map +1 -0
- package/dist/mcp/handlers-n8n-manager.d.ts +2 -1
- package/dist/mcp/handlers-n8n-manager.d.ts.map +1 -1
- package/dist/mcp/handlers-n8n-manager.js +193 -5
- package/dist/mcp/handlers-n8n-manager.js.map +1 -1
- package/dist/mcp/handlers-workflow-diff.d.ts +2 -1
- package/dist/mcp/handlers-workflow-diff.d.ts.map +1 -1
- package/dist/mcp/handlers-workflow-diff.js +30 -2
- package/dist/mcp/handlers-workflow-diff.js.map +1 -1
- package/dist/mcp/index.js +0 -0
- package/dist/mcp/minimal-test.d.ts +3 -0
- package/dist/mcp/minimal-test.d.ts.map +1 -0
- package/dist/mcp/minimal-test.js +83 -0
- package/dist/mcp/minimal-test.js.map +1 -0
- package/dist/mcp/server.d.ts.map +1 -1
- package/dist/mcp/server.js +5 -2
- package/dist/mcp/server.js.map +1 -1
- package/dist/mcp/stderr-test.d.ts +2 -0
- package/dist/mcp/stderr-test.d.ts.map +1 -0
- package/dist/mcp/stderr-test.js +32 -0
- package/dist/mcp/stderr-test.js.map +1 -0
- package/dist/mcp/tool-docs/special/code-node-guide.d.ts +3 -0
- package/dist/mcp/tool-docs/special/code-node-guide.d.ts.map +1 -0
- package/dist/mcp/tool-docs/special/code-node-guide.js +61 -0
- package/dist/mcp/tool-docs/special/code-node-guide.js.map +1 -0
- package/dist/mcp/tool-docs/special/index.d.ts +2 -0
- package/dist/mcp/tool-docs/special/index.d.ts.map +1 -0
- package/dist/mcp/tool-docs/special/index.js +6 -0
- package/dist/mcp/tool-docs/special/index.js.map +1 -0
- package/dist/mcp/tool-docs/templates/get-node-for-task.d.ts +3 -0
- package/dist/mcp/tool-docs/templates/get-node-for-task.d.ts.map +1 -0
- package/dist/mcp/tool-docs/templates/get-node-for-task.js +50 -0
- package/dist/mcp/tool-docs/templates/get-node-for-task.js.map +1 -0
- package/dist/mcp/tool-docs/workflow_management/n8n-autofix-workflow.d.ts.map +1 -1
- package/dist/mcp/tool-docs/workflow_management/n8n-autofix-workflow.js +47 -11
- package/dist/mcp/tool-docs/workflow_management/n8n-autofix-workflow.js.map +1 -1
- package/dist/mcp/tool-docs/workflow_management/n8n-update-partial-workflow.d.ts.map +1 -1
- package/dist/mcp/tool-docs/workflow_management/n8n-update-partial-workflow.js +2 -0
- package/dist/mcp/tool-docs/workflow_management/n8n-update-partial-workflow.js.map +1 -1
- package/dist/mcp/tools-documentation-new.d.ts +6 -0
- package/dist/mcp/tools-documentation-new.d.ts.map +1 -0
- package/dist/mcp/tools-documentation-new.js +163 -0
- package/dist/mcp/tools-documentation-new.js.map +1 -0
- package/dist/mcp/tools-n8n-manager.d.ts.map +1 -1
- package/dist/mcp/tools-n8n-manager.js +55 -1
- package/dist/mcp/tools-n8n-manager.js.map +1 -1
- package/dist/mcp/ultra-minimal.d.ts +8 -0
- package/dist/mcp/ultra-minimal.d.ts.map +1 -0
- package/dist/mcp/ultra-minimal.js +93 -0
- package/dist/mcp/ultra-minimal.js.map +1 -0
- package/dist/mcp/working-test.d.ts +3 -0
- package/dist/mcp/working-test.d.ts.map +1 -0
- package/dist/mcp/working-test.js +48 -0
- package/dist/mcp/working-test.js.map +1 -0
- package/dist/mcp/wrapper.sh +28 -0
- package/dist/scripts/debug-n8n-auth.d.ts +3 -0
- package/dist/scripts/debug-n8n-auth.d.ts.map +1 -0
- package/dist/scripts/debug-n8n-auth.js +97 -0
- package/dist/scripts/debug-n8n-auth.js.map +1 -0
- package/dist/scripts/debug-node.d.ts +3 -0
- package/dist/scripts/debug-node.d.ts.map +1 -0
- package/dist/scripts/debug-node.js +59 -0
- package/dist/scripts/debug-node.js.map +1 -0
- package/dist/scripts/generate-metadata-only.d.ts +3 -0
- package/dist/scripts/generate-metadata-only.d.ts.map +1 -0
- package/dist/scripts/generate-metadata-only.js +146 -0
- package/dist/scripts/generate-metadata-only.js.map +1 -0
- package/dist/scripts/generate-metadata-sync.d.ts +3 -0
- package/dist/scripts/generate-metadata-sync.d.ts.map +1 -0
- package/dist/scripts/generate-metadata-sync.js +136 -0
- package/dist/scripts/generate-metadata-sync.js.map +1 -0
- package/dist/scripts/test-ai-workflow-validation.d.ts +3 -0
- package/dist/scripts/test-ai-workflow-validation.d.ts.map +1 -0
- package/dist/scripts/test-ai-workflow-validation.js +191 -0
- package/dist/scripts/test-ai-workflow-validation.js.map +1 -0
- package/dist/scripts/test-api-headers.d.ts +3 -0
- package/dist/scripts/test-api-headers.d.ts.map +1 -0
- package/dist/scripts/test-api-headers.js +35 -0
- package/dist/scripts/test-api-headers.js.map +1 -0
- package/dist/scripts/test-autofix-workflow.js +3 -3
- package/dist/scripts/test-autofix-workflow.js.map +1 -1
- package/dist/scripts/test-batch-processing.d.ts +3 -0
- package/dist/scripts/test-batch-processing.d.ts.map +1 -0
- package/dist/scripts/test-batch-processing.js +144 -0
- package/dist/scripts/test-batch-processing.js.map +1 -0
- package/dist/scripts/test-docker-config-simulation.d.ts +3 -0
- package/dist/scripts/test-docker-config-simulation.d.ts.map +1 -0
- package/dist/scripts/test-docker-config-simulation.js +74 -0
- package/dist/scripts/test-docker-config-simulation.js.map +1 -0
- package/dist/scripts/test-enhanced-validation.d.ts +3 -0
- package/dist/scripts/test-enhanced-validation.d.ts.map +1 -0
- package/dist/scripts/test-enhanced-validation.js +117 -0
- package/dist/scripts/test-enhanced-validation.js.map +1 -0
- package/dist/scripts/test-error-handling-validation.d.ts +3 -0
- package/dist/scripts/test-error-handling-validation.d.ts.map +1 -0
- package/dist/scripts/test-error-handling-validation.js +340 -0
- package/dist/scripts/test-error-handling-validation.js.map +1 -0
- package/dist/scripts/test-issue-45-fix.d.ts +3 -0
- package/dist/scripts/test-issue-45-fix.d.ts.map +1 -0
- package/dist/scripts/test-issue-45-fix.js +136 -0
- package/dist/scripts/test-issue-45-fix.js.map +1 -0
- package/dist/scripts/test-issue-74.d.ts +3 -0
- package/dist/scripts/test-issue-74.d.ts.map +1 -0
- package/dist/scripts/test-issue-74.js +112 -0
- package/dist/scripts/test-issue-74.js.map +1 -0
- package/dist/scripts/test-lazy-config.d.ts +3 -0
- package/dist/scripts/test-lazy-config.d.ts.map +1 -0
- package/dist/scripts/test-lazy-config.js +60 -0
- package/dist/scripts/test-lazy-config.js.map +1 -0
- package/dist/scripts/test-limited-results.d.ts +3 -0
- package/dist/scripts/test-limited-results.d.ts.map +1 -0
- package/dist/scripts/test-limited-results.js +76 -0
- package/dist/scripts/test-limited-results.js.map +1 -0
- package/dist/scripts/test-mcp-n8n-update-partial.d.ts +3 -0
- package/dist/scripts/test-mcp-n8n-update-partial.d.ts.map +1 -0
- package/dist/scripts/test-mcp-n8n-update-partial.js +138 -0
- package/dist/scripts/test-mcp-n8n-update-partial.js.map +1 -0
- package/dist/scripts/test-mcp-tools.d.ts +3 -0
- package/dist/scripts/test-mcp-tools.d.ts.map +1 -0
- package/dist/scripts/test-mcp-tools.js +36 -0
- package/dist/scripts/test-mcp-tools.js.map +1 -0
- package/dist/scripts/test-metadata-generation.d.ts +3 -0
- package/dist/scripts/test-metadata-generation.d.ts.map +1 -0
- package/dist/scripts/test-metadata-generation.js +96 -0
- package/dist/scripts/test-metadata-generation.js.map +1 -0
- package/dist/scripts/test-n8n-manager-integration.d.ts +3 -0
- package/dist/scripts/test-n8n-manager-integration.d.ts.map +1 -0
- package/dist/scripts/test-n8n-manager-integration.js +122 -0
- package/dist/scripts/test-n8n-manager-integration.js.map +1 -0
- package/dist/scripts/test-n8n-validate-workflow.d.ts +3 -0
- package/dist/scripts/test-n8n-validate-workflow.d.ts.map +1 -0
- package/dist/scripts/test-n8n-validate-workflow.js +125 -0
- package/dist/scripts/test-n8n-validate-workflow.js.map +1 -0
- package/dist/scripts/test-node-level-properties.d.ts +3 -0
- package/dist/scripts/test-node-level-properties.d.ts.map +1 -0
- package/dist/scripts/test-node-level-properties.js +196 -0
- package/dist/scripts/test-node-level-properties.js.map +1 -0
- package/dist/scripts/test-node-suggestions.js +1 -1
- package/dist/scripts/test-node-suggestions.js.map +1 -1
- package/dist/scripts/test-nodes.d.ts +3 -0
- package/dist/scripts/test-nodes.d.ts.map +1 -0
- package/dist/scripts/test-nodes.js +91 -0
- package/dist/scripts/test-nodes.js.map +1 -0
- package/dist/scripts/test-single-workflow.d.ts +3 -0
- package/dist/scripts/test-single-workflow.d.ts.map +1 -0
- package/dist/scripts/test-single-workflow.js +112 -0
- package/dist/scripts/test-single-workflow.js.map +1 -0
- package/dist/scripts/test-template-validation.d.ts +3 -0
- package/dist/scripts/test-template-validation.d.ts.map +1 -0
- package/dist/scripts/test-template-validation.js +142 -0
- package/dist/scripts/test-template-validation.js.map +1 -0
- package/dist/scripts/test-templates.d.ts +4 -0
- package/dist/scripts/test-templates.d.ts.map +1 -0
- package/dist/scripts/test-templates.js +99 -0
- package/dist/scripts/test-templates.js.map +1 -0
- package/dist/scripts/test-tools-documentation.d.ts +2 -0
- package/dist/scripts/test-tools-documentation.d.ts.map +1 -0
- package/dist/scripts/test-tools-documentation.js +41 -0
- package/dist/scripts/test-tools-documentation.js.map +1 -0
- package/dist/scripts/test-transactional-diff.d.ts +2 -0
- package/dist/scripts/test-transactional-diff.d.ts.map +1 -0
- package/dist/scripts/test-transactional-diff.js +240 -0
- package/dist/scripts/test-transactional-diff.js.map +1 -0
- package/dist/scripts/test-update-partial-debug.d.ts +3 -0
- package/dist/scripts/test-update-partial-debug.d.ts.map +1 -0
- package/dist/scripts/test-update-partial-debug.js +92 -0
- package/dist/scripts/test-update-partial-debug.js.map +1 -0
- package/dist/scripts/test-updatenode-null-fix.d.ts +3 -0
- package/dist/scripts/test-updatenode-null-fix.d.ts.map +1 -0
- package/dist/scripts/test-updatenode-null-fix.js +156 -0
- package/dist/scripts/test-updatenode-null-fix.js.map +1 -0
- package/dist/scripts/test-validation-parity.d.ts +2 -0
- package/dist/scripts/test-validation-parity.d.ts.map +1 -0
- package/dist/scripts/test-validation-parity.js +153 -0
- package/dist/scripts/test-validation-parity.js.map +1 -0
- package/dist/scripts/test-version-extraction.d.ts +2 -0
- package/dist/scripts/test-version-extraction.d.ts.map +1 -0
- package/dist/scripts/test-version-extraction.js +74 -0
- package/dist/scripts/test-version-extraction.js.map +1 -0
- package/dist/scripts/test-webhook-autofix.js +1 -1
- package/dist/scripts/test-webhook-autofix.js.map +1 -1
- package/dist/scripts/test-workflow-diff.d.ts +3 -0
- package/dist/scripts/test-workflow-diff.d.ts.map +1 -0
- package/dist/scripts/test-workflow-diff.js +328 -0
- package/dist/scripts/test-workflow-diff.js.map +1 -0
- package/dist/scripts/test-workflow-validation.d.ts +3 -0
- package/dist/scripts/test-workflow-validation.d.ts.map +1 -0
- package/dist/scripts/test-workflow-validation.js +238 -0
- package/dist/scripts/test-workflow-validation.js.map +1 -0
- package/dist/services/breaking-change-detector.d.ts +38 -0
- package/dist/services/breaking-change-detector.d.ts.map +1 -0
- package/dist/services/breaking-change-detector.js +184 -0
- package/dist/services/breaking-change-detector.js.map +1 -0
- package/dist/services/breaking-changes-registry.d.ts +28 -0
- package/dist/services/breaking-changes-registry.d.ts.map +1 -0
- package/dist/services/breaking-changes-registry.js +200 -0
- package/dist/services/breaking-changes-registry.js.map +1 -0
- package/dist/services/enhanced-config-validator.d.ts.map +1 -1
- package/dist/services/enhanced-config-validator.js +32 -0
- package/dist/services/enhanced-config-validator.js.map +1 -1
- package/dist/services/n8n-api-client.d.ts +1 -0
- package/dist/services/n8n-api-client.d.ts.map +1 -1
- package/dist/services/n8n-api-client.js +26 -4
- package/dist/services/n8n-api-client.js.map +1 -1
- package/dist/services/node-migration-service.d.ts +44 -0
- package/dist/services/node-migration-service.d.ts.map +1 -0
- package/dist/services/node-migration-service.js +231 -0
- package/dist/services/node-migration-service.js.map +1 -0
- package/dist/services/node-version-service.d.ts +63 -0
- package/dist/services/node-version-service.d.ts.map +1 -0
- package/dist/services/node-version-service.js +215 -0
- package/dist/services/node-version-service.js.map +1 -0
- package/dist/services/post-update-validator.d.ts +59 -0
- package/dist/services/post-update-validator.d.ts.map +1 -0
- package/dist/services/post-update-validator.js +231 -0
- package/dist/services/post-update-validator.js.map +1 -0
- package/dist/services/workflow-auto-fixer.d.ts +10 -2
- package/dist/services/workflow-auto-fixer.d.ts.map +1 -1
- package/dist/services/workflow-auto-fixer.js +124 -3
- package/dist/services/workflow-auto-fixer.js.map +1 -1
- package/dist/services/workflow-diff-engine.d.ts +1 -0
- package/dist/services/workflow-diff-engine.d.ts.map +1 -1
- package/dist/services/workflow-diff-engine.js +23 -1
- package/dist/services/workflow-diff-engine.js.map +1 -1
- package/dist/services/workflow-ownership.d.ts +35 -0
- package/dist/services/workflow-ownership.d.ts.map +1 -0
- package/dist/services/workflow-ownership.js +195 -0
- package/dist/services/workflow-ownership.js.map +1 -0
- package/dist/services/workflow-versioning-service.d.ts +102 -0
- package/dist/services/workflow-versioning-service.d.ts.map +1 -0
- package/dist/services/workflow-versioning-service.js +264 -0
- package/dist/services/workflow-versioning-service.js.map +1 -0
- package/dist/sse-server.d.ts +8 -0
- package/dist/sse-server.d.ts.map +1 -0
- package/dist/sse-server.js +652 -0
- package/dist/sse-server.js.map +1 -0
- package/dist/types/session-restoration.d.ts +25 -0
- package/dist/types/session-restoration.d.ts.map +1 -0
- package/dist/types/session-restoration.js +3 -0
- package/dist/types/session-restoration.js.map +1 -0
- package/dist/types/sse.d.ts +42 -0
- package/dist/types/sse.d.ts.map +1 -0
- package/dist/types/sse.js +3 -0
- package/dist/types/sse.js.map +1 -0
- package/dist/types/workflow-diff.d.ts +1 -0
- package/dist/types/workflow-diff.d.ts.map +1 -1
- package/dist/types/workflow-diff.js.map +1 -1
- package/dist/utils/sse-session-manager.d.ts +23 -0
- package/dist/utils/sse-session-manager.d.ts.map +1 -0
- package/dist/utils/sse-session-manager.js +178 -0
- package/dist/utils/sse-session-manager.js.map +1 -0
- package/dist/utils/version-utils.d.ts +16 -0
- package/dist/utils/version-utils.d.ts.map +1 -0
- package/dist/utils/version-utils.js +129 -0
- package/dist/utils/version-utils.js.map +1 -0
- package/package.json +1 -10
|
@@ -0,0 +1,117 @@
|
|
|
1
|
+
#!/usr/bin/env ts-node
|
|
2
|
+
"use strict";
|
|
3
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
4
|
+
const config_validator_1 = require("../services/config-validator");
|
|
5
|
+
const enhanced_config_validator_1 = require("../services/enhanced-config-validator");
|
|
6
|
+
const database_adapter_1 = require("../database/database-adapter");
|
|
7
|
+
const node_repository_1 = require("../database/node-repository");
|
|
8
|
+
async function testValidation() {
|
|
9
|
+
const db = await (0, database_adapter_1.createDatabaseAdapter)('./data/nodes.db');
|
|
10
|
+
const repository = new node_repository_1.NodeRepository(db);
|
|
11
|
+
console.log('🧪 Testing Enhanced Validation System\n');
|
|
12
|
+
console.log('='.repeat(60));
|
|
13
|
+
console.log('\n📧 Test Case 1: Slack Send Message');
|
|
14
|
+
console.log('-'.repeat(40));
|
|
15
|
+
const slackConfig = {
|
|
16
|
+
resource: 'message',
|
|
17
|
+
operation: 'send',
|
|
18
|
+
channel: '#general',
|
|
19
|
+
text: 'Hello from n8n!'
|
|
20
|
+
};
|
|
21
|
+
const slackNode = repository.getNode('nodes-base.slack');
|
|
22
|
+
if (slackNode && slackNode.properties) {
|
|
23
|
+
console.log('\n❌ OLD Validation (validate_node_config):');
|
|
24
|
+
const oldResult = config_validator_1.ConfigValidator.validate('nodes-base.slack', slackConfig, slackNode.properties);
|
|
25
|
+
console.log(` Errors: ${oldResult.errors.length}`);
|
|
26
|
+
console.log(` Warnings: ${oldResult.warnings.length}`);
|
|
27
|
+
console.log(` Visible Properties: ${oldResult.visibleProperties.length}`);
|
|
28
|
+
if (oldResult.errors.length > 0) {
|
|
29
|
+
console.log('\n Sample errors:');
|
|
30
|
+
oldResult.errors.slice(0, 3).forEach(err => {
|
|
31
|
+
console.log(` - ${err.message}`);
|
|
32
|
+
});
|
|
33
|
+
}
|
|
34
|
+
console.log('\n✅ NEW Validation (validate_node_operation):');
|
|
35
|
+
const newResult = enhanced_config_validator_1.EnhancedConfigValidator.validateWithMode('nodes-base.slack', slackConfig, slackNode.properties, 'operation');
|
|
36
|
+
console.log(` Errors: ${newResult.errors.length}`);
|
|
37
|
+
console.log(` Warnings: ${newResult.warnings.length}`);
|
|
38
|
+
console.log(` Mode: ${newResult.mode}`);
|
|
39
|
+
console.log(` Operation: ${newResult.operation?.resource}/${newResult.operation?.operation}`);
|
|
40
|
+
if (newResult.examples && newResult.examples.length > 0) {
|
|
41
|
+
console.log('\n 📚 Examples provided:');
|
|
42
|
+
newResult.examples.forEach(ex => {
|
|
43
|
+
console.log(` - ${ex.description}`);
|
|
44
|
+
});
|
|
45
|
+
}
|
|
46
|
+
if (newResult.nextSteps && newResult.nextSteps.length > 0) {
|
|
47
|
+
console.log('\n 🎯 Next steps:');
|
|
48
|
+
newResult.nextSteps.forEach(step => {
|
|
49
|
+
console.log(` - ${step}`);
|
|
50
|
+
});
|
|
51
|
+
}
|
|
52
|
+
}
|
|
53
|
+
console.log('\n\n📊 Test Case 2: Google Sheets Append (with errors)');
|
|
54
|
+
console.log('-'.repeat(40));
|
|
55
|
+
const sheetsConfigBad = {
|
|
56
|
+
operation: 'append',
|
|
57
|
+
};
|
|
58
|
+
const sheetsNode = repository.getNode('nodes-base.googleSheets');
|
|
59
|
+
if (sheetsNode && sheetsNode.properties) {
|
|
60
|
+
const result = enhanced_config_validator_1.EnhancedConfigValidator.validateWithMode('nodes-base.googleSheets', sheetsConfigBad, sheetsNode.properties, 'operation');
|
|
61
|
+
console.log(`\n Validation result:`);
|
|
62
|
+
console.log(` Valid: ${result.valid}`);
|
|
63
|
+
console.log(` Errors: ${result.errors.length}`);
|
|
64
|
+
if (result.errors.length > 0) {
|
|
65
|
+
console.log('\n Errors found:');
|
|
66
|
+
result.errors.forEach(err => {
|
|
67
|
+
console.log(` - ${err.message}`);
|
|
68
|
+
if (err.fix)
|
|
69
|
+
console.log(` Fix: ${err.fix}`);
|
|
70
|
+
});
|
|
71
|
+
}
|
|
72
|
+
if (result.examples && result.examples.length > 0) {
|
|
73
|
+
console.log('\n 📚 Working examples provided:');
|
|
74
|
+
result.examples.forEach(ex => {
|
|
75
|
+
console.log(` - ${ex.description}:`);
|
|
76
|
+
console.log(` ${JSON.stringify(ex.config, null, 2).split('\n').join('\n ')}`);
|
|
77
|
+
});
|
|
78
|
+
}
|
|
79
|
+
}
|
|
80
|
+
console.log('\n\n💬 Test Case 3: Slack Update Message');
|
|
81
|
+
console.log('-'.repeat(40));
|
|
82
|
+
const slackUpdateConfig = {
|
|
83
|
+
resource: 'message',
|
|
84
|
+
operation: 'update',
|
|
85
|
+
channel: '#general',
|
|
86
|
+
text: 'Updated message'
|
|
87
|
+
};
|
|
88
|
+
if (slackNode && slackNode.properties) {
|
|
89
|
+
const result = enhanced_config_validator_1.EnhancedConfigValidator.validateWithMode('nodes-base.slack', slackUpdateConfig, slackNode.properties, 'operation');
|
|
90
|
+
console.log(`\n Validation result:`);
|
|
91
|
+
console.log(` Valid: ${result.valid}`);
|
|
92
|
+
console.log(` Errors: ${result.errors.length}`);
|
|
93
|
+
result.errors.forEach(err => {
|
|
94
|
+
console.log(` - Property: ${err.property}`);
|
|
95
|
+
console.log(` Message: ${err.message}`);
|
|
96
|
+
console.log(` Fix: ${err.fix}`);
|
|
97
|
+
});
|
|
98
|
+
}
|
|
99
|
+
console.log('\n\n📈 Summary: Old vs New Validation');
|
|
100
|
+
console.log('='.repeat(60));
|
|
101
|
+
console.log('\nOLD validate_node_config:');
|
|
102
|
+
console.log(' ❌ Validates ALL properties regardless of operation');
|
|
103
|
+
console.log(' ❌ Many false positives for complex nodes');
|
|
104
|
+
console.log(' ❌ Generic error messages');
|
|
105
|
+
console.log(' ❌ No examples or next steps');
|
|
106
|
+
console.log('\nNEW validate_node_operation:');
|
|
107
|
+
console.log(' ✅ Only validates properties for selected operation');
|
|
108
|
+
console.log(' ✅ 80%+ reduction in false positives');
|
|
109
|
+
console.log(' ✅ Operation-specific error messages');
|
|
110
|
+
console.log(' ✅ Includes working examples when errors found');
|
|
111
|
+
console.log(' ✅ Provides actionable next steps');
|
|
112
|
+
console.log(' ✅ Auto-fix suggestions for common issues');
|
|
113
|
+
console.log('\n✨ The enhanced validation makes AI agents much more effective!');
|
|
114
|
+
db.close();
|
|
115
|
+
}
|
|
116
|
+
testValidation().catch(console.error);
|
|
117
|
+
//# sourceMappingURL=test-enhanced-validation.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"test-enhanced-validation.js","sourceRoot":"","sources":["../../src/scripts/test-enhanced-validation.ts"],"names":[],"mappings":";;;AAWA,mEAA+D;AAC/D,qFAAgF;AAChF,mEAAqE;AACrE,iEAA6D;AAG7D,KAAK,UAAU,cAAc;IAC3B,MAAM,EAAE,GAAG,MAAM,IAAA,wCAAqB,EAAC,iBAAiB,CAAC,CAAC;IAC1D,MAAM,UAAU,GAAG,IAAI,gCAAc,CAAC,EAAE,CAAC,CAAC;IAE1C,OAAO,CAAC,GAAG,CAAC,yCAAyC,CAAC,CAAC;IACvD,OAAO,CAAC,GAAG,CAAC,GAAG,CAAE,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC;IAG7B,OAAO,CAAC,GAAG,CAAC,sCAAsC,CAAC,CAAC;IACpD,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC;IAE5B,MAAM,WAAW,GAAG;QAClB,QAAQ,EAAE,SAAS;QACnB,SAAS,EAAE,MAAM;QACjB,OAAO,EAAE,UAAU;QACnB,IAAI,EAAE,iBAAiB;KACxB,CAAC;IAEF,MAAM,SAAS,GAAG,UAAU,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;IACzD,IAAI,SAAS,IAAI,SAAS,CAAC,UAAU,EAAE,CAAC;QAEtC,OAAO,CAAC,GAAG,CAAC,4CAA4C,CAAC,CAAC;QAC1D,MAAM,SAAS,GAAG,kCAAe,CAAC,QAAQ,CAAC,kBAAkB,EAAE,WAAW,EAAE,SAAS,CAAC,UAAU,CAAC,CAAC;QAClG,OAAO,CAAC,GAAG,CAAC,aAAa,SAAS,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC;QACpD,OAAO,CAAC,GAAG,CAAC,eAAe,SAAS,CAAC,QAAQ,CAAC,MAAM,EAAE,CAAC,CAAC;QACxD,OAAO,CAAC,GAAG,CAAC,yBAAyB,SAAS,CAAC,iBAAiB,CAAC,MAAM,EAAE,CAAC,CAAC;QAC3E,IAAI,SAAS,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAChC,OAAO,CAAC,GAAG,CAAC,oBAAoB,CAAC,CAAC;YAClC,SAAS,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE;gBACzC,OAAO,CAAC,GAAG,CAAC,SAAS,GAAG,CAAC,OAAO,EAAE,CAAC,CAAC;YACtC,CAAC,CAAC,CAAC;QACL,CAAC;QAGD,OAAO,CAAC,GAAG,CAAC,+CAA+C,CAAC,CAAC;QAC7D,MAAM,SAAS,GAAG,mDAAuB,CAAC,gBAAgB,CACxD,kBAAkB,EAClB,WAAW,EACX,SAAS,CAAC,UAAU,EACpB,WAAW,CACZ,CAAC;QACF,OAAO,CAAC,GAAG,CAAC,aAAa,SAAS,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC;QACpD,OAAO,CAAC,GAAG,CAAC,eAAe,SAAS,CAAC,QAAQ,CAAC,MAAM,EAAE,CAAC,CAAC;QACxD,OAAO,CAAC,GAAG,CAAC,WAAW,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC;QACzC,OAAO,CAAC,GAAG,CAAC,gBAAgB,SAAS,CAAC,SAAS,EAAE,QAAQ,IAAI,SAAS,CAAC,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;QAE/F,IAAI,SAAS,CAAC,QAAQ,IAAI,SAAS,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACxD,OAAO,CAAC,GAAG,CAAC,2BAA2B,CAAC,CAAC;YACzC,SAAS,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE;gBAC9B,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC;YACzC,CAAC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,SAAS,CAAC,SAAS,IAAI,SAAS,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC1D,OAAO,CAAC,GAAG,CAAC,oBAAoB,CAAC,CAAC;YAClC,SAAS,CAAC,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE;gBACjC,OAAO,CAAC,GAAG,CAAC,SAAS,IAAI,EAAE,CAAC,CAAC;YAC/B,CAAC,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAGD,OAAO,CAAC,GAAG,CAAC,wDAAwD,CAAC,CAAC;IACtE,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC;IAE5B,MAAM,eAAe,GAAG;QACtB,SAAS,EAAE,QAAQ;KAEpB,CAAC;IAEF,MAAM,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,yBAAyB,CAAC,CAAC;IACjE,IAAI,UAAU,IAAI,UAAU,CAAC,UAAU,EAAE,CAAC;QACxC,MAAM,MAAM,GAAG,mDAAuB,CAAC,gBAAgB,CACrD,yBAAyB,EACzB,eAAe,EACf,UAAU,CAAC,UAAU,EACrB,WAAW,CACZ,CAAC;QAEF,OAAO,CAAC,GAAG,CAAC,wBAAwB,CAAC,CAAC;QACtC,OAAO,CAAC,GAAG,CAAC,YAAY,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC;QACxC,OAAO,CAAC,GAAG,CAAC,aAAa,MAAM,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC;QAEjD,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7B,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAC;YACjC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE;gBAC1B,OAAO,CAAC,GAAG,CAAC,SAAS,GAAG,CAAC,OAAO,EAAE,CAAC,CAAC;gBACpC,IAAI,GAAG,CAAC,GAAG;oBAAE,OAAO,CAAC,GAAG,CAAC,cAAc,GAAG,CAAC,GAAG,EAAE,CAAC,CAAC;YACpD,CAAC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,MAAM,CAAC,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClD,OAAO,CAAC,GAAG,CAAC,mCAAmC,CAAC,CAAC;YACjD,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE;gBAC3B,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,CAAC,WAAW,GAAG,CAAC,CAAC;gBACxC,OAAO,CAAC,GAAG,CAAC,SAAS,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC;YAC1F,CAAC,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAGD,OAAO,CAAC,GAAG,CAAC,0CAA0C,CAAC,CAAC;IACxD,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC;IAE5B,MAAM,iBAAiB,GAAG;QACxB,QAAQ,EAAE,SAAS;QACnB,SAAS,EAAE,QAAQ;QACnB,OAAO,EAAE,UAAU;QAEnB,IAAI,EAAE,iBAAiB;KACxB,CAAC;IAEF,IAAI,SAAS,IAAI,SAAS,CAAC,UAAU,EAAE,CAAC;QACtC,MAAM,MAAM,GAAG,mDAAuB,CAAC,gBAAgB,CACrD,kBAAkB,EAClB,iBAAiB,EACjB,SAAS,CAAC,UAAU,EACpB,WAAW,CACZ,CAAC;QAEF,OAAO,CAAC,GAAG,CAAC,wBAAwB,CAAC,CAAC;QACtC,OAAO,CAAC,GAAG,CAAC,YAAY,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC;QACxC,OAAO,CAAC,GAAG,CAAC,aAAa,MAAM,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC;QAEjD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE;YAC1B,OAAO,CAAC,GAAG,CAAC,mBAAmB,GAAG,CAAC,QAAQ,EAAE,CAAC,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,kBAAkB,GAAG,CAAC,OAAO,EAAE,CAAC,CAAC;YAC7C,OAAO,CAAC,GAAG,CAAC,cAAc,GAAG,CAAC,GAAG,EAAE,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;IACL,CAAC;IAGD,OAAO,CAAC,GAAG,CAAC,uCAAuC,CAAC,CAAC;IACrD,OAAO,CAAC,GAAG,CAAC,GAAG,CAAE,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC;IAC7B,OAAO,CAAC,GAAG,CAAC,6BAA6B,CAAC,CAAC;IAC3C,OAAO,CAAC,GAAG,CAAC,sDAAsD,CAAC,CAAC;IACpE,OAAO,CAAC,GAAG,CAAC,4CAA4C,CAAC,CAAC;IAC1D,OAAO,CAAC,GAAG,CAAC,4BAA4B,CAAC,CAAC;IAC1C,OAAO,CAAC,GAAG,CAAC,+BAA+B,CAAC,CAAC;IAE7C,OAAO,CAAC,GAAG,CAAC,gCAAgC,CAAC,CAAC;IAC9C,OAAO,CAAC,GAAG,CAAC,sDAAsD,CAAC,CAAC;IACpE,OAAO,CAAC,GAAG,CAAC,uCAAuC,CAAC,CAAC;IACrD,OAAO,CAAC,GAAG,CAAC,uCAAuC,CAAC,CAAC;IACrD,OAAO,CAAC,GAAG,CAAC,iDAAiD,CAAC,CAAC;IAC/D,OAAO,CAAC,GAAG,CAAC,oCAAoC,CAAC,CAAC;IAClD,OAAO,CAAC,GAAG,CAAC,4CAA4C,CAAC,CAAC;IAE1D,OAAO,CAAC,GAAG,CAAC,kEAAkE,CAAC,CAAC;IAEhF,EAAE,CAAC,KAAK,EAAE,CAAC;AACb,CAAC;AAGD,cAAc,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"test-error-handling-validation.d.ts","sourceRoot":"","sources":["../../src/scripts/test-error-handling-validation.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,340 @@
|
|
|
1
|
+
#!/usr/bin/env npx tsx
|
|
2
|
+
"use strict";
|
|
3
|
+
var __importDefault = (this && this.__importDefault) || function (mod) {
|
|
4
|
+
return (mod && mod.__esModule) ? mod : { "default": mod };
|
|
5
|
+
};
|
|
6
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
7
|
+
const workflow_validator_1 = require("../services/workflow-validator");
|
|
8
|
+
const node_repository_1 = require("../database/node-repository");
|
|
9
|
+
const database_adapter_1 = require("../database/database-adapter");
|
|
10
|
+
const enhanced_config_validator_1 = require("../services/enhanced-config-validator");
|
|
11
|
+
const logger_1 = require("../utils/logger");
|
|
12
|
+
const path_1 = __importDefault(require("path"));
|
|
13
|
+
const logger = new logger_1.Logger({ prefix: '[Test Error Handling]' });
|
|
14
|
+
const testWorkflows = [
|
|
15
|
+
{
|
|
16
|
+
name: 'Deprecated continueOnFail',
|
|
17
|
+
workflow: {
|
|
18
|
+
name: 'Test Deprecated Error Handling',
|
|
19
|
+
nodes: [
|
|
20
|
+
{
|
|
21
|
+
id: '1',
|
|
22
|
+
name: 'Webhook',
|
|
23
|
+
type: 'n8n-nodes-base.webhook',
|
|
24
|
+
position: [100, 100],
|
|
25
|
+
parameters: {
|
|
26
|
+
path: 'test-webhook',
|
|
27
|
+
httpMethod: 'POST'
|
|
28
|
+
},
|
|
29
|
+
continueOnFail: true
|
|
30
|
+
},
|
|
31
|
+
{
|
|
32
|
+
id: '2',
|
|
33
|
+
name: 'HTTP Request',
|
|
34
|
+
type: 'n8n-nodes-base.httpRequest',
|
|
35
|
+
position: [300, 100],
|
|
36
|
+
parameters: {
|
|
37
|
+
url: 'https://api.example.com/data',
|
|
38
|
+
method: 'GET'
|
|
39
|
+
},
|
|
40
|
+
continueOnFail: true
|
|
41
|
+
}
|
|
42
|
+
],
|
|
43
|
+
connections: {
|
|
44
|
+
'Webhook': {
|
|
45
|
+
main: [[{ node: 'HTTP Request', type: 'main', index: 0 }]]
|
|
46
|
+
}
|
|
47
|
+
}
|
|
48
|
+
}
|
|
49
|
+
},
|
|
50
|
+
{
|
|
51
|
+
name: 'Modern onError property',
|
|
52
|
+
workflow: {
|
|
53
|
+
name: 'Test Modern Error Handling',
|
|
54
|
+
nodes: [
|
|
55
|
+
{
|
|
56
|
+
id: '1',
|
|
57
|
+
name: 'Webhook',
|
|
58
|
+
type: 'n8n-nodes-base.webhook',
|
|
59
|
+
position: [100, 100],
|
|
60
|
+
parameters: {
|
|
61
|
+
path: 'test-webhook',
|
|
62
|
+
httpMethod: 'POST'
|
|
63
|
+
},
|
|
64
|
+
onError: 'continueRegularOutput',
|
|
65
|
+
alwaysOutputData: true
|
|
66
|
+
},
|
|
67
|
+
{
|
|
68
|
+
id: '2',
|
|
69
|
+
name: 'HTTP Request',
|
|
70
|
+
type: 'n8n-nodes-base.httpRequest',
|
|
71
|
+
position: [300, 100],
|
|
72
|
+
parameters: {
|
|
73
|
+
url: 'https://api.example.com/data',
|
|
74
|
+
method: 'POST',
|
|
75
|
+
sendBody: true,
|
|
76
|
+
contentType: 'json',
|
|
77
|
+
jsonBody: '{"test": "data"}'
|
|
78
|
+
},
|
|
79
|
+
onError: 'continueRegularOutput',
|
|
80
|
+
retryOnFail: true,
|
|
81
|
+
maxTries: 3,
|
|
82
|
+
waitBetweenTries: 2000,
|
|
83
|
+
alwaysOutputData: true
|
|
84
|
+
}
|
|
85
|
+
],
|
|
86
|
+
connections: {
|
|
87
|
+
'Webhook': {
|
|
88
|
+
main: [[{ node: 'HTTP Request', type: 'main', index: 0 }]]
|
|
89
|
+
}
|
|
90
|
+
}
|
|
91
|
+
}
|
|
92
|
+
},
|
|
93
|
+
{
|
|
94
|
+
name: 'Missing error handling',
|
|
95
|
+
workflow: {
|
|
96
|
+
name: 'Test Missing Error Handling',
|
|
97
|
+
nodes: [
|
|
98
|
+
{
|
|
99
|
+
id: '1',
|
|
100
|
+
name: 'Webhook',
|
|
101
|
+
type: 'n8n-nodes-base.webhook',
|
|
102
|
+
position: [100, 100],
|
|
103
|
+
parameters: {
|
|
104
|
+
path: 'test-webhook',
|
|
105
|
+
httpMethod: 'POST'
|
|
106
|
+
}
|
|
107
|
+
},
|
|
108
|
+
{
|
|
109
|
+
id: '2',
|
|
110
|
+
name: 'HTTP Request',
|
|
111
|
+
type: 'n8n-nodes-base.httpRequest',
|
|
112
|
+
position: [300, 100],
|
|
113
|
+
parameters: {
|
|
114
|
+
url: 'https://api.example.com/data',
|
|
115
|
+
method: 'GET'
|
|
116
|
+
}
|
|
117
|
+
},
|
|
118
|
+
{
|
|
119
|
+
id: '3',
|
|
120
|
+
name: 'Postgres',
|
|
121
|
+
type: 'n8n-nodes-base.postgres',
|
|
122
|
+
position: [500, 100],
|
|
123
|
+
parameters: {
|
|
124
|
+
operation: 'insert',
|
|
125
|
+
table: 'users',
|
|
126
|
+
columns: 'name,email'
|
|
127
|
+
}
|
|
128
|
+
}
|
|
129
|
+
],
|
|
130
|
+
connections: {
|
|
131
|
+
'Webhook': {
|
|
132
|
+
main: [[{ node: 'HTTP Request', type: 'main', index: 0 }]]
|
|
133
|
+
},
|
|
134
|
+
'HTTP Request': {
|
|
135
|
+
main: [[{ node: 'Postgres', type: 'main', index: 0 }]]
|
|
136
|
+
}
|
|
137
|
+
}
|
|
138
|
+
}
|
|
139
|
+
},
|
|
140
|
+
{
|
|
141
|
+
name: 'Mixed error handling',
|
|
142
|
+
workflow: {
|
|
143
|
+
name: 'Test Mixed Error Handling',
|
|
144
|
+
nodes: [
|
|
145
|
+
{
|
|
146
|
+
id: '1',
|
|
147
|
+
name: 'Manual Trigger',
|
|
148
|
+
type: 'n8n-nodes-base.manualTrigger',
|
|
149
|
+
position: [100, 100],
|
|
150
|
+
parameters: {}
|
|
151
|
+
},
|
|
152
|
+
{
|
|
153
|
+
id: '2',
|
|
154
|
+
name: 'OpenAI',
|
|
155
|
+
type: 'n8n-nodes-base.openAi',
|
|
156
|
+
position: [300, 100],
|
|
157
|
+
parameters: {
|
|
158
|
+
resource: 'chat',
|
|
159
|
+
operation: 'create',
|
|
160
|
+
model: 'gpt-3.5-turbo',
|
|
161
|
+
messages: {
|
|
162
|
+
values: [
|
|
163
|
+
{
|
|
164
|
+
role: 'user',
|
|
165
|
+
content: 'Hello'
|
|
166
|
+
}
|
|
167
|
+
]
|
|
168
|
+
}
|
|
169
|
+
},
|
|
170
|
+
onError: 'continueRegularOutput',
|
|
171
|
+
retryOnFail: true,
|
|
172
|
+
maxTries: 5,
|
|
173
|
+
waitBetweenTries: 5000
|
|
174
|
+
},
|
|
175
|
+
{
|
|
176
|
+
id: '3',
|
|
177
|
+
name: 'Slack',
|
|
178
|
+
type: 'n8n-nodes-base.slack',
|
|
179
|
+
position: [500, 100],
|
|
180
|
+
parameters: {
|
|
181
|
+
resource: 'message',
|
|
182
|
+
operation: 'post',
|
|
183
|
+
channel: '#general',
|
|
184
|
+
text: 'AI response: {{ $json.message }}'
|
|
185
|
+
},
|
|
186
|
+
continueOnFail: true
|
|
187
|
+
}
|
|
188
|
+
],
|
|
189
|
+
connections: {
|
|
190
|
+
'Manual Trigger': {
|
|
191
|
+
main: [[{ node: 'OpenAI', type: 'main', index: 0 }]]
|
|
192
|
+
},
|
|
193
|
+
'OpenAI': {
|
|
194
|
+
main: [[{ node: 'Slack', type: 'main', index: 0 }]]
|
|
195
|
+
}
|
|
196
|
+
}
|
|
197
|
+
}
|
|
198
|
+
},
|
|
199
|
+
{
|
|
200
|
+
name: 'Error output routing',
|
|
201
|
+
workflow: {
|
|
202
|
+
name: 'Test Error Output Routing',
|
|
203
|
+
nodes: [
|
|
204
|
+
{
|
|
205
|
+
id: '1',
|
|
206
|
+
name: 'Manual Trigger',
|
|
207
|
+
type: 'n8n-nodes-base.manualTrigger',
|
|
208
|
+
position: [100, 200],
|
|
209
|
+
parameters: {}
|
|
210
|
+
},
|
|
211
|
+
{
|
|
212
|
+
id: '2',
|
|
213
|
+
name: 'HTTP Request',
|
|
214
|
+
type: 'n8n-nodes-base.httpRequest',
|
|
215
|
+
position: [300, 200],
|
|
216
|
+
parameters: {
|
|
217
|
+
url: 'https://api.example.com/data',
|
|
218
|
+
method: 'GET'
|
|
219
|
+
},
|
|
220
|
+
onError: 'continueErrorOutput',
|
|
221
|
+
retryOnFail: true,
|
|
222
|
+
maxTries: 2,
|
|
223
|
+
alwaysOutputData: true
|
|
224
|
+
},
|
|
225
|
+
{
|
|
226
|
+
id: '3',
|
|
227
|
+
name: 'Success Handler',
|
|
228
|
+
type: 'n8n-nodes-base.set',
|
|
229
|
+
position: [500, 100],
|
|
230
|
+
parameters: {
|
|
231
|
+
mode: 'manual',
|
|
232
|
+
assignments: {
|
|
233
|
+
assignments: [
|
|
234
|
+
{
|
|
235
|
+
id: '1',
|
|
236
|
+
name: 'status',
|
|
237
|
+
value: 'success',
|
|
238
|
+
type: 'string'
|
|
239
|
+
}
|
|
240
|
+
]
|
|
241
|
+
}
|
|
242
|
+
}
|
|
243
|
+
},
|
|
244
|
+
{
|
|
245
|
+
id: '4',
|
|
246
|
+
name: 'Error Handler',
|
|
247
|
+
type: 'n8n-nodes-base.set',
|
|
248
|
+
position: [500, 300],
|
|
249
|
+
parameters: {
|
|
250
|
+
mode: 'manual',
|
|
251
|
+
assignments: {
|
|
252
|
+
assignments: [
|
|
253
|
+
{
|
|
254
|
+
id: '1',
|
|
255
|
+
name: 'status',
|
|
256
|
+
value: 'error',
|
|
257
|
+
type: 'string'
|
|
258
|
+
}
|
|
259
|
+
]
|
|
260
|
+
}
|
|
261
|
+
}
|
|
262
|
+
}
|
|
263
|
+
],
|
|
264
|
+
connections: {
|
|
265
|
+
'Manual Trigger': {
|
|
266
|
+
main: [[{ node: 'HTTP Request', type: 'main', index: 0 }]]
|
|
267
|
+
},
|
|
268
|
+
'HTTP Request': {
|
|
269
|
+
main: [[{ node: 'Success Handler', type: 'main', index: 0 }]],
|
|
270
|
+
error: [[{ node: 'Error Handler', type: 'main', index: 0 }]]
|
|
271
|
+
}
|
|
272
|
+
}
|
|
273
|
+
}
|
|
274
|
+
}
|
|
275
|
+
];
|
|
276
|
+
async function testErrorHandlingValidation() {
|
|
277
|
+
try {
|
|
278
|
+
const dbPath = path_1.default.join(process.cwd(), 'nodes.db');
|
|
279
|
+
const dbAdapter = await (0, database_adapter_1.createDatabaseAdapter)(dbPath);
|
|
280
|
+
const nodeRepository = new node_repository_1.NodeRepository(dbAdapter);
|
|
281
|
+
const validator = new workflow_validator_1.WorkflowValidator(nodeRepository, enhanced_config_validator_1.EnhancedConfigValidator);
|
|
282
|
+
logger.info('Testing error handling validation...\n');
|
|
283
|
+
for (const test of testWorkflows) {
|
|
284
|
+
logger.info(`\n${'='.repeat(80)}`);
|
|
285
|
+
logger.info(`Test: ${test.name}`);
|
|
286
|
+
logger.info('='.repeat(80));
|
|
287
|
+
const result = await validator.validateWorkflow(test.workflow, {
|
|
288
|
+
validateNodes: true,
|
|
289
|
+
validateConnections: true,
|
|
290
|
+
validateExpressions: false,
|
|
291
|
+
profile: 'strict'
|
|
292
|
+
});
|
|
293
|
+
logger.info(`\nValidation result: ${result.valid ? '✅ VALID' : '❌ INVALID'}`);
|
|
294
|
+
if (result.errors.length > 0) {
|
|
295
|
+
logger.error('\nErrors:');
|
|
296
|
+
result.errors.forEach((error, index) => {
|
|
297
|
+
logger.error(` ${index + 1}. ${error.nodeName ? `[${error.nodeName}] ` : ''}${error.message}`);
|
|
298
|
+
if (error.details) {
|
|
299
|
+
logger.error(` Details: ${JSON.stringify(error.details)}`);
|
|
300
|
+
}
|
|
301
|
+
});
|
|
302
|
+
}
|
|
303
|
+
if (result.warnings.length > 0) {
|
|
304
|
+
logger.warn('\nWarnings:');
|
|
305
|
+
result.warnings.forEach((warning, index) => {
|
|
306
|
+
logger.warn(` ${index + 1}. ${warning.nodeName ? `[${warning.nodeName}] ` : ''}${warning.message}`);
|
|
307
|
+
});
|
|
308
|
+
}
|
|
309
|
+
if (result.suggestions.length > 0) {
|
|
310
|
+
logger.info('\nSuggestions:');
|
|
311
|
+
result.suggestions.forEach((suggestion, index) => {
|
|
312
|
+
logger.info(` ${index + 1}. ${suggestion}`);
|
|
313
|
+
});
|
|
314
|
+
}
|
|
315
|
+
const errorHandlingWarnings = result.warnings.filter((w) => w.message.includes('error handling') ||
|
|
316
|
+
w.message.includes('continueOnFail') ||
|
|
317
|
+
w.message.includes('onError'));
|
|
318
|
+
if (errorHandlingWarnings.length > 0) {
|
|
319
|
+
logger.info('\n📋 Error Handling Specific Issues:');
|
|
320
|
+
errorHandlingWarnings.forEach((warning) => {
|
|
321
|
+
logger.info(` - [${warning.nodeName}] ${warning.message}`);
|
|
322
|
+
});
|
|
323
|
+
}
|
|
324
|
+
logger.info('\nStatistics:');
|
|
325
|
+
logger.info(` Total nodes: ${result.statistics.totalNodes}`);
|
|
326
|
+
logger.info(` Enabled nodes: ${result.statistics.enabledNodes}`);
|
|
327
|
+
logger.info(` Valid connections: ${result.statistics.validConnections}`);
|
|
328
|
+
}
|
|
329
|
+
logger.info('\n✅ Error handling validation tests completed!');
|
|
330
|
+
}
|
|
331
|
+
catch (error) {
|
|
332
|
+
logger.error('Test failed:', error);
|
|
333
|
+
process.exit(1);
|
|
334
|
+
}
|
|
335
|
+
}
|
|
336
|
+
testErrorHandlingValidation().catch(error => {
|
|
337
|
+
logger.error('Fatal error:', error);
|
|
338
|
+
process.exit(1);
|
|
339
|
+
});
|
|
340
|
+
//# sourceMappingURL=test-error-handling-validation.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"test-error-handling-validation.js","sourceRoot":"","sources":["../../src/scripts/test-error-handling-validation.ts"],"names":[],"mappings":";;;;;;AAQA,uEAAmE;AACnE,iEAA6D;AAC7D,mEAAqE;AACrE,qFAAgF;AAChF,4CAAyC;AACzC,gDAAwB;AAExB,MAAM,MAAM,GAAG,IAAI,eAAM,CAAC,EAAE,MAAM,EAAE,uBAAuB,EAAE,CAAC,CAAC;AAG/D,MAAM,aAAa,GAAG;IACpB;QACE,IAAI,EAAE,2BAA2B;QACjC,QAAQ,EAAE;YACR,IAAI,EAAE,gCAAgC;YACtC,KAAK,EAAE;gBACL;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,SAAS;oBACf,IAAI,EAAE,wBAAwB;oBAC9B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,IAAI,EAAE,cAAc;wBACpB,UAAU,EAAE,MAAM;qBACnB;oBACD,cAAc,EAAE,IAAI;iBACrB;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,cAAc;oBACpB,IAAI,EAAE,4BAA4B;oBAClC,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,GAAG,EAAE,8BAA8B;wBACnC,MAAM,EAAE,KAAK;qBACd;oBACD,cAAc,EAAE,IAAI;iBACrB;aACF;YACD,WAAW,EAAE;gBACX,SAAS,EAAE;oBACT,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBAC3D;aACF;SACF;KACF;IACD;QACE,IAAI,EAAE,yBAAyB;QAC/B,QAAQ,EAAE;YACR,IAAI,EAAE,4BAA4B;YAClC,KAAK,EAAE;gBACL;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,SAAS;oBACf,IAAI,EAAE,wBAAwB;oBAC9B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,IAAI,EAAE,cAAc;wBACpB,UAAU,EAAE,MAAM;qBACnB;oBACD,OAAO,EAAE,uBAAuB;oBAChC,gBAAgB,EAAE,IAAI;iBACvB;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,cAAc;oBACpB,IAAI,EAAE,4BAA4B;oBAClC,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,GAAG,EAAE,8BAA8B;wBACnC,MAAM,EAAE,MAAM;wBACd,QAAQ,EAAE,IAAI;wBACd,WAAW,EAAE,MAAM;wBACnB,QAAQ,EAAE,kBAAkB;qBAC7B;oBACD,OAAO,EAAE,uBAAuB;oBAChC,WAAW,EAAE,IAAI;oBACjB,QAAQ,EAAE,CAAC;oBACX,gBAAgB,EAAE,IAAI;oBACtB,gBAAgB,EAAE,IAAI;iBACvB;aACF;YACD,WAAW,EAAE;gBACX,SAAS,EAAE;oBACT,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBAC3D;aACF;SACF;KACF;IACD;QACE,IAAI,EAAE,wBAAwB;QAC9B,QAAQ,EAAE;YACR,IAAI,EAAE,6BAA6B;YACnC,KAAK,EAAE;gBACL;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,SAAS;oBACf,IAAI,EAAE,wBAAwB;oBAC9B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,IAAI,EAAE,cAAc;wBACpB,UAAU,EAAE,MAAM;qBACnB;iBAEF;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,cAAc;oBACpB,IAAI,EAAE,4BAA4B;oBAClC,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,GAAG,EAAE,8BAA8B;wBACnC,MAAM,EAAE,KAAK;qBACd;iBAEF;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,UAAU;oBAChB,IAAI,EAAE,yBAAyB;oBAC/B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,SAAS,EAAE,QAAQ;wBACnB,KAAK,EAAE,OAAO;wBACd,OAAO,EAAE,YAAY;qBACtB;iBAEF;aACF;YACD,WAAW,EAAE;gBACX,SAAS,EAAE;oBACT,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBAC3D;gBACD,cAAc,EAAE;oBACd,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBACvD;aACF;SACF;KACF;IACD;QACE,IAAI,EAAE,sBAAsB;QAC5B,QAAQ,EAAE;YACR,IAAI,EAAE,2BAA2B;YACjC,KAAK,EAAE;gBACL;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,gBAAgB;oBACtB,IAAI,EAAE,8BAA8B;oBACpC,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE,EAAE;iBACf;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,QAAQ;oBACd,IAAI,EAAE,uBAAuB;oBAC7B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,QAAQ,EAAE,MAAM;wBAChB,SAAS,EAAE,QAAQ;wBACnB,KAAK,EAAE,eAAe;wBACtB,QAAQ,EAAE;4BACR,MAAM,EAAE;gCACN;oCACE,IAAI,EAAE,MAAM;oCACZ,OAAO,EAAE,OAAO;iCACjB;6BACF;yBACF;qBACF;oBACD,OAAO,EAAE,uBAAuB;oBAChC,WAAW,EAAE,IAAI;oBACjB,QAAQ,EAAE,CAAC;oBACX,gBAAgB,EAAE,IAAI;iBACvB;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,OAAO;oBACb,IAAI,EAAE,sBAAsB;oBAC5B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,QAAQ,EAAE,SAAS;wBACnB,SAAS,EAAE,MAAM;wBACjB,OAAO,EAAE,UAAU;wBACnB,IAAI,EAAE,kCAAkC;qBACzC;oBACD,cAAc,EAAE,IAAI;iBACrB;aACF;YACD,WAAW,EAAE;gBACX,gBAAgB,EAAE;oBAChB,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBACrD;gBACD,QAAQ,EAAE;oBACR,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBACpD;aACF;SACF;KACF;IACD;QACE,IAAI,EAAE,sBAAsB;QAC5B,QAAQ,EAAE;YACR,IAAI,EAAE,2BAA2B;YACjC,KAAK,EAAE;gBACL;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,gBAAgB;oBACtB,IAAI,EAAE,8BAA8B;oBACpC,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE,EAAE;iBACf;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,cAAc;oBACpB,IAAI,EAAE,4BAA4B;oBAClC,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,GAAG,EAAE,8BAA8B;wBACnC,MAAM,EAAE,KAAK;qBACd;oBACD,OAAO,EAAE,qBAAqB;oBAC9B,WAAW,EAAE,IAAI;oBACjB,QAAQ,EAAE,CAAC;oBACX,gBAAgB,EAAE,IAAI;iBACvB;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,iBAAiB;oBACvB,IAAI,EAAE,oBAAoB;oBAC1B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,IAAI,EAAE,QAAQ;wBACd,WAAW,EAAE;4BACX,WAAW,EAAE;gCACX;oCACE,EAAE,EAAE,GAAG;oCACP,IAAI,EAAE,QAAQ;oCACd,KAAK,EAAE,SAAS;oCAChB,IAAI,EAAE,QAAQ;iCACf;6BACF;yBACF;qBACF;iBACF;gBACD;oBACE,EAAE,EAAE,GAAG;oBACP,IAAI,EAAE,eAAe;oBACrB,IAAI,EAAE,oBAAoB;oBAC1B,QAAQ,EAAE,CAAC,GAAG,EAAE,GAAG,CAAqB;oBACxC,UAAU,EAAE;wBACV,IAAI,EAAE,QAAQ;wBACd,WAAW,EAAE;4BACX,WAAW,EAAE;gCACX;oCACE,EAAE,EAAE,GAAG;oCACP,IAAI,EAAE,QAAQ;oCACd,KAAK,EAAE,OAAO;oCACd,IAAI,EAAE,QAAQ;iCACf;6BACF;yBACF;qBACF;iBACF;aACF;YACD,WAAW,EAAE;gBACX,gBAAgB,EAAE;oBAChB,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBAC3D;gBACD,cAAc,EAAE;oBACd,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,iBAAiB,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;oBAC7D,KAAK,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,eAAe,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;iBAC7D;aACF;SACF;KACF;CACF,CAAC;AAEF,KAAK,UAAU,2BAA2B;IACxC,IAAI,CAAC;QAEH,MAAM,MAAM,GAAG,cAAI,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,UAAU,CAAC,CAAC;QACpD,MAAM,SAAS,GAAG,MAAM,IAAA,wCAAqB,EAAC,MAAM,CAAC,CAAC;QACtD,MAAM,cAAc,GAAG,IAAI,gCAAc,CAAC,SAAS,CAAC,CAAC;QAGrD,MAAM,SAAS,GAAG,IAAI,sCAAiB,CAAC,cAAc,EAAE,mDAAuB,CAAC,CAAC;QAEjF,MAAM,CAAC,IAAI,CAAC,wCAAwC,CAAC,CAAC;QAEtD,KAAK,MAAM,IAAI,IAAI,aAAa,EAAE,CAAC;YACjC,MAAM,CAAC,IAAI,CAAC,KAAK,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;YACnC,MAAM,CAAC,IAAI,CAAC,SAAS,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;YAClC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC;YAG5B,MAAM,MAAM,GAAG,MAAM,SAAS,CAAC,gBAAgB,CAAC,IAAI,CAAC,QAAe,EAAE;gBACpE,aAAa,EAAE,IAAI;gBACnB,mBAAmB,EAAE,IAAI;gBACzB,mBAAmB,EAAE,KAAK;gBAC1B,OAAO,EAAE,QAAQ;aAClB,CAAC,CAAC;YAGH,MAAM,CAAC,IAAI,CAAC,wBAAwB,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC;YAE9E,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBAC7B,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;gBAC1B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,KAAU,EAAE,KAAa,EAAE,EAAE;oBAClD,MAAM,CAAC,KAAK,CAAC,KAAK,KAAK,GAAG,CAAC,KAAK,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC;oBAChG,IAAI,KAAK,CAAC,OAAO,EAAE,CAAC;wBAClB,MAAM,CAAC,KAAK,CAAC,iBAAiB,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;oBACjE,CAAC;gBACH,CAAC,CAAC,CAAC;YACL,CAAC;YAED,IAAI,MAAM,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBAC/B,MAAM,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;gBAC3B,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,OAAY,EAAE,KAAa,EAAE,EAAE;oBACtD,MAAM,CAAC,IAAI,CAAC,KAAK,KAAK,GAAG,CAAC,KAAK,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,OAAO,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,OAAO,CAAC,OAAO,EAAE,CAAC,CAAC;gBACvG,CAAC,CAAC,CAAC;YACL,CAAC;YAED,IAAI,MAAM,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBAClC,MAAM,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;gBAC9B,MAAM,CAAC,WAAW,CAAC,OAAO,CAAC,CAAC,UAAkB,EAAE,KAAa,EAAE,EAAE;oBAC/D,MAAM,CAAC,IAAI,CAAC,KAAK,KAAK,GAAG,CAAC,KAAK,UAAU,EAAE,CAAC,CAAC;gBAC/C,CAAC,CAAC,CAAC;YACL,CAAC;YAGD,MAAM,qBAAqB,GAAG,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAM,EAAE,EAAE,CAC9D,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAC;gBACpC,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAC;gBACpC,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,CAC9B,CAAC;YAEF,IAAI,qBAAqB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBACrC,MAAM,CAAC,IAAI,CAAC,sCAAsC,CAAC,CAAC;gBACpD,qBAAqB,CAAC,OAAO,CAAC,CAAC,OAAY,EAAE,EAAE;oBAC7C,MAAM,CAAC,IAAI,CAAC,QAAQ,OAAO,CAAC,QAAQ,KAAK,OAAO,CAAC,OAAO,EAAE,CAAC,CAAC;gBAC9D,CAAC,CAAC,CAAC;YACL,CAAC;YAED,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YAC7B,MAAM,CAAC,IAAI,CAAC,kBAAkB,MAAM,CAAC,UAAU,CAAC,UAAU,EAAE,CAAC,CAAC;YAC9D,MAAM,CAAC,IAAI,CAAC,oBAAoB,MAAM,CAAC,UAAU,CAAC,YAAY,EAAE,CAAC,CAAC;YAClE,MAAM,CAAC,IAAI,CAAC,wBAAwB,MAAM,CAAC,UAAU,CAAC,gBAAgB,EAAE,CAAC,CAAC;QAC5E,CAAC;QAED,MAAM,CAAC,IAAI,CAAC,gDAAgD,CAAC,CAAC;IAEhE,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,CAAC,KAAK,CAAC,cAAc,EAAE,KAAK,CAAC,CAAC;QACpC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;AACH,CAAC;AAGD,2BAA2B,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE;IAC1C,MAAM,CAAC,KAAK,CAAC,cAAc,EAAE,KAAK,CAAC,CAAC;IACpC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;AAClB,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"test-issue-45-fix.d.ts","sourceRoot":"","sources":["../../src/scripts/test-issue-45-fix.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,136 @@
|
|
|
1
|
+
#!/usr/bin/env node
|
|
2
|
+
"use strict";
|
|
3
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
4
|
+
const dotenv_1 = require("dotenv");
|
|
5
|
+
const logger_1 = require("../utils/logger");
|
|
6
|
+
const n8n_validation_1 = require("../services/n8n-validation");
|
|
7
|
+
(0, dotenv_1.config)();
|
|
8
|
+
function testCleanWorkflowFunctions() {
|
|
9
|
+
logger_1.logger.info('Testing Issue #45 Fix: cleanWorkflowForUpdate should not add default settings\n');
|
|
10
|
+
logger_1.logger.info('=== Test 1: cleanWorkflowForUpdate without settings ===');
|
|
11
|
+
const workflowWithoutSettings = {
|
|
12
|
+
id: 'test-123',
|
|
13
|
+
name: 'Test Workflow',
|
|
14
|
+
nodes: [],
|
|
15
|
+
connections: {},
|
|
16
|
+
active: false,
|
|
17
|
+
createdAt: '2024-01-01T00:00:00.000Z',
|
|
18
|
+
updatedAt: '2024-01-01T00:00:00.000Z',
|
|
19
|
+
versionId: 'version-123'
|
|
20
|
+
};
|
|
21
|
+
const cleanedUpdate = (0, n8n_validation_1.cleanWorkflowForUpdate)(workflowWithoutSettings);
|
|
22
|
+
if ('settings' in cleanedUpdate) {
|
|
23
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForUpdate added settings when it should not have');
|
|
24
|
+
logger_1.logger.error(' Found settings:', JSON.stringify(cleanedUpdate.settings));
|
|
25
|
+
}
|
|
26
|
+
else {
|
|
27
|
+
logger_1.logger.info('✅ PASS: cleanWorkflowForUpdate did not add settings');
|
|
28
|
+
}
|
|
29
|
+
logger_1.logger.info('\n=== Test 2: cleanWorkflowForUpdate with existing settings ===');
|
|
30
|
+
const workflowWithSettings = {
|
|
31
|
+
...workflowWithoutSettings,
|
|
32
|
+
settings: {
|
|
33
|
+
executionOrder: 'v1',
|
|
34
|
+
saveDataErrorExecution: 'none',
|
|
35
|
+
saveDataSuccessExecution: 'none',
|
|
36
|
+
saveManualExecutions: false,
|
|
37
|
+
saveExecutionProgress: false
|
|
38
|
+
}
|
|
39
|
+
};
|
|
40
|
+
const cleanedUpdate2 = (0, n8n_validation_1.cleanWorkflowForUpdate)(workflowWithSettings);
|
|
41
|
+
if ('settings' in cleanedUpdate2) {
|
|
42
|
+
const settingsMatch = JSON.stringify(cleanedUpdate2.settings) === JSON.stringify(workflowWithSettings.settings);
|
|
43
|
+
if (settingsMatch) {
|
|
44
|
+
logger_1.logger.info('✅ PASS: cleanWorkflowForUpdate preserved existing settings without modification');
|
|
45
|
+
}
|
|
46
|
+
else {
|
|
47
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForUpdate modified existing settings');
|
|
48
|
+
logger_1.logger.error(' Original:', JSON.stringify(workflowWithSettings.settings));
|
|
49
|
+
logger_1.logger.error(' Cleaned:', JSON.stringify(cleanedUpdate2.settings));
|
|
50
|
+
}
|
|
51
|
+
}
|
|
52
|
+
else {
|
|
53
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForUpdate removed existing settings');
|
|
54
|
+
}
|
|
55
|
+
logger_1.logger.info('\n=== Test 3: cleanWorkflowForUpdate with partial settings ===');
|
|
56
|
+
const workflowWithPartialSettings = {
|
|
57
|
+
...workflowWithoutSettings,
|
|
58
|
+
settings: {
|
|
59
|
+
executionOrder: 'v1'
|
|
60
|
+
}
|
|
61
|
+
};
|
|
62
|
+
const cleanedUpdate3 = (0, n8n_validation_1.cleanWorkflowForUpdate)(workflowWithPartialSettings);
|
|
63
|
+
if ('settings' in cleanedUpdate3) {
|
|
64
|
+
const settingsKeys = cleanedUpdate3.settings ? Object.keys(cleanedUpdate3.settings) : [];
|
|
65
|
+
const hasOnlyExecutionOrder = settingsKeys.length === 1 &&
|
|
66
|
+
cleanedUpdate3.settings?.executionOrder === 'v1';
|
|
67
|
+
if (hasOnlyExecutionOrder) {
|
|
68
|
+
logger_1.logger.info('✅ PASS: cleanWorkflowForUpdate preserved partial settings without adding defaults');
|
|
69
|
+
}
|
|
70
|
+
else {
|
|
71
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForUpdate added default properties to partial settings');
|
|
72
|
+
logger_1.logger.error(' Original keys:', Object.keys(workflowWithPartialSettings.settings || {}));
|
|
73
|
+
logger_1.logger.error(' Cleaned keys:', settingsKeys);
|
|
74
|
+
}
|
|
75
|
+
}
|
|
76
|
+
else {
|
|
77
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForUpdate removed partial settings');
|
|
78
|
+
}
|
|
79
|
+
logger_1.logger.info('\n=== Test 4: cleanWorkflowForCreate should add default settings ===');
|
|
80
|
+
const newWorkflow = {
|
|
81
|
+
name: 'New Workflow',
|
|
82
|
+
nodes: [],
|
|
83
|
+
connections: {}
|
|
84
|
+
};
|
|
85
|
+
const cleanedCreate = (0, n8n_validation_1.cleanWorkflowForCreate)(newWorkflow);
|
|
86
|
+
if ('settings' in cleanedCreate && cleanedCreate.settings) {
|
|
87
|
+
const hasDefaults = cleanedCreate.settings.executionOrder === 'v1' &&
|
|
88
|
+
cleanedCreate.settings.saveDataErrorExecution === 'all' &&
|
|
89
|
+
cleanedCreate.settings.saveDataSuccessExecution === 'all' &&
|
|
90
|
+
cleanedCreate.settings.saveManualExecutions === true &&
|
|
91
|
+
cleanedCreate.settings.saveExecutionProgress === true;
|
|
92
|
+
if (hasDefaults) {
|
|
93
|
+
logger_1.logger.info('✅ PASS: cleanWorkflowForCreate correctly adds default settings');
|
|
94
|
+
}
|
|
95
|
+
else {
|
|
96
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForCreate added settings but not with correct defaults');
|
|
97
|
+
logger_1.logger.error(' Settings:', JSON.stringify(cleanedCreate.settings));
|
|
98
|
+
}
|
|
99
|
+
}
|
|
100
|
+
else {
|
|
101
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForCreate did not add default settings');
|
|
102
|
+
}
|
|
103
|
+
logger_1.logger.info('\n=== Test 5: cleanWorkflowForUpdate removes read-only fields ===');
|
|
104
|
+
const workflowWithReadOnly = {
|
|
105
|
+
...workflowWithoutSettings,
|
|
106
|
+
staticData: { some: 'data' },
|
|
107
|
+
pinData: { node1: 'data' },
|
|
108
|
+
tags: ['tag1', 'tag2'],
|
|
109
|
+
isArchived: true,
|
|
110
|
+
usedCredentials: ['cred1'],
|
|
111
|
+
sharedWithProjects: ['proj1'],
|
|
112
|
+
triggerCount: 5,
|
|
113
|
+
shared: true,
|
|
114
|
+
active: true
|
|
115
|
+
};
|
|
116
|
+
const cleanedReadOnly = (0, n8n_validation_1.cleanWorkflowForUpdate)(workflowWithReadOnly);
|
|
117
|
+
const removedFields = [
|
|
118
|
+
'id', 'createdAt', 'updatedAt', 'versionId', 'meta',
|
|
119
|
+
'staticData', 'pinData', 'tags', 'isArchived',
|
|
120
|
+
'usedCredentials', 'sharedWithProjects', 'triggerCount',
|
|
121
|
+
'shared', 'active'
|
|
122
|
+
];
|
|
123
|
+
const hasRemovedFields = removedFields.some(field => field in cleanedReadOnly);
|
|
124
|
+
if (!hasRemovedFields) {
|
|
125
|
+
logger_1.logger.info('✅ PASS: cleanWorkflowForUpdate correctly removed all read-only fields');
|
|
126
|
+
}
|
|
127
|
+
else {
|
|
128
|
+
const foundFields = removedFields.filter(field => field in cleanedReadOnly);
|
|
129
|
+
logger_1.logger.error('❌ FAIL: cleanWorkflowForUpdate did not remove these fields:', foundFields);
|
|
130
|
+
}
|
|
131
|
+
logger_1.logger.info('\n=== Test Summary ===');
|
|
132
|
+
logger_1.logger.info('All tests completed. The fix ensures that cleanWorkflowForUpdate only removes fields');
|
|
133
|
+
logger_1.logger.info('without adding default settings, preventing the n8n API validation error.');
|
|
134
|
+
}
|
|
135
|
+
testCleanWorkflowFunctions();
|
|
136
|
+
//# sourceMappingURL=test-issue-45-fix.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"test-issue-45-fix.js","sourceRoot":"","sources":["../../src/scripts/test-issue-45-fix.ts"],"names":[],"mappings":";;;AASA,mCAAgC;AAChC,4CAAyC;AACzC,+DAA4F;AAI5F,IAAA,eAAM,GAAE,CAAC;AAET,SAAS,0BAA0B;IACjC,eAAM,CAAC,IAAI,CAAC,iFAAiF,CAAC,CAAC;IAG/F,eAAM,CAAC,IAAI,CAAC,yDAAyD,CAAC,CAAC;IACvE,MAAM,uBAAuB,GAAa;QACxC,EAAE,EAAE,UAAU;QACd,IAAI,EAAE,eAAe;QACrB,KAAK,EAAE,EAAE;QACT,WAAW,EAAE,EAAE;QACf,MAAM,EAAE,KAAK;QACb,SAAS,EAAE,0BAA0B;QACrC,SAAS,EAAE,0BAA0B;QACrC,SAAS,EAAE,aAAa;KACzB,CAAC;IAEF,MAAM,aAAa,GAAG,IAAA,uCAAsB,EAAC,uBAAuB,CAAC,CAAC;IAEtE,IAAI,UAAU,IAAI,aAAa,EAAE,CAAC;QAChC,eAAM,CAAC,KAAK,CAAC,uEAAuE,CAAC,CAAC;QACtF,eAAM,CAAC,KAAK,CAAC,oBAAoB,EAAE,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC;IAC7E,CAAC;SAAM,CAAC;QACN,eAAM,CAAC,IAAI,CAAC,qDAAqD,CAAC,CAAC;IACrE,CAAC;IAGD,eAAM,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAAC;IAC/E,MAAM,oBAAoB,GAAa;QACrC,GAAG,uBAAuB;QAC1B,QAAQ,EAAE;YACR,cAAc,EAAE,IAAI;YACpB,sBAAsB,EAAE,MAAM;YAC9B,wBAAwB,EAAE,MAAM;YAChC,oBAAoB,EAAE,KAAK;YAC3B,qBAAqB,EAAE,KAAK;SAC7B;KACF,CAAC;IAEF,MAAM,cAAc,GAAG,IAAA,uCAAsB,EAAC,oBAAoB,CAAC,CAAC;IAEpE,IAAI,UAAU,IAAI,cAAc,EAAE,CAAC;QACjC,MAAM,aAAa,GAAG,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,QAAQ,CAAC,KAAK,IAAI,CAAC,SAAS,CAAC,oBAAoB,CAAC,QAAQ,CAAC,CAAC;QAChH,IAAI,aAAa,EAAE,CAAC;YAClB,eAAM,CAAC,IAAI,CAAC,iFAAiF,CAAC,CAAC;QACjG,CAAC;aAAM,CAAC;YACN,eAAM,CAAC,KAAK,CAAC,2DAA2D,CAAC,CAAC;YAC1E,eAAM,CAAC,KAAK,CAAC,cAAc,EAAE,IAAI,CAAC,SAAS,CAAC,oBAAoB,CAAC,QAAQ,CAAC,CAAC,CAAC;YAC5E,eAAM,CAAC,KAAK,CAAC,aAAa,EAAE,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC,CAAC;QACvE,CAAC;IACH,CAAC;SAAM,CAAC;QACN,eAAM,CAAC,KAAK,CAAC,0DAA0D,CAAC,CAAC;IAC3E,CAAC;IAGD,eAAM,CAAC,IAAI,CAAC,gEAAgE,CAAC,CAAC;IAC9E,MAAM,2BAA2B,GAAa;QAC5C,GAAG,uBAAuB;QAC1B,QAAQ,EAAE;YACR,cAAc,EAAE,IAAI;SAErB;KACF,CAAC;IAEF,MAAM,cAAc,GAAG,IAAA,uCAAsB,EAAC,2BAA2B,CAAC,CAAC;IAE3E,IAAI,UAAU,IAAI,cAAc,EAAE,CAAC;QACjC,MAAM,YAAY,GAAG,cAAc,CAAC,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QACzF,MAAM,qBAAqB,GAAG,YAAY,CAAC,MAAM,KAAK,CAAC;YAC1B,cAAc,CAAC,QAAQ,EAAE,cAAc,KAAK,IAAI,CAAC;QAC9E,IAAI,qBAAqB,EAAE,CAAC;YAC1B,eAAM,CAAC,IAAI,CAAC,mFAAmF,CAAC,CAAC;QACnG,CAAC;aAAM,CAAC;YACN,eAAM,CAAC,KAAK,CAAC,6EAA6E,CAAC,CAAC;YAC5F,eAAM,CAAC,KAAK,CAAC,mBAAmB,EAAE,MAAM,CAAC,IAAI,CAAC,2BAA2B,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,CAAC;YAC3F,eAAM,CAAC,KAAK,CAAC,kBAAkB,EAAE,YAAY,CAAC,CAAC;QACjD,CAAC;IACH,CAAC;SAAM,CAAC;QACN,eAAM,CAAC,KAAK,CAAC,yDAAyD,CAAC,CAAC;IAC1E,CAAC;IAGD,eAAM,CAAC,IAAI,CAAC,sEAAsE,CAAC,CAAC;IACpF,MAAM,WAAW,GAAG;QAClB,IAAI,EAAE,cAAc;QACpB,KAAK,EAAE,EAAE;QACT,WAAW,EAAE,EAAE;KAChB,CAAC;IAEF,MAAM,aAAa,GAAG,IAAA,uCAAsB,EAAC,WAAW,CAAC,CAAC;IAE1D,IAAI,UAAU,IAAI,aAAa,IAAI,aAAa,CAAC,QAAQ,EAAE,CAAC;QAC1D,MAAM,WAAW,GACf,aAAa,CAAC,QAAQ,CAAC,cAAc,KAAK,IAAI;YAC9C,aAAa,CAAC,QAAQ,CAAC,sBAAsB,KAAK,KAAK;YACvD,aAAa,CAAC,QAAQ,CAAC,wBAAwB,KAAK,KAAK;YACzD,aAAa,CAAC,QAAQ,CAAC,oBAAoB,KAAK,IAAI;YACpD,aAAa,CAAC,QAAQ,CAAC,qBAAqB,KAAK,IAAI,CAAC;QAExD,IAAI,WAAW,EAAE,CAAC;YAChB,eAAM,CAAC,IAAI,CAAC,gEAAgE,CAAC,CAAC;QAChF,CAAC;aAAM,CAAC;YACN,eAAM,CAAC,KAAK,CAAC,6EAA6E,CAAC,CAAC;YAC5F,eAAM,CAAC,KAAK,CAAC,cAAc,EAAE,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC;QACvE,CAAC;IACH,CAAC;SAAM,CAAC;QACN,eAAM,CAAC,KAAK,CAAC,6DAA6D,CAAC,CAAC;IAC9E,CAAC;IAGD,eAAM,CAAC,IAAI,CAAC,mEAAmE,CAAC,CAAC;IACjF,MAAM,oBAAoB,GAAQ;QAChC,GAAG,uBAAuB;QAC1B,UAAU,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE;QAC5B,OAAO,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE;QAC1B,IAAI,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC;QACtB,UAAU,EAAE,IAAI;QAChB,eAAe,EAAE,CAAC,OAAO,CAAC;QAC1B,kBAAkB,EAAE,CAAC,OAAO,CAAC;QAC7B,YAAY,EAAE,CAAC;QACf,MAAM,EAAE,IAAI;QACZ,MAAM,EAAE,IAAI;KACb,CAAC;IAEF,MAAM,eAAe,GAAG,IAAA,uCAAsB,EAAC,oBAAoB,CAAC,CAAC;IAErE,MAAM,aAAa,GAAG;QACpB,IAAI,EAAE,WAAW,EAAE,WAAW,EAAE,WAAW,EAAE,MAAM;QACnD,YAAY,EAAE,SAAS,EAAE,MAAM,EAAE,YAAY;QAC7C,iBAAiB,EAAE,oBAAoB,EAAE,cAAc;QACvD,QAAQ,EAAE,QAAQ;KACnB,CAAC;IAEF,MAAM,gBAAgB,GAAG,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,KAAK,IAAI,eAAe,CAAC,CAAC;IAE/E,IAAI,CAAC,gBAAgB,EAAE,CAAC;QACtB,eAAM,CAAC,IAAI,CAAC,uEAAuE,CAAC,CAAC;IACvF,CAAC;SAAM,CAAC;QACN,MAAM,WAAW,GAAG,aAAa,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,KAAK,IAAI,eAAe,CAAC,CAAC;QAC5E,eAAM,CAAC,KAAK,CAAC,6DAA6D,EAAE,WAAW,CAAC,CAAC;IAC3F,CAAC;IAED,eAAM,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;IACtC,eAAM,CAAC,IAAI,CAAC,sFAAsF,CAAC,CAAC;IACpG,eAAM,CAAC,IAAI,CAAC,2EAA2E,CAAC,CAAC;AAC3F,CAAC;AAGD,0BAA0B,EAAE,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"test-issue-74.d.ts","sourceRoot":"","sources":["../../src/scripts/test-issue-74.ts"],"names":[],"mappings":""}
|