Content
78%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 lean, well-structured skill with genuinely executable search commands and a concrete output template. The main defects are the dangling REFERENCE.md pointer (the bundle contains no reference files), the lack of an error-recovery loop at the validation checkpoint, and example-specific commands without genericized placeholders.
Suggestions
Ship the referenced REFERENCE.md (or remove the pointer and inline the minimal template details) — the link currently resolves to nothing, breaking progressive disclosure entirely.
Add a short feedback loop at the validation checkpoint: what to do when a listed file fails typecheck or a search hit doesn't fit the map (e.g., re-trace that subtree or correct the entry-point list).
Generalize the rg commands with explicit placeholder symbols (e.g., 'rg -n "import .*from .*<lib-path>"' with a note to substitute the entry-point export) so the commands model the pattern, not just one example.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~70-line body is lean and assumes Claude's competence: no concept explanations, just operational steps, concrete rg commands, an output template, and terse anti-patterns. Nothing beyond a formatting nit (an empty 'Skip' table cell) is trimmable. | 5 / 5 |
Actionability | Concrete, copy-paste-able rg commands for both trace directions ('rg -n "import .*from .*libs/auth"', 'rg -n "from .*libs/" src/auth/') plus a full markdown output template. Not a 5 because the commands hardcode example symbols (libs/auth, AuthCard, priceRange) that the reader must substitute, with no generic placeholder pattern shown. | 4 / 5 |
Workflow Clarity | Clear 4-step sequence (entry points → trace outward → trace inward → build map) with an explicit validation checkpoint ('every search hit lands in the map; all listed files open without errors'). Not a 5 because there is no error-recovery loop and the typecheck is delegated vaguely to a 'codebase-tool slot' rather than a runnable command; not a 3 because the checkpoint is explicit, not implicit. | 4 / 5 |
Progressive Disclosure | The body correctly keeps detail out of SKILL.md and clearly signals a one-level-deep pointer ('Full template, depth levels, partition derivation, Team Lead integration: see [REFERENCE.md](./REFERENCE.md)'), but that file does not exist in the bundle (no references/ directory at all), so the disclosure promise fails at load time. Scoring against the actual bundle structure per the rubric guideline, this lands at 'some structure but could be better organized' rather than a working split. | 3 / 5 |
Total | 16 / 20 Passed |