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.
The body is a tightly written, repo-specific operational guide: an explicit mode-selection flow, copy-paste commands, safety guardrails around force-pushes and tag replacement, and a single well-signaled reference file holding the full command checklist. Its only weakness is mild duplication of the check commands and changelog format between SKILL.md and the reference file.
Suggestions
Trim the duplicated bilingual changelog template from the body (or from references/release-checklist.md §7) so the format is stated in exactly one place, keeping the body's pointer to the reference.
Replace the inline list of `pnpm run format` / `i18n` / `lint` commands with a single line deferring to the checklist's 'Recommended checks' section, since the reference already enumerates them.
Consider moving the 'Common Recovery Case' numbered steps into the checklist as mode 4b to reduce overlap with the 'update existing release branch' instructions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence — e.g. 'Check whether `release/<version>` exists locally or on `origin`' with no git tutorial — but there is duplication with the bundle reference: the check commands (`pnpm run format` / `i18n` / `lint`) and the bilingual changelog format block appear in both SKILL.md and references/release-checklist.md. This fits anchor 4 ('efficient; minor instances that could be trimmed') rather than 5, where every token would earn its place. | 4 / 5 |
Actionability | Guidance is fully executable: exact commands (`pnpm run release:ff -- release/<version> --tag v<version>`, `--force-with-lease`), a copy-paste changelog template with date format, and a pointer to references/release-checklist.md for the full command sequence. Specific examples cover the common cases (four modes, recovery case); not score 4 because there are no gaps in executable detail. | 5 / 5 |
Workflow Clarity | The sequence is clear (inspect state → choose mode → update metadata → run checks → branch policy → publish → clean up) with explicit validation checkpoints for the destructive steps: 'If a local or remote tag already exists on the wrong commit, stop and ask', 'Use this only after the release PR is approved', '--force-with-lease', and 'separate blocking failures from unrelated existing warnings'. This matches the anchor with explicit validation, error-recovery guidance, and a recovery case; not score 4 because no meaningful checkpoint is missing. | 5 / 5 |
Progressive Disclosure | The body is a well-organized overview that defers exact commands to a real, clearly signaled one-level-deep reference ('Use [references/release-checklist.md](references/release-checklist.md) for exact commands') and points to the repository's docs/release-flow.md for full policy — the file exists in the bundle and is referenced with its purpose stated. Content is appropriately split with easy navigation; not score 4 because navigation and signaling are clean, not just mostly clean. | 5 / 5 |
Total | 19 / 20 Passed |