Content
77%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 well-structured, lean workflow loop with explicit validation gates, feedback, and stop conditions — the body itself is close to exemplary. The weak point is the bundle: most referenced reference/workflow files are missing, and several bundled scripts are orphans, breaking the otherwise good navigation structure.
Suggestions
Ship the referenced files (MERGE-GATE.md, SALVAGE-PR.md, SECURITY-SWEEP.md, TEST-GAPS.md, HOTSPOTS.md, SEQUENCE-WORK.md, PR-REVIEW-PRIORITIES.md, RISKY-AREAS.md, STATE-SCHEMA.md, ../_shared/pr-follow-up.md) with the skill, or inline the critical steps they contain.
Reference or document the orphaned scripts in the bundle (check-gates.ts, triage.ts, hotspots.ts, handoff-summary.ts, run-trusted-check-gates.sh) so the SKILL.md navigation covers the actual bundle contents.
Remove the duplicated merge-decision guidance (Autonomy vs. Stop and Ask) and the repeated version-progress command block to tighten the body further.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence — no concept explanations, tight command blocks, one-outcome-per-pass rule. Minor trimming is possible: merge-decision guidance appears in both the Autonomy block ("Never merge. Ask the user about merge") and the Stop and Ask section, and the version-progress command is shown twice verbatim. Not 5 due to that redundancy; not 3 because the padding is minor. | 4 / 5 |
Actionability | Concrete executable commands ("node --no-warnings .agents/skills/.../version-target.ts", state.ts invocation) and a specific decision list with objective criteria ("CI passes, no conflicts remain, and tests cover the change"). Not 5 because the executable detail for most actions is delegated to referenced workflow files (MERGE-GATE.md, SALVAGE-PR.md, etc.) that are not present in the bundle, so the reader cannot actually follow them; not 3 because the top-level commands and selection criteria are executable as written. | 4 / 5 |
Workflow Clarity | A clear four-step sequence with explicit validation (pre-push publication guard: "Immediately before a push, confirm that the remote branch still points to that commit. Stop when another workflow publishes first"), a feedback loop (re-run the progress script in Step 4), and a checklist (Stop and Ask When section). The destructive-operation validation cap does not apply since push/approve actions have explicit gates. Not 4 because checkpoints, error recovery (backlog fallback, stop conditions), and reporting are all explicit rather than implicit. | 5 / 5 |
Progressive Disclosure | References are clearly signaled and one level deep (References section plus inline links per action), but scored against the actual bundle: the referenced files PR-REVIEW-PRIORITIES.md, RISKY-AREAS.md, STATE-SCHEMA.md, MERGE-GATE.md, SALVAGE-PR.md, SECURITY-SWEEP.md, TEST-GAPS.md, HOTSPOTS.md, SEQUENCE-WORK.md, and ../_shared/pr-follow-up.md are all absent from the bundle, so most navigation targets are broken; additionally several bundled scripts (check-gates.ts, triage.ts, hotspots.ts, handoff-summary.ts) are never referenced. Not 2 because the body itself is well structured and clearly signals its references; not 4 because missing referenced files are more than a minor organization gap. | 3 / 5 |
Total | 16 / 20 Passed |