Skip to content

fix(cli): wire top-level status command#69

Merged
yan5xu merged 1 commit intoepiral:mainfrom
ox01024:fix/status-command-help
Mar 19, 2026
Merged

fix(cli): wire top-level status command#69
yan5xu merged 1 commit intoepiral:mainfrom
ox01024:fix/status-command-help

Conversation

@ox01024
Copy link
Copy Markdown
Contributor

@ox01024 ox01024 commented Mar 19, 2026

Summary

  • add the documented top-level status CLI route
  • keep status checks read-only by using the managed-browser probe instead of discovery that may launch/write state

Testing

  • pnpm test

Closes #68

@yan5xu
Copy link
Copy Markdown
Collaborator

yan5xu commented Mar 19, 2026

LGTM — clean fix, correctly wires the status command and eliminates the side-effect in isDaemonRunning().

One thing I noticed: case "daemon": (line ~407) falls through to case "close": with no break. This means bb-browser daemon silently runs the close command — likely a pre-existing bug. Not blocking this PR, but worth a follow-up issue.

Merging.

@yan5xu yan5xu merged commit 3a15411 into epiral:main Mar 19, 2026
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.

Top-level status command is listed in help but fails as unknown command

2 participants