Content
88%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 tight, high-judgment process skill: it sequences the work clearly, builds in validation and error-recovery around a destructive operation, and adds opinions Claude would not generate on its own. Its only weaknesses are the absence of example commands for the PR-inspection phase and the lack of any referenced material, both minor. Strong overall — no suggestions needed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — it never explains what a rebase is or how git works, and directives like "Do not describe a PR as current merely because Git reports no conflicts" and "Age alone is not evidence, and sunk effort is not a reason to keep a PR alive" each earn their tokens. Every section adds judgment guidance Claude would not reliably produce on its own, matching anchor 5 ("Lean and efficient; assumes Claude's competence; every token earns its place"). | 5 / 5 |
Actionability | The guidance is mostly executable for an instruction-only skill: concrete commands ("`--force-with-lease` against the head SHA observed before the rebase", "`git range-diff`"), a precise six-step rebase procedure, and three crisply defined dispositions (MODERNIZE/REWORK/CLOSE). It falls short of anchor 5 only because the PR-intake phase gives no example commands for fetching or inspecting the PR (e.g., gh pr view or fetching the PR head ref), leaving minor gaps. | 4 / 5 |
Workflow Clarity | The process is explicitly sequenced — establish intent, check what changed around it, choose a disposition, then a numbered rebase workflow — with validation checkpoints (range-diff comparison, focused checks, distinguishing real from pre-existing failures) and error-recovery loops ("If the lease fails, fetch and reassess rather than overwriting new work"; a conflict revealing an architecture disagreement means "stop and reconsider the disposition"). The destructive force-push operation is fully guarded, matching anchor 5. | 5 / 5 |
Progressive Disclosure | The single ~95-line file is well-organized into scannable sections (Guardrails, Establish intent, Check what changed, Disposition, Rebase, Report) with nothing inlined that clearly belongs in a separate file, and the cross-reference to the `leaving-pr-comments` skill is clearly signaled. It scores 4 rather than 5 because there are no bundle files or referenced deep-dive material at all, and the body exceeds the under-50-line simple-skill case the rubric exempts for a top score. | 4 / 5 |
Total | 18 / 20 Passed |