Content
80%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.
A lean, well-structured instruction-only skill whose reporting format and triage rules are fully actionable. Its one real weakness is workflow clarity: the multi-merge scan sequence is implicit and no validation step confirms findings before they are reported or acted on.
Suggestions
Add an explicit ordered workflow (1. list merges from the last 7 days, 2. scan each diff for the listed items, 3. triage per the rules, 4. emit the output template).
Include a validation checkpoint such as "confirm each TODO/FIXME was introduced by this merge's diff, not pre-existing" before reporting, since this is a batch operation over many merges.
Show one concrete retrieval command (e.g., `git log --merges --since="7 days ago" origin/main`) to close the actionability gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line is a directive or concrete enumeration — "Only scan merges from the last 7 days", "Large refactors → ticket, not auto-fix", "Medium+ risk paths → escalate-human" — with no explanation of concepts Claude already knows; the untrusted-input block states loop-specific behavioral rules rather than generic prompt-injection theory. | 5 / 5 |
Actionability | Concrete, actionable guidance throughout: a copy-paste output template ("### PR #N — title (merged DATE)" with risk/effort/action enums) and explicit triage rules; the gap is that no example command shows how to retrieve recent merges (e.g., `git log --merges --since=7.days` or `gh pr list --merged`), leaving a minor execution gap. | 4 / 5 |
Workflow Clarity | The sequence (retrieve merges → scan each → triage → report) is implicit in the section layout but never explicitly ordered, and this batch operation over multiple merges lacks an explicit validation checkpoint (e.g., confirming a TODO is newly introduced rather than pre-existing, or that a flagged link is actually broken); the batch-operation cap at 3 applies. | 3 / 5 |
Progressive Disclosure | The body is under 50 lines with no bundle files and no content that belongs in separate references; sections ("Output per merge", "What to look for", "Rules", "Untrusted input") are well-organized, and the single background link is clearly signaled and one level deep, satisfying the simple-skill exception for a top score. | 5 / 5 |
Total | 17 / 20 Passed |