Content
82%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 tightly written orchestrator skill: near-optimal token efficiency, concrete CLI commands, explicit decision-point and guard checkpoints, and well-signaled one-level-deep references. Its main weaknesses are details deferred to reference files that are missing from the bundle (intent-frame schema, task/knowledge command semantics) and an inconsistent, unverifiable reference path convention.
Suggestions
Inline a minimal CometIntentFrame field list or example in §1 so intent routing is actionable without the missing intent-frame.md.
Fix the inconsistent reference path in §2 (`reference/scripts.md#...` vs `comet-classic/reference/scripts.md` used everywhere else) and verify all 11 referenced files actually ship in the bundle.
Add a compact phase-exit checklist (exit checks pass → read `agent.continuation` → invoke only its next Skill) to consolidate the transition sequence currently spread across §3 and auto-transition.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense imperative instruction with no concept explanations and no padding — lines like "proposal records goals and scope; spec defines behavior and acceptance; design_doc points to the single formal technical design" compress an entire artifact model into one line. Every sentence is operational and nothing restates knowledge Claude already has, matching the 'lean and efficient; every token earns its place' anchor. | 5 / 5 |
Actionability | Concrete, copy-ready commands are given for the main paths (`comet classic workspace resolve <change-name> --json`, `comet state select`, `comet resume-probe . --stdin --json`, `comet state transition <name> preset-escalate`) plus a route→Skill dispatch table. It falls short of 5 because key executable details are deferred to reference files absent from this bundle — e.g. "fill a CometIntentFrame" and "run `comet task`" have no inline field list, schema, or output example. | 4 / 5 |
Workflow Clarity | A clear numbered sequence (resolve target/workspace → load phase → complete authorized work) with explicit checkpoints: phase Guards, "phase exit checks", decision points that "block dependent work until the user answers explicitly", preset escalation on changed-file thresholds, and recovery procedures. Not 5 because the control flow for any given situation is scattered across four sections and the compressed phrasing requires re-reading to reconstruct the order. | 4 / 5 |
Progressive Disclosure | The structure is exemplary for a complex skill: an overview that delegates ~11 one-level-deep reference files, each tied to an explicit read trigger ("read Ambient Resume in context-recovery.md", "read the chosen preset's escalation rules"). Docked one level because the referenced files do not exist in this bundle — the `comet-classic/reference/...` paths resolve nowhere and §2 inconsistently uses a different style (`[...](reference/scripts.md#task-context-and-artifact-language)`) — so navigation cannot be verified and the skill is unexecutable as shipped. | 4 / 5 |
Total | 17 / 20 Passed |