fix: align validator guidance with n8n-mcp 2.63.0 (v1.22.0)#36
Merged
Conversation
The n8n_generate_workflow tool was removed from n8n-mcp; drop it from the tools-expert skill (categories, quick-reference, WORKFLOW_GUIDE section) and the router skill's tool list. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…d by Romuald Członkowski - www.aiadvisors.pl/en) n8n-mcp 2.63.0 overhauled the validators (czlonkowski/n8n-mcp#911): the classic false positives are fixed at the source and validation profiles now genuinely differ (minimal/runtime = errors + security/deprecation; ai-friendly/strict add best-practice advisories). This release updates the pack to describe current behavior: - n8n-validation-expert: removed the fabricated 'known issues #304/#306/#338' guidance and the stale '40% of warnings acceptable / ai-friendly reduces FPs by 60%' claims; reframed FALSE_POSITIVES.md around profile-gated advisories; refreshed profile docs, error catalog examples, and review checklist cross-references - n8n-mcp-tools-expert: accurate profile gating in VALIDATION_GUIDE - n8n-expression-syntax: precise template-literal guidance (backticks with ${} interpolation are valid INSIDE {{ }}; bare template literals without {{ }} remain literal text); optional chaining and string-keyed bracket access documented as valid - n8n-code-javascript: bare-object returns auto-wrap in All Items mode (canonical [{json}] still recommended); removed obsolete validator-appeasement advice ({{ }} in strings, this.helpers, regex $ anchors) Every rewritten behavioral claim was verified by running probe workflows through the 2.63.0 validator build under all four profiles. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the n8n skills pack documentation to match the corrected validator behavior in n8n-mcp ≥ 2.63.0, removing prior “false-positive” workarounds and clarifying what is must-fix vs advisory across validation profiles. Also bumps the pack version to v1.22.0 and refreshes distribution artifacts.
Changes:
- Reframes validator guidance across profiles (minimal/runtime/ai-friendly/strict) and removes/rewrites outdated “false positive” guidance.
- Updates expression and Code-node docs to reflect current supported syntax/return-shape behavior.
- Bumps plugin/build versions and refreshes dist zips.
Reviewed changes
Copilot reviewed 16 out of 36 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| skills/using-n8n-mcp-skills/SKILL.md | Updates router guidance and tool list references. |
| skills/n8n-validation-expert/SKILL.md | Revises validation profiles, warning semantics, and false-positive guidance. |
| skills/n8n-validation-expert/REVIEW_CHECKLIST.md | Updates review checklist items and cross-references (notably error-handling). |
| skills/n8n-validation-expert/README.md | Updates summary claims and profile descriptions to match ≥2.63.0 behavior. |
| skills/n8n-validation-expert/FALSE_POSITIVES.md | Reframes “false positives” as advisories and updates examples/strategy guidance. |
| skills/n8n-validation-expert/ERROR_CATALOG.md | Updates examples/notes for validator behavior (expressions, operator shapes, typeVersion notes). |
| skills/n8n-mcp-tools-expert/WORKFLOW_GUIDE.md | Removes n8n_generate_workflow documentation section. |
| skills/n8n-mcp-tools-expert/VALIDATION_GUIDE.md | Updates profile gating guidance and related explanations. |
| skills/n8n-mcp-tools-expert/SKILL.md | Removes n8n_generate_workflow references from the skill index. |
| skills/n8n-expression-syntax/COMMON_MISTAKES.md | Clarifies template literals/concatenation behavior inside vs outside {{ }}. |
| skills/n8n-code-javascript/SKILL.md | Updates Code-node return-shape guidance and common pitfalls. |
| skills/n8n-code-javascript/README.md | Aligns README guidance with updated return-shape behavior. |
| skills/n8n-code-javascript/ERROR_PATTERNS.md | Refines error taxonomy (return shape vs prior “wrapper” rules) and clarifies {{ }} cases. |
| dist/README.md | Documents distribution packages and install instructions (currently version-mismatched). |
| dist/n8n-subworkflows-v1.22.0.zip | Updated built distribution artifact. |
| dist/n8n-self-hosting-v1.22.0.zip | Updated built distribution artifact. |
| dist/n8n-multi-instance-v1.22.0.zip | Updated built distribution artifact. |
| dist/n8n-code-tool-v1.22.0.zip | Updated built distribution artifact. |
| build.sh | Bumps build version to 1.22.0. |
| .claude-plugin/plugin.json | Bumps plugin version to 1.22.0. |
| .claude-plugin/marketplace.json | Bumps marketplace version to 1.22.0. |
Comments suppressed due to low confidence (1)
skills/n8n-validation-expert/ERROR_CATALOG.md:503
- ERROR_CATALOG currently contradicts itself about whether static validation resolves node names inside expressions. It says validation does not resolve node names/paths inside expressions (and references
$('Node')), but the examples still show validation errors like "Referenced node ... does not exist" for$node['...']expressions. This needs to be reconciled so readers don’t get misled about what validation can/can’t catch.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
50
to
62
| **Types**: | ||
| - `best_practice` - Recommended but not required | ||
| - `deprecated` - Using old API/feature | ||
| - `performance` - Potential performance issue | ||
| - `best_practice` - Recommended but not required — surfaces under `ai-friendly` / `strict` only | ||
| - `deprecated` - Using old API/feature — surfaces under every profile | ||
| - `security` - Hardcoded secrets, unauthenticated webhooks — surfaces under every profile | ||
| - `performance` - Potential performance issue — advisory, `ai-friendly` / `strict` | ||
|
|
||
| **Example**: | ||
| **Example** (best-practice — appears under `ai-friendly` / `strict`): | ||
| ```json | ||
| { | ||
| "type": "best_practice", | ||
| "property": "errorHandling", | ||
| "message": "Slack API can have rate limits", | ||
| "suggestion": "Add onError: 'continueRegularOutput' with retryOnFail" | ||
| "type": "warning", | ||
| "nodeName": "Slack", | ||
| "message": "Slack API can have rate limits and transient failures" | ||
| } |
czlonkowski
added a commit
to czlonkowski/n8n-mcp
that referenced
this pull request
Jul 3, 2026
…3.0 (Concieved by Romuald Członkowski - www.aiadvisors.pl/en) (#916) Syncs data/skills from n8n-skills v1.22.0 (czlonkowski/n8n-skills#36): validator guidance now describes 2.63.0 behavior (real profile gating, fixed false-positive classes, precise template-literal/optional-chaining guidance, bare-object return auto-wrap), plus all skills added to the pack since the last sync (agents, error-handling, binary-and-data, code-tool, subworkflows, multi-instance, self-hosting, router). sync-skills.ts now skips skill-creator eval workspace directories (skills/*-workspace/) so local eval debris never ships in artifacts. Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
n8n-mcp 2.63.0 (czlonkowski/n8n-mcp#911) fixed the validator's false-positive classes at the source and repaired profile gating. This release updates the pack so its guidance describes current behavior instead of working around bugs that no longer exist.
Changes
{{ }}; a bare template literal without{{ }}remains literal text. Optional chaining and string-keyed bracket access documented as valid.[{json: {...}}]still recommended); removed obsolete validator-appeasement advice.Verification
Every rewritten behavioral claim was verified empirically by running probe workflows through the 2.63.0 validator build under all four profiles (~30 probes: template literals, optional chaining, IF v1/v2 shapes, responseNode webhooks, onError variants, Code-node returns, profile gating). A consistency sweep confirmed zero leftover stale markers across the whole pack.
Version 1.21.1 → 1.22.0; dist zips rebuilt.
Note: the pack targets n8n-mcp ≥ 2.63.0 behavior — merge after czlonkowski/n8n-mcp#911 ships.
Concieved by Romuald Członkowski - www.aiadvisors.pl/en
🤖 Generated with Claude Code