fix(desktop): correct ACP provider install semantics for Goose and Codex

Goose has native ACP support (no separate adapter), so the install
script belongs in cli_install_commands, not adapter_install_commands.
Also sets underlying_cli to "goose" since the CLI binary exists.

Codex ACP adapter is available on npm as @zed-industries/codex-acp —
enable auto-install and point docs link to the actual adapter repo
instead of the main Codex repo.
This commit is contained in:
Will Pfleger
2026-05-22 18:44:06 -04:00
parent 338f0aff1d
commit 4696cc4cd6
@@ -66,9 +66,9 @@ const KNOWN_ACP_PROVIDERS: &[KnownAcpProvider] = &[
avatar_url: GOOSE_AVATAR_URL,
mcp_command: None,
mcp_hooks: false,
underlying_cli: None,
cli_install_commands: &[],
adapter_install_commands: &["curl -fsSL https://github.com/block-open-source/goose/releases/download/stable/download_cli.sh | CONFIGURE=false bash"],
underlying_cli: Some("goose"),
cli_install_commands: &["curl -fsSL https://github.com/block-open-source/goose/releases/download/stable/download_cli.sh | CONFIGURE=false bash"],
adapter_install_commands: &[],
install_instructions_url: "https://block.github.io/goose/",
install_hint: "Install Goose via the official install script.",
},
@@ -96,9 +96,9 @@ const KNOWN_ACP_PROVIDERS: &[KnownAcpProvider] = &[
mcp_hooks: false,
underlying_cli: Some("codex"),
cli_install_commands: &[],
adapter_install_commands: &[],
install_instructions_url: "https://github.com/openai/codex",
install_hint: "The codex-acp adapter must be built from source. See the GitHub repo.",
adapter_install_commands: &["npm install -g @zed-industries/codex-acp"],
install_instructions_url: "https://github.com/zed-industries/codex-acp",
install_hint: "Install the Codex ACP adapter via npm.",
},
KnownAcpProvider {
id: "sprout-agent",