Skip to content
This repository was archived by the owner on Apr 11, 2026. It is now read-only.

Commit cdb5bff

Browse files
z23ccclaude
andcommitted
docs: update CLAUDE.md and docs to new sequential phase/step IDs
- CLAUDE.md: Step 1b→6, Phase 5b→11, Phase 2.5→6, Phase 5→10 - docs/flowctl.md: worker-phase sequences, canonical order, TDD/review IDs - docs/skill-anatomy.md: phase naming convention now integer-only Task: fn-104.6 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 34da701 commit cdb5bff

3 files changed

Lines changed: 17 additions & 17 deletions

File tree

CLAUDE.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ Rust: clippy for linting, cargo test for tests. No TypeScript, no npm. Skills an
8585
- **Review comparison**: `flowctl review-backend --compare <files>` or `--epic <id>` detects consensus/conflict across review receipts (auto-archived to `.flow/reviews/`)
8686
- **Domain tagging**: `flowctl task create --domain <domain>` tags tasks (frontend/backend/architecture/testing/docs/ops/general), filterable via `tasks --domain`
8787
- **Epic archival**: `flowctl epic archive <id>` moves closed epic + tasks + specs + reviews to `.flow/.archive/`; `flowctl epic clean` archives all closed epics at once
88-
- **Learning loop**: plan injects memory (Step 1b), worker saves lessons (Phase 5b, included in default sequence when memory.enabled is true), epic close prompts retro, retro verifies stale entries via `flowctl memory verify <id>`
88+
- **Learning loop**: plan injects memory (Step 6), worker saves lessons (Phase 11, included in default sequence when memory.enabled is true), epic close prompts retro, retro verifies stale entries via `flowctl memory verify <id>`
8989
- **Task duration**: `flowctl done` auto-tracks `duration_seconds` from start to completion, rendered in evidence
9090
- **File ownership**: `flowctl task create --files <paths>` declares owned files; `flowctl files --epic <id>` shows ownership map + conflict detection
9191
- **File locking (Teams)**: `flowctl lock --task <id> --files <paths>` acquires runtime file locks; `flowctl unlock --task <id>` releases on completion; `flowctl lock-check --file <path>` inspects lock state; `flowctl unlock --all` clears all locks between waves
@@ -95,9 +95,9 @@ Rust: clippy for linting, cargo test for tests. No TypeScript, no npm. Skills an
9595
- **Review circuit breaker**: impl-review fix loop capped at `MAX_REVIEW_ITERATIONS` (default 3) — prevents infinite NEEDS_WORK cycles
9696
- **Auto-improve analysis-driven**: generates custom program.md from codebase analysis (hotspots, lint, coverage, memory) with Action Catalog ranked by impact — not static templates
9797
- **Auto-improve quantitative**: captures before/after metrics per experiment, commit messages include delta `[lint:23→21]`
98-
- **Worker self-review**: Phase 2.5 runs guard + structured diff review (correctness, quality, performance, testing) before commit
98+
- **Worker self-review**: Phase 6 runs guard + structured diff review (correctness, quality, performance, testing) before commit
9999
- **Plan auto-execute**: `/flow-code:plan` defaults to auto-execute work after planning (Teams mode handles any task count); `--plan-only` to opt out
100-
- **Goal-backward verification**: worker Phase 5 re-reads acceptance criteria and verifies each is actually satisfied before completing
100+
- **Goal-backward verification**: worker Phase 10 re-reads acceptance criteria and verifies each is actually satisfied before completing
101101
- **Full-auto by default**: `/flow-code:plan` and `/flow-code:work` require zero interactive questions — AI reads git state, `.flow/` config, and request context to make branch, review, and research decisions autonomously. Default mode is Worktree + Teams + Phase-Gate (all three active). Work resumes from `.flow/` state on every startup (not a special "resume mode"). All tasks done → auto push + draft PR (`--no-pr` to skip)
102102
- **Cross-platform**: flowctl is a single Rust binary (macOS/Linux). RP plan-review auto-degrades to Codex on platforms where rp-cli is unavailable. Bash hooks degrade gracefully on Windows (skip, don't block)
103103
- **Session start**: CLAUDE.md instruction (not an enforced hook) — if `.flow/` exists, run `flowctl status --interrupted` to check for unfinished work from a previous session and resume with the suggested `/flow-code:work <id>` command

docs/flowctl.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,7 @@ flowctl guard --layer frontend [--json]
531531

532532
Exits non-zero if any guard fails. Output includes per-command pass/fail status.
533533

534-
Workers use this for baseline check (Phase 1) and verification (Phase 5) — one command replaces manual test/lint/typecheck invocations.
534+
Workers use this for baseline check (Phase 2) and verification (Phase 10) — one command replaces manual test/lint/typecheck invocations.
535535

536536
### invariants
537537

@@ -556,7 +556,7 @@ flowctl invariants check [--json]
556556
- **Fix:** [how to fix if violated]
557557
```
558558

559-
Workers check invariants in Phase 1 (baseline) and Phase 5 (verification). Planners check during Step 1 to ensure tasks don't violate constraints.
559+
Workers check invariants in Phase 2 (baseline) and Phase 10 (verification). Planners check during Step 4 to ensure tasks don't violate constraints.
560560

561561
### stack
562562

@@ -842,7 +842,7 @@ flowctl worker-prompt --task fn-1.1 --bootstrap [--tdd] [--review rp|codex] [--j
842842
Options:
843843
- `--task ID` (required): Task ID for context
844844
- `--bootstrap`: Output minimal ~200 token prompt that instructs the worker to call `worker-phase next` in a loop
845-
- `--tdd`: Include TDD Phase 2a in the prompt
845+
- `--tdd`: Include TDD Phase 4 in the prompt
846846
- `--review rp|codex`: Include review Phase 4
847847
- `--team`: Include Teams mode instructions (default in phase-gate)
848848
- `--json`: JSON output with `prompt` and `estimated_tokens` fields
@@ -859,10 +859,10 @@ flowctl worker-phase next --task fn-1.1 [--tdd] [--review rp|codex] --json
859859
flowctl worker-phase done --task fn-1.1 --phase <PHASE_ID> [--tdd] [--review rp|codex] --json
860860
```
861861

862-
**Default phase sequence**: `0 → 1 → 2 → 2.5 → 356`
863-
- With `--tdd`: adds Phase 2a (test-first)
864-
- With `--review`: adds Phase 4 (impl-review)
865-
- Canonical order: `0, 1, 2a, 2, 2.5, 3, 4, 5, 5b, 6`
862+
**Default phase sequence**: `1 → 2 → 5 → 6710 → 12`
863+
- With `--tdd`: adds Phase 4 (test-first)
864+
- With `--review`: adds Phase 8 (impl-review)
865+
- Canonical order: `1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12`
866866

867867
Phase progress is stored per-task in runtime state. `next` returns `{"phase": "<id>", "content": "...", "all_done": false}`. When all phases are complete, returns `{"phase": null, "all_done": true}`.
868868

docs/skill-anatomy.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ The heart of the skill. Step-by-step workflow the agent follows. Must be specifi
8181
**Good:** "Run `cargo test --all` and verify zero failures"
8282
**Bad:** "Make sure the tests work"
8383

84-
For flow-code skills, phases follow the convention: Phase 1, Phase 2, Phase 2.5 (verify), Phase 3 (commit), etc. Reference `flowctl` commands where applicable:
84+
For flow-code skills, phases follow the convention: Phase 1, Phase 2, Phase 3, etc. (always integers). Reference `flowctl` commands where applicable:
8585
```bash
8686
$FLOWCTL guard # Run all guards
8787
$FLOWCTL invariants check # Check architecture invariants
@@ -137,13 +137,13 @@ Flow-code workers produce evidence JSON. Skills should specify what evidence the
137137
### Phase Naming
138138
Follow the worker agent convention:
139139
- Phase 1: Re-anchor (read spec)
140-
- Phase 2: Implement
141-
- Phase 2.5: Verify & Fix
142-
- Phase 3: Commit
143-
- Phase 4: Review
144-
- Phase 5: Complete
140+
- Phase 5: Implement
141+
- Phase 6: Verify & Fix
142+
- Phase 7: Commit
143+
- Phase 8: Review
144+
- Phase 10: Complete
145145

146-
Skills that define their own phases should use this numbering style (Phase 1, Phase 1.5, Phase 2) for consistency with the worker pipeline.
146+
Phase IDs are always integers. Skills that define their own phases should use sequential integers (Phase 1, Phase 2, Phase 3) for consistency with the worker pipeline.
147147

148148
### Cross-Skill References
149149
Reference other skills by name, don't duplicate:

0 commit comments

Comments
 (0)