pi-minimal-footer 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -7,6 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.1] - 2026-10-06
+
+### Fixed
+
+- Keep quota requests and timers stopped after another extension removes or replaces the footer, even when the model changes; explicit reinstallation restores polling
+- Use all spare columns for responsive quota bars when the left group is absent
+- Fall back to default settings when the JSON root is null, a scalar, or an array
+
+### Changed
+
+- Clarify renderer ownership and layout budgeting with rationale comments; isolate reset-label formatting and name shared bar-size limits
+
 ## [0.3.0] - 2026-10-06
 
 ### Added
@@ -129,7 +141,8 @@ First npm-ready release.
 
 - Project scaffold — `extensions/index.ts` with basic footer structure, `package.json` with pi extension manifest, `README.md`, `LICENSE` (MIT)
 
-[Unreleased]: https://github.com/Ryu-CZ/pi-minimal-footer/compare/v0.3.0...HEAD
+[Unreleased]: https://github.com/Ryu-CZ/pi-minimal-footer/compare/v0.3.1...HEAD
+[0.3.1]: https://github.com/Ryu-CZ/pi-minimal-footer/compare/v0.3.0...v0.3.1
 [0.3.0]: https://github.com/Ryu-CZ/pi-minimal-footer/compare/v0.2.0...v0.3.0
 [0.2.0]: https://github.com/Ryu-CZ/pi-minimal-footer/compare/v0.1.3...v0.2.0
 [0.1.3]: https://github.com/Ryu-CZ/pi-minimal-footer/compare/v0.1.2...v0.1.3

package/README.md

@@ -17,8 +17,6 @@ Path and branch on the left; extension statuses, model, context, and provider qu
 
 ![Footer preview: ~/git/project   main        🧠 Karpathy  🪽 Icarus  model · 42/200k · ⣿⣿⣿⣤⠀⠀⠀⠀⠀⠀↻1d8h](media/github-preview.png)
 
-*Screenshot shows an earlier layout; the text preview reflects current source.*
-
 ## Install
 
 Requires Node.js >=22.19.0.

package/extensions/index.ts

@@ -17,7 +17,7 @@ import { truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import { readFileSync, writeFileSync, existsSync, mkdirSync } from "node:fs";
 import { join, dirname, sep } from "node:path";
 import { homedir } from "node:os";
-import { UsageLimits } from "./lib/usage-limits.js";
+import { BASE_BAR_CELLS, MAX_BAR_CELLS, UsageLimits } from "./lib/usage-limits.js";
 
 // ── Settings ──────────────────────────────────────────────────────────
 
@@ -52,7 +52,10 @@ function settingsPath(): string {
 function readSettings(): Settings {
   try {
     if (!existsSync(settingsPath())) return {};
-    return JSON.parse(readFileSync(settingsPath(), "utf-8")) as Settings;
+    const settings: unknown = JSON.parse(readFileSync(settingsPath(), "utf-8"));
+    // Valid JSON can still be null or a scalar; a broken preference file must not prevent loading.
+    if (settings === null || typeof settings !== "object" || Array.isArray(settings)) return {};
+    return settings as Settings;
   } catch {
     return {};
   }
@@ -65,7 +68,7 @@ function writeSettings(patch: Partial<Settings>): void {
     const current = readSettings();
     writeFileSync(path, JSON.stringify({ ...current, ...patch }, null, 2) + "\n");
   } catch {
-    // best-effort
+    // A read-only settings file must not crash the extension.
   }
 }
 
@@ -135,10 +138,14 @@ const LOCATION_GAP_WIDTH = 3;
 const MIN_TEXT_WIDTH = 4;
 const MIN_STATUS_WIDTH = 12;
 
-function buildLine(width: number, path: string, statuses: string, branch: string, model: string, context: string,
-  usage: (available: number, maxCells?: number) => string | null, statusSeparator: string, locationSeparator: string): string {
+function buildLine(
+  width: number, path: string, statuses: string, branch: string, model: string, context: string,
+  usage: (available: number, maxCells?: number) => string | null,
+  statusSeparator: string, locationSeparator: string,
+): string {
   if (width <= 0) return "";
   let core = [model, context].filter(Boolean).join(MODEL_GAP);
+  // Budget the compact bar first; its expanded size must not drive truncation decisions.
   const quota = usage(width) ?? "";
   const quotaGapWidth = quota && core ? QUOTA_GAP.length : 0;
   const coreBudget = Math.max(0, width - visibleWidth(quota) - quotaGapWidth);
@@ -164,16 +171,18 @@ function buildLine(width: number, path: string, statuses: string, branch: string
     }
   }
   let right = [fittedStatuses, protectedRight].filter(Boolean).join(statusSeparator);
-  const locationGapWidth = right ? LOCATION_GAP_WIDTH : 0;
-  const leftBudget = Math.max(0, width - visibleWidth(right) - locationGapWidth);
+  const reservedLocationGapWidth = right ? LOCATION_GAP_WIDTH : 0;
+  const leftBudget = Math.max(0, width - visibleWidth(right) - reservedLocationGapWidth);
   const fittedBranch = visibleWidth(branch) <= leftBudget ? branch : "";
   const pathGapWidth = fittedBranch && path ? visibleWidth(locationSeparator) : 0;
   const pathBudget = leftBudget - visibleWidth(fittedBranch) - pathGapWidth;
   const fittedPath = pathBudget >= MIN_TEXT_WIDTH ? truncateToWidth(path, pathBudget, "...") : "";
   const left = [fittedPath, fittedBranch].filter(Boolean).join(locationSeparator);
   if (quota) {
-    const spareWidth = Math.max(0, width - visibleWidth(left) - visibleWidth(right) - locationGapWidth);
-    const expandedQuota = usage(visibleWidth(quota) + spareWidth, 10) ?? quota;
+    // A dropped left group needs no divider; those columns belong to the quota bar instead.
+    const interGroupGapWidth = left && right ? LOCATION_GAP_WIDTH : 0;
+    const spareWidth = Math.max(0, width - visibleWidth(left) - visibleWidth(right) - interGroupGapWidth);
+    const expandedQuota = usage(visibleWidth(quota) + spareWidth, MAX_BAR_CELLS) ?? quota;
     const expandedCore = [core, expandedQuota].filter(Boolean).join(QUOTA_GAP);
     right = [fittedStatuses, expandedCore].filter(Boolean).join(statusSeparator);
   }
@@ -190,6 +199,12 @@ export default function (pi: ExtensionAPI) {
   let disposeFooter: (() => void) | null = null;
   const usageLimits = new UsageLimits(() => requestRender?.());
 
+  function selectUsage(ctx: ExtensionContext): void {
+    // The preference may remain enabled after another extension replaces our footer.
+    // Poll only while we own a live renderer, or model changes can resurrect hidden requests.
+    usageLimits.select(ctx, enabled && disposeFooter !== null);
+  }
+
   function install(ctx: ExtensionContext): void {
     config = readConfig();
     enabled = config.enabled !== false;
@@ -210,6 +225,7 @@ export default function (pi: ExtensionAPI) {
         if (disposed) return;
         disposed = true;
         unsub();
+        // A superseded renderer must not stop its replacement's quota polling.
         if (requestRender === request) {
           requestRender = null;
           usageLimits.stop();
@@ -220,7 +236,7 @@ export default function (pi: ExtensionAPI) {
 
       return {
         render(width: number): string[] {
-          const skills = config.showSkills
+          const statuses = config.showSkills
             ? [...footerData.getExtensionStatuses().values()].filter((s) => s.trim())
             : [];
           const branch = config.showGitBranch ? footerData.getGitBranch() : null;
@@ -228,11 +244,11 @@ export default function (pi: ExtensionAPI) {
           const line = buildLine(
             width,
             config.showPath ? theme.fg("dim", abbreviateHome(state.cwd, homedir())) : "",
-            skills.length ? statusSeparator + skills.map((s) => theme.fg("dim", s)).join(statusSeparator) : "",
+            statuses.length ? statusSeparator + statuses.map((status) => theme.fg("dim", status)).join(statusSeparator) : "",
             branch ? theme.fg("dim", ` ${branch}`) : "",
             config.showModel ? theme.bold(state.model) : "",
             config.showContext ? theme.fg("dim", theme.bold(state.context)) : "",
-            (available, maxCells = 5) => usageLimits.line(available, theme, maxCells),
+            (available, maxCells = BASE_BAR_CELLS) => usageLimits.line(available, theme, maxCells),
             statusSeparator,
             config.powerlineSeparator ? theme.fg("dim", "  ") : " ",
           );
@@ -242,7 +258,7 @@ export default function (pi: ExtensionAPI) {
         dispose,
       };
     });
-    usageLimits.select(ctx, enabled);
+    selectUsage(ctx);
   }
 
   /** Cheap refresh: update plain state and request one render. */
@@ -260,7 +276,7 @@ export default function (pi: ExtensionAPI) {
   });
 
   pi.on("model_select", async (_event, ctx) => {
-    usageLimits.select(ctx, enabled);
+    selectUsage(ctx);
     refresh(ctx);
   });
 

package/extensions/lib/usage-limits.ts

@@ -6,6 +6,29 @@ import { parseUsageWindows, record, shortest, supportedOrigin, usageRequest, win
 import type { QuotaWindow } from "./quota-providers.js";
 
 const REFRESH_MS = 4 * 60_000;
+const REQUEST_TIMEOUT_MS = 5000;
+const HOUR_MS = 60 * 60_000;
+const DAY_MS = 24 * HOUR_MS;
+const CELL_STEPS = 8;
+const PARTIAL_CELL_FILLS = ["", "⡀", "⣀", "⣄", "⣤", "⣦", "⣶", "⣷"];
+
+// Start compact so extra precision never steals space from the other footer fields.
+export const BASE_BAR_CELLS = 5;
+export const MAX_BAR_CELLS = 10;
+
+function formatResetLabel(resetAt: number | null, now: number): string {
+  if (resetAt === null) return "";
+  const remaining = resetAt - now;
+  // Clock time loses the day for distant resets; minutes add noise at that scale.
+  if (remaining > 10 * DAY_MS) return `↻${Math.floor(remaining / DAY_MS)}d`;
+  if (remaining > DAY_MS) {
+    const days = Math.floor(remaining / DAY_MS);
+    const hours = Math.floor(remaining % DAY_MS / HOUR_MS);
+    return `↻${days}d${hours}h`;
+  }
+  const date = new Date(resetAt);
+  return `↻${String(date.getHours()).padStart(2, "0")}:${String(date.getMinutes()).padStart(2, "0")}`;
+}
 
 function abortable<T>(promise: Promise<T>, signal: AbortSignal): Promise<T> {
   return new Promise((resolve, reject) => {
@@ -15,7 +38,7 @@ function abortable<T>(promise: Promise<T>, signal: AbortSignal): Promise<T> {
     };
     signal.addEventListener("abort", abort, { once: true });
     if (signal.aborted) abort();
-    // Observe late failures even after abort has settled the caller's await.
+    // Credential and body promises may ignore abort; observe late failures to avoid unhandled rejections.
     promise.then((value) => {
       signal.removeEventListener("abort", abort);
       resolve(value);
@@ -179,9 +202,10 @@ export class UsageLimits {
     const provider = this.provider!;
     const modelId = model.id;
     const controller = new AbortController();
+    // Identity checks below keep late responses from restoring a replaced provider's quota.
     this.request = controller;
     this.attemptedAt = Date.now();
-    const timeout = setTimeout(() => controller.abort(), 5000);
+    const timeout = setTimeout(() => controller.abort(), REQUEST_TIMEOUT_MS);
     timeout.unref();
     try {
       const request = await abortable(usageRequest(ctx, model), controller.signal);
@@ -208,27 +232,19 @@ export class UsageLimits {
     }
   }
 
-  line(width: number, theme: Theme, maxCells = 10): string | null {
+  line(width: number, theme: Theme, maxCells = MAX_BAR_CELLS): string | null {
     const window = this.selectedWindow()?.[1];
     if (!this.ctx || !window) return null;
     const now = Date.now();
-    const date = window.resetAt === null ? null : new Date(window.resetAt);
-    const remaining = window.resetAt === null ? 0 : window.resetAt - now;
-    const day = 24 * 60 * 60 * 1000;
-    let time = date ? `↻${String(date.getHours()).padStart(2, "0")}:${String(date.getMinutes()).padStart(2, "0")}` : "";
-    if (remaining > 10 * day) {
-      time = `↻${Math.floor(remaining / day)}d`;
-    } else if (remaining > day) {
-      time = `↻${Math.floor(remaining / day)}d${Math.floor(remaining % day / (60 * 60 * 1000))}h`;
-    }
-    if (width < (time ? time.length : 1)) return null;
-    const cells = Math.min(maxCells, Math.max(0, width - time.length));
-    const steps = Math.round(window.used / 100 * cells * 8);
-    const filled = "⣿".repeat(Math.floor(steps / 8)) + ["", "⡀", "⣀", "⣄", "⣤", "⣦", "⣶", "⣷"][steps % 8];
-    const empty = "⠀".repeat(cells - Math.ceil(steps / 8));
+    const resetLabel = formatResetLabel(window.resetAt, now);
+    if (width < (resetLabel.length || 1)) return null;
+    const cells = Math.min(maxCells, Math.max(0, width - resetLabel.length));
+    const steps = Math.round(window.used / 100 * cells * CELL_STEPS);
+    const filled = "⣿".repeat(Math.floor(steps / CELL_STEPS)) + PARTIAL_CELL_FILLS[steps % CELL_STEPS];
+    const empty = "⠀".repeat(cells - Math.ceil(steps / CELL_STEPS));
     const stale = this.stale || (window.resetAt !== null && now >= window.resetAt);
     const color = stale ? "dim" : window.used >= 92 ? "error" : window.used >= 85 ? "warning" : "success";
     const bar = cells ? theme.fg(color, filled) + theme.fg("dim", empty) : "";
-    return bar + theme.fg("dim", time);
+    return bar + theme.fg("dim", resetLabel);
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-minimal-footer",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "An opinionated minimal footer extension for Pi — clean status line with path, extension statuses, model, and context usage.",
   "keywords": [
     "pi-package",