Content
93%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 exemplarily lean skill body: executable scaffold command, concrete fill and acceptance criteria, and clean one-level-deep progressive disclosure into real reference files. The only gap is that the failure/retry loop for the findings-upsert acceptance check is delegated to the reference rather than stated explicitly.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~30-line body is dense and lean: every line directs action ("Replace every `TODO(agent)` marker", "Keep slugs stable across runs", "Plans go to `<plansDir>/<run>/<slug>.md`") with zero padding and no explanation of concepts Claude already knows. Clearly matches the anchor-5 example of every token earning its place. | 5 / 5 |
Actionability | Provides a copy-paste-ready scaffold command with enumerated flags and area values (`pnpm exec tsx .agents/skills/refactor-plan/scripts/new-plan.ts --run <id> --slug <kebab> ... --area <Framework|Slides|...>`), the exact output location, concrete bug-plan variant behavior, and an explicit done criterion (findings-upsert.ts acceptance). Delegates detail to real, existing reference files — fully executable guidance matching anchor 5. | 5 / 5 |
Workflow Clarity | Clear sequence across sections — judge first (rubric.md, read diffs, find a named repeated mechanism), scaffold (command), fill (replace markers per plan-guide.md), with an explicit end-state validation ("A plan is finished when fragility-common's `findings-upsert.ts` accepts it"). The recovery loop when findings-upsert.ts rejects a plan (fix markers/confidence, re-run) is only implied via the reference rather than stated in the body, which is the minor checkpoint gap of anchor 4 rather than the explicit feedback loop of anchor 5. | 4 / 5 |
Progressive Disclosure | The body is a concise overview with two well-signaled, one-level-deep references — "`references/rubric.md` separates fragile from fast-moving from coupled, and sets confidence" and "following `references/plan-guide.md`" — both of which exist in the bundle and carry the detail, while executable logic lives in `scripts/new-plan.ts` (with `lib.ts`). No nested doc chains, content appropriately split; matches anchor 5. | 5 / 5 |
Total | 19 / 20 Passed |