You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The orchestrator's phase-skip logic checks for "approved plan exists" and "plan section complete" to decide whether to skip PREPARE and ARCHITECT phases. However, it doesn't account for research/investigation tasks within the plan that explicitly require PREPARE-phase execution.
What happened
During SDK optimization orchestration (2026-02-19), the orchestrator:
Detected an approved design doc and implementation plan
Skipped PREPARE (rationale: "approved plan with complete sections")
User also noted ARCHITECT shouldn't be pre-skipped since PREPARE outcomes could require architectural decisions
Root cause
The completeness check in pact-completeness.md checks for 6 incompleteness signals (unchecked research, TBD values, forward references, etc.) but doesn't explicitly check for investigation/research tasks in the implementation plan that require runtime execution during PREPARE.
Proposed Fix
Add a 7th incompleteness signal: "Implementation plan contains research/investigation tasks (go/no-go items, feasibility studies, audit tasks)"
Update pact-completeness.md with the new signal
Update the phase-skip guidance in orchestrate.md to explicitly call out research tasks as a reason NOT to skip PREPARE
Consider: ARCHITECT should not be pre-skipped when PREPARE is running, since PREPARE outcomes may require architectural decisions (add guidance to post-PREPARE re-assessment)
Problem
The orchestrator's phase-skip logic checks for "approved plan exists" and "plan section complete" to decide whether to skip PREPARE and ARCHITECT phases. However, it doesn't account for research/investigation tasks within the plan that explicitly require PREPARE-phase execution.
What happened
During SDK optimization orchestration (2026-02-19), the orchestrator:
Root cause
The completeness check in
pact-completeness.mdchecks for 6 incompleteness signals (unchecked research, TBD values, forward references, etc.) but doesn't explicitly check for investigation/research tasks in the implementation plan that require runtime execution during PREPARE.Proposed Fix
pact-completeness.mdwith the new signalorchestrate.mdto explicitly call out research tasks as a reason NOT to skip PREPAREFiles to modify
pact-plugin/protocols/pact-completeness.mdpact-plugin/commands/orchestrate.md(phase-skip section + post-PREPARE re-assessment)Reference
docs/plans/2026-02-19-sdk-optimization-design.md