[timeleft--]

Summary

Adds a new consultation mode to porch that delegates phase reviews to the parent session instead of spawning cold-start consult API calls.

• When porch.consultation is set to "parent" in af-config.json, porch emits phase-review-* gates at phase boundaries instead of running consult commands
• The parent (architect) session reviews builder work and approves gates via porch approve
• Gate names are scoped to phase/planPhase/iteration to prevent stale gates from blocking after rollback
• Falls back to "default" mode (standard consult commands) when config is missing or invalid

Closes #614

Changes

• config.ts: New loadConsultationMode() function reads porch.consultation from af-config.json
• next.ts: Gate-intercept logic in handleBuildVerify() — emits phase-review-{phase}-{planPhase}-iter{N} gates; next() checks for pending phase-review gates before advancing
• index.ts: Shows CONSULTATION: parent in porch status output
• config.test.ts: 8 unit tests for loadConsultationMode()

Configuration

{
  "porch": {
    "consultation": "parent"
  }
}

Gate format

phase-review-{phase}-{planPhase}-iter{iteration}

Example: phase-review-implement-phase_1-iter1

Dogfooding

This feature has been dogfooded extensively on the MachineWisdomAI fork:

• Full ASPIR lifecycle with 2 concurrent builders (bug: ASPIR builder-prompt references SPIR, spawn --task skips porch init, fragile phases check #619, bug: builder worktrees missing codev/protocols, af spawn requires GitHub issue with CLI backend #620) — all phase-review gates fired and were reviewed/approved by the parent session
• BUGFIX protocol cycle on the fava-trails project
• Both multi-phase (ASPIR) and single-phase (BUGFIX) protocols validated

Test plan

• loadConsultationMode() returns "parent" when configured, "default" otherwise (8 unit tests)
• Phase-review gates emit at each phase boundary in parent mode
• Stale gates from previous iterations don't block after rollback
• porch status displays consultation mode
• All 229 porch tests pass
• Builds clean with no type errors

🤖 Generated with Claude Code

[waleedkadous]

Hey — thanks for this work! Just a heads up: PR #624 (merged as part of the v3.0.0-rc.8 config overhaul) added pluggable consultation models to porch, including a "parent" mode that emits phase-review-{phase} gates instead of running consult commands.

Could you take a look and let us know whether #624 already covers what this PR was doing, or if there's still functionality here that's additive? The main things I'd want to compare are the gate naming convention and the rollback-safety logic.

Also note that af-config.json has been replaced by .codev/config.json with a unified config loader, so this PR would need updating either way.

[timeleft--]

Closing — v3.0.0 (PR #624) implemented pluggable consultation models including parent-delegated mode via porch.consultation.models: 'parent'. This PR's functionality is fully covered by the new architecture.