@bevel-software/platform-core-backend 0.28.1 → 0.30.0

package/src/shared/workspace-id.ts

@@ -34,3 +34,27 @@ export function branchForWorkspaceId(workspaceId: string): string {
     return workspaceId;
   }
 }
+
+/**
+ * Canonicalize a workspace id so every row keyed on one — a queued save, a
+ * file lock — and every reader of those rows key on the SAME string.
+ *
+ * The id reaches `enqueue` and a lock's `acquire` via a route `:id` path
+ * param, which Express URL-decodes — so a slashed feature branch arrives as `alice/feature`. But the
+ * worker claims per `WorkspaceService.knownWorkspaces()`, whose ids are
+ * `encodeURIComponent(branch)` → `alice%2Ffeature`. Those strings differ, so
+ * `WHERE workspace_id = …` never matched and the row was never drained —
+ * silently stranding every human save on a feature branch (protected branches
+ * have no `/`, so encoded == decoded and they were unaffected).
+ *
+ * `encodeURIComponent(decodeURIComponent(id))` is idempotent and collapses both
+ * forms to the encoded id `knownWorkspaces()` uses. A malformed `%` sequence
+ * (which `decodeURIComponent` would throw on) is left untouched.
+ */
+export function canonicalWorkspaceId(id: string): string {
+  try {
+    return encodeURIComponent(decodeURIComponent(id));
+  } catch {
+    return id;
+  }
+}

package/src/test-setup.ts

@@ -1,4 +1,6 @@
 import { branchModelFromEnv, configureBranchModel } from '@bevel-software/platform-shared';
+import { afterAll } from 'vitest';
+import { removeTestRepositoryRoots } from './__tests__/test-repositories.js';
 
 /**
  * Apply the branch model before any suite imports something that reads it.
@@ -16,3 +18,6 @@ import { branchModelFromEnv, configureBranchModel } from '@bevel-software/platfo
  * place the historical two-branch pair is written.
  */
 configureBranchModel(branchModelFromEnv());
+
+// The repositories `testKbRepository` placed beside each suite's fixture root.
+afterAll(removeTestRepositoryRoots);