Content
75%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.
The body is an operationally dense, command-rich orchestration guide with genuine feedback loops and error-recovery steps, and it correctly delegates the rules to chips.md. Its weaknesses are uniform rather than severe: duplicated checks, row-logging syntax left implicit, checkpoints buried in very long paragraphs, and auxiliary tool references hidden mid-step.
Suggestions
Show the exact `append` command and row format for at least the most common ledger rows (`started`, `landed`, `carded`, `updated`) the way Offer step 3 already does for a new chip, so the core logging operations are copy-paste ready instead of delegated to chips.md.
Split Sweep step 5 into separate numbered sub-steps (dependency wait, effort check, card offer, active-plan check) so its five interleaved checks and validation checkpoints are individually visible rather than buried in one paragraph.
Remove the duplication between Sweep steps 3 and 5 — state the dependency/effort checks once and reference them from the other step — and collect the auxiliary tool pointers (status.mjs, scan.mjs, .agents/pstack.json pageTopic.field) into a short named list near the top instead of scattering them mid-step.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no padding or explanation of concepts Claude already knows — every sentence is operational — which is above the level-3 'could be tightened' anchor. It stops short of level 5 because of real duplication: the dependency and effort checks appear in both Sweep step 3 ("judge its dependencies as step 5 does", "check its effort ... as step 5 does") and again in full in step 5, and the "A runtime with no session tool" caveat recurs three times. | 4 / 5 |
Actionability | Mostly executable: concrete commands appear throughout ("node .agents/pstack/decisions-check.mjs <command>", "chips cleanup --sessions <file> --session <id>", the near-verbatim append TSV invocation in Offer step 3). It misses level 5 because the core row-logging operations ("log `landed` with `patch:`", "log `updated` with its waits") never show the exact command or row syntax — that syntax is delegated to chips.md — and tools like "the dismiss tool" and "the suggest tool" are named generically. | 4 / 5 |
Workflow Clarity | The 7-step Sweep and the numbered Offer/Start/Drop sequences include explicit feedback loops and error recovery ("Run `chips --session <id>` again and repeat until the Do list shows only the card pass", "When `append` refuses a batch, fix or drop the refused row ... and append the rest", "once the cleanup scan's `--landed` check passes"), so the destructive/batch cap of 3 does not apply. It falls below level 5 because checkpoints are buried inside dense paragraphs — step 5 alone interleaves five distinct checks (dependencies, effort, carding, the 20-card limit, active-plan writes) in a single block, and many checkpoints are delegated to chips.md sections rather than stated inline. | 4 / 5 |
Progressive Disclosure | The bundle is a single SKILL.md (no references/, scripts/ or assets/ exist), well organized with a mode table linking to Sweep/Offer/Start/Drop sections, and one clearly signaled one-level external reference ("The law is `.agents/pstack/rules/chips.md` ... Read it in full before this session's first ledger write"). It misses level 5 because the body inlines a large amount of orchestration detail that a reader must hold at once, and secondary tool references ("status.mjs", "scan.mjs", "pageTopic.field in `.agents/pstack.json`") are buried mid-paragraph in Sweep step 5 rather than surfaced as a navigation aid. | 4 / 5 |
Total | 16 / 20 Passed |