cross-tab-worker-databus 0.21.42 → 0.21.43

package/CHANGELOG.md

@@ -1,3 +1,16 @@
+## [0.21.43] - 2026-09-26
+
+Two claims about the storage boundary turned out to be narrower than the storage was, and a public clock could report `NaN` into the health summary. No library behavior, export, option, default, frame or storage key changed beyond the one guard; the coverage floors and the zero-count arm ledger are `0.21.42`'s.
+
+### Fixed
+
+- **A caller-injected dedup clock that returns a non-finite number poisoned every comparison that reads it.** `dedup.now` is a public option, and a non-wall-clock host is exactly what it exists for — one whose reading can be *missing* rather than merely unusual. Since every comparison in the dedup manager is arithmetic on a timestamp, a single `NaN` reading did not degrade one field: the adaptive window's `elapsed >= ADAPTIVE_WINDOW_MS` never fired, the hot path's `now - timestamp > ttlMs` became `NaN > ttlMs` (false, so nothing expired), the periodic sweep pruned nothing, and `getDedupStats().ttlMs` reported **`NaN`** — which `JSON.stringify` writes as **`null`**, so a health summary or a diagnostics log carried a null where a duration belongs. Measured on the pre-fix class: `ttlMs` → `NaN`, serialized → `{"ttlMs":null}`, and a message ID stayed suppressed for as long as it stayed in the map. The manager now holds the last finite reading, which is the degradation that changes least — time stands still, expiry pauses, the adaptive window keeps its last value — where substituting `Date.now()` would silently mix two clocks and zeroing would expire the whole map on the next call. The guard sits at the clock rather than at each comparison, which is the shape this repository already uses for the same class of input in `trace.ts`, `websocket.ts` and `replay-manager.ts`. Pinned by two cases whose legs fail apart: removing the guard fails both, while removing only the constructor's fallback — the one site with no previous reading to hold — fails the second alone, because a `NaN` baseline would make every later `elapsed` non-finite too. `docs/api.md` and its Chinese mirror state the guarantee where `dedup.now` is documented.
+
+### Added
+
+- **The storage boundary is now stated where it was widest, not only where it was found.** `docs/configuration.md` and `docs/architecture.md` each said the coordination records are isolated per `clusterKey` and that Topic plaintext reaches a durable store only through two named opt-ins; the architecture guide carried that claim in a *dedicated* section ("Two DataBus instances with different `clusterKey` values … operate on completely isolated storage namespaces") and as an absolute above its key listing ("All keys are isolated by `createOpaqueKey(clusterKey)`"). Both now name the exception the second opt-in introduces — its IndexedDB object store is keyed by the plaintext Topic and its database name defaults to the storage prefix, so it is not partitioned by `clusterKey` at all, and `dbName` per cluster is what separates them — in both languages.
+- **The key/plaintext pairing rule is now described the way the code enforces it.** The protocol table's row "`topicKey === createOpaqueKey(topic)`" became unconditionally true in `0.21.42`, but the prose beneath it still said only that a pair which *disagrees* is a substitution — which reads as a rule about mismatches and leaves a frame carrying no `topicKey` sounding like a tolerated legacy shape, which is the reading that once kept a guard in place. The shipped text now says the comparison is unconditional, that a missing field is a disagreement of the same kind, and that a custom environment should always send the field. Both languages.
+
 ## [0.21.42] - 2026-09-26
 
 The library's central privacy claim — that a topic's plaintext never reaches localStorage — was enforced on keys by accident and on values by nothing, and the guard that protects the coordination plane's key/topic pair was written to tolerate a missing field it never had to tolerate. Both halves of that sentence were found by re-auditing claims rather than code: the first by measuring which mutations anything noticed, the second by enumerating the readers of a pair. One behavioral change, no API, option, default, frame or storage-key change.

package/dist/centrifuge.js

@@ -7,7 +7,7 @@ import {
   publicationMetadata,
   selectWorkerBackend,
   serializeError
-} from "./chunk-VWVYUSIN.js";
+} from "./chunk-NTMMSR4W.js";
 import {
   CENTRIFUGE_INPUT_TYPE,
   CENTRIFUGE_OUTPUT_TYPE,

package/dist/{chunk-VWVYUSIN.js → chunk-NTMMSR4W.js}

@@ -2366,6 +2366,8 @@ var DedupManager = class {
   seenMessageIds = /* @__PURE__ */ new Map();
   sweepTimer = null;
   windowStartedAt = 0;
+  /** The most recent finite reading of `now`, and the fallback for a broken one. */
+  lastFiniteNow = 0;
   windowAccepted = 0;
   suppressed = 0;
   accepted = 0;
@@ -2377,14 +2379,46 @@ var DedupManager = class {
     this.sweepMs = options.sweepMs;
     this.now = options.now;
     this.trace = options.trace;
-    this.windowStartedAt = this.now();
+    this.lastFiniteNow = Number.isFinite(this.now()) ? this.now() : 0;
+    this.windowStartedAt = this.lastFiniteNow;
+  }
+  /**
+   * The injected clock, normalized.
+   *
+   * `dedup.now` is a public option and a non-wall-clock host is exactly the case it
+   * exists for — one whose reading can be *missing* rather than merely unusual. Every
+   * comparison this class makes is arithmetic on a timestamp, so a single non-finite
+   * reading does not degrade one field: `now - timestamp > ttlMs` becomes
+   * `NaN > ttlMs` (false, so nothing expires), the adaptive window's
+   * `elapsed >= ADAPTIVE_WINDOW_MS` never fires, and `getStats().ttlMs` reports
+   * `NaN` — which `JSON.stringify` turns into `null`, so a health summary or a
+   * diagnostics log would carry a null where a duration belongs. Measured on the
+   * pre-fix class: `getStats().ttlMs` → `NaN`, serialized → `null`, and an id
+   * stayed suppressed for as long as it stayed in the map.
+   *
+   * Holding the last finite reading is the degradation that changes least: time
+   * stands still, expiry pauses, and the adaptive window keeps the value it last
+   * computed. Substituting `Date.now()` would be worse — it silently mixes two
+   * clocks — and zeroing would expire the whole map on the next call. This is the
+   * same class of guard `trace.ts` carries on its own public clock, and that
+   * `websocket.ts` and `replay-manager.ts` apply to their numeric options; it is one
+   * helper rather than a condition per site because the broken input is the *clock*,
+   * not any particular comparison.
+   */
+  nowMs() {
+    const reading = this.now();
+    if (Number.isFinite(reading)) {
+      this.lastFiniteNow = reading;
+      return reading;
+    }
+    return this.lastFiniteNow;
   }
   /** True when a publication carrying `messageId` was already seen. Records the
    * ID and updates counters/trace on acceptance. Disabled or ID-less messages
    * always pass through. */
   isDuplicate(messageId, topic) {
     if (!this.enabled || !messageId) return false;
-    const now = this.now();
+    const now = this.nowMs();
     const ttlMs = this.currentTtl();
     for (const [id, timestamp] of this.seenMessageIds) {
       if (now - timestamp > ttlMs) this.seenMessageIds.delete(id);
@@ -2435,12 +2469,12 @@ var DedupManager = class {
     this.seenMessageIds.clear();
     this.suppressed = 0;
     this.accepted = 0;
-    this.windowStartedAt = this.now();
+    this.windowStartedAt = this.nowMs();
     this.windowAccepted = 0;
   }
   /** Remove IDs whose timestamp predates the effective TTL cutoff. */
   pruneExpired() {
-    const cutoff = this.now() - this.currentTtl();
+    const cutoff = this.nowMs() - this.currentTtl();
     for (const [id, timestamp] of this.seenMessageIds) {
       if (timestamp < cutoff) this.seenMessageIds.delete(id);
     }
@@ -2451,7 +2485,7 @@ var DedupManager = class {
    * every ADAPTIVE_WINDOW_MS. */
   currentTtl() {
     if (!this.adaptiveBounds) return this.ttlMs;
-    const now = this.now();
+    const now = this.nowMs();
     const elapsed = now - this.windowStartedAt;
     if (elapsed >= ADAPTIVE_WINDOW_MS) {
       this.windowStartedAt = now;
@@ -2475,7 +2509,7 @@ var SDK_VERSION = (
   // declaration's type is `string | undefined` and the export's is `string`
   // (dropping the conditional is TS2322, measured). Recorded here so the leg reads
   // as a shape the harness cannot reach rather than as a case nobody exercised.
-  true ? "0.21.42" : ""
+  true ? "0.21.43" : ""
 );
 
 // src/core/data-bus.ts
@@ -3715,4 +3749,4 @@ export {
   parseDataBusPublication,
   selectWorkerBackend
 };
-//# sourceMappingURL=chunk-VWVYUSIN.js.map
+//# sourceMappingURL=chunk-NTMMSR4W.js.map