Content
85%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 exceptionally actionable, well-structured diagnosis skill: concrete commands, hard validation gates, clear mode triggers, and a clean one-level reference split — all referenced files verified present. The only real cost is length; a few dense rule passages (regression-guard edge cases, reporter-environment probes) could be tightened or pushed to a reference file.
Suggestions
Move the regression-guard edge-case detail (shell bracket-form failures, negative-assertion pairing) into references/failure-patterns.md and keep a two-line rule with the link in SKILL.md.
Tighten the reporter-environment probe paragraph in the Runtime Evidence Ladder into a short checklist (what to print, what to omit, what to ask back) — it currently reads as one 100+ word run-on sentence.
The authorization boundary is stated twice in near-full form (Outcome Contract 'Authorization' bullet and Hard Rules 'Fix the cause within the authorized scope'); consolidate into one canonical statement with the other cross-referencing it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with original project-specific rules and explains nothing Claude already knows (no 'what is git bisect' padding), but several passages are over-long and convoluted — the ~200-word regression-guard sub-rules covering shell bracket forms and negative assertions, and the reporter-environment probe paragraph. This is 'mostly efficient but could be tightened' rather than the minor-trimming of anchor 4. | 3 / 5 |
Actionability | Guidance is fully executable for an instruction-only skill: concrete commands ("git status --short --branch -uall", "git diff <last-good>..HEAD -- <suspect path>", "grep -rn"), a fill-in hypothesis template ("I believe the root cause is [X] because [evidence]"), an exact output block, and a worked example of a testable hypothesis ("Stale cache in useUser at src/hooks/user.ts:42 because the dependency array is missing userId"). | 5 / 5 |
Workflow Clarity | The diagnostic workflow is clearly sequenced with explicit validation checkpoints and feedback loops: a numbered Runtime Evidence Ladder, hard-stop gates ("Same symptom after a fix is a hard stop"), confirm-or-discard with probe-driven re-orientation, and a red-green regression verification procedure. Bisect mode even sequences worktree safety checks before any bisect command. | 5 / 5 |
Progressive Disclosure | The main file is an overview with mode-dispatch, and five specialized playbooks (durable-context, failure-patterns, ime-unicode, logging-techniques, rendering-debug) are one level deep, each referenced with an explicit activation trigger ("load references/logging-techniques.md before changing code"). All five referenced paths exist in references/, and nothing is nested deeper. | 5 / 5 |
Total | 18 / 20 Passed |