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.
An exemplary instruction-only skill body: lean, fully executable, with a well-sequenced workflow that includes cache short-circuiting, per-project failure isolation, and a validate-and-retry checkpoint. The only weakness is reference resolution risk — the skill_dir find pattern ('repo-context') does not match the frontmatter name ('ghost-repo-context'), and no bundle files were provided to verify the referenced files exist.
Suggestions
Align the Setup discovery path with the skill's actual directory/frontmatter name (find '*/skills/ghost-repo-context/SKILL.md' or derive skill_dir from the loaded skill's own path) so detector.md, summarizer.md, and template-repo.md reliably resolve.
State explicitly what to do if <skill_dir>/detector.md or summarizer.md is missing (e.g., report an error) so the reference chain cannot fail silently.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~55-line body contains zero concept explanations and no padding: every section (Inputs, Tool Restrictions, Setup, cache check, Workflow) carries operational information only. It assumes Claude's competence throughout, e.g. the one-line cache_dir derivation and the terse step descriptions — every token earns its place. | 5 / 5 |
Actionability | Both bash blocks are copy-paste ready, inputs are enumerated as concrete key=value pairs, the exact output string ("Repository context is at: <cache_dir>/repo.md") is given, and every edge case is handled explicitly: cache hit short-circuit, no projects detected, per-project summarization failure, and one-retry on malformed output. Guidance is fully executable with no gaps. | 5 / 5 |
Workflow Clarity | Five clearly sequenced steps with a cache-first short-circuit, explicit data flow between steps (detection output feeding summarizer inputs), and a genuine feedback loop: step 4 reads repo.md back, verifies expected sections from the template, and retries the write once before reporting an error. Sequence, checkpoints, and error recovery are all explicit. | 5 / 5 |
Progressive Disclosure | The body is a clean overview with well-signaled, one-level-deep references to detector.md, summarizer.md, and template-repo.md, each tied to a specific workflow step. However, no bundle files exist to verify these references resolve, and the Setup find targets '*/skills/repo-context/SKILL.md' while the frontmatter name is 'ghost-repo-context' — a path mismatch that could break reference resolution, leaving a minor navigation gap below anchor 5. | 4 / 5 |
Total | 19 / 20 Passed |