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.
A well-structured, actionable instruction skill with explicit sequencing and validation gates for destructive merge operations, and clean one-level-deep references. Minor conciseness gains are available by consolidating the repeated rescan/refresh rules.
Suggestions
Consolidate the rescan-on-state-change and refresh-before-write rules into a single stated rule in 'Run the Loop', then reference it from the other sections instead of restating it.
Tighten 'Apply Common Decisions' so its items don't restate guidance already covered in 'Run the Loop'; keep it to decisions not derivable from the loop steps.
Consider surfacing one or two concrete command examples for the most common write actions (e.g., the draft-PR creation via nemoclaw-contributor-create-pr) to lift actionability toward fully copy-paste-ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and almost entirely operational with no padding about concepts Claude already knows, but the rescan-on-state-change and refresh-before-write guidance recurs across several sections and could be consolidated. | 4 / 5 |
Actionability | For an instruction-only skill it gives concrete, specific guidance — named sub-skills to load, an explicit PR-body field list, exact commands to avoid (gh run rerun, gh workflow run .github/workflows/e2e.yaml) — with execution detail delegated to referenced files. | 4 / 5 |
Workflow Clarity | The loop is a clearly sequenced multi-step process with explicit validation checkpoints ('Run both checks immediately before the write', 'stop editing it' while waiting for CI/review, 'Re-read the PR and rules immediately before merge') and feedback loops for obsolete-work and failed-merge containment. | 5 / 5 |
Progressive Disclosure | The body is a concise overview pointing to one-level-deep references that all exist (queue-and-ownership.md, review-and-merge.md, continuity-handoff.md), each clearly signaled inline and appropriately split out for easy navigation. | 5 / 5 |
Total | 18 / 20 Passed |