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 lean, highly executable wrapper skill with an explicitly gated workflow (dry-run, verify, exit-status watch, readback) and disciplined delegation to the owning release-lanes rule. The only gap is that a couple of peripheral steps (stale-PR close, run-id resolution) lack concrete commands.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is pure commands, constraints, and stop conditions — 'node tooling/scripts/release-branch-prs.mjs sync-main-to-next --dry-run', 'Do not run autoreview', 'stop only for: dirty checkout... real source conflicts...' — with no concept explanation or padding; every line earns its place. | 5 / 5 |
Actionability | The core workflow is copy-paste ready with full flags and JSON field lists ('gh run list --branch next --limit 10 --json databaseId,name,workflowName...'), but two steps have minor gaps: 'Close stale sync/main-to-next PRs' has no closing command, and '<release-run-id>' placeholders must be filled from earlier output. | 4 / 5 |
Workflow Clarity | The sequence is explicit with validation at every stage: dry-run gate, push, 'verify-main-to-next-sync', 'gh run watch ... --exit-status', npm/GitHub readback, and stale-PR cleanup only after verification passes — plus a dedicated Hard Stops section covering failures and repair limits. | 5 / 5 |
Progressive Disclosure | With no bundle files, the skill is a compact, well-sectioned wrapper (Default Mode, Keep It Fast, Hard Stops, Handoff) that delegates architecture to a single one-level reference: 'loading .agents/rules/release-lanes.mdc' and 'release-lanes owns the release architecture, script contract, hard stops, and handoff shape'. | 5 / 5 |
Total | 19 / 20 Passed |