Content
81%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.
This is a strong instruction-only skill: a clearly sequenced multi-phase workflow with concrete commands, named tool calls, explicit validation checkpoints, and a well-defined rejection/retry feedback loop. The only weaknesses are minor — some redundancy between the Behavior Rules and earlier sections, and a few spots where an example (transition payload, plan excerpt) would make execution even more copy-paste ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and procedural with no explanations of concepts Claude already knows, and nearly every sentence carries operational instruction (e.g., "Run `git rev-parse --show-toplevel` and `git remote get-url origin`"). It is not a 5 because the Behavior Rules section partially restates earlier content ("One terminal call" repeats the opening paragraph and Phase 4; "Decide and record" repeats the Decision rule), which could be trimmed. | 4 / 5 |
Actionability | Guidance is concrete and executable: exact commands (`git rev-parse --show-toplevel`, `git remote get-url origin`, `gh repo view --json visibility`), a named terminal tool call with parameters (`factory_transition_work_item`, `stage: "execute"`, `expectedRevision` from the `factory-phase` signal, rationale max 1000 chars), and an exact artifact path (`.artifacts/plans/issue-<number>.md`). It is not a 5 because a few specifics are still abstract — no example transition payload or sample plan snippet — though per the scoring notes absence of code in an instruction-only skill is not itself penalized. | 4 / 5 |
Workflow Clarity | The sequence is explicit and ordered (Repository Guard → Phase 1 Verify → Phase 2 Design → Phase 3 Write Plan → Phase 4 Transition) with explicit validation checkpoints (repo verification before the first edit and again before push; verify inherited understanding against current code) and an explicit error-recovery feedback loop ("If it is rejected, read the stated reason, address it... and retry once corrected"). Validation steps are present, so the destructive/batch cap does not apply, and the Behavior Rules section closes with a checklist — matching the top anchor. | 5 / 5 |
Progressive Disclosure | No bundle files exist, and the content is well-organized into clearly headed sections with everything appropriately placed inline for a self-contained workflow skill. It is not a 5 because the body (~63 lines) exceeds the under-50-line simple-skill exception and a couple of sections (the untrusted-data handling paragraph, the Repository Guard detail) could arguably be split into a reference file; it is well above a 3 since nothing that belongs in a separate file is bloating the main body. | 4 / 5 |
Total | 17 / 20 Passed |