Content
88%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 strong, dense procedural skill: an unambiguous sequenced workflow with validation gates and approval checkpoints, fully executable commands for every provider path, and lean rule sections that only justify constraints Claude could not infer. The only structural improvement would be moving the GitLab template-adaptation detail into a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes competence (no explanation of what a PR is), with commands and rules stated tersely; parenthetical rationales ('a heredoc-to-file can silently land 0 bytes', 'Em-dashes read as machine-written') are project-specific constraints, not padding. Not 5 because a few asides ('renders fine on GitLab', the em-dash rationale) could be trimmed without losing guidance. | 4 / 5 |
Actionability | Fully executable throughout: exact gather-context commands, provider detection rules, a fallback command chain ('devflow check # or, if unavailable: yarn lint && yarn build && yarn test --changedSince=main'), a complete structured body template, and copy-paste-ready 'gh pr create' and 'glab mr create' invocations. Specific examples cover the common GitHub, GitLab, and unknown-provider cases. | 5 / 5 |
Workflow Clarity | A clearly sequenced 10-step pipeline with explicit validation checkpoints and feedback handling: checks run at step 3, step 4 gates on issues ('ask the user whether to fix first or proceed'), step 7 requires approval of the draft, and the draft-when-unverified rule plus 'Never open a PR/MR with failing checks' close the loop. The creation step includes a checklist for the resulting PR. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned (Steps, Description rules, Provider assembly, Creating the PR/MR, Important) and external detail is delegated by pointer to CLAUDE.md and lib/utils.sh rather than inlined. Not 5 because the ~30-line GitLab template-adaptation subsection is sizeable inline content with no reference-file split, leaving minor organization gaps; not 3 because navigation is easy and nothing is buried. | 4 / 5 |
Total | 18 / 20 Passed |