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 unusually disciplined instruction-only skill: terse, fully operational commands, a clearly sequenced diagnostic workflow with repeated evidence-validation checkpoints, and an explicit handoff contract with scope boundaries. The two gaps are minor: the report-verifier invocation is described rather than shown, and the one external reference (../_shared/base-green.md) lives outside the bundle and cannot be confirmed to exist.
Suggestions
Show one concrete invocation of scripts/ci/release-green-result.mjs (arguments, expected output) instead of only describing its contract, so the verification step is copy-paste executable.
Either bundle the shared base-green guidance as a reference file inside the skill (e.g. references/base-green.md) or inline the few rules from ../_shared/base-green.md that this skill depends on, so the skill is self-contained and the reference can be verified.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and dense throughout: no background concept is explained, every sentence is operational ('Build in a worktree with physical dependencies, never an external node_modules symlink. Preserve stdout/stderr, process receipts and the validator's own exit.'), and boundaries are stated once without padding. Fully matches anchor 5's 'every token earns its place'. | 5 / 5 |
Actionability | Includes copy-paste commands ('npm run check:release-green -- --quick', '--full-ci --with-build --hermetic --json'), concrete script paths, and a specific handoff field list. Falls short of anchor 5 because the report verifier's usage is described but never shown ('use scripts/ci/release-green-result.mjs when available... supply the values from the run') and flag selection is delegated to 'Inspect scripts/quality/validate-release-green.mjs' rather than stated. | 4 / 5 |
Workflow Clarity | Clear sequenced workflow — read the shared base doc, resolve SHAs, isolate a worktree, select a profile, execute, interpret against the exact base, hand off — with explicit validation checkpoints throughout ('Read the report even when the outer job is green', 'Missing, contradictory, stale, cancelled or timed-out evidence cannot establish PASS', 'Re-query live refs before using the result for a later decision') and a concrete handoff checklist. The skill is diagnostic, not destructive, so no cap applies. | 5 / 5 |
Progressive Disclosure | A ~52-line, well-sectioned single file whose only delegated reference ('Read ../_shared/base-green.md before interpreting remote status') is clearly signaled and one level deep. Not a 5: that reference sits outside the skill's own bundle (no references/, scripts/, or assets/ directories exist alongside SKILL.md), so the essential shared guidance cannot be verified from the skill's structure — a minor organization gap against anchor 5's 'appropriately split' content. | 4 / 5 |
Total | 18 / 20 Passed |