Content
85%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 an exemplary lean orchestration document: exact commands, a validated four-step loop with explicit gates, and a clear escalation checklist. Its one real weakness is that its progressive-disclosure skeleton is not backed by the bundle — most referenced workflow files are missing and most bundled scripts are undiscoverable from the skill.
Suggestions
Ship the nine referenced .md 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) or remove the dangling references.
Make the unreferenced scripts (check-gates.ts, triage.ts, hotspots.ts, handoff-summary.ts, run-trusted-check-gates.sh) discoverable — e.g. via the References section or from the workflow docs that invoke them.
Clarify the location/ownership of ../_shared/pr-follow-up.md, since the Step 2 prerequisite depends entirely on it and it lives outside this skill's bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense rules and copy-paste commands with zero concept explanations — it assumes Claude already knows git, CI, and PR workflows ("Never merge. Ask the user about merge, product-scope, and architecture decisions."). Every section (autonomy rules, action selection, commit hygiene, stop-and-ask, /loop integration) carries new instructions, matching "lean and efficient; every token earns its place". | 5 / 5 |
Actionability | Step 1 and Step 4 give exact, executable commands ("node --no-warnings .agents/skills/nemoclaw-maintainer-day/scripts/version-target.ts", the state.ts history invocation), and Step 2 is a concrete ordered decision tree. Not a 5 because the actual execution details of each selected action are delegated to reference files (MERGE-GATE.md, SALVAGE-PR.md, etc.) that are absent from the bundle, leaving a gap beyond "minor" in end-to-end executability. | 4 / 5 |
Workflow Clarity | The four-step loop is clearly sequenced with explicit validation checkpoints: PR follow-up must "complete ... successfully for one unchanged latest PR commit", approval requires "CI passes, no conflicts remain, and tests cover the change" with no unresolved correctness or security issue, and the pre-push publication guard ("confirm that the remote branch still points to that commit") plus post-pass verification (re-run progress script) close the feedback loop. The "Stop and Ask When" checklist handles error/edge paths, matching the top anchor. | 5 / 5 |
Progressive Disclosure | The design is right — a lean overview with a References section and well-signaled, one-level-deep pointers — but scored against the actual bundle it breaks down: none of the nine referenced .md files (PR-REVIEW-PRIORITIES.md, MERGE-GATE.md, SALVAGE-PR.md, SECURITY-SWEEP.md, TEST-GAPS.md, HOTSPOTS.md, SEQUENCE-WORK.md, RISKY-AREAS.md, STATE-SCHEMA.md) exist, ../_shared/pr-follow-up.md points outside the bundle, and six of the nine scripts in scripts/ (including the 84KB check-gates.ts) are never referenced. That is more than the "minor organization gaps" of a 4, so it lands at "some structure but could be better organized". | 3 / 5 |
Total | 17 / 20 Passed |