Work Execution Command
Execute work efficiently while maintaining quality and finishing features.
Introduction
This command takes a work document (plan or specification) or a bare prompt describing the work, and executes it systematically. The focus is on shipping complete features by understanding requirements quickly, following existing patterns, and maintaining quality throughout.
Beta rollout note: Invoke ce-work-beta manually when you want to trial Codex delegation. During the beta period, planning and workflow handoffs remain pointed at stable ce-work to avoid dual-path orchestration complexity.
Input Document
#$ARGUMENTS
Argument Parsing
Parse $ARGUMENTS for the following optional tokens. Strip each recognized token before interpreting the remainder as the plan file path or bare prompt.
| Token |
Example |
Effect |
delegate:codex |
delegate:codex |
Activate Codex delegation mode for plan execution |
delegate:local |
delegate:local |
Deactivate delegation even if enabled in config |
All tokens are optional. When absent, fall back to the resolution chain below.
Fuzzy activation: Also recognize imperative delegation-intent phrases such as "use codex", "delegate to codex", "codex mode", or "delegate mode" as equivalent to delegate:codex. A bare mention of "codex" in a prompt (e.g., "fix codex converter bugs") must NOT activate delegation -- only clear delegation intent triggers it.
Fuzzy deactivation: Also recognize phrases such as "no codex", "local mode", "standard mode" as equivalent to delegate:local.
Settings Resolution Chain
After extracting tokens from arguments, resolve the delegation state using this precedence chain:
- Argument flag --
delegate:codex or delegate:local from the current invocation (highest priority)
- Config file -- extract settings from the config block below. Value
codex for work_delegate activates delegation; false deactivates.
- Hard default --
false (delegation off)
Config (pre-resolved):
!cat "$(git rev-parse --show-toplevel 2>/dev/null)/.compound-engineering/config.local.yaml" 2>/dev/null || echo '__NO_CONFIG__'
If the block above contains YAML key-value pairs, extract values for the keys listed below.
If it shows __NO_CONFIG__, the file does not exist — all settings fall through to defaults.
If it shows an unresolved command string, read .compound-engineering/config.local.yaml from the repo root using the native file-read tool (e.g., Read in Claude Code, read_file in Codex). If the file does not exist, all settings fall through to defaults.
If any setting has an unrecognized value, fall through to the hard default for that setting. For optional settings without a hard default (work_delegate_model, work_delegate_effort), an unrecognized or unparseable value resolves to unset — the corresponding flag is omitted from the codex exec invocation so Codex resolves from ~/.codex/config.toml. Never substitute an invalid value into the CLI flags.
Config keys:
work_delegate -- codex or default false
work_delegate_consent -- true or default false
work_delegate_sandbox -- yolo (default) or full-auto
work_delegate_decision -- auto (default) or ask
work_delegate_model -- Codex model to use. Optional — when unset or unparseable, defers to the user's ~/.codex/config.toml default. Passthrough — any non-empty string is accepted as valid; only YAML parse failures or empty values resolve to unset.
work_delegate_effort -- one of minimal, low, medium, high, or xhigh. Optional — when unset or set to a value outside this enum, resolves to unset and defers to the user's ~/.codex/config.toml default.
Store the resolved state for downstream consumption:
delegation_active -- boolean, whether delegation mode is on
delegation_source -- argument or config or default -- how delegation was resolved (used by environment guard to decide notification verbosity)
sandbox_mode -- yolo or full-auto (from config or default yolo)
consent_granted -- boolean (from config work_delegate_consent)
delegate_model -- string from config, or unset (defer to Codex config)
delegate_effort -- string from config, or unset (defer to Codex config). Floor for per-batch effort selection; not passed directly to codex exec.
effective_effort -- per-batch derived value (default | medium | high | xhigh), computed before each batch from delegate_effort and the picked level per references/codex-delegation-workflow.md ("Per-Batch Effort"). Feeds the codex exec invocation in place of delegate_effort.
Execution Workflow
Phase 0: Input Triage
Determine how to proceed based on what was provided in <input_document>.
Plan document (input is a file path to an existing plan or specification): read the plan's metadata first — YAML frontmatter for a markdown plan, or the visible header text for an HTML plan (both formats carry the same fields). If it carries execution: knowledge-work, this is a non-code plan — read references/non-code-execution.md and follow that carve-out instead of the rest of this workflow. Otherwise (the field is absent or execution: code) → skip to Phase 1 and run the normal code lifecycle. (The marker check lives here, inside plan-document handling, because detecting the marker requires already having a file; "Bare prompt" below is unaffected.)
Bare prompt (input is a description of work, not a file path):
Scan the work area
- Identify files likely to change based on the prompt
- Find existing test files for those areas (search for test/spec files that import, reference, or share names with the implementation files)
- Note local patterns and conventions in the affected areas
Assess complexity and route
| Complexity |
Signals |
Action |
| Trivial |
1-2 files, no behavioral change (typo, config, rename) |
Proceed to Phase 1 step 2 (environment setup), then implement directly — no task list, no execution loop. Apply Test Discovery if the change touches behavior-bearing code |
| Small / Medium |
Clear scope, under ~10 files |
Build a task list from discovery. Proceed to Phase 1 step 2 |
| Large |
Cross-cutting, architectural decisions, 10+ files, touches auth/payments/migrations |
Inform the user this would benefit from /ce-brainstorm or /ce-plan to surface edge cases and scope boundaries. Honor their choice. If proceeding, build a task list and continue to Phase 1 step 2 |
Phase 1: Quick Start
Read Plan and Clarify (skip if arriving from Phase 0 with a bare prompt)
- Read the work document completely
- Treat the plan as a decision artifact, not an execution script
- If the plan includes sections such as
Implementation Units, Work Breakdown, Requirements (or legacy Requirements Trace), Files, Test Scenarios, or Verification, use those as the primary source material for execution
- Check for
Execution note on each implementation unit — these carry the plan's execution posture signal for that unit (for example, test-first or characterization-first). Note them when creating tasks.
- Check for a
Deferred to Implementation or Implementation-Time Unknowns section — these are questions the planner intentionally left for you to resolve during execution. Note them before starting so they inform your approach rather than surprising you mid-task
- Check for a
Scope Boundaries section — these are explicit non-goals. Refer back to them if implementation starts pulling you toward adjacent work
- Review any references or links provided in the plan
- If the user explicitly asks for TDD, test-first, or characterization-first execution in this session, honor that request even if the plan has no
Execution note
- If anything is unclear or ambiguous, ask clarifying questions now
- If clarifying questions were needed above, get user approval on the resolved answers. If no clarifications were needed, proceed without a separate approval step — plan scope is the plan's authority, not something to renegotiate
- Do not skip this - better to ask questions now than build the wrong thing
- Do not edit the plan body during execution. The plan is a decision artifact; progress lives in git commits and the task tracker, not the plan.
ce-work does not mutate the plan — whether it shipped is derived from git, not recorded in the doc. Legacy plans may contain - [ ] / - [x] marks on unit headings or a status: field — ignore them as state; per-unit completion is determined during execution by reading the current file state.
Setup Environment
First, check the current branch:
current_branch=$(git branch --show-current)
default_branch=$(git symbolic-ref refs/remotes/origin/HEAD 2>/dev/null | sed 's@^refs/remotes/origin/@@')
# Fallback if remote HEAD isn't set
if [ -z "$default_branch" ]; then
default_branch=$(git rev-parse --verify origin/main >/dev/null 2>&1 && echo "main" || echo "master")
fi
If already on a feature branch (not the default branch):
First, check whether the branch name is meaningful — a name like feat/crowd-sniff or fix/email-validation tells future readers what the work is about. Auto-generated worktree names (e.g., worktree-jolly-beaming-raven) or other opaque names do not.
If the branch name is meaningless or auto-generated, suggest renaming it before continuing:
git branch -m <meaningful-name>
Derive the new name from the plan title or work description (e.g., feat/crowd-sniff). Present the rename as a recommended option alongside continuing as-is.
Then ask: "Continue working on [current_branch], or create a new branch?"
- If continuing (with or without rename), proceed to step 3
- If creating new, follow Option A or B below
If on the default branch, choose how to proceed:
Option A: Create a new branch
git pull origin [default_branch]
git checkout -b feature-branch-name
Use a meaningful name based on the work (e.g., feat/user-authentication, fix/email-validation).
Option B: Use a worktree (recommended for parallel development)
skill: ce-worktree
# The skill will create a new branch from the default branch in an isolated worktree
Option C: Continue on the default branch
- Requires explicit user confirmation
- Only proceed after user explicitly says "yes, commit to [default_branch]"
- Never commit directly to the default branch without explicit permission
Recommendation: Use worktree if:
- You want to work on multiple features simultaneously
- You want to keep the default branch clean while experimenting
- You plan to switch between branches frequently
Create Task List (skip if Phase 0 already built one, or if Phase 0 routed as Trivial)
- Use the platform's task tracking tool (
TaskCreate/TaskUpdate/TaskList in Claude Code, update_plan in Codex, or the equivalent on other harnesses) to break the plan into actionable tasks
- Derive tasks from the plan's implementation units, dependencies, files, test targets, and verification criteria
- When the plan defines U-IDs for Implementation Units, preserve the unit's U-ID as a prefix in the task subject (e.g., "U3: Add parser coverage"). This keeps blocker references, deferred-work notes, and final summaries anchored to the same identifier the plan uses, so progress and traceability remain unambiguous across plan edits
- Carry each unit's
Execution note into the task when present
- For each unit, read the
Patterns to follow field before implementing — these point to specific files or conventions to mirror
- Use each unit's
Verification field as the primary "done" signal for that task
- Do not expect the plan to contain implementation code, micro-step TDD instructions, or exact shell commands
- Include dependencies between tasks
- Prioritize based on what needs to be done first
- Include testing and quality check tasks
- Keep tasks specific and completable
Choose Execution Strategy
Delegation routing gate: If delegation_active is true AND the input is a plan file (not a bare prompt), read references/codex-delegation-workflow.md and follow its Pre-Delegation Checks and Delegation Decision flow. If all checks pass and delegation proceeds, force serial execution and proceed directly to Phase 2 using the workflow's batched execution loop. If any check disables delegation, fall through to the standard strategy table below. If delegation is active but the input is a bare prompt (no plan file), set delegation_active to false with a brief note: "Codex delegation requires a plan file -- using standard mode." and continue with the standard strategy selection below.
After creating the task list, decide how to execute based on the plan's size and dependency structure:
| Strategy |
When to use |
| Inline |
1-2 small tasks, or tasks needing user interaction mid-flight. Default for bare-prompt work — bare prompts rarely produce enough structured context to justify subagent dispatch |
| Serial subagents |
3+ tasks with dependencies between them. Each subagent gets a fresh context window focused on one unit — prevents context degradation across many tasks. Requires plan-unit metadata (Goal, Files, Approach, Test scenarios) |
| Parallel subagents |
3+ tasks that pass the Parallel Safety Check (below). Dispatch independent units simultaneously, run dependent units after their prerequisites complete. Requires plan-unit metadata |
Parallel Safety Check — required before choosing parallel dispatch:
- Build a file-to-unit mapping from every candidate unit's
Files: section (Create, Modify, and Test paths)
- Check for intersection — any file path appearing in 2+ units means overlap
- If overlap is found AND worktree isolation is unavailable: downgrade to serial subagents. Log the reason (e.g., "Units 2 and 4 share
config/routes.rb — using serial dispatch"). Serial subagents still provide context-window isolation without shared-directory write races.
- If overlap is found AND worktree isolation is available: parallel dispatch is still safe — subagents work in isolation, and the overlap surfaces as a predictable merge conflict the orchestrator handles via the post-batch flow below. Log the predicted overlap so the post-batch flow knows which merges to expect conflicts on.
Even with no file overlap, parallel subagents sharing the orchestrator's working directory face git index contention (concurrent staging/committing corrupts the index) and test interference (concurrent test runs pick up each other's in-progress changes). Worktree isolation eliminates both; the shared-directory fallback constraints below mitigate them.
Subagent isolation — give each parallel subagent its own working tree:
- Claude Code (
Agent tool): pass isolation: "worktree" and run_in_background: true. The harness creates a per-subagent worktree under .claude/worktrees/agent-<id> on its own branch. Verify .claude/worktrees/ is gitignored before relying on this.
- Other platforms without built-in worktree isolation (e.g., Codex
spawn_agent, Pi subagent): subagents share the orchestrator's directory.
Subagent dispatch uses your available subagent or task spawning mechanism. For each unit, give the subagent:
- The full plan file path (for overall context)
- The specific unit's Goal, Files, Approach, Execution note, Patterns, Test scenarios, and Verification
- Any resolved deferred questions relevant to that unit
- Instruction to check whether the unit's test scenarios cover all applicable categories (happy paths, edge cases, error paths, integration) and supplement gaps before writing tests
Shared-directory fallback constraints — apply only when worktree isolation is unavailable:
- Instruct each subagent: "Do not stage files (
git add), create commits, or run the project test suite. The orchestrator handles testing, staging, and committing after all parallel units complete."
- These constraints prevent git index contention and test interference between concurrent subagents.
- With worktree isolation active, omit these constraints — subagents may stage, commit, and run their unit's tests within their own worktree branch.
Permission mode: Omit the mode parameter when dispatching subagents so the user's configured permission settings apply. Do not pass mode: "auto" — it overrides user-level settings like bypassPermissions.
After each subagent completes (serial mode):
- Review the subagent's diff — verify changes match the unit's scope and
Files: list
- Run the relevant test suite to confirm the tree is healthy
- If tests fail, diagnose and fix before proceeding — do not dispatch dependent units on a broken tree
- Update the task list (do not edit the plan body — progress is carried by the commit)
- Dispatch the next unit
After all parallel subagents in a batch complete (worktree-isolated mode):
- Wait for every subagent in the current parallel batch to finish.
- For each completed subagent, in dependency order: review the worktree's diff against the orchestrator's branch. If the subagent did not commit its own work, stage and commit it inside that worktree.
- Merge each subagent's branch into the orchestrator's branch sequentially in dependency order. If a merge conflict surfaces, abort the merge (
git merge --abort) and re-dispatch the conflicting unit serially against the now-merged tree — hand-resolving silently picks a side and discards one unit's intent. (Predicted overlap from the Parallel Safety Check surfaces here as a conflict, not as silent data loss in shared-directory mode.)
- After each merge, run the relevant test suite. If tests fail, diagnose and fix before merging the next branch.
- Update the task list (progress is carried by the merge commits).
- After merging, remove each subagent's worktree and delete its branch. Use the absolute path and branch name returned in the subagent's result.
- Unlock the worktree first — the harness locks per-subagent worktrees:
git worktree unlock <absolute-path>
- Remove the worktree:
git worktree remove <absolute-path>
- Delete the branch:
git branch -d <branch-name> (the branch outlives the worktree by default and accumulates as orphans if not cleaned up; -d lowercase refuses to delete unmerged branches, which is the safety we want — if it fails, investigate before forcing)
- Dispatch the next batch of independent units, or the next dependent unit.
After all parallel subagents in a batch complete (shared-directory fallback):
- Wait for every subagent in the current parallel batch to finish before acting on any of their results
- Cross-check for discovered file collisions: compare the actual files modified by all subagents in the batch (not just their declared
Files: lists). Subagents may create or modify files not anticipated during planning — this is expected, since plans describe what not how. A collision only matters when 2+ subagents in the same batch modified the same file. In a shared working directory, only the last writer's version survives — the other unit's changes to that file are lost. If a collision is detected: commit all non-colliding files from all units first, then re-run the affected units serially for the shared file so each builds on the other's committed work
- For each completed unit, in dependency order: review the diff, run the relevant test suite, stage only that unit's files, and commit with a conventional message derived from the unit's Goal
- If tests fail after committing a unit's changes, diagnose and fix before committing the next unit
- Update the task list (do not edit the plan body — progress is carried by the commits just made)
- Dispatch the next batch of independent units, or the next dependent unit
Phase 2: Execute
Task Execution Loop
For each task in priority order:
while (tasks remain):
- Mark task as in-progress
- Read any referenced files from the plan or discovered during Phase 0
- **If the unit's work is already present and matches the plan's intent** (files exist with the expected capability, or the unit's `Verification` criteria are already satisfied by the current code), the work has likely shipped on a prior branch or session. Verify it matches, mark the task complete, and move on. Do not silently reimplement.
- Look for similar patterns in codebase
- Find existing test files for implementation files being changed (Test Discovery — see below)
- If delegation_active: branch to the Codex Delegation Execution Loop
(see `references/codex-delegation-workflow.md`)
- Otherwise: implement following existing conventions
- Add, update, or remove tests to match implementation changes (see Test Discovery below)
- Run System-Wide Test Check (see below)
- Run tests after changes
- Assess testing coverage: did this task change behavior? If yes, were tests written or updated? If no tests were added, is the justification deliberate (e.g., pure config, no behavioral change)?
- Mark task as completed
- Evaluate for incremental commit (see below)
When a unit carries an Execution note, honor it. For test-first units, write the failing test before implementation for that unit. For characterization-first units, capture existing behavior before changing it. For units without an Execution note, proceed pragmatically.
Guardrails for execution posture:
- Do not write the test and implementation in the same step when working test-first
- Do not skip verifying that a new test fails before implementing the fix or feature
- Do not over-implement beyond the current behavior slice when working test-first
- Skip test-first discipline for trivial renames, pure configuration, and pure styling work
Test Discovery — Before implementing changes to a file, find its existing test files (search for test/spec files that import, reference, or share naming patterns with the implementation file). When a plan specifies test scenarios or test files, start there, then check for additional test coverage the plan may not have enumerated. Changes to implementation files should be accompanied by corresponding test updates — new tests for new behavior, modified tests for changed behavior, removed or updated tests for deleted behavior.
Test Scenario Completeness — Before writing tests for a feature-bearing unit, check whether the plan's Test scenarios cover all categories that apply to this unit. If a category is missing or scenarios are vague (e.g., "validates correctly" without naming inputs and expected outcomes), supplement from the unit's own context before writing tests:
| Category |
When it applies |
How to derive if missing |
| Happy path |
Always for feature-bearing units |
Read the unit's Goal and Approach for core input/output pairs |
| Edge cases |
When the unit has meaningful boundaries (inputs, state, concurrency) |
Identify boundary values, empty/nil inputs, and concurrent access patterns |
| Error/failure paths |
When the unit has failure modes (validation, external calls, permissions) |
Enumerate invalid inputs the unit should reject, permission/auth denials it should enforce, and downstream failures it should handle |
| Integration |
When the unit crosses layers (callbacks, middleware, multi-service) |
Identify the cross-layer chain and write a scenario that exercises it without mocks |
System-Wide Test Check — Before marking a task done, pause and ask:
| Question |
What to do |
| What fires when this runs? Callbacks, middleware, observers, event handlers — trace two levels out from your change. |
Read the actual code (not docs) for callbacks on models you touch, middleware in the request chain, after_* hooks. |
| Do my tests exercise the real chain? If every dependency is mocked, the test proves your logic works in isolation — it says nothing about the interaction. |
Write at least one integration test that uses real objects through the full callback/middleware chain. No mocks for the layers that interact. |
| Can failure leave orphaned state? If your code persists state (DB row, cache, file) before calling an external service, what happens when the service fails? Does retry create duplicates? |
Trace the failure path with real objects. If state is created before the risky call, test that failure cleans up or that retry is idempotent. |
| What other interfaces expose this? Mixins, DSLs, alternative entry points (Agent vs Chat vs ChatMethods). |
Grep for the method/behavior in related classes. If parity is needed, add it now — not as a follow-up. |
| Do error strategies align across layers? Retry middleware + application fallback + framework error handling — do they conflict or create double execution? |
List the specific error classes at each layer. Verify your rescue list matches what the lower layer actually raises. |
When to skip: Leaf-node changes with no callbacks, no state persistence, no parallel interfaces. If the change is purely additive (new helper method, new view partial), the check takes 10 seconds and the answer is "nothing fires, skip."
When this matters most: Any change that touches models with callbacks, error handling with fallback/retry, or functionality exposed through multiple interfaces.
Incremental Commits
After completing each task, evaluate whether to create an incremental commit:
| Commit when... |
Don't commit when... |
| Logical unit complete (model, service, component) |
Small part of a larger unit |
| Tests pass + meaningful progress |
Tests failing |
| About to switch contexts (backend → frontend) |
Purely scaffolding with no behavior |
| About to attempt risky/uncertain changes |
Would need a "WIP" commit message |
Heuristic: "Can I write a commit message that describes a complete, valuable change? If yes, commit. If the message would be 'WIP' or 'partial X', wait."
If the plan has Implementation Units, use them as a starting guide for commit boundaries — but adapt based on what you find during implementation. A unit might need multiple commits if it's larger than expected, or small related units might land together. Use each unit's Goal to inform the commit message.
Commit workflow:
# 1. Verify tests pass (use project's test command)
# Examples: bin/rails test, npm test, pytest, go test, etc.
# 2. Stage only files related to this logical unit (not `git add .`)
git add <files related to this logical unit>
# 3. Commit with conventional message
git commit -m "feat(scope): description of this unit"
Handling merge conflicts: If conflicts arise during rebasing or merging, resolve them immediately. Incremental commits make conflict resolution easier since each commit is small and focused.
Note: Incremental commits use clean conventional messages without attribution footers. The final Phase 4 commit/PR includes the full attribution.
Parallel subagent mode: Commit ownership is split by isolation mode (see Phase 1 Step 4):
- Worktree-isolated: subagents may stage and commit inside their own worktree branch; the orchestrator merges those branches in dependency order after the batch.
- Shared-directory fallback: subagents do not commit; the orchestrator stages and commits each unit after the entire parallel batch completes.
Follow Existing Patterns
- The plan should reference similar code - read those files first
- Match naming conventions exactly
- Reuse existing components where possible
- Follow project coding standards (see AGENTS.md; use CLAUDE.md only if the repo still keeps a compatibility shim)
- When in doubt, grep for similar implementations
Test Continuously
- Run relevant tests after each significant change
- Don't wait until the end to test
- Fix failures immediately
- Add new tests for new behavior, update tests for changed behavior, remove tests for deleted behavior
- Unit tests with mocks prove logic in isolation. Integration tests with real objects prove the layers work together. If your change touches callbacks, middleware, or error handling — you need both.
Simplify as You Go
After completing a cluster of related implementation units (or every 2-3 units), review recently changed files for simplification opportunities — consolidate duplicated patterns, extract shared helpers, and improve code reuse and efficiency. This is especially valuable when using subagents, since each agent works with isolated context and can't see patterns emerging across units.
Don't simplify after every single unit — early patterns may look duplicated but diverge intentionally in later units. Wait for a natural phase boundary or when you notice accumulated complexity.
If ce-simplify-code is available, invoke it at phase boundaries (especially before Phase 3 when the diff is >=30 lines). Otherwise, review the changed files yourself for reuse and consolidation opportunities.
Figma Design Sync (if applicable)
For UI work with Figma designs:
- Implement components following design specs
- Use ce-figma-design-sync agent iteratively to compare
- Fix visual differences identified
- Repeat until implementation matches design
Frontend Design Guidance (if applicable)
For UI tasks without a Figma design -- where the implementation touches view, template, component, layout, or page files, creates user-visible routes, or the plan contains explicit UI/frontend/design language:
- Load the
ce-frontend-design skill before implementing
- Follow its detection, guidance, and verification flow
- If the skill produced a verification screenshot, it satisfies Phase 4's screenshot requirement -- no need to capture separately. If the skill fell back to mental review (no browser access), Phase 4's screenshot capture still applies
Track Progress
- Keep the task list updated as you complete tasks
- Note any blockers or unexpected discoveries
- Create new tasks if scope expands
- Keep user informed of major milestones
- When the plan defines U-IDs for Implementation Units, or the plan or origin document carries stable R-IDs (and optionally A/F/AE IDs), reference them in blockers, deferred-work notes, task summaries, and final verification — not routine status updates. U-IDs anchor units across plan edits; R/A/F/AE anchor product intent across the brainstorm-plan handoff. Use the IDs the plan supplies and do not invent ones it does not. This preserves traceability without burying signal under noise.
Phase 3-4: Quality Check and Finishing Work
When all Phase 2 tasks are complete and execution transitions to quality check, read references/shipping-workflow.md for the full shipping workflow: quality checks, code review, final validation, PR creation, and notification.
Codex Delegation Mode
When delegation_active is true after argument parsing, read references/codex-delegation-workflow.md for the complete delegation workflow: pre-checks, batching, prompt template, execution loop, and result classification.
Key Principles
Start Fast, Execute Faster
- Get clarification once at the start, then execute
- Don't wait for perfect understanding - ask questions and move
- The goal is to finish the feature, not create perfect process
The Plan is Your Guide
- Work documents should reference similar code and patterns
- Load those references and follow them
- Don't reinvent - match what exists
Test As You Go
- Run tests after each change, not at the end
- Fix failures immediately
- Continuous testing prevents big surprises
Quality is Built In
- Follow existing patterns
- Write tests for new code
- Run linting before pushing
- Review when Tier 1 is available or Tier 2 criteria match (see
shipping-workflow.md)
Ship Complete Features
- Mark all tasks completed before moving on
- Don't leave features 80% done
- A finished feature that ships beats a perfect feature that doesn't
Common Pitfalls to Avoid
- Analysis paralysis - Don't overthink, read the plan and execute
- Skipping clarifying questions - Ask now, not after building wrong thing
- Ignoring plan references - The plan has links for a reason
- Testing at the end - Test continuously or suffer later
- Forgetting to track progress - Update task status as you go or lose track of what's done
- 80% done syndrome - Finish the feature, don't move on early
- Skipping review without reason — Use Tier 1 when available; escalate to Tier 2 only on criteria in
shipping-workflow.md; document when both are skipped
- Re-scoping the plan into human-time phases - The plan's Implementation Units define the scope of execution. Do not estimate human-hours per unit, propose multi-day breakdowns, or ask the user to pick a subset of units for "this session". Agents execute at agent speed, and context-window pressure is addressed by subagent dispatch (Phase 1 Step 4), not by phased sessions. If a plan-file input is genuinely too large for a single execution, say so plainly and suggest the user return to
/ce-plan to reduce scope — don't invent session phases as a workaround. For bare-prompt input, Phase 0's Large routing already handles oversized work
1---2name: ce-work-beta3description: [BETA] Execute work with external delegate support. Same as ce-work but includes experimental Codex delegation mode for token-conserving code implementation.4---56# Work Execution Command78Execute work efficiently while maintaining quality and finishing features.910## Introduction1112This command takes a work document (plan or specification) or a bare prompt describing the work, and executes it systematically. The focus is on **shipping complete features** by understanding requirements quickly, following existing patterns, and maintaining quality throughout.1314**Beta rollout note:** Invoke `ce-work-beta` manually when you want to trial Codex delegation. During the beta period, planning and workflow handoffs remain pointed at stable `ce-work` to avoid dual-path orchestration complexity.1516## Input Document1718<input_document> #$ARGUMENTS </input_document>1920## Argument Parsing2122Parse `$ARGUMENTS` for the following optional tokens. Strip each recognized token before interpreting the remainder as the plan file path or bare prompt.2324| Token | Example | Effect |25|-------|---------|--------|26| `delegate:codex` | `delegate:codex` | Activate Codex delegation mode for plan execution |27| `delegate:local` | `delegate:local` | Deactivate delegation even if enabled in config |2829All tokens are optional. When absent, fall back to the resolution chain below.3031**Fuzzy activation:** Also recognize imperative delegation-intent phrases such as "use codex", "delegate to codex", "codex mode", or "delegate mode" as equivalent to `delegate:codex`. A bare mention of "codex" in a prompt (e.g., "fix codex converter bugs") must NOT activate delegation -- only clear delegation intent triggers it.3233**Fuzzy deactivation:** Also recognize phrases such as "no codex", "local mode", "standard mode" as equivalent to `delegate:local`.3435### Settings Resolution Chain3637After extracting tokens from arguments, resolve the delegation state using this precedence chain:38391. **Argument flag** -- `delegate:codex` or `delegate:local` from the current invocation (highest priority)402. **Config file** -- extract settings from the config block below. Value `codex` for `work_delegate` activates delegation; `false` deactivates.413. **Hard default** -- `false` (delegation off)4243**Config (pre-resolved):**44!`cat "$(git rev-parse --show-toplevel 2>/dev/null)/.compound-engineering/config.local.yaml" 2>/dev/null || echo '__NO_CONFIG__'`4546If the block above contains YAML key-value pairs, extract values for the keys listed below.47If it shows `__NO_CONFIG__`, the file does not exist — all settings fall through to defaults.48If it shows an unresolved command string, read `.compound-engineering/config.local.yaml` from the repo root using the native file-read tool (e.g., Read in Claude Code, read_file in Codex). If the file does not exist, all settings fall through to defaults.4950If any setting has an unrecognized value, fall through to the hard default for that setting. For optional settings without a hard default (`work_delegate_model`, `work_delegate_effort`), an unrecognized or unparseable value resolves to **unset** — the corresponding flag is omitted from the `codex exec` invocation so Codex resolves from `~/.codex/config.toml`. Never substitute an invalid value into the CLI flags.5152Config keys:53- `work_delegate` -- `codex` or default `false`54- `work_delegate_consent` -- `true` or default `false`55- `work_delegate_sandbox` -- `yolo` (default) or `full-auto`56- `work_delegate_decision` -- `auto` (default) or `ask`57- `work_delegate_model` -- Codex model to use. Optional — when unset or unparseable, defers to the user's `~/.codex/config.toml` default. Passthrough — any non-empty string is accepted as valid; only YAML parse failures or empty values resolve to unset.58- `work_delegate_effort` -- one of `minimal`, `low`, `medium`, `high`, or `xhigh`. Optional — when unset or set to a value outside this enum, resolves to unset and defers to the user's `~/.codex/config.toml` default.5960Store the resolved state for downstream consumption:61- `delegation_active` -- boolean, whether delegation mode is on62- `delegation_source` -- `argument` or `config` or `default` -- how delegation was resolved (used by environment guard to decide notification verbosity)63- `sandbox_mode` -- `yolo` or `full-auto` (from config or default `yolo`)64- `consent_granted` -- boolean (from config `work_delegate_consent`)65- `delegate_model` -- string from config, or unset (defer to Codex config)66- `delegate_effort` -- string from config, or unset (defer to Codex config). Floor for per-batch effort selection; not passed directly to `codex exec`.67- `effective_effort` -- per-batch derived value (`default | medium | high | xhigh`), computed before each batch from `delegate_effort` and the picked level per `references/codex-delegation-workflow.md` ("Per-Batch Effort"). Feeds the `codex exec` invocation in place of `delegate_effort`.6869---7071## Execution Workflow7273### Phase 0: Input Triage7475Determine how to proceed based on what was provided in `<input_document>`.7677**Plan document** (input is a file path to an existing plan or specification): read the plan's metadata first — YAML frontmatter for a markdown plan, or the visible header text for an HTML plan (both formats carry the same fields). If it carries `execution: knowledge-work`, this is a **non-code plan** — read `references/non-code-execution.md` and follow that carve-out instead of the rest of this workflow. Otherwise (the field is absent or `execution: code`) → skip to Phase 1 and run the normal code lifecycle. (The marker check lives here, inside plan-document handling, because detecting the marker requires already having a file; "Bare prompt" below is unaffected.)7879**Bare prompt** (input is a description of work, not a file path):80811. **Scan the work area**8283 - Identify files likely to change based on the prompt84 - Find existing test files for those areas (search for test/spec files that import, reference, or share names with the implementation files)85 - Note local patterns and conventions in the affected areas86872. **Assess complexity and route**8889 | Complexity | Signals | Action |90 |-----------|---------|--------|91 | **Trivial** | 1-2 files, no behavioral change (typo, config, rename) | Proceed to Phase 1 step 2 (environment setup), then implement directly — no task list, no execution loop. Apply Test Discovery if the change touches behavior-bearing code |92 | **Small / Medium** | Clear scope, under ~10 files | Build a task list from discovery. Proceed to Phase 1 step 2 |93 | **Large** | Cross-cutting, architectural decisions, 10+ files, touches auth/payments/migrations | Inform the user this would benefit from `/ce-brainstorm` or `/ce-plan` to surface edge cases and scope boundaries. Honor their choice. If proceeding, build a task list and continue to Phase 1 step 2 |9495---9697### Phase 1: Quick Start98991. **Read Plan and Clarify** _(skip if arriving from Phase 0 with a bare prompt)_100101 - Read the work document completely102 - Treat the plan as a decision artifact, not an execution script103 - If the plan includes sections such as `Implementation Units`, `Work Breakdown`, `Requirements` (or legacy `Requirements Trace`), `Files`, `Test Scenarios`, or `Verification`, use those as the primary source material for execution104 - Check for `Execution note` on each implementation unit — these carry the plan's execution posture signal for that unit (for example, test-first or characterization-first). Note them when creating tasks.105 - Check for a `Deferred to Implementation` or `Implementation-Time Unknowns` section — these are questions the planner intentionally left for you to resolve during execution. Note them before starting so they inform your approach rather than surprising you mid-task106 - Check for a `Scope Boundaries` section — these are explicit non-goals. Refer back to them if implementation starts pulling you toward adjacent work107 - Review any references or links provided in the plan108 - If the user explicitly asks for TDD, test-first, or characterization-first execution in this session, honor that request even if the plan has no `Execution note`109 - If anything is unclear or ambiguous, ask clarifying questions now110 - If clarifying questions were needed above, get user approval on the resolved answers. If no clarifications were needed, proceed without a separate approval step — plan scope is the plan's authority, not something to renegotiate111 - **Do not skip this** - better to ask questions now than build the wrong thing112 - **Do not edit the plan body during execution.** The plan is a decision artifact; progress lives in git commits and the task tracker, not the plan. `ce-work` does not mutate the plan — whether it shipped is derived from git, not recorded in the doc. Legacy plans may contain `- [ ]` / `- [x]` marks on unit headings or a `status:` field — ignore them as state; per-unit completion is determined during execution by reading the current file state.1131142. **Setup Environment**115116 First, check the current branch:117118 ```bash119 current_branch=$(git branch --show-current)120 default_branch=$(git symbolic-ref refs/remotes/origin/HEAD 2>/dev/null | sed 's@^refs/remotes/origin/@@')121122 # Fallback if remote HEAD isn't set123 if [ -z "$default_branch" ]; then124 default_branch=$(git rev-parse --verify origin/main >/dev/null 2>&1 && echo "main" || echo "master")125 fi126 ```127128 **If already on a feature branch** (not the default branch):129130 First, check whether the branch name is **meaningful** — a name like `feat/crowd-sniff` or `fix/email-validation` tells future readers what the work is about. Auto-generated worktree names (e.g., `worktree-jolly-beaming-raven`) or other opaque names do not.131132 If the branch name is meaningless or auto-generated, suggest renaming it before continuing:133 ```bash134 git branch -m <meaningful-name>135 ```136 Derive the new name from the plan title or work description (e.g., `feat/crowd-sniff`). Present the rename as a recommended option alongside continuing as-is.137138 Then ask: "Continue working on `[current_branch]`, or create a new branch?"139 - If continuing (with or without rename), proceed to step 3140 - If creating new, follow Option A or B below141142 **If on the default branch**, choose how to proceed:143144 **Option A: Create a new branch**145 ```bash146 git pull origin [default_branch]147 git checkout -b feature-branch-name148 ```149 Use a meaningful name based on the work (e.g., `feat/user-authentication`, `fix/email-validation`).150151 **Option B: Use a worktree (recommended for parallel development)**152 ```bash153 skill: ce-worktree154 # The skill will create a new branch from the default branch in an isolated worktree155 ```156157 **Option C: Continue on the default branch**158 - Requires explicit user confirmation159 - Only proceed after user explicitly says "yes, commit to [default_branch]"160 - Never commit directly to the default branch without explicit permission161162 **Recommendation**: Use worktree if:163 - You want to work on multiple features simultaneously164 - You want to keep the default branch clean while experimenting165 - You plan to switch between branches frequently1661673. **Create Task List** _(skip if Phase 0 already built one, or if Phase 0 routed as Trivial)_168 - Use the platform's task tracking tool (`TaskCreate`/`TaskUpdate`/`TaskList` in Claude Code, `update_plan` in Codex, or the equivalent on other harnesses) to break the plan into actionable tasks169 - Derive tasks from the plan's implementation units, dependencies, files, test targets, and verification criteria170 - When the plan defines U-IDs for Implementation Units, preserve the unit's U-ID as a prefix in the task subject (e.g., "U3: Add parser coverage"). This keeps blocker references, deferred-work notes, and final summaries anchored to the same identifier the plan uses, so progress and traceability remain unambiguous across plan edits171 - Carry each unit's `Execution note` into the task when present172 - For each unit, read the `Patterns to follow` field before implementing — these point to specific files or conventions to mirror173 - Use each unit's `Verification` field as the primary "done" signal for that task174 - Do not expect the plan to contain implementation code, micro-step TDD instructions, or exact shell commands175 - Include dependencies between tasks176 - Prioritize based on what needs to be done first177 - Include testing and quality check tasks178 - Keep tasks specific and completable1791804. **Choose Execution Strategy**181182 **Delegation routing gate:** If `delegation_active` is true AND the input is a plan file (not a bare prompt), read `references/codex-delegation-workflow.md` and follow its Pre-Delegation Checks and Delegation Decision flow. If all checks pass and delegation proceeds, force **serial execution** and proceed directly to Phase 2 using the workflow's batched execution loop. If any check disables delegation, fall through to the standard strategy table below. If delegation is active but the input is a bare prompt (no plan file), set `delegation_active` to false with a brief note: "Codex delegation requires a plan file -- using standard mode." and continue with the standard strategy selection below.183184 After creating the task list, decide how to execute based on the plan's size and dependency structure:185186 | Strategy | When to use |187 |----------|-------------|188 | **Inline** | 1-2 small tasks, or tasks needing user interaction mid-flight. **Default for bare-prompt work** — bare prompts rarely produce enough structured context to justify subagent dispatch |189 | **Serial subagents** | 3+ tasks with dependencies between them. Each subagent gets a fresh context window focused on one unit — prevents context degradation across many tasks. Requires plan-unit metadata (Goal, Files, Approach, Test scenarios) |190 | **Parallel subagents** | 3+ tasks that pass the Parallel Safety Check (below). Dispatch independent units simultaneously, run dependent units after their prerequisites complete. Requires plan-unit metadata |191192 **Parallel Safety Check** — required before choosing parallel dispatch:193194 1. Build a file-to-unit mapping from every candidate unit's `Files:` section (Create, Modify, and Test paths)195 2. Check for intersection — any file path appearing in 2+ units means overlap196 3. **If overlap is found AND worktree isolation is unavailable**: downgrade to serial subagents. Log the reason (e.g., "Units 2 and 4 share `config/routes.rb` — using serial dispatch"). Serial subagents still provide context-window isolation without shared-directory write races.197 4. **If overlap is found AND worktree isolation is available**: parallel dispatch is still safe — subagents work in isolation, and the overlap surfaces as a predictable merge conflict the orchestrator handles via the post-batch flow below. Log the predicted overlap so the post-batch flow knows which merges to expect conflicts on.198199 Even with no file overlap, parallel subagents sharing the orchestrator's working directory face git index contention (concurrent staging/committing corrupts the index) and test interference (concurrent test runs pick up each other's in-progress changes). Worktree isolation eliminates both; the shared-directory fallback constraints below mitigate them.200201 **Subagent isolation** — give each parallel subagent its own working tree:202 - **Claude Code (`Agent` tool):** pass `isolation: "worktree"` and `run_in_background: true`. The harness creates a per-subagent worktree under `.claude/worktrees/agent-<id>` on its own branch. Verify `.claude/worktrees/` is gitignored before relying on this.203 - **Other platforms** without built-in worktree isolation (e.g., Codex `spawn_agent`, Pi `subagent`): subagents share the orchestrator's directory.204205 **Subagent dispatch** uses your available subagent or task spawning mechanism. For each unit, give the subagent:206 - The full plan file path (for overall context)207 - The specific unit's Goal, Files, Approach, Execution note, Patterns, Test scenarios, and Verification208 - Any resolved deferred questions relevant to that unit209 - Instruction to check whether the unit's test scenarios cover all applicable categories (happy paths, edge cases, error paths, integration) and supplement gaps before writing tests210211 **Shared-directory fallback constraints** — apply only when worktree isolation is unavailable:212 - Instruct each subagent: "Do not stage files (`git add`), create commits, or run the project test suite. The orchestrator handles testing, staging, and committing after all parallel units complete."213 - These constraints prevent git index contention and test interference between concurrent subagents.214 - With worktree isolation active, omit these constraints — subagents may stage, commit, and run their unit's tests within their own worktree branch.215216 **Permission mode:** Omit the `mode` parameter when dispatching subagents so the user's configured permission settings apply. Do not pass `mode: "auto"` — it overrides user-level settings like `bypassPermissions`.217218 **After each subagent completes (serial mode):**219 1. Review the subagent's diff — verify changes match the unit's scope and `Files:` list220 2. Run the relevant test suite to confirm the tree is healthy221 3. If tests fail, diagnose and fix before proceeding — do not dispatch dependent units on a broken tree222 4. Update the task list (do not edit the plan body — progress is carried by the commit)223 5. Dispatch the next unit224225 **After all parallel subagents in a batch complete (worktree-isolated mode):**226 1. Wait for every subagent in the current parallel batch to finish.227 2. For each completed subagent, in dependency order: review the worktree's diff against the orchestrator's branch. If the subagent did not commit its own work, stage and commit it inside that worktree.228 3. Merge each subagent's branch into the orchestrator's branch sequentially in dependency order. **If a merge conflict surfaces, abort the merge (`git merge --abort`) and re-dispatch the conflicting unit serially against the now-merged tree** — hand-resolving silently picks a side and discards one unit's intent. (Predicted overlap from the Parallel Safety Check surfaces here as a conflict, not as silent data loss in shared-directory mode.)229 4. After each merge, run the relevant test suite. If tests fail, diagnose and fix before merging the next branch.230 5. Update the task list (progress is carried by the merge commits).231 6. After merging, remove each subagent's worktree and delete its branch. Use the absolute path and branch name returned in the subagent's result.232 - Unlock the worktree first — the harness locks per-subagent worktrees: `git worktree unlock <absolute-path>`233 - Remove the worktree: `git worktree remove <absolute-path>`234 - Delete the branch: `git branch -d <branch-name>` (the branch outlives the worktree by default and accumulates as orphans if not cleaned up; `-d` lowercase refuses to delete unmerged branches, which is the safety we want — if it fails, investigate before forcing)235 7. Dispatch the next batch of independent units, or the next dependent unit.236237 **After all parallel subagents in a batch complete (shared-directory fallback):**238 1. Wait for every subagent in the current parallel batch to finish before acting on any of their results239 2. Cross-check for discovered file collisions: compare the actual files modified by all subagents in the batch (not just their declared `Files:` lists). Subagents may create or modify files not anticipated during planning — this is expected, since plans describe *what* not *how*. A collision only matters when 2+ subagents in the same batch modified the same file. In a shared working directory, only the last writer's version survives — the other unit's changes to that file are lost. If a collision is detected: commit all non-colliding files from all units first, then re-run the affected units serially for the shared file so each builds on the other's committed work240 3. For each completed unit, in dependency order: review the diff, run the relevant test suite, stage only that unit's files, and commit with a conventional message derived from the unit's Goal241 4. If tests fail after committing a unit's changes, diagnose and fix before committing the next unit242 5. Update the task list (do not edit the plan body — progress is carried by the commits just made)243 6. Dispatch the next batch of independent units, or the next dependent unit244245### Phase 2: Execute2462471. **Task Execution Loop**248249 For each task in priority order:250251 ```252 while (tasks remain):253 - Mark task as in-progress254 - Read any referenced files from the plan or discovered during Phase 0255 - **If the unit's work is already present and matches the plan's intent** (files exist with the expected capability, or the unit's `Verification` criteria are already satisfied by the current code), the work has likely shipped on a prior branch or session. Verify it matches, mark the task complete, and move on. Do not silently reimplement.256 - Look for similar patterns in codebase257 - Find existing test files for implementation files being changed (Test Discovery — see below)258 - If delegation_active: branch to the Codex Delegation Execution Loop259 (see `references/codex-delegation-workflow.md`)260 - Otherwise: implement following existing conventions261 - Add, update, or remove tests to match implementation changes (see Test Discovery below)262 - Run System-Wide Test Check (see below)263 - Run tests after changes264 - Assess testing coverage: did this task change behavior? If yes, were tests written or updated? If no tests were added, is the justification deliberate (e.g., pure config, no behavioral change)?265 - Mark task as completed266 - Evaluate for incremental commit (see below)267 ```268269 When a unit carries an `Execution note`, honor it. For test-first units, write the failing test before implementation for that unit. For characterization-first units, capture existing behavior before changing it. For units without an `Execution note`, proceed pragmatically.270271 Guardrails for execution posture:272 - Do not write the test and implementation in the same step when working test-first273 - Do not skip verifying that a new test fails before implementing the fix or feature274 - Do not over-implement beyond the current behavior slice when working test-first275 - Skip test-first discipline for trivial renames, pure configuration, and pure styling work276277 **Test Discovery** — Before implementing changes to a file, find its existing test files (search for test/spec files that import, reference, or share naming patterns with the implementation file). When a plan specifies test scenarios or test files, start there, then check for additional test coverage the plan may not have enumerated. Changes to implementation files should be accompanied by corresponding test updates — new tests for new behavior, modified tests for changed behavior, removed or updated tests for deleted behavior.278279 **Test Scenario Completeness** — Before writing tests for a feature-bearing unit, check whether the plan's `Test scenarios` cover all categories that apply to this unit. If a category is missing or scenarios are vague (e.g., "validates correctly" without naming inputs and expected outcomes), supplement from the unit's own context before writing tests:280281 | Category | When it applies | How to derive if missing |282 |----------|----------------|------------------------|283 | **Happy path** | Always for feature-bearing units | Read the unit's Goal and Approach for core input/output pairs |284 | **Edge cases** | When the unit has meaningful boundaries (inputs, state, concurrency) | Identify boundary values, empty/nil inputs, and concurrent access patterns |285 | **Error/failure paths** | When the unit has failure modes (validation, external calls, permissions) | Enumerate invalid inputs the unit should reject, permission/auth denials it should enforce, and downstream failures it should handle |286 | **Integration** | When the unit crosses layers (callbacks, middleware, multi-service) | Identify the cross-layer chain and write a scenario that exercises it without mocks |287288 **System-Wide Test Check** — Before marking a task done, pause and ask:289290 | Question | What to do |291 |----------|------------|292 | **What fires when this runs?** Callbacks, middleware, observers, event handlers — trace two levels out from your change. | Read the actual code (not docs) for callbacks on models you touch, middleware in the request chain, `after_*` hooks. |293 | **Do my tests exercise the real chain?** If every dependency is mocked, the test proves your logic works *in isolation* — it says nothing about the interaction. | Write at least one integration test that uses real objects through the full callback/middleware chain. No mocks for the layers that interact. |294 | **Can failure leave orphaned state?** If your code persists state (DB row, cache, file) before calling an external service, what happens when the service fails? Does retry create duplicates? | Trace the failure path with real objects. If state is created before the risky call, test that failure cleans up or that retry is idempotent. |295 | **What other interfaces expose this?** Mixins, DSLs, alternative entry points (Agent vs Chat vs ChatMethods). | Grep for the method/behavior in related classes. If parity is needed, add it now — not as a follow-up. |296 | **Do error strategies align across layers?** Retry middleware + application fallback + framework error handling — do they conflict or create double execution? | List the specific error classes at each layer. Verify your rescue list matches what the lower layer actually raises. |297298 **When to skip:** Leaf-node changes with no callbacks, no state persistence, no parallel interfaces. If the change is purely additive (new helper method, new view partial), the check takes 10 seconds and the answer is "nothing fires, skip."299300 **When this matters most:** Any change that touches models with callbacks, error handling with fallback/retry, or functionality exposed through multiple interfaces.3013023032. **Incremental Commits**304305 After completing each task, evaluate whether to create an incremental commit:306307 | Commit when... | Don't commit when... |308 |----------------|---------------------|309 | Logical unit complete (model, service, component) | Small part of a larger unit |310 | Tests pass + meaningful progress | Tests failing |311 | About to switch contexts (backend → frontend) | Purely scaffolding with no behavior |312 | About to attempt risky/uncertain changes | Would need a "WIP" commit message |313314 **Heuristic:** "Can I write a commit message that describes a complete, valuable change? If yes, commit. If the message would be 'WIP' or 'partial X', wait."315316 If the plan has Implementation Units, use them as a starting guide for commit boundaries — but adapt based on what you find during implementation. A unit might need multiple commits if it's larger than expected, or small related units might land together. Use each unit's Goal to inform the commit message.317318 **Commit workflow:**319 ```bash320 # 1. Verify tests pass (use project's test command)321 # Examples: bin/rails test, npm test, pytest, go test, etc.322323 # 2. Stage only files related to this logical unit (not `git add .`)324 git add <files related to this logical unit>325326 # 3. Commit with conventional message327 git commit -m "feat(scope): description of this unit"328 ```329330 **Handling merge conflicts:** If conflicts arise during rebasing or merging, resolve them immediately. Incremental commits make conflict resolution easier since each commit is small and focused.331332 **Note:** Incremental commits use clean conventional messages without attribution footers. The final Phase 4 commit/PR includes the full attribution.333334 **Parallel subagent mode:** Commit ownership is split by isolation mode (see Phase 1 Step 4):335 - **Worktree-isolated:** subagents may stage and commit inside their own worktree branch; the orchestrator merges those branches in dependency order after the batch.336 - **Shared-directory fallback:** subagents do not commit; the orchestrator stages and commits each unit after the entire parallel batch completes.3373383. **Follow Existing Patterns**339340 - The plan should reference similar code - read those files first341 - Match naming conventions exactly342 - Reuse existing components where possible343 - Follow project coding standards (see AGENTS.md; use CLAUDE.md only if the repo still keeps a compatibility shim)344 - When in doubt, grep for similar implementations3453464. **Test Continuously**347348 - Run relevant tests after each significant change349 - Don't wait until the end to test350 - Fix failures immediately351 - Add new tests for new behavior, update tests for changed behavior, remove tests for deleted behavior352 - **Unit tests with mocks prove logic in isolation. Integration tests with real objects prove the layers work together.** If your change touches callbacks, middleware, or error handling — you need both.3533545. **Simplify as You Go**355356 After completing a cluster of related implementation units (or every 2-3 units), review recently changed files for simplification opportunities — consolidate duplicated patterns, extract shared helpers, and improve code reuse and efficiency. This is especially valuable when using subagents, since each agent works with isolated context and can't see patterns emerging across units.357358 Don't simplify after every single unit — early patterns may look duplicated but diverge intentionally in later units. Wait for a natural phase boundary or when you notice accumulated complexity.359360 If **`ce-simplify-code`** is available, invoke it at phase boundaries (especially before Phase 3 when the diff is >=30 lines). Otherwise, review the changed files yourself for reuse and consolidation opportunities.3613626. **Figma Design Sync** (if applicable)363364 For UI work with Figma designs:365366 - Implement components following design specs367 - Use ce-figma-design-sync agent iteratively to compare368 - Fix visual differences identified369 - Repeat until implementation matches design3703717. **Frontend Design Guidance** (if applicable)372373 For UI tasks without a Figma design -- where the implementation touches view, template, component, layout, or page files, creates user-visible routes, or the plan contains explicit UI/frontend/design language:374375 - Load the `ce-frontend-design` skill before implementing376 - Follow its detection, guidance, and verification flow377 - If the skill produced a verification screenshot, it satisfies Phase 4's screenshot requirement -- no need to capture separately. If the skill fell back to mental review (no browser access), Phase 4's screenshot capture still applies3783798. **Track Progress**380 - Keep the task list updated as you complete tasks381 - Note any blockers or unexpected discoveries382 - Create new tasks if scope expands383 - Keep user informed of major milestones384 - When the plan defines U-IDs for Implementation Units, or the plan or origin document carries stable R-IDs (and optionally A/F/AE IDs), reference them in blockers, deferred-work notes, task summaries, and final verification — not routine status updates. U-IDs anchor units across plan edits; R/A/F/AE anchor product intent across the brainstorm-plan handoff. Use the IDs the plan supplies and do not invent ones it does not. This preserves traceability without burying signal under noise.385386### Phase 3-4: Quality Check and Finishing Work387388When all Phase 2 tasks are complete and execution transitions to quality check, read `references/shipping-workflow.md` for the full shipping workflow: quality checks, code review, final validation, PR creation, and notification.389390---391392## Codex Delegation Mode393394When `delegation_active` is true after argument parsing, read `references/codex-delegation-workflow.md` for the complete delegation workflow: pre-checks, batching, prompt template, execution loop, and result classification.395396---397398## Key Principles399400### Start Fast, Execute Faster401402- Get clarification once at the start, then execute403- Don't wait for perfect understanding - ask questions and move404- The goal is to **finish the feature**, not create perfect process405406### The Plan is Your Guide407408- Work documents should reference similar code and patterns409- Load those references and follow them410- Don't reinvent - match what exists411412### Test As You Go413414- Run tests after each change, not at the end415- Fix failures immediately416- Continuous testing prevents big surprises417418### Quality is Built In419420- Follow existing patterns421- Write tests for new code422- Run linting before pushing423- Review when Tier 1 is available or Tier 2 criteria match (see `shipping-workflow.md`)424425### Ship Complete Features426427- Mark all tasks completed before moving on428- Don't leave features 80% done429- A finished feature that ships beats a perfect feature that doesn't430431## Common Pitfalls to Avoid432433- **Analysis paralysis** - Don't overthink, read the plan and execute434- **Skipping clarifying questions** - Ask now, not after building wrong thing435- **Ignoring plan references** - The plan has links for a reason436- **Testing at the end** - Test continuously or suffer later437- **Forgetting to track progress** - Update task status as you go or lose track of what's done438- **80% done syndrome** - Finish the feature, don't move on early439- **Skipping review without reason** — Use Tier 1 when available; escalate to Tier 2 only on criteria in `shipping-workflow.md`; document when both are skipped440- **Re-scoping the plan into human-time phases** - The plan's Implementation Units define the scope of execution. Do not estimate human-hours per unit, propose multi-day breakdowns, or ask the user to pick a subset of units for "this session". Agents execute at agent speed, and context-window pressure is addressed by subagent dispatch (Phase 1 Step 4), not by phased sessions. If a plan-file input is genuinely too large for a single execution, say so plainly and suggest the user return to `/ce-plan` to reduce scope — don't invent session phases as a workaround. For bare-prompt input, Phase 0's Large routing already handles oversized work