Content
60%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.
The content is highly actionable with abundant executable examples, but it is significantly redundant and teaches generic JavaScript Claude already knows. Critically, it cites four reference files that do not exist in the bundle, so its progressive-disclosure structure fails in practice.
Suggestions
Remove duplicate treatments: state the webhook '.body' nesting and return-format rules once each instead of repeating them across Essential Rules, dedicated sections, and Error Prevention.
Cut generic JavaScript guidance ('Use Descriptive Variable Names', 'Prefer Array Methods Over Loops', 'Debug with console.log()') that Claude already knows, keeping only n8n-specific behavior.
Either bundle the referenced files (DATA_ACCESS.md, COMMON_PATTERNS.md, ERROR_PATTERNS.md, BUILTIN_FUNCTIONS.md) and trim the inlined duplicates from SKILL.md, or remove the dangling references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Noticeably verbose with repeated sections: webhook '.body' nesting is explained three times (Essential Rules #4, 'Critical: Webhook Data Structure', Error Prevention #5), return-format rules appear in three places, and the Best Practices section teaches generic JavaScript Claude already knows ('Use Descriptive Variable Names', 'Prefer Array Methods Over Loops', 'Debug with console.log()'). Not 3 because the padding spans multiple whole sections rather than occasional loose sentences. | 2 / 5 |
Actionability | Fully executable, copy-paste-ready code throughout: Quick Start template, mode-selection examples, data-access patterns, correct/incorrect return-format pairs, $helpers.httpRequest, DateTime, and $jmespath examples covering the common cases. Not 4 because the examples are complete, runnable, and span the skill's typical usage. | 5 / 5 |
Workflow Clarity | Clear sequencing (Quick Start → mode selection → data access → return format → patterns → errors) with a decision shortcut for mode choice and a 'Quick Reference Checklist' acting as a pre-deploy checkpoint. Not 5 because there is no explicit validate→fix→retry feedback loop or testing step; not 3 because sequence and checklist checkpoints are present. | 4 / 5 |
Progressive Disclosure | The body repeatedly signals 'See: DATA_ACCESS.md', 'COMMON_PATTERNS.md', 'ERROR_PATTERNS.md', and 'BUILTIN_FUNCTIONS.md', but no bundle files exist — all references are dangling. Meanwhile the ~700-line body inlines the very content (patterns overview, built-in reference) it claims those files hold. Not 3 because references that resolve to nothing make navigation fail entirely; not 1 because section structure is present and references are clearly signaled. | 2 / 5 |
Total | 13 / 20 Passed |