@herbertgao/pi-extensions 2026.9.0 → 2026.9.1

package/node_modules/pi-lens/dist/clients/dispatch/runners/helm-render.js

@@ -893,7 +893,6 @@ const helmRenderRunner = {
     id: "helm-render",
     appliesTo: ["yaml", "helm-template"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     timeoutMs: RENDER_TIMEOUT_MS + TRIVY_TIMEOUT_MS + 10_000,
     async run(ctx) {

package/node_modules/pi-lens/dist/clients/dispatch/runners/htmlhint.js

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

package/node_modules/pi-lens/dist/clients/dispatch/runners/javac.js

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

package/node_modules/pi-lens/dist/clients/dispatch/runners/ktlint.js

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

package/node_modules/pi-lens/dist/clients/dispatch/runners/lsp.js

@@ -93,7 +93,6 @@ const lspRunner = {
     // is now the only step this seam needs (#1545).
     appliesTo: getLspCapableKinds(),
     priority: PRIORITY.LSP_PRIMARY,
-    enabledByDefault: true,
     timeoutMs: LSP_RUNNER_TIMEOUT_MS,
     async run(ctx) {
         const diagnosticPath = resolveRunnerPath(ctx.cwd, ctx.filePath);

package/node_modules/pi-lens/dist/clients/dispatch/runners/markdownlint.js

@@ -94,7 +94,6 @@ const markdownlintRunner = {
     id: "markdownlint",
     appliesTo: ["markdown"],
     priority: PRIORITY.DOC_QUALITY,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/node_modules/pi-lens/dist/clients/dispatch/runners/mypy.js

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

package/node_modules/pi-lens/dist/clients/dispatch/runners/oxlint.js

@@ -6,9 +6,7 @@
  *
  * Requires: oxlint (npm install -g oxlint)
  */
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { walkUpDirs } from "../../path-utils.js";
+import { findLocalBinUpwards } from "../../package-manager.js";
 import { safeSpawnAsync } from "../../safe-spawn.js";
 import { truncatedByOutputCap } from "../../spawn-output-cap.js";
 import { getJstsLintPolicyForCwd, hasVitePlusConfig, } from "../../tool-policy.js";
@@ -30,21 +28,19 @@ const OXLINT_NO_FILES_REPORT_KEYS = new Set([
     "threads_count",
     "start_time",
 ]);
+/**
+ * Vite+'s `vp` from the project's own `node_modules/.bin`.
+ *
+ * Delegates to THE shared bin walker (#2544 review F1). This was a private,
+ * UNCEILINGED copy of that walk, while `formatters.ts`'s `oxfmtFormatter`
+ * resolved the SAME binary — `vp` — through the ceilinged `findInNodeModules`:
+ * one `vp` resolver stopped at `$HOME` and the other climbed past it to the
+ * filesystem root, so which policy applied depended on whether the format path
+ * or the lint path asked. A `node_modules/.bin/vp` at or above `$HOME` can
+ * never be this project's own Vite+ install (#2514).
+ */
 function resolveLocalVp(cwd) {
-    const isWin = process.platform === "win32";
-    for (const dir of walkUpDirs(cwd)) {
-        const candidates = isWin
-            ? [
-                path.join(dir, "node_modules", ".bin", "vp.cmd"),
-                path.join(dir, "node_modules", ".bin", "vp"),
-            ]
-            : [path.join(dir, "node_modules", ".bin", "vp")];
-        for (const candidate of candidates) {
-            if (fs.existsSync(candidate))
-                return candidate;
-        }
-    }
-    return null;
+    return findLocalBinUpwards("vp", cwd) ?? null;
 }
 async function resolveVitePlusCommand(cwd) {
     const local = resolveLocalVp(cwd);
@@ -60,7 +56,6 @@ const oxlintRunner = {
     id: "oxlint",
     appliesTo: ["jsts"],
     priority: PRIORITY.LINT_SECONDARY,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();

package/node_modules/pi-lens/dist/clients/dispatch/runners/php-lint.js

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

package/node_modules/pi-lens/dist/clients/dispatch/runners/phpstan.js

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

package/node_modules/pi-lens/dist/clients/dispatch/runners/prisma-validate.js

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

package/node_modules/pi-lens/dist/clients/dispatch/runners/psscriptanalyzer.js

@@ -269,7 +269,6 @@ const psScriptAnalyzerRunner = {
     id: "psscriptanalyzer",
     appliesTo: ["powershell"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cmd = await resolvePowerShellCmd();

package/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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.
@@ -49,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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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/node_modules/pi-lens/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, "\\$&");
-}