Content
82%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 content is a well-organized, actionable gatekeeper spec with strong workflow sequencing and explicit validation checkpoints. Its main weakness is mild verbosity in the inlined policy/guardrail sections and some detail that could live in the referenced workflow files.
Suggestions
Tighten the 'No fabricated policies' and approval-guardrail paragraphs — the enumerated Stage 0/1-pre/1c sub-cases read as policy detail that already lives in references/pr-workflow.md and could be condensed to a pointer.
Move the detailed escalation/criteria enumeration out of SKILL.md into references/pr-workflow.md, keeping only the one-line rule plus a 'see references' pointer to reduce token cost.
Consider a short 'Stages at a glance' table (Stage → comment → decision) at the top of the Workflow section so the sequence is scannable before diving into pre-flight detail.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence (terse command snippets, no concept explanation), but some rule sections are quite verbose and could be tightened, e.g. the long 'No fabricated policies' and approval-guardrail paragraphs. | 4 / 5 |
Actionability | Provides concrete executable commands (`gh issue view ... --json ...`, `enter_worktree(name: "triage")`) and specific guardrails, though several key details (stage templates, escalation criteria) are deferred to references rather than being inline. | 4 / 5 |
Workflow Clarity | The multi-stage workflow is clearly sequenced with explicit pre-flight checkpoints, a duplicate-guard, and validation/feedback guidance ('Validate immediately', 'not verified: <reason>'); destructive/batch operations (auto-approval, CHANGES_REQUESTED) carry explicit guardrails and error-recovery loops. | 5 / 5 |
Progressive Disclosure | Structure is good: SKILL.md is an overview with clearly signaled one-level-deep references (`references/issue-workflow.md`, `references/pr-workflow.md`) that exist as real files. The body inlines a fair amount of detailed rule/policy content that arguably belongs in the referenced workflow files, which is a minor organization gap. | 4 / 5 |
Total | 17 / 20 Passed |