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.
An operationally dense, copy-paste-ready runbook with an explicit sequenced workflow, validation checkpoints, gated destructive operations, and well-signaled pointers to repo runbook and validation tooling. Its only weaknesses are minor: a few trimmable justification passages and an inlined pre-flight section that could live in a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every line is a command or a repo-specific constraint Claude cannot infer ("Versions live in git tags only", "A release is done when the run is green and the GitHub release is published", the v0.18.52 checksum-mismatch failure mode), with no explanation of concepts Claude already knows. It is not 5 because a few passages could be trimmed — the incident 2026-09-09 backstory and its Warden mechanics ("warden.yml skips them and warden-clearance.yml requires head_repository == repository") state more justification than needed to act — and not 3 because there is no padded or redundant material beyond those minor spots. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout: `pnpm release:cut` variants, `gh workflow run "Release App" ... -f bump=patch`, the gh+jq PR-triage scripts, `gh run watch <run-id> --exit-status`, `gh release view ... --json assets`, the `curl -sI` asset spot-check, and `npm view openwork-server version`. It covers the common cases (default cut, manual equivalent, expedited admin tag, rerun/recovery, verification); the only placeholders (vX.Y.Z, <sha>, <run-id>) are appropriately parameterized. Not 4 — no gaps exist. | 5 / 5 |
Workflow Clarity | Clear, ordered sequence — pre-flight fork-PR triage, cut, watch, verify, validate — with explicit validation checkpoints ("never when the tag is created" done-criterion, `--exit-status` watch, asset-name verification, the validate-a-release gate before declaring rollout-safe) and explicit error-recovery loops ("If the run fails before the release is published: land the fix on dev ... cut the next patch"). The destructive tag-delete is gated ("Only delete/recreate a tag after verifying the GitHub release is still draft-only") and ship-anyway decisions require a recorded decision. Not 4 — checkpoints and recovery paths are explicit, not implicit. | 5 / 5 |
Progressive Disclosure | A well-sectioned single file with clear headers and one-level-deep, clearly signaled pointers to repo materials ("Full runbook: docs/RELEASING.md", scripts/release/stamp-version.mjs and verify-tag.mjs, the validate-a-release skill path). No bundle files (references/, scripts/, assets/) exist, so this scores the file's own structure: good organization, but the ~55-line pre-flight triage section with two gh/jq scripts is inlined where a reference file could carry it, and the skill is well over the 50-line simple-skill exception. Not 5 (not a lean overview with split-out references); not 3 (structure and navigation are clear, references are signaled, not buried). | 4 / 5 |
Total | 18 / 20 Passed |