Skip to content

fix: keep repeated launcher on existing bridge - #1776

Open
k9ight000 wants to merge 1 commit into
BigPizzaV3:mainfrom
k9ight000:codex/delete-conversation-fix
Open

fix: keep repeated launcher on existing bridge#1776
k9ight000 wants to merge 1 commit into
BigPizzaV3:mainfrom
k9ight000:codex/delete-conversation-fix

Conversation

@k9ight000

Copy link
Copy Markdown

Problem

Launching CodexPlusPlus a second time can acquire the existing-instance path, start a temporary helper, reinject the page, and then exit. The page is left pointing at the temporary helper, so conversation delete requests fail even though the delete controls remain visible.

Fix

  • Keep the repeated-launcher path limited to activating the existing Codex window.
  • Do not start another helper, reinject the renderer, or start another bridge watchdog.
  • Add a regression assertion that the existing-instance path cannot recreate bridge components.

Validation

  • cargo fmt --check --package codex-plus-launcher
  • cargo test -p codex-plus-launcher --no-run
  • cargo test -p codex-plus-core --test launcher -j 1 -- --skip app_paths_resolves_portable_current_link_to_directory_version — 74 passed

The skipped test requires Windows symbolic-link privilege and is unrelated to this change. The published diff contains only apps/codex-plus-launcher/src/main.rs and was scanned for credentials and local user paths.

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