Content
96%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 an exemplary instruction-only skill: fully executable commands, a clearly sequenced workflow with explicit validation gates and stop conditions, and dense repo-specific detail with no filler. Its only structural weakness is that everything lives inline in one long SKILL.md with no bundle files, where a portion of the reference material (e.g., the module table) could be split out.
Suggestions
Consider moving the module-mapping table and type list into a references/ file (e.g. references/modules.md), keeping only the core rules inline, to shrink the always-loaded SKILL.md body.
The interaction principle paragraph could be condensed to one sentence, since tool-level permission prompts already imply most of it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is command-dense with essentially no padding: it teaches nothing Claude already knows and spends its tokens exclusively on repo-specific knowledge (the module-mapping table, the CI title regex location, the draft-file convention, the interaction policy). Not 4: there is no over-explanation left to trim — even prose sections like "Do not reproduce the template from memory — it changes" justify their tokens with a failure mode. | 5 / 5 |
Actionability | Every step is copy-paste executable: `git branch --show-current`, `gh pr view --json number,title,body`, `git push -u origin HEAD`, `tail -n +3 .pr-drafts/<branch-name>.md > /tmp/pr-body.md` with `--body-file`, plus a concrete filename convention and file format example. Not 4: common cases (new PR vs. existing PR, title vs. body extraction) are each covered with exact commands. | 5 / 5 |
Workflow Clarity | The four-step sequence is explicit with real validation checkpoints and feedback loops: stop on `main`, "If `make quality` fails: stop and report errors", stop when no commits are ahead, and "A `risky` verdict stops the PR: report it and wait for the user". Not 4: validation is explicit at every risky point (quality gate, review gate, empty-branch guard) rather than having minor gaps. | 5 / 5 |
Progressive Disclosure | No bundle files exist; the ~155-line body is well-sectioned (Interaction Principle, Steps, Common Pitfalls) and its external references (`.github/PULL_REQUEST_TEMPLATE.md`, `.github/workflows/check_pr_title.yml`) are clearly signaled and one level deep. Not 5: the content is a single monolithic inline file — the 17-row module-mapping table in particular is sizable enough that a split into a reference file could be argued, so structure is good rather than optimally split. | 4 / 5 |
Total | 19 / 20 Passed |