Content
92%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 excellent instruction-only skill: lean, dense with non-obvious domain constraints, and structured as a sequenced audit workflow with reproduction, regression-test, and merge-gate validation loops. The only gap is the absence of any literal commands or GraphQL query examples, which keeps actionability at 4.
Suggestions
Include one or two copy-paste examples for the most fragile steps — e.g., a sample GraphQL query for fetching thread-aware review state, or the exact gh command shape for re-fetching head SHA/checks — so the workflow's least standard operations are fully executable without relying on AGENTS.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line is a directive; there is no padding, no explanation of concepts Claude already knows, and no over-justification. Lines like "Do not treat a flat comment list as proof that all review threads are resolved" add non-obvious domain constraints, matching the 'lean and efficient; every token earns its place' anchor. Not a 4 because no section could be trimmed without losing actionable content. | 5 / 5 |
Actionability | Highly concrete guidance: exact artifacts to read ("AGENTS.md", "docs/openapi/latest.json"), "disposable app 6759231657", "GitHub GraphQL" for thread state, and "Re-fetch the head SHA ... after every push". It falls short of the 'fully executable, copy-paste ready' anchor because no literal commands or query examples are included — exact commands are delegated to AGENTS.md — leaving minor gaps rather than major ones. | 4 / 5 |
Workflow Clarity | A clearly sequenced six-phase workflow (contract → inspect → verify → fix → merge gate → hand off) with explicit validation checkpoints: "Reproduce each defect before changing code", "confirm the failure", "rerun the focused test", a full merge-gate checklist, and re-fetching state after every push. This feedback-loop-rich structure with a checklist matches the top anchor; destructive/batch operations here are gated by validation, so no cap applies. | 5 / 5 |
Progressive Disclosure | The body is a well-organized single SKILL.md with clear section headers, no bundle files, and nothing inlined that belongs in a separate file. Detailed authority/review rules are correctly delegated one level deep to AGENTS.md with clearly signaled references, matching the 'clear overview with well-signaled one-level-deep references' anchor. | 5 / 5 |
Total | 19 / 20 Passed |