@herbertgao/pi-extensions 2026.8.11 → 2026.8.13

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

@@ -5,6 +5,7 @@ import { getLinterPolicyForCwd } from "../../tool-policy.js";
 import { PRIORITY } from "../priorities.js";
 import { createAvailabilityChecker, resolveAvailableOrInstall, } from "./utils/runner-helpers.js";
 import { spawnFailedWithNoOutput } from "./utils/spawn-outcome.js";
+import { finishParsedRun } from "./utils/tool-failure.js";
 const terragrunt = createAvailabilityChecker("terragrunt", ".exe");
 function normalizeSeverity(raw) {
     if (typeof raw === "number")
@@ -149,15 +150,19 @@ const terragruntRunner = {
         if (spawnFailedWithNoOutput(result))
             return SKIPPED;
         const diagnostics = parseTerragruntOutput(result.stdout || "", ctx.filePath, absPath);
-        if (diagnostics.length === 0) {
-            return { status: "succeeded", diagnostics: [], semantic: "none" };
-        }
-        const hasErrors = diagnostics.some((d) => d.severity === "error");
-        return {
-            status: hasErrors ? "failed" : "succeeded",
+        return finishParsedRun({
+            tool: "terragrunt",
+            ctx,
+            result,
             diagnostics,
-            semantic: hasErrors ? "blocking" : "warning",
-        };
+            classify: (diagnostics) => {
+                const hasErrors = diagnostics.some((d) => d.severity === "error");
+                return {
+                    status: hasErrors ? "failed" : "succeeded",
+                    semantic: hasErrors ? "blocking" : "warning",
+                };
+            },
+        });
     },
 };
 export default terragruntRunner;

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

@@ -5,6 +5,7 @@ import { findNearestDirWithAnyBasename } from "../../workspace-topology.js";
 import { PRIORITY } from "../priorities.js";
 import { createAvailabilityChecker, resolveAvailableOrInstall, } from "./utils/runner-helpers.js";
 import { parseToolRun } from "./utils/tool-failure.js";
+import { finishParsedRun } from "./utils/tool-failure.js";
 const tflint = createAvailabilityChecker("tflint", ".exe");
 const TFLINT_CONFIG = ".tflint.hcl";
 /**
@@ -91,15 +92,19 @@ const tflintRunner = {
         if (run.skipped)
             return run.skipped;
         const diagnostics = run.diagnostics;
-        if (diagnostics.length === 0) {
-            return { status: "succeeded", diagnostics: [], semantic: "none" };
-        }
-        const hasErrors = diagnostics.some((d) => d.severity === "error");
-        return {
-            status: hasErrors ? "failed" : "succeeded",
+        return finishParsedRun({
+            tool: "tflint",
+            ctx,
+            result,
             diagnostics,
-            semantic: hasErrors ? "blocking" : "warning",
-        };
+            classify: (diagnostics) => {
+                const hasErrors = diagnostics.some((d) => d.severity === "error");
+                return {
+                    status: hasErrors ? "failed" : "succeeded",
+                    semantic: hasErrors ? "blocking" : "warning",
+                };
+            },
+        });
     },
 };
 export default tflintRunner;

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

@@ -300,7 +300,9 @@ const ENTITY_EXTRACTION_LINE_THRESHOLD = 5;
  */
 function relativeForIgnoreGlob(filePath, root) {
     const rel = path.relative(root, filePath);
-    const normalized = (rel.startsWith("..") ? filePath : rel).split(path.sep).join("/");
+    const normalized = (rel.startsWith("..") ? filePath : rel)
+        .split(path.sep)
+        .join("/");
     return normalized;
 }
 function matchesIgnorePaths(filePath, root, patterns) {
@@ -311,7 +313,17 @@ function matchesIgnorePaths(filePath, root, patterns) {
 }
 const treeSitterRunner = {
     id: "tree-sitter",
-    appliesTo: ["jsts", "python", "go", "rust", "ruby", "cxx", "csharp", "php", "css"],
+    appliesTo: [
+        "jsts",
+        "python",
+        "go",
+        "rust",
+        "ruby",
+        "cxx",
+        "csharp",
+        "php",
+        "css",
+    ],
     priority: PRIORITY.STRUCTURAL_ANALYSIS,
     enabledByDefault: true,
     skipTestFiles: false, // Run on test files too (structural issues matter there)
@@ -323,7 +335,9 @@ const treeSitterRunner = {
             logTreeSitter({
                 phase: "runner_skip",
                 filePath: ctx.filePath,
-                reason: isTreeSitterWasmAborted() ? "wasm_aborted" : "client_unavailable",
+                reason: isTreeSitterWasmAborted()
+                    ? "wasm_aborted"
+                    : "client_unavailable",
                 status: "skipped",
             });
             return { status: "skipped", diagnostics: [], semantic: "none" };

package/node_modules/pi-lens/dist/clients/dispatch/runners/utils/candidate-probe.js

@@ -80,5 +80,11 @@ export async function probeAvailabilityCandidates(candidates, probeArgs, timeout
             // above rather than through a thrown spawn.
         }
     }
-    return { foundPath: null, sawTransient, transientCause, hostStallMs, evidence };
+    return {
+        foundPath: null,
+        sawTransient,
+        transientCause,
+        hostStallMs,
+        evidence,
+    };
 }

package/node_modules/pi-lens/dist/clients/dispatch/runners/utils/runner-helpers.js

@@ -19,6 +19,7 @@ import { getServersForFileWithConfig, isServerDisabled, } from "../../../lsp/con
 import { findGlobalBinary } from "../../../package-manager.js";
 import { safeSpawnAsync } from "../../../safe-spawn.js";
 import { getToolCommandSpec, shouldAutoInstallTool, } from "../../../tool-policy.js";
+import { isInSpawnTimeoutCooldown } from "../../../spawn-timeout-cooldown.js";
 import { classifyProbeFailure, createAvailabilityLatch, describeInstallAttempt, describeProbeEvidence, isLatchingOutcome, logAvailabilityDecision, startHostStallSampler, transientRetryDelayMs, } from "./availability-policy.js";
 export { createAvailabilityLatch, classifyProbeFailure, describeProbeEvidence, describeUnavailability, isTransientDecision, logAvailabilityDecision, startHostStallSampler, } from "./availability-policy.js";
 /**
@@ -777,6 +778,27 @@ export function createAvailabilityChecker(command, windowsExt = "", versionArgs
                 return false;
             }
             const cmd = await findCommand(resolvedCwd);
+            // #1995: a command cooling down after a RUNTIME timeout (lint or
+            // autofix lane blew its real budget) must not re-probe on every
+            // edit - the positive verdict is effectively cooled. Consult-only:
+            // probe timeouts do NOT arm the cooldown here, because a single
+            // slow --version under host stall (#1467) is transient evidence,
+            // and session-long suppression from it would contradict the
+            // #1494 retry ladder this classification exists for.
+            if (isInSpawnTimeoutCooldown(cmd)) {
+                noteDecision(cache, resolvedCwd, {
+                    available: false,
+                    outcome: "transient",
+                    cause: "probe-timeout",
+                    elapsedMs: 0,
+                    classifiedBy: "caller",
+                    evidence: {
+                        command: cmd,
+                        status: null,
+                    },
+                });
+                return false;
+            }
             const env = await options.environment?.(resolvedCwd);
             // The probe budget is enforced by a HOST-side timer, so host event-loop
             // stalls are charged to the child. Measure the stall that overlapped the
@@ -856,7 +878,8 @@ export function createAvailabilityChecker(command, windowsExt = "", versionArgs
             outcome: cache.outcome,
             cause: cache.cause,
             elapsedMs: cache.elapsedMs,
-            latched: cache.available !== false || isLatchingOutcome(cache.outcome ?? "missing"),
+            latched: cache.available !== false ||
+                isLatchingOutcome(cache.outcome ?? "missing"),
             retryAtMs: cache.retryAtMs,
         };
     }

package/node_modules/pi-lens/dist/clients/dispatch/runners/utils/tool-failure.js

@@ -186,3 +186,38 @@ export function parseToolRun(tool, input, parse, options = {}) {
     }
     return { skipped: null, diagnostics, parsedNothing };
 }
+export function finishParsedRun(input) {
+    const raw = `${input.result.stdout ?? ""}${input.result.stderr ?? ""}`;
+    const status = input.result.status ?? null;
+    if (input.diagnostics.length === 0) {
+        if (status !== 0 && raw.trim().length > 0) {
+            return {
+                status: "failed",
+                diagnostics: [
+                    {
+                        id: `${input.tool}:parse-error:1`,
+                        message: `${input.tool} exited ${status} but its output could not be parsed`,
+                        filePath: input.ctx.filePath,
+                        line: 1,
+                        column: 1,
+                        severity: "warning",
+                        semantic: "warning",
+                        tool: input.tool,
+                    },
+                ],
+                semantic: "warning",
+            };
+        }
+        return { status: "succeeded", diagnostics: [], semantic: "none" };
+    }
+    const mapped = input.classify?.(input.diagnostics);
+    if (mapped) {
+        return { ...mapped, diagnostics: input.diagnostics };
+    }
+    const hasBlocking = input.diagnostics.some((d) => d.semantic === "blocking");
+    return {
+        status: hasBlocking ? "failed" : "succeeded",
+        diagnostics: input.diagnostics,
+        semantic: hasBlocking ? "blocking" : "warning",
+    };
+}

package/node_modules/pi-lens/dist/clients/dispatch/runners/yaml-rule-parser.js

@@ -106,9 +106,7 @@ export function loadYamlRulesFresh(ruleDir, severityFilter) {
         hash.update("\0");
     }
     const signature = hash.digest("hex");
-    const cache = severityFilter === "error"
-        ? contentBlockingRulesCache
-        : contentRulesCache;
+    const cache = severityFilter === "error" ? contentBlockingRulesCache : contentRulesCache;
     const cached = cache.get(ruleDir);
     if (cached?.signature === signature)
         return cached.rules;

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

@@ -3,6 +3,7 @@ import { getLinterPolicyForCwd, hasYamllintConfig } from "../../tool-policy.js";
 import { PRIORITY } from "../priorities.js";
 import { createAvailabilityChecker, resolveToolCommandWithInstallFallback, } from "./utils/runner-helpers.js";
 import { parseToolRun } from "./utils/tool-failure.js";
+import { finishParsedRun } from "./utils/tool-failure.js";
 const yamllint = createAvailabilityChecker("yamllint", ".exe");
 export { hasYamllintConfig };
 function parseYamllintParsable(raw, filePath) {
@@ -72,15 +73,12 @@ const yamllintRunner = {
         if (run.skipped)
             return run.skipped;
         const diagnostics = run.diagnostics;
-        if (diagnostics.length === 0) {
-            return { status: "succeeded", diagnostics: [], semantic: "none" };
-        }
-        const hasBlocking = diagnostics.some((d) => d.semantic === "blocking");
-        return {
-            status: hasBlocking ? "failed" : "succeeded",
+        return finishParsedRun({
+            tool: "yamllint",
+            ctx,
+            result,
             diagnostics,
-            semantic: hasBlocking ? "blocking" : "warning",
-        };
+        });
     },
 };
 export default yamllintRunner;

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

@@ -48,7 +48,7 @@ const zigCheckRunner = {
     skipTestFiles: false,
     async run(ctx) {
         const cwd = ctx.cwd || process.cwd();
-        if (!(await (zig.isAvailableAsync(cwd)))) {
+        if (!(await zig.isAvailableAsync(cwd))) {
             return { status: "skipped", diagnostics: [], semantic: "none" };
         }
         const cmd = zig.getCommand(cwd);

package/node_modules/pi-lens/dist/clients/dispatch/suppress-writer.js

@@ -56,7 +56,8 @@ export function insertSuppressComment(content, filePath, line, rule) {
             .filter(Boolean);
         if (!rules.includes(rule))
             rules.push(rule);
-        lines[aboveIdx] = existingAbove.slice(0, match.index) + match[1] + rules.join(", ");
+        lines[aboveIdx] =
+            existingAbove.slice(0, match.index) + match[1] + rules.join(", ");
         return lines.join("\n");
     }
     // Match the indentation of the flagged line so the inserted comment lines

package/node_modules/pi-lens/dist/clients/dispatch/types.js

@@ -10,4 +10,9 @@
  *
  * The dispatcher must handle these semantics consistently.
  */
-export {};
+/** Closed telemetry taxonomy for expected runner skips. */
+export const RUNNER_SKIP_REASONS = ["no-files-matched"];
+/** Runtime guard for untyped/plugin-provided runner results. */
+export function isRunnerSkipReason(value) {
+    return RUNNER_SKIP_REASONS.some((reason) => reason === value);
+}

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

@@ -3,9 +3,13 @@ export function convertLspDiagnostics(diags, filePath, options = {}) {
     return diags
         .filter((d) => d.range?.start?.line !== undefined)
         .map((d, idx) => {
-        const severityMap = { 1: "error", 2: "warning", 4: "hint" };
+        const severityMap = {
+            1: "error",
+            2: "warning",
+            4: "hint",
+        };
         const severity = severityMap[d.severity] ?? "info";
-        const semantic = d.severity === 1 ? "blocking" : (d.severity === 2 ? "warning" : "none");
+        const semantic = d.severity === 1 ? "blocking" : d.severity === 2 ? "warning" : "none";
         const code = String(d.code ?? "unknown");
         // #692: identity ALWAYS derives from the diagnostic's own source — never
         // from a caller-supplied scan label (see `scanOrigin`'s doc comment above).

package/node_modules/pi-lens/dist/clients/extension-log.js

@@ -137,7 +137,7 @@ let moduleLoadWindowOpen = false;
  * the same as an empty store, so it reads as "no window" rather than throwing.
  */
 export function isConsoleCaptureActive() {
-    return (moduleLoadWindowOpen || consoleCaptureStorage?.getStore() === true);
+    return moduleLoadWindowOpen || consoleCaptureStorage?.getStore() === true;
 }
 /**
  * Open the module-evaluation window. Called by `clients/console-guard-install.ts`
@@ -207,7 +207,7 @@ function inCaptureWindow(fn) {
  * the terminal.
  */
 function isCaptureSeam(prop) {
-    return prop === "on" || (typeof prop === "string" && prop.startsWith("register"));
+    return (prop === "on" || (typeof prop === "string" && prop.startsWith("register")));
 }
 /**
  * Wrap every function value found in `value`, in place. Handles the two
@@ -312,7 +312,9 @@ export function withConsoleCaptureWindows(api) {
             // value instead of tripping that invariant (S2a) — a frozen host API
             // loses the capture window for that member, but keeps working.
             const descriptor = Reflect.getOwnPropertyDescriptor(target, prop);
-            if (descriptor && descriptor.writable === false && descriptor.configurable === false) {
+            if (descriptor &&
+                descriptor.writable === false &&
+                descriptor.configurable === false) {
                 return descriptor.value;
             }
             const cached = wrapperCache.get(prop);

package/node_modules/pi-lens/dist/clients/feature-hints.js

@@ -81,7 +81,14 @@ export function inferTrustBoundaries(nameOrPath) {
         boundaries.add("user-input");
         boundaries.add("process-exec");
     }
-    if (hasHintToken(normalized, ["auth", "login", "token", "session", "oauth", "jwt"])) {
+    if (hasHintToken(normalized, [
+        "auth",
+        "login",
+        "token",
+        "session",
+        "oauth",
+        "jwt",
+    ])) {
         boundaries.add("auth");
         boundaries.add("user-input");
     }

package/node_modules/pi-lens/dist/clients/file-kinds.js

@@ -7,24 +7,10 @@
 import { basename, extname } from "node:path";
 // --- Extension Maps ---
 export const KIND_EXTENSIONS = {
-    clojure: [
-        ".clj",
-        ".cljc",
-        ".cljs",
-        ".edn",
-    ],
-    cmake: [
-        ".cmake",
-    ],
-    csharp: [
-        ".cs",
-    ],
-    css: [
-        ".css",
-        ".less",
-        ".sass",
-        ".scss",
-    ],
+    clojure: [".clj", ".cljc", ".cljs", ".edn"],
+    cmake: [".cmake"],
+    csharp: [".cs"],
+    css: [".css", ".less", ".sass", ".scss"],
     cue: [".cue"],
     // From llvm-project/clang/lib/Driver/Types.cpp clang::driver::types::lookupTypeForExtension:
     cxx: [
@@ -61,49 +47,18 @@ export const KIND_EXTENSIONS = {
         ".cl",
         ".clcpp",
     ],
-    dart: [
-        ".dart",
-    ],
-    docker: [
-        ".dockerfile",
-    ],
-    elixir: [
-        ".ex",
-        ".exs",
-    ],
-    fish: [
-        ".fish",
-    ],
-    fsharp: [
-        ".fs",
-        ".fsi",
-        ".fsx",
-    ],
-    gleam: [
-        ".gleam",
-    ],
-    go: [
-        ".go",
-    ],
-    haskell: [
-        ".hs",
-        ".lhs",
-    ],
-    "helm-template": [
-        ".tpl",
-    ],
-    html: [
-        ".htm",
-        ".html",
-    ],
-    java: [
-        ".java",
-    ],
-    json: [
-        ".json",
-        ".json5",
-        ".jsonc",
-    ],
+    dart: [".dart"],
+    docker: [".dockerfile"],
+    elixir: [".ex", ".exs"],
+    fish: [".fish"],
+    fsharp: [".fs", ".fsi", ".fsx"],
+    gleam: [".gleam"],
+    go: [".go"],
+    haskell: [".hs", ".lhs"],
+    "helm-template": [".tpl"],
+    html: [".htm", ".html"],
+    java: [".java"],
+    json: [".json", ".json5", ".jsonc"],
     jsts: [
         ".cjs",
         ".cts",
@@ -116,75 +71,25 @@ export const KIND_EXTENSIONS = {
         ".tsx",
         ".vue",
     ],
-    kotlin: [
-        ".kt",
-        ".kts",
-    ],
-    lua: [
-        ".lua",
-    ],
-    markdown: [
-        ".md",
-        ".mdx",
-    ],
-    nix: [
-        ".nix",
-    ],
-    ocaml: [
-        ".ml",
-        ".mli",
-    ],
-    php: [
-        ".php",
-    ],
-    powershell: [
-        ".ps1",
-        ".psm1",
-        ".psd1",
-    ],
-    prisma: [
-        ".prisma",
-    ],
-    python: [
-        ".py",
-        ".pyi",
-    ],
-    ruby: [
-        ".gemspec",
-        ".rake",
-        ".rb",
-        ".ru",
-    ],
-    rust: [
-        ".rs",
-    ],
-    shell: [
-        ".bash",
-        ".sh",
-        ".zsh",
-    ],
-    sql: [
-        ".sql",
-    ],
-    swift: [
-        ".swift",
-    ],
-    terraform: [
-        ".tf",
-        ".tfvars",
-    ],
+    kotlin: [".kt", ".kts"],
+    lua: [".lua"],
+    markdown: [".md", ".mdx"],
+    nix: [".nix"],
+    ocaml: [".ml", ".mli"],
+    php: [".php"],
+    powershell: [".ps1", ".psm1", ".psd1"],
+    prisma: [".prisma"],
+    python: [".py", ".pyi"],
+    ruby: [".gemspec", ".rake", ".rb", ".ru"],
+    rust: [".rs"],
+    shell: [".bash", ".sh", ".zsh"],
+    sql: [".sql"],
+    swift: [".swift"],
+    terraform: [".tf", ".tfvars"],
     terragrunt: [],
-    toml: [
-        ".toml",
-    ],
-    yaml: [
-        ".yaml",
-        ".yml",
-    ],
-    zig: [
-        ".zig",
-        ".zon",
-    ],
+    toml: [".toml"],
+    yaml: [".yaml", ".yml"],
+    zig: [".zig", ".zon"],
 };
 /** Return whether a path has an extension registered for the given file kind. */
 export function hasKindExtension(filePath, kind) {

package/node_modules/pi-lens/dist/clients/file-utils.js

@@ -5,7 +5,8 @@ import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
 import { minimatch } from "./deps/minimatch.js";
-import { collectTrackedFiles, getTrackedFilesSnapshot } from "./git-tracked-ignore.js";
+import { isInSpawnTimeoutCooldown, noteSpawnTimeout, } from "./spawn-timeout-cooldown.js";
+import { collectTrackedFiles, getTrackedFilesSnapshot, } from "./git-tracked-ignore.js";
 import { getGlobalIgnorePatterns, getPiLensGlobalConfigPath, } from "./lens-config.js";
 import { normalizeEphemeralMapKey, normalizeFilePath } from "./path-utils.js";
 import { findPiLensConfigInDir, findPiLensProjectConfig, loadPiLensConfigInDir, loadPiLensProjectConfig, } from "./project-lens-config.js";
@@ -577,12 +578,27 @@ export async function detectFileChangedAfterCommand(filePath, command, args, cwd
     catch {
         return 0;
     }
+    // #1995: a command cooling down after a spawn timeout must not be handed
+    // a second budget - autofix is one of the lanes that paid twice for a
+    // wedged .cmd shim. Return "no fix applied" (an honest not-checked).
+    if (isInSpawnTimeoutCooldown(command))
+        return 0;
     const result = await safeSpawnAsync(command, args, {
         timeout: 30000,
         cwd,
     });
     if (result.error)
         return 0;
+    if (result.failure === "timeout") {
+        noteSpawnTimeout({
+            tool: path.basename(command).replace(/\.[^.]+$/, ""),
+            command,
+            phase: "autofix",
+            durationMs: 30000,
+            teardown: result.timeoutTeardown,
+        });
+        return 0;
+    }
     if (result.status !== 0 && !ignoreStatuses.includes(result.status ?? -1)) {
         return 0;
     }