Agent Diagnostic
Used the create-github-issue and create-github-pr skills. Investigated mise run test:rust browser launch behavior in crates/openshell-cli/src/run.rs and crates/openshell-cli/src/auth.rs. The unit test gateway_add_cloud_rolls_back_on_auth_failure invokes gateway_add("https://127.0.0.1:1", ...) with browser suppression disabled so that the auth flow is treated as a real failure and the rollback path is exercised. browser_auth_flow then constructs /auth/connect?callback_port=...&code=... and calls open_browser_url, which runs the platform browser opener (open on macOS) before the test eventually fails authentication.
Description
Actual behavior: Running mise run test:rust locally can open a browser window to a URL like https://127.0.0.1:1/auth/connect?callback_port=53520&code=VFB-YQGF.
Expected behavior: Local test runs should not launch GUI browser windows. The auth rollback test should still exercise the failure rollback behavior without invoking the system browser.
Reproduction Steps
- Run
mise run test:rust locally.
- Observe a browser window or tab opening to
https://127.0.0.1:1/auth/connect?... during the Rust test suite.
Environment
- OS: macOS
- Docker: Not required
- OpenShell:
main at 25abc9e3
Logs
https://127.0.0.1:1/auth/connect?callback_port=53520&code=VFB-YQGF
Agent Diagnostic
Used the
create-github-issueandcreate-github-prskills. Investigatedmise run test:rustbrowser launch behavior incrates/openshell-cli/src/run.rsandcrates/openshell-cli/src/auth.rs. The unit testgateway_add_cloud_rolls_back_on_auth_failureinvokesgateway_add("https://127.0.0.1:1", ...)with browser suppression disabled so that the auth flow is treated as a real failure and the rollback path is exercised.browser_auth_flowthen constructs/auth/connect?callback_port=...&code=...and callsopen_browser_url, which runs the platform browser opener (openon macOS) before the test eventually fails authentication.Description
Actual behavior: Running
mise run test:rustlocally can open a browser window to a URL likehttps://127.0.0.1:1/auth/connect?callback_port=53520&code=VFB-YQGF.Expected behavior: Local test runs should not launch GUI browser windows. The auth rollback test should still exercise the failure rollback behavior without invoking the system browser.
Reproduction Steps
mise run test:rustlocally.https://127.0.0.1:1/auth/connect?...during the Rust test suite.Environment
mainat25abc9e3Logs