pi-lens 2.2.0 → 2.2.1

package/CHANGELOG.md

@@ -298,6 +298,11 @@ All notable changes to pi-lens will be documented in this file.
 ### Changed
 - **Improved ast-grep tool descriptions**: Better pattern guidance to prevent overly broad searches.
 
+## [2.2.1] - 2026-03-29
+
+### Fixed
+- **No auto-install**: Runners (biome, pyright) now use direct CLI commands instead of `npx`. If not installed, gracefully skip instead of attempting to download.
+
 ## [2.2.0] - 2026-03-29
 
 ### Added

package/README.md

@@ -395,7 +395,7 @@ Each rule includes a `message` and `note` that are shown in diagnostics, so the
 | `type-coverage` | `npm i -D type-coverage` | TypeScript `any` coverage percentage |
 | `madge` | `npm i -D madge` | Circular dependency detection |
 | `ruff` | `pip install ruff` | Python lint + format + autofix |
-| `pyright` | `npx pyright` (auto-installed) | Python type-checking |
+| `pyright` | `pip install pyright` or `npm i -g pyright` | Python type-checking (optional, graceful skip if not installed) |
 
 ---
 

package/clients/dispatch/runners/biome.js

@@ -1,27 +1,38 @@
 /**
  * Biome runner for dispatch system
+ *
+ * Requires: @biomejs/biome (npm install -D @biomejs/biome)
  */
 import { spawnSync } from "node:child_process";
+// Cache biome availability check
+let biomeAvailable = null;
+function isBiomeAvailable() {
+    if (biomeAvailable !== null)
+        return biomeAvailable;
+    // Check if biome CLI is available (do NOT auto-install via npx)
+    const check = spawnSync("biome", ["--version"], {
+        encoding: "utf-8",
+        timeout: 5000,
+        shell: true,
+    });
+    biomeAvailable = !check.error && check.status === 0;
+    return biomeAvailable;
+}
 const biomeRunner = {
     id: "biome-lint",
     appliesTo: ["jsts", "json"],
     priority: 10,
     enabledByDefault: true,
     async run(ctx) {
-        // Check if biome is available
-        const check = spawnSync("npx", ["@biomejs/biome", "--version"], {
-            encoding: "utf-8",
-            timeout: 5000,
-            shell: true,
-        });
-        if (check.error || check.status !== 0) {
+        // Skip if biome is not installed
+        if (!isBiomeAvailable()) {
             return { status: "skipped", diagnostics: [], semantic: "none" };
         }
-        // Run biome check
+        // Run biome check (use direct command, not npx)
         const args = ctx.autofix
             ? ["check", "--write", ctx.filePath]
             : ["check", ctx.filePath];
-        const result = spawnSync("npx", ["@biomejs/biome", ...args], {
+        const result = spawnSync("biome", args, {
             encoding: "utf-8",
             timeout: 30000,
             shell: true,

package/clients/dispatch/runners/biome.ts

@@ -1,5 +1,7 @@
 /**
  * Biome runner for dispatch system
+ *
+ * Requires: @biomejs/biome (npm install -D @biomejs/biome)
  */
 
 import { spawnSync } from "node:child_process";
@@ -10,6 +12,22 @@ import type {
 	RunnerResult,
 } from "../types.js";
 
+// Cache biome availability check
+let biomeAvailable: boolean | null = null;
+
+function isBiomeAvailable(): boolean {
+	if (biomeAvailable !== null) return biomeAvailable;
+
+	// Check if biome CLI is available (do NOT auto-install via npx)
+	const check = spawnSync("biome", ["--version"], {
+		encoding: "utf-8",
+		timeout: 5000,
+		shell: true,
+	});
+	biomeAvailable = !check.error && check.status === 0;
+	return biomeAvailable;
+}
+
 const biomeRunner: RunnerDefinition = {
 	id: "biome-lint",
 	appliesTo: ["jsts", "json"],
@@ -17,23 +35,17 @@ const biomeRunner: RunnerDefinition = {
 	enabledByDefault: true,
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
-		// Check if biome is available
-		const check = spawnSync("npx", ["@biomejs/biome", "--version"], {
-			encoding: "utf-8",
-			timeout: 5000,
-			shell: true,
-		});
-
-		if (check.error || check.status !== 0) {
+		// Skip if biome is not installed
+		if (!isBiomeAvailable()) {
 			return { status: "skipped", diagnostics: [], semantic: "none" };
 		}
 
-		// Run biome check
+		// Run biome check (use direct command, not npx)
 		const args = ctx.autofix
 			? ["check", "--write", ctx.filePath]
 			: ["check", ctx.filePath];
 
-		const result = spawnSync("npx", ["@biomejs/biome", ...args], {
+		const result = spawnSync("biome", args, {
 			encoding: "utf-8",
 			timeout: 30000,
 			shell: true,

package/clients/dispatch/runners/pyright.js

@@ -3,18 +3,38 @@
  *
  * Provides real Python type-checking (not just linting).
  * Catches type errors like: result: str = add(1, 2)  # Type "int" not assignable to "str"
+ *
+ * Requires: pyright (pip install pyright or npm install -g pyright)
  */
 import { spawnSync } from "node:child_process";
+// Cache pyright availability check
+let pyrightAvailable = null;
+function isPyrightAvailable() {
+    if (pyrightAvailable !== null)
+        return pyrightAvailable;
+    // Check if pyright CLI is available (do NOT auto-install via npx)
+    const check = spawnSync("pyright", ["--version"], {
+        encoding: "utf-8",
+        timeout: 5000,
+        shell: true,
+    });
+    pyrightAvailable = !check.error && check.status === 0;
+    return pyrightAvailable;
+}
 const pyrightRunner = {
     id: "pyright",
     appliesTo: ["python"],
     priority: 5, // Higher priority than ruff (10) - type errors are more important
     enabledByDefault: true,
     async run(ctx) {
-        // Run pyright with JSON output
-        const result = spawnSync("npx", ["pyright", "--outputjson", ctx.filePath], {
+        // Skip if pyright is not installed
+        if (!isPyrightAvailable()) {
+            return { status: "skipped", diagnostics: [], semantic: "none" };
+        }
+        // Run pyright with JSON output (use direct command, not npx)
+        const result = spawnSync("pyright", ["--outputjson", ctx.filePath], {
             encoding: "utf-8",
-            timeout: 60000, // Pyright can be slower on first run
+            timeout: 60000,
             shell: true,
         });
         // Pyright returns non-zero when errors found, that's OK

package/clients/dispatch/runners/pyright.ts

@@ -3,6 +3,8 @@
  *
  * Provides real Python type-checking (not just linting).
  * Catches type errors like: result: str = add(1, 2)  # Type "int" not assignable to "str"
+ *
+ * Requires: pyright (pip install pyright or npm install -g pyright)
  */
 
 import { spawnSync } from "node:child_process";
@@ -13,6 +15,22 @@ import type {
 	RunnerResult,
 } from "../types.js";
 
+// Cache pyright availability check
+let pyrightAvailable: boolean | null = null;
+
+function isPyrightAvailable(): boolean {
+	if (pyrightAvailable !== null) return pyrightAvailable;
+
+	// Check if pyright CLI is available (do NOT auto-install via npx)
+	const check = spawnSync("pyright", ["--version"], {
+		encoding: "utf-8",
+		timeout: 5000,
+		shell: true,
+	});
+	pyrightAvailable = !check.error && check.status === 0;
+	return pyrightAvailable;
+}
+
 const pyrightRunner: RunnerDefinition = {
 	id: "pyright",
 	appliesTo: ["python"],
@@ -20,10 +38,15 @@ const pyrightRunner: RunnerDefinition = {
 	enabledByDefault: true,
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
-		// Run pyright with JSON output
-		const result = spawnSync("npx", ["pyright", "--outputjson", ctx.filePath], {
+		// Skip if pyright is not installed
+		if (!isPyrightAvailable()) {
+			return { status: "skipped", diagnostics: [], semantic: "none" };
+		}
+
+		// Run pyright with JSON output (use direct command, not npx)
+		const result = spawnSync("pyright", ["--outputjson", ctx.filePath], {
 			encoding: "utf-8",
-			timeout: 60000, // Pyright can be slower on first run
+			timeout: 60000,
 			shell: true,
 		});
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "2.2.0",
+	"version": "2.2.1",
 	"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": {