Content
71%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 dense, highly prescriptive hub skill: it assumes the reader's competence, provides concrete commands and explicit validation verdict-handling per harness, and delegates detail via clearly anchored one-level-deep links. Its weaknesses are extreme clause density in several rule paragraphs, inlining of reference-grade per-harness operational detail, and reliance on workflow files that are not present in the evaluated bundle. Overall a strong instruction-only skill whose structure would benefit from offloading the Long-Running Validation detail to a reference file.
Suggestions
Move the per-harness Long-Running Validation rows (Claude Code, Codex, Pi, warden hosts) into a reference file such as references/dev-validate-run.md, keeping only the routing rule and a table of harness→route in SKILL.md.
Ship workflows/dev-implement.md and workflows/dev-fix.md in the skill bundle — the body's section anchors (§ 1-11, § 5) point to them but they are absent, so the core process cannot be followed from the provided files.
Break the longest multi-clause rules (the Round Contract rebasing paragraph and Engineering Rules exceptions) into shorter declarative statements or a table so each rule states one constraint.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body contains essentially no padding and no explanations of concepts Claude already knows — every sentence carries project-specific rules like "A test whose premise died is deleted whole in the commit that kills the premise", assuming high competence. It falls short of 5 because several passages (e.g., the Round Contract rebasing paragraph and the Long-Running Validation harness rows) are so clause-dense that they could be tightened or tabulated, and a few rules are restated across Round Contract and Validation. | 4 / 5 |
Actionability | Concrete, copy-paste-shaped commands with explicit placeholders are given throughout: ".agents/skills/orch/scripts/dev-validate-run --worktree [WORKTREE_PATH]", "git diff --no-renames --name-only [PREVIOUS_ROUND_COMMIT] HEAD -- <each loaded file's repo path>", and "bg_task action: \"spawn\" with notifyOnExit: true", plus explicit output interpretation ("state=timeout and state=lost are both failed validations"). It is not 5 because the two core workflows themselves are only named as arrow chains and delegated to workflow files whose steps are not shown in this body. | 4 / 5 |
Workflow Clarity | Sequences are stated explicitly ("activate → plan → implement → validate → commit → QA labels → summary → artifact → return (§ 1-11)") and validation is unusually explicit, including the run-notes question/answer protocol, interrupted-run recovery, and verdict routing for fail states. It stays at 4 rather than 5 because the detailed step-by-step execution lives in the referenced workflow files rather than here, and a few dense passages (the rebase-diff rule, the Engineering Rules exceptions) require re-reading to follow. | 4 / 5 |
Progressive Disclosure | Section organization is good and cross-references are clearly signaled with one-level-deep markdown links (e.g., "[dev-implement.md § 5. Validate](workflows/dev-implement.md#5-validate)", "[../orch/references/waiter-launch.md](../orch/references/waiter-launch.md)"), but the skill's own bundle files referenced as core content (workflows/dev-implement.md, workflows/dev-fix.md) are not present in the bundle, and ~40 lines of reference-grade operational detail (the per-harness Long-Running Validation rows, Round Contract channel mechanics) are inlined in SKILL.md. This lands on the 'content that should be separate is inline' side of the 3 anchor rather than the 'most content is appropriately placed' of 4. | 3 / 5 |
Total | 15 / 20 Passed |