@mitway/sdk 0.2.4 → 0.3.0

package/dist/index.cjs

@@ -893,6 +893,11 @@ var Database = class {
 // src/modules/realtime.ts
 var import_socket = require("socket.io-client");
 var PRESENCE_HEARTBEAT_MS = 2e4;
+function makeChannelError(code, message) {
+  const err = new Error(message);
+  err.code = code;
+  return err;
+}
 var DEFAULT_CONNECT_TIMEOUT_MS = 1e4;
 var RealtimeChannel = class {
   constructor(topic, realtime, options = {}) {
@@ -963,12 +968,19 @@ var RealtimeChannel = class {
       this.statusCallback?.("SUBSCRIBED");
     } catch (err) {
       this.state = "errored";
-      this.statusCallback?.("CHANNEL_ERROR", {
-        code: "REJOIN_FAILED",
-        message: err instanceof Error ? err.message : String(err)
-      });
+      this.statusCallback?.(
+        "CHANNEL_ERROR",
+        makeChannelError("REJOIN_FAILED", err instanceof Error ? err.message : String(err))
+      );
     }
   }
+  // ── implementation signature (not in public type surface).
+  // The callback type is intentionally broad: each overload above pins a
+  // specific payload shape, but TypeScript overload resolution needs the
+  // implementation to accept the union of every narrow callback without
+  // the contravariance conflict (TS2394). `any` is the standard escape
+  // hatch for this exact pattern and is confined to this one line — the
+  // public surface users see is strictly typed via the overloads.
   on(type, filter, callback) {
     if (type === "postgres_changes") {
       this.bindings.push({
@@ -1070,18 +1082,18 @@ var RealtimeChannel = class {
         } catch (err) {
           this.state = "errored";
           const message = err instanceof Error ? err.message : String(err);
-          this.statusCallback?.("CHANNEL_ERROR", {
-            code: "SUBSCRIBE_FAILED",
-            message
-          });
+          this.statusCallback?.(
+            "CHANNEL_ERROR",
+            makeChannelError("SUBSCRIBE_FAILED", message)
+          );
         }
       },
       (err) => {
         this.state = "errored";
-        this.statusCallback?.("CHANNEL_ERROR", {
-          code: "CONNECT_FAILED",
-          message: err.message
-        });
+        this.statusCallback?.(
+          "CHANNEL_ERROR",
+          makeChannelError("CONNECT_FAILED", err.message)
+        );
       }
     );
     return this;
@@ -1134,6 +1146,12 @@ var RealtimeChannel = class {
    * events, write to your own application table and enable
    * `postgres_changes` on it; the SDK will surface the INSERT as a
    * `postgres_changes` event without a separate channel.send call.
+   *
+   * The returned promise always resolves with the server ack, so callers
+   * can `await channel.send(...)` to confirm delivery + get the server-
+   * assigned `message_id`. There's no performance cost — Socket.IO piggy-
+   * backs the ack on the same frame. Callers that don't need it just
+   * don't await.
    */
   async send(args) {
     if (args.type !== "broadcast") {
@@ -1160,9 +1178,7 @@ var RealtimeChannel = class {
     if (!socket) {
       throw new MitwayBaasError("Socket not connected", 503, "NOT_CONNECTED");
     }
-    const broadcastCfg = this.options.config?.broadcast;
-    const wantAck = broadcastCfg?.ack === true;
-    const self = broadcastCfg?.self;
+    const self = this.options.config?.broadcast?.self;
     const wirePayload = {
       channel: this.topic,
       event: args.event,
@@ -1171,10 +1187,6 @@ var RealtimeChannel = class {
     if (self === false) {
       wirePayload.self = false;
     }
-    if (!wantAck) {
-      socket.emit("realtime:publish", wirePayload);
-      return;
-    }
     return await new Promise((resolve) => {
       socket.emit(
         "realtime:publish",
@@ -1215,7 +1227,7 @@ var RealtimeChannel = class {
         if (!b.subscriptionId || !pcEvent.ids.includes(b.subscriptionId)) {
           continue;
         }
-        const matchesEvent = b.filter.event === "*" || b.filter.event === pcEvent.data.type;
+        const matchesEvent = b.filter.event === "*" || b.filter.event === pcEvent.data.eventType;
         if (!matchesEvent) {
           continue;
         }
@@ -1274,7 +1286,13 @@ var RealtimeChannel = class {
         continue;
       }
       try {
-        b.callback(payload);
+        if (payload.event === "sync") {
+          b.callback();
+        } else if (payload.event === "join") {
+          b.callback(payload);
+        } else {
+          b.callback(payload);
+        }
       } catch {
       }
     }
@@ -1293,7 +1311,7 @@ var RealtimeChannel = class {
           "realtime:subscribe",
           { channel: this.topic, private: this.isPrivate },
           (ack) => {
-            if (ack.ok) {
+            if (ack.status === "ok") {
               resolve();
             } else {
               reject(new Error(ack.error?.message ?? "subscribe failed"));
@@ -1317,7 +1335,7 @@ var RealtimeChannel = class {
               filter: b.filter.filter
             },
             (ack) => {
-              if (ack.ok && ack.subscription_id) {
+              if (ack.status === "ok" && ack.subscription_id) {
                 b.subscriptionId = ack.subscription_id;
                 resolve();
               } else {
@@ -1363,13 +1381,14 @@ var Realtime = class {
    * The optional `opts` argument lets the caller configure the channel:
    *   * `config.private` — enable subscribe-side authorization against
    *     `realtime.authorize_subscribe(...)` on the tenant DB.
-   *   * `config.broadcast.ack` — `channel.send()` resolves with the
-   *     server's ack (message_id) instead of fire-and-forget.
    *   * `config.broadcast.self` — `false` excludes the sender from the
    *     fan-out (defaults to `true`).
    *   * `config.presence.key` — stable presence key to group multiple
    *     tabs of the same user under one entry.
    *
+   * `channel.send()` always resolves with the server ack (see its own
+   * docstring); there is no separate opt-in needed.
+   *
    * Options are locked in when the channel is first created; subsequent
    * `.channel('same')` calls with different opts are ignored. Pass a
    * different topic to get a different-configured channel.