trm/processor
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs: update task 1.5.3 done section and ROADMAP status

Browse Source
This commit is contained in:
Julian Çuni
2026-05-02 17:40:19 +02:00
parent bf5c358668
commit 90605614f6
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.planning/phase-1-5-live-broadcast/03-subscription-registry.md
+5 -1
View File
@@ -223,4 +223,8 @@ Hooked into the `ws.on('close', ...)` handler in `server.ts`.
## Done
(Filled in when the task lands.)
Landed in `38de4bc`. Key deviations from spec:
- `canAccessEvent` signature is `(cookieHeader: string, eventId: string)` rather than `(user: AuthenticatedUser, eventId: string)` because `AuthenticatedUser` doesn't carry the cookie — the cookie lives on `LiveConnection.cookieHeader`. The call site passes `conn.cookieHeader` directly.
- Added `'error'` to `ErrorCode` in `protocol.ts` to handle transient authz failures; the spec omitted this case.
- `SnapshotProvider` interface is defined in `registry.ts` and defaults to a stub returning `[]`; task 1.5.5 injects the real implementation via the optional parameter.
- Snapshot fetching is integrated into the subscribe flow already (calls `fetchSnapshot` on success), making task 1.5.5 a pure injection of a better provider rather than a structural change to the subscribe flow.