fix(core,desktop): clear subscription usage when logged out

Hide and purge cached quota when subscription OAuth is missing or
logged out; reject usage queries without credentials on the core side.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: joohw

core/internal/buildinfo/buildinfo.go

@@ -4,7 +4,7 @@ import "strings"
 
 // Set at link time via -ldflags (see .goreleaser.yaml).
 var (
-	Version = "dev0.1.103"
+	Version = "dev0.1.104"
 	Commit  = "none"
 	Date    = "unknown"
 )

core/internal/desktop/usage.go

@@ -80,6 +80,9 @@ func resolveVendorCredentials(vendor profile.Profile) (baseURL, apiKey, template
 func querySubscriptionVendorUsage(vendor profile.Profile) usage.Result {
 	flat := vendor
 	profile.HydrateSubscriptionCredentials(&flat)
+	if strings.TrimSpace(flat.APIKey) == "" {
+		return usage.Result{Success: false, Kind: "subscription", Error: "subscription not logged in"}
+	}
 	return usage.QuerySubscriptionUsage(flat.SubscriptionProviderID, flat.APIKey, flat.AccountID)
 }
 

core/internal/desktop/usage_test.go

@@ -25,3 +25,19 @@ func TestResolveVendorCredentialsFallsBackToModelConnection(t *testing.T) {
 		t.Fatalf("template = %q want %q", template, usage.TemplateAuto)
 	}
 }
+
+func TestQuerySubscriptionVendorUsageRequiresLogin(t *testing.T) {
+	profile.SetClaudeCredentialsPathOverride(t.TempDir() + "/missing-claude.json")
+	t.Cleanup(func() { profile.SetClaudeCredentialsPathOverride("") })
+
+	result := querySubscriptionVendorUsage(profile.Profile{
+		Kind:                   "subscription",
+		SubscriptionProviderID: "claude-code",
+	})
+	if result.Success {
+		t.Fatal("expected usage query to fail without subscription credentials")
+	}
+	if result.Error != "subscription not logged in" {
+		t.Fatalf("error = %q want %q", result.Error, "subscription not logged in")
+	}
+}

electron/package.json

@@ -1,7 +1,7 @@
 {
   "name": "clovapi-switcher",
   "private": true,
-  "version": "0.2.12",
+  "version": "0.2.13",
   "description": "ClovAPI Switcher desktop app",
   "main": "main.js",
   "type": "commonjs",

electron/ui/src/components/ProfilesPanel.svelte

@@ -9,6 +9,7 @@
     subscriptionStatusForVendor,
     vendorSummaryLine,
     subscriptionIsUsable,
+    shouldShowVendorUsage,
   } from "../lib/helpers";
   import { displayVendorName, i18n, t } from "../lib/i18n";
   import {
@@ -17,7 +18,7 @@
     openProfilesVendor,
     queryVendorUsage,
     store,
-    vendorUsageSummary,
+    vendorUsageSummaryForVendor,
     canFetchVendorModels,
   } from "../lib/store.svelte";
   import ListRow from "./ListRow.svelte";
@@ -75,7 +76,7 @@
 
   function canQueryUsage(vendor: (typeof vendorList)[number]) {
     if (vendor.kind === "subscription") {
-      return subscriptionStatusForVendor(vendor, store.subscriptions)?.active === true;
+      return shouldShowVendorUsage(vendor, store.subscriptions);
     }
     if (vendor.kind === "api") {
       return Boolean((vendor.baseUrl && vendor.apiKey) || vendor.models?.some((model) => model.baseUrl && model.apiKey));
@@ -86,7 +87,7 @@
   function summaryLine(vendor: (typeof vendorList)[number]) {
     const sub = subscriptionStatusForVendor(vendor, store.subscriptions);
     const base = vendorSummaryLine(vendor, sub, store.ollamaInstalled);
-    const usage = vendor.kind === "api" || vendor.kind === "subscription" ? vendorUsageSummary(vendor.name) : "";
+    const usage = vendorUsageSummaryForVendor(vendor, store.subscriptions);
     return usage ? `${base} · ${usage}` : base;
   }
 

electron/ui/src/components/VendorDetailPanel.svelte

@@ -23,7 +23,7 @@
     isSubscriptionLogging,
     openModelDialog,
     openProfileDialog,
-    vendorUsageSummary,
+    vendorUsageSummaryForVendor,
     removeVendorModel,
     runModelTest,
     runSubscriptionLogin,
@@ -90,7 +90,7 @@
 
   const cardDescription = $derived.by(() => {
     void i18n.locale;
-    const usage = vendor.kind === "api" || vendor.kind === "subscription" ? vendorUsageSummary(vendor.name) : "";
+    const usage = vendorUsageSummaryForVendor(vendor, store.subscriptions);
     const parts: string[] = [];
     if (vendor.kind === "subscription") {
       parts.push(formatSubscriptionSummary(subscription?.summary || ""));

electron/ui/src/lib/helpers.ts

@@ -467,6 +467,19 @@ export function subscriptionIsUsable(subscription: SubscriptionItem | undefined)
   return subscription.active !== false;
 }
 
+/** Whether vendor quota/balance should be queried or shown in the UI. */
+export function shouldShowVendorUsage(vendor: Vendor, subscriptions: SubscriptionItem[]): boolean {
+  if (vendor.kind === "local") return false;
+  if (vendor.kind === "subscription") {
+    return subscriptionIsUsable(subscriptionStatusForVendor(vendor, subscriptions));
+  }
+  if (vendor.kind === "api") {
+    if (vendor.baseUrl && vendor.apiKey) return true;
+    return Boolean(vendor.models?.some((model) => model.baseUrl && model.apiKey));
+  }
+  return false;
+}
+
 export type CliBindingOption = {
   value: string;
   label: string;

electron/ui/src/lib/store/store.svelte.ts

@@ -27,7 +27,13 @@ export {
   removeVendorModel,
 } from "./profiles";
 export { canFetchVendorModels, isVendorFetching, fetchVendorModels } from "./vendor-models";
-export { queryVendorUsage, vendorUsageSummary, isVendorUsageLoading } from "./vendor-usage";
+export {
+  queryVendorUsage,
+  vendorUsageSummary,
+  vendorUsageSummaryForVendor,
+  clearVendorUsage,
+  isVendorUsageLoading,
+} from "./vendor-usage";
 export {
   refreshProxyStatus,
   refreshProxyLogs,

electron/ui/src/lib/store/subscriptions.ts

@@ -13,6 +13,7 @@ import { clearModelTest } from "./model-tests";
 import { runModelTest } from "./model-runner";
 import { loadProfilesFromDisk, persistProfiles } from "./profiles";
 import { fetchVendorModels } from "./vendor-models";
+import { clearVendorUsage, pruneVendorUsageForSubscriptions } from "./vendor-usage";
 import { store } from "./state.svelte";
 import { toast } from "../toast";
 import type { SubscriptionItem, SubscriptionVendorRow } from "../../global";
@@ -39,6 +40,7 @@ export async function refreshSubscriptions() {
   } catch {
     store.subscriptions = [];
   }
+  pruneVendorUsageForSubscriptions(store.subscriptions);
 }
 
 export async function cancelSubscriptionLogin(providerId: string) {
@@ -98,6 +100,7 @@ export async function runSubscriptionLogout(providerId: string, label: string) {
   const previousVendor = getSubscriptionVendors(store.profiles).find(
     (item) => item.subscriptionProviderId === providerId,
   );
+  if (previousVendor?.name) clearVendorUsage(previousVendor.name);
   const result = await bridge.authLogout(providerId);
   if (!result?.ok) {
     toast.error(result?.error || t("toast.logoutFailed"));

electron/ui/src/lib/store/vendor-usage.ts

@@ -1,4 +1,5 @@
-import type { Vendor, VendorUsageData, VendorUsageResult } from "../../global";
+import type { SubscriptionItem, Vendor, VendorUsageData, VendorUsageResult } from "../../global";
+import { getSubscriptionVendors, shouldShowVendorUsage } from "../helpers";
 import { t } from "../i18n";
 import { toast } from "../toast";
 import { store } from "./state.svelte";
@@ -7,6 +8,26 @@ export function vendorUsageSummary(vendorName: string): string {
   return store.vendorUsage[vendorName]?.summary || "";
 }
 
+export function vendorUsageSummaryForVendor(vendor: Vendor, subscriptions: SubscriptionItem[]): string {
+  if (!shouldShowVendorUsage(vendor, subscriptions)) return "";
+  return vendorUsageSummary(vendor.name);
+}
+
+export function clearVendorUsage(vendorName: string): void {
+  const name = String(vendorName || "").trim();
+  if (!name) return;
+  delete store.vendorUsage[name];
+  delete store.vendorUsageLoading[name];
+}
+
+export function pruneVendorUsageForSubscriptions(subscriptions: SubscriptionItem[]): void {
+  for (const vendor of getSubscriptionVendors(store.profiles)) {
+    if (!shouldShowVendorUsage(vendor, subscriptions)) {
+      clearVendorUsage(vendor.name);
+    }
+  }
+}
+
 export function isVendorUsageLoading(vendorName: string): boolean {
   return Boolean(store.vendorUsageLoading[vendorName]);
 }
@@ -74,6 +95,10 @@ export async function queryVendorUsage(vendor: Vendor, options: { silent?: boole
   const name = String(vendor?.name || "").trim();
   if (!name) return;
   if (vendor.kind === "local") return;
+  if (vendor.kind === "subscription" && !shouldShowVendorUsage(vendor, store.subscriptions)) {
+    clearVendorUsage(name);
+    return;
+  }
   if (vendor.kind === "api" && !vendorHasUsageCredentials(vendor)) return;
   const bridge = window.clovapiCli;
   if (!bridge?.profilesUsage) {
@@ -87,7 +112,12 @@ export async function queryVendorUsage(vendor: Vendor, options: { silent?: boole
     const result = await bridge.profilesUsage(name);
     if (!result?.ok || !result.usage?.success) {
       const message = result?.error || result?.usage?.error || t("toast.vendorUsageFailed");
-      if (options.silent) return;
+      if (vendor.kind === "subscription") {
+        clearVendorUsage(name);
+        if (options.silent) return;
+      } else if (options.silent) {
+        return;
+      }
       store.vendorUsage[name] = { summary: message, rows: [], error: message };
       if (!options.silent) toast.error(message);
       return;