@ai-matrx/auth 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog — @ai-matrx/auth
 
+## 0.3.1
+- `clearStaleHeld` fix: a held choice already stale when the read began is cleared even when no organization can be answered,
+  and a choice made while the read was in flight is never cleared. Test: `active-organization.test.ts`.
+
 ## 0.3.0
 ### auth consolidation tail (round 3)
 

package/dist/organizations.cjs

@@ -583,6 +583,7 @@ function createActiveOrganization(options) {
       const memberships = options.accountChoices ? orderOrganizations(rawMemberships, favourites) : rawMemberships;
       const readAccount = () => account ? Promise.resolve(account) : readAccountOrganizationChoices(client, person.id);
       const before = prior;
+      const heldAtStart = options.clearStaleHeld ? await held.read(person.id) : null;
       const linkOptions = options.link;
       let resolved;
       let linkDecision = null;
@@ -609,15 +610,14 @@ function createActiveOrganization(options) {
         });
       }
       if (gen !== generation) return current;
+      if (heldAtStart && !memberships.some((m) => m.id === heldAtStart) && await held.read(person.id) === heldAtStart) {
+        await held.clear();
+      }
       if (!resolved) {
         publish({ ...IDLE, status: "none", userId: person.id, error: "no-memberships" });
         return current;
       }
       if (options.holdResolved ?? true) await held.write(person.id, resolved.organization.id);
-      if (options.clearStaleHeld) {
-        const stale = await held.read(person.id);
-        if (stale && !memberships.some((m) => m.id === stale)) await held.clear();
-      }
       const wasOrg = before.status === "ready" || before.status === "unchecked" ? before.organization : null;
       const lapsed = wasOrg && wasOrg.id !== resolved.organization.id && !memberships.some((m) => m.id === wasOrg.id) ? wasOrg : null;
       if (linkDecision && linkDecision.kind !== "no-link") options.link?.onDecision?.(linkDecision);

package/dist/organizations.js

@@ -503,6 +503,7 @@ function createActiveOrganization(options) {
       const memberships = options.accountChoices ? orderOrganizations(rawMemberships, favourites) : rawMemberships;
       const readAccount = () => account ? Promise.resolve(account) : readAccountOrganizationChoices(client, person.id);
       const before = prior;
+      const heldAtStart = options.clearStaleHeld ? await held.read(person.id) : null;
       const linkOptions = options.link;
       let resolved;
       let linkDecision = null;
@@ -529,15 +530,14 @@ function createActiveOrganization(options) {
         });
       }
       if (gen !== generation) return current;
+      if (heldAtStart && !memberships.some((m) => m.id === heldAtStart) && await held.read(person.id) === heldAtStart) {
+        await held.clear();
+      }
       if (!resolved) {
         publish({ ...IDLE, status: "none", userId: person.id, error: "no-memberships" });
         return current;
       }
       if (options.holdResolved ?? true) await held.write(person.id, resolved.organization.id);
-      if (options.clearStaleHeld) {
-        const stale = await held.read(person.id);
-        if (stale && !memberships.some((m) => m.id === stale)) await held.clear();
-      }
       const wasOrg = before.status === "ready" || before.status === "unchecked" ? before.organization : null;
       const lapsed = wasOrg && wasOrg.id !== resolved.organization.id && !memberships.some((m) => m.id === wasOrg.id) ? wasOrg : null;
       if (linkDecision && linkDecision.kind !== "no-link") options.link?.onDecision?.(linkDecision);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ai-matrx/auth",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "AI Matrx sign-in, session and organizations for every app: sign in/out, session storage and refresh per platform (browser, Next.js, extension, desktop, Node), the current person, their organizations and favourites, the one active organization (held requests, never a list filter), and the credentials @ai-matrx/data and @ai-matrx/api read.",
   "type": "module",
   "main": "./dist/index.cjs",
@@ -211,5 +211,5 @@
     "test": "vitest run",
     "verify:tarball": "node ./scripts/verify-tarball.mjs"
   },
-  "gitHead": "032b64c11a538c439562215e8dc33fba11bc0050"
+  "gitHead": "63d7fd1430cbe05b6ed9f0e638ae4c630232497a"
 }