Content
72%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 lean, token-efficient policy overview with strong warrant criteria, binding-semantics rules, and clean workflow dispatch. Weaknesses concentrate in execution wiring: the `check` validation command is orphaned from the workflows, no example CLI run is shown, and most referenced detail files are missing from the bundle.
Suggestions
Wire `check` into the workflows as an explicit validation checkpoint, e.g., "after any INDEX or record edit, run `decisions check`; fix and re-run until clean."
Ship the referenced `workflows/`, `schemas/`, and `templates/` files alongside SKILL.md (or inline the minimal creation steps) so every referenced path resolves within the bundle.
Add one concrete example invocation with its output shape (e.g., `decisions search "auth"`) so CLI usage is copy-paste ready without consulting --help first.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~50 lines of dense policy with zero padding: "A record exists to stop a reversal: a reviewer or a future agent, reading the code alone, would undo the choice because the code cannot show its reason" carries a full warrant rule in one sentence, and CLI detail is deferred to "decisions --help". Nothing explains concepts Claude already knows, and every token earns its place. | 5 / 5 |
Actionability | Concrete guidance throughout: the CLI invocation ".agents/skills/decider/scripts/decisions <command> [options]", exact file pointers ("workflows/create-decision.md", "schemas/decision-format.md", "templates/decision-entry.md"), a warning against a nonexistent command ("There is no bare `issue` action; use `search --issue`"), and a literal approval sentence to say. It falls short of 5 because no example command run or output shape is shown inline — usage is delegated entirely to --help and the referenced files. | 4 / 5 |
Workflow Clarity | The Workflows table gives clean trigger-to-file dispatch, the reading protocol is ordered ("Read a hit's INDEX row first", "Read an active or superseded record's document before treating it as binding"), and the approval gate is explicit ("Never create a decision document without explicit user approval"). But the `check` action is listed among the CLI's actions yet never sequenced into any flow, and supersession/removal — operations that edit the INDEX and existing records — have no inline validation loop; per the guideline capping workflows with such operations at 3, this cannot score 4. | 3 / 5 |
Progressive Disclosure | SKILL.md is a proper overview with clearly signaled, one-level-deep references (the Workflows table, the "Format:" line, `decisions --help`), and the CLI script does exist in the bundle (scripts/decisions). However, the referenced `workflows/create-decision.md`, `workflows/update-decision.md`, `schemas/decision-format.md`, `templates/decision-entry.md`, and `templates/index-row.md` are not present in the actual bundle, so navigation to the core procedural detail cannot be completed or verified — more than the minor organization gap of a 4. | 3 / 5 |
Total | 15 / 20 Passed |