@acodeninja/persist 3.0.0-next.20 → 3.0.0-next.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@acodeninja/persist",
-  "version": "3.0.0-next.20",
+  "version": "3.0.0-next.22",
   "description": "A JSON based data modelling and persistence module with alternate storage mechanisms.",
   "type": "module",
   "scripts": {

package/src/Connection.js

@@ -189,11 +189,12 @@ export default class Connection {
 
             if (modelToProcessHasChanged) modelsToPut.push(modelToProcess);
 
+            const modelToProcessConstructor = this.#getModelConstructorFromId(modelToProcess.id).name;
+
             if (
                 Boolean(modelToProcess.constructor.indexedProperties().length) &&
                 (!currentModel || !_.isEqual(currentModel.toIndexData(), modelToProcess.toIndexData()))
             ) {
-                const modelToProcessConstructor = this.#getModelConstructorFromId(modelToProcess.id);
                 modelsToReindex[modelToProcessConstructor] = modelsToReindex[modelToProcessConstructor] || [];
                 modelsToReindex[modelToProcessConstructor].push(modelToProcess);
             }
@@ -202,7 +203,6 @@ export default class Connection {
                 Boolean(modelToProcess.constructor.searchProperties().length) &&
                 (!currentModel || !_.isEqual(currentModel.toSearchData(), modelToProcess.toSearchData()))
             ) {
-                const modelToProcessConstructor = this.#getModelConstructorFromId(modelToProcess.id);
                 modelsToReindexSearch[modelToProcessConstructor] = modelsToReindexSearch[modelToProcessConstructor] || [];
                 modelsToReindexSearch[modelToProcessConstructor].push(modelToProcess);
             }
@@ -223,7 +223,7 @@ export default class Connection {
                 const index = await this.#storage.getIndex(modelConstructor);
 
                 await this.#storage.putIndex(modelConstructor, {
-                    ...index || {},
+                    ...index,
                     ...Object.fromEntries(models.map(m => [m.id, m.toIndexData()])),
                 });
             })),
@@ -232,7 +232,7 @@ export default class Connection {
                 const index = await this.#storage.getSearchIndex(modelConstructor);
 
                 await this.#storage.putSearchIndex(modelConstructor, {
-                    ...index || {},
+                    ...index,
                     ...Object.fromEntries(models.map(m => [m.id, m.toSearchData()])),
                 });
             })),
@@ -277,7 +277,8 @@ export default class Connection {
 
             if (!modelsToDelete.includes(currentModel.id)) modelsToDelete.push(currentModel.id);
 
-            const modelToProcessConstructor = this.#getModelConstructorFromId(modelToProcess.id);
+            const modelToProcessConstructor = this.#getModelConstructorFromId(modelToProcess.id).name;
+
             indexActions[modelToProcessConstructor] = indexActions[modelToProcessConstructor] ?? [];
             searchIndexActions[modelToProcessConstructor] = searchIndexActions[modelToProcessConstructor] ?? [];
 
@@ -437,9 +438,9 @@ export default class Connection {
 
         const engine = CreateTransactionalStorageEngine(transactions, this.#storage);
 
-        const connection = new this.constructor(engine, this.#cache, Object.values(this.#models));
+        const transaction = new this.constructor(engine, this.#cache, Object.values(this.#models));
 
-        connection.commit = async () => {
+        transaction.commit = async () => {
             try {
                 for (const [index, transaction] of transactions.entries()) {
                     try {
@@ -449,6 +450,12 @@ export default class Connection {
                         if (transaction.method === 'deleteModel')
                             transactions[index].original = await this.#storage.getModel(transaction.args[0]);
 
+                        if (transaction.method === 'putIndex')
+                            transactions[index].original = await this.#storage.getIndex(transaction.args[0]);
+
+                        if (transaction.method === 'putSearchIndex')
+                            transactions[index].original = await this.#storage.getSearchIndex(transaction.args[0]);
+
                         await this.#storage[transaction.method](...transaction.args);
 
                         transactions[index].committed = true;
@@ -458,17 +465,24 @@ export default class Connection {
                     }
                 }
             } catch (error) {
-                await Promise.all(
-                    transactions
-                        .filter(t => t.committed && t.original)
-                        .map(t => this.#storage.putModel(t.original)),
-                );
+                for (const transaction of transactions) {
+                    if (transaction.committed && transaction.original) {
+                        if (['putModel', 'deleteModel'].includes(transaction.method))
+                            await this.#storage.putModel(transaction.original);
+
+                        if ('putIndex' === transaction.method)
+                            await this.#storage.putIndex(transaction.args[0], transaction.original);
+
+                        if ('putSearchIndex' === transaction.method)
+                            await this.#storage.putSearchIndex(transaction.args[0], transaction.original);
+                    }
+                }
 
                 throw new CommitFailedTransactionError(transactions, error);
             }
         };
 
-        return connection;
+        return transaction;
     }
 
     /**
@@ -525,13 +539,11 @@ export default class Connection {
                     })),
             )
             .flat()
-            .reduce((accumulator, {containingModel, propertyName, propertyProperty}) => ({
-                ...accumulator,
-                [containingModel]: {
-                    ...accumulator[containingModel] || {},
-                    [propertyName]: propertyProperty,
-                },
-            }), {});
+            .reduce((accumulator, {containingModel, propertyName, propertyProperty}) => {
+                accumulator[containingModel] = accumulator[containingModel] ?? {};
+                accumulator[containingModel][propertyName] = propertyProperty;
+                return accumulator;
+            }, {});
     }
 
     /**

package/src/engine/storage/HTTPStorageEngine.js

@@ -29,7 +29,7 @@ export default class HTTPStorageEngine extends StorageEngine {
      * Get a model
      * @param {string} id
      * @throws ModelNotFoundStorageEngineError
-     * @return Promise<Model>
+     * @return Promise<Object>
      */
     getModel(id) {
         return this.#processFetch(this.#generateURL([id]), this.#getReadOptions())
@@ -43,7 +43,7 @@ export default class HTTPStorageEngine extends StorageEngine {
 
     /**
      * Update a model
-     * @param {object} model
+     * @param {Object} model
      * @throws HTTPRequestFailedError
      * @return Promise<void>
      */
@@ -75,7 +75,7 @@ export default class HTTPStorageEngine extends StorageEngine {
     /**
      * Get a model's index data
      * @param {Model.constructor} modelConstructor
-     * @return Promise<void>
+     * @return Promise<Record<String, Object>>
      */
     getIndex(modelConstructor) {
         return this.#processFetch(
@@ -88,7 +88,7 @@ export default class HTTPStorageEngine extends StorageEngine {
     /**
      * Put a model's index data
      * @param {Model.constructor} modelConstructor
-     * @param {object} index
+     * @param {Record<String, Object>} index
      * @throws MethodNotImplementedStorageEngineError
      * @return Promise<void>
      */
@@ -103,7 +103,7 @@ export default class HTTPStorageEngine extends StorageEngine {
      * Get a model's raw search index data
      * @param {Model.constructor} modelConstructor
      * @throws MethodNotImplementedStorageEngineError
-     * @return Promise<object>
+     * @return Promise<Record<String, Object>>
      */
     getSearchIndex(modelConstructor) {
         return this.#processFetch(

package/src/engine/storage/S3StorageEngine.js

@@ -48,7 +48,7 @@ class S3StorageEngine extends StorageEngine {
 
     /**
      * Upload an object to S3
-     * @param {object} model - The model object to upload.
+     * @param {Object} model - The model object to upload.
      * @returns {Promise<void>}
      */
     async putModel(model) {
@@ -87,7 +87,7 @@ class S3StorageEngine extends StorageEngine {
      * Get a model's index data
      * @param {Model.constructor} modelConstructor
      * @throws MethodNotImplementedStorageEngineError
-     * @return Promise<object>
+     * @return Promise<Record<String, Object>>
      */
     async getIndex(modelConstructor) {
         const Key = this.#generatePath([modelConstructor.name, '_index.json']);
@@ -106,7 +106,7 @@ class S3StorageEngine extends StorageEngine {
     /**
      * Put a model's index data
      * @param {Model.constructor} modelConstructor
-     * @param {object} index
+     * @param {Record<String, Object>} index
      * @throws MethodNotImplementedStorageEngineError
      * @return Promise<void>
      */

package/src/engine/storage/StorageEngine.js

@@ -13,7 +13,7 @@ export default class StorageEngine {
      * @param {string} _id
      * @throws MethodNotImplementedStorageEngineError
      * @throws ModelNotFoundStorageEngineError
-     * @return Promise<Model>
+     * @return Promise<Object>
      */
     getModel(_id) {
         return Promise.reject(new MethodNotImplementedStorageEngineError('getModel', this));
@@ -21,7 +21,7 @@ export default class StorageEngine {
 
     /**
      * Update a model
-     * @param {object} _model
+     * @param {Object} _model
      * @throws MethodNotImplementedStorageEngineError
      * @return Promise<void>
      */
@@ -44,7 +44,7 @@ export default class StorageEngine {
      * Get a model's index data
      * @param {Model.constructor} _modelConstructor
      * @throws MethodNotImplementedStorageEngineError
-     * @return Promise<void>
+     * @return Promise<Record<String, Object>>
      */
     getIndex(_modelConstructor) {
         return Promise.reject(new MethodNotImplementedStorageEngineError('getIndex', this));
@@ -53,7 +53,7 @@ export default class StorageEngine {
     /**
      * Put a model's index data
      * @param {Model.constructor} _modelConstructor
-     * @param {object} _data
+     * @param {Record<String, Object>} _data
      * @throws MethodNotImplementedStorageEngineError
      * @return Promise<void>
      */
@@ -65,7 +65,7 @@ export default class StorageEngine {
      * Get a model's raw search index data
      * @param {Model.constructor} _modelConstructor
      * @throws MethodNotImplementedStorageEngineError
-     * @return Promise<object>
+     * @return Promise<Record<String, Object>>
      */
     getSearchIndex(_modelConstructor) {
         return Promise.reject(new MethodNotImplementedStorageEngineError('getSearchIndex', this));
@@ -76,7 +76,7 @@ export default class StorageEngine {
      * @param {Model.constructor} _constructor
      * @param {Record<string, object>} _index
      * @throws MethodNotImplementedStorageEngineError
-     * @return Promise<void>
+     * @return Promise<Record<String, Object>>
      */
     putSearchIndex(_constructor, _index) {
         return Promise.reject(new MethodNotImplementedStorageEngineError('putSearchIndex', this));