pi-lens 2.2.2 → 2.2.3

package/clients/dispatch/runners/pyright.js

@@ -7,18 +7,44 @@
  * Requires: pyright (pip install pyright or npm install -g pyright)
  */
 import { spawnSync } from "node:child_process";
+import * as fs from "node:fs";
+import * as path from "node:path";
 // Cache pyright availability check
 let pyrightAvailable = null;
-function isPyrightAvailable() {
+let pyrightCommand = null;
+/**
+ * Find pyright command, checking venv first, then global.
+ * Looks in .venv/bin, venv/bin (Unix), .venv/Scripts, venv/Scripts (Windows)
+ */
+function findPyrightCommand(cwd) {
+    // Check common venv locations
+    const venvPaths = [
+        ".venv/bin/pyright",
+        "venv/bin/pyright",
+        ".venv/Scripts/pyright.exe",
+        "venv/Scripts/pyright.exe",
+    ];
+    for (const venvPath of venvPaths) {
+        const fullPath = path.join(cwd, venvPath);
+        if (fs.existsSync(fullPath)) {
+            return `"${fullPath}"`; // Quote for Windows paths with spaces
+        }
+    }
+    // Fall back to global
+    return "pyright";
+}
+function isPyrightAvailable(cwd) {
     if (pyrightAvailable !== null)
         return pyrightAvailable;
-    // Check if pyright CLI is available (do NOT auto-install via npx)
-    const check = spawnSync("pyright", ["--version"], {
+    const command = findPyrightCommand(cwd || process.cwd());
+    const check = spawnSync(command, ["--version"], {
         encoding: "utf-8",
         timeout: 5000,
         shell: true,
     });
     pyrightAvailable = !check.error && check.status === 0;
+    if (pyrightAvailable)
+        pyrightCommand = command;
     return pyrightAvailable;
 }
 const pyrightRunner = {
@@ -28,11 +54,11 @@ const pyrightRunner = {
     enabledByDefault: true,
     async run(ctx) {
         // Skip if pyright is not installed
-        if (!isPyrightAvailable()) {
+        if (!isPyrightAvailable(ctx.cwd || process.cwd())) {
             return { status: "skipped", diagnostics: [], semantic: "none" };
         }
-        // Run pyright with JSON output (use direct command, not npx)
-        const result = spawnSync("pyright", ["--outputjson", ctx.filePath], {
+        // Run pyright with JSON output (use venv-local or global command)
+        const result = spawnSync(pyrightCommand, ["--outputjson", ctx.filePath], {
             encoding: "utf-8",
             timeout: 60000,
             shell: true,

package/clients/dispatch/runners/pyright.ts

@@ -8,6 +8,8 @@
  */
 
 import { spawnSync } from "node:child_process";
+import * as fs from "node:fs";
+import * as path from "node:path";
 import type {
 	Diagnostic,
 	DispatchContext,
@@ -17,17 +19,44 @@ import type {
 
 // Cache pyright availability check
 let pyrightAvailable: boolean | null = null;
+let pyrightCommand: string | null = null;
 
-function isPyrightAvailable(): boolean {
+/**
+ * Find pyright command, checking venv first, then global.
+ * Looks in .venv/bin, venv/bin (Unix), .venv/Scripts, venv/Scripts (Windows)
+ */
+function findPyrightCommand(cwd: string): string {
+	// Check common venv locations
+	const venvPaths = [
+		".venv/bin/pyright",
+		"venv/bin/pyright",
+		".venv/Scripts/pyright.exe",
+		"venv/Scripts/pyright.exe",
+	];
+
+	for (const venvPath of venvPaths) {
+		const fullPath = path.join(cwd, venvPath);
+		if (fs.existsSync(fullPath)) {
+			return `"${fullPath}"`; // Quote for Windows paths with spaces
+		}
+	}
+
+	// Fall back to global
+	return "pyright";
+}
+
+function isPyrightAvailable(cwd?: string): boolean {
 	if (pyrightAvailable !== null) return pyrightAvailable;
 
-	// Check if pyright CLI is available (do NOT auto-install via npx)
-	const check = spawnSync("pyright", ["--version"], {
+	const command = findPyrightCommand(cwd || process.cwd());
+
+	const check = spawnSync(command, ["--version"], {
 		encoding: "utf-8",
 		timeout: 5000,
 		shell: true,
 	});
 	pyrightAvailable = !check.error && check.status === 0;
+	if (pyrightAvailable) pyrightCommand = command;
 	return pyrightAvailable;
 }
 
@@ -39,12 +68,12 @@ const pyrightRunner: RunnerDefinition = {
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
 		// Skip if pyright is not installed
-		if (!isPyrightAvailable()) {
+		if (!isPyrightAvailable(ctx.cwd || process.cwd())) {
 			return { status: "skipped", diagnostics: [], semantic: "none" };
 		}
 
-		// Run pyright with JSON output (use direct command, not npx)
-		const result = spawnSync("pyright", ["--outputjson", ctx.filePath], {
+		// Run pyright with JSON output (use venv-local or global command)
+		const result = spawnSync(pyrightCommand!, ["--outputjson", ctx.filePath], {
 			encoding: "utf-8",
 			timeout: 60000,
 			shell: true,

package/clients/dispatch/runners/ruff.js

@@ -2,29 +2,62 @@
  * Ruff runner for dispatch system
  *
  * Ruff handles both formatting and linting for Python files.
+ * Supports venv-local installations.
  */
 import { spawnSync } from "node:child_process";
+import * as fs from "node:fs";
+import * as path from "node:path";
 import { stripAnsi } from "../../sanitize.js";
+// Cache ruff availability check
+let ruffAvailable = null;
+let ruffCommand = null;
+/**
+ * Find ruff command, checking venv first, then global.
+ */
+function findRuffCommand(cwd) {
+    const venvPaths = [
+        ".venv/bin/ruff",
+        "venv/bin/ruff",
+        ".venv/Scripts/ruff.exe",
+        "venv/Scripts/ruff.exe",
+    ];
+    for (const venvPath of venvPaths) {
+        const fullPath = path.join(cwd, venvPath);
+        if (fs.existsSync(fullPath)) {
+            return `"${fullPath}"`;
+        }
+    }
+    return "ruff";
+}
+function isRuffAvailable(cwd) {
+    if (ruffAvailable !== null)
+        return ruffAvailable;
+    const command = findRuffCommand(cwd || process.cwd());
+    const check = spawnSync(command, ["--version"], {
+        encoding: "utf-8",
+        timeout: 5000,
+        shell: true,
+    });
+    ruffAvailable = !check.error && check.status === 0;
+    if (ruffAvailable)
+        ruffCommand = command;
+    return ruffAvailable;
+}
 const ruffRunner = {
     id: "ruff-lint",
     appliesTo: ["python"],
     priority: 10,
     enabledByDefault: true,
     async run(ctx) {
-        // Check if ruff is available
-        const check = spawnSync("ruff", ["--version"], {
-            encoding: "utf-8",
-            timeout: 5000,
-            shell: true,
-        });
-        if (check.error || check.status !== 0) {
+        // Skip if ruff is not installed
+        if (!isRuffAvailable(ctx.cwd || process.cwd())) {
             return { status: "skipped", diagnostics: [], semantic: "none" };
         }
         // Run ruff check
         const args = ctx.autofix
             ? ["check", "--fix", ctx.filePath]
             : ["check", ctx.filePath];
-        const result = spawnSync("ruff", args, {
+        const result = spawnSync(ruffCommand, args, {
             encoding: "utf-8",
             timeout: 30000,
             shell: true,

package/clients/dispatch/runners/ruff.ts

@@ -2,9 +2,12 @@
  * Ruff runner for dispatch system
  *
  * Ruff handles both formatting and linting for Python files.
+ * Supports venv-local installations.
  */
 
 import { spawnSync } from "node:child_process";
+import * as fs from "node:fs";
+import * as path from "node:path";
 import { stripAnsi } from "../../sanitize.js";
 import type {
 	Diagnostic,
@@ -13,6 +16,46 @@ import type {
 	RunnerResult,
 } from "../types.js";
 
+// Cache ruff availability check
+let ruffAvailable: boolean | null = null;
+let ruffCommand: string | null = null;
+
+/**
+ * Find ruff command, checking venv first, then global.
+ */
+function findRuffCommand(cwd: string): string {
+	const venvPaths = [
+		".venv/bin/ruff",
+		"venv/bin/ruff",
+		".venv/Scripts/ruff.exe",
+		"venv/Scripts/ruff.exe",
+	];
+
+	for (const venvPath of venvPaths) {
+		const fullPath = path.join(cwd, venvPath);
+		if (fs.existsSync(fullPath)) {
+			return `"${fullPath}"`;
+		}
+	}
+
+	return "ruff";
+}
+
+function isRuffAvailable(cwd?: string): boolean {
+	if (ruffAvailable !== null) return ruffAvailable;
+
+	const command = findRuffCommand(cwd || process.cwd());
+	const check = spawnSync(command, ["--version"], {
+		encoding: "utf-8",
+		timeout: 5000,
+		shell: true,
+	});
+
+	ruffAvailable = !check.error && check.status === 0;
+	if (ruffAvailable) ruffCommand = command;
+	return ruffAvailable;
+}
+
 const ruffRunner: RunnerDefinition = {
 	id: "ruff-lint",
 	appliesTo: ["python"],
@@ -20,14 +63,8 @@ const ruffRunner: RunnerDefinition = {
 	enabledByDefault: true,
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
-		// Check if ruff is available
-		const check = spawnSync("ruff", ["--version"], {
-			encoding: "utf-8",
-			timeout: 5000,
-			shell: true,
-		});
-
-		if (check.error || check.status !== 0) {
+		// Skip if ruff is not installed
+		if (!isRuffAvailable(ctx.cwd || process.cwd())) {
 			return { status: "skipped", diagnostics: [], semantic: "none" };
 		}
 
@@ -36,7 +73,7 @@ const ruffRunner: RunnerDefinition = {
 			? ["check", "--fix", ctx.filePath]
 			: ["check", ctx.filePath];
 
-		const result = spawnSync("ruff", args, {
+		const result = spawnSync(ruffCommand!, args, {
 			encoding: "utf-8",
 			timeout: 30000,
 			shell: true,

package/package.json

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