@powersync/common 1.41.1 → 1.43.0

package/dist/bundle.node.mjs

@@ -6702,7 +6702,7 @@ function requireDist () {
 
 var distExports = requireDist();
 
-var version = "1.41.1";
+var version = "1.43.0";
 var PACKAGE = {
 	version: version};
 
@@ -7450,9 +7450,11 @@ class AbstractRemote {
         // Create a new stream splitting the response at line endings while also handling cancellations
         // by closing the reader.
         const reader = res.body.getReader();
+        let readerReleased = false;
         // This will close the network request and read stream
         const closeReader = async () => {
             try {
+                readerReleased = true;
                 await reader.cancel();
             }
             catch (ex) {
@@ -7460,17 +7462,21 @@ class AbstractRemote {
             }
             reader.releaseLock();
         };
+        const stream = new DataStream({
+            logger: this.logger,
+            mapLine: mapLine
+        });
         abortSignal?.addEventListener('abort', () => {
             closeReader();
+            stream.close();
         });
         const decoder = this.createTextDecoder();
         let buffer = '';
-        const stream = new DataStream({
-            logger: this.logger,
-            mapLine: mapLine
-        });
         const l = stream.registerListener({
             lowWater: async () => {
+                if (stream.closed || abortSignal?.aborted || readerReleased) {
+                    return;
+                }
                 try {
                     let didCompleteLine = false;
                     while (!didCompleteLine) {
@@ -8630,6 +8636,7 @@ class TriggerManagerImpl {
             await hooks?.beforeCreate?.(tx);
             await tx.execute(/* sql */ `
         CREATE TEMP TABLE ${destination} (
+          operation_id INTEGER PRIMARY KEY AUTOINCREMENT,
           id TEXT,
           operation TEXT,
           timestamp TEXT,
@@ -8740,17 +8747,20 @@ class TriggerManagerImpl {
                     const callbackResult = await options.onChange({
                         ...tx,
                         destinationTable: destination,
-                        withDiff: async (query, params) => {
+                        withDiff: async (query, params, options) => {
                             // Wrap the query to expose the destination table
+                            const operationIdSelect = options?.castOperationIdAsText
+                                ? 'id, operation, CAST(operation_id AS TEXT) as operation_id, timestamp, value, previous_value'
+                                : '*';
                             const wrappedQuery = /* sql */ `
                   WITH
                     DIFF AS (
                       SELECT
-                        *
+                        ${operationIdSelect}
                       FROM
                         ${destination}
                       ORDER BY
-                        timestamp ASC
+                        operation_id ASC
                     ) ${query}
                 `;
                             return tx.getAll(wrappedQuery, params);
@@ -8764,13 +8774,14 @@ class TriggerManagerImpl {
                         id,
                         ${contextColumns.length > 0
                                 ? `${contextColumns.map((col) => `json_extract(value, '$.${col}') as ${col}`).join(', ')},`
-                                : ''} operation as __operation,
+                                : ''} operation_id as __operation_id,
+                        operation as __operation,
                         timestamp as __timestamp,
                         previous_value as __previous_value
                       FROM
                         ${destination}
                       ORDER BY
-                        __timestamp ASC
+                        __operation_id ASC
                     ) ${query}
                 `;
                             return tx.getAll(wrappedQuery, params);