@powersync/service-module-mongodb-storage 0.3.0 → 0.3.1

package/src/storage/MongoBucketStorage.ts

@@ -5,7 +5,7 @@ import * as timers from 'timers/promises';
 
 import { storage, sync, utils } from '@powersync/service-core';
 
-import { DisposableObserver, logger } from '@powersync/lib-services-framework';
+import { DisposableObserver, ErrorCode, logger, ServiceError } from '@powersync/lib-services-framework';
 import { v4 as uuid } from 'uuid';
 
 import * as lib_mongo from '@powersync/lib-service-mongodb';
@@ -433,7 +433,7 @@ export class MongoBucketStorage
       clusterTime = time;
     });
     if (clusterTime == null) {
-      throw new Error('Could not get clusterTime');
+      throw new ServiceError(ErrorCode.PSYNC_S2401, 'Could not get clusterTime');
     }
 
     if (signal.aborted) {

package/src/storage/implementation/MongoBucketBatch.ts

@@ -2,7 +2,16 @@ import { mongo } from '@powersync/lib-service-mongodb';
 import { SqlEventDescriptor, SqliteRow, SqlSyncRules } from '@powersync/service-sync-rules';
 import * as bson from 'bson';
 
-import { container, DisposableObserver, errors, logger } from '@powersync/lib-services-framework';
+import {
+  container,
+  DisposableObserver,
+  ErrorCode,
+  errors,
+  logger,
+  ReplicationAssertionError,
+  ServiceAssertionError,
+  ServiceError
+} from '@powersync/lib-services-framework';
 import { SaveOperationTag, storage, utils } from '@powersync/service-core';
 import * as timers from 'node:timers/promises';
 import { PowerSyncMongo } from './db.js';
@@ -140,7 +149,7 @@ export class MongoBucketBatch
     this.batch = resumeBatch;
 
     if (last_op == null) {
-      throw new Error('Unexpected last_op == null');
+      throw new ReplicationAssertionError('Unexpected last_op == null');
     }
 
     this.persisted_op = last_op;
@@ -294,7 +303,7 @@ export class MongoBucketBatch
           return null;
         }
       } else {
-        throw new Error(`${record.tag} not supported with skipExistingRows: true`);
+        throw new ReplicationAssertionError(`${record.tag} not supported with skipExistingRows: true`);
       }
     }
 
@@ -348,7 +357,7 @@ export class MongoBucketBatch
         afterData = new bson.Binary(bson.serialize(after!));
         // We additionally make sure it's <= 15MB - we need some margin for metadata.
         if (afterData.length() > MAX_ROW_SIZE) {
-          throw new Error(`Row too large: ${afterData.length()}`);
+          throw new ServiceError(ErrorCode.PSYNC_S1002, `Row too large: ${afterData.length()}`);
         }
       } catch (e) {
         // Replace with empty values, equivalent to TOAST values
@@ -548,7 +557,7 @@ export class MongoBucketBatch
         logger.info(`${this.slot_name} ${description} - try ${flushTry}`);
       }
       if (flushTry > 20 && Date.now() > lastTry) {
-        throw new Error('Max transaction tries exceeded');
+        throw new ServiceError(ErrorCode.PSYNC_S1402, 'Max transaction tries exceeded');
       }
 
       const next_op_id_doc = await this.db.op_id_sequence.findOneAndUpdate(

package/src/storage/implementation/MongoCompactor.ts

@@ -1,5 +1,5 @@
 import { mongo } from '@powersync/lib-service-mongodb';
-import { logger } from '@powersync/lib-services-framework';
+import { logger, ReplicationAssertionError } from '@powersync/lib-services-framework';
 import { storage, utils } from '@powersync/service-core';
 
 import { PowerSyncMongo } from './db.js';
@@ -335,7 +335,9 @@ export class MongoCompactor {
                   }
                 }
               } else {
-                throw new Error(`Unexpected ${op.op} operation at ${op._id.g}:${op._id.b}:${op._id.o}`);
+                throw new ReplicationAssertionError(
+                  `Unexpected ${op.op} operation at ${op._id.g}:${op._id.b}:${op._id.o}`
+                );
               }
             }
             if (!gotAnOp) {

package/src/storage/implementation/MongoIdSequence.ts

@@ -1,3 +1,5 @@
+import { ReplicationAssertionError } from '@powersync/lib-services-framework';
+
 /**
  * Manages op_id or similar sequence in memory.
  *
@@ -9,7 +11,7 @@ export class MongoIdSequence {
 
   constructor(last: bigint) {
     if (typeof last != 'bigint') {
-      throw new Error(`BigInt required, got ${last} ${typeof last}`);
+      throw new ReplicationAssertionError(`BigInt required, got ${last} ${typeof last}`);
     }
     this._last = last;
   }

package/src/storage/implementation/MongoStorageProvider.ts

@@ -1,5 +1,5 @@
 import * as lib_mongo from '@powersync/lib-service-mongodb';
-import { logger } from '@powersync/lib-services-framework';
+import { logger, ServiceAssertionError } from '@powersync/lib-services-framework';
 import { storage } from '@powersync/service-core';
 import { MongoStorageConfig } from '../../types/types.js';
 import { MongoBucketStorage } from '../MongoBucketStorage.js';
@@ -16,7 +16,9 @@ export class MongoStorageProvider implements storage.BucketStorageProvider {
     const { storage } = resolvedConfig;
     if (storage.type != this.type) {
       // This should not be reached since the generation should be managed externally.
-      throw new Error(`Cannot create MongoDB bucket storage with provided config ${storage.type} !== ${this.type}`);
+      throw new ServiceAssertionError(
+        `Cannot create MongoDB bucket storage with provided config ${storage.type} !== ${this.type}`
+      );
     }
 
     const decodedConfig = MongoStorageConfig.decode(storage as any);

package/src/storage/implementation/MongoSyncBucketStorage.ts

@@ -1,6 +1,6 @@
 import * as lib_mongo from '@powersync/lib-service-mongodb';
 import { mongo } from '@powersync/lib-service-mongodb';
-import { DisposableObserver, logger } from '@powersync/lib-services-framework';
+import { DisposableObserver, logger, ServiceAssertionError } from '@powersync/lib-services-framework';
 import { storage, utils } from '@powersync/service-core';
 import { SqliteJsonRow, SqliteJsonValue, SqlSyncRules } from '@powersync/service-sync-rules';
 import * as bson from 'bson';
@@ -344,7 +344,7 @@ export class MongoSyncBucketStorage
 
         start ??= dataBuckets.get(bucket);
         if (start == null) {
-          throw new Error(`data for unexpected bucket: ${bucket}`);
+          throw new ServiceAssertionError(`data for unexpected bucket: ${bucket}`);
         }
         currentBatch = {
           bucket,
@@ -479,7 +479,7 @@ export class MongoSyncBucketStorage
       }
     );
     if (doc == null) {
-      throw new Error('Cannot find sync rules status');
+      throw new ServiceAssertionError('Cannot find sync rules status');
     }
 
     return {

package/src/storage/implementation/MongoSyncRulesLock.ts

@@ -1,6 +1,6 @@
 import crypto from 'crypto';
 
-import { logger } from '@powersync/lib-services-framework';
+import { ErrorCode, logger, ServiceError } from '@powersync/lib-services-framework';
 import { storage } from '@powersync/service-core';
 import { PowerSyncMongo } from './db.js';
 
@@ -33,7 +33,10 @@ export class MongoSyncRulesLock implements storage.ReplicationLock {
     );
 
     if (doc == null) {
-      throw new Error(`Sync rules: ${sync_rules.id} have been locked by another process for replication.`);
+      throw new ServiceError(
+        ErrorCode.PSYNC_S1003,
+        `Sync rules: ${sync_rules.id} have been locked by another process for replication.`
+      );
     }
     return new MongoSyncRulesLock(db, sync_rules.id, lockId);
   }

package/test/src/storage_compacting.test.ts

@@ -3,4 +3,9 @@ import { register } from '@powersync/service-core-tests';
 import { describe } from 'vitest';
 import { INITIALIZED_MONGO_STORAGE_FACTORY } from './util.js';
 
-describe('Mongo Sync Bucket Storage Compact', () => register.registerCompactTests<MongoCompactOptions>(INITIALIZED_MONGO_STORAGE_FACTORY, { clearBatchLimit: 2, moveBatchLimit: 1, moveBatchQueryLimit: 1 }));
+describe('Mongo Sync Bucket Storage Compact', () =>
+  register.registerCompactTests<MongoCompactOptions>(INITIALIZED_MONGO_STORAGE_FACTORY, {
+    clearBatchLimit: 2,
+    moveBatchLimit: 1,
+    moveBatchQueryLimit: 1
+  }));