Content
81%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 highly actionable, well-validated workflow document: every step has a concrete command, explicit checkpoints, and defined failure paths. The main cost is token efficiency — repeated issue-acceptance policy and a redundant options table inflate the body without adding guidance Claude does not already have.
Suggestions
State the issue-acceptance rule (and its dependency-update/flaky-test exceptions) once in a single section and reference it elsewhere instead of restating it in Prerequisites, 'Verify Contributor Access and Issue Acceptance', and 'Link to an Issue'.
Drop or trim the 'Useful Options' table, since Claude already knows standard gh pr create flags; keep only project-specific notes such as the conventional-commit title requirement.
Move the repo-specific policy detail (vouch requirements, security disclosure, config-doc cross-checks) into a short reference file and keep SKILL.md to the core PR-creation workflow.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean commands and directives with no explanation of concepts Claude already knows, but the issue-acceptance policy is restated three times in overlapping forms (Prerequisites, 'Verify Contributor Access and Issue Acceptance', 'Link to an Issue') and the 'Useful Options' table re-lists standard gh flags, so it could be meaningfully tightened. | 3 / 5 |
Actionability | Fully executable guidance throughout: concrete gh/git commands, a copy-paste-ready example PR with heredoc body covering the common case, exact title formats, and a populated description template. | 5 / 5 |
Workflow Clarity | Clear sequenced workflow (verify access/issues, check docs and infra, verify branch state, push, create, post-create) with explicit validation gates ('Should NOT be main', gh repo view/gh issue view checks, 'Resolve failed lookups before proceeding') and recovery paths ('preserve local work and report which issue needs acceptance'; re-run acceptance after label changes). | 5 / 5 |
Progressive Disclosure | A single, well-sectioned file that correctly delegates project policy to existing repo artifacts (CONTRIBUTING.md, .github/PULL_REQUEST_TEMPLATE.md, the sync-agent-infra skill) instead of duplicating them, but at roughly 270 lines the dense project-policy prose is a candidate for a separate reference file. | 4 / 5 |
Total | 17 / 20 Passed |