Skip to content

fix(desktop): clarify CLI runtime setup#2680

Draft
atishpatel wants to merge 3 commits into
mainfrom
agent/windows-onboarding-verify
Draft

fix(desktop): clarify CLI runtime setup#2680
atishpatel wants to merge 3 commits into
mainfrom
agent/windows-onboarding-verify

Conversation

@atishpatel

Copy link
Copy Markdown
Contributor

Why

Installing the Codex, Claude, or Goose desktop app does not install the command-line harness Buzz needs. The current UI makes that distinction unclear, links some missing-CLI states to adapter documentation, and can report a successful install from the installer exit code even when runtime discovery still fails. On Windows, Buzz also invokes Goose's Bash installer, which writes the executable somewhere Buzz does not discover.

What

  • distinguish missing vendor CLIs from missing or outdated ACP adapters in runtime metadata and UI guidance
  • link Codex, Claude Code, and Goose missing-CLI states to their official CLI installation documentation
  • explain in Settings, onboarding, and agent configuration that the desktop app alone is not sufficient
  • use Goose's official PowerShell installer on Windows
  • refresh PATH and rediscover the requested runtime after installation, keeping the control retryable if the runtime is still unavailable
  • add Rust and Playwright regression coverage for Windows installer selection, CLI/adapter guidance, false-success prevention, verified installs, and onboarding copy

Risk Assessment

Medium. This changes desktop onboarding and runtime installation behavior. Successful installs now require the runtime catalog to verify availability; previously hidden discovery failures will surface as actionable errors instead of a false success state.

References

Validation

  • just desktop-typecheck
  • just desktop-test — 3,455 passed
  • focused Rust post-install verification tests
  • focused Playwright Doctor/onboarding coverage (in progress; CI and local sequential rerun will provide final results)

Generated with Codex

Co-authored-by: Atish Patel <atish@squareup.com>
Signed-off-by: Atish Patel <atish@squareup.com>
Co-authored-by: Codex <noreply@openai.com>
Comment thread desktop/src-tauri/src/managed_agents/discovery.rs Outdated
atishpatel and others added 2 commits July 23, 2026 22:41
Co-authored-by: Atish Patel <atish@squareup.com>
Signed-off-by: Atish Patel <atish@squareup.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Atish Patel <atish@squareup.com>
Signed-off-by: Atish Patel <atish@squareup.com>
Co-authored-by: Codex <noreply@openai.com>
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