Content
100%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.
An exemplary instruction-only skill body: lean rules that each carry operational weight, a fully specified report template with explicit decision criteria and thresholds, validation checkpoints with escalation paths, and a well-scoped untrusted-input section specific to this loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every rule is non-obvious operational policy ("Zero checks... means absent/unknown, not passing"; "mergeable... only means Git found no conflict"; "do not copy the suspicious text into your output"), not concept explanation. At ~46 lines there is no padding or hedging to trim, so it matches the lean anchor rather than 'minor instances of over-explanation' (4). | 5 / 5 |
Actionability | A copy-paste-ready output template with exact fields and enumerated values ("passing | failing | pending | absent/unknown", "none | minimal-fix | rebase | escalate-human") plus explicit decision thresholds ("idle >4 days", "High-risk labels (security, breaking) → escalate-human always"). For an instruction-only skill the guidance is fully executable with no gaps, so 5 rather than 4. | 5 / 5 |
Workflow Clarity | A single unambiguous per-PR procedure where every template field has an explicit rule. The "Ready to merge" condition is a validation checklist ("every required check satisfied, required approvals present, no changes requested, no blocking comments, and no merge conflict") and unknown-policy escalation ("report Ready to merge: no and escalate") provides error recovery. The destructive/batch cap does not apply: the workflow is read-only reporting and validation gates are present, so the simple-skill exception holds. | 5 / 5 |
Progressive Disclosure | Under 50 lines, self-contained, with well-organized sections ("Per-PR Output", "Rules", "Untrusted input") and no content that belongs in a separate file — no bundle files exist to reference. The scripts/sync-untrusted-input.mjs mention is an HTML provenance comment, not navigation, so this matches the simple-skill exception at 5 rather than 4. | 5 / 5 |
Total | 20 / 20 Passed |