jazz-tools 0.20.3 → 0.20.5

package/src/tools/subscribe/SubscriptionCache.ts

@@ -12,6 +12,14 @@ import { SubscriptionScope } from "./SubscriptionScope.js";
 import type { BranchDefinition } from "./types.js";
 import { isEqualRefsToResolve } from "./utils.js";
 
+function copyResolve(resolve: RefsToResolve<any>): RefsToResolve<any> {
+  if (typeof resolve !== "object" || resolve === null) {
+    return resolve;
+  }
+
+  return { ...resolve };
+}
+
 interface CacheEntry {
   subscriptionScope: SubscriptionScope<any>;
   schema: CoValueClassOrSchema;
@@ -233,10 +241,11 @@ export class SubscriptionCache {
     };
 
     // Create cache entry with initial subscriber count (starts at 0)
+    // Clone resolve to prevent mutation by SubscriptionScope.subscribeToKey from affecting cache lookups
     const entry: CacheEntry = {
       subscriptionScope,
       schema,
-      resolve,
+      resolve: copyResolve(resolve),
       branch,
       subscriberCount: subscriptionScope.subscribers.size,
       unsubscribeFromScope: subscriptionScope.onSubscriberChange(

package/src/tools/subscribe/SubscriptionScope.ts

@@ -153,7 +153,7 @@ export class SubscriptionScope<D extends CoValue> {
   }
 
   trackLoadingPerformance(source: string) {
-    if (!SubscriptionScope.isProfilingEnabled) {
+    if (!SubscriptionScope.isProfilingEnabled || !crypto.randomUUID) {
       return;
     }
 

package/src/tools/tests/testStorage.ts

@@ -45,26 +45,28 @@ class LibSQLSqliteAsyncDriver implements SQLiteDatabaseDriverAsync {
   }
 }
 
+function deleteDb(dbPath: string) {
+  try {
+    unlinkSync(dbPath);
+  } catch (error) {
+    console.error(error);
+  }
+}
+
 export async function createAsyncStorage({ filename }: { filename?: string }) {
+  const dbPath = getDbPath(filename);
   const storage = await getSqliteStorageAsync(
-    new LibSQLSqliteAsyncDriver(getDbPath(filename)),
+    new LibSQLSqliteAsyncDriver(dbPath),
   );
 
   onTestFinished(async () => {
     await storage.close();
+    deleteDb(dbPath);
   });
 
   return storage;
 }
 
 export function getDbPath(defaultDbPath?: string) {
-  const dbPath = defaultDbPath ?? join(tmpdir(), `test-${randomUUID()}.db`);
-
-  if (!defaultDbPath) {
-    onTestFinished(() => {
-      unlinkSync(dbPath);
-    });
-  }
-
-  return dbPath;
+  return defaultDbPath ?? join(tmpdir(), `test-${randomUUID()}.db`);
 }