Plan: Remove execute phase and reorder deliver before review#
Summary#
Remove all references to the deprecated execute pipeline phase and fix phase ordering so deliver comes before review across all UI components, type definitions, and constants in the control center.
Files#
| File | Action | Description |
|---|---|---|
app/pipeline/page.tsx | modify | Remove execute from KANBAN_PHASES array and tasksByPhase init; reorder deliver before review |
app/config/page.tsx | modify | Remove execute from SKILL_KEYS array; reorder deliver before review |
lib/api/tasks.ts | modify | Remove phaseExecute field from Task interface; reorder phaseDeliver before phaseReview |
app/pipeline/[taskId]/page.tsx | modify | Remove execute entry from PHASE_FIELDS; reorder deliver before review in PHASE_FIELDS and the hardcoded dropdown (line ~185) |
Steps#
- S1: Update phase arrays and constants — Remove
executefromKANBAN_PHASESinapp/pipeline/page.tsx,SKILL_KEYSinapp/config/page.tsx, andPHASE_FIELDS+ dropdown inapp/pipeline/[taskId]/page.tsx. Reorder all to:refine, plan, deliver, review, compound. - S2: Update Task interface and data structures — Remove
phaseExecutefromTaskinterface inlib/api/tasks.ts. Removeexecutekey fromtasksByPhaseobject init inapp/pipeline/page.tsx. Reorder field declarations sophaseDeliverprecedesphaseReview.
Verification#
npm run lintpasses with zero warningsnpm run buildcompiles successfully- Grep for
executein phase-related contexts returns zero hits
Risks#
phaseExecutein theTaskinterface mirrors the backend API response shape. Ifagent-corestill sends this field, removing it is safe (TS will ignore extra props), but any code readingtask.phaseExecutewill break at compile time — the build step will catch this.