Content
81%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 a lean, well-sequenced orchestration card with concrete dispatch code, explicit feedback loops, and a definition of done — strong on token efficiency and workflow clarity. Its critical defect is progressive disclosure: it defers nearly all operative detail to a rules/ directory that is absent from the bundle, so as shipped the skill's navigation chain is broken and one of its two bundle files is orphaned.
Suggestions
Ship the rules/ directory (config.md, intent.md, local-stack.md, auth.md, browser-verification.md, findings.md, fix-loop.md, report.md, lessons.md, pr-comment.md, setup.md, delegation.md) or inline the essential content of each into SKILL.md — as delivered, every run-card step links to a file that does not exist, breaking the skill's navigation entirely.
Reference scripts/verify-comment.mjs from the body (e.g., in the PR-comment step pointing to rules/pr-comment.md, which should invoke it) so the only executable bundle asset is discoverable and its role is clear.
In the run card, name which placeholders each rule resolves (e.g., '<env> comes from environments.yml, <driver> from rules/local-stack.md') so a reader can act even when a referenced file cannot be loaded.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational — modes, a flag table, a numbered run card, verbatim dispatch code, and a Definition of Done checklist — with no padding and no explanation of concepts Claude already knows. Lines like 'Start it detached, logs to procs/<name>.log, poll the ready URL' pack maximal instruction into minimal tokens, matching the lean/efficient anchor; nothing reads as filler that would drop it to 4. | 5 / 5 |
Actionability | The two-witness block gives copy-paste-ready Skill()/Agent() invocations with exact flags ('--env <env>', '--windows .agent/<branch>/verify/probes.jsonl', '--no-telemetry --no-comment'), and the run card specifies concrete artifacts (intent.md with 'AC-n' verbatim and tagged rows, probes.jsonl with 'window_start'/'window_end'/'log_offsets', findings.jsonl, procs/<name>.log). It stops short of fully executable because each step's operative detail is deferred to rules/*.md files and placeholders like <env> and <driver> are resolved by config rules not inlined, leaving minor gaps the body alone cannot close. | 4 / 5 |
Workflow Clarity | The run card is an explicitly ordered 0-6 sequence ('Do these in order') with validation at each turn ('No .agents/aw/environments.yml, or no entry for the environment → run setup first; never guess'), a feedback loop for error recovery ('After a fix, re-run the probe that found it; stop at ✅ verified or at --loops'), defect routing ('record it, then route around it without changing code'), an explicit verdict rule, and a Definition of Done checklist. This matches the anchor requiring clear sequence, explicit validation, feedback loops, and checklists; nothing about the batch/fix operations lacks verification. | 5 / 5 |
Progressive Disclosure | The body is structured as a good overview with well-signaled one-level-deep links (rules/config.md, rules/intent.md, rules/fix-loop.md, etc.), but scored against the actual bundle: 12 of the 13 referenced paths (the entire rules/ directory) do not exist — only references/design.md is present — and the shipped scripts/verify-comment.mjs is never referenced from the body. Every run-card step points to a missing file, so navigation is broken rather than merely imperfect, fitting the minimal/broken-structure anchor rather than 3 (where references exist but are poorly organized). | 2 / 5 |
Total | 16 / 20 Passed |