@rpamis/comet 0.1.7 → 0.2.0

package/README.md

@@ -9,6 +9,8 @@
  ╚═════╝ ╚═════╝ ╚═╝     ╚═╝╚══════╝   ╚═╝
 ```
 
+> 中文版：[README-zh.md](README-zh.md)
+
 **OpenSpec + Superpowers dual-star development workflow** — one command from idea to archive.
 
 OpenSpec handles **WHAT** (outlines, proposals, spec lifecycle, archiving). Superpowers handles **HOW** (technical design, planning, execution, wrap-up). Comet chains both into a five-phase automated pipeline.
@@ -41,6 +43,9 @@ comet init
 | Command | Description |
 |---------|-------------|
 | `comet init [path]` | Initialize Comet workflow |
+| `comet status [path]` | Show active changes and workflow status |
+| `comet doctor [path]` | Diagnose Comet installation health |
+| `comet update [path]` | Update comet skills to latest version |
 | `comet --help` | Show help |
 | `comet --version` | Show version |
 
@@ -51,6 +56,15 @@ comet init
 | `--yes` | Non-interactive mode, auto-select detected platforms |
 | `--skip-existing` | Skip already installed components |
 | `--overwrite` | Overwrite already installed components |
+| `--json` | Output structured JSON |
+
+### status / doctor / update Options
+
+| Option | Applies to | Description |
+|--------|-----------|-------------|
+| `--json` | `status`, `doctor` | Output structured JSON |
+| `--language <lang>` | `update` | Language for skills (`en`, `zh`) |
+| `--scope <scope>` | `update` | Install scope (`global`, `project`)|
 
 ## Supported Platforms
 
@@ -90,12 +104,14 @@ After `comet init`, three groups of skills are installed to the selected platfor
 | `/comet-hotfix` | Preset: Quick bug fix (skips brainstorming) |
 | `/comet-tweak` | Preset: Small change (skips brainstorming and full plan) |
 
-### Guard Scripts
+### Guard & Automation Scripts
 
 | Script | Purpose |
 |--------|---------|
-| `comet-guard.sh` | Phase transition guard — validates exit conditions before phase transitions |
+| `comet-guard.sh` | Phase transition guard — validates exit conditions, `--apply` auto-updates `.comet.yaml` |
+| `comet-archive.sh` | One-command archive — validates state, syncs specs, moves to archive, updates status |
 | `comet-yaml-validate.sh` | Schema validator — validates `.comet.yaml` structure and field values |
+| `comet-state.sh` | Unified state management — init/set/get/check/scale, agents' exclusive YAML interface |
 
 ### OpenSpec Skills
 
@@ -110,14 +126,14 @@ Development methodology: brainstorming, TDD, subagent-driven development, code r
 ```
 /comet
   ↓ auto-detect
-/comet-open ——→ /comet-design ——→ /comet-build ——→ /comet-verify ——→ /comet-archive
-  (OpenSpec)      (Superpowers)     (Superpowers)     (Both)          (OpenSpec)
+/comet-open  -->  /comet-design  -->  /comet-build  -->  /comet-verify  -->  /comet-archive
+(OpenSpec)         (Superpowers)       (Superpowers)       (Both)           (OpenSpec)
 
 /comet-hotfix (preset path, skips brainstorming)
-  open ——→ build ——→ verify ——→ archive
+  open  -->  build  -->  verify  -->  archive
 
 /comet-tweak (preset path, skips brainstorming and full plan)
-  open ——→ lightweight build ——→ light verify ——→ archive
+  open  -->  lightweight build  -->  light verify  -->  archive
 ```
 
 ### Five Phases
@@ -160,15 +176,17 @@ Comet uses a decoupled state architecture with separate YAML files:
 
 ### Reliability Features
 
-Comet includes three-layer defense to ensure agent execution reliability:
+Comet ensures agent execution reliability through automated state transitions:
 
 1. **Entry Verification** — Each phase validates preconditions before execution
    - Checks file existence, state consistency, and phase transitions
    - Outputs `[HARD STOP]` with actionable suggestions if validation fails
 
-2. **Write-Then-Verify** — Every state write is immediately verified
-   - After updating `.comet.yaml`, agents must verify field values
-   - Automatic retry mechanism (up to 2 attempts) on mismatch
+2. **Automated State Transitions** — `comet-guard.sh --apply` updates `.comet.yaml` automatically
+   - All phase transitions (design → build → verify → archive) use `guard --apply`
+   - No manual state editing required — eliminates write-verification errors
+   - `comet-state.sh` is the agents' exclusive interface for state operations
+   - Guard and archive scripts use `comet-state.sh` internally for state management
 
 3. **Schema Validation** — `comet-yaml-validate.sh` ensures data integrity
    - Validates required fields (10 fields)
@@ -176,6 +194,12 @@ Comet includes three-layer defense to ensure agent execution reliability:
    - Validates referenced file paths exist
    - Detects unknown/typos fields
 
+4. **Archive Automation** — `comet-archive.sh` handles the full archive flow in one command
+   - Validates entry state, syncs delta specs to main specs
+   - Annotates design doc and plan frontmatter
+   - Moves change to archive directory and updates `archived: true`
+   - Supports `--dry-run` for preview
+
 **Security**: Path traversal protection on all change name inputs
 
 ## Project Structure
@@ -185,8 +209,10 @@ your-project/
 ├── .claude/skills/              # Platform skills dir (Comet + OpenSpec + Superpowers)
 │   ├── comet/SKILL.md
 │   │   └── scripts/
-│   │       ├── comet-guard.sh       # Phase transition guard
-│   │       └── comet-yaml-validate.sh # Schema validator
+│   │       ├── comet-guard.sh       # Phase transition guard (--apply auto-updates state)
+│   │       ├── comet-archive.sh     # One-command archive automation
+│   │       ├── comet-yaml-validate.sh # Schema validator
+│   │       └── comet-state.sh       # Unified state management (init/set/get/check/scale)
 │   ├── comet-*/SKILL.md
 │   ├── openspec-*/SKILL.md
 │   └── brainstorming/SKILL.md
@@ -221,8 +247,14 @@ pnpm dev
 # Build
 pnpm build
 
-# Test
+# Test (unit + coverage)
 pnpm test
+pnpm test:coverage
+pnpm test:shell         # bats shell tests
+
+# Lint & format
+pnpm lint
+pnpm format
 ```
 
 See [CHANGELOG.md](CHANGELOG.md) for version history and updates.

package/assets/manifest.json

@@ -3,6 +3,8 @@
   "skills": [
     "comet/SKILL.md",
     "comet/scripts/comet-guard.sh",
+    "comet/scripts/comet-state.sh",
+    "comet/scripts/comet-archive.sh",
     "comet/scripts/comet-yaml-validate.sh",
     "comet-open/SKILL.md",
     "comet-design/SKILL.md",

package/assets/skills/comet/SKILL.md

@@ -1,219 +1,214 @@
----
-name: comet
-description: "Comet — OpenSpec + Superpowers dual-star development workflow. Start with /comet for automatic phase detection and dispatch to subcommands. Five phases: open → design → build → verify → archive."
----
-
-# Comet — OpenSpec + Superpowers Dual-Star Development Workflow
-
-OpenSpec and Superpowers orbit the same goal like a binary star system. OpenSpec handles WHAT, Superpowers handles HOW.
-
-```
-OpenSpec handles WHAT  — outline, proposal, spec lifecycle, archive
-Superpowers handles HOW — technical design, planning, execution, closing
-```
-
-**Core principle: brainstorming cannot be skipped. Every change must undergo deep design (except hotfix and tweak presets).**
-
-<IMPORTANT>
-## Phase Transitions
-
-A single `/comet` invocation starts from the detected phase and prompts or advances to the next phase when exit conditions are met.
-
-Flow chain: open → design → build → verify → archive
-
-Nodes requiring user participation:
-1. Confirm design approach during brainstorming
-2. Select execution mode during build phase
-3. Decide to fix or accept deviation when verify fails
-4. Choose branch handling method for finishing-branch
-5. Encounter upgrade conditions (hotfix/tweak → full workflow)
-
-Agents should not skip these decision points; other unambiguous phase transitions can proceed automatically.
-</IMPORTANT>
-
-## Automatic Phase Detection
-
-### Step 0: Active Change Discovery
-
-**Execute immediately:**
-
-1. Run `openspec list --json` to get all active changes
-2. For each change, check `docs/superpowers/specs/` and `docs/superpowers/plans/` for associated files to determine phase and progress
-
-**Branch logic:**
-
-| Situation | Action |
-|-----------|--------|
-| No active change | → Invoke `/comet-open` |
-| Exactly 1 active change | → Auto-select, enter phase determination (Step 1) |
-| Multiple active changes | → List for user selection |
-
-**Multi-change selection**: Display using AskUserQuestion, example format:
-
-```
-| # | Change Name | Current Phase | Task Progress |
-|---|-------------|---------------|---------------|
-| 1 | xxx-feature | Build | 3/5 tasks |
-| 2 | yyy-fix | Design | No design doc |
-```
-
-After user selection, enter phase determination for the selected change.
-
-**Preset detection**:
-- If user explicitly describes as bug fix / hotfix and hotfix conditions are met, directly invoke `/comet-hotfix` (skip selection).
-- If user explicitly describes as copy, config, docs, prompt, or small non-bug adjustment and tweak conditions are met, directly invoke `/comet-tweak` (skip selection).
-
-### Step 1: Comet State Metadata Reading
-
-Prefer reading `openspec/changes/<name>/.comet.yaml`. If not available, fall back to `openspec status --change "<name>" --json`, `tasks.md`, and `docs/superpowers/` file checks.
-
-Recommended metadata structure:
-
-```yaml
-workflow: full
-phase: build
-design_doc: docs/superpowers/specs/YYYY-MM-DD-topic-design.md
-plan: docs/superpowers/plans/YYYY-MM-DD-feature.md
-build_mode: subagent-driven-development
-isolation: branch
-verify_mode: light
-verify_result: pending
-verified_at: null
-archived: false
-```
-
-Field meanings:
-
-| Field | Meaning |
-|-------|---------|
-| `workflow` | `full`, `hotfix`, or `tweak` |
-| `phase` | Current phase: `open`, `design`, `build`, `verify`, `archive` |
-| `design_doc` | Associated Superpowers Design Doc path, can be empty |
-| `plan` | Associated Superpowers Plan path, can be empty |
-| `build_mode` | Selected execution mode, can be empty |
-| `isolation` | `branch` or `worktree`, workspace isolation method, defaults to `branch` |
-| `verify_mode` | `light` or `full`, can be empty |
-| `verify_result` | `pending`, `pass`, or `fail` |
-| `verified_at` | Verification pass time, can be empty |
-| `archived` | Whether change is archived |
-
-### Step 2: Phase Determination
-
-For the selected change, determine current state in the following order:
-
-1. **`archived: true` or change moved to archive** → Workflow complete
-2. **`verify_result: pass` and `archived` is not `true`** → Invoke `/comet-archive`
-3. **`phase: verify` or tasks.md all checked** → Invoke `/comet-verify`
-4. **`phase: build` or has Design Doc but plan/execution incomplete** → Invoke `/comet-build`
-5. **`phase: design` or has change but no Design Doc** → Invoke `/comet-design`
-6. **No active change or state undeterminable** → Invoke `/comet-open`
-
-If metadata conflicts with file state, use verifiable file state as source of truth and correct `.comet.yaml` before continuing phase.
-
----
-
-## Subcommands
-
-| Command | Phase | Owner | Artifacts |
-|---------|-------|-------|-----------|
-| `/comet-open` | 1. Open | OpenSpec | proposal.md、design.md、tasks.md |
-| `/comet-design` | 2. Deep Design | Superpowers | Design Doc、delta spec |
-| `/comet-build` | 3. Plan & Build | Superpowers | Implementation plan、code commits |
-| `/comet-verify` | 4. Verify & Close | Both | Verification report、branch handling |
-| `/comet-archive` | 5. Archive | OpenSpec | delta→main spec sync、design doc markup、archive |
-| `/comet-hotfix` | Preset path | Both | Quick fix (skip brainstorming) |
-| `/comet-tweak` | Preset path | Both | Small change (skip brainstorming and full plan) |
-
----
-
-## Flow Diagram
-
-```
-/comet
-  ↓ Auto-detect
-/comet-open ──→ /comet-design ──→ /comet-build ──→ /comet-verify ──→ /comet-archive
-  (OpenSpec)      (Superpowers)     (Superpowers)     (Both)          (OpenSpec)
-
-/comet-hotfix（preset path, skip brainstorming）
-  open ──→ build ──→ verify ──→ archive
-    ↑ If upgrade triggered → supplement Design Doc → return to full workflow
-
-/comet-tweak（preset path, skip brainstorming and full plan）
-  open ──→ lightweight build ──→ light verify ──→ archive
-    ↑ If upgrade triggered → supplement Design Doc → return to full workflow
-```
-
----
-
-## Error Handling
-
-| Scenario | Handling |
-|----------|----------|
-| `openspec list --json` fails | Check if openspec is installed, prompt user to run `openspec init` |
-| Sub-skill unavailable (e.g., `superpowers:brainstorming`) | Stop workflow, prompt to install or enable corresponding skill |
-| `.comet.yaml` malformed or missing | Use file state as source of truth (tasks.md、docs/superpowers/), correct metadata then continue |
-| Maven compile/test fails | Return to build phase for fixes, do not enter verify |
-| Incomplete change directory structure | Fill missing files according to `comet-open` artifact requirements |
-
----
-
-## Quick Reference
-
-### Script Location
-
-Comet phase guard script `comet-guard.sh` and archive script `comet-archive.sh` are distributed with the skill package, located in `comet/scripts/` directory.
-**Do not hardcode platform paths**, self-locate at runtime with:
-
-```bash
-COMET_GUARD=$(find . -path '*/comet/scripts/comet-guard.sh' -type f -print -quit)
-bash "$COMET_GUARD" <change-name> <phase>
-```
-
-**Auto state update**: Guard supports `--apply` flag, automatically updating `.comet.yaml` state fields after checks pass:
-
-```bash
-bash "$COMET_GUARD" <change-name> <phase> --apply
-```
-
-**Archive script**: Complete all archive steps in one command:
-
-```bash
-COMET_ARCHIVE=$(find . -path '*/comet/scripts/comet-archive.sh' -type f -print -quit)
-bash "$COMET_ARCHIVE" <change-name>
-```
-
-In subsequent documentation, `bash $COMET_GUARD <change> <phase>` refers to this command. After loading comet, agents should cache `COMET_GUARD` path in shell environment to avoid repeated `find`.
-
-### File Structure
-
-```
-openspec/                              # OpenSpec — WHAT
-├── config.yaml
-├── changes/
-│   ├── <name>/                        # Active change
-│   │   ├── .openspec.yaml
-│   │   ├── .comet.yaml
-│   │   ├── proposal.md                # Why + What
-│   │   ├── design.md                  # High-level architecture decisions
-│   │   ├── specs/<capability>/spec.md # Delta capability spec
-│   │   └── tasks.md                   # Task checklist
-│   └── archive/YYYY-MM-DD-<name>/     # Archived
-└── specs/<capability>/spec.md         # Main specs (overwritten from delta at archive)
-
-docs/superpowers/                      # Superpowers — HOW
-├── specs/YYYY-MM-DD-<topic>-design.md # Design doc (technical RFC, mark status at archive)
-└── plans/YYYY-MM-DD-<feature>.md      # Implementation plan (file header contains change association metadata)
-```
-
-### Best Practices
-
-1. **brainstorming cannot be skipped** — Every change must undergo deep design (except hotfix and tweak)
-2. **delta spec is a living document** — Freely modify during phase 3, sync at archive
-3. **Keep tasks.md in sync** — Check off each completed task
-4. **Commit frequently** — One commit per task, message reflects design intent
-5. **Verify before archive** — Execute `/comet-archive` only after `/comet-verify` passes
-6. **Classify incremental updates** — Small edits, medium brainstorming, large new changes
-7. **Plan must associate with change** — File header contains `change:` and `design-doc:` metadata
-8. **Archive closure** — design doc and plan must mark `archived-with` status
-9. **Modifying existing features** — Just open a new change; brainstorming reads existing specs as context naturally
-10. **Preset has limits** — Switch to full workflow promptly when hotfix/tweak meet upgrade conditions
+---
+name: comet
+description: "Comet — OpenSpec + Superpowers dual-star development workflow. Start with /comet for automatic phase detection and dispatch to subcommands. Five phases: open → design → build → verify → archive."
+---
+
+# Comet — OpenSpec + Superpowers Dual-Star Development Workflow
+
+OpenSpec and Superpowers orbit the same goal like a binary star system. OpenSpec handles WHAT, Superpowers handles HOW.
+
+```
+OpenSpec handles WHAT  — outline, proposal, spec lifecycle, archive
+Superpowers handles HOW — technical design, planning, execution, closing
+```
+
+**Core principle: brainstorming cannot be skipped. Every change must undergo deep design (except hotfix and tweak presets).**
+
+---
+
+## Decision Core
+
+Agents need only read this section for decision-making. Refer to the Reference Appendix as needed.
+
+### Automatic Phase Detection
+
+**Step 0: Active Change Discovery**
+
+1. Run `openspec list --json` to get all active changes
+2. For each change, check `docs/superpowers/specs/` and `docs/superpowers/plans/` for associated files to determine phase and progress
+
+| Situation | Action |
+|-----------|--------|
+| No active change | → Invoke `/comet-open` |
+| Exactly 1 active change | → Auto-select, enter Step 1 |
+| Multiple active changes | → List for user selection |
+
+**Preset detection**:
+- User describes as bug fix / hotfix + meets hotfix conditions → directly invoke `/comet-hotfix`
+- User describes as copy, config, docs, prompt or small non-bug adjustment + meets tweak conditions → directly invoke `/comet-tweak`
+
+**Step 1: Read `.comet.yaml` state metadata**
+
+Prefer reading `openspec/changes/<name>/.comet.yaml`. If not available, fall back to `openspec status --change "<name>" --json`, `tasks.md`, and `docs/superpowers/` file checks.
+
+**Step 2: Phase Determination** (check in order, first match wins)
+
+1. `archived: true` or change moved to archive → Workflow complete
+2. `verify_result: pass` and `archived` is not `true` → Invoke `/comet-archive`
+3. `phase: verify` or tasks.md all checked → Invoke `/comet-verify`
+4. `phase: build` or has Design Doc but plan/execution incomplete → Invoke `/comet-build`
+5. `phase: design` or has change but no Design Doc → Invoke `/comet-design`
+6. No active change or state undeterminable → Invoke `/comet-open`
+
+If metadata conflicts with file state, use verifiable file state as source of truth and correct `.comet.yaml` before continuing.
+
+### Preset Upgrade Criteria
+
+**hotfix → full** (upgrade if any condition met):
+- Change involves **3+ files**
+- Architecture changes (new modules, new interfaces, new dependencies)
+- Database schema changes
+- Fix introduces new public API
+- Fix scope exceeds a single function/module
+
+**tweak → full** (upgrade if any condition met):
+- Change involves **5+ files**
+- Cross-module coordination required
+- **5+** new test cases needed
+- Config item additions or deletions (not value changes)
+
+### Error Handling Quick Reference
+
+| Scenario | Handling |
+|----------|----------|
+| `openspec list --json` fails | Check if openspec is installed, prompt user to run `openspec init` |
+| Sub-skill unavailable | Stop workflow, prompt to install or enable the corresponding skill |
+| `.comet.yaml` malformed or missing | Use file state as source of truth, correct with `bash $COMET_STATE set` then continue |
+| Maven compile/test fails | Return to build phase for fixes, do not enter verify |
+| Incomplete change directory structure | Fill missing files according to `comet-open` artifact requirements |
+
+### Phase Transitions
+
+<IMPORTANT>
+A single `/comet` invocation starts from the detected phase and advances to the next phase when exit conditions are met.
+
+Flow chain: open → design → build → verify → archive
+
+Nodes requiring user participation:
+1. Confirm design approach during brainstorming
+2. Select execution mode during build phase
+3. Decide to fix or accept deviation when verify fails
+4. Choose branch handling method for finishing-branch
+5. Encounter upgrade conditions (hotfix/tweak → full workflow)
+
+Agents should not skip these decision points; other unambiguous phase transitions can proceed automatically.
+</IMPORTANT>
+
+---
+
+## Subcommand Quick Reference
+
+| Command | Phase | Owner | Artifacts |
+|---------|-------|-------|-----------|
+| `/comet-open` | 1. Open | OpenSpec | proposal.md, design.md, tasks.md |
+| `/comet-design` | 2. Deep Design | Superpowers | Design Doc, delta spec |
+| `/comet-build` | 3. Plan and Build | Superpowers | Implementation plan, code commits |
+| `/comet-verify` | 4. Verify and Close | Both | Verification report, branch handling |
+| `/comet-archive` | 5. Archive | OpenSpec | delta→main spec sync, design doc markup, archive |
+| `/comet-hotfix` | Preset path | Both | Quick fix (skip brainstorming) |
+| `/comet-tweak` | Preset path | Both | Small change (skip brainstorming and full plan) |
+
+```
+/comet
+  ↓ Auto-detect
+/comet-open ──→ /comet-design ──→ /comet-build ──→ /comet-verify ──→ /comet-archive
+  (OpenSpec)      (Superpowers)     (Superpowers)     (Both)          (OpenSpec)
+
+/comet-hotfix (preset, skip brainstorming)
+  open ──→ build ──→ verify ──→ archive
+    ↑ If upgrade triggered → supplement Design Doc → return to full workflow
+
+/comet-tweak (preset, skip brainstorming and full plan)
+  open ──→ lightweight build ──→ light verify ──→ archive
+    ↑ If upgrade triggered → supplement Design Doc → return to full workflow
+```
+
+---
+
+## Reference Appendix
+
+### .comet.yaml Field Reference
+
+```yaml
+workflow: full
+phase: build
+design_doc: docs/superpowers/specs/YYYY-MM-DD-topic-design.md
+plan: docs/superpowers/plans/YYYY-MM-DD-feature.md
+build_mode: subagent-driven-development
+isolation: branch
+verify_mode: light
+verify_result: pending
+verified_at: null
+archived: false
+```
+
+| Field | Meaning |
+|-------|---------|
+| `workflow` | `full`, `hotfix`, or `tweak` |
+| `phase` | Current phase: `open`, `design`, `build`, `verify`, `archive` |
+| `design_doc` | Associated Superpowers Design Doc path, can be empty |
+| `plan` | Associated Superpowers Plan path, can be empty |
+| `build_mode` | Selected execution mode, can be empty |
+| `isolation` | `branch` or `worktree`, workspace isolation method, defaults to `branch` |
+| `verify_mode` | `light` or `full`, can be empty |
+| `verify_result` | `pending`, `pass`, or `fail` |
+| `verified_at` | Verification pass time, can be empty |
+| `archived` | Whether change is archived |
+
+### Script Location
+
+Comet scripts are distributed in `comet/scripts/`. **Do not hardcode paths** — locate once, cache in env vars:
+
+```bash
+COMET_GUARD="${COMET_GUARD:-$(find . -path '*/comet/scripts/comet-guard.sh' -type f -print -quit)}"
+COMET_STATE="${COMET_STATE:-$(find . -path '*/comet/scripts/comet-state.sh' -type f -print -quit)}"
+COMET_ARCHIVE="${COMET_ARCHIVE:-$(find . -path '*/comet/scripts/comet-archive.sh' -type f -print -quit)}"
+```
+
+**Auto state update**: Guard supports `--apply` flag, automatically updating `.comet.yaml` state fields after checks pass:
+
+```bash
+bash "$COMET_GUARD" <change-name> <phase> --apply
+```
+
+**Archive script**: Complete all archive steps in one command:
+
+```bash
+bash "$COMET_ARCHIVE" <change-name>
+```
+
+After loading comet, agents should run the three variable assignments above once, then reuse `$COMET_GUARD`, `$COMET_STATE`, `$COMET_ARCHIVE` throughout the session.
+
+### File Structure
+
+```
+openspec/                              # OpenSpec — WHAT
+├── config.yaml
+├── changes/
+│   ├── <name>/                        # Active change
+│   │   ├── .openspec.yaml
+│   │   ├── .comet.yaml
+│   │   ├── proposal.md                # Why + What
+│   │   ├── design.md                  # High-level architecture decisions
+│   │   ├── specs/<capability>/spec.md # Delta capability spec
+│   │   └── tasks.md                   # Task checklist
+│   └── archive/YYYY-MM-DD-<name>/     # Archived
+└── specs/<capability>/spec.md         # Main specs (overwritten from delta at archive)
+
+docs/superpowers/                      # Superpowers — HOW
+├── specs/YYYY-MM-DD-<topic>-design.md # Design doc (technical RFC, mark status at archive)
+└── plans/YYYY-MM-DD-<feature>.md      # Implementation plan (file header contains change association metadata)
+```
+
+### Best Practices
+
+1. **brainstorming cannot be skipped** — Every change must undergo deep design (except hotfix and tweak)
+2. **delta spec is a living document** — Freely modify during phase 3, sync at archive
+3. **Keep tasks.md in sync** — Check off each completed task
+4. **Commit frequently** — One commit per task, message reflects design intent
+5. **Verify before archive** — Execute `/comet-archive` only after `/comet-verify` passes
+6. **Classify incremental updates** — Small edits, medium brainstorming, large new changes
+7. **Plan must associate with change** — File header contains `change:` and `design-doc:` metadata
+8. **Archive closure** — design doc and plan must mark `archived-with` status
+9. **Modifying existing features** — Just open a new change
+10. **Preset has limits** — Switch to full workflow promptly when hotfix/tweak meet upgrade conditions

package/assets/skills/comet/scripts/comet-archive.sh

@@ -37,6 +37,8 @@ validate_change_name "$CHANGE"
 
 CHANGE_DIR="openspec/changes/$CHANGE"
 YAML="$CHANGE_DIR/.comet.yaml"
+SCRIPT_DIR="$(cd "$(dirname "$(readlink -f "$0" 2>/dev/null || echo "$0")" 2>/dev/null || dirname "$0")" && pwd)"
+STATE_SH="$SCRIPT_DIR/comet-state.sh"
 TODAY=$(date +%Y-%m-%d)
 ARCHIVE_NAME="${TODAY}-${CHANGE}"
 ARCHIVE_DIR="openspec/changes/archive/${ARCHIVE_NAME}"
@@ -61,8 +63,12 @@ echo "=== Comet Archive: $CHANGE ===" >&2
 
 yaml_field() {
   local field="$1"
-  if [ -f "$YAML" ]; then
-    grep "^${field}:" "$YAML" | sed "s/^${field}: *//" | tr -d '"' | tr -d "'"
+  if [ -f "$STATE_SH" ]; then
+    bash "$STATE_SH" get "$CHANGE" "$field" 2>/dev/null
+  else
+    if [ -f "$YAML" ]; then
+      grep "^${field}:" "$YAML" | sed "s/^${field}: *//" | tr -d '"' | tr -d "'"
+    fi
   fi
 }