Skip to content

Improve remote-control daemon UX#22562

Merged
owenlin0 merged 2 commits into
mainfrom
owen/improve_error_msg
May 14, 2026
Merged

Improve remote-control daemon UX#22562
owenlin0 merged 2 commits into
mainfrom
owen/improve_error_msg

Conversation

@owenlin0
Copy link
Copy Markdown
Collaborator

@owenlin0 owenlin0 commented May 13, 2026

Why

codex remote-control manages the app-server daemon with remote_control enabled, but it previously only exposed an implicit start path. Once started, there was no obvious top-level remote-control command for stopping the daemon; users had to know about the lower-level codex app-server daemon stop command.

The startup failure for missing managed installs was also ambiguous. codex remote-control and daemon bootstrap require the standalone Codex install under CODEX_HOME/packages/standalone/current/codex, but the old error only said to install Codex first, which is unclear when another codex binary is already on PATH. Now we add an explicit instruction for how to get the standalone Codex install.

What changed

  • Converts codex remote-control into a command group while preserving bare codex remote-control as the existing start behavior.
  • Adds codex remote-control start as the explicit start path.
  • Adds codex remote-control stop, which maps to app-server daemon stop.
  • Updates the shared daemon managed-install error to name the missing standalone path, explain why that install is required, provide the installer command, and tell users to rerun the command they just tried.

Verification

  • cargo test -p codex-app-server-daemon
  • cargo test -p codex-cli
  • ./target/debug/codex remote-control --help

@owenlin0 owenlin0 changed the title fix: better error message for codexd when standalone install is missing Improve missing standalone install error for remote control May 13, 2026
@owenlin0 owenlin0 changed the title Improve missing standalone install error for remote control Improve remote-control daemon UX May 13, 2026
@owenlin0 owenlin0 marked this pull request as ready for review May 13, 2026 23:22
@owenlin0 owenlin0 requested a review from euroelessar May 13, 2026 23:23
@owenlin0 owenlin0 merged commit 512f8f8 into main May 14, 2026
31 checks passed
@owenlin0 owenlin0 deleted the owen/improve_error_msg branch May 14, 2026 01:04
@github-actions github-actions Bot locked and limited conversation to collaborators May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants