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.
A tight, chronological instruction set with concrete BAD/GOOD examples, explicit pre-submission validation checks (existing PR, diff-vs-goal, secrets), and a command-level fallback for a known failure mode. The only notable gap is that the primary `gh pr create` path is never shown as a command, leaving the reader to construct it from the REST fallback's fields.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~44-line body is pure instruction with no explanations of concepts Claude already knows (no PR primer, no gh walkthrough); every token adds a convention or checkpoint Claude doesn't inherently know, e.g. "PR titles usually become commit messages, so follow the repository's title conventions" and the rate-limit fallback. This matches the anchor 'Lean and efficient; assumes Claude's competence; every token earns its place' — the BAD/GOOD quotes are illustrative contrasts, not padding. | 5 / 5 |
Actionability | Concrete, executable guidance is present: the REST fallback `gh api repos/<owner>/<repo>/pulls -f title=... -f head=... -f base=... -F body=@<file>`, BAD/GOOD title and description examples, and the attribution template `Created with <model> in <harness>.`. It stays below anchor 5 because the primary `gh pr create` invocation and the command for checking whether a PR already exists are left implicit — minor gaps rather than 'fully executable ... covers the common cases'. | 4 / 5 |
Workflow Clarity | A clear chronological sequence (check existing PR → review diff → title → description → attribution → secrets check → submit) with explicit validation checkpoints ('Before filing, check whether a PR for this branch already exists', 'Before submission, check the title, description, quotes, and attachments for secrets') and an error-recovery feedback loop ('If `gh pr create` fails because the GraphQL rate limit is used up, open it through REST'). This matches anchor 5; PR creation is not destructive/batch, so no cap applies, and anchor 4's 'minor validation gaps' understates the three explicit checks plus fallback. | 5 / 5 |
Progressive Disclosure | The skill is under 50 lines, single-purpose, with no bundle files (references/, scripts/, assets/ are absent), so nothing needs externalizing; cross-references to sibling skills (`babysit-pr`, `leaving-pr-comments`) are clearly signaled exactly at the point of need ('If the user also asked to babysit it, continue with the `babysit-pr` skill'). Per the simple-skill guideline, this earns the top score with well-organized chronological content. | 5 / 5 |
Total | 19 / 20 Passed |