pi-mega-compact 0.15.0 → 0.15.2

package/extensions/dashboard-client/src/components/SessionContextGauges.tsx

@@ -19,7 +19,12 @@
 
 import type React from "react";
 import type { ActiveSession, SessionsResponse } from "@contracts";
-import { Card, CardContent, CardHeader, CardTitle } from "../components/ui/card";
+import {
+	Card,
+	CardContent,
+	CardHeader,
+	CardTitle,
+} from "../components/ui/card";
 
 export interface SessionContextGaugesProps {
 	/** Active sessions from /api/sessions (may be null while loading). */
@@ -107,7 +112,11 @@ function SessionGauge({ data }: { data: GaugeData }): React.ReactElement {
 				<div className="gauge-fill" style={{ width: `${fillWidth}%` }} />
 			</div>
 			<p className="gauge-label session-gauge-sub">{sublabel}</p>
-			{data.sublabel && <p className="session-gauge-meta truncate" title={data.sublabel}>{data.sublabel}</p>}
+			{data.sublabel && (
+				<p className="session-gauge-meta truncate" title={data.sublabel}>
+					{data.sublabel}
+				</p>
+			)}
 		</div>
 	);
 }
@@ -135,11 +144,26 @@ export function SessionContextGauges({
 	];
 
 	// Other active sessions (exclude the launcher by sessionId to avoid dup).
-	const others: ActiveSession[] = (sessions?.sessions ?? []).filter(
+	// Also dedup BY PID: the heartbeat schema PK is (pid, session_id), so a
+	// process that restarted into a new session_id leaves a stale
+	// (pid, old_session_id) row. That stale row has a different sessionId so
+	// the launcher filter above doesn't catch it, and it surfaces as a
+	// phantom 0% box (null tokens). Keep only the most-recently-seen row per
+	// pid (rows are sorted by lastSeen DESC below before the loop).
+	const othersRaw: ActiveSession[] = (sessions?.sessions ?? []).filter(
 		(s) => s.sessionId !== launcherSessionId,
 	);
-	// Sort by lastSeen descending (most recent first).
-	others.sort((a, b) => b.lastSeen - a.lastSeen);
+	othersRaw.sort((a, b) => b.lastSeen - a.lastSeen);
+	const seenPid = new Set<number>();
+	const others: ActiveSession[] = othersRaw.filter((s) => {
+		// Skip stale rows with no token sample — they have nothing to gauge
+		// and are always either a fresh session (briefly) or a dead lingering
+		// row. Either way a 0% / "?" gauge is noise.
+		if (s.tokens == null && s.percent == null) return false;
+		if (seenPid.has(s.pid)) return false;
+		seenPid.add(s.pid);
+		return true;
+	});
 	for (const s of others) {
 		gauges.push({
 			key: s.sessionId,
@@ -163,21 +187,21 @@ export function SessionContextGauges({
 				<CardTitle>Context per Session (live)</CardTitle>
 			</CardHeader>
 			<CardContent>
-			{hasError ? (
-				<div className="text-sm text-muted-foreground">
-					Error loading sessions: {error!.message}
-				</div>
-			) : showLoading ? (
-				<div className="text-sm text-muted-foreground">Loading sessions…</div>
-			) : (
-				<div className="session-gauges-scroll">
-					<div className="session-gauges-grid">
-						{gauges.map((g) => (
-							<SessionGauge key={g.key} data={g} />
-						))}
+				{hasError ? (
+					<div className="text-sm text-muted-foreground">
+						Error loading sessions: {error!.message}
 					</div>
-				</div>
-			)}
+				) : showLoading ? (
+					<div className="text-sm text-muted-foreground">Loading sessions…</div>
+				) : (
+					<div className="session-gauges-scroll">
+						<div className="session-gauges-grid">
+							{gauges.map((g) => (
+								<SessionGauge key={g.key} data={g} />
+							))}
+						</div>
+					</div>
+				)}
 			</CardContent>
 		</Card>
 	);

package/extensions/dashboard-client/src/hooks/useApi.ts

@@ -24,15 +24,45 @@ export interface UseApiOptions {
 	retryBaseMs?: number;
 }
 
-// SPRINT-D1-REMAINING: integrate retryWithBackoff from utils/retry.ts.
-// SPRINT-D1-REMAINING: integrate staleness detection from utils/staleness.ts.
+// SPRINT-D1: retry with exponential backoff on fetch failure, so a
+// transient network blip (dashboard server still starting, ECONNRESET, a
+// single malformed response) auto-recovers instead of surfacing a hard
+// error screen that forces a manual page refresh. Crosses any fetch this
+// hook backs (snapshot, sessions, ...).
+function sleep(ms: number): Promise<void> {
+	return new Promise((r) => setTimeout(r, ms));
+}
+
+/** Fetch with bounded exponential backoff retry. The retry schedule is
+ *  {0, retryBaseMs, retryBaseMs*2, retryBaseMs*4, ...} capped at retryBaseMs*8,
+ *  so maxRetries=3 yields ~0ms + 500ms + 1000ms + 2000ms before giving up. */
+async function fetchWithRetry<T>(
+	fetchFn: () => Promise<T>,
+	maxRetries: number,
+	retryBaseMs: number,
+): Promise<T> {
+	let lastErr: unknown;
+	for (let attempt = 0; attempt <= maxRetries; attempt++) {
+		try {
+			return await fetchFn();
+		} catch (err) {
+			lastErr = err;
+			if (attempt === maxRetries) break;
+			// Exponential backoff capped at 8x base to avoid long stalls.
+			const delay = Math.min(retryBaseMs * 2 ** attempt, retryBaseMs * 8);
+			await sleep(delay);
+		}
+	}
+	throw lastErr instanceof Error ? lastErr : new Error(String(lastErr));
+}
+
 // SPRINT-T1-REMAINING: add Authorization header when auth token present.
 
 export function useApi<T>(
 	fetchFn: () => Promise<T>,
 	options: UseApiOptions = {},
 ): UseApiResult<T> {
-	const { pollInterval = 0 } = options;
+	const { pollInterval = 0, maxRetries = 3, retryBaseMs = 500 } = options;
 	const [data, setData] = useState<T | null>(null);
 	const [error, setError] = useState<Error | null>(null);
 	const [loading, setLoading] = useState(true);
@@ -43,7 +73,10 @@ export function useApi<T>(
 		setLoading(true);
 		setError(null);
 		try {
-			const result = await fetchFn();
+			// Retry transient failures with exponential backoff so a single
+			// network blip on page load auto-recovers instead of showing a
+			// hard error that requires a manual refresh.
+			const result = await fetchWithRetry(fetchFn, maxRetries, retryBaseMs);
 			if (mountedRef.current) {
 				setData(result);
 				setLastFetchedAt(Date.now());
@@ -57,7 +90,7 @@ export function useApi<T>(
 				setLoading(false);
 			}
 		}
-	}, [fetchFn]);
+	}, [fetchFn, maxRetries, retryBaseMs]);
 
 	useEffect(() => {
 		mountedRef.current = true;

package/extensions/dashboard-client/src/tabs/OverviewTab.tsx

@@ -124,7 +124,9 @@ export default function OverviewTab({
 	// which widget's modal is open (null = none)
 	const [openWidget, setOpenWidget] = useState<string | null>(null);
 	// perf samples fetched for the opened perf-backed widget
-	const [perfData, setPerfData] = useState<Array<{ ts: number; value: number }>>([]);
+	const [perfData, setPerfData] = useState<
+		Array<{ ts: number; value: number }>
+	>([]);
 	const [perfLoading, setPerfLoading] = useState(false);
 	const [perfError, setPerfError] = useState<Error | null>(null);
 
@@ -148,9 +150,7 @@ export default function OverviewTab({
 			})
 			.catch((err) => {
 				if (!cancelled) {
-					setPerfError(
-						err instanceof Error ? err : new Error(String(err)),
-					);
+					setPerfError(err instanceof Error ? err : new Error(String(err)));
 				}
 			})
 			.finally(() => {
@@ -177,7 +177,15 @@ export default function OverviewTab({
 	if (loading && !snapshot)
 		return <div className="tab-stub">Loading snapshot…</div>;
 	if (error && !snapshot)
-		return <div className="tab-stub">Error: {error.message}</div>;
+		return (
+			<div className="tab-stub">
+				<p>Error loading snapshot: {error.message}</p>
+				<p className="text-xs text-muted-foreground mt-2">
+					Auto-retried with backoff and will re-poll every 5s — no action
+					needed.
+				</p>
+			</div>
+		);
 	if (!snapshot) return <div className="tab-stub">No snapshot data.</div>;
 
 	// Cast to RuntimeSnapshot for tierPct (runtime-only field).
@@ -389,15 +397,15 @@ export default function OverviewTab({
 				onClose={() => setOpenWidget(null)}
 			>
 				{perfLoading ? (
-					<p className="py-8 text-center text-sm text-muted-foreground">Loading…</p>
+					<p className="py-8 text-center text-sm text-muted-foreground">
+						Loading…
+					</p>
 				) : perfError ? (
-					<p className="py-8 text-center text-sm text-red-400">{perfError.message}</p>
+					<p className="py-8 text-center text-sm text-red-400">
+						{perfError.message}
+					</p>
 				) : (
-					<PerfLineChart
-						data={perfData}
-						label="Cache hit %"
-						color="#58a6ff"
-					/>
+					<PerfLineChart data={perfData} label="Cache hit %" color="#58a6ff" />
 				)}
 			</WidgetDetailModal>
 
@@ -407,9 +415,13 @@ export default function OverviewTab({
 				onClose={() => setOpenWidget(null)}
 			>
 				{perfLoading ? (
-					<p className="py-8 text-center text-sm text-muted-foreground">Loading…</p>
+					<p className="py-8 text-center text-sm text-muted-foreground">
+						Loading…
+					</p>
 				) : perfError ? (
-					<p className="py-8 text-center text-sm text-red-400">{perfError.message}</p>
+					<p className="py-8 text-center text-sm text-red-400">
+						{perfError.message}
+					</p>
 				) : (
 					<PerfLineChart
 						data={perfData}
@@ -425,9 +437,13 @@ export default function OverviewTab({
 				onClose={() => setOpenWidget(null)}
 			>
 				{perfLoading ? (
-					<p className="py-8 text-center text-sm text-muted-foreground">Loading…</p>
+					<p className="py-8 text-center text-sm text-muted-foreground">
+						Loading…
+					</p>
 				) : perfError ? (
-					<p className="py-8 text-center text-sm text-red-400">{perfError.message}</p>
+					<p className="py-8 text-center text-sm text-red-400">
+						{perfError.message}
+					</p>
 				) : (
 					<PerfLineChart
 						data={perfData}

package/extensions/dashboard-server/resolve-snapshot.ts

@@ -5,11 +5,17 @@
  * active pi session may be running in a different repo. This module queries
  * the repo registry to find the most recently active repo's dashboard.json,
  * so the Overview always shows live data from the repo pi is actually running
- * in — not the stale launcher dir.
+ * in.
+ *
+ * The launch stateDir is a candidate like any other (NOT excluded): when
+ * `/dashboard` was run in the repo you are still coding in, the launch dir
+ * IS the most-recently-active repo and must win. Excluding it would serve a
+ * stale other-repo snapshot, surfacing a phantom 0% "this session" gauge
+ * plus a duplicate live-session gauge.
  *
  * Resolution order:
- * 1. Most recently seen repo (by last_seen in repo_registry) whose
- *    dashboard.json exists and has a non-null updatedAt
+ * 1. Most recently seen repo (by last_seen in repo_registry, launch dir
+ *    included) whose dashboard.json exists and has a non-null updatedAt
  * 2. The static launch stateDir's dashboard.json (original behavior)
  */
 
@@ -35,23 +41,30 @@ interface RepoRegistryRow {
 
 /**
  * Find the most recently active repo's stateDir by scanning the repo registry
- * for repos whose dashboard.json exists and was written recently. Returns null
- * if no suitable repo is found.
+ * for the repo with the highest last_seen whose dashboard.json exists and has
+ * a non-null updatedAt. Returns null if no suitable repo is found.
+ *
+ * The launch stateDir is NOT excluded: if it is itself the most-recently-active
+ * repo (the common case — `/dashboard` was run in the repo you are still
+ * coding in), its live dashboard.json must be served. Excluding it would cause
+ * a stale *other*-repo snapshot to win, producing a phantom 0% "this session"
+ * gauge + a duplicate live-session gauge.
  */
-function mostRecentRepoStateDir(staticStateDir: string): string | null {
+function mostRecentRepoStateDir(_staticStateDir: string): string | null {
 	const indexPath = join(getIndexDir(), "index.sqlite");
 	if (!existsSync(indexPath)) return null;
 	let db;
 	try {
-		const { DatabaseSync } = require("node:sqlite") as typeof import("node:sqlite");
+		const { DatabaseSync } =
+			require("node:sqlite") as typeof import("node:sqlite");
 		db = new DatabaseSync(indexPath, { readOnly: true });
 		const rows = db
 			.prepare(
 				`SELECT state_dir, last_seen FROM repo_registry
-				 WHERE state_dir IS NOT NULL AND state_dir != ?
+				 WHERE state_dir IS NOT NULL AND last_seen IS NOT NULL
 				 ORDER BY last_seen DESC`,
 			)
-			.all(staticStateDir) as unknown as RepoRegistryRow[];
+			.all() as unknown as RepoRegistryRow[];
 		for (const r of rows) {
 			const dashPath = join(r.state_dir, "dashboard.json");
 			if (!existsSync(dashPath)) continue;
@@ -95,4 +108,3 @@ export function resolveSnapshot(
 		stateDir: staticStateDir,
 	};
 }
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-mega-compact",
-  "version": "0.15.0",
+  "version": "0.15.2",
   "description": "Layered, local, vector-backed context compressor for pi — supersede/collapse/cluster compaction with deduped inline recall.",
   "type": "module",
   "license": "BSD-3-Clause",