pi-lens 4.1.2 → 4.1.4

package/dist/clients/dispatch/runners/pyright.js

@@ -8,6 +8,7 @@
  */
 import { logExtension } from "../../extension-log.js";
 import { getLSPService } from "../../lsp/index.js";
+import { augmentPythonEnvironment, detectPythonEnvironment, } from "../../python-environment.js";
 import { safeSpawnAsync } from "../../safe-spawn.js";
 import { PRIORITY } from "../priorities.js";
 import { createAvailabilityChecker, resolveAvailableOrInstall, } from "./utils/runner-helpers.js";
@@ -17,7 +18,6 @@ const pyrightRunner = {
     id: "pyright",
     appliesTo: ["python"],
     priority: PRIORITY.LSP_FALLBACK,
-    enabledByDefault: true,
     timeoutMs: 75_000,
     async run(ctx) {
         // Always allow pyright CLI fallback even when LSP is enabled.
@@ -52,12 +52,15 @@ const pyrightRunner = {
         if (!cmd) {
             return { status: "skipped", diagnostics: [], semantic: "none" };
         }
-        // Run pyright with JSON output. Pass cwd so pyright resolves
-        // pyrightconfig.json / the project venv from the project root rather than
-        // falling back to process.cwd() (which mis-resolves in multi-root setups).
+        // Pyright's Node CLI falls back to `python` on PATH when the project has no
+        // explicit Pyright config. Apply the same environment the Python LSP uses so
+        // a conventional project .venv resolves without shell activation.
+        const pythonEnvironment = await detectPythonEnvironment(cwd);
+        const env = augmentPythonEnvironment(process.env, pythonEnvironment);
         const result = await safeSpawnAsync(cmd, ["--outputjson", ctx.filePath], {
             timeout: 60000,
             cwd,
+            env,
         });
         // Pyright returns non-zero when errors found, that's OK
         if (result.error) {

package/dist/clients/dispatch/runners/rubocop.js

@@ -54,7 +54,6 @@ const rubocopRunner = {
     id: "rubocop",
     appliesTo: ["ruby"],
     priority: PRIORITY.FORMAT_AND_LINT_PRIMARY,
-    enabledByDefault: true,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();
         const policy = getLinterPolicyForCwd(ctx.filePath, cwd);

package/dist/clients/dispatch/runners/ruff.js

@@ -49,7 +49,6 @@ const ruffRunner = {
     id: "ruff-lint",
     appliesTo: ["python"],
     priority: PRIORITY.FORMAT_AND_LINT_PRIMARY,
-    enabledByDefault: true,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();
         const policy = getLinterPolicyForCwd(ctx.filePath, cwd);

package/dist/clients/dispatch/runners/rust-clippy.js

@@ -5,14 +5,13 @@
  */
 import { dirname, isAbsolute, join, resolve } from "node:path";
 import { findNearestContaining } from "../../path-utils.js";
-import { RustClient } from "../../rust-client.js";
+import { rustClient } from "../../rust-client.js";
 import { safeSpawnAsync } from "../../safe-spawn.js";
 import { stripAnsi } from "../../sanitize.js";
 import { getLazyInstallAttempt, tryLazyInstall, } from "./utils/lazy-installer.js";
 import { describeInstallAttempt, logAvailabilityDecision, } from "./utils/availability-policy.js";
 import { PRIORITY } from "../priorities.js";
 import { createCwdCachedProbe } from "./utils/runner-helpers.js";
-const rustClient = new RustClient();
 // Cached per-cwd `cargo clippy --version` probe (#120). Before this, the
 // probe fired on every Rust file save in a project where clippy was already
 // installed.
@@ -26,17 +25,22 @@ const rustClient = new RustClient();
 // The verdict is governed by the shared availability policy (#1494): a probe
 // timeout expires on a cooldown, so only a genuinely absent clippy latches.
 const CLIPPY_PROBE_BUDGET_MS = 8000;
-const makeClippyProbe = (cargoExe) => createCwdCachedProbe((cwd) => safeSpawnAsync(cargoExe, ["clippy", "--version"], {
+let clippyProbeRevision = 0;
+const makeClippyProbe = (cargoExe, flightKeyComponent = cargoExe) => createCwdCachedProbe((cwd) => safeSpawnAsync(cargoExe, ["clippy", "--version"], {
     timeout: CLIPPY_PROBE_BUDGET_MS,
     cwd,
-}), { tool: "clippy", budgetMs: CLIPPY_PROBE_BUDGET_MS });
+}), {
+    tool: "clippy",
+    budgetMs: CLIPPY_PROBE_BUDGET_MS,
+    flightKeyComponent,
+});
 const clippyAvailabilityByCargo = new Map();
 function getClippyProbe(cargoExe) {
     return (clippyAvailabilityByCargo.get(cargoExe) ?? refreshClippyProbe(cargoExe));
 }
 /** Replace the cached probe so a post-install state is observed. */
 function refreshClippyProbe(cargoExe) {
-    const created = makeClippyProbe(cargoExe);
+    const created = makeClippyProbe(cargoExe, `${cargoExe}#refresh-${++clippyProbeRevision}`);
     clippyAvailabilityByCargo.set(cargoExe, created);
     return created;
 }
@@ -44,7 +48,6 @@ const rustClippyRunner = {
     id: "rust-clippy",
     appliesTo: ["rust"],
     priority: PRIORITY.SPECIALIZED_ANALYSIS,
-    enabledByDefault: true,
     timeoutMs: 90_000,
     async run(ctx) {
         // Resolve cargo path using platform-aware lookup (handles ~/.cargo/bin on Windows)

package/dist/clients/dispatch/runners/shellcheck.js

@@ -108,7 +108,6 @@ const shellcheckRunner = {
     id: "shellcheck",
     appliesTo: ["shell"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false, // Shell scripts in test directories should still be checked
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/shfmt.js

@@ -29,7 +29,6 @@ const shfmtRunner = {
     id: "shfmt",
     appliesTo: ["shell"],
     priority: PRIORITY.FORMAT_AND_LINT_PRIMARY,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/spellcheck.js

@@ -80,7 +80,6 @@ const spellcheckRunner = {
     id: "spellcheck",
     appliesTo: ["markdown"],
     priority: PRIORITY.DOC_QUALITY,
-    enabledByDefault: true,
     skipTestFiles: false, // Check docs in test files too
     async run(ctx) {
         // Skip if typos-cli is not installed

package/dist/clients/dispatch/runners/spotbugs.js

@@ -190,7 +190,6 @@ const spotbugsRunner = {
     id: "spotbugs",
     appliesTo: ["java", "kotlin"],
     priority: PRIORITY.DEEP_LANGUAGE_ANALYSIS,
-    enabledByDefault: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();
         const classesDir = findCompiledClassesDir(cwd);

package/dist/clients/dispatch/runners/sqlfluff.js

@@ -115,7 +115,6 @@ const sqlfluffRunner = {
     id: "sqlfluff",
     appliesTo: ["sql"],
     priority: PRIORITY.SQL_LINT,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/stylelint.js

@@ -114,7 +114,6 @@ const stylelintRunner = {
     id: "stylelint",
     appliesTo: ["css"],
     priority: PRIORITY.FORMAT_AND_LINT_PRIMARY,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/swiftlint.js

@@ -113,7 +113,6 @@ const swiftlintRunner = {
     id: "swiftlint",
     appliesTo: ["swift"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/taplo.js

@@ -104,7 +104,6 @@ const taploRunner = {
     id: "taplo",
     appliesTo: ["toml"],
     priority: PRIORITY.FORMAT_AND_LINT_PRIMARY,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/terragrunt.js

@@ -126,7 +126,6 @@ const terragruntRunner = {
     id: "terragrunt",
     appliesTo: ["terragrunt"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/tflint.js

@@ -50,7 +50,6 @@ const tflintRunner = {
     id: "tflint",
     appliesTo: ["terraform"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/tree-sitter.js

@@ -325,7 +325,6 @@ const treeSitterRunner = {
         "css",
     ],
     priority: PRIORITY.STRUCTURAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false, // Run on test files too (structural issues matter there)
     async run(ctx) {
         // Use singleton client — WASM must never be re-initialized after first call

package/dist/clients/dispatch/runners/trivy-config.js

@@ -146,7 +146,6 @@ const trivyConfigRunner = {
     id: "trivy-config",
     appliesTo: ["docker", "yaml", "terraform", "json"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/dist/clients/dispatch/runners/utils/availability-policy.js

@@ -460,6 +460,9 @@ export function logAvailabilityDecision(decision, filePath = "<pi-lens>") {
         durationMs: Math.round(decision.elapsedMs),
         metadata: {
             tool: decision.tool,
+            ...(decision.producer !== undefined && {
+                producer: decision.producer,
+            }),
             verdict: decision.verdict,
             outcome: decision.outcome,
             cause: decision.cause,

package/dist/clients/dispatch/runners/utils/diagnostic-parsers.js

@@ -4,6 +4,8 @@
  * Common patterns for parsing tool output into standardized diagnostics.
  * Supports the common `file:line:col: message` format used by most linters.
  */
+import { stripAnsi } from "../../../sanitize.js";
+import { escapeRegExp } from "../../../string-utils.js";
 import { getAutofixCapability } from "../../../tool-policy.js";
 /**
  * Create a parser for line-based tool output.
@@ -13,7 +15,7 @@ export function createLineParser(config) {
     return (raw, filePath) => {
         const diagnostics = [];
         // Optionally strip ANSI codes (for tools that output colored text)
-        const clean = config.stripAnsi !== false ? raw.replace(/\x1b\[[0-9;]*m/g, "") : raw;
+        const clean = config.stripAnsi !== false ? stripAnsi(raw) : raw;
         const lines = clean.split("\n").filter((l) => l.trim());
         for (const line of lines) {
             const match = line.match(config.regex);
@@ -144,6 +146,3 @@ export function createSimpleParser(tool, options = {}) {
         return diagnostics;
     };
 }
-function escapeRegExp(string) {
-    return string.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-}