pi-lens 1.3.11 → 1.3.13

package/CHANGELOG.md

@@ -2,6 +2,50 @@
 
 All notable changes to pi-lens will be documented in this file.
 
+## [1.3.11] - 2026-03-25
+
+### Added
+- **Actionable feedback messages**: All real-time warnings now include specific guidance on what to do.
+- **Code entropy metric**: Shannon entropy in bits (threshold: >3.5 indicates risky AI-induced complexity).
+- **Advanced pattern matching**: `/lens-booboo` now finds structurally similar functions (e.g., `formatDate` and `formatTimestamp`).
+- **Duplicate export detection**: Warns when redefining a function that already exists in the codebase.
+- **Biome formatting noise removed**: Only lint issues shown in real-time; use `/lens-format` for formatting.
+
+## [1.3.10] - 2026-03-25
+
+### Added
+- **Actionable complexity warnings**: Real-time feedback when metrics break limits with specific fix guidance.
+
+## [1.3.9] - 2026-03-25
+
+### Fixed
+- **Entropy calculation**: Corrected to use bits with 3.5-bit threshold for AI-induced complexity.
+
+## [1.3.8] - 2026-03-25
+
+### Added
+- **Code entropy metric**: Shannon entropy to detect repetitive or unpredictable code patterns.
+
+## [1.3.7] - 2026-03-25
+
+### Added
+- **Advanced pattern matching in `/lens-booboo`**: Finds structurally similar functions across the codebase.
+
+## [1.3.6] - 2026-03-25
+
+### Added
+- **Duplicate export detection on write**: Warns when defining a function that already exists elsewhere.
+
+## [1.3.5] - 2026-03-25
+
+### Changed
+- **Consistent command prefix**: All commands now start with `lens-`.
+  - `/find-todos` → `/lens-todos`
+  - `/dead-code` → `/lens-dead-code`
+  - `/check-deps` → `/lens-deps`
+  - `/format` → `/lens-format`
+  - `/design-review` + `/lens-metrics` → `/lens-booboo`
+
 ## [1.5.0] - 2026-03-23
 
 ### Added

package/README.md

@@ -15,8 +15,38 @@ Real-time code quality feedback for [pi](https://github.com/mariozechner/pi-codi
 | **Biome** | Lint + format for JS/TS/JSX/TSX/CSS/JSON. Auto-fix disabled by default, use `/lens-format` to apply |
 | **Ruff** | Lint + format for Python. Auto-fixes on every write by default |
 | **Test Runner** | Runs corresponding test file when you edit source code (vitest, jest, pytest). Silent if no test file exists. |
-| **Complexity Metrics** | AST-based analysis: Maintainability Index, Cyclomatic/Cognitive Complexity, Halstead Volume, nesting depth, function length. |
+| **Complexity Metrics** | AST-based analysis: Maintainability Index, Cyclomatic/Cognitive Complexity, Halstead Volume, nesting depth, function length, code entropy. |
 | **jscpd** | Code duplication detection. Warns when editing a file that has duplicates with other files in the project. |
+| **Duplicate Exports** | Detects when you redefine a function that already exists elsewhere in the codebase. |
+
+### Actionable feedback
+
+All warnings include actionable guidance — the agent sees what to do, not just what's wrong:
+
+```
+[TypeScript] 2 issue(s):
+  [error] L10: Type 'string' is not assignable to type 'number'
+
+[ast-grep] 1 structural issue(s) — 1 warning(s):
+  no-console-log: console.log found (L15)
+    → Use a proper logging framework or remove before committing.
+
+[jscpd] 1 duplicate block(s) involving utils.ts:
+  15 lines — helpers.ts:20
+  → Extract duplicated code to a shared utility function
+
+[Duplicate Exports] 1 function(s) already exist:
+  formatDate (already in helpers.ts)
+  → Import the existing function instead of redefining it
+
+[Complexity Warnings]
+  ⚠ Maintainability dropped to 55 — extract logic into helper functions
+  ⚠ High entropy (4.2 bits) — follow project conventions
+
+[Tests] ✗ 1/3 failed, 2 passed
+  ✗ should format date
+  → Fix failing tests before proceeding
+```
 
 ### Pre-write hints
 

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.11",
+	"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