pi-hypercharm-provider 1.3.28 → 1.3.29

package/README.md

@@ -138,11 +138,13 @@ DeepSeek V4 models use the `deepseek` thinking format — the same native format
 
 ## Footer Status
 
-A Neuralwatt-style status line sits below the editor. It appears after the
-session's first HyperCharm turn completes (never before — no half-empty line
-on fresh sessions or other providers), refreshes its balance when the agent
-run fully settles, and makes no status-related API calls in sessions that
-never use HyperCharm:
+A Neuralwatt-style status line sits below the editor. It appears as soon as a
+HyperCharm model is selected — the account side (team, balance, rate limits)
+renders when the session-start or model-select credits fetch lands, and the
+session side joins it after the first HyperCharm turn completes. Selecting
+another provider hides it (`hideOnOtherProvider` defaults to `true`), the
+balance refreshes when the agent run fully settles, and sessions that never
+use HyperCharm make no status-related API calls:
 
 ```
 ⚡ 1.24 hc · 7 req                       Xu's Team ◆ 249 hc · 996/1k/h · 29d

package/index.ts

@@ -33,12 +33,15 @@
  *   The right side compresses across progressive tiers as the terminal
  *   narrows. The balance flips to a ⚠ warning at/below lowBalanceHc.
  *
- *   Lifecycle (mirrors pi-neuralwatt-provider): nothing renders before this
- *   session's first HyperCharm turn completes, so fresh sessions and other
- *   providers' sessions see no half-empty line. Credits/team are prefetched
- *   on session start or model select when a HyperCharm model is active, so
- *   the first turn ends with data already cached. The balance is polled
- *   again on pi's agent_settled event (fires only once no automatic retry,
+ *   Lifecycle (mirrors pi-neuralwatt-provider): selecting a HyperCharm model
+ *   shows the line — the account side renders as soon as the credits/team
+ *   prefetch lands, and the session side (spend/requests) joins it on the
+ *   first completed turn. hideOnOtherProvider (default true) clears
+ *   everything the moment the active model belongs to another provider.
+ *   Credits/team are prefetched on session start or model select when a
+ *   HyperCharm model is active, so the first turn ends with data already
+ *   cached. The balance is polled again on pi's agent_settled event (fires
+ *   only once no automatic retry,
  *   compaction, or queued continuation can follow) — and nowhere else, so
  *   sessions without HyperCharm turns make zero status-related API calls.
  *   Between polls the balance moves optimistically: each turn's
@@ -827,10 +830,13 @@ function renderStatus(ctx: ExtensionContext): void {
 		widgetGlyphClampNotified = true;
 		ctx.ui.notify("HyperCharm: widget glyphs stay ASCII on this terminal — unicode glyphs overflow legacy mintty/Cygwin cell widths. Statusbar is unaffected.", "info");
 	}
-	// Show only after HyperCharm activity this session (like pi-neuralwatt):
-	// no empty-gap line on fresh sessions, no stale account glare on other
-	// providers' sessions.
-	const accountVisible = statusConfig.account !== "off" && accountHasData(account) && hasActivity;
+	// Show while HyperCharm is the selected provider — the account side renders
+	// as soon as the session_start/model_select credits fetch lands, with no
+	// need to wait for a turn — and once this session recorded HyperCharm
+	// activity, which is what keeps the line alive after a switch when
+	// hideOnOtherProvider is false. The default true clears it on the switch.
+	const visible = hasActivity || provider === PROVIDER_ID;
+	const accountVisible = statusConfig.account !== "off" && accountHasData(account) && visible;
 	const lowBalance =
 		statusConfig.lowBalanceHc !== null && account.balance !== null && account.balance <= statusConfig.lowBalanceHc;
 	const sessionLine = statusConfig.session !== "off" ? buildSessionLine(sessionStats, glyphs) : undefined;
@@ -1151,7 +1157,7 @@ export default function (pi: ExtensionAPI) {
 
 		loadStatusConfig();
 		resetStatusState();
-		updateStatus(ctx); // clears any carryover; activity-gated, renders nothing yet
+		updateStatus(ctx); // clears any carryover; the account side lands with the credits fetch
 		// Re-register so our identity (custom api + streamSimple) always wins
 		// over anything that touched provider registration during load.
 		pi.registerProvider(PROVIDER_ID, makeProviderConfig());
@@ -1187,8 +1193,11 @@ export default function (pi: ExtensionAPI) {
 		updateStatus(ctx);
 		const model: any = (event as any).model;
 		if (model?.provider === PROVIDER_ID && cachedApiKey) {
+			// Both refreshes repaint when they land: selection alone must fill in
+			// the account side (balance now, team/auth atoms a moment later)
+			// instead of leaving a bare gem until the next turn.
 			updateStatusAfter(refreshCredits(cachedApiKey, statusAbort?.signal ?? undefined, false), ctx);
-			void refreshAccountMeta(cachedApiKey, statusAbort?.signal ?? undefined);
+			updateStatusAfter(refreshAccountMeta(cachedApiKey, statusAbort?.signal ?? undefined), ctx);
 		}
 	});
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hypercharm-provider",
-  "version": "1.3.28",
+  "version": "1.3.29",
   "description": "HyperCharm provider extension for pi - Access DeepSeek, GLM, Kimi, Qwen, MiniMax, Gemma, and GPT-OSS models through the Charm Hyper API",
   "type": "module",
   "main": "index.ts",

package/tests/provider.integration.test.ts

@@ -80,6 +80,11 @@ function captureUI(runner) {
 	const notifications = [];
 	const statusKeys = [];
 	const widgetKeys = [];
+	// Install (component factory) vs clear (undefined), split so a test can
+	// assert both "the widget popped up" and "the widget went away".
+	const widgetInstalls = [];
+	const widgetClears = [];
+	const widgetRenders = [];
 	const base = runner.createContext().ui ?? {};
 	const ui = {
 		...base,
@@ -90,12 +95,24 @@ function captureUI(runner) {
 		setStatus: (key) => {
 			statusKeys.push(String(key));
 		},
-		setWidget: (key) => {
-			widgetKeys.push(String(key));
+		setWidget: (key, value) => {
+			const name = String(key);
+			widgetKeys.push(name);
+			if (value === undefined) {
+				widgetClears.push(name);
+				return;
+			}
+			widgetInstalls.push(name);
+			try {
+				const component = value(undefined, ui.theme);
+				widgetRenders.push(component?.render ? component.render(80).join("\n") : "");
+			} catch {
+				widgetRenders.push("");
+			}
 		},
 	};
 	runner.setUIContext(ui, "tui");
-	return { notifications, statusKeys, widgetKeys };
+	return { notifications, statusKeys, widgetKeys, widgetInstalls, widgetClears, widgetRenders };
 }
 
 async function load(options = {}) {
@@ -414,3 +431,55 @@ test("co-installs with the official identifier surface without interference", as
 		harness.session.dispose();
 	}
 });
+
+test("shows the footer widget on model selection and clears it when the provider is deselected", async () => {
+	const requests = [];
+	const json = (body) => new Response(JSON.stringify(body), { status: 200, headers: { "content-type": "application/json" } });
+	const accountApi = async (input) => {
+		const url = String(input);
+		requests.push(url);
+		if (url.endsWith("/credits")) return json({ balance: 249 });
+		if (url.endsWith("/teams")) return json({ items: [{ name: "Xu's Team" }] });
+		if (url.endsWith("/devices")) return json({ items: [] });
+		if (url.endsWith("/provider")) return json({ models: [FIXTURE_MODEL] });
+		throw new Error("unexpected request: " + url);
+	};
+	const harness = await load({ extensionPaths: [extensionPath, officialSurfacePath], fetchImpl: accountApi });
+	try {
+		// The model_select handler only refreshes credits once session_start has
+		// cached a key, so wait for the catalog refresh that follows resolution.
+		await waitFor("session-start catalog refresh", () =>
+			requests.some((url) => url.endsWith("/provider")) ? true : undefined,
+		);
+
+		const ui = captureUI(harness.runner);
+		const ours = harness.runtime.getModel("hypercharm", "deepseek-v4-flash");
+		assert.ok(ours, "embedded catalog serves deepseek-v4-flash");
+		await harness.session.setModel(ours);
+		// No turn_end is ever emitted in this test: selection alone must install
+		// the widget, with the account side already filled in.
+		await waitFor("widget installed on selection", () => (ui.widgetInstalls.includes("hypercharm") ? true : undefined));
+		const filled = await waitFor("account side in the widget", () =>
+			ui.widgetRenders.find((line) => line.includes("Xu's Team")),
+		);
+		assert.match(filled, /249 hc/, "balance renders from the credits prefetch");
+
+		const installsWhileSelected = ui.widgetInstalls.length;
+		await harness.credentials.modify("hyper", async () => ({ type: "api_key", key: "fixture-official-key" }));
+		const official = harness.runtime.getModel("hyper", "glm-5.3");
+		assert.ok(official, "the official-surface fixture serves glm-5.3");
+		await harness.session.setModel(official);
+		await waitFor("widget cleared on deselect", () => (ui.widgetClears.includes("hypercharm") ? true : undefined));
+
+		// Deselected stays deselected: another provider's turn must not bring the
+		// line back (hideOnOtherProvider defaults to true).
+		await harness.runner.emit({ type: "turn_end", turnIndex: 0, message: assistantMessage(), toolResults: [] });
+		assert.equal(
+			ui.widgetInstalls.length,
+			installsWhileSelected,
+			"no reinstall while another provider's model is active",
+		);
+	} finally {
+		harness.session.dispose();
+	}
+});