Content
88%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 strong, thoroughly executable skill body: exact pinned commands, complete edge-case handling (refusals, missing sessions, output labels), and a confirmation gate before any mutating mark. The only weaknesses are minor — a few over-packed sentences that could be tightened, and a single-file layout whose label table could move to a reference file as the skill grows.
Suggestions
Conciseness: split the multi-clause passages — the IN RESULT table cell and the final bullet of section 4 ("A hand-made pass sees which files a call named, but not...") — into shorter standalone directives so each rule is scannable at a glance.
Progressive disclosure: consider moving the section 1 label-interpretation table into a reference file, keeping only the command, its options, and the routing rules in SKILL.md, so the main body stays an overview.
Conciseness: the opening block before section 1 mixes tool facts, default private-file patterns, and run-location rules in one dense paragraph; grouping them under short labels would aid scanning.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient — no explanations of concepts Claude already knows; every section is operational ("By default it treats these as private: .env*, *.env, .npmrc, secrets/, .ssh/, id_rsa*"). It is not a 5: a few passages pack several directives into single sentences — the IN RESULT table cell combines three instructions plus a caveat, and the final bullet of section 4 chains four clauses ("A hand-made pass sees which files a call named, but not a value printed by a search that named no file, which is the route that matters most...") — which could be split or trimmed without losing meaning. It is not a 3: there is no padding or over-explanation, only density that occasionally needs a second read. | 4 / 5 |
Actionability | Everything is copy-paste executable: `npx -y @agentwhy/cli@0.4.1 check`, `start --detach --quiet`, `check --mark rotated <path>` with `--note` and `--unmark`, `init`, options `--since 30d`, `--full`, `--share`, and even the exact permission strings ("Bash(npx -y @agentwhy/cli@0.4.1 check)") to add to settings. It is not a 4: the common cases are all covered, including interpretation of each output label, the no-sessions-found case, and the command-refused path — nothing needed to execute is missing. | 5 / 5 |
Workflow Clarity | The process is clearly routed by user intent (question → check, section 1; report or Codex question → page, section 2) with explicit error-recovery loops: no sessions stored → ask which folder the user worked in; command refused → section 3's two concrete remedies. Mutating operations carry a confirmation checkpoint ("Never mark a line the user has not confirmed"), and the tool's own non-destructive nature is stated ("writes nothing, and exits 0 whatever it found"). It is not a 4: validation checkpoints and failure feedback paths are explicit, not merely implied — the destructive-operation cap does not apply since the core flow is read-only. | 5 / 5 |
Progressive Disclosure | A single well-sectioned file (numbered sections 1–5 plus Limits) with external detail correctly deferred rather than inlined: detection gaps point to the project's own "docs/detection.md" and the report detail to agentwhy's page. It is not a 5: at ~110 lines, the full label-interpretation table and the rules sections sit inline in SKILL.md; while defensible at this size, the table in particular is the kind of material a reference file would absorb if the skill grows. It is not a 3: structure is clear, navigation is easy, and the one external reference is clearly signaled and one level deep. | 4 / 5 |
Total | 18 / 20 Passed |