mirror of
https://github.com/czlonkowski/n8n-mcp.git
synced 2026-01-30 06:22:04 +00:00
v2.28.7
869 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
Romuald Członkowski
|
cdaa29e7a2 |
fix: memory leak in session removal - close MCP server properly (#471) (#472)
- Add close() method to N8NDocumentationMCPServer that: - Calls server.close() (MCP SDK cleanup) - Clears internal cache - Nullifies service references to help GC - Update removeSession() to call server.close() before releasing references Root cause: removeSession() deleted server from map but didn't call cleanup Evidence: Production server memory grew ~1GB in 43 minutes (10% to 35%) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Romuald Członkowski <romualdczlonkowski@MacBook-Pro-Romuald.local> Co-authored-by: Claude <noreply@anthropic.com>v2.28.7 |
||
|
Romuald Członkowski
|
60479e0eb4 |
test: update tests for v2.28.5 behavior changes (v2.28.6) (#470)
- Update n8n-version tests: 'v' prefix now supported in version strings
- Update n8n-validation tests: empty settings now return minimal defaults
{ executionOrder: 'v1' } instead of {} to avoid API rejection (Issue #431)
Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Romuald Członkowski <romualdczlonkowski@MacBook-Pro-Romuald.local>
Co-authored-by: Claude <noreply@anthropic.com>
v2.28.6
|
||
|
Romuald Członkowski
|
90407f845d |
fix: version-aware settings filtering for n8n API compatibility (#467)
Fixes #464, #465, #466 - Add n8n version detection via /rest/settings endpoint - Filter workflow settings based on n8n version support: - All versions: 7 core properties - n8n 1.37.0+: adds executionOrder - n8n 1.119.0+: adds callerPolicy, callerIds, timeSavedPerExecution, availableInMCP - Add read-only field filtering (activeVersionId, activeVersion) - Version cache with 5-minute TTL for server upgrades - Race condition protection with promise-based locking - Type validation for version string responses Thanks to @thesved for this contribution! Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|
Romuald Członkowski
|
b5759c4146 |
chore: bump version to 2.28.5 and add changelog entry
Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|
Romuald Członkowski
|
1524fd5a08 |
fix: address code review issues for PR #467
- Add 5-minute TTL to version cache (prevents stale data after upgrades) - Add type validation for version string from settings response - Support 'v' prefix in version strings (e.g., v1.2.3) - Fix race condition in getVersion() using promise-based locking - Fix empty settings regression (Issue #431) - use minimal defaults 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|
thesved
|
5057481e70 | chore: add pre-built dist folder for npx usage | ||
|
thesved
|
a70d96a373 |
Add version-aware settings filtering for n8n API compatibility
This commit adds automatic detection of n8n server version and filters workflow settings properties based on what the target version supports. Changes: - Add n8n-version.ts service for version detection via /rest/settings - Fix nested response structure handling (data.data.versionCli) - Add N8nVersionInfo and N8nSettingsData types - Update N8nApiClient with getVersion() and version-aware filtering - Clean workflow settings before API update based on detected version Version compatibility: - n8n < 1.37.0: 7 core properties only - n8n 1.37.0+: adds executionOrder - n8n 1.119.0+: adds callerPolicy, callerIds, timeSavedPerExecution, availableInMCP Fixes #466 |
||
|
thesved
|
934124fa7b |
Fix: n8n_update_partial_workflow fails with additional properties error (#466)
The n8n Public API uses strict JSON schema validation (additionalProperties: false).
The cleanWorkflowForUpdate() function was sending properties that the API rejects.
Changes:
- Filter out activeVersionId and activeVersion (read-only fields)
- Update settings whitelist to include all 12 properties from n8n OpenAPI spec
- Simplify settings handling (empty {} is now accepted by n8n API)
Source of truth for allowed properties:
- https://github.com/n8n-io/n8n/blob/master/packages/cli/src/public-api/v1/handlers/workflows/spec/schemas/workflow.yml
- https://github.com/n8n-io/n8n/blob/master/packages/cli/src/public-api/v1/handlers/workflows/spec/schemas/workflowSettings.yml
Fixes #466
Conceived by Romuald Członkowski - www.aiadvisors.pl/en
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
|
||
|
Romuald Członkowski
|
c2dc7c9c51 |
feat: configurable MAX_SESSIONS via N8N_MCP_MAX_SESSIONS env var (v2.28.4) (#469)
* fix: n8n_test_workflow webhookId resolution and form handling (v2.28.2) ## Bug Fixes - **webhookId Resolution**: Fixed trigger handlers using `node.id` instead of `node.webhookId` for building webhook URLs. This caused 404 errors when nodes had custom IDs. - **Chat Trigger URL**: Fixed chat triggers using wrong URL pattern. Now correctly uses `/webhook/<webhookId>/chat` endpoint. - **Form Content-Type**: Fixed form triggers failing with "Expected multipart/form-data" error by switching to proper multipart encoding. ## Enhancements - **Form Field Types**: Added support for all n8n form field types (text, textarea, email, number, password, date, dropdown, checkbox, file, hidden) - **Checkbox Arrays**: Automatically converts arrays to `field[]` format - **Helpful Warnings**: Reports missing required fields with names and labels - **Error Hints**: Provides complete field structure on failure Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: address code review issues for form trigger improvements - Add form-data as direct dependency (was only in devDependencies) - Add TypeScript interfaces (FormFieldValue, FormFieldOption) replacing any types - Add FORM_FIELD_TYPES constants for type-safe switch statements - Add isValidBase64() validation for file uploads with size limits - Add MAX_FILE_SIZE_BYTES (10MB) constant with validation - Update form-handler.test.ts for FormData instead of JSON - Update trigger-detector.test.ts for chat URL /chat suffix Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * feat: configurable MAX_SESSIONS via N8N_MCP_MAX_SESSIONS env var (v2.28.3) (#468) Make MAX_SESSIONS limit configurable for multi-tenant SaaS deployments. - Add N8N_MCP_MAX_SESSIONS environment variable (default: 100) - Include safety floor with Math.max(1, ...) to prevent invalid configs - Update documentation in LIBRARY_USAGE.md, SESSION_PERSISTENCE.md - Update CLAUDE.md and CHANGELOG.md Fixes #468 Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: add QEMU setup for multi-arch Docker builds Add docker/setup-qemu-action@v3 before Buildx setup to enable proper QEMU emulation for linux/arm64 builds on GitHub Actions. Fixes CI Docker build failure with "execve: No such file or directory" 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Romuald Członkowski <romualdczlonkowski@MacBook-Pro-Romuald.local>v2.28.4 |
||
|
Romuald Członkowski
|
527e9874ab |
chore: update n8n to 1.122.4 and remove ludwig's templates (v2.28.3) (#463)
- Updated n8n from 1.121.2 to 1.122.4 - Updated n8n-core from 1.120.1 to 1.121.1 - Updated n8n-workflow from 1.118.1 to 1.119.1 - Updated @n8n/n8n-nodes-langchain from 1.120.1 to 1.121.1 - Rebuilt node database with 544 nodes (438 from n8n-nodes-base, 106 from @n8n/n8n-nodes-langchain) - Removed 7 templates from creator "ludwig" at author's request (IDs: 2795, 2816, 2825, 2850, 2869, 2939, 3847) - Updated README badge with new n8n version - Updated CHANGELOG with dependency changes Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <noreply@anthropic.com>v2.28.3 |
||
|
Romuald Członkowski
|
ef9b6f6341 | fix: n8n_test_workflow webhookId resolution and form handling (v2.28.2) (#462) v2.28.2 | ||
|
Romuald Członkowski
|
3188d209b7 |
fix: AI connection type propagation and get_node improvements (v2.28.1) (#461)
* fix: AI connection type propagation and get_node improvements (v2.28.1) Bug fixes: - Issue #458: addConnection now preserves AI connection types (ai_tool, ai_memory, ai_languageModel) instead of defaulting to 'main' - Fixed false positive "AI Agent has no tools connected" validation warning Enhancements: - Added expectedFormat field to resourceLocator properties in get_node output - Added versionNotice field to make typeVersion more prominent Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * test: add missing test coverage for PR #461 improvements - Added test for AI Agent validation positive case (tools properly connected) - Added 3 tests for expectedFormat on resourceLocator properties Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.28.1 |
||
|
Romuald Członkowski
|
33690c5650 |
feat: rename n8n_trigger_webhook_workflow to n8n_test_workflow with multi-trigger support (#460)
* feat: rename n8n_trigger_webhook_workflow to n8n_test_workflow with multi-trigger support - Rename tool from n8n_trigger_webhook_workflow to n8n_test_workflow - Add support for webhook, form, and chat triggers (auto-detection) - Implement modular trigger system with registry pattern - Add trigger detector for automatic trigger type inference - Remove execute trigger type (n8n public API limitation) - Add comprehensive tests for trigger detection and handlers The tool now auto-detects trigger type from workflow structure and supports all externally-triggerable workflows via n8n's public API. Note: Direct workflow execution (Schedule/Manual triggers) requires n8n's instance-level MCP access, not available via REST API. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: add SSRF protection to webhook handler and update tests - Add SSRF URL validation to webhook-handler.ts (critical security fix) Aligns with existing SSRF protection in form-handler.ts and chat-handler.ts - Update parameter-validation.test.ts to use new n8n_test_workflow tool name Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * feat: n8n_test_workflow unified trigger tool (v2.28.0) Added new `n8n_test_workflow` tool replacing `n8n_trigger_webhook_workflow`: Features: - Auto-detects trigger type (webhook/form/chat) from workflow - Supports multiple trigger types with type-specific parameters - SSRF protection for all trigger handlers - Extensible handler architecture with registry pattern Changes: - Fixed Zod schema to remove invalid 'execute' trigger type - Updated README.md tool documentation - Added CHANGELOG entry for v2.28.0 - Bumped version to 2.28.0 Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * test: add comprehensive unit tests for trigger handlers Added 87 unit tests across 4 test files to improve code coverage: - base-handler.test.ts (19 tests) - 100% coverage - webhook-handler.test.ts (22 tests) - 100% coverage - chat-handler.test.ts (23 tests) - 100% coverage - form-handler.test.ts (23 tests) - 100% coverage Tests cover: - Input validation and parameter handling - SSRF protection integration - HTTP method handling and URL building - Error response formatting - Execution paths for all trigger types Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.28.0 |
||
|
Romuald Członkowski
|
ddf9556759 |
feat: n8n_deploy_template deploy-first with auto-fix (v2.27.2) (#457)
* feat: n8n_deploy_template deploy-first with auto-fix (v2.27.2) Improved template deployment to deploy first, then automatically fix common issues. This dramatically improves deployment success rates for templates with expression format issues. Key Changes: - Deploy-first behavior: templates deployed before validation - Auto-fix runs automatically after deployment (configurable via `autoFix`) - Returns `fixesApplied` array showing all corrections made - Fixed expression validator "nested expressions" false positive - Fixed Zod schema missing `typeversion-upgrade` and `version-migration` fix types Testing: 87% deployment success rate across 15 diverse templates 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: address code review findings for deploy template Code review fixes: - CRITICAL: Update test schema to use `autoFix` instead of old `validate` parameter - WARNING: Add `AppliedFix` and `AutofixResultData` interfaces for type safety - WARNING: Add `autoFixStatus` field to response (success/failed/skipped) - WARNING: Report auto-fix failure in response message Changes: - tests/unit/mcp/handlers-deploy-template.test.ts: Fixed schema and test cases - src/mcp/handlers-n8n-manager.ts: Added type definitions, autoFixStatus tracking - src/mcp/tool-docs/workflow_management/n8n-deploy-template.ts: Updated docs 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.27.2 |
||
|
Romuald Członkowski
|
7d9b456887 |
fix: pin MCP SDK version in Docker build files (v2.27.1) (#456)
* fix: pin MCP SDK version in Docker build files (#454) The Docker image 2.27.0 was missing the Zod fix from #450 because: - package.runtime.json had @modelcontextprotocol/sdk@^1.13.2 - Dockerfile builder had @modelcontextprotocol/sdk@^1.12.1 Both now use the pinned version 1.20.1 (no caret) to match package.json. Also pinned zod@3.24.1 in Dockerfile for consistency. Fixes #454 Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * chore: bump version to 2.27.1 and update CHANGELOG - Version bump from 2.27.0 to 2.27.1 - Added CHANGELOG entry for #454 fix (Docker SDK version) - Added missing CHANGELOG entry for 2.27.0 (n8n_deploy_template) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.27.1 |
||
|
devangkantharia
|
2f5a857142 |
Added Antigravity Setup Instructions (#452)
* Add Antigravity setup documentation Document the setup process for Antigravity with n8n MCP server, including preconditions, installation steps, configuration, and best practices. * Add Antigravity integration guide to README Added a new section for Antigravity integration. |
||
|
Romuald Członkowski
|
e7dd04b471 |
feat: add n8n_deploy_template tool for one-click template deployment (v2.27.0) (#453)
* feat: add n8n_deploy_template tool for one-click template deployment (v2.27.0) Add new MCP tool that deploys n8n.io workflow templates directly to user's n8n instance in a single operation. Features: - Fetch template from local database - Extract and report required credentials - Optionally strip credentials (default: true) - Optionally auto-upgrade node typeVersions (default: true) - Optionally validate before deployment (default: true) - Return workflow ID, URL, and setup guidance Parameters: - templateId (required): Template ID from n8n.io - name (optional): Custom workflow name - autoUpgradeVersions (default: true) - validate (default: true) - stripCredentials (default: true) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: address code review findings for n8n_deploy_template - Fix health check tool count (12 → 13) to include new tool - Add templateId validation (must be positive integer) - Use deep copy of workflow to prevent template mutation - Expand unit tests with negative/zero/decimal validation cases Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * docs: update README with n8n_deploy_template tool - Update management tools count from 12 to 13 - Add n8n_deploy_template to the tools list Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: prevent workflow validator from mutating node types The validator was incorrectly mutating node types from full form (n8n-nodes-base.*) to short form (nodes-base.*) during validation. This caused deployed workflows to show "?" icons in n8n UI because the API requires full form node types. Also updated SplitInBatches detection to check both node type forms since workflows may contain either format. Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * test: update tool counts in handlers-n8n-manager test Update expected managementTools count from 12 to 13 and totalAvailable from 19 to 20 to account for the new n8n_deploy_template tool. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: pin MCP SDK and Zod versions to prevent Zod v4 resolution Fixes #440, #444, #446, #447, #450 Root cause: package.json declared `"@modelcontextprotocol/sdk": "^1.20.1"` which allowed npm to resolve to SDK 1.23.0. That version accepts `"zod": "^3.25 || ^4.0"`, causing npm to deduplicate to Zod v4. SDK 1.23.0's `isZ4Schema()` function crashes when called with undefined, which happens for plain JSON Schema objects. Changes: - Pin SDK to exact version 1.20.1 (removes ^ prefix) - Pin Zod to exact version 3.24.1 (removes ^ prefix) - Add CI workflow to verify fresh installs get compatible versions The new CI workflow: - Packs and installs package fresh (without lockfile) - Verifies SDK is exactly 1.20.1 - Verifies Zod is NOT v4 (blocks 4.x.x) - Runs weekly to catch upstream dependency changes Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * chore: improve dependency-check workflow based on code review - Add workflow_dispatch for manual triggering/debugging - Add explicit "not found" handling for version detection failures - Use regex pattern for Zod v4 check to catch pre-release versions - Add Zod error pattern detection in functionality test - Capture stderr output for comprehensive error checking Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.27.0 |
||
|
Romuald Członkowski
|
c7e7bda505 |
fix: remove historical migration info from tools documentation (v2.26.5) (#448)
- Remove "Replaces X, Y, Z..." sentences from full.description in: - get_node, validate_node, search_templates, n8n_executions, n8n_get_workflow - Remove version/issue references from n8n_update_partial_workflow - Clean up consolidation comments in index.ts - Documentation now starts directly with functional content - Estimated token savings: ~128 tokens per full documentation request Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <noreply@anthropic.com>v2.26.5 |
||
|
Romuald Członkowski
|
bac4936c6d |
fix: add n8n 1.121 availableInMCP and callerPolicy settings support (v2.26.4) (#445)
* fix: add n8n 1.121 availableInMCP and callerPolicy settings support (v2.26.4) n8n 1.121 introduced a new workflow setting `availableInMCP` (boolean) that controls whether a workflow is "Available in MCP". The sanitization whitelist was missing this field, causing it to be silently stripped during workflow updates. Changes: - Added `availableInMCP` to Zod schema in workflowSettingsSchema - Added `availableInMCP` and `callerPolicy` to safeSettingsProperties whitelist - Both settings are now preserved during workflow updates - Settings can be toggled via updateSettings operation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - www.aiadvisors.pl/en * test: update tests for callerPolicy and availableInMCP whitelist changes Updated 5 tests in n8n-validation.test.ts that expected callerPolicy to be filtered out. Since callerPolicy and availableInMCP are now whitelisted (n8n 1.121+), the tests now verify these settings are preserved during workflow updates. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.26.4 |
||
|
Romuald Członkowski
|
25784142fe | fix: address tools documentation gaps and outdated references (v2.26.3) (#443) v2.26.3 | ||
|
Romuald Członkowski
|
f770043d3d |
Revise quick start section in README.md
Removed quick start instructions and example JSON configuration for n8n-MCP. |
||
|
Romuald Członkowski
|
1be06c217f |
fix: synchronize tool documentation with v2.26.0 tool consolidation (v2.26.2) (#442)
* fix: synchronize tool documentation with v2.26.0 tool consolidation (v2.26.2) - Delete 23 obsolete documentation files for removed tools - Create consolidated documentation for get_node, validate_node, n8n_executions - Update search_templates with all searchModes - Update n8n_get_workflow with all modes - Fix stale relatedTools references - Update tools-documentation.ts overview to reflect 19 consolidated tools Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: address code review - fix remaining stale tool references - Fix relatedTools in system/tools-documentation.ts (get_node_for_task → search_templates) - Fix relatedTools in validation/validate-workflow.ts (remove references to removed tools) - Fix relatedTools in n8n-autofix-workflow.ts (remove references to removed tools) - Update tools-n8n-friendly.ts with consolidated tools (validate_node, get_node, search_templates) Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: address final code review - fix remaining stale references - Fix ai-agents-guide.ts: get_node_essentials → get_node, remove list_ai_tools - Fix get-template.ts: list_node_templates → search_templates, remove get_templates_for_task - Fix n8n-list-workflows.ts: n8n_get_workflow_minimal → n8n_get_workflow, n8n_list_executions → n8n_executions - Fix n8n-trigger-webhook-workflow.ts: n8n_get_execution/n8n_list_executions → n8n_executions - Fix n8n-delete-workflow.ts: n8n_get_workflow_minimal → n8n_get_workflow, n8n_delete_execution → n8n_executions - Fix CHANGELOG date typo: 2025-01-25 → 2025-11-25 Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * test: adjust comprehensive docs threshold after tool consolidation Reduce expected character count from 5000 to 4000 in tool-invocation.test.ts to account for reduced documentation after v2.26.0 tool consolidation (31→19 tools, actual output is ~4645 chars). Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.26.2 |
||
|
Romuald Członkowski
|
c974947c84 |
chore: update n8n to 1.121.2 (#441)
* chore: update n8n to 1.121.2 and bump version to 2.26.1 - Updated n8n from 1.120.3 to 1.121.2 - Updated n8n-core from 1.119.2 to 1.120.1 - Updated n8n-workflow from 1.117.0 to 1.118.1 - Updated @n8n/n8n-nodes-langchain from 1.119.1 to 1.120.1 - Rebuilt node database with 545 nodes (439 from n8n-nodes-base, 106 from @n8n/n8n-nodes-langchain) - Updated README badge with new n8n version - Updated CHANGELOG with dependency changes Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * chore: expand template database to 2,768 templates - Added 170 new workflow templates from n8n.io - Sanitized 27 templates containing API tokens - Updated CHANGELOG with template expansion info Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.26.1 |
||
|
Romuald Członkowski
|
ff69e4ccca |
feat: Tool Consolidation - Reduce MCP Tools by 38% (v2.26.0) (#439)
* feat: Remove 9 low-value tools and consolidate n8n_health_check (v2.25.0) Telemetry-driven tool cleanup to improve API clarity: **Removed Tools (9):** - list_nodes - Use search_nodes instead - list_ai_tools - Use search_nodes with isAITool filter - list_tasks - Low usage (0.02%) - get_database_statistics - Use n8n_health_check - list_templates - Use search_templates or get_templates_for_task - get_node_as_tool_info - Documented in get_node - validate_workflow_connections - Use validate_workflow - validate_workflow_expressions - Use validate_workflow - n8n_list_available_tools - Use n8n_health_check - n8n_diagnostic - Merged into n8n_health_check **Consolidated Tool:** - n8n_health_check now supports mode='diagnostic' for detailed troubleshooting **Tool Count:** - Before: 38 tools - After: 31 tools (18% reduction) Concieved by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: cleanup stale references and update tests after tool removal - Remove handleListAvailableTools dead code from handlers-n8n-manager.ts - Update error messages to reference n8n_health_check(mode="diagnostic") instead of n8n_diagnostic - Update tool counts in diagnostic messages (14 doc tools, 31 total) - Fix error-handling.test.ts to use valid tools (search_nodes, tools_documentation) - Remove obsolete list-tools.test.ts integration tests - Remove unused ListToolsResponse type from response-types.ts - Update tools.ts QUICK REFERENCE to remove list_nodes references - Update tools-documentation.ts to remove references to removed tools - Update tool-docs files to remove stale relatedTools references - Fix tools.test.ts to not test removed tools (list_nodes, list_ai_tools, etc.) - Fix parameter-validation.test.ts to not test removed tools - Update handlers-n8n-manager.test.ts error message expectations All 399 MCP unit tests now pass. Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: update integration tests to use valid tools after v2.25.0 removal Replaced all references to removed tools in integration tests: - list_nodes -> search_nodes - get_database_statistics -> tools_documentation - list_ai_tools -> search_nodes/tools_documentation - list_tasks -> tools_documentation - get_node_as_tool_info -> removed test section Updated test files: - tests/integration/mcp-protocol/basic-connection.test.ts - tests/integration/mcp-protocol/performance.test.ts - tests/integration/mcp-protocol/session-management.test.ts - tests/integration/mcp-protocol/test-helpers.ts - tests/integration/mcp-protocol/tool-invocation.test.ts - tests/integration/telemetry/mcp-telemetry.test.ts - tests/unit/mcp/disabled-tools.test.ts - tests/unit/mcp/tools-documentation.test.ts Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * feat: Tool consolidation v2.26.0 - reduce tools by 38% (31 → 19) Major consolidation of MCP tools using mode-based parameters for better AI agent ergonomics: Node Tools: - get_node_documentation → get_node with mode='documentation' - search_node_properties → get_node with mode='search_properties' - get_property_dependencies → removed Validation Tools: - validate_node_operation + validate_node_minimal → validate_node with mode param Template Tools: - list_node_templates → search_templates with searchMode='nodes' - search_templates_by_metadata → search_templates with searchMode='metadata' - get_templates_for_task → search_templates with searchMode='task' Workflow Getters: - n8n_get_workflow_details/structure/minimal → n8n_get_workflow with mode param Execution Tools: - n8n_list/get/delete_execution → n8n_executions with action param Test updates for all consolidated tools. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * docs: comprehensive README update for v2.26.0 tool consolidation - Quick Start: Added hosted service (dashboard.n8n-mcp.com) as primary option - Self-hosting: Renamed options to A (npx), B (Docker), C (Local), D (Railway) - Removed: "Memory Leak Fix (v2.20.2)" section (outdated) - Removed: "Known Issues" section (outdated container management) - Claude Project Setup: Updated all tool references to v2.26.0 consolidated tools - validate_node({mode: 'minimal'|'full'}) instead of separate tools - search_templates({searchMode: ...}) unified template search - get_node({mode: 'docs'|'search_properties'}) for documentation - n8n_executions({action: ...}) unified execution management - Available MCP Tools: Updated to show 19 consolidated tools (7 core + 12 mgmt) - Recent Updates: Simplified to just link to CHANGELOG.md 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: update tool count from 31 to 19 in diagnostic message 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix(tests): update tool count expectations for v2.26.0 Update handlers-n8n-manager.test.ts to expect new consolidated tool counts (7/12/19) after v2.26.0 tool consolidation. Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.26.0 |
||
|
czlonkowski
|
9ee4b9492f | Merge branch 'feature/session-persistence-api' | ||
|
czlonkowski
|
4df9558b3e |
docs: add comprehensive session persistence production guide
Created detailed production documentation for the session persistence API
covering implementation, security, best practices, and troubleshooting.
Documentation includes:
- Architecture overview and session state components
- Complete API reference with examples
- Security considerations (encryption, key management)
- Implementation examples (Express, Kubernetes, Docker Compose)
- Best practices (timeouts, monitoring, graceful shutdown)
- Performance considerations and limits
- Comprehensive troubleshooting guide
- Version compatibility matrix
Target audience: Production engineers deploying n8n-mcp in multi-tenant
environments with zero-downtime requirements.
Related: Session persistence API fixes in commit
|
||
|
Romuald Członkowski
|
05424f66af |
feat: Session Persistence API for Zero-Downtime Deployments (v2.24.1) (#438)
* feat: Add session persistence API for zero-downtime deployments (v2.24.1) Implements export/restore functionality for MCP sessions to support container restarts without losing user sessions. This enables zero-downtime deployments for multi-tenant platforms and Kubernetes/Docker environments. New Features: - exportSessionState() - Export active sessions to JSON - restoreSessionState() - Restore sessions from exported data - SessionState type - Serializable session structure - Comprehensive test suite (22 tests, 100% passing) Implementation Details: - Only exports sessions with valid n8nApiUrl and n8nApiKey - Automatically filters expired sessions (respects sessionTimeout) - Validates context structure using existing validation - Handles null/invalid sessions gracefully with warnings - Enforces MAX_SESSIONS limit during restore (100 sessions) - Dormant sessions recreate transport/server on first request Files Modified: - src/http-server-single-session.ts: Core export/restore logic - src/mcp-engine.ts: Public API wrapper methods - src/types/session-state.ts: Type definitions - tests/: Comprehensive unit tests Security Note: Session data contains plaintext n8n API keys. Downstream applications MUST encrypt session data before persisting to disk. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * feat: implement 7 critical session persistence API fixes for production readiness This commit implements all 7 critical fixes identified in the code review to make the session persistence API production-ready for zero-downtime container deployments in multi-tenant environments. Fixes implemented: 1. Made instanceId optional in SessionState interface 2. Removed redundant validation, properly using validateInstanceContext() 3. Fixed race condition in MAX_SESSIONS check using real-time count 4. Added comprehensive security logging with logSecurityEvent() helper 5. Added duplicate session ID detection during export with Set tracking 6. Added date parsing validation with isNaN checks for Invalid Date objects 7. Restructured null checks for proper TypeScript type narrowing Changes: - src/types/session-state.ts: Made instanceId optional - src/http-server-single-session.ts: Implemented all validation and security fixes - tests/unit/http-server/session-persistence.test.ts: Fixed MAX_SESSIONS test All 13 session persistence unit tests passing. All 9 MCP engine session persistence tests passing. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.24.1 |
||
|
czlonkowski
|
5d2c5df53e |
feat: implement 7 critical session persistence API fixes for production readiness
This commit implements all 7 critical fixes identified in the code review to make the session persistence API production-ready for zero-downtime container deployments in multi-tenant environments. Fixes implemented: 1. Made instanceId optional in SessionState interface 2. Removed redundant validation, properly using validateInstanceContext() 3. Fixed race condition in MAX_SESSIONS check using real-time count 4. Added comprehensive security logging with logSecurityEvent() helper 5. Added duplicate session ID detection during export with Set tracking 6. Added date parsing validation with isNaN checks for Invalid Date objects 7. Restructured null checks for proper TypeScript type narrowing Changes: - src/types/session-state.ts: Made instanceId optional - src/http-server-single-session.ts: Implemented all validation and security fixes - tests/unit/http-server/session-persistence.test.ts: Fixed MAX_SESSIONS test All 13 session persistence unit tests passing. All 9 MCP engine session persistence tests passing. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|
czlonkowski
|
f5cf1e2934 |
feat: Add session persistence API for zero-downtime deployments (v2.24.1)
Implements export/restore functionality for MCP sessions to support container restarts without losing user sessions. This enables zero-downtime deployments for multi-tenant platforms and Kubernetes/Docker environments. New Features: - exportSessionState() - Export active sessions to JSON - restoreSessionState() - Restore sessions from exported data - SessionState type - Serializable session structure - Comprehensive test suite (22 tests, 100% passing) Implementation Details: - Only exports sessions with valid n8nApiUrl and n8nApiKey - Automatically filters expired sessions (respects sessionTimeout) - Validates context structure using existing validation - Handles null/invalid sessions gracefully with warnings - Enforces MAX_SESSIONS limit during restore (100 sessions) - Dormant sessions recreate transport/server on first request Files Modified: - src/http-server-single-session.ts: Core export/restore logic - src/mcp-engine.ts: Public API wrapper methods - src/types/session-state.ts: Type definitions - tests/: Comprehensive unit tests Security Note: Session data contains plaintext n8n API keys. Downstream applications MUST encrypt session data before persisting to disk. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en |
||
|
Romuald Członkowski
|
9050967cd6 |
Release v2.24.0: Unified get_node Tool with Code Review Fixes (#437)
* feat(tools): unify node information retrieval with get_node tool Implements v2.24.0 featuring a unified node information tool that consolidates get_node_info and get_node_essentials functionality while adding version history and type structure metadata capabilities. Key Features: - Unified get_node tool with progressive detail levels (minimal/standard/full) - Version history access (versions, compare, breaking changes, migrations) - Type structure metadata integration from v2.23.0 - Token-efficient defaults optimized for AI agents - Backward-compatible via private method preservation Breaking Changes: - Removed get_node_info tool (replaced by get_node with detail='full') - Removed get_node_essentials tool (replaced by get_node with detail='standard') - Tool count: 40 → 39 tools Implementation: - src/mcp/tools.ts: Added unified get_node tool definition - src/mcp/server.ts: Implemented getNode() with 7 mode-specific methods - Type structure integration via TypeStructureService.getStructure() - Updated documentation in CHANGELOG.md and README.md - Version bumped to 2.24.0 Token Costs: - minimal: ~200 tokens (basic metadata) - standard: ~1000-2000 tokens (essential properties, default) - full: ~3000-8000 tokens (complete information) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en Co-Authored-By: Claude <noreply@anthropic.com> * docs: update tools-documentation.ts to reference unified get_node tool Updated all references from deprecated get_node_essentials and get_node_info to the new unified get_node tool with appropriate detail levels. Changes: - Standard Workflow Pattern: Updated to show get_node with detail levels - Configuration Tools: Replaced two separate tool descriptions with unified get_node - Performance Characteristics: Updated to reference get_node detail levels - Usage Notes: Updated recommendation to use get_node with detail='standard' This completes the v2.24.0 unified get_node tool implementation. All 13/13 test scenarios passed in n8n-mcp-tester agent validation. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - www.aiadvisors.pl/en * test: update tests to reference unified get_node tool Updated test files to replace references to deprecated get_node_info and get_node_essentials tools with the new unified get_node tool. Changes: - tests/unit/mcp/tools.test.ts: Updated get_node tests and removed references to get_node_essentials in toolsWithExamples array and categories object - tests/unit/mcp/parameter-validation.test.ts: Updated all get_node_info references to get_node throughout the test suite Test results: Successfully reduced test failures from 11 to 3 non-critical failures: - 1 description length test (expected for unified tool with comprehensive docs) - 1 database initialization issue (test infrastructure, not related to changes) - 1 timeout issue (unrelated to changes) All get_node_info → get_node migration tests now pass successfully. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - www.aiadvisors.pl/en * fix: implement all code review fixes for v2.24.0 unified get_node tool Comprehensive improvements addressing all critical, high-priority, and code quality issues identified in code review. ## Critical Fixes (Phase 1) - Add missing getNode mock in parameter-validation tests - Shorten tool description from 670 to 288 characters (under 300 limit) ## High Priority Fixes (Phase 2) - Add null safety check in enrichPropertyWithTypeInfo (prevent crashes on null properties) - Add nodeType context to all error messages in handleVersionMode (better debugging) - Optimize version summary fetch (conditional on detail level, skip for minimal mode) - Add comprehensive parameter validation for detail and mode with clear error messages ## Code Quality Improvements (Phase 3) - Refactor property enrichment with new enrichPropertiesWithTypeInfo helper (eliminate duplication) - Add TypeScript interfaces for all return types (replace any with proper union types) - Implement version data caching with 24-hour TTL (improve performance) - Enhance JSDoc documentation with detailed parameter explanations ## New TypeScript Interfaces - VersionSummary: Version metadata structure - NodeMinimalInfo: ~200 token response for minimal detail - NodeStandardInfo: ~1-2K token response for standard detail - NodeFullInfo: ~3-8K token response for full detail - VersionHistoryInfo: Version history response - VersionComparisonInfo: Version comparison response - NodeInfoResponse: Union type for all possible responses ## Testing - All 130 test files passed (3778 tests, 42 skipped) - Build successful with no TypeScript errors - Proper test mocking for unified get_node tool Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: update integration tests to use unified get_node tool Replace all references to deprecated get_node_info and get_node_essentials with the new unified get_node tool in integration tests. ## Changes - Replace get_node_info → get_node in 6 integration test files - Replace get_node_essentials → get_node in 2 integration test files - All tool calls now use unified interface ## Files Updated - tests/integration/mcp-protocol/error-handling.test.ts - tests/integration/mcp-protocol/performance.test.ts - tests/integration/mcp-protocol/session-management.test.ts - tests/integration/mcp-protocol/tool-invocation.test.ts - tests/integration/mcp-protocol/protocol-compliance.test.ts - tests/integration/telemetry/mcp-telemetry.test.ts This fixes CI test failures caused by calling removed tools. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * test: add comprehensive tests for unified get_node tool Add 81 comprehensive unit tests for the unified get_node tool to improve code coverage of the v2.24.0 implementation. ## Test Coverage ### Parameter Validation (6 tests) - Invalid detail/mode validation with clear error messages - All valid parameter combinations - Default values and node type normalization ### Info Mode Tests (21 tests) - Minimal detail: Basic metadata only, no version info (~200 tokens) - Standard detail: Essentials with version info (~1-2K tokens) - Full detail: Complete info with version info (~3-8K tokens) - includeTypeInfo and includeExamples parameter handling ### Version Mode Tests (24 tests) - versions: Version history and details - compare: Version comparison with proper error handling - breaking: Breaking changes with upgradeSafe flags - migrations: Auto-migratable changes detection ### Helper Methods (18 tests) - enrichPropertyWithTypeInfo: Null safety, type handling, structure hints - enrichPropertiesWithTypeInfo: Array handling, mixed properties - getVersionSummary: Caching with 24-hour TTL ### Error Handling (3 tests) - Repository initialization checks - NodeType context in error messages - Invalid mode/detail handling ### Integration Tests (8 tests) - Mode routing logic - Cache effectiveness across calls - Type safety validation - Edge cases (empty data, alternatives, long names) ## Results - 81 tests passing - 100% coverage of new get_node methods - All parameter combinations tested - All error conditions covered Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: update integration test assertions for unified get_node tool Updated integration tests to match the new unified get_node response structure: - error-handling.test.ts: Added detail='full' parameter for large payload test - tool-invocation.test.ts: Updated property assertions for standard/full detail levels - Fixed duplicate describe block and comparison logic Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: correct property names in integration test for standard detail Updated test to check for requiredProperties and commonProperties instead of essentialProperties to match actual get_node response structure. Conceived by Romuald Członkowski - www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.24.0 |
||
|
Romuald Członkowski
|
717d6f927f |
Release v2.23.0: Type Structure Validation (Phases 1-4) (#434)
* feat: implement Phase 1 - Type Structure Definitions Phase 1 Complete: Type definitions and service layer for all 22 n8n NodePropertyTypes New Files: - src/types/type-structures.ts (273 lines) * TypeStructure and TypePropertyDefinition interfaces * Type guards: isComplexType, isPrimitiveType, isTypeStructure * ComplexPropertyType and PrimitivePropertyType unions - src/constants/type-structures.ts (677 lines) * Complete definitions for all 22 NodePropertyTypes * Structures for complex types (filter, resourceMapper, etc.) * COMPLEX_TYPE_EXAMPLES with real-world usage patterns - src/services/type-structure-service.ts (441 lines) * Static service class with 15 public methods * Type querying, validation, and metadata access * No database dependencies (code-only constants) - tests/unit/types/type-structures.test.ts (14 tests) - tests/unit/constants/type-structures.test.ts (39 tests) - tests/unit/services/type-structure-service.test.ts (64 tests) Modified Files: - src/types/index.ts - Export new type-structures module Test Results: - 117 tests passing (100% pass rate) - 99.62% code coverage (exceeds 90% target) - Zero breaking changes Key Features: - Complete coverage of all 22 n8n NodePropertyTypes - Real-world examples from actual workflows - Validation infrastructure ready for Phase 2 integration - Follows project patterns (static services, type guards) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * feat: implement Phase 2 type structure validation integration Integrates TypeStructureService into EnhancedConfigValidator to validate complex property types (filter, resourceMapper, assignmentCollection, resourceLocator) against their expected structures. **Changes:** 1. Enhanced Config Validator (src/services/enhanced-config-validator.ts): - Added `properties` parameter to `addOperationSpecificEnhancements()` - Implemented `validateSpecialTypeStructures()` - detects and validates special types - Implemented `validateComplexTypeStructure()` - deep validation for each type - Implemented `validateFilterOperations()` - validates filter operator/operation pairs 2. Test Coverage (tests/unit/services/enhanced-config-validator-type-structures.test.ts): - 23 comprehensive test cases - Filter validation: combinator, conditions, operation compatibility - ResourceMapper validation: mappingMode values - AssignmentCollection validation: assignments array structure - ResourceLocator validation: mode and value fields (3 tests skipped for debugging) **Validation Features:** - ✅ Filter: Validates combinator ('and'/'or'), conditions array, operator types - ✅ Filter Operations: Type-specific operation validation (string, number, boolean, dateTime, array) - ✅ ResourceMapper: Validates mappingMode ('defineBelow'/'autoMapInputData') - ✅ AssignmentCollection: Validates assignments array presence and type - ⚠️ ResourceLocator: Basic validation (needs debugging - 3 tests skipped) **Test Results:** - 20/23 new tests passing (87% success rate) - 97+ existing tests still passing - ZERO breaking changes **Next Steps:** - Debug resourceLocator test failures - Integrate structure definitions into MCP tools (getNodeEssentials, getNodeInfo) - Update tools documentation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: add type guard for condition.operator in validateFilterOperations Addresses code review warning W1 by adding explicit type checking for condition.operator before accessing its properties. This prevents potential runtime errors if operator is not an object. **Change:** - Added `typeof condition.operator !== 'object'` check in validateFilterOperations **Impact:** - More robust validation - Prevents edge case runtime errors - All tests still passing (20/23) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * feat: complete Phase 3 real-world type structure validation Implemented and validated type structure definitions against 91 real-world workflow templates from n8n.io with 100% pass rate. **Validation Results:** - Pass Rate: 100% (target: >95%) ✅ - False Positive Rate: 0% (target: <5%) ✅ - Avg Validation Time: 0.01ms (target: <50ms) ✅ - Templates Tested: 91 templates, 616 nodes, 776 validations **Changes:** 1. Filter Operations Enhancement (enhanced-config-validator.ts) - Added exists, notExists, isNotEmpty operations to all filter types - Fixed 6 validation errors for field existence checks - Operations now match real-world n8n workflow usage 2. Google Sheets Node Validator (node-specific-validators.ts) - Added validateGoogleSheets() to filter credential-provided fields - Removes false positives for sheetId (comes from credentials at runtime) - Fixed 113 validation errors (91% of all failures) 3. Phase 3 Validation Script (scripts/test-structure-validation.ts) - Loads and validates top 100 templates by popularity - Tests filter, resourceMapper, assignmentCollection, resourceLocator types - Generates detailed statistics and error reports - Supports compressed workflow data (gzip + base64) 4. npm Script (package.json) - Added test:structure-validation script using tsx All success criteria met for Phase 3 real-world validation. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: resolve duplicate validateGoogleSheets function (CRITICAL) Fixed build-breaking duplicate function implementation found in code review. **Issue:** - Two validateGoogleSheets() implementations at lines 234 and 1717 - Caused TypeScript compilation error: TS2393 duplicate function - Blocked all builds and deployments **Solution:** - Merged both implementations into single function at line 234 - Removed sheetId validation check (comes from credentials) - Kept all operation-specific validation logic - Added error filtering at end to remove credential-provided field errors - Maintains 100% pass rate on Phase 3 validation (776/776 validations) **Validation Confirmed:** - TypeScript compilation: ✅ Success - Phase 3 validation: ✅ 100% pass rate maintained - All 4 special types: ✅ 100% pass rate (filter, resourceMapper, assignmentCollection, resourceLocator) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * feat: complete Phase 3 real-world validation with 100% pass rate Phase 3: Real-World Type Structure Validation - COMPLETED Results: - 91 templates tested (616 nodes with special types) - 776 property validations performed - 100.00% pass rate (776/776 passed) - 0.00% false positive rate - 0.01ms average validation time (500x better than 50ms target) Type-specific results: - filter: 93/93 passed (100.00%) - resourceMapper: 69/69 passed (100.00%) - assignmentCollection: 213/213 passed (100.00%) - resourceLocator: 401/401 passed (100.00%) Changes: - Add scripts/test-structure-validation.ts for standalone validation - Add integration test suite for real-world structure validation - Update implementation plan with Phase 3 completion details - All success criteria exceeded (>95% pass rate, <5% FP, <50ms) Edge cases fixed: - Filter operations: Added exists, notExists, isNotEmpty support - Google Sheets: Properly handle credential-provided fields Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * feat: complete Phase 4 documentation and polish Phase 4: Documentation & Polish - COMPLETED Changes: - Created docs/TYPE_STRUCTURE_VALIDATION.md (239 lines) - comprehensive technical reference - Updated CLAUDE.md with Phase 1-3 completion and architecture updates - Added minimal structure validation notes to tools-documentation.ts (progressive discovery) Documentation approach: - Separate brief technical reference file (no README bloat) - Minimal one-line mentions in tools documentation - Comprehensive internal documentation (CLAUDE.md) - Respects progressive discovery principle All Phase 1-4 complete: - Phase 1: Type Structure Definitions ✅ - Phase 2: Validation Integration ✅ - Phase 3: Real-World Validation ✅ (100% pass rate) - Phase 4: Documentation & Polish ✅ Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: correct line counts and dates in Phase 4 documentation Code review feedback fixes: 1. Fixed line counts in TYPE_STRUCTURE_VALIDATION.md: - Type Definitions: 273 → 301 lines (actual) - Type Structures: 677 → 741 lines (actual) - Service Layer: 441 → 427 lines (actual) 2. Fixed completion dates: - Changed from 2025-01-21 to 2025-11-21 (November, not January) - Updated in both TYPE_STRUCTURE_VALIDATION.md and CLAUDE.md 3. Enhanced filter example: - Added rightValue field for completeness - Example now shows complete filter condition structure All corrections per code-reviewer agent feedback. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * chore: release v2.23.0 - Type Structure Validation (Phases 1-4) Version bump from 2.22.21 to 2.23.0 (minor version bump for new backwards-compatible feature) Changes: - Comprehensive CHANGELOG.md entry documenting all 4 phases - Version bumped in package.json, package.runtime.json, package-lock.json - Database included (consistent with release pattern) Type Structure Validation Feature (v2.23.0): - Phase 1: 22 complete type structures defined - Phase 2: Validation integrated in all MCP tools - Phase 3: 100% pass rate on 776 real-world validations (91 templates, 616 nodes) - Phase 4: Documentation and polish completed Key Metrics: - 100% pass rate on 776 validations - 0.01ms average validation time (500x faster than target) - 0% false positive rate - Zero breaking changes (100% backward compatible) - Automatic, zero-configuration operation Semantic Versioning: - Minor version bump (2.22.21 → 2.23.0) for new backwards-compatible feature - No breaking changes - All existing functionality preserved Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: update tests for Type Structure Validation improvements in v2.23.0 CI test failures fixed for Type Structure Validation: 1. Google Sheets validator test (node-specific-validators.test.ts:313-328) - Test now expects 'range' error instead of 'sheetId' error - sheetId is credential-provided and excluded from configuration validation - Validation correctly prioritizes user-provided fields 2. If node workflow validation test (workflow-fixed-collection-validation.test.ts:164-178) - Test now expects 3 errors instead of 1 - Type Structure Validation catches multiple filter structure errors: * Missing combinator field * Missing conditions field * Invalid nested structure (conditions.values) - Comprehensive error detection is correct behavior Both tests now correctly verify the improved validation behavior introduced in the Type Structure Validation system (v2.23.0). Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en --------- Co-authored-by: Claude <noreply@anthropic.com>v2.23.0 |
||
|
Romuald Członkowski
|
fc37907348 | fix: resolve empty settings validation error in workflow updates (#431) (#432) v2.22.21 | ||
|
Romuald Członkowski
|
47d9f55dc5 |
chore: update n8n to 1.120.3 and bump version to 2.22.20 (#430)
- Updated n8n from 1.119.1 to 1.120.3 - Updated n8n-core from 1.118.0 to 1.119.2 - Updated n8n-workflow from 1.116.0 to 1.117.0 - Updated @n8n/n8n-nodes-langchain from 1.118.0 to 1.119.1 - Rebuilt node database with 544 nodes (439 from n8n-nodes-base, 105 from @n8n/n8n-nodes-langchain) - Updated README badge with new n8n version - Updated CHANGELOG with dependency changes Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <noreply@anthropic.com>v2.22.20 |
||
|
Romuald Członkowski
|
5575630711 |
fix: eliminate stack overflow in session removal (#427) (#428)
Critical bug fix for production crashes during session cleanup. **Root Cause:** Infinite recursion caused by circular event handler chain: - removeSession() called transport.close() - transport.close() triggered onclose event handler - onclose handler called removeSession() again - Loop continued until stack overflow **Solution:** Delete transport from registry BEFORE closing to break circular reference: 1. Store transport reference 2. Delete from this.transports first 3. Close transport after deletion 4. When onclose fires, transport no longer found, no recursion **Impact:** - Eliminates "RangeError: Maximum call stack size exceeded" errors - Fixes session cleanup crashes every 5 minutes in production - Prevents potential memory leaks from failed cleanup **Testing:** - Added regression test for infinite recursion prevention - All 39 session management tests pass - Build and typecheck succeed Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en Closes #427v2.22.19 |
||
|
Romuald Członkowski
|
1bbfaabbc2 |
fix: add structural hash tracking for workflow mutations (#422)
* feat: add structural hashes and success tracking for workflow mutations Enables cross-referencing workflow_mutations with telemetry_workflows by adding structural hashes (nodeTypes + connections) alongside existing full hashes. **Database Changes:** - Added workflow_structure_hash_before/after columns - Added is_truly_successful computed column - Created 3 analytics views: successful_mutations, mutation_training_data, mutations_with_workflow_quality - Created 2 helper functions: get_mutation_success_rate_by_intent(), get_mutation_crossref_stats() **Code Changes:** - Updated mutation-tracker.ts to generate both hash types - Updated mutation-types.ts with new fields - Auto-converts to snake_case via existing toSnakeCase() function **Testing:** - Added 5 new unit tests for structural hash generation - All 17 tests passing **Tooling:** - Created backfill script to populate hashes for existing 1,499 mutations - Created comprehensive documentation (STRUCTURAL_HASHES.md) **Impact:** - Before: 0% cross-reference match rate - After: Expected 60-70% match rate (post-backfill) - Unlocks quality impact analysis, training data curation, and mutation pattern insights Conceived by Romuald Członkowski - www.aiadvisors.pl/en * fix: correct test operation types for structural hash tests Fixed TypeScript errors in mutation-tracker tests by adding required 'updates' parameter to updateNode operations. Used 'as any' for test operations to maintain backward compatibility while tests are updated. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * chore: remove documentation files from tracking Removed internal documentation files from version control: - Telemetry implementation docs - Implementation roadmap - Disabled tools analysis docs These files are for internal reference only. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * chore: remove telemetry documentation files from tracking Removed all telemetry analysis and documentation files from root directory. These files are for internal reference only and should not be in version control. Files removed: - TELEMETRY_ANALYSIS*.md - TELEMETRY_MUTATION_SPEC.md - TELEMETRY_*_DATASET.md - VALIDATION_ANALYSIS*.md 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * chore: bump version to 2.22.18 and update CHANGELOG Version 2.22.18 adds structural hash tracking for workflow mutations, enabling cross-referencing with workflow quality data and automated success detection. Key changes: - Added workflowStructureHashBefore/After fields - Added isTrulySuccessful computed field - Enhanced mutation tracking with structural hashes - All tests passing (17/17) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * chore: remove migration and documentation files from PR Removed internal database migration files and documentation from version control: - docs/migrations/ - docs/telemetry/ Updated CHANGELOG to remove database migration references. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Conceived by Romuald Członkowski - https://www.aiadvisors.pl/env2.22.18 |
||
|
Romuald Członkowski
|
597bd290b6 |
fix: critical telemetry improvements for data quality and security (#421)
* fix: critical telemetry improvements for data quality and security Fixed three critical issues in workflow mutation telemetry: 1. Fixed Inconsistent Sanitization (Security Critical) - Problem: 30% of workflows unsanitized, exposing credentials/tokens - Solution: Use robust WorkflowSanitizer.sanitizeWorkflowRaw() - Impact: 100% sanitization with 17 sensitive patterns redacted - Files: workflow-sanitizer.ts, mutation-tracker.ts 2. Enabled Validation Data Capture (Data Quality) - Problem: Zero validation metrics captured (all NULL) - Solution: Add pre/post mutation validation with WorkflowValidator - Impact: Measure mutation quality, track error resolution - Non-blocking validation that captures errors/warnings - Files: handlers-workflow-diff.ts 3. Improved Intent Capture (Data Quality) - Problem: 92.62% generic "Partial workflow update" intents - Solution: Enhanced docs + automatic intent inference - Impact: Meaningful intents auto-generated from operations - Files: n8n-update-partial-workflow.ts, handlers-workflow-diff.ts Expected Results: - 100% sanitization coverage (up from 70%) - 100% validation capture (up from 0%) - 50%+ meaningful intents (up from 7.33%) Version bumped to 2.22.17 🤖 Generated with [Claude Code](https://claude.com/claude-code) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en Co-Authored-By: Claude <noreply@anthropic.com> * perf: implement validator instance caching to avoid redundant initialization - Add module-level cached WorkflowValidator instance - Create getValidator() helper to reuse validator across mutations - Update pre/post mutation validation to use cached instance - Avoids redundant NodeSimilarityService initialization on every mutation Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: restore backward-compatible sanitization with context preservation Fixed CI test failures by updating WorkflowSanitizer to use pattern-specific placeholders while maintaining backward compatibility: Changes: - Convert SENSITIVE_PATTERNS to PatternDefinition objects with specific placeholders - Update sanitizeString() to preserve context (Bearer prefix, URL paths) - Refactor sanitizeObject() to handle sensitive fields vs URL fields differently - Remove overly greedy field patterns that conflicted with token patterns Pattern-specific placeholders: - [REDACTED_URL_WITH_AUTH] for URLs with credentials - [REDACTED_TOKEN] for long tokens (32+ chars) - [REDACTED_APIKEY] for OpenAI-style keys - Bearer [REDACTED] for Bearer tokens (preserves "Bearer " prefix) - [REDACTED] for generic sensitive fields Test Results: - All 13 mutation-tracker tests passing - URL with auth: preserves path after credentials - Long tokens: properly detected and marked - OpenAI keys: correctly identified - Bearer tokens: prefix preserved - Sensitive field names: generic redaction for non-URL fields Fixes #421 CI failures Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: prevent double-redaction in workflow sanitizer Added safeguard to stop pattern matching once a placeholder is detected, preventing token patterns from matching text inside placeholders like [REDACTED_URL_WITH_AUTH]. Also expanded database URL pattern to match full URLs including port and path, and updated test expectations to match context-preserving sanitization. Fixes: - Database URLs now properly sanitized to [REDACTED_URL_WITH_AUTH] - Prevents [[REDACTED]] double-redaction issue - All 25 workflow-sanitizer tests passing - No regression in mutation-tracker tests Conceived by Romuald Członkowski - www.aiadvisors.pl/en --------- Co-authored-by: Claude <noreply@anthropic.com>v2.22.17 |
||
|
Romuald Członkowski
|
99c5907b71 |
feat: enhance workflow mutation telemetry for better AI responses (#419)
* feat: add comprehensive telemetry for partial workflow updates Implement telemetry infrastructure to track workflow mutations from partial update operations. This enables data-driven improvements to partial update tooling by capturing: - Workflow state before and after mutations - User intent and operation patterns - Validation results and improvements - Change metrics (nodes/connections modified) - Success/failure rates and error patterns New Components: - Intent classifier: Categorizes mutation patterns - Intent sanitizer: Removes PII from user instructions - Mutation validator: Ensures data quality before tracking - Mutation tracker: Coordinates validation and metric calculation Extended Components: - TelemetryManager: New trackWorkflowMutation() method - EventTracker: Mutation queue management - BatchProcessor: Mutation data flushing to Supabase MCP Tool Enhancements: - n8n_update_partial_workflow: Added optional 'intent' parameter - n8n_update_full_workflow: Added optional 'intent' parameter - Both tools now track mutations asynchronously Database Schema: - New workflow_mutations table with 20+ fields - Comprehensive indexes for efficient querying - Supports deduplication and data analysis This telemetry system is: - Privacy-focused (PII sanitization, anonymized users) - Non-blocking (async tracking, silent failures) - Production-ready (batching, retries, circuit breaker) - Backward compatible (all parameters optional) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: correct SQL syntax for expression index in workflow_mutations schema The expression index for significant changes needs double parentheses around the arithmetic expression to be valid PostgreSQL syntax. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: enable RLS policies for workflow_mutations table Enable Row-Level Security and add policies: - Allow anonymous (anon) inserts for telemetry data collection - Allow authenticated reads for data analysis and querying These policies are required for the telemetry system to function correctly with Supabase, as the MCP server uses the anon key to insert mutation data. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: reduce mutation auto-flush threshold from 5 to 2 Lower the auto-flush threshold for workflow mutations from 5 to 2 to ensure more timely data persistence. Since mutations are less frequent than regular telemetry events, a lower threshold provides: - Faster data persistence (don't wait for 5 mutations) - Better testing experience (easier to verify with fewer operations) - Reduced risk of data loss if process exits before threshold - More responsive telemetry for low-volume mutation scenarios This complements the existing 5-second periodic flush and process exit handlers, ensuring mutations are persisted promptly. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * fix: improve mutation telemetry error logging and diagnostics Changes: - Upgrade error logging from debug to warn level for better visibility - Add diagnostic logging to track mutation processing - Log telemetry disabled state explicitly - Add context info (sessionId, intent, operationCount) to error logs - Remove 'await' from telemetry calls to make them truly non-blocking This will help identify why mutations aren't being persisted to the workflow_mutations table despite successful workflow operations. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en * feat: enhance workflow mutation telemetry for better AI responses Improve workflow mutation tracking to capture comprehensive data that helps provide better responses when users update workflows. This enhancement collects workflow state, user intent, and operation details to enable more context-aware assistance. Key improvements: - Reduce auto-flush threshold from 5 to 2 for more reliable mutation tracking - Add comprehensive workflow and credential sanitization to mutation tracker - Document intent parameter in workflow update tools for better UX - Fix mutation queue handling in telemetry manager (flush now handles 3 queues) - Add extensive unit tests for mutation tracking and validation (35 new tests) Technical changes: - mutation-tracker.ts: Multi-layer sanitization (workflow, node, parameter levels) - batch-processor.ts: Support mutation data flushing to Supabase - telemetry-manager.ts: Auto-flush mutations at threshold 2, track mutations queue - handlers-workflow-diff.ts: Track workflow mutations with sanitized data - Tests: 13 tests for mutation-tracker, 22 tests for mutation-validator The intent parameter messaging emphasizes user benefit ("helps to return better response") rather than technical implementation details. Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * chore: bump version to 2.22.16 with telemetry changelog Updated package.json and package.runtime.json to version 2.22.16. Added comprehensive CHANGELOG entry documenting workflow mutation telemetry enhancements for better AI-powered workflow assistance. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en Co-Authored-By: Claude <noreply@anthropic.com> * fix: resolve TypeScript lint errors in telemetry tests Fixed type issues in mutation-tracker and mutation-validator tests: - Import and use MutationToolName enum instead of string literals - Fix ValidationResult.errors to use proper object structure - Add UpdateNodeOperation type assertion for operation with nodeName All TypeScript errors resolved, lint now passes. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>v2.22.16 |
||
|
Romuald Członkowski
|
77151e013e | chore: update n8n to 1.119.1 (#414) v2.22.15 | ||
|
Romuald Członkowski
|
14f3b9c12a |
Merge pull request #411 from czlonkowski/feat/disabled-tools-env-var
feat: Add DISABLED_TOOLS environment variable for tool filtering (Issue #410)v2.22.14 |
||
|
czlonkowski
|
eb362febd6 |
test: Add critical missing tests for DISABLED_TOOLS feature
Add tests for two critical features identified by code review: 1. 10KB Safety Limit Test: - Verify DISABLED_TOOLS environment variable is truncated at 10KB - Test with 15KB input to ensure truncation works - Confirm first tools are parsed, last tools are excluded - Prevents DoS attacks from massive environment variables 2. Security Information Disclosure Test: - Verify error messages only reveal attempted tool name - Ensure full list of disabled tools is NOT leaked - Critical security test to prevent configuration disclosure - Tests defense against information leakage attacks Test Coverage: - Total tests: 47 (up from 45) - Both tests passing - Addresses critical gaps from code review Files Modified: - tests/unit/mcp/disabled-tools-additional.test.ts Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|
czlonkowski
|
821ace310e |
refactor: Improve DISABLED_TOOLS implementation based on code review
Performance Optimization: - Add caching to getDisabledTools() to prevent 3x parsing per request - Cache result as instance property disabledToolsCache - Reduces overhead from 3x to 1x per server instance Security Improvements: - Fix information disclosure in error responses - Only reveal the attempted tool name, not full list of disabled tools - Prevents leaking security configuration details Safety Limits: - Add 10KB maximum length for DISABLED_TOOLS environment variable - Add 200-tool maximum limit to prevent abuse - Include warnings when limits are exceeded Code Quality: - Add clarifying comment for defense-in-depth guard in executeTool() - Change logging level from info to debug for frequent operations - Add comprehensive JSDoc to TestableN8NMCPServer test classes - Document test wrapper pattern and exposed methods Test Updates: - Update test to verify 200-tool safety limit enforcement - All 45 tests passing with improved coverage Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|
czlonkowski
|
53252adc68 |
feat: Add DISABLED_TOOLS environment variable for tool filtering (Issue #410)
Added DISABLED_TOOLS environment variable to filter specific tools from registration at startup, enabling deployment-specific tool configuration for multi-tenant deployments, security hardening, and feature flags. ## Implementation - Added getDisabledTools() method to parse comma-separated tool names from env var - Modified ListToolsRequestSchema handler to filter both documentation and management tools - Modified CallToolRequestSchema handler to reject disabled tool calls with clear error messages - Added defense-in-depth guard in executeTool() method ## Features - Environment variable format: DISABLED_TOOLS=tool1,tool2,tool3 - O(1) lookup performance using Set data structure - Clear error messages with TOOL_DISABLED code - Backward compatible (no DISABLED_TOOLS = all tools enabled) - Comprehensive logging for observability ## Use Cases - Multi-tenant: Hide tools that check global env vars - Security: Disable management tools in production - Feature flags: Gradually roll out new tools - Deployment-specific: Different tool sets for cloud vs self-hosted ## Testing - 45 comprehensive tests (all passing) - 95% feature code coverage - Unit tests + additional test scenarios - Performance tested with 1000 tools (<100ms) ## Files Modified - src/mcp/server.ts - Core implementation (~40 lines) - .env.example, .env.docker - Configuration documentation - tests/unit/mcp/disabled-tools*.test.ts - Comprehensive tests - package.json, package.runtime.json - Version bump to 2.22.14 - CHANGELOG.md - Full documentation Resolves #410 Conceived by Romuald Członkowski - www.aiadvisors.pl/en |
||
|
Romuald Członkowski
|
2010d77ed8 |
Merge pull request #407 from czlonkowski/feat/telemetry-quick-wins-validation-errors
feat: Telemetry-driven quick wins to reduce AI agent validation errors by 30-40%v2.22.13 |
||
|
czlonkowski
|
caf9383ba1 |
test: Add comprehensive edge case coverage for telemetry quick wins
Added 20 edge case tests based on code review recommendations: **Duplicate ID Validation (4 tests)**: - Multiple duplicate IDs (3+ nodes with same ID) - Duplicate IDs with same node type - Duplicate IDs with empty/null node names - Duplicate IDs with missing node properties **AI Agent Validator (16 tests)**: maxIterations edge cases (7 tests): - Boundary values: 0 (reject), 1 (accept), 51 (warn), MAX_SAFE_INTEGER (warn) - Invalid types: NaN (reject), negative decimal (reject) - Threshold testing: 50 vs 51 promptType validation (4 tests): - Whitespace-only text (reject) - Very long text 3200+ chars (accept) - undefined/null text (reject) System message validation (5 tests): - Empty/whitespace messages (suggest adding) - Very long messages >1000 chars (accept) - Special characters, emojis, unicode (accept) - Multi-line formatting (accept) - Boundary: 19 chars (warn), 20 chars (accept) **Test Quality Improvements**: - Fixed flaky system message test (changed from expect.stringContaining to .some()) - All tests are deterministic - Comprehensive inline comments - Follows existing test patterns All 20 new tests passing. Zero regressions. Concieved by Romuald Członkowski - www.aiadvisors.pl/en |
||
|
czlonkowski
|
8728a808ac |
fix: AI Agent validator not executing due to nodeType format mismatch (Critical)
Fixed critical bug where AI Agent validator never executed, missing 179 configuration errors (30% of all telemetry-identified failures). The Bug: - Switch case checked for '@n8n/n8n-nodes-langchain.agent' (full package format) - But nodeType was normalized to 'nodes-langchain.agent' before reaching switch - Result: AI Agent validator never matched, never executed The Fix: - Changed case to 'nodes-langchain.agent' to match normalized format - Now correctly catches prompt configuration, maxIterations, error handling issues Files Changed: - src/services/enhanced-config-validator.ts:322 - Fixed nodeType format - tests/unit/services/enhanced-config-validator.test.ts - Added validateAIAgent to mock and verification test - CHANGELOG.md - Added bug fix section to 2.22.13 (not separate version) Testing: - npm test -- tests/unit/services/enhanced-config-validator.test.ts - ✓ All 51 tests pass including new AI Agent validation test Discovery: Discovered by n8n-mcp-tester agent during post-deployment verification of 2.22.13 improvements. The agent attempted to validate an AI Agent node configuration and discovered the validator was never being called. Impact: - Without fix: 179 AI Agent configuration errors (30%) go undetected - With fix: All AI Agent validation rules now execute correctly Version: 2.22.13 (kept under same version as original implementation) Concieved by Romuald Członkowski - www.aiadvisors.pl/en |
||
|
czlonkowski
|
60ab66d64d |
feat: telemetry-driven quick wins to reduce AI agent validation errors by 30-40%
Enhanced tools documentation, duplicate ID errors, and AI Agent validator based on telemetry analysis of 593 validation errors across 3 categories: - 378 errors: Duplicate node IDs (64%) - 179 errors: AI Agent configuration (30%) - 36 errors: Other validations (6%) Quick Win #1: Enhanced tools documentation (src/mcp/tools-documentation.ts) - Added prominent warnings to call get_node_essentials() FIRST before configuring nodes - Emphasized 5KB vs 100KB+ size difference between essentials and full info - Updated workflow patterns to prioritize essentials over get_node_info Quick Win #2: Improved duplicate ID error messages (src/services/workflow-validator.ts) - Added crypto import for UUID generation examples - Enhanced error messages with node indices, names, and types - Included crypto.randomUUID() example in error messages - Helps AI agents understand EXACTLY which nodes conflict and how to fix Quick Win #3: Added AI Agent node-specific validator (src/services/node-specific-validators.ts) - Validates prompt configuration (promptType + text requirement) - Checks maxIterations bounds (1-50 recommended) - Suggests error handling (onError + retryOnFail) - Warns about high iteration limits (cost/performance impact) - Integrated into enhanced-config-validator.ts Test Coverage: - Added duplicate ID validation tests (workflow-validator.test.ts) - Added AI Agent validator tests (node-specific-validators.test.ts:2312-2491) - All new tests passing (3527 total passing) Version: 2.22.12 → 2.22.13 Expected Impact: 30-40% reduction in AI agent validation errors Technical Details: - Telemetry analysis: 593 validation errors (Dec 2024 - Jan 2025) - 100% error recovery rate maintained (validation working correctly) - Root cause: Documentation/guidance gaps, not validation logic failures - Solution: Proactive guidance at decision points References: - Telemetry analysis findings - Issue #392 (helpful error messages pattern) - Existing Slack validator pattern (node-specific-validators.ts:98-230) Concieved by Romuald Członkowski - www.aiadvisors.pl/en |
||
|
Romuald Członkowski
|
eee52a7f53 |
Merge pull request #406 from czlonkowski/fix/helpful-error-changes-vs-updates
fix: Add helpful error messages for 'changes' vs 'updates' parameter (Issue #392)v2.22.12 |
||
|
czlonkowski
|
a66cb18cce |
fix: Add helpful error messages for 'changes' vs 'updates' parameter (Issue #392)
Fixed cryptic "Cannot read properties of undefined (reading 'name')" error when
users mistakenly use 'changes' instead of 'updates' in updateNode operations.
Changes:
- Added early validation in validateUpdateNode() to detect common parameter mistake
- Provides clear, educational error messages with examples
- Fixed outdated documentation example in VS_CODE_PROJECT_SETUP.md
- Added comprehensive test coverage (2 test cases)
Error Messages:
- Before: "Diff engine error: Cannot read properties of undefined (reading 'name')"
- After: "Invalid parameter 'changes'. The updateNode operation requires 'updates'
(not 'changes'). Example: {type: "updateNode", nodeId: "abc", updates: {...}}"
Testing:
- Test coverage: 85% confidence (production ready)
- n8n-mcp-tester: All 3 test cases passed
- Code review: Approved with minor optional suggestions
Impact:
- AI agents now receive actionable error messages
- Self-correction enabled through clear examples
- Zero breaking changes (backward compatible)
- Follows existing patterns from Issue #249
Files Modified:
- src/services/workflow-diff-engine.ts (10 lines added)
- docs/VS_CODE_PROJECT_SETUP.md (1 line fixed)
- tests/unit/services/workflow-diff-engine.test.ts (2 tests added)
- CHANGELOG.md (comprehensive entry)
- package.json (version bump to 2.22.12)
Fixes #392
Conceived by Romuald Członkowski - www.aiadvisors.pl/en
|
||
|
Romuald Członkowski
|
0e0f0998af | Merge pull request #403 from czlonkowski/feat/workflow-activation-operations v2.22.11 | ||
|
czlonkowski
|
08a4be8370 |
fix: Add missing typeVersion to workflow activation test nodes
Fixed TypeScript linting errors in workflow-diff-engine.test.ts by adding typeVersion: 1 to all test nodes that were missing it. Fixes CI linting failures in Test Suite workflow. Conceived by Romuald Członkowski - www.aiadvisors.pl/en |