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.
The body is a dense but highly actionable phase workflow: exact CLI invocations, field-level validation, error-recovery feedback loops, and clearly signaled external references. Its main weakness is redundancy — recovery and re-check rules are repeated across sections, inflating token cost without adding guidance.
Suggestions
Consolidate the cold-recovery/binding rules (currently duplicated across §0a, Step 3, and 自动衔接下一阶段) into a single reference such as comet-classic/reference/auto-transition.md or a dedicated recovery section, keeping only a one-line pointer in each place.
Merge the §0c compatibility capability checks with Step 2's preflight list so each capability requirement (changeRoot, applyRequires, requires, resolvedOutputPath) is stated exactly once.
Extract the Step 2/3 dependency-closure generation loop (the numbered status/instructions/artifacts cycle and blocked handling) into a reference file, leaving the step-level sequence in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is operational (no teaching of known concepts) but noticeably redundant: the cold-recovery and binding rules of §0a are restated in Step 3 and the auto-transition section, "不重复 next、select 或 check" appears in at least four places, and the §0c compatibility checks overlap Step 2's preflight. This matches 'mostly efficient but could be tightened'; not 2 because there is no padding or concept explanation, and not 4 because the repetition is substantial rather than minor. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands with exact flags and placeholders ("comet state init <name> full --isolation <selected-isolation>", "comet classic openspec --agent-json -- status --change \"<name>\" --json"), plus concrete field-level checks (changeRoot, applyRequires, missingDeps, approvalHash) and explicit stop conditions. The common cases are covered end-to-end. | 5 / 5 |
Workflow Clarity | A clearly sequenced workflow (0, 0a–0c, 1, 1b, 2, 3, 4, 5) with explicit validation checkpoints — closure validation via "comet state artifacts <name> --json" after each artifact, entry-state verification via "comet state check <name> open", guard preflight with approvalHash — and feedback loops (re-validate on failure, re-preflight and re-confirm after content changes). Stop conditions name the exact blocked artifacts and missingDeps to report. | 5 / 5 |
Progressive Disclosure | Good structure: separate concerns (workspace.md, decision-point.md, auto-transition.md, batch-changes.md, sdk-methods.md) are delegated to clearly signaled one-level-deep references, and no local bundle files exist so nothing is misplaced. Not 5 because some references are bare code paths rather than links, and the dense Step 2/3 dependency-closure loop logic would fit better in a reference file; not 3 because the split between inline workflow and referenced detail is mostly appropriate. | 4 / 5 |
Total | 17 / 20 Passed |