Content
56%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 dense, domain-specific rulebook with strong validation discipline (known-bad controls, noise rules, rerun contracts) but written in long, repetitive prose that buries its executable commands. The bundle exists, yet the body's script references resolve outside it and one reference file is orphaned, so navigation does not exploit the bundle layout.
Suggestions
Tighten the prose: remove the duplicated 'before creating a plan, running a benchmark or changing runtime code' read-directives and the restated defer/gate rule, and split the multi-clause noise/planted-arm rules into a short table or bullet list.
Point script invocations at the bundle's own files (e.g. scripts/paired-timing.mjs instead of .agents/rules/benchmark/scripts/paired-timing.mjs) and reference references/autoresearch.md from the body so every bundle file is discoverable.
Move the ~100-line Pre-Acceptance Architecture Probe law into a references/ file, keeping a short step summary inline, so the probe's executable commands (proof-worktree.mjs, paired-timing.mjs, build-app-if-stale.mjs) stand out in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Almost every line carries project-specific rules rather than concepts Claude already knows, but the prose could be tightened substantially: the instruction to read the playbook "before creating a plan, running a benchmark or changing runtime code" appears twice nearly verbatim (lines 19-20 and 26-27), the defer/gate rule is restated in step 5 and again in the following paragraph, and multi-clause run-ons like the cohort-noise rule ("A cohort's noise is the larger of its same-code gap and the furthest any pair ... because a same-code arm divides by the same base runs ...") demand several re-reads. Matches mostly-efficient-but-could-be-tightened, not noticeably padded with unnecessary explanations (2). | 3 / 5 |
Actionability | There are genuinely concrete commands scattered throughout ("node tooling/scripts/proof-worktree.mjs", "node .agents/rules/benchmark/scripts/paired-timing.mjs", "node tooling/scripts/review-ledger.mjs lookup <scope>", "pnpm bench:editor:install", "--reporter=verbose --silent=false"), but the central Pre-Acceptance Probe workflow is dense policy law ("Reuse an honest existing budget when one exists", "Size each line relative to the baseline and above the timer resolution") whose executable mechanics are delegated to a playbook and scripts outside this bundle. Some concrete guidance, incomplete within the body — matching anchor 3 rather than mostly-executable anchor 4. | 3 / 5 |
Workflow Clarity | The probe is a numbered 5-step sequence with abundant validation checkpoints and feedback loops: a planted-slowdown control the budget line must reject, a known-bad input that "must fail with the probe's own assertion text", inconclusive results branching to defer/gate, and conclusive misses sending the target "back to `architect` at once". Not a 5 because the full end-to-end workflow is interleaved across external cross-references (the perf-issue playbook, `pstack:benchmark-checklist`, `verify`'s command recipes) that a reader must assemble themselves, leaving minor sequence gaps in the body alone. | 4 / 5 |
Progressive Disclosure | The two key references are clearly signaled with when-to-read guidance ("For `review <scope-or-plan>`, read the full [performance review method](./references/performance-review.md)"; "Read [methodology.md](./references/methodology.md) completely before creating a plan..."), but the structure has real gaps against the actual bundle: `references/autoresearch.md` is never mentioned, the ~100-line Pre-Acceptance Probe law is inlined where a reference would fit, and script paths point outside the bundle (`.agents/rules/benchmark/scripts/paired-timing.mjs`, `.agents/rules/benchmark/scripts/benchmark-contract.test.mjs`) instead of the colocated `scripts/` files. Between anchor 3 and anchor 4; the orphaned file, path mismatches, and heavy inline section pull it to 3. | 3 / 5 |
Total | 13 / 20 Passed |