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 skill with a strong output contract and a meaningful prompt-injection defense. The body is token-efficient and actionable; its weakness is the implicit workflow — no explicit scan sequence or verification checkpoints, and a couple of undefined references ('state', the routing choice between minimal-fix and skip).
Suggestions
Add an explicit ordered scan sequence with the concrete command to enumerate merges (e.g. `gh pr list --merged --base main --since <date>`) so the entry step is unambiguous.
Add validation checkpoints: verify a TODO/FIXME is newly introduced in the merge (compare against the pre-merge tree) and confirm doc links actually fail to resolve before reporting them as broken.
Define the 'state' file (where it lives, what it contains) and complete the routing rules so it is clear when to choose minimal-fix vs. skip.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~38-line body is lean with zero concept explanation: a copy-paste output template, a five-item detection list, five terse routing rules, and a substantive untrusted-input policy. Every token earns its place; not 4 because there are no over-explanations to trim (the safety block is load-bearing for an autonomous loop that ingests PR/issue text). | 5 / 5 |
Actionability | Concrete, executable guidance: an exact output template (file:line, risk/effort enums, four loop-action choices), named detection targets, and routing rules ("Large refactors → ticket", "Medium+ risk → escalate-human", 7-day window). Not 5 due to minor gaps: the "state" file is referenced but never defined, no command for enumerating recent merges, and no rule for when to choose minimal-fix vs. skip. | 4 / 5 |
Workflow Clarity | The sequence (enumerate merges → scan each for listed items → emit per-merge report → route action) is coherent but only implicit via section organization, and checkpoints are missing or implicit — e.g. "introduced in merge" states a filter without any method to verify it, and there is no verification step before reporting (confirming a doc link is actually broken). Not 4 because most checkpoints are absent rather than minor gaps; the read-only nature of the scan keeps it above level 2. | 3 / 5 |
Progressive Disclosure | Under 50 lines with no external content needed; well-organized sections (output format, detection list, rules, safety policy) with the one external pointer being a clearly signaled background link. Per the simple-skill scoring note, this earns 5 with just well-organized sections; no bundle files exist, so no navigation burden applies. | 5 / 5 |
Total | 17 / 20 Passed |