Content
71%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A highly actionable, n8n-specific reference with excellent correct/incorrect code examples and decision routing, weakened by repetitive JavaScript-first messaging, basic-Python explanations, and four prominently referenced companion files that are missing from the bundle. The dangling references are the most consequential defect since the body repeatedly defers detail to them.
Suggestions
Ship the four referenced files (DATA_ACCESS.md, COMMON_PATTERNS.md, ERROR_PATTERNS.md, STANDARD_LIBRARY.md) or remove/inline their references — the body defers substantial detail to files that do not exist in the bundle.
Consolidate the JavaScript-first recommendation to one section and delete its repeats in Essential Rules, the When-to-Use section, and the closing line; likewise fold the duplicated Essential Rules into their dedicated sections.
Cut Best Practices items that teach basic Python Claude already knows (.get() defaults, list comprehension vs manual loop) and keep only n8n-specific guidance; add an explicit run → error → ERROR_PATTERNS fix → retry loop for the deployment checklist.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient n8n-specific content, but with real padding: the JavaScript-first recommendation is repeated in at least four places ("⚠️ Important: JavaScript First", "Essential Rules #1", "When to Use Python vs JavaScript", the closing line); "Essential Rules" duplicates the later Data Access / Return Format / Limitations sections; and Best Practices items 1–3 (.get(), list comprehension vs manual loop) explain basic Python Claude already knows. Not anchor 2 because the bulk is genuinely non-obvious n8n knowledge (webhook body nesting, _node.first() semantics, SplitInBatches outputs); not anchor 4 because the repetition and beginner-Python explanations are more than minor trims. | 3 / 5 |
Actionability | Every section gives copy-paste-ready executable code with correct-vs-incorrect pairs ("# ❌ WRONG ... # ✅ CORRECT"), named error classes (ModuleNotFoundError, KeyError), concrete module lists, and specific workaround routes (HTTP Request node, Set/Filter/IF nodes, search_nodes/get_node/validate_node MCP calls). Matches the anchor-5 pattern of runnable examples covering the common cases. | 5 / 5 |
Workflow Clarity | Decision routing is well sequenced — JavaScript-first check → mode selection (All Items vs Each Item with when-to-use) → Beta vs Native mode → data access patterns → Python/JS/other-node decision lists — and the closing "Quick Reference Checklist" acts as a pre-deploy validation gate. Not anchor 5 because there is no explicit error-recovery feedback loop (run → read error → apply ERROR_PATTERNS fix → retry) for the fragile batch/deployment cases it warns about. | 4 / 5 |
Progressive Disclosure | The in-SKILL.md structure is good — lean overview sections with well-signaled, one-level-deep pointers to [DATA_ACCESS.md], [COMMON_PATTERNS.md], [ERROR_PATTERNS.md], [STANDARD_LIBRARY.md], each annotated with what it contains — but none of these four referenced files exist in the bundle (no references/, scripts/, or assets/ directories at all). Scored against the actual bundle structure, the promised detail layer is inaccessible, which is a structural failure beyond the 'minor organization gaps' of anchor 4; better than anchor 2 because the overview itself is genuinely well organized and signals references clearly. | 3 / 5 |
Total | 15 / 20 Passed |