Skip to content

code-mode standalone: Cutover to new process - #27727

Closed
cconger wants to merge 11 commits into
mainfrom
cconger/code-mode-host-4-exclusive-cutover
Closed

code-mode standalone: Cutover to new process#27727
cconger wants to merge 11 commits into
mainfrom
cconger/code-mode-host-4-exclusive-cutover

Conversation

@cconger

@cconger cconger commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Compare with previous

This is phase 3 of a 4 phase stack:

  1. Add protocol and host crates for new IPC code mode implementation code-mode standalone: extract protocol and add host crate #27724
  2. Create the new standalone binary code-mode standalone: add host binary  #27725
  3. Create a new IPC CodeModeSessionProvider to use new binary code-mode standalone: Add client and consume with packaging #27726
  4. Remove v8 from core and only use IPC provider

Complete the process isolation and remove V8 from the core dependency graph.

  • Make the IPC client the default code-mode session provider.
  • Remove the direct codex-code-mode dependency from codex-core.
  • Move remaining shared type and helper consumers onto codex-code-mode-protocol.
  • Ensure V8 is linked only into the standalone host binary.

@cconger
cconger requested a review from a team as a code owner June 12, 2026 00:51

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 15e7cfbbcf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

return sibling;
}
}
PathBuf::from(executable_name)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep code mode working in single-binary installs

When codex-code-mode-host is not installed beside the CLI (for example Cargo installs or single-binary release archives), this falls back to PATH and spawning fails, so code-mode execution regresses from the prior in-process behavior for those CLI install surfaces. guidance

Useful? React with 👍 / 👎.

@cconger
cconger force-pushed the cconger/code-mode-host-4-exclusive-cutover branch 5 times, most recently from d1b3292 to 08ebec8 Compare June 12, 2026 03:07
@cconger
cconger force-pushed the cconger/code-mode-host-4-exclusive-cutover branch from 08ebec8 to 181bcfe Compare June 12, 2026 05:14
@cconger
cconger force-pushed the cconger/code-mode-host-4-exclusive-cutover branch from 181bcfe to c1bf7b7 Compare June 12, 2026 05:40
@cconger
cconger force-pushed the cconger/code-mode-host-4-exclusive-cutover branch from c1bf7b7 to 32ed022 Compare June 12, 2026 05:58
@cconger cconger closed this Jun 25, 2026
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