Skip to content

chore(daemon): read status through transport layer#2040

Open
jackwener wants to merge 1 commit into
mainfrom
chore/daemon-transport-boundary
Open

chore(daemon): read status through transport layer#2040
jackwener wants to merge 1 commit into
mainfrom
chore/daemon-transport-boundary

Conversation

@jackwener

Copy link
Copy Markdown
Owner

Summary

  • route passive daemon/profile/doctor status reads through daemon-transport instead of daemon-client
  • keep daemon-client focused on command dispatch while preserving its existing re-exports
  • update daemon and doctor tests to mock the lower-level transport boundary

Verification

  • npm run typecheck
  • npm test -- --run src/commands/daemon.test.ts src/doctor.test.ts
  • npm run build

Note: I also tried npm test -- --run src/commands/daemon.test.ts src/doctor.test.ts src/cli.test.ts. The cli tests showed pre-existing environment-sensitive failures because this machine has a default Browser Bridge profile configured, so BrowserBridge.connect received a contextId the assertions do not allow. I did not change those assertions in this cleanup PR.

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