Content
76%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.
The body is a tightly written, highly actionable API reference with copy-paste-ready calls and clear response-handling guidance. Its main gap is workflow structure: destructive/mutating operations lack an explicit validation-and-feedback-loop sequence.
Suggestions
Add a short numbered workflow for mutating operations (e.g. create PR) that includes an explicit verify step and a fix-and-retry loop on non-2xx responses.
De-duplicate the Authorization-header warning so it lives in only one place, or explicitly mark the repetition as deliberate reinforcement.
Consider splitting the full endpoint catalog into a REFERENCE.md and leaving SKILL.md as an overview, which would let progressive disclosure reach the top anchor.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and action-oriented and assumes Claude already knows GitHub/API concepts (no 'what is a PR' padding); minor redundancy (the 'do NOT add an Authorization header' warning appears in both the intro and Common Mistakes) keeps it just below 5. | 4 / 5 |
Actionability | Copy-paste-ready http(...) calls cover issues, PRs, repos, search, and authenticated-user queries, and the Response Handling section gives concrete parsing code (e.g. pr = r["body"]; title = pr["title"]) — specific examples cover the common cases. | 5 / 5 |
Workflow Clarity | A request → status-check → parse → surface pattern with fail-fast validation is present in Response Handling, but there is no explicit sequenced workflow or validate→fix→retry feedback loop for destructive operations (create PR/issue), so the destructive-ops cap holds at 3. | 3 / 5 |
Progressive Disclosure | Sections are well-organized (API Patterns, Response Handling, Common Mistakes) and the file is self-contained with no nested references; it stops short of 5 because, with no bundle files and no one-level-deep file references, the endpoint catalog is entirely inline rather than split out. | 4 / 5 |
Total | 16 / 20 Passed |