Content
96%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.
Excellent reference-style content: terse, verified, error-message-driven, and fully actionable, with a strong closing checklist that embeds validation into the workflow. The one real defect is progressive disclosure — the body's only external reference, COMMON_PATTERNS.md, is missing from the bundle, leaving a dangling link.
Suggestions
Include COMMON_PATTERNS.md in the skill bundle (e.g. under references/) or remove the Reference section, since the promised '12 import-free patterns' are currently unreachable and the link 404s.
If COMMON_PATTERNS.md is added, briefly restate its one-or-two highest-value patterns (or the safe-nested-access idiom) inline so the most common need survives a broken or skipped reference.
Consider moving the Related-skills and Reference sections adjacent to each other (or merging them) so all outbound pointers live in one navigable block.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Python competence: dense tables (inputs, migration, sandbox limits, return shapes, onError matrix) with no re-teaching of basic Python or generic n8n concepts, e.g. 'Everything else in plain Python works (verified): comprehensions, generators, lambdas...'. Version pins ('Observed on n8n 2.38.5') are provenance notes with an explicit re-check caveat rather than date-conditional instructions, so the time-sensitivity penalty does not apply; every token earns its place, matching the 5 anchor. | 5 / 5 |
Actionability | Copy-paste-ready code for both modes ('return [{"json": ...} for it in _items ...]', 'row = _item["json"]'), exact node config JSON, a legacy-to-native migration table with literal replacements, a one-line import probe ('import json + return [{"json": {"ok": True}}}]'), and exact fixes paired to exact error strings ('NameError: name 'type' is not defined' → 'isinstance(x, dict)'). Fully executable guidance covering the common cases, matching the 5 anchor. | 5 / 5 |
Workflow Clarity | The decision flow is clear and sequenced: JavaScript first (with an explicit fallback gatekeeper), mode/variable matching, dict-only access, no unconfirmed imports, correct return shape — culminating in a Checklist with an explicit validation checkpoint ('Ran a real test execution and inspected the output items (validation alone won't catch the traps above)') and feedback loops (the import probe node, 'test-run before relying on it' for pairedItem). This matches the 5 anchor (clear sequence, explicit validation, feedback loops, checklist) rather than the 4 anchor's minor validation gaps. | 5 / 5 |
Progressive Disclosure | Sections are well organized and the single reference is clearly signaled and one level deep ('**[COMMON_PATTERNS.md](COMMON_PATTERNS.md)**: 12 import-free patterns, each verified on a live n8n instance'), but that referenced file does not exist anywhere in the bundle — no references/ directory and no COMMON_PATTERNS.md — so the link is dangling and the promised detailed content is unavailable. This is above the 3 anchor (the reference is clearly signaled and no detail content is inlined) but the broken reference is more than the 5 anchor's 'easy navigation' allows. | 4 / 5 |
Total | 19 / 20 Passed |