@almadar/workspace 0.11.12 → 0.11.13

package/dist/__tests__/race-restore-vs-create.test.d.ts

@@ -0,0 +1,20 @@
+/**
+ * Regression test for the create:false / create mint race (2026-09-04 Kura
+ * first-open defect).
+ *
+ * `openWorkspace({ create: false, ... })` is a "pure read" only when the
+ * local cache already has the workspace. When it misses locally but a
+ * `restore` backend yields files, `resolveLifecycle`'s restore branch mints
+ * a fresh session dir via `mintSessionDir` — the same directory-creating
+ * path a concurrent `create` (default) call takes. Before the fix,
+ * `create:false` bypassed the in-flight dedupe map entirely, so a
+ * `GET .../git/history` (`create:false`) racing a `GET .../files`
+ * (`create`) on first open both missed `findLocalWorkspaceDir`, both
+ * restored, and two workspace directories were minted for one
+ * (userId, appId).
+ *
+ * After the fix, both paths share one in-flight map keyed by
+ * (backend, userId, appId): whichever call resolves first wins, and the
+ * other awaits it instead of minting its own directory.
+ */
+export {};

package/dist/index.js

@@ -2511,28 +2511,32 @@ function openWorkspaceCacheKey(opts) {
   const backendTag = opts.backend === "memory" ? "memory" : "local";
   return `${backendTag}:${opts.userId}:${opts.appId}`;
 }
+function registerInFlightOpen(key, opts) {
+  const pending = (async () => {
+    try {
+      const ws = await openWorkspaceInternal(opts);
+      if (ws === null) openWorkspaceInFlight.delete(key);
+      return ws;
+    } catch (err) {
+      openWorkspaceInFlight.delete(key);
+      throw err;
+    }
+  })();
+  openWorkspaceInFlight.set(key, pending);
+  return pending;
+}
 async function openWorkspace(opts) {
-  if (opts.create === false) {
+  const key = openWorkspaceCacheKey(opts);
+  if (key === null) {
     return openWorkspaceInternal(opts);
   }
-  const key = openWorkspaceCacheKey(opts);
-  if (key !== null) {
+  for (; ; ) {
     const cached = openWorkspaceInFlight.get(key);
-    if (cached) return cached;
-    const pending = (async () => {
-      try {
-        const ws = await openWorkspaceInternal(opts);
-        if (ws === null) throw new Error("openWorkspace: null resolution without create:false");
-        return ws;
-      } catch (err) {
-        openWorkspaceInFlight.delete(key);
-        throw err;
-      }
-    })();
-    openWorkspaceInFlight.set(key, pending);
-    return pending;
+    if (!cached) return registerInFlightOpen(key, opts);
+    const result = await cached;
+    if (result !== null) return result;
+    if (opts.create === false) return null;
   }
-  return openWorkspaceInternal(opts);
 }
 var WORKSPACE_OWNED_ENTRIES = /* @__PURE__ */ new Set([".almadar", "orbitals", "schema.orb", ".git", ".gitignore"]);
 async function guardForeignMintTarget(backend, workDir) {