Content
56%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 body is highly actionable with concrete, contrasted code examples and a clear decision workflow, but it is noticeably verbose — its three critical rules are each repeated three times and it teaches basic Python Claude already knows — and its four advertised reference files are missing from the bundle, breaking progressive disclosure.
Suggestions
Eliminate the triple duplication: state the webhook ["body"] nesting, the return format, and the no-external-libraries rule once each in their own sections and have the Essential Rules and Error Prevention sections point to them instead of restating them.
Ship the four referenced files (DATA_ACCESS.md, COMMON_PATTERNS.md, ERROR_PATTERNS.md, STANDARD_LIBRARY.md) in the bundle or remove the dangling 'See:' references and the 'Related Files' section.
Remove the 'Best Practices' section's basic Python instruction (.get(), list comprehensions, None handling) and keep only n8n-specific guidance, trimming the body to the n8n-specific rules, mode guide, and a few representative patterns.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body duplicates its three critical rules three times each (webhook ["body"] nesting in Essential Rules, its own section, and Error #5; return format in three places; the no-external-libraries limitation in three places; the standard-library listing in three places), and the 'Best Practices' section teaches basic Python Claude already knows (.get(), list comprehensions, None handling). This is several padded/duplicated sections, matching anchor 2 rather than the incidental over-explanation of anchor 3. | 2 / 5 |
Actionability | Mostly copy-paste-ready code with correct/incorrect contrasts covering the common cases (transformation, filtering, aggregation, validation, statistics), but a few examples are not executable as written — Pattern 2 calls an undefined process_data(data) and Quick Start/Pattern 2 use datetime.now() without the required import shown — giving the minor gaps of anchor 4 rather than anchor 5. | 4 / 5 |
Workflow Clarity | A clear decision sequence (consider JavaScript first → select mode → data-access pattern → return format) capped by an explicit pre-deploy 'Quick Reference Checklist' and an error-prevention section for recovery, matching anchor 4's 'most checkpoints present with minor validation gaps'; it is not a fragile destructive/batch workflow requiring validate-fix-retry loops. | 4 / 5 |
Progressive Disclosure | Structure is good and references are clearly signaled ('See: DATA_ACCESS.md', 'Related Files'), but none of the four referenced files (DATA_ACCESS.md, COMMON_PATTERNS.md, ERROR_PATTERNS.md, STANDARD_LIBRARY.md) exist in the bundle, so navigation is broken; additionally, the inlined Standard Library Reference and Common Patterns sections duplicate content that the skill claims lives in those separate files, fitting anchor 3. | 3 / 5 |
Total | 13 / 20 Passed |