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.
A highly actionable, well-sequenced orchestration workflow with concrete commands, decision tables, validation gates, and explicit recovery paths. Its main cost is density: repeated do-not-restate rules and two very long policy paragraphs make it heavier on the context window than the material requires.
Suggestions
Consolidate the repeated 'do not repeat select/check/next/reload' guidance into a single early rule (e.g., in Step 0) instead of restating it in Steps 0, 1, 2, and the closing section.
Move the evidence validity, input-scope, check-policy.json, and record-check rules from the exit-conditions section into a dedicated reference file (e.g., comet-classic/reference/build-evidence.md), keeping only the guard/check commands and the commit-after-guard ordering in SKILL.md.
Break the single-paragraph build_mode/tdd_mode/review_mode policy sentences into the existing tables or short bullets so each rule (e.g., 'Full autonomous requires standard or thorough') is scannable rather than embedded in multi-clause sentences.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly all content is novel operational policy (comet CLI semantics, evidence validity, phase transitions) that Claude would not know, avoiding score-2 territory, but it is 'mostly efficient... could be tightened': the anti-repetition rules ("Do not repeat next, select, or check", "do not reload", "do not repeat Verify's final review") recur in at least four sections, and the exit-conditions evidence paragraph is a ~400-word wall that buries key rules. It does not reach score 4 because several sections need restructuring or trimming to be efficient. | 3 / 5 |
Actionability | Commands are copy-paste ready with concrete example invocations — `comet state set <name> build_mode autonomous subagent_dispatch null tdd_mode tdd review_mode standard --json`, `comet guard <change-name> build --apply`, `comet check run <change-name> build --local -- <program> [args...]`, `comet state task-complete <name> <task-id> --expect <revision> --json` — plus decision tables (build_mode, tdd_mode/review_mode, scope Small/Medium/Large) covering common cases. This matches the 'fully executable; copy-paste ready... specific examples cover the common cases' anchor. | 5 / 5 |
Workflow Clarity | Steps 0-5 are clearly sequenced with explicit validation checkpoints (entry state check with BLOCKED/ERROR handling, guard --apply as a phase gate, an exit-conditions checklist), feedback loops for error recovery (invalid evidence names changed files and reruns only the listed scope; CRITICAL/IMPORTANT findings must be resolved; branch mismatch pauses under decision-point), and dedicated recovery sections. This matches the top anchor including its checklist and error-recovery criteria. | 5 / 5 |
Progressive Disclosure | The bundle has no references/scripts/assets of its own, but the body cleanly signals one-level-deep external references at point of need (classic-layout.md, context-recovery.md, decision-point.md, subagent-dispatch.md, debug-gate.md, dirty-worktree.md, auto-transition.md, scripts.md), and sections are well organized. It stays at 'good structure; minor organization gaps' rather than 5 because dense reference-grade policy (the evidence-input-scope, check-policy, and record-check paragraphs in the exit section) is inlined as unbroken walls of text that belong in a separate reference file. | 4 / 5 |
Total | 17 / 20 Passed |