Content
92%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 tight, eight-step debugging procedure with explicit validation gates (reproduce-first, a regression test that must fail without the fix, real-consumer old-vs-new verification) and a well-designed failure escape hatch for repeated wrong fixes. Commands and criteria are mostly executable as written with only minor gaps, and the short, self-contained structure needs no bundle split.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Quotes: "Reproduce on current `main` with `--format json --quiet`. Save one command that shows the wrong result. Change no code before this command exists." — ~30 lines of pure operational directives with zero concept explanations. Even the build note ("A release build adds a slow LTO link and gives the same findings") is non-obvious project knowledge that earns its tokens; it is not 4 because there is no over-explanation left to trim. | 5 / 5 |
Actionability | Quotes: "`--format json --quiet`", "clear the `.fallow/` cache first", "`cargo build -p fallow-cli`", "a regression test that fails without the fix" — concrete flags, commands, and precise pass/fail criteria throughout. Not 5: the reproduce command is a fragment rather than a complete invocation, "Reduce to a minimal fixture" and step 8's "Run `review` with the affected surface reviewers" are directions without exact commands; not 3 because the guidance is largely executable as written. | 4 / 5 |
Workflow Clarity | Quotes: eight numbered steps with explicit validation gates ("Change no code before this command exists", "a regression test that fails without the fix", "compare old versus new output" on a real consumer) and a genuine error-recovery feedback loop ("After 3 failed fixes, stop and write down the premise that they share. Question that premise before a fourth fix."). Not 4: no validation gap is visible — reproduce-first, failing-test gate, and real-consumer comparison are all explicit. | 5 / 5 |
Progressive Disclosure | The skill is under 50 lines, self-contained, and well organized as a single numbered procedure with a heading; no bundle files (references/, scripts/, assets/) exist to split content into. The single external pointer ("test evidence in `docs/development/quality-gates.md`") is one level deep and clearly signaled, so the simple-skill exception applies and there are no organization gaps to justify a 4. | 5 / 5 |
Total | 19 / 20 Passed |