@acodeninja/persist 3.0.0-next.17 → 3.0.0-next.19

package/README.md

@@ -9,6 +9,7 @@ A JSON based data modelling and persistence library with alternate storage mecha
 
 [![DeepSource](https://app.deepsource.com/gh/acodeninja/persist.svg/?label=active+issues&show_trend=true&token=Vd8_PJuRwwoq4_uBJ0_ymc06)](https://app.deepsource.com/gh/acodeninja/persist/)
 [![DeepSource](https://app.deepsource.com/gh/acodeninja/persist.svg/?label=code+coverage&show_trend=true&token=Vd8_PJuRwwoq4_uBJ0_ymc06)](https://app.deepsource.com/gh/acodeninja/persist/)
+![CodeRabbit PR Reviews](https://img.shields.io/coderabbit/prs/github/acodeninja/persist?utm_source=oss&utm_medium=github&utm_campaign=acodeninja%2Fpersist&labelColor=171717&color=FF570A&link=https%3A%2F%2Fcoderabbit.ai&label=CodeRabbit+Reviews)
 
 ## Features
 
@@ -52,6 +53,7 @@ class Person extends Persist.Model {
 
 ```javascript
 import Persist from '@acodeninja/persist';
+import {S3Client} from "@aws-sdk/client-s3";
 import S3StorageEngine from '@acodeninja/persist/storage/s3';
 
 const engine = new S3StorageEngine({
@@ -74,7 +76,7 @@ await connection.put(person);
 ## Find out more
 
 - [Defining Models](./docs/defining-models.md)
-- [Model Property Types](./docs/model-properties)
+- [Model Property Types](./docs/model-properties.md)
 - [Models as Properties](./docs/models-as-properties.md)
 - [Structured Queries](./docs/structured-queries.md)
 - [Search Queries](./docs/search-queries.md)

package/docs/storage-engines.md

@@ -8,6 +8,7 @@ To store models using an S3 Bucket, use the `S3` storage engine. To use the `S3`
 
 ```javascript
 import Persist from "@acodeninja/persist";
+import {S3Client} from "@aws-sdk/client-s3";
 import S3StorageEngine from "@acodeninja/persist/storage/s3";
 
 const connection = Persist.registerConnection('remote', new S3StorageEngine({

package/package.json

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

package/src/Connection.js

@@ -272,7 +272,7 @@ export default class Connection {
             if (!Object.keys(this.#models).includes(modelToProcess.constructor.name))
                 throw new ModelNotRegisteredConnectionError(modelToProcess, this.#storage);
 
-            const currentModel = modelCache[model.id] ?? await this.get(model.id);
+            const currentModel = modelCache[modelToProcess.id] ?? await this.get(modelToProcess.id);
             modelCache[currentModel.id] = currentModel;
 
             if (!modelsToDelete.includes(currentModel.id)) modelsToDelete.push(currentModel.id);
@@ -458,9 +458,11 @@ export default class Connection {
                     }
                 }
             } catch (error) {
-                for (const transaction of transactions.filter(t => t.committed && t.original)) {
-                    this.#storage.putModel(transaction.original);
-                }
+                await Promise.all(
+                    transactions
+                        .filter(t => t.committed && t.original)
+                        .map(t => this.#storage.putModel(t.original)),
+                );
 
                 throw new CommitFailedTransactionError(transactions, error);
             }