cross-tab-worker-databus 0.20.96 → 0.21.0

package/dist/cjs/index.cjs

@@ -703,6 +703,12 @@ function assertClusterOptions(options) {
   }
   assertLoadWeightingOptions(options.loadWeighting);
 }
+function assertPublicTopic(operation, topic) {
+  if (topic !== "") return;
+  throw new TypeError(
+    `CrossTabDataBus.${operation}("") addresses a channel no transport can route; use a non-empty topic.`
+  );
+}
 
 // src/core/cluster.ts
 var CLUSTER_PROTOCOL_VERSION = 1;
@@ -2466,7 +2472,7 @@ var DedupManager = class {
 };
 
 // src/core/version.ts
-var SDK_VERSION = true ? "0.20.96" : "";
+var SDK_VERSION = true ? "0.21.0" : "";
 
 // src/core/data-bus.ts
 var DEFAULT_REPLAY_MAX_PER_TOPIC = 100;
@@ -2557,10 +2563,6 @@ var CrossTabDataBus = class {
   // themselves demand: the failure path starts an on-demand reopen instead of
   // stranding them until some unrelated future operation arrives.
   recoveryWaiters = 0;
-  // Latch for the empty-topic deprecation warning so a hot publish path cannot
-  // fill the console. It is per-bus and never reset: the warning is about the
-  // caller's code, not about a transient runtime condition.
-  emptyTopicWarned = false;
   /** Monotonic generation incremented on every successful transport open.
    * Stays in lockstep with `lastSuccessAt` so callers can detect that the
    * transport has been reopened even if the timestamp window is short. */
@@ -2955,8 +2957,11 @@ var CrossTabDataBus = class {
    * unsubscribe function for convenience. During an explicit stop() the
    * registration is rejected through onError and a no-op cleanup is returned,
    * so a late subscriber cannot leak into a future restart.
+   * @throws {TypeError} when `topic` is `''` — no transport can address such a
+   *   channel, so the subscription could never receive anything.
    */
   subscribe(topic, handler, options) {
+    assertPublicTopic("subscribe", topic);
     if (this.stopping) {
       this.reportError(new Error(
         "CrossTabDataBus is stopping; subscribe() was not registered. Wait for stop() to resolve, then call start() before subscribing again."
@@ -2965,7 +2970,6 @@ var CrossTabDataBus = class {
       };
     }
     this.ensureStarted();
-    if (topic === "") this.warnEmptyTopic("subscribe");
     const handlers = this.topicHandlers.get(topic) ?? /* @__PURE__ */ new Set();
     const wasUnused = handlers.size === 0;
     handlers.add(handler);
@@ -3015,20 +3019,10 @@ var CrossTabDataBus = class {
   resetDedup() {
     this.dedupManager.reset();
   }
-  /** Warn (once per bus) that an empty topic is deprecated, without changing
-   * behavior yet. `''` flows through routing as a literal channel, so the
-   * subscription it creates can never be addressed by a transport: the message
-   * silently goes nowhere. A future minor rejects it at this boundary. */
-  warnEmptyTopic(operation) {
-    if (this.emptyTopicWarned) return;
-    this.emptyTopicWarned = true;
-    console.warn(
-      `cross-tab-worker-databus: ${operation}("") addresses a channel no transport can route. Use a non-empty topic; passing "" is planned to throw in a future minor.`
-    );
-  }
-  /** Publish a message to `topic`. The owning Worker delivers it to the transport. */
+  /** Publish a message to `topic`. The owning Worker delivers it to the transport.
+   * @throws {TypeError} when `topic` is `''`; see `subscribe()`. */
   publish(topic, data, options) {
-    if (topic === "") this.warnEmptyTopic("publish");
+    assertPublicTopic("publish", topic);
     this.ensureStarted();
     if (this.rejectPublishDuringStop("publish")) return;
     if (!this.cluster.publish(topic, data, options)) {
@@ -3043,11 +3037,14 @@ var CrossTabDataBus = class {
    * in one tick. Per-item dedup / replay / ordering is preserved; each item
    * may carry its own `messageId` / `timestamp` via `options`. Empty array is
    * a no-op; single-item array delegates to `publish()`.
+   * @throws {TypeError} when `topic` is `''`; see `subscribe()`. Checked before
+   *   the empty-array no-op, so `publishBatch('', [])` throws too — the topic is
+   *   the caller's bug regardless of whether anything is carried.
    */
   publishBatch(topic, items) {
+    assertPublicTopic("publishBatch", topic);
     this.ensureStarted();
     if (items.length === 0) return;
-    if (topic === "") this.warnEmptyTopic("publishBatch");
     if (this.rejectPublishDuringStop("publishBatch")) return;
     if (items.length === 1) {
       const first = items[0];