Content
85%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 strong, highly actionable reference body with exemplary code examples, decision guides, and a validation checklist, weakened by repeated restatements of the same critical facts and — more seriously — by referencing four detail files (DATA_ACCESS.md, COMMON_PATTERNS.md, ERROR_PATTERNS.md, BUILTIN_FUNCTIONS.md) that are missing from the bundle. Progressive disclosure is structurally sound but functionally broken until those files ship.
Suggestions
Ship the four referenced bundle files (DATA_ACCESS.md, COMMON_PATTERNS.md, ERROR_PATTERNS.md, BUILTIN_FUNCTIONS.md) or remove/inline their 'See' pointers — every progressive-disclosure link currently dangles.
Deduplicate the repeated critical facts: state the $helpers-undefined sandbox note, the auto-wrap return behavior, and the webhook .body nesting once each (in the Built-ins, Return Format, and Webhook sections respectively) and have the other mentions point there instead of restating.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — no JS tutorials, no 'what is n8n' padding, and high-value specifics like the per-item cost table and sandbox flags. It falls between anchors 3 and 4 because several key facts repeat: the $helpers-undefined note appears in Essential Rules #5 and again in the Built-ins section, the auto-wrap return behavior is explained in Rules #3, the Return Format section, and Top Mistakes #3, and webhook .body nesting appears three times. Noticeably above the midpoint, but the redundancy is trimmable. | 4 / 5 |
Actionability | Fully executable throughout: quick-start template, mode-specific examples, wrong/right code pairs for returns and webhook access, exact env flags (N8N_BLOCK_ENV_ACCESS_IN_NODE, N8N_RUNNERS_ALLOWED_BUILT_IN_MODULES), and exact failure strings ('Code doesn't return items properly', ReferenceError, paired_item_no_info). Copy-paste ready and covers the common cases; not a 4 because there are no pseudocode gaps. | 5 / 5 |
Workflow Clarity | Clear decision sequencing with explicit checkpoints: a mode-selection guide with a decision shortcut, an ordered transform gatekeeper (expression → IIFE → Code node), wrong/right error-recovery pairs, and a pre-deploy Quick Reference Checklist that functions as validation for batch/deployment risk. Not a 4 because validation checkpoints (checklist, error lookup) are explicit rather than implied. | 5 / 5 |
Progressive Disclosure | Section structure is excellent — every section ends with a well-signaled one-level-deep pointer ('**See**: [DATA_ACCESS.md](DATA_ACCESS.md) → ...') — but the actual bundle contains no referenced files at all: DATA_ACCESS.md, COMMON_PATTERNS.md, ERROR_PATTERNS.md, and BUILTIN_FUNCTIONS.md do not exist in the skill directory, so every link dangles and navigation to the detail content is impossible. This is a genuine organization defect, not the 'minor organization gaps' of anchor 4, but the in-body structure is too good for anchor 2's 'minimal structure'. | 3 / 5 |
Total | 17 / 20 Passed |