refactor: migrate solo to workspace store

This commit is contained in:
Janpeter Visser 2026-05-10 07:27:43 +02:00
parent 90c68ef8de
commit 2d42e2b954
13 changed files with 1191 additions and 526 deletions

View file

@ -7,11 +7,9 @@
// productId niet null is; sluit de stream als productId null wordt.
// - Reconnect met exponential backoff (1s → 30s, reset bij ready).
// - PBI-74: stream blijft open op tab hidden (geen close meer). Bij
// hidden→visible en bij window 'online' triggeren we router.refresh()
// zodat gemiste events alsnog binnenkomen via een verse server-render
// (re-fetcht initialTasks → initTasks reset solo-store). Postgres NOTIFY
// heeft geen replay, dus zonder deze resync zouden hidden-tab events
// permanent verloren zijn — zelfde fix als Story 5 voor backlog-realtime.
// hidden→visible en bij window 'online' triggeren we een directe
// workspace-store resync. Postgres NOTIFY heeft geen replay, dus zonder deze
// resync zouden hidden-tab events permanent verloren zijn.
// - Cleanup op unmount.
// - Connection-status (status, showConnectingIndicator) wordt naar de
// solo-store geschreven; UI-componenten lezen daar uit.
@ -24,7 +22,6 @@
import { useEffect, useRef } from 'react'
import { flushSync } from 'react-dom'
import { useRouter } from 'next/navigation'
import { useSoloStore } from '@/stores/solo-store'
import type { ClaudeJobEvent, JobState, RealtimeEvent, RealtimeStatus } from '@/stores/solo-store'
@ -33,7 +30,6 @@ const BACKOFF_MAX_MS = 30_000
const CONNECTING_INDICATOR_DELAY_MS = 4_000
export function useSoloRealtime(productId: string | null) {
const router = useRouter()
const sourceRef = useRef<EventSource | null>(null)
const backoffRef = useRef<number>(BACKOFF_START_MS)
const reconnectTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null)
@ -97,10 +93,9 @@ export function useSoloRealtime(productId: string | null) {
backoffRef.current = BACKOFF_START_MS
scheduleIndicator('open')
readyCountRef.current += 1
// PBI-74: latere ready = post-reconnect → resync via router.refresh()
// zodat gemiste tasks-state via re-render initial-prop binnenkomt.
// PBI-74: latere ready = post-reconnect → directe workspace-resync.
if (readyCountRef.current > 1) {
router.refresh()
void useSoloStore.getState().resyncActiveScopes('reconnect')
}
})
@ -189,19 +184,19 @@ export function useSoloRealtime(productId: string | null) {
// PBI-74: stream blijft open op hidden. Reconnect alleen als de stream
// door netwerkfout/server-close weg is en de tab visible is. Bij iedere
// visible-overgang triggeren we router.refresh() — gemiste events tijdens
// throttling/freeze worden via een verse server-render alsnog opgepakt.
// visible-overgang triggeren we een store-resync — gemiste events tijdens
// throttling/freeze worden via de solo-workspace route alsnog opgepakt.
const onVisibility = () => {
if (document.visibilityState !== 'visible') return
if (sourceRef.current === null) {
backoffRef.current = BACKOFF_START_MS
connect()
}
router.refresh()
void useSoloStore.getState().resyncActiveScopes('visible')
}
const onOnline = () => {
router.refresh()
void useSoloStore.getState().resyncActiveScopes('reconnect')
}
connect()
@ -215,5 +210,5 @@ export function useSoloRealtime(productId: string | null) {
close()
readyCountRef.current = 0
}
}, [productId, router])
}, [productId])
}