Content
82%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 high-quality instruction-only skill body: every rule is operational and non-obvious, gates have explicit pass conditions, and the output format is exactly specified down to comment prefixes and placement. Remaining gaps are minor — a few rationale sentences to trim, an implicit interaction between a failed Gate 1 and later gates, and data tables that could move to a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with repo-specific rules Claude cannot know (companion-file table, accepted-issue label criteria, churn strike classes) and explains no general concepts, but a few rationale sentences could be trimmed — e.g. 'Both front matter flags keep local agents from loading this skill; the bot reaches it through .github/copilot-instructions.md' and 'New Non-blocking findings on unchanged code are what keep reviews from ending'. This fits anchor 4 ('minor instances of over-explanation') rather than anchor 5 ('every token earns its place') and is far above anchor 3. | 4 / 5 |
Actionability | Fully executable instruction guidance: exact comment prefixes ('Gate:', 'Scope:', 'Churn:') with exact placement (first changed file, first hunk of the part that should move, the regressing line), exact label and path names to check, a strike escalation ladder, and a copy-paste-ready issue-draft template. Per the scoring notes, absence of code in an instruction-only skill is not penalized; this matches the level-5 anchor. | 5 / 5 |
Workflow Clarity | Clear sequence (Step 0 fetch table → gate-selection table → Gates 1–3 → output) with explicit pass/fail checkpoints per gate and an escalation ladder (strike 1 warning → strike 2 abandon). A minor gap keeps it at anchor 4 rather than 5: whether Gates 2–3 still run after Gate 1 fails ('then stop') is resolved only implicitly via the Principles line 'When a gate fails, report it and stop: no line-by-line review', not stated at the gate itself. It is well above anchor 3, whose checkpoints are missing or implicit. | 4 / 5 |
Progressive Disclosure | No bundle files (references/, scripts/, assets/) exist, so this scores on the body alone: a single well-sectioned file with clearly signaled one-level references to the sibling skill ([nic-code-review](../nic-code-review/SKILL.md)) and CONTRIBUTING.md anchors. Structure is good with minor gaps — the large companion/scope tables are core operational content and justify inline placement, but the ~150-line body with three big data tables could plausibly split some material into a reference file. Anchor 4 ('good structure, most content appropriately placed, references mostly clear') fits better than anchor 5, which requires a deliberate overview-to-detail split. | 4 / 5 |
Total | 17 / 20 Passed |