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.
A well-structured, actionable skill body with clear phased workflows and real bundled scripts, but it is let down by repeated slug/path rules that bloat the token budget and by references to a templates/ directory that is not actually shipped with the skill.
Suggestions
Consolidate the slug-derivation rules into one canonical section and reference it elsewhere (e.g., 'see Slug derivation') instead of restating defaults, shorter-form, and collision rules three or four times.
Ship the referenced templates/ directory (README.template.md, workflow-entry.template.md, architecture.template.md, ai-orientation.template.md, focus-workflows.template.md, focus-architecture.template.md, workflows.template.md) or remove the template references and inline the minimal scaffolds needed.
Add an explicit validate→fix→retry feedback loop for the overwrite-prone regeneration step (e.g., 'regenerate, run git diff generated-docs/, review and re-run affected scripts if output is stale') to push workflow clarity to the top anchor.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense and does not explain concepts Claude already knows, but the slug-derivation rules are restated three to four times (Invocation, Output, 'Slug derivation', and Quality Rules) and the 'no README/ai-orientation in the subfolder' and '../ path' rules are repeated, which could be tightened into a single canonical statement. | 3 / 5 |
Actionability | Provides executable commands (the scripts table plus per-phase 'node skills/document-this/scripts/*.mjs' invocations) and a clear file-output map; minor gaps remain in that template placeholder filling is referenced rather than shown with concrete examples. | 4 / 5 |
Workflow Clarity | Phases 1–5 and F1–F4 are clearly sequenced with explicit checkpoints (empty-scope halt, 'refuse to invent the feature', ripple check, 'when scripts disagree with disk, trust disk'), though it stops short of a formal validate→fix→retry feedback loop and the 'git diff' review is only suggested. | 4 / 5 |
Progressive Disclosure | Section structure is good and the seven scripts are properly externalized into a real scripts/ bundle, but the body references seven template files under templates/ that do not exist in the bundle — broken references that would dead-end an agent following the skill. | 3 / 5 |
Total | 14 / 20 Passed |