fix: skip accounts with all workspaces disabled during rotation

Accounts with tracked workspaces could remain eligible for family
selection even after every workspace had been disabled. That leaves a
workspace-exhausted account in the rotation pool and can block fallback
selection unnecessarily.

Treat accounts whose tracked workspaces are all disabled as unavailable
for selection, while keeping legacy workspace-less accounts eligible.

Author: ndycode

lib/accounts.ts

@@ -535,6 +535,7 @@ export class AccountManager {
 		const account = this.getAccountByIndex(index);
 		if (!account) return false;
 		if (account.enabled === false) return false;
+		if (!this.hasEnabledWorkspaces(account)) return false;
 		clearExpiredRateLimits(account);
 		return (
 			!isRateLimitedForFamily(account, family, model) &&
@@ -606,6 +607,7 @@ export class AccountManager {
 			const account = this.accounts[idx];
 			if (!account) continue;
 			if (account.enabled === false) continue;
+			if (!this.hasEnabledWorkspaces(account)) continue;
 
 			clearExpiredRateLimits(account);
 			if (
@@ -636,6 +638,7 @@ export class AccountManager {
 			const account = this.accounts[idx];
 			if (!account) continue;
 			if (account.enabled === false) continue;
+			if (!this.hasEnabledWorkspaces(account)) continue;
 
 			clearExpiredRateLimits(account);
 			if (
@@ -666,6 +669,7 @@ export class AccountManager {
 			.map((account): AccountWithMetrics | null => {
 				if (!account) return null;
 				if (account.enabled === false) return null;
+				if (!this.hasEnabledWorkspaces(account)) return null;
 				clearExpiredRateLimits(account);
 				const isAvailable =
 					!isRateLimitedForFamily(account, family, model) &&

test/accounts.test.ts

@@ -427,6 +427,55 @@ describe("AccountManager", () => {
     expect(manager.isAccountAvailableForFamily(3, "codex")).toBe(true);
   });
 
+  it("treats accounts with all tracked workspaces disabled as unavailable for selection", () => {
+    const now = Date.now();
+    const stored = {
+      version: 3 as const,
+      activeIndex: 0,
+      activeIndexByFamily: { codex: 0 },
+      accounts: [
+        {
+          refreshToken: "token-workspace-disabled",
+          addedAt: now,
+          lastUsed: now,
+          workspaces: [
+            { id: "workspace-1", name: "Workspace 1", enabled: false },
+            { id: "workspace-2", name: "Workspace 2", enabled: false },
+          ],
+          currentWorkspaceIndex: 0,
+        },
+        {
+          refreshToken: "token-ready",
+          addedAt: now,
+          lastUsed: now - 10_000,
+        },
+      ],
+    };
+
+    const manager = new AccountManager(undefined, stored as never);
+
+    expect(manager.isAccountAvailableForFamily(0, "codex")).toBe(false);
+    expect(manager.getCurrentOrNextForFamily("codex")?.refreshToken).toBe("token-ready");
+    expect(manager.getNextForFamily("codex")?.refreshToken).toBe("token-ready");
+    expect(manager.getCurrentOrNextForFamilyHybrid("codex")?.refreshToken).toBe("token-ready");
+  });
+
+  it("keeps workspace-less legacy accounts eligible for selection", () => {
+    const now = Date.now();
+    const stored = {
+      version: 3 as const,
+      activeIndex: 0,
+      activeIndexByFamily: { codex: 0 },
+      accounts: [{ refreshToken: "token-legacy", addedAt: now, lastUsed: now }],
+    };
+
+    const manager = new AccountManager(undefined, stored as never);
+
+    expect(manager.hasEnabledWorkspaces(manager.getAccountByIndex(0)!)).toBe(true);
+    expect(manager.isAccountAvailableForFamily(0, "codex")).toBe(true);
+    expect(manager.getCurrentOrNextForFamily("codex")?.refreshToken).toBe("token-legacy");
+  });
+
   it("returns false for invalid account index in availability checks", () => {
     const now = Date.now();
     const stored = {