Content
63%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 a highly actionable, deeply validated build procedure with excellent executable examples, but it is roughly twice the length it needs to be due to cross-section duplication, and it inlines substantial reference-grade material while shipping unsubstituted template placeholders at a mandatory decision point. Strong on execution guidance, weak on token efficiency and file organization.
Suggestions
Consolidate the Gateway credits rules (Escalation, Credential Rules, and Gateway credits Preference), the placeholder rules (Placeholders, SDK Code Rules, Workflow Rules 1), and the inserted-node payload semantics (Repair Strategy vs Workflow Rules 7) into single canonical sections, with other mentions reduced to a one-line pointer — this alone would cut a substantial fraction of the duplication.
Move the Expression Reference, SDK Patterns Reference, Gateway credits Enumeration, and data-table diagnostic heuristics into dedicated files under references/ (like the existing error-workflows.md and compositional-workflows.md pattern) so SKILL.md reads as the build process plus pointers.
Resolve or remove the raw template tokens {{GROUPING_GUIDANCE_PLACEHOLDER}} and {{TOP_LEVEL_ITEM_CEILING_PLACEHOLDER}} in the Node Groups and Completion sections — a mandatory build decision (step 6) currently points at criteria that are not present in the file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | At 1047 lines / ~8,100 words the body is dense and non-generic (no explanations of concepts Claude already knows), but it duplicates itself: routing rules appear twice ("Do not load planning or create-tasks first" in Routing and again in the same section), Gateway credits rules spread across Escalation, Credential Rules, and a dedicated Gateway credits Preference section, placeholder rules appear in three places, and inserted-node payload semantics are stated in Repair Strategy ("an inserted write/create node replaces the payload flowing into the next node") and again verbatim as Workflow Rules 7. Mostly efficient content that clearly could be tightened — the 3 anchor — rather than padded (not 2) or with only minor trim (not 4). | 3 / 5 |
Actionability | Guidance is nearly fully executable: the exact validate command ("node --import tsx node_modules/@n8n/workflow-sdk/dist/cli/index.js validate <filePath>"), complete TypeScript patterns with correct-vs-wrong wiring examples, exact tool-call signatures, and precise placeholder/locator shapes. It stops short of 5 because "{{GROUPING_GUIDANCE_PLACEHOLDER}}" and "{{TOP_LEVEL_ITEM_CEILING_PLACEHOLDER}}" are unsubstituted template tokens — the grouping criteria and ceiling value the Mandatory Process and Completion sections depend on are literally absent. | 4 / 5 |
Workflow Clarity | The 12-step Mandatory Process has explicit validation (step 7 validate, step 8 build), feedback loops (step 10: edit the same file, re-validate, rebuild), a repair budget ("one repair attempt per unique failure signature"), and a Verification Contract with explicit stop conditions — the destructive/batch validation cap does not apply since validation is present. It misses 5 because the mandatory grouping decision checkpoint (step 6 "See [Node Groups](#node-groups) for the criteria") resolves to a placeholder token with no criteria, and the process is scattered across five interleaved sections (Mandatory Process, Repair Strategy, Escalation, Verification Contract, Completion) that restate one another. | 4 / 5 |
Progressive Disclosure | References are clearly signaled by name and all resolve to real bundle files ("load the `error-workflows` reference", "load the `compositional-workflows` reference", "follow `model-selection`", "load the `credential-recipe-research` skill" — each exists in references/), and the 41KB post-build flow is properly externalized. But the body itself is a 1000-line monolith carrying Expression Reference, SDK Patterns Reference, Gateway credits enumeration, and data-table query heuristics that belong in reference files, and reference chains run two levels deep (SKILL.md → post-build-flow.md → trigger-input-data-shapes.md / config-evals.md). Some structure, but content that should be separate is inline — the 3 anchor. | 3 / 5 |
Total | 14 / 20 Passed |