fix: fix memory error when closing entities.

Entities were getting freed before they had finished getting saved.

Author: zicklag

change/@muni-town-leaf-ed823780-b098-440a-ba32-1651b34b201a.json

@@ -3,5 +3,5 @@
   "comment": "fix memory error when closing entities.",
   "packageName": "@muni-town/leaf",
   "email": "[email protected]",
-  "dependentChangeType": "patch"
+  "dependentChangeType": "prerelease"
 }

package.json

@@ -4,6 +4,7 @@
     "url": "https://github.com/muni-town/leaf"
   },
   "scripts": {
+    "change": "npx beachball change",
     "build-api-docs": "turbo build-api-docs",
     "syncserver": "PORT=8095 DB_FILE=./data/data.sqlite turbo syncserver",
     "publish-packages": "turbo build && beachball publish -m 'chore: publish packages.'"

packages/leaf/src/index.ts

@@ -643,7 +643,6 @@ export class Peer {
 
   /** Commit the entity, stop syncing it, and flush it to storage. */
   close(id: IntoEntityId): Promise<void> {
-    // console.log('close');
     return new Promise((resolve) => {
       // NOTE: we queue a microtask here because if you have _just_ committed an entity, and then
       // you call this function, the change callbacks on the entity have not yet bent run, and the
@@ -660,7 +659,11 @@ export class Peer {
           if (entity) {
             // This will trigger a write to storage
             entity.doc.commit();
-            entity.free();
+            // For the same reason mentioned in the nte above, we have to wait until the next tick
+            // to actually free the entity after the commit.
+            queueMicrotask(() => {
+              entity.free();
+            });
           }
 
           this.#rawUnload(entIdStr);