pi-lens 1.3.12 → 1.3.13

package/clients/complexity-client.ts

@@ -291,6 +291,17 @@ export class ComplexityClient {
       warnings.push(`High entropy (${metrics.codeEntropy.toFixed(1)} bits) — follow project conventions`);
     }
 
+    // Comments ratio (>30% = excessive comments, AI slop signal)
+    const totalLines = metrics.linesOfCode + metrics.commentLines;
+    if (totalLines > 10 && metrics.commentLines / totalLines > 0.3) {
+      warnings.push(`Excessive comments (${Math.round(metrics.commentLines / totalLines * 100)}%) — remove obvious comments`);
+    }
+
+    // Verbose code (long functions with low complexity = overly verbose)
+    if (metrics.avgFunctionLength > 30 && metrics.cyclomaticComplexity < 3) {
+      warnings.push(`Verbose code (avg ${Math.round(metrics.avgFunctionLength)} lines, low complexity) — simplify or extract`);
+    }
+
     return warnings;
   }
 

package/index.ts

@@ -711,10 +711,24 @@ export default function (pi: ExtensionAPI) {
 
 			const diags = tsClient.getDiagnostics(filePath);
 			if (diags.length > 0) {
-				lspOutput += `\n\n[TypeScript] ${diags.length} issue(s):\n`;
-				for (const d of diags.slice(0, 10)) {
-					const label = d.severity === 2 ? "Warning" : "Error";
-					lspOutput += `  [${label}] L${d.range.start.line + 1}: ${d.message}\n`;
+				// Separate unused imports (TS6133, TS6196) from other diagnostics
+				const unusedImports = diags.filter(d => d.code === 6133 || d.code === 6196);
+				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`;
+					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`;
+					}
 				}
 			}
 		}

package/package.json

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

package/rules/ast-grep-rules/rules/magic-numbers.yml

@@ -0,0 +1,9 @@
+id: magic-numbers
+language: TypeScript
+message: "Magic number detected — use a named constant"
+severity: hint
+note: |
+  Hardcoded numbers make code harder to understand and maintain.
+  Extract to a named constant with a descriptive name.
+rule:
+  kind: number

package/rules/ast-grep-rules/rules/silent-failure.yml

@@ -0,0 +1,14 @@
+id: silent-failure
+language: TypeScript
+message: "Silent failure detected — log or rethrow the error"
+severity: warning
+note: |
+  Empty catch blocks swallow errors silently.
+  Always log the error or rethrow it to aid debugging.
+rule:
+  kind: catch_clause
+  has:
+    kind: statement_block
+    not:
+      has:
+        kind: throw_statement