pi-lens 2.2.3 → 2.2.5

package/clients/dispatch/dispatcher.js

@@ -41,17 +41,31 @@ export function registerRunner(runner) {
 export function getRunner(id) {
     return globalRegistry.get(id);
 }
-export function getRunnersForKind(kind) {
+export function getRunnersForKind(kind, filePath) {
     if (!kind)
         return [];
     const runners = [];
+    const isTestFile = filePath ? isTest(filePath) : false;
     for (const runner of globalRegistry.values()) {
+        // Skip runners that shouldn't run on test files
+        if (isTestFile && runner.skipTestFiles)
+            continue;
         if (runner.appliesTo.includes(kind) || runner.appliesTo.length === 0) {
             runners.push(runner);
         }
     }
     return runners.sort((a, b) => a.priority - b.priority);
 }
+function isTest(filePath) {
+    const normalized = filePath.replace(/\\/g, "/");
+    return (normalized.includes(".test.") ||
+        normalized.includes(".spec.") ||
+        normalized.includes("/test/") ||
+        normalized.includes("/tests/") ||
+        normalized.includes("__tests__/") ||
+        normalized.includes("test-utils") ||
+        normalized.startsWith("test-"));
+}
 export function listRunners() {
     return Array.from(globalRegistry.values());
 }

package/clients/dispatch/dispatcher.ts

@@ -65,10 +65,16 @@ export function getRunner(id: string): RunnerDefinition | undefined {
 
 export function getRunnersForKind(
 	kind: FileKind | undefined,
+	filePath?: string,
 ): RunnerDefinition[] {
 	if (!kind) return [];
 	const runners: RunnerDefinition[] = [];
+	const isTestFile = filePath ? isTest(filePath) : false;
+
 	for (const runner of globalRegistry.values()) {
+		// Skip runners that shouldn't run on test files
+		if (isTestFile && runner.skipTestFiles) continue;
+
 		if (runner.appliesTo.includes(kind) || runner.appliesTo.length === 0) {
 			runners.push(runner);
 		}
@@ -76,6 +82,19 @@ export function getRunnersForKind(
 	return runners.sort((a, b) => a.priority - b.priority);
 }
 
+function isTest(filePath: string): boolean {
+	const normalized = filePath.replace(/\\/g, "/");
+	return (
+		normalized.includes(".test.") ||
+		normalized.includes(".spec.") ||
+		normalized.includes("/test/") ||
+		normalized.includes("/tests/") ||
+		normalized.includes("__tests__/") ||
+		normalized.includes("test-utils") ||
+		normalized.startsWith("test-")
+	);
+}
+
 export function listRunners(): RunnerDefinition[] {
 	return Array.from(globalRegistry.values());
 }

package/clients/dispatch/runners/ast-grep.js

@@ -13,6 +13,7 @@ const astGrepRunner = {
     appliesTo: ["jsts", "python", "go", "rust", "cxx"],
     priority: 30,
     enabledByDefault: false,
+    skipTestFiles: true, // Many rules are noisy in tests
     async run(ctx) {
         // Check if ast-grep is available
         const check = spawnSync("sg", ["--version"], {

package/clients/dispatch/runners/ast-grep.ts

@@ -21,6 +21,7 @@ const astGrepRunner: RunnerDefinition = {
 	appliesTo: ["jsts", "python", "go", "rust", "cxx"],
 	priority: 30,
 	enabledByDefault: false,
+	skipTestFiles: true, // Many rules are noisy in tests
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
 		// Check if ast-grep is available

package/clients/dispatch/types.ts

@@ -96,6 +96,8 @@ export interface RunnerDefinition {
 	appliesTo: readonly FileKind[];
 	priority: number;
 	enabledByDefault: boolean;
+	/** Skip this runner for test files (false positive reduction) */
+	skipTestFiles?: 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.3",
+	"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": {