dangernoodle-io / TaipanMiner / 26204791209

90%

push

github

refactor(webui): event-bus context; components own their SSE state machines (#424)

App.svelte previously created updateAvailableState + blockFoundState,
held a separate SseClient with an explicit eventHandlers map, and
threaded the state objects into Header / UpdateBadgeContainer /
BlockFoundBanner as props. Adding a new event topic meant editing
App.svelte to import the state, register it in eventHandlers, and pass
it down — three coupled changes for what's logically one consumer.

Replace with an event-bus context (lib/eventBus.svelte.ts):

- App.svelte instantiates the bus, setContext(EVENT_BUS_KEY, bus),
  start/stop in onMount. Nothing else about topics.
- The bus owns one SseClient over /api/events, fans out by topic, and
  rebuilds the underlying client only when a *new* topic set is
  subscribed (no churn on re-subscribes of an already-attached topic).
- BlockFoundBanner / UpdateBadgeContainer each create their own state
  machine and call bus.subscribe(TOPIC, state.handleMessage) on mount.
  Components encapsulate their topic dependency instead of asking
  App.svelte to wire it.

Adding a new event-driven UI surface is now one component change with
no App.svelte edits.

2511 of 3185 branches covered (78.84%)

Branch coverage included in aggregate %.

53 of 53 new or added lines in 5 files covered. (100.0%)

5098 of 5273 relevant lines covered (96.68%)

156607.08 hits per line