Title#

feat: implement #501 — Remove execute phase and align CC APIs with current pipeline

Problem#

The execute phase column and its references remain in the database entity, DTOs, and test fixtures despite having been removed from PHASE_ORDER and routing maps. This causes API inconsistencies and misleading test fixtures.

Task / Link#

Closes #501

Changes#

• Remove phaseExecute column from TaskEntity
• Remove phaseExecute: 'pending' from createTask() defaults in TaskStateService
• Remove 'execute' from PHASES arrays in PhaseOverrideDto and StatusOverrideDto, reorder to [refine, plan, deliver, review, compound]
• Update all test fixtures across 9 spec files to replace phaseExecute, currentPhase: 'execute', and execute_completed/execute_failed event types with valid phases (deliver)

Notes#

• Existing SQLite DB rows retain the phase_execute column harmlessly — TypeORM stops reading/writing it without a migration
• event-bus.service.spec.ts (not in original plan) also had a currentPhase: 'execute' fixture; fixed

Testing#

• npm run lint — passed (zero warnings)
• npm run test — 790 tests passed, 0 failures