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.
The body is an exemplary orchestration document: lean, imperative, with concrete commands, explicit validation with revert-on-failure loops, and clearly hard safety rules. Its one real defect is bundle-level: the two files it designates as the simplify engine (rules/simplify.md, rules/quick-pass.md) are absent, leaving the most load-bearing references dangling and Pass B not fully executable from the shipped skill.
Suggestions
Ship the missing rules/simplify.md and rules/quick-pass.md in the bundle (or inline the item list and quick-pass subset), since the body references them five times as the skill's core engine and they are currently unreachable paths.
Define or point to where "the project's fast check" is specified, so the apply-then-revert-on-red loop is executable from SKILL.md alone rather than depending on the absent rules file.
Inline one concrete example of a mechanical simplify item (e.g., a restating-comment before/after) so Pass B retains some actionable guidance even when the referenced engine file cannot be loaded.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and imperative throughout — mode tables, exact commands, and a report template with no concept explanations (nothing teaches what git or a PR is). Even the one apparent explanation ("pr-reviewer is an agent, and Skill('pr-reviewer') errors with 'Unknown skill'") is a non-obvious operational gotcha, not padding. The only near-duplication ("Never stash, reset, or discard the user's uncommitted work" in Step 1 and Hard rules) is deliberate safety restatement, so the content matches 'every token earns its place'. | 5 / 5 |
Actionability | Mostly copy-paste-ready: `git rev-parse --abbrev-ref HEAD`, `PR_URL=$(gh pr view --json url -q .url)`, the exact `Task(subagent_type="pr-reviewer", ...)` form, and ready commit commands. It is not fully executable though: Pass B's actual behavior ("Read and follow rules/simplify.md with items 1–9" and "the project's fast check") is delegated to files that are not present in the bundle, so the core simplify mechanics cannot be executed from what ships here — a gap beyond the anchor-5 bar of 'specific examples cover the common cases'. | 4 / 5 |
Workflow Clarity | Steps 0–4 are clearly sequenced with explicit checkpoints and feedback loops: dirty-tree warning before committing, empty-diff and non-code-only exits, per-change validation with rollback ("each behind the project's fast check, reverting any change that turns it red"), "skip a pass with no edits", and a structured report with a Reverted list. This satisfies the batch-operation validation requirement (score would cap at 3 if missing), matching the anchor's explicit validation and error-recovery loop. | 5 / 5 |
Progressive Disclosure | The in-body structure and signaling are good — a one-line purpose, a contents index, and clearly signaled one-level links ("[rules/simplify.md](./rules/simplify.md)", "[references/callers.md](./references/callers.md)", each with a stated purpose). However, scored against the actual bundle: rules/simplify.md and rules/quick-pass.md are referenced repeatedly as the skill's engine, yet only references/callers.md exists — the two core referenced paths are dangling, so the split content is not actually reachable. That is more than the 'minor organization gaps' of anchor 4; it fits anchor 3's 'references present but' structurally unresolved. | 3 / 5 |
Total | 17 / 20 Passed |