pi-lens 2.2.4 → 2.2.5

package/clients/dispatch/dispatcher.js

@@ -13,7 +13,6 @@
  * - OutputSemantic: How to display (blocking, warning, silent, etc.)
  * - BaselineStore: Track pre-existing issues for delta mode
  */
-import * as fs from "node:fs";
 import { detectFileKind } from "../file-kinds.js";
 // --- In-Memory Baseline Store ---
 export function createBaselineStore() {
@@ -32,8 +31,6 @@ export function createBaselineStore() {
 }
 // --- Runner Registry ---
 const globalRegistry = new Map();
-// Track last-run mtime per file+runner to skip unchanged files
-const lastRunMtimes = new Map(); // key: `${runnerId}:${filePath}` -> mtimeMs
 export function registerRunner(runner) {
     if (globalRegistry.has(runner.id)) {
         console.error(`[dispatch] Duplicate runner: ${runner.id}`);
@@ -223,22 +220,6 @@ export async function dispatchForFile(ctx, groups) {
 }
 // --- Run Single Runner ---
 async function runRunner(ctx, runner, defaultSemantic) {
-    // Skip if file unchanged (optimization for expensive runners)
-    if (runner.skipIfUnchanged) {
-        const cacheKey = `${runner.id}:${ctx.filePath}`;
-        try {
-            const stats = fs.statSync(ctx.filePath);
-            const lastMtime = lastRunMtimes.get(cacheKey);
-            if (lastMtime && stats.mtimeMs <= lastMtime) {
-                return { status: "skipped", diagnostics: [], semantic: "none" };
-            }
-            // Update mtime after run (below)
-            lastRunMtimes.set(cacheKey, stats.mtimeMs);
-        }
-        catch {
-            // File doesn't exist or stat failed, continue with run
-        }
-    }
     try {
         const result = await runner.run(ctx);
         return {

package/clients/dispatch/dispatcher.ts

@@ -14,7 +14,6 @@
  * - BaselineStore: Track pre-existing issues for delta mode
  */
 
-import * as fs from "node:fs";
 import type { FileKind } from "../file-kinds.js";
 import { detectFileKind } from "../file-kinds.js";
 
@@ -52,9 +51,6 @@ export function createBaselineStore(): BaselineStore {
 
 const globalRegistry = new Map<string, RunnerDefinition>();
 
-// Track last-run mtime per file+runner to skip unchanged files
-const lastRunMtimes = new Map<string, number>(); // key: `${runnerId}:${filePath}` -> mtimeMs
-
 export function registerRunner(runner: RunnerDefinition): void {
 	if (globalRegistry.has(runner.id)) {
 		console.error(`[dispatch] Duplicate runner: ${runner.id}`);
@@ -311,22 +307,6 @@ async function runRunner(
 	runner: RunnerDefinition,
 	defaultSemantic: OutputSemantic,
 ): Promise<RunnerResult> {
-	// Skip if file unchanged (optimization for expensive runners)
-	if (runner.skipIfUnchanged) {
-		const cacheKey = `${runner.id}:${ctx.filePath}`;
-		try {
-			const stats = fs.statSync(ctx.filePath);
-			const lastMtime = lastRunMtimes.get(cacheKey);
-			if (lastMtime && stats.mtimeMs <= lastMtime) {
-				return { status: "skipped", diagnostics: [], semantic: "none" };
-			}
-			// Update mtime after run (below)
-			lastRunMtimes.set(cacheKey, stats.mtimeMs);
-		} catch {
-			// File doesn't exist or stat failed, continue with run
-		}
-	}
-
 	try {
 		const result = await runner.run(ctx);
 		return {

package/clients/dispatch/runners/type-safety.js

@@ -12,7 +12,6 @@ const typeSafetyRunner = {
     appliesTo: ["jsts"],
     priority: 20,
     enabledByDefault: true,
-    skipIfUnchanged: true, // Skip type-coverage if file unchanged
     async run(ctx) {
         // Only check TypeScript files
         if (!ctx.filePath.match(/\.tsx?$/)) {

package/clients/dispatch/runners/type-safety.ts

@@ -20,7 +20,6 @@ const typeSafetyRunner: RunnerDefinition = {
 	appliesTo: ["jsts"],
 	priority: 20,
 	enabledByDefault: true,
-	skipIfUnchanged: true, // Skip type-coverage if file unchanged
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
 		// Only check TypeScript files

package/clients/dispatch/types.ts

@@ -98,8 +98,6 @@ export interface RunnerDefinition {
 	enabledByDefault: boolean;
 	/** Skip this runner for test files (false positive reduction) */
 	skipTestFiles?: boolean;
-	/** Skip if file unchanged since last run (check mtime) */
-	skipIfUnchanged?: boolean;
 	/** Check if runner should run */
 	when?: (ctx: DispatchContext) => Promise<boolean> | boolean;
 	/** Execute the runner */

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "2.2.4",
+	"version": "2.2.5",
 	"type": "module",
 	"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": {