@bismawy/pi-arnative 0.5.2 → 0.5.3

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## [0.5.3] - 2026-10-14
+
+### Fixed
+- A new session showed the previous session's token counts. The footer's 2s `getUsage` cache is keyed on branch length, and the module is not re-imported by `/new`, so a fresh session whose branch happened to reach the same length read the cached sums instead of its own. The cache is now cleared in the non-reload `session_start` branch (`extensions/footer.ts`).
+- **The cell was invisible: the footer still matched the value on `jev-eye`.** The value became `Eye: ● ON (Clef Flash)`, which contains no `jev-eye` anywhere, so `value.includes("jev-eye")` went false and the whole cell fell through to the generic path — no icon, no dim backend suffix, just grey text. The router now recognises the cell by the label its value carries (`isJevEyeStatus`, `^\s*Eye\s*:`), and the self-check pins that rule from both sides: every mode's real value is recognised, `Jev: ● ON` and `MCP: 2 servers enabled` are not, and the value is asserted to contain no internal key name. Proved negative: reverting the rule to the substring test makes both `node extensions/footer.ts` and pi-jev-eye's `render-check` fail.
+- **One space after the glyph, not two.** The two spaces existed because the previous glyph's ink overran its cell (`\uedcf`); with `\uf21b` in place they read as a gap in `  Eye: ● ON (Clef Flash)`. Pinned in pi-jev-eye's render check as one space, with an explicit assert that the glyph is not followed by two.
+
+### Changed
+- **The cell glyph is U+F21B (`fa-user_secret`), chosen by the operator.** U+EDCF (`fa-user_shield`) was the previous one; both exist in JetBrainsMono Nerd Font, and the codepoint is pinned by number in pi-jev-eye's render check because a Nerd glyph that renders blank on another platform is the bug the ponytail cell had (`\uf51` -> `\uef04`).
+- **The pi-jev-eye status cell renders `Eye: ● ON (Backend)`, parsed from the value instead of hardcoded.** The format is `Eye: ● ON (Clef Flash)` when a Clef model answers and `Eye: ● ON (Jev)` when TypeSafe or OpenRouter does. `jevEyeStatusLine()` reads the label before the first colon and the backend from the trailing parentheses — so nothing about the cell is a literal in this repo, and an older pi-jev-eye value with no suffix gains no guessed one.
+- **The backend suffix takes `dim`, while `Eye:` and the state keep their tiers.** A backend painted like the label would bury `Eye: ● ON`, which is the case this cell exists for; the spaced `Clef Flash` (not a nested `clef (flash)`) reads as one label on a single line.
+- **The status line moved out of the render closure**, next to `ponytailStatusLine`, so the self-check in `node extensions/footer.ts` drives the exact path the footer uses instead of a copy of it.
+- Dropped two unused imports (`CustomMessageComponent`, `isPointerOrFocusInput`) and corrected three stale statements: the usage estimate comment named `prices.json` where the code reads `models.json`, the footer header listed two of the three footer presets, and the README's local-test snippet ran one extension where `npm run dev` runs all eight (`extensions/ui-render-tweaks.ts`, `lib/usage-store.ts`, `extensions/footer.ts`, `test/footer-minimal.test.ts`, `README.md`).
+
+### Added
+- Self-check for the pi-jev-eye cell: both requested forms (`Eye: ● ON (Clef Flash)`, `Eye: ● ON (Jev)`), the tierless `(Clef)`, an upstream `(Clef Omni)`, `OFF` taking the dim bullet, `REVIEW` keeping its warning bullet, and a suffix-less value drawing no parenthesis.
+
 ## [0.5.2] - 2026-10-13
 
 ### Fixed

package/README.md

@@ -46,7 +46,7 @@ Apply a theme via `/settings` → **Theme** → `arnative` (or variant). Require
 
 To test locally:
 ```bash
-pi --extension ./extensions/footer.ts
+npm run dev
 ```
 
 ## Shortcuts

package/extensions/footer.ts

@@ -2,7 +2,8 @@
  * Arnative footer (fixed 3-row grid). Row 1: cwd | active time | branch — model + thinking;
  * rows 2-3: extension status | tok/s · cache · token, each side wrapped in its half of the
  * width. The clock counts active time, not wall-clock age — see {@link ActiveClock}.
- * Preset (`/arnative footers`), live: "Arnative (Full)" (default) or "Pi (system)".
+ * Preset (`/arnative footers`), live: "Arnative (Full)" (default), "Arnative (Minimal)"
+ * or "Pi (system)".
  */
 import type { AssistantMessage } from "@earendil-works/pi-ai";
 import { CustomEditor, FooterComponent, type ExtensionAPI, type ExtensionContext } from "@earendil-works/pi-coding-agent";
@@ -210,6 +211,55 @@ export function currentToolVerb(active: ReadonlyMap<string, string>): string {
 	return verb;
 }
 
+/**
+ * The pi-jev-eye status cell: `Eye: ● MODE (Backend)`. pi-jev-eye writes the value plain because a status
+ * is a fact, not a layout; this decides the painting.
+ *
+ * The trailing parenthesised backend is a modifier, so it takes `dim` while the `Eye:` label and the state
+ * keep the soft and state tiers — that is what makes `(Clef Flash)` recede and `Eye: ● ON` stay readable
+ * at a glance. The backend name itself is read out of the value, never hardcoded here: the extension
+ * reports the active backend from `/jev-eye models`, so the footer cannot know it in advance.
+ */
+/**
+ * Is this status value pi-jev-eye's own cell? It is matched on the LABEL the value carries (`Eye:`), not on
+ * the internal key name: the value is `Eye: ● ON (Clef Flash)` and contains no `jev-eye` anywhere, so the
+ * old `value.includes("jev-eye")` test went false and the whole cell fell through to the generic path —
+ * which is exactly how it silently lost its icon and its dim backend suffix.
+ *
+ * Recognising the label keeps the rule honest: if pi-jev-eye renames its label, this test fails loudly in
+ * the self-check below instead of degrading to a plain grey cell.
+ */
+export const isJevEyeStatus = (status: string): boolean => /^\s*Eye\s*:/i.test(stripAnsi(status));
+
+export function jevEyeStatusLine(
+	status: string,
+	acc: (text: string) => string,
+	dim: (text: string) => string,
+	soft: (text: string) => string,
+	theme?: { fg?: (color: string, text: string) => string },
+): string {
+	const clean = stripAnsi(status);
+	const isOff = clean.includes("OFF") || clean.includes("\u25CB");
+	const isReview = /REVIEW/i.test(clean);
+	let bullet = isOff ? dim("\u25CB") : acc("\u25CF");
+	if (isReview && typeof theme?.fg === "function") {
+		try {
+			bullet = theme.fg("warning", "\u25CF");
+		} catch {
+			bullet = acc("\u25CF");
+		}
+	}
+	const label = isOff ? "OFF" : isReview ? "REVIEW" : "ON";
+	// `Eye: ● ON (Clef Flash)` -> the base before the first colon, the backend in the trailing parentheses.
+	const base = clean.split(":")[0]!.trim() || "Eye";
+	const at = clean.lastIndexOf("(");
+	const backend = at >= 0 && clean.trimEnd().endsWith(")") ? clean.slice(at, clean.lastIndexOf(")") + 1).trim() : "";
+	// One space, not the two the previous glyph needed: \uf21b's ink fits its own cell, so the label sits
+	// right after it. The space before the backend stays outside the dim run, so the modifier does not
+	// start with a painted blank.
+	return `${acc("\uf21b")} ${soft(`${base}:`)} ${bullet} ${soft(label)}${backend ? ` ${dim(backend)}` : ""}`;
+}
+
 /**
  * The ponytail status cell. The horse lives at U+EF04 (fa-horse). U+EF51
  * (fa-circle_radiation) carries no outline in the Windows Nerd Font build, so it
@@ -848,6 +898,10 @@ export default function (pi: ExtensionAPI) {
 			currentModel = ctx.model;
 			currentThinkingLevel = ctx.thinkingLevel;
 			git = null;
+			// A new session reuses this module (no re-import), so the 2s cache would serve the
+			// previous session's sums whenever the fresh branch happens to be the same length.
+			usageScanAt = 0;
+			usageScanLen = -1;
 		} else {
 			currentModel = (globalThis as Record<symbol, any>)[MODEL_KEY] ?? ctx.model;
 			currentThinkingLevel = (globalThis as Record<symbol, any>)[THINKING_KEY] ?? ctx.thinkingLevel;
@@ -948,22 +1002,7 @@ export default function (pi: ExtensionAPI) {
 									return `${acc("\uf233")} ${secondarySoft(m ? m[0] : rest || "MCP")}`;
 								}
 								if (s.includes("ponytail")) return ponytailStatusLine(s, acc, dim, secondarySoft);
-								if (s.includes("jev-eye")) {
-									const isOff = s.includes("OFF") || s.includes("○");
-									const isReview = /REVIEW/i.test(s);
-									let bullet = isOff ? dim("○") : acc("●");
-									if (isReview) {
-										try {
-											bullet = theme.fg("warning", "●");
-										} catch {
-											bullet = acc("●");
-										}
-									}
-									const label = isOff ? "OFF" : isReview ? "REVIEW" : "ON";
-									// Two spaces: \uedcf's ink is wider than MCP's \uf233 or ponytail's \uef04,
-									// so it needs the extra cell to clear the label.
-									return `${acc("\uedcf")}  ${secondarySoft("Jev:")} ${bullet} ${secondarySoft(label)}`;
-								}
+								if (isJevEyeStatus(s)) return jevEyeStatusLine(s, acc, dim, secondarySoft, theme);
 								let clean = stripAnsi(s);
 								clean = clean.replace(/\uFFFD/g, "").replace(/\?{1,2}\s*/g, "");
 								return secondarySoft(clean.trim());
@@ -1438,6 +1477,58 @@ if (isMain(import.meta.url)) {
 	assert(!ponyLine.includes("\uef51"), "the blank \\uef51 glyph must never come back");
 	assert(ponytailStatusLine("ponytail: LITE", (s) => s, (s) => s, (s) => s).includes("LITE"), "the ponytail mode is read from the status text");
 
+	// Self-check: the status router must recognise the pi-jev-eye cell by the label its VALUE carries.
+	// The old test was `value.includes("jev-eye")`, and the value is `Eye: ● ON (Clef Flash)` — no
+	// `jev-eye` anywhere — so the branch went false and the cell silently lost its icon and dim suffix.
+	// This pins the recognition rule itself, at the same place the router uses it.
+	assert(isJevEyeStatus("Eye: ● ON (Clef Flash)"), "the value pi-jev-eye publishes is recognised as its own cell");
+	assert(isJevEyeStatus("Eye: ○ OFF (Jev)"), "and so is every other mode's value");
+	assert(!isJevEyeStatus("Jev: ● ON"), "the previous format is not claimed as this cell");
+	assert(!isJevEyeStatus("MCP: 2 servers enabled"), "and neither is another extension's status");
+	assert(
+		!stripAnsi("Eye: ● ON (Clef Flash)").includes("jev-eye"),
+		"the value carries no internal key name, which is what broke the old matcher",
+	);
+
+	// Self-check: the pi-jev-eye status cell. The value is `Eye: ● MODE (Backend)`; the backend is read out
+	// of that string rather than hardcoded, and it takes `dim` while the base label and the state stay
+	// readable — a backend painted like the label would bury `Eye: ● ON`, which is the case this cell
+	// exists for. Both footer formats the operator asked for are pinned here.
+	const jevPlain = (s: string) => s;
+	const jevDim = (s: string) => `[dim]${s}[/dim]`;
+	assert(
+		jevEyeStatusLine("Eye: ● ON (Clef Flash)", jevPlain, jevDim, jevPlain, undefined) ===
+			"\uf21b" + " Eye: ● ON [dim](Clef Flash)[/dim]",
+		"Clef Flash is dim while `Eye:` and the state are not",
+	);
+	assert(
+		jevEyeStatusLine("Eye: ● ON (Jev)", jevPlain, jevDim, jevPlain, undefined) === "\uf21b" + " Eye: ● ON [dim](Jev)[/dim]",
+		"TypeSafe and OpenRouter both read as (Jev)",
+	);
+	assert(
+		jevEyeStatusLine("Eye: ● ON (Clef)", jevPlain, jevDim, jevPlain, undefined) === "\uf21b" + " Eye: ● ON [dim](Clef)[/dim]",
+		"the untiered Clef model draws no tier word",
+	);
+	assert(
+		jevEyeStatusLine("Eye: ● ON (Clef Omni)", jevPlain, jevDim, jevPlain, undefined) === "\uf21b" + " Eye: ● ON [dim](Clef Omni)[/dim]",
+		"a model added upstream draws its own tier",
+	);
+	assert(
+		jevEyeStatusLine("Eye: ○ OFF (Jev)", jevPlain, jevDim, jevPlain, undefined) === "\uf21b" + " Eye: [dim]○[/dim] OFF [dim](Jev)[/dim]",
+		"OFF takes the dim bullet",
+	);
+	assert(
+		jevEyeStatusLine("Eye: ● REVIEW (Clef Flash)", jevPlain, jevDim, jevPlain, { fg: (color, text) => `<${color}>${text}</${color}>` }) ===
+			"\uf21b" + " Eye: <warning>●</warning> REVIEW [dim](Clef Flash)[/dim]",
+		"REVIEW keeps the warning bullet",
+	);
+	// A value with no parenthesised backend (an older pi-jev-eye) must not gain a stray `(`: the suffix is
+	// drawn only when the value actually carries one.
+	assert(
+		jevEyeStatusLine("Eye: ● ON", jevPlain, jevDim, jevPlain, undefined) === "\uf21b" + " Eye: ● ON",
+		"no backend suffix means no parenthesis, never a guessed one",
+	);
+
 	// Self-check: session age survives /resume (header timestamp wins over now).
 	const NOW = Date.parse("2026-10-08T12:00:00.000Z");
 	assert(resolveSessionStartMs("resume", "2026-10-08T09:00:00.000Z", NOW) === Date.parse("2026-10-08T09:00:00.000Z"), "resume restores the session age from the header");

package/extensions/ui-render-tweaks.ts

@@ -16,7 +16,6 @@
  */
 import {
 	CompactionSummaryMessageComponent,
-	CustomMessageComponent,
 	ExtensionSelectorComponent,
 	getMarkdownTheme,
 	initTheme,

package/lib/usage-store.ts

@@ -117,7 +117,7 @@ export interface UsageSummary {
 		cost: number;
 		/** Provider-reported cost only (never includes the estimate). */
 		reportedCost: number;
-		/** Cost reconstructed from `prices.json`. Shown with a `~`, never folded into `cost`. */
+		/** Cost reconstructed from `models.json` rates. Shown with a `~`, never folded into `cost`. */
 		estimatedCost: number;
 	};
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bismawy/pi-arnative",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "Refined aesthetics. Cohesive tools. Built for Pi. Warm themes, rounded tool boxes, tabbed header, compact footer and a /usage dashboard.",
   "keywords": [
     "pi-package",