Content
77%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-engineered orchestration hub: lean, highly actionable routing rules with copy-paste CLI commands, an explicit decision-point checklist, and strong validation/error-recovery loops across the five phases. The weak point is progressive disclosure — the body correctly defers detail to reference files, but those files are absent from the provided bundle and the reference paths use two inconsistent conventions.
Suggestions
Ship the referenced files with the skill bundle and verify all ~11 paths resolve (classic-layout.md, context-recovery.md, intent-frame.md, workspace.md, scripts.md, auto-transition.md, decision-point.md, dirty-worktree.md, debug-gate.md, comet-yaml-fields.md, file-structure.md were all absent from the provided bundle).
Standardize on one reference path convention (the `comet-classic/reference/...` inline form vs. `reference/scripts.md#anchor` markdown links are mixed), and consider a short table of reference files with their trigger conditions near the top so navigation is scannable in one place.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational throughout: it never explains concepts Claude already knows and almost every sentence carries a routing rule, a condition, or a command. A few long multi-clause sentences (e.g., the workspace/isolation paragraph) could be tightened, matching anchor 4 (efficient; minor instances that could be trimmed) rather than 5 (every token earns its place). | 4 / 5 |
Actionability | Concrete, executable commands are given with full flags — `comet resume-probe . --stdin --json`, `comet classic workspace resolve <change-name> --json`, `comet state select`, `comet state next <name> --json`, `comet state transition <name> preset-escalate` — plus a bash block and explicit conditional rules ("只有 auto_resume 自动进入"). Minor gaps keep it at anchor 4: `comet task` and `comet knowledge remember` are invoked without syntax (deferred to reference/scripts.md), and several commands use placeholders the reader must substitute. | 4 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (numbered sections: determine workspace → load phase → continue work) with a phase routing table, an explicit checklist of blocking decision points ("以下确认不能…跳过"), validation gates (Guard checks, Verify's single final integration review, "只有实际运行过测试或构建,才能声称完成相应检查"), and error-recovery feedback loops (context-recovery, debug-gate's investigate-then-fix, state-update rejection handling). This matches anchor 5 (clear sequence, explicit validation, feedback loops, checklists); the destructive/batch cap does not apply since verification and authorization checks are explicitly required before archive/commit/push. | 5 / 5 |
Progressive Disclosure | The in-body discipline is strong — each of ~11 reference files is read only under an explicit trigger with a named section (e.g., "读取 context-recovery.md 的 'Ambient Resume'", "需要状态字段含义时读 comet-yaml-fields.md"), one level deep. However, scored against the actual bundle: no references/, scripts/, or assets/ directories are present, so none of the referenced files can be verified to exist, and two inconsistent path conventions are used (`comet-classic/reference/...` inline code vs. `[任务上下文与产物语言](reference/scripts.md#…)` markdown links). This lands at anchor 3 (structure present, but organization/navigation cannot be confirmed) rather than 4-5. | 3 / 5 |
Total | 16 / 20 Passed |