Skip to content

[Refactor] Consolidate code-mode turn lifecycle state - #3009

Open
roomote-roomote[bot] wants to merge 1 commit into
developfrom
refactor/code-mode-turn-lifecycle-33ko1cz12jlsj
Open

roomote-roomote[bot] wants to merge 1 commit into
developfrom
refactor/code-mode-turn-lifecycle-33ko1cz12jlsj

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Created by Roomote. View the task or mention @roomote-roomote for follow-up asks.

What changed

  • Replaced five independently mutable code-mode turn fields with one turn-scoped lifecycle owner.
  • Modeled inactive, awaiting-server, and server-bound phases explicitly so capability, directory, mounted integration IDs, and leased-server state move together.
  • Moved same-turn mounting and retry bookkeeping behind that owner, with focused coverage for inactive turns, server binding, duplicate suppression, and next-turn retry after a rejected live mount.

Why this change was made

Code-mode integration activation and mid-turn refresh depended on partial state spread across the Fast orchestration service. Keeping those invariants in one owner reduces invalid combinations and makes the lifecycle safer to extend.

Impact

There is no intended user-facing behavior change. Same-turn connection refresh, next-turn durability, and collision fallback are preserved, while the orchestration service carries less mutable lifecycle state. The change was kept narrow around the active per-tool approvals work in #3001 to reduce merge overlap.

@roomote-community

roomote-community Bot commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed 5520c5f

@roomote-roomote
roomote-roomote Bot marked this pull request as ready for review September 20, 2026 03:39

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant