Fix #396: client fetch storm

crueber commented

2026-09-12 18:27:51 +00:00

Owner

Fixes #396 (client fetch storm: ~5 GET/sec across all endpoints in bursts from one tab).\n\nLoop found (reproduced headlessly against the real data-layer modules with a fetch counter): no effect loop and no key churn — the idle layer is quiet. The unbounded path is SSE invalidation: invalidate() always starts a new generation (TTL bypass, the #41 mutation rule) and the §4 coalescer batched only within one microtask, so frames arriving in separate tasks each flushed a full refetch round per cached key (measured: 64-frame spread replay = ~192 GETs; a sha key queued explicitly + via prefix fetched 2x/frame). Fix: the scheduleInvalidate flush (SSE path only) dedupes prefix expansion per flush and TTL-gates each key (08 §6 table; Infinity = immutable windows never SSE-refetched); mutation invalidate() stays eager. Tasks-overlay busy cadence 1.5s→5s (hottest endpoint). No backend change (server emission is event-driven, not 5Hz spam). No new deps.\n\nTests: web/test/unit/fetch-rate-guard.test.js (8 tests: sync/spread/mixed burst budgets, post-TTL liveness, immutable pin, mutation-eager #41 pin, unknown/uncached no-ops, BUSY/IDLE cadence pins) — fail 5/8 pre-fix, pass post-fix. Full node suite 796/796 green; vite+esbuild green; E9 evid harness green. Docs: 12_web_ui.md §2.9 + Decisions, features/08 §4 + Decisions.

crueber added 1 commit

2026-09-12 18:27:52 +00:00

Fix #396 : TTL-aware SSE invalidation + calmer tasks poll 842423331f

12_web_ui.md §2.4/§2.9 + features/08 §4: the SSE flush now dedupes prefix
expansion per flush and skips already-fresh entries (08 §6 TTL table),
so sustained frame rates decay to TTL cadence; mutation invalidate()
stays eager (#41). Tasks busy cadence 1.5s -> 5s. Cover:
web/test/unit/fetch-rate-guard.test.js (burst budgets, post-TTL
liveness, mutation-eager pin, cadence pins).

crueber referenced this pull request

2026-09-12 18:28:03 +00:00

Client fetch storm: SPA polls every endpoint ~5/sec in bursts #396

crueber added 1 commit

2026-09-12 18:31:56 +00:00

Fix #396 : ttlForKey comment — collaborators seeds at 30s, overview/ops unseeded d1aa97c1af

crueber referenced this pull request

2026-09-12 18:32:10 +00:00

Client fetch storm: SPA polls every endpoint ~5/sec in bursts #396

crueber merged commit d89c7e70bd into main

2026-09-12 18:32:29 +00:00

crueber referenced this pull request from a commit