pi-lens 2.0.1 → 2.0.3

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

@@ -256,25 +256,32 @@ export default function (pi: ExtensionAPI) {
 					const output = result.stdout || result.stderr || "";
 					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());
-
-						for (const line of lines) {
-							try {
-								const item = JSON.parse(line);
-								const ruleId = item.ruleId || item.name || "unknown";
-								const ruleDesc = astGrepClient.getRuleDescription?.(ruleId);
-								const message = ruleDesc?.message || item.message || ruleId;
-								const lineNum = item.labels?.[0]?.range?.start?.line ||
-									item.spans?.[0]?.range?.start?.line || 0;
-
-								issues.push({
-									line: lineNum + 1,
-									rule: ruleId,
-									message: message,
-								});
-							} catch {
-								// Skip unparseable lines
+
+						// ast-grep outputs either a JSON array or NDJSON (one object per line)
+						// biome-ignore lint/suspicious/noExplicitAny: ast-grep JSON output is untyped
+						const parseItems = (raw: string): Record<string, any>[] => {
+							const trimmed = raw.trim();
+							if (trimmed.startsWith("[")) {
+								try { return JSON.parse(trimmed); } catch { return []; }
 							}
+							return raw.split("\n").flatMap((l: string) => {
+								try { return [JSON.parse(l)]; } catch { return []; }
+							});
+						};
+
+						for (const item of parseItems(output)) {
+							const ruleId = item.ruleId || item.rule?.title || item.name || "unknown";
+							const ruleDesc = astGrepClient.getRuleDescription?.(ruleId);
+							const message = ruleDesc?.message || item.message || ruleId;
+							const lineNum = item.labels?.[0]?.range?.start?.line ||
+								item.spans?.[0]?.range?.start?.line ||
+								item.range?.start?.line || 0;
+
+							issues.push({
+								line: lineNum + 1,
+								rule: ruleId,
+								message: message,
+							});
 						}
 
 						if (issues.length > 0) {

package/package.json

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