@syncular/server-hono 0.0.1-60 → 0.0.1

package/src/ws.ts

@@ -17,23 +17,12 @@ export interface PresenceEntry {
   metadata?: Record<string, unknown>;
 }
 
-/**
- * Push response data sent back to the client over WS
- */
-export interface WsPushResponseData {
-  requestId: string;
-  ok: boolean;
-  status: string;
-  commitSeq?: number;
-  results: Array<{ opIndex: number; status: string; [k: string]: unknown }>;
-}
-
 /**
  * WebSocket event data for sync notifications
  */
 export interface SyncWebSocketEvent {
   /** Event type */
-  event: 'sync' | 'heartbeat' | 'error' | 'presence' | 'push-response';
+  event: 'sync' | 'heartbeat' | 'error' | 'presence';
   /** Data payload */
   data: {
     /** New cursor position (for sync events) */
@@ -49,12 +38,6 @@ export interface SyncWebSocketEvent {
       metadata?: Record<string, unknown>;
       entries?: PresenceEntry[];
     };
-    /** Push response data (for push-response events) */
-    requestId?: string;
-    ok?: boolean;
-    status?: string;
-    commitSeq?: number;
-    results?: Array<{ opIndex: number; status: string; [k: string]: unknown }>;
     /** Timestamp */
     timestamp: number;
   };
@@ -64,8 +47,8 @@ export interface SyncWebSocketEvent {
  * WebSocket connection controller for managing active connections
  */
 export interface WebSocketConnection {
-  /** Send a sync notification, optionally with inline change data */
-  sendSync(cursor: number, changes?: unknown[]): void;
+  /** Send a sync notification */
+  sendSync(cursor: number): void;
   /** Send a heartbeat */
   sendHeartbeat(): void;
   /** Send a presence event */
@@ -77,8 +60,6 @@ export interface WebSocketConnection {
     metadata?: Record<string, unknown>;
     entries?: PresenceEntry[];
   }): void;
-  /** Send a push response back to the client */
-  sendPushResponse(data: WsPushResponseData): void;
   /** Send an error and close */
   sendError(message: string): void;
   /** Close the connection */
@@ -116,16 +97,15 @@ export function createWebSocketConnection(
     actorId: args.actorId,
     clientId: args.clientId,
     transportPath: args.transportPath,
-    sendSync(cursor: number, changes?: unknown[]) {
+    sendSync(cursor: number) {
       if (!connection.isOpen) return;
-      const payload: Record<string, unknown> = {
-        cursor,
-        timestamp: Date.now(),
-      };
-      if (changes && changes.length > 0) {
-        payload.changes = changes;
-      }
-      const ok = safeSend(ws, JSON.stringify({ event: 'sync', data: payload }));
+      const ok = safeSend(
+        ws,
+        JSON.stringify({
+          event: 'sync',
+          data: { cursor, timestamp: Date.now() },
+        })
+      );
       if (!ok) closed = true;
     },
     sendHeartbeat() {
@@ -154,17 +134,6 @@ export function createWebSocketConnection(
       );
       if (!ok) closed = true;
     },
-    sendPushResponse(data: WsPushResponseData) {
-      if (!connection.isOpen) return;
-      const ok = safeSend(
-        ws,
-        JSON.stringify({
-          event: 'push-response',
-          data: { ...data, timestamp: Date.now() },
-        })
-      );
-      if (!ok) closed = true;
-    },
     sendError(message: string) {
       if (connection.isOpen) {
         safeSend(
@@ -606,16 +575,10 @@ export class WebSocketConnectionManager {
    * Notify clients that new data is available for the given scopes.
    * Dedupes connections that match multiple scopes.
    */
-  /**
-   * Maximum serialized size (bytes) for inline WS change delivery.
-   * Larger payloads fall back to cursor-only notification.
-   */
-  private static readonly WS_INLINE_MAX_BYTES = 64 * 1024;
-
   notifyScopeKeys(
     scopeKeys: string[],
     cursor: number,
-    opts?: { excludeClientIds?: string[]; changes?: unknown[] }
+    opts?: { excludeClientIds?: string[] }
   ): void {
     const exclude = new Set(opts?.excludeClientIds ?? []);
     const targets = new Set<WebSocketConnection>();
@@ -626,23 +589,10 @@ export class WebSocketConnectionManager {
       for (const conn of conns) targets.add(conn);
     }
 
-    // Size guard: only deliver inline changes if under threshold
-    let inlineChanges: unknown[] | undefined;
-    if (opts?.changes && opts.changes.length > 0) {
-      const serialized = JSON.stringify(opts.changes);
-      if (serialized.length <= WebSocketConnectionManager.WS_INLINE_MAX_BYTES) {
-        inlineChanges = opts.changes;
-      }
-    }
-
     for (const conn of targets) {
       if (!conn.isOpen) continue;
       if (exclude.has(conn.clientId)) continue;
-      if (inlineChanges) {
-        conn.sendSync(cursor, inlineChanges);
-      } else {
-        conn.sendSync(cursor);
-      }
+      conn.sendSync(cursor);
     }
   }