pi-lens 2.0.39 → 2.0.41

package/CHANGELOG.md

@@ -2,12 +2,18 @@
 
 All notable changes to pi-lens will be documented in this file.
 
+## [2.0.40] - 2026-03-27
+
+### Changed
+- **Passive capture on every file edit**: `captureSnapshot()` now called from `tool_call` hook with 5s debounce. Zero latency — reuses complexity metrics already computed for real-time feedback.
+- **Skip duplicate snapshots**: Same commit + same MI = no write (reduces noise).
+
 ## [2.0.39] - 2026-03-27
 
 ### Added
 - **Historical metrics tracking**: New `clients/metrics-history.ts` module captures complexity snapshots per commit. Tracks MI, cognitive complexity, and nesting depth across sessions.
 - **Trend analysis in `/lens-metrics`**: New "Trend" column shows 📈/📉/➡️ with MI delta. "Trend Summary" section aggregates improving/stable/regressing counts with worst regressions.
-- **Passive capture**: Snapshots captured on session start cache + every `/lens-metrics` run. Max 20 snapshots per file (sliding window).
+- **Passive capture**: Snapshots captured on every file edit (tool_call hook) + `/lens-metrics` run. Max 20 snapshots per file (sliding window).
 
 ## [2.0.38] - 2026-03-27
 

package/clients/metrics-history.js

@@ -64,11 +64,19 @@ export function saveHistory(history) {
     const historyPath = path.join(historyDir, "metrics-history.json");
     fs.writeFileSync(historyPath, JSON.stringify(history, null, 2));
 }
+// In-memory cache to avoid loading/saving on every capture
+let pendingHistory = null;
+let saveTimer = null;
+const SAVE_DEBOUNCE_MS = 5000; // Save at most every 5 seconds
 /**
  * Capture a snapshot for a file's current metrics
+ * Auto-saves to disk (debounced) for passive tracking
  */
-export function captureSnapshot(filePath, metrics, history) {
-    const hist = history ?? loadHistory();
+export function captureSnapshot(filePath, metrics) {
+    // Use in-memory cache if available, otherwise load from disk
+    if (!pendingHistory) {
+        pendingHistory = loadHistory();
+    }
     const relativePath = path.relative(process.cwd(), filePath);
     const commit = getCurrentCommit();
     const snapshot = {
@@ -79,8 +87,13 @@ export function captureSnapshot(filePath, metrics, history) {
         nesting: metrics.maxNestingDepth,
         lines: metrics.linesOfCode,
     };
-    const existing = hist.files[relativePath];
+    const existing = pendingHistory.files[relativePath];
     if (existing) {
+        // Skip if same commit + same MI (no change worth recording)
+        const latest = existing.latest;
+        if (latest.commit === commit && latest.mi === snapshot.mi) {
+            return;
+        }
         // Append to history (cap at MAX_HISTORY_PER_FILE)
         existing.history.push(snapshot);
         if (existing.history.length > MAX_HISTORY_PER_FILE) {
@@ -91,21 +104,55 @@ export function captureSnapshot(filePath, metrics, history) {
     }
     else {
         // New file
-        hist.files[relativePath] = {
+        pendingHistory.files[relativePath] = {
             latest: snapshot,
             history: [snapshot],
             trend: "stable",
         };
     }
-    return hist;
+    // Debounced save to disk
+    if (saveTimer)
+        clearTimeout(saveTimer);
+    saveTimer = setTimeout(() => {
+        if (pendingHistory) {
+            saveHistory(pendingHistory);
+            pendingHistory = null;
+        }
+    }, SAVE_DEBOUNCE_MS);
 }
 /**
- * Capture snapshots for multiple files
+ * Capture snapshots for multiple files (explicit, immediate save)
+ * Used by /lens-metrics for batch capture
  */
 export function captureSnapshots(files) {
     let history = loadHistory();
     for (const file of files) {
-        history = captureSnapshot(file.filePath, file.metrics, history);
+        const relativePath = path.relative(process.cwd(), file.filePath);
+        const commit = getCurrentCommit();
+        const snapshot = {
+            commit,
+            timestamp: new Date().toISOString(),
+            mi: Math.round(file.metrics.maintainabilityIndex * 10) / 10,
+            cognitive: file.metrics.cognitiveComplexity,
+            nesting: file.metrics.maxNestingDepth,
+            lines: file.metrics.linesOfCode,
+        };
+        const existing = history.files[relativePath];
+        if (existing) {
+            existing.history.push(snapshot);
+            if (existing.history.length > MAX_HISTORY_PER_FILE) {
+                existing.history = existing.history.slice(-MAX_HISTORY_PER_FILE);
+            }
+            existing.latest = snapshot;
+            existing.trend = computeTrend(existing.history);
+        }
+        else {
+            history.files[relativePath] = {
+                latest: snapshot,
+                history: [snapshot],
+                trend: "stable",
+            };
+        }
     }
     saveHistory(history);
     return history;

package/clients/metrics-history.ts

@@ -99,8 +99,14 @@ export function saveHistory(history: MetricsHistory): void {
 	fs.writeFileSync(historyPath, JSON.stringify(history, null, 2));
 }
 
+// In-memory cache to avoid loading/saving on every capture
+let pendingHistory: MetricsHistory | null = null;
+let saveTimer: ReturnType<typeof setTimeout> | null = null;
+const SAVE_DEBOUNCE_MS = 5000; // Save at most every 5 seconds
+
 /**
  * Capture a snapshot for a file's current metrics
+ * Auto-saves to disk (debounced) for passive tracking
  */
 export function captureSnapshot(
 	filePath: string,
@@ -110,9 +116,12 @@ export function captureSnapshot(
 		maxNestingDepth: number;
 		linesOfCode: number;
 	},
-	history?: MetricsHistory,
-): MetricsHistory {
-	const hist = history ?? loadHistory();
+): void {
+	// Use in-memory cache if available, otherwise load from disk
+	if (!pendingHistory) {
+		pendingHistory = loadHistory();
+	}
+
 	const relativePath = path.relative(process.cwd(), filePath);
 	const commit = getCurrentCommit();
 
@@ -125,9 +134,14 @@ export function captureSnapshot(
 		lines: metrics.linesOfCode,
 	};
 
-	const existing = hist.files[relativePath];
+	const existing = pendingHistory.files[relativePath];
 
 	if (existing) {
+		// Skip if same commit + same MI (no change worth recording)
+		const latest = existing.latest;
+		if (latest.commit === commit && latest.mi === snapshot.mi) {
+			return;
+		}
 		// Append to history (cap at MAX_HISTORY_PER_FILE)
 		existing.history.push(snapshot);
 		if (existing.history.length > MAX_HISTORY_PER_FILE) {
@@ -137,18 +151,26 @@ export function captureSnapshot(
 		existing.trend = computeTrend(existing.history);
 	} else {
 		// New file
-		hist.files[relativePath] = {
+		pendingHistory.files[relativePath] = {
 			latest: snapshot,
 			history: [snapshot],
 			trend: "stable",
 		};
 	}
 
-	return hist;
+	// Debounced save to disk
+	if (saveTimer) clearTimeout(saveTimer);
+	saveTimer = setTimeout(() => {
+		if (pendingHistory) {
+			saveHistory(pendingHistory);
+			pendingHistory = null;
+		}
+	}, SAVE_DEBOUNCE_MS);
 }
 
 /**
- * Capture snapshots for multiple files
+ * Capture snapshots for multiple files (explicit, immediate save)
+ * Used by /lens-metrics for batch capture
  */
 export function captureSnapshots(
 	files: Array<{
@@ -164,7 +186,34 @@ export function captureSnapshots(
 	let history = loadHistory();
 
 	for (const file of files) {
-		history = captureSnapshot(file.filePath, file.metrics, history);
+		const relativePath = path.relative(process.cwd(), file.filePath);
+		const commit = getCurrentCommit();
+
+		const snapshot: MetricSnapshot = {
+			commit,
+			timestamp: new Date().toISOString(),
+			mi: Math.round(file.metrics.maintainabilityIndex * 10) / 10,
+			cognitive: file.metrics.cognitiveComplexity,
+			nesting: file.metrics.maxNestingDepth,
+			lines: file.metrics.linesOfCode,
+		};
+
+		const existing = history.files[relativePath];
+
+		if (existing) {
+			existing.history.push(snapshot);
+			if (existing.history.length > MAX_HISTORY_PER_FILE) {
+				existing.history = existing.history.slice(-MAX_HISTORY_PER_FILE);
+			}
+			existing.latest = snapshot;
+			existing.trend = computeTrend(existing.history);
+		} else {
+			history.files[relativePath] = {
+				latest: snapshot,
+				history: [snapshot],
+				trend: "stable",
+			};
+		}
 	}
 
 	saveHistory(history);

package/index.ts

@@ -14,7 +14,7 @@ import { buildInterviewer } from "./clients/interviewer.js";
 import { JscpdClient } from "./clients/jscpd-client.js";
 import { KnipClient } from "./clients/knip-client.js";
 import { MetricsClient } from "./clients/metrics-client.js";
-import { captureSnapshots, getTrendSummary, formatTrendCell } from "./clients/metrics-history.js";
+import { captureSnapshot, captureSnapshots, getTrendSummary, formatTrendCell } from "./clients/metrics-history.js";
 import { RuffClient } from "./clients/ruff-client.js";
 import { RustClient } from "./clients/rust-client.js";
 import { getSourceFiles } from "./clients/scan-utils.js";
@@ -452,8 +452,8 @@ export default function (pi: ExtensionAPI) {
 
 			// All files table (sorted by MI ascending)
 			report += `## All Files\n\n`;
-			report += `| Grade | File | MI | Cognitive | Nesting | LOC | Trend |\n`;
-			report += `|-------|------|-----|-----------|---------|-----|-------|\n`;
+			report += `| Grade | File | MI | Cognitive | LOC | Entropy | Trend |\n`;
+			report += `|-------|------|-----|-----------|-----|---------|-------|\n`;
 
 			const sorted = [...results].sort(
 				(a, b) => a.maintainabilityIndex - b.maintainabilityIndex,
@@ -470,8 +470,9 @@ export default function (pi: ExtensionAPI) {
 				// Make path relative for readability
 				const relPath = path.relative(targetPath, f.filePath);
 				const trendCell = formatTrendCell(f.filePath, history);
+				const entropyCell = f.codeEntropy > 0 ? f.codeEntropy.toFixed(2) : "—";
 
-				report += `| ${grade} | ${relPath} | ${mi.toFixed(1)} | ${f.cognitiveComplexity} | ${f.maxNestingDepth} | ${f.linesOfCode} | ${trendCell} |\n`;
+				report += `| ${grade} | ${relPath} | ${mi.toFixed(1)} | ${f.cognitiveComplexity} | ${f.linesOfCode} | ${entropyCell} | ${trendCell} |\n`;
 			}
 			report += `\n`;
 
@@ -902,7 +903,7 @@ export default function (pi: ExtensionAPI) {
 		);
 		if (!nodeFs.existsSync(filePath)) return;
 
-		// Record complexity baseline for TS/JS files
+		// Record complexity baseline for TS/JS files + capture history snapshot
 		if (
 			complexityClient.isSupportedFile(filePath) &&
 			!complexityBaselines.has(filePath)
@@ -910,6 +911,13 @@ export default function (pi: ExtensionAPI) {
 			const baseline = complexityClient.analyzeFile(filePath);
 			if (baseline) {
 				complexityBaselines.set(filePath, baseline);
+				// Capture snapshot for historical tracking (async, non-blocking)
+				captureSnapshot(filePath, {
+					maintainabilityIndex: baseline.maintainabilityIndex,
+					cognitiveComplexity: baseline.cognitiveComplexity,
+					maxNestingDepth: baseline.maxNestingDepth,
+					linesOfCode: baseline.linesOfCode,
+				});
 			}
 		}
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "2.0.39",
+	"version": "2.0.41",
 	"description": "Real-time code quality feedback for pi — TypeScript LSP, Biome, ast-grep, Ruff, complexity metrics, duplicate detection. Includes automated fix loop (/lens-booboo-fix) and interactive architectural refactoring (/lens-booboo-refactor) with browser-based interviews.",
 	"repository": {
 		"type": "git",