pi-lens 2.0.0 → 2.0.2

package/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 All notable changes to pi-lens will be documented in this file.
 
+## [2.0.1] - 2026-03-25
+
+### Fixed
+- **ast-grep in `/lens-booboo` was silently dropping all results** — newer ast-grep versions exit `0` with `--json` even when issues are found; fixed the exit code check.
+- **Renamed "Design Smells" to "ast-grep"** in booboo report — the scan runs all 65 rules (security, correctness, style, design), not just design smells.
+
+### Changed
+- **Stronger real-time feedback messages** — all messages now use severity emoji and imperative language:
+  - `🔴 Fix N TypeScript error(s) — these must be resolved`
+  - `🧹 Remove N unused import(s) — they are dead code`
+  - `🔴 You introduced N new structural violation(s) — fix before moving on`
+  - `🟠 You introduced N new Biome violation(s) — fix before moving on`
+  - `🟡 Complexity issues — refactor when you get a chance`
+  - `🟠 This file has N duplicate block(s) — extract to shared utilities`
+  - `🔴 Do not redefine — N function(s) already exist elsewhere`
+- **Biome fix command is now a real bash command** — `npx @biomejs/biome check --write <file>` instead of `/lens-format` (which is a pi UI command, not runnable from agent tools).
+- **Complexity warnings skip test files in real-time** — same exclusion as lens-booboo.
+
 ## [2.0.0] - 2026-03-25
 
 ### Added

package/README.md

@@ -24,30 +24,29 @@ Real-time code quality feedback for [pi](https://github.com/mariozechner/pi-codi
 ast-grep and Biome run in **delta mode** — only violations *introduced by the current edit* are shown. Pre-existing issues are silent. Fixed violations are acknowledged.
 
 ```
-[TypeScript] 2 issue(s):
-  [error] L10: Type 'string' is not assignable to type 'number'
+🔴 Fix 2 TypeScript error(s) — these must be resolved:
+  L10: Type 'string' is not assignable to type 'number'
 
-[ast-grep] +1 new issue(s) introduced:
-  no-var: Use 'const' or 'let' instead of 'var' (L23) [fixable]
+🔴 You introduced 1 new structural violation(s) — fix before moving on:
+  no-var: Use 'const' or 'let' instead of 'var' (L23)
     → var has function scope and can lead to unexpected hoisting behavior.
-  (18 total)
+  (18 total remaining)
 
-[ast-grep] ✓ Fixed: no-console-log (-1)
+✅ ast-grep: fixed no-console-log (-1)
 
-[Biome] +1 new issue(s) introduced:
+🟠 You introduced 1 new Biome violation(s) — fix before moving on:
   L23:5 [style/useConst] This let declares a variable that is only assigned once.
-  1 fixable — run /lens-format
-  (4 total)
+  → Auto-fixable: `npx @biomejs/biome check --write utils.ts`
+  (4 total remaining)
 
-[jscpd] 1 duplicate block(s) involving utils.ts:
-  15 lines — helpers.ts:20
-  → Extract duplicated code to a shared utility function
+🟠 This file has 1 duplicate block(s) — extract to shared utilities:
+  15 lines duplicated with helpers.ts:20
 
-[Duplicate Exports] 1 function(s) already exist:
+🔴 Do not redefine — 1 function(s) already exist elsewhere:
   formatDate (already in helpers.ts)
-  → Import the existing function instead of redefining it
+  → Import the existing function instead
 
-[Complexity Warnings]
+🟡 Complexity issues — refactor when you get a chance:
   ⚠ Maintainability dropped to 55 — extract logic into helper functions
   ⚠ AI-style comments (6) — remove hand-holding comments
   ⚠ Many try/catch blocks (7) — consolidate error handling

package/index.ts

@@ -254,7 +254,7 @@ export default function (pi: ExtensionAPI) {
 					});
 
 					const output = result.stdout || result.stderr || "";
-					if (output.trim() && result.status === 1) {
+					if (output.trim() && result.status !== undefined) {
 						let issues: Array<{line: number; rule: string; message: string}> = [];
 						const lines = output.split("\n").filter((l: string) => l.trim());
 
@@ -279,7 +279,7 @@ export default function (pi: ExtensionAPI) {
 
 						if (issues.length > 0) {
 							// UI summary (truncated)
-							let report = `[Design Smells] ${issues.length} issue(s) found:\n`;
+							let report = `[ast-grep] ${issues.length} issue(s) found:\n`;
 							for (const issue of issues.slice(0, 20)) {
 								report += `  L${issue.line}: ${issue.rule} — ${issue.message}\n`;
 							}
@@ -289,7 +289,7 @@ export default function (pi: ExtensionAPI) {
 							parts.push(report);
 
 							// Full report for file
-							let fullSection = `## Design Smells\n\n**${issues.length} issue(s) found**\n\n`;
+							let fullSection = `## ast-grep (Structural Issues)\n\n**${issues.length} issue(s) found**\n\n`;
 							fullSection += "| Line | Rule | Message |\n|------|------|--------|\n";
 							for (const issue of issues) {
 								fullSection += `| ${issue.line} | ${issue.rule} | ${issue.message} |\n`;
@@ -1103,18 +1103,26 @@ export default function (pi: ExtensionAPI) {
 				const otherDiags = diags.filter(d => d.code !== 6133 && d.code !== 6196);
 
 				if (unusedImports.length > 0) {
-					lspOutput += `\n\n[Unused Imports] ${unusedImports.length} imported but never used:\n`;
+					lspOutput += `\n\n🧹 Remove ${unusedImports.length} unused import(s) — they are dead code:\n`;
 					for (const d of unusedImports.slice(0, 10)) {
 						lspOutput += `  L${d.range.start.line + 1}: ${d.message}\n`;
 					}
-					lspOutput += `  → Remove unused imports to reduce noise\n`;
 				}
 
 				if (otherDiags.length > 0) {
-					lspOutput += `\n\n[TypeScript] ${otherDiags.length} issue(s):\n`;
-					for (const d of otherDiags.slice(0, 10)) {
-						const label = d.severity === 2 ? "Warning" : "Error";
-						lspOutput += `  [${label}] L${d.range.start.line + 1}: ${d.message}\n`;
+					const errors = otherDiags.filter(d => d.severity !== 2);
+					const warnings = otherDiags.filter(d => d.severity === 2);
+					if (errors.length > 0) {
+						lspOutput += `\n\n🔴 Fix ${errors.length} TypeScript error(s) — these must be resolved:\n`;
+						for (const d of errors.slice(0, 10)) {
+							lspOutput += `  L${d.range.start.line + 1}: ${d.message}\n`;
+						}
+					}
+					if (warnings.length > 0) {
+						lspOutput += `\n\n🟡 ${warnings.length} TypeScript warning(s) — address before moving on:\n`;
+						for (const d of warnings.slice(0, 10)) {
+							lspOutput += `  L${d.range.start.line + 1}: ${d.message}\n`;
+						}
 					}
 				}
 			}
@@ -1157,10 +1165,10 @@ export default function (pi: ExtensionAPI) {
 				}
 			} else {
 				if (diags.length > 0)
-					lspOutput += `\n\n${ruffClient.formatDiagnostics(diags)}`;
+					lspOutput += `\n\n🟠 Fix ${diags.length} Ruff issue(s):\n${ruffClient.formatDiagnostics(diags)}`;
 				if (fmtReport) lspOutput += `\n\n${fmtReport}`;
 				if (fixable.length > 0 || hasFormatIssues) {
-					lspOutput += `\n\n[Ruff] ${fixable.length} fixable — enable --autofix-ruff flag to auto-fix`;
+					lspOutput += `\n  → Enable --autofix-ruff to auto-fix ${fixable.length} of these on every write`;
 				}
 			}
 		}
@@ -1195,17 +1203,16 @@ export default function (pi: ExtensionAPI) {
 			}
 
 			if (newViolations.length > 0) {
-				lspOutput += `\n\n[ast-grep] +${newViolations.length} new issue(s) introduced:\n`;
+				const hasFixable = newViolations.some(v => v.fix);
+				lspOutput += `\n\n🔴 You introduced ${newViolations.length} new structural violation(s) — fix before moving on:\n`;
 				lspOutput += astGrepClient.formatDiagnostics(newViolations);
+				if (hasFixable) lspOutput += `\n  Some are fixable — check the → hints above`;
 			}
 			if (fixedRules.length > 0) {
-				lspOutput += `\n\n[ast-grep] ✓ Fixed: ${fixedRules.join(", ")}`;
+				lspOutput += `\n\n✅ ast-grep: fixed ${fixedRules.join(", ")}`;
 			}
-			// Show total count quietly so context isn't lost
-			if (after.length > 0 && newViolations.length === 0 && fixedRules.length === 0) {
-				// no change — show nothing
-			} else if (after.length > 0) {
-				lspOutput += `\n  (${after.length} total)`;
+			if (after.length > 0 && newViolations.length > 0) {
+				lspOutput += `\n  (${after.length} total remaining)`;
 			}
 		}
 
@@ -1253,17 +1260,15 @@ export default function (pi: ExtensionAPI) {
 			} else {
 				if (newDiags.length > 0) {
 					const fixable = newDiags.filter((d) => d.fixable);
-					lspOutput += `\n\n[Biome] +${newDiags.length} new issue(s) introduced:\n`;
+					lspOutput += `\n\n🟠 You introduced ${newDiags.length} new Biome violation(s) — fix before moving on:\n`;
 					lspOutput += biomeClient.formatDiagnostics(newDiags, filePath);
-					if (fixable.length > 0) lspOutput += `\n\n[Biome] ${fixable.length} fixable — run /lens-format`;
+					if (fixable.length > 0) lspOutput += `\n  → Auto-fixable: \`npx @biomejs/biome check --write ${path.basename(filePath)}\``;
 				}
 				if (fixedRules.length > 0) {
-					lspOutput += `\n\n[Biome] ✓ Fixed: ${fixedRules.join(", ")}`;
+					lspOutput += `\n\n✅ Biome: fixed ${fixedRules.join(", ")}`;
 				}
-				if (after.length > 0 && newDiags.length === 0 && fixedRules.length === 0) {
-					// no change — show nothing
-				} else if (after.length > 0) {
-					lspOutput += `\n  (${after.length} total)`;
+				if (after.length > 0 && newDiags.length > 0) {
+					lspOutput += `\n  (${after.length} total remaining)`;
 				}
 			}
 		}
@@ -1291,11 +1296,14 @@ export default function (pi: ExtensionAPI) {
 			if (metrics) {
 				const warnings = complexityClient.checkThresholds(metrics);
 				if (warnings.length > 0) {
-					let warningReport = `[Complexity Warnings]\n`;
-					for (const w of warnings) {
-						warningReport += `  ⚠ ${w}\n`;
+					const isTestFile = /\.(test|spec)\.[jt]sx?$/.test(filePath);
+					if (!isTestFile) {
+						let warningReport = `🟡 Complexity issues — refactor when you get a chance:\n`;
+						for (const w of warnings) {
+							warningReport += `  ⚠ ${w}\n`;
+						}
+						lspOutput += `\n\n${warningReport}`;
 					}
-					lspOutput += `\n\n${warningReport}`;
 				}
 			}
 		}
@@ -1335,17 +1343,16 @@ export default function (pi: ExtensionAPI) {
 			);
 			if (fileClones.length > 0) {
 				dbg(`  jscpd: ${fileClones.length} duplicate(s) involving ${filePath}`);
-				let dupReport = `[jscpd] ${fileClones.length} duplicate block(s) involving ${path.basename(filePath)}:\n`;
+				let dupReport = `🟠 This file has ${fileClones.length} duplicate block(s) — extract to shared utilities:\n`;
 				for (const clone of fileClones.slice(0, 3)) {
 					const other = path.resolve(clone.fileA) === path.resolve(filePath)
 						? `${path.basename(clone.fileB)}:${clone.startB}`
 						: `${path.basename(clone.fileA)}:${clone.startA}`;
-					dupReport += `  ${clone.lines} lines — ${other}\n`;
+					dupReport += `  ${clone.lines} lines duplicated with ${other}\n`;
 				}
 				if (fileClones.length > 3) {
 					dupReport += `  ... and ${fileClones.length - 3} more\n`;
 				}
-				dupReport += `  → Extract duplicated code to a shared utility function\n`;
 				lspOutput += `\n\n${dupReport}`;
 			}
 		}
@@ -1365,11 +1372,11 @@ export default function (pi: ExtensionAPI) {
 				}
 				if (dupes.length > 0) {
 					dbg(`  duplicate exports: ${dupes.length} found`);
-					let exportReport = `[Duplicate Exports] ${dupes.length} function(s) already exist:\n`;
+					let exportReport = `🔴 Do not redefine — ${dupes.length} function(s) already exist elsewhere:\n`;
 					for (const dupe of dupes.slice(0, 5)) {
 						exportReport += `  ${dupe}\n`;
 					}
-					exportReport += `  → Import the existing function instead of redefining it\n`;
+					exportReport += `  → Import the existing function instead\n`;
 					lspOutput += `\n\n${exportReport}`;
 				}
 				// Update cache with new exports

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "2.0.0",
+	"version": "2.0.2",
 	"description": "Real-time code feedback for pi — TypeScript LSP, Biome, ast-grep, Ruff, TODO scanner, dead code, duplicate detection, type coverage",
 	"repository": {
 		"type": "git",