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 strongly structured operational skill: every phase has executable commands, three explicit operator checkpoints with STOP gates, and a lint feedback loop that blocks pushing on failure. Detail is correctly pushed to real one-level bundle files. The only deduction is minor redundancy in repeated boundary statements and the duplicated trigger list.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and assumes competence (exact git/gh/script commands, no concept explanations), but a few spots could be trimmed: the tagging boundary is restated in the intro, Phase 2, anti-patterns, Phase 7, and the Boundary section, and the 'When to use' trigger list duplicates the frontmatter description. This sits above 'minor instances of over-explanation' being absent but below 'every token earns its place'. | 4 / 5 |
Actionability | Fully executable throughout: `git rev-parse --abbrev-ref HEAD`, `git describe --tags --abbrev=0`, `scripts/list-changes-since-tag.sh` with its documented JSON output fields, `scripts/bump-version.sh <new-version>`, `scripts/verify-lint-mirror.sh` with concrete failure cures (`uv run --extra dev ruff check src/ tests/ --fix`), and the full `git commit` / `git push` / `gh pr create --body-file` sequence with a PR-body template. Copy-paste ready and covers the common cases including failure paths. | 5 / 5 |
Workflow Clarity | A clear 7-phase sequence with explicit validation checkpoints: STOP if on main, wait for operator confirm after the bump decision and changelog diff, halt on non-zero from bump-version.sh, and a lint gate with a feedback loop (fail -> cure -> re-run, 'Do NOT push or open the PR until lint is green'). Error recovery guidance is specific per failure mode, matching the top anchor. | 5 / 5 |
Progressive Disclosure | The SKILL.md body is an orchestration overview that defers detail to real, one-level-deep bundle files (assets/semver-rubric.md, assets/entry-sanitizer.md, assets/pr-body-template.md, scripts/*.sh — all verified present, none nesting further) and explicitly references external primitives instead of duplicating them ('Do not duplicate their content; reference them'). Well-signaled and easy to navigate. | 5 / 5 |
Total | 19 / 20 Passed |