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 a tightly written, highly actionable runbook: concrete git/gh commands, ordered steps, explicit stop conditions and blocker rules, and a clean single external reference to pr-style. Its few imperfections are the broken default-branch fallback in Step 1, a duplicated `gh pr edit` snippet, and a vestigial Command Template heading — none of which materially impede execution.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean and command-driven — it assumes competence with git/gh and never explains background concepts — with only minor trimmable fat: the duplicate `gh pr edit --body "<generated description>"` snippet immediately followed by the HEREDOC version, and the vestigial "## Command Template" heading block. This matches 'efficient; minor instances of over-explanation that could be trimmed' rather than the every-token-earns-its-place level 5. | 4 / 5 |
Actionability | Nearly everything is copy-paste-ready: `gh pr view --json number,url,title,body --jq ...`, `git log --oneline ${default_branch}..HEAD`, `git diff --stat`, and the HEREDOC `gh pr edit --body` pattern, plus first-match-wins template selection rules and an exact report format. Minor gaps keep it from 5: the `git symbolic-ref ... | sed ... || echo "main"` fallback is broken (sed succeeds on empty input, so `|| echo "main"` never fires and default_branch ends up empty), and the body itself is a placeholder. | 4 / 5 |
Workflow Clarity | Steps 0–6 are clearly sequenced with explicit checkpoints: "If no PR exists for the current branch, stop and report", unfilled template sections are "a blocker — report it rather than posting an unfilled template", and ambiguity triggers "ask the user which template applies". It falls short of anchor 5 only because there is no post-update verification or preview/feedback loop before overwriting the existing PR body. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), and the ~107-line body is well organized into numbered sections with a single clearly signaled one-level-deep external reference ("Read ~/.agents/skills/pr-style/SKILL.md"). Minor gaps — the leftover nested "# Update PR Description" heading under "## Command Template" and no true overview section — place it at 'good structure; minor organization gaps' rather than the fully clean split of anchor 5. | 4 / 5 |
Total | 16 / 20 Passed |