jazz-tools 0.10.14 → 0.10.15

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> jazz-tools@0.10.14 build /home/runner/_work/jazz/jazz/packages/jazz-tools
+> jazz-tools@0.10.15 build /home/runner/_work/jazz/jazz/packages/jazz-tools
 > tsup
 
 CLI Building entry: {"index":"src/index.ts","testing":"src/testing.ts"}
@@ -10,9 +10,9 @@
 CLI Cleaning output folder
 ESM Build start
 ESM dist/index.js              1.50 KB
-ESM dist/testing.js            6.28 KB
-ESM dist/chunk-5YDDEUNX.js     115.67 KB
+ESM dist/testing.js            6.18 KB
+ESM dist/chunk-5USJBXLW.js     116.50 KB
 ESM dist/index.js.map          259.00 B
-ESM dist/testing.js.map        12.41 KB
-ESM dist/chunk-5YDDEUNX.js.map 275.90 KB
-ESM ⚡️ Build success in 38ms
+ESM dist/testing.js.map        12.24 KB
+ESM dist/chunk-5USJBXLW.js.map 277.65 KB
+ESM ⚡️ Build success in 59ms

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # jazz-tools
 
+## 0.10.15
+
+### Patch Changes
+
+- 2f99de0: Avoid race conditions on context creation and run the anonymous migration only when the hook is provided
+- Updated dependencies [f86e278]
+  - cojson@0.10.15
+
 ## 0.10.14
 
 ### Patch Changes

package/dist/{chunk-5YDDEUNX.js → chunk-5USJBXLW.js}

@@ -3591,7 +3591,7 @@ import { cojsonInternals as cojsonInternals4 } from "cojson";
 var JazzContextManager = class {
   constructor() {
     this.authSecretStorage = new AuthSecretStorage();
-    this.authenticating = false;
+    this.keepContextOpen = false;
     this.logOut = async () => {
       if (!this.context || !this.props) {
         return;
@@ -3606,6 +3606,14 @@ var JazzContextManager = class {
       }
       this.context.done();
     };
+    this.shouldMigrateAnonymousAccount = async () => {
+      if (!this.props?.onAnonymousAccountDiscarded) {
+        return false;
+      }
+      const prevCredentials = await this.authSecretStorage.get();
+      const wasAnonymous = this.authSecretStorage.getIsAuthenticated(prevCredentials) === false;
+      return wasAnonymous;
+    };
     /**
      * Authenticates the user with the given credentials
      */
@@ -3614,13 +3622,12 @@ var JazzContextManager = class {
         throw new Error("Props required");
       }
       const prevContext = this.context;
-      const prevCredentials = await this.authSecretStorage.get();
-      const wasAnonymous = this.authSecretStorage.getIsAuthenticated(prevCredentials) === false;
-      this.authenticating = true;
+      const migratingAnonymousAccount = await this.shouldMigrateAnonymousAccount();
+      this.keepContextOpen = migratingAnonymousAccount;
       await this.createContext(this.props, { credentials }).finally(() => {
-        this.authenticating = false;
+        this.keepContextOpen = false;
       });
-      if (wasAnonymous) {
+      if (migratingAnonymousAccount) {
         await this.handleAnonymousAccountMigration(prevContext);
       }
     };
@@ -3629,18 +3636,17 @@ var JazzContextManager = class {
         throw new Error("Props required");
       }
       const prevContext = this.context;
-      const prevCredentials = await this.authSecretStorage.get();
-      const wasAnonymous = this.authSecretStorage.getIsAuthenticated(prevCredentials) === false;
-      this.authenticating = true;
+      const migratingAnonymousAccount = await this.shouldMigrateAnonymousAccount();
+      this.keepContextOpen = migratingAnonymousAccount;
       await this.createContext(this.props, {
         newAccountProps: {
           secret: accountSecret,
           creationProps
         }
       }).finally(() => {
-        this.authenticating = false;
+        this.keepContextOpen = false;
       });
-      if (wasAnonymous) {
+      if (migratingAnonymousAccount) {
         await this.handleAnonymousAccountMigration(prevContext);
       }
       if (this.context && "me" in this.context) {
@@ -3661,12 +3667,27 @@ var JazzContextManager = class {
     return new InMemoryKVStore();
   }
   async createContext(props, authProps) {
+    this.props = props;
+    const { promise, resolve } = createResolvablePromise();
+    const prevPromise = this.contextPromise;
+    this.contextPromise = promise;
+    await prevPromise;
+    try {
+      const result = await this.getNewContext(props, authProps);
+      await this.updateContext(props, result, authProps);
+      resolve();
+    } catch (error) {
+      resolve();
+      throw error;
+    }
+  }
+  async getNewContext(props, authProps) {
     props;
     authProps;
     throw new Error("Not implemented");
   }
   async updateContext(props, context, authProps) {
-    if (!this.authenticating) {
+    if (!this.keepContextOpen) {
       this.context?.done();
     }
     this.context = context;
@@ -3728,6 +3749,13 @@ var JazzContextManager = class {
     }
   }
 };
+function createResolvablePromise() {
+  let resolve;
+  const promise = new Promise((res) => {
+    resolve = res;
+  });
+  return { promise, resolve };
+}
 
 // src/auth/DemoAuth.ts
 var DemoAuth = class {
@@ -4045,4 +4073,4 @@ export {
   consumeInviteLink
 };
 /* istanbul ignore file -- @preserve */
-//# sourceMappingURL=chunk-5YDDEUNX.js.map
+//# sourceMappingURL=chunk-5USJBXLW.js.map