Content
70%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 delivers strong, actionable domain content with a well-sequenced validate-fix-revalidate loop and explicit recovery branches. Its main weaknesses are repetition (telemetry stats stated three times, error types listed twice, a redundant Summary), broken references to nonexistent detail files, and catalog-level content that belongs in those referenced files.
Suggestions
Create the referenced ERROR_CATALOG.md and FALSE_POSITIVES.md files (or remove the 'Detailed Guides' section) — the links currently point to files that do not exist in the bundle.
Cut the repeated telemetry statistics ("7,841 occurrences", "23s thinking / 58s fixing") and the closing Summary section, which restates earlier sections almost verbatim.
Merge the 'Error Severity Levels' type list with the 'Common Error Types' section so each error type appears once with its fix, and show one concrete `get_node` call since it is repeatedly prescribed as the first step.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is dense, genuinely non-obvious domain knowledge (error types, validation profiles, auto-sanitization rules, false-positive guidance), but there is noticeable padding: the telemetry stats ("7,841 occurrences", "23 seconds thinking", "58 seconds fixing") are repeated three times, the five error types are listed in full in both 'Error Severity Levels' and 'Common Error Types', and the closing Summary restates prior sections. This fits anchor 3 ('mostly efficient but includes some unnecessary explanation or could be tightened') rather than anchor 2 because most content is domain-specific reference material, not concepts Claude already knows. | 3 / 5 |
Actionability | Each common error type pairs a concrete error payload with a copy-paste fix (e.g. `config.channel = "#general"`, `config.limit = 100`, `config.text = "={{$json.name}}"`), and recovery strategies show exact tool invocations like `n8n_update_partial_workflow({ id, operations: [{ type: "cleanStaleConnections" }] })`. It falls short of anchor 5 only through minor gaps: `get_node` is cited four times as a key step but never shown, and the referenced ERROR_CATALOG.md / FALSE_POSITIVES.md files do not exist in the bundle. | 4 / 5 |
Workflow Clarity | The core multi-step process is explicitly sequenced with a validation feedback loop ("Validate → Read errors → Fix → Validate again; Repeat until valid"), recovery strategies branch by failure mode, and the auto-fix flow embeds a verification checkpoint ("applyFixes: false // Preview first", then apply). This matches anchor 5 ('Clear sequence with explicit validation steps; feedback loops for error recovery; checklists'), reinforced by the Do/Don't checklist. | 5 / 5 |
Progressive Disclosure | Section headers are clear and consistent throughout, but the 'Detailed Guides' section points to ERROR_CATALOG.md and FALSE_POSITIVES.md, which do not exist in any bundle directory (no references/, scripts/, or assets/), and ~690 lines of catalog-level detail (error types, false-positive cases) are inlined in SKILL.md rather than split into those files. This fits anchor 3 ('content that should be separate is inline'); it is not anchor 2 because the internal structure is extensive and easy to navigate, not minimal. | 3 / 5 |
Total | 15 / 20 Passed |