Skip to content

fix(cli): respect BROWSER for dashboard launch on Linux#423

Open
tsubasakong wants to merge 1 commit intoRightNow-AI:mainfrom
tsubasakong:lucas/issue-389-browser-env-fallback
Open

fix(cli): respect BROWSER for dashboard launch on Linux#423
tsubasakong wants to merge 1 commit intoRightNow-AI:mainfrom
tsubasakong:lucas/issue-389-browser-env-fallback

Conversation

@tsubasakong
Copy link

@tsubasakong tsubasakong commented Mar 8, 2026

Summary

Fixes #389

  • respect the Linux BROWSER environment variable before falling back to xdg-open
  • support %s placeholders as well as appended URL arguments so users can pass flags like --no-sandbox
  • add focused unit tests for the browser command construction helper

Fixes #389

Changes

  • respect the Linux BROWSER environment variable before falling back to xdg-open
  • support %s placeholders as well as appended URL arguments so users can pass flags like --no-sandbox
  • add focused unit tests for the browser command construction helper

Testing

  • cargo clippy --workspace --all-targets -- -D warnings passes
  • cargo test --workspace passes
  • Live integration tested (if applicable)

Testing notes:

  • git diff --check
  • Rust toolchain is not installed on this host, so compile/test validation is deferred to CI.

Security

  • No new unsafe code
  • No secrets or API keys in diff
  • User input validated at boundaries

Signed-off-by: Frank <97429702+tsubasakong@users.noreply.github.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.

BUG : openfang init fails with Sandbox EPERM error on Linux when falling back to web dashboard

1 participant