pi-mega-compact 0.14.1 → 0.15.1

package/extensions/dashboard-server/api-contracts/endpoints/types.ts

@@ -348,3 +348,39 @@ export interface PerfResponse {
 	/** Diagnostic counters from the live snapshot, or null if unavailable. */
 	readonly diag: PerfDiag | null;
 }
+
+/**
+ * Query parameters for GET /api/perf/samples. `kind` selects which perf metric
+ * to return raw samples for; `minutes` controls the rolling window size.
+ */
+export interface PerfSamplesQuery {
+	/** Perf metric kind (turn_latency_ms, tps, disk_write_ms, cache_hit_pct, ...). Required. */
+	readonly kind: string;
+	/** Rolling window size in minutes (default: 60). Optional. */
+	readonly minutes?: number;
+}
+
+/** A single raw perf sample for time-series charting. */
+export interface PerfSamplePoint {
+	/** Epoch ms timestamp of the sample. */
+	readonly ts: number;
+	/** Sample value for the requested kind. */
+	readonly value: number;
+	/** Optional stringified metadata associated with the sample. */
+	readonly meta?: string;
+}
+
+/**
+ * Response for GET /api/perf/samples. Raw perf samples for a single kind,
+ * ascending by ts, for chart drill-down in the Perf tab.
+ */
+export interface PerfSamplesResponse {
+	/** Array of raw samples ascending by ts. */
+	readonly samples: PerfSamplePoint[];
+	/** The requested perf kind. */
+	readonly kind: string;
+	/** The rolling window size in minutes used. */
+	readonly minutes: number;
+	/** Epoch ms timestamp when the response was generated. */
+	readonly updatedAt: number;
+}

package/extensions/dashboard-server/api-contracts/endpoints.ts

@@ -31,6 +31,9 @@ export type {
 	PerfDiag,
 	PerfQuery,
 	PerfResponse,
+	PerfSamplesQuery,
+	PerfSamplePoint,
+	PerfSamplesResponse,
 	GameScoreRow,
 	GameScoresQuery,
 	AchievementRow,

package/extensions/dashboard-server/api-contracts/index.ts

@@ -140,6 +140,9 @@ export type {
 	PerfDiag,
 	PerfQuery,
 	PerfResponse,
+	PerfSamplesQuery,
+	PerfSamplePoint,
+	PerfSamplesResponse,
 	GameScoreRow,
 	GameScoresQuery,
 	AchievementRow,

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/extensions/dashboard-server/routes-game.ts

@@ -8,6 +8,7 @@ import type { IncomingMessage, ServerResponse } from "node:http";
 
 import type { RouteContext } from "./routes-core.js";
 import type { GameMetric } from "../../src/game/scoring.js";
+import type { PerfKind } from "../../src/store/sqlite/perf-samples.js";
 
 // ---------------------------------------------------------------------------
 // handleGameState — "/api/game-state" (GET + PUT)
@@ -351,6 +352,77 @@ export function handlePerf(
 	return true;
 }
 
+// ---------------------------------------------------------------------------
+// handlePerfSamples — "/api/perf/samples"
+// ---------------------------------------------------------------------------
+
+export function handlePerfSamples(
+	req: IncomingMessage,
+	res: ServerResponse,
+	ctx: RouteContext,
+): boolean {
+	if (!req.url?.startsWith("/api/perf/samples")) return false;
+
+	const { stateDir } = ctx;
+
+	// /api/perf/samples — dashboard Perf tab chart drill-down. GET returns raw
+	// perf_samples for a single kind (?kind=<k>&minutes=<n>) ascending by ts so
+	// the client can render a time-series for any perf metric (turn_latency_ms,
+	// tps, disk_write_ms, cache_hit_pct, db_recompute_ms, etc.). `kind` is
+	// required and validated against the PERF_KINDS allow-list (400 on unknown);
+	// `minutes` defaults to 60 and is capped at 1440 (24h). Reads perf_samples
+	// via the same require()'d sqlite helper as /api/perf — no new SQLite code.
+	// Non-GET -> 405. PREVENT-PI-004: loopback.
+	const psReq = createRequire(import.meta.url);
+	const { readPerfSamples, PERF_KINDS } = psReq(
+		"../../src/store/sqlite.js",
+	) as typeof import("../../src/store/sqlite.js");
+	if (req.method !== "GET") {
+		res.writeHead(405, { "Content-Type": "application/json" }); // guardrails-allow PREVENT-PI-004: loopback dashboard response (local)
+		res.end(JSON.stringify({ error: "method_not_allowed" }));
+		return true;
+	}
+	try {
+		const url = new URL(req.url, "http://x"); // guardrails-allow PREVENT-PI-004: localhost dashboard URL base (loopback-only)
+		const kindParam = url.searchParams.get("kind") ?? "";
+		if (!(PERF_KINDS as readonly string[]).includes(kindParam)) {
+			res.writeHead(400, { "Content-Type": "application/json" }); // guardrails-allow PREVENT-PI-004: loopback dashboard response (local)
+			res.end(JSON.stringify({ error: "unknown_kind", kind: kindParam }));
+			return true;
+		}
+		let minutes = Number(url.searchParams.get("minutes") ?? "60");
+		if (!Number.isFinite(minutes) || minutes <= 0) minutes = 60;
+		minutes = Math.min(minutes, 1440); // cap at 24h
+		const sinceTs = Date.now() - minutes * 60_000;
+		const kind = kindParam as PerfKind; // validated against PERF_KINDS above
+		const rows = readPerfSamples(stateDir, sinceTs, kind); // guardrails-allow PREVENT-PI-004: local SQLite read (loopback dashboard)
+		const samples = rows.map((r) => ({
+			ts: r.ts,
+			value: r.value,
+			...((r.meta != null && typeof r.meta === "object"
+				? { meta: JSON.stringify(r.meta) }
+				: r.meta != null
+					? { meta: String(r.meta) }
+					: {}) as { meta?: string }),
+		}));
+		res.writeHead(200, { "Content-Type": "application/json" });
+		res.end(
+			JSON.stringify({
+				samples,
+				kind: kindParam,
+				minutes,
+				updatedAt: Date.now(),
+			}),
+		);
+	} catch (e) {
+		res.writeHead(500, { "Content-Type": "application/json" });
+		res.end(
+			JSON.stringify({ error: "perf_samples_unavailable", detail: String(e) }),
+		);
+	}
+	return true;
+}
+
 // ---------------------------------------------------------------------------
 // handleAchievements — "/api/achievements"
 // ---------------------------------------------------------------------------

package/extensions/dashboard-server/routes.ts

@@ -13,6 +13,7 @@ export {
 	handleGameState,
 	handleGameScores,
 	handlePerf,
+	handlePerfSamples,
 	handleAchievements,
 } from "./routes-game.js";
 export { handleEvents, handleSessions } from "./routes-sessions.js";

package/extensions/dashboard-server/server.ts

@@ -33,6 +33,7 @@ import {
 	handleGameState,
 	handleGameScores,
 	handlePerf,
+	handlePerfSamples,
 	handleAchievements,
 	handleSessions,
 	handleTopics,
@@ -247,6 +248,7 @@ export async function launchDashboardServer(
 		if (handleEvents(req, res, ctx)) return;
 		if (handleGameState(req, res, ctx)) return;
 		if (handleGameScores(req, res, ctx)) return;
+		if (handlePerfSamples(req, res, ctx)) return;
 		if (handlePerf(req, res, ctx)) return;
 		if (handleAchievements(req, res, ctx)) return;
 		if (handleSessions(req, res, ctx)) return;

package/package.json

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