@powersync/common 1.44.0 → 1.46.0

package/src/db/ConnectionClosedError.ts

@@ -0,0 +1,23 @@
+/**
+ * Thrown when an underlying database connection is closed.
+ * This is particularly relevant when worker connections are marked as closed while
+ * operations are still in progress.
+ */
+export class ConnectionClosedError extends Error {
+  static NAME = 'ConnectionClosedError';
+
+  static MATCHES(input: any) {
+    /**
+     * If there are weird package issues which cause multiple versions of classes to be present, the instanceof
+     * check might fail. This also performs a failsafe check.
+     * This might also happen if the Error is serialized and parsed over a bridging channel like a MessagePort.
+     */
+    return (
+      input instanceof ConnectionClosedError || (input instanceof Error && input.name == ConnectionClosedError.NAME)
+    );
+  }
+  constructor(message: string) {
+    super(message);
+    this.name = ConnectionClosedError.NAME;
+  }
+}

package/src/db/crud/SyncStatus.ts

@@ -1,7 +1,7 @@
-import { CoreStreamSubscription } from '../../client/sync/stream/core-instruction.js';
 import { SyncClientImplementation } from '../../client/sync/stream/AbstractStreamingSyncImplementation.js';
-import { InternalProgressInformation, ProgressWithOperations, SyncProgress } from './SyncProgress.js';
+import { CoreStreamSubscription } from '../../client/sync/stream/core-instruction.js';
 import { SyncStreamDescription, SyncSubscriptionDescription } from '../../client/sync/sync-streams.js';
+import { InternalProgressInformation, ProgressWithOperations, SyncProgress } from './SyncProgress.js';
 
 export type SyncDataFlowStatus = Partial<{
   downloading: boolean;
@@ -250,13 +250,32 @@ export class SyncStatus {
     return {
       connected: this.connected,
       connecting: this.connecting,
-      dataFlow: this.dataFlowStatus,
+      dataFlow: {
+        ...this.dataFlowStatus,
+        uploadError: this.serializeError(this.dataFlowStatus.uploadError),
+        downloadError: this.serializeError(this.dataFlowStatus.downloadError)
+      },
       lastSyncedAt: this.lastSyncedAt,
       hasSynced: this.hasSynced,
       priorityStatusEntries: this.priorityStatusEntries
     };
   }
 
+  /**
+   * Not all errors are serializable over a MessagePort. E.g. some `DomExceptions` fail to be passed across workers. 
+   * This explicitly serializes errors in the SyncStatus.
+   */
+  protected serializeError(error?: Error) {
+    if (typeof error == 'undefined') {
+      return undefined;
+    }
+    return {
+      name: error.name,
+      message: error.message,
+      stack: error.stack
+    };
+  }
+
   private static comparePriorities(a: SyncPriorityStatus, b: SyncPriorityStatus) {
     return b.priority - a.priority; // Reverse because higher priorities have lower numbers
   }

package/src/index.ts

@@ -21,6 +21,7 @@ export * from './client/sync/stream/streaming-sync-types.js';
 export * from './client/sync/sync-streams.js';
 
 export * from './client/ConnectionManager.js';
+export * from './db/ConnectionClosedError.js';
 export { ProgressWithOperations, SyncProgress } from './db/crud/SyncProgress.js';
 export * from './db/crud/SyncStatus.js';
 export * from './db/crud/UploadQueueStatus.js';
@@ -28,13 +29,16 @@ export * from './db/DBAdapter.js';
 export * from './db/schema/Column.js';
 export * from './db/schema/Index.js';
 export * from './db/schema/IndexedColumn.js';
+export * from './db/schema/RawTable.js';
 export * from './db/schema/Schema.js';
 export * from './db/schema/Table.js';
 export * from './db/schema/TableV2.js';
 
 export * from './client/Query.js';
+export { MEMORY_TRIGGER_CLAIM_MANAGER } from './client/triggers/MemoryTriggerClaimManager.js';
 export * from './client/triggers/sanitizeSQL.js';
 export * from './client/triggers/TriggerManager.js';
+export { TriggerManagerImpl } from './client/triggers/TriggerManagerImpl.js';
 export * from './client/watched/GetAllQuery.js';
 export * from './client/watched/processors/AbstractQueryProcessor.js';
 export * from './client/watched/processors/comparators.js';

package/src/utils/DataStream.ts

@@ -110,6 +110,18 @@ export class DataStream<ParsedData, SourceData = any> extends BaseObserver<DataS
       return null;
     }
 
+    // Wait for any pending processing to complete first.
+    // This ensures we register our listener before calling processQueue(),
+    // avoiding a race where processQueue() sees no reader and returns early.
+    if (this.processingPromise) {
+      await this.processingPromise;
+    }
+
+    // Re-check after await - stream may have closed while we were waiting
+    if (this.closed) {
+      return null;
+    }
+
     return new Promise((resolve, reject) => {
       const l = this.registerListener({
         data: async (data) => {
@@ -151,7 +163,7 @@ export class DataStream<ParsedData, SourceData = any> extends BaseObserver<DataS
 
     const promise = (this.processingPromise = this._processQueue());
     promise.finally(() => {
-      return (this.processingPromise = null);
+      this.processingPromise = null;
     });
     return promise;
   }
@@ -190,7 +202,6 @@ export class DataStream<ParsedData, SourceData = any> extends BaseObserver<DataS
     }
 
     if (this.dataQueue.length > 0) {
-      // Next tick
       setTimeout(() => this.processQueue());
     }
   }