@herbertgao/pi-extensions 2026.9.0 → 2026.9.2

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

@@ -35,7 +35,6 @@ const hadolintRunner = {
     id: "hadolint",
     appliesTo: ["docker"],
     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/helm-lint.js

@@ -144,7 +144,6 @@ const helmLintRunner = {
     id: "helm-lint",
     appliesTo: ["yaml", "helm-template"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     timeoutMs: 35_000,
     async run(ctx) {

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

@@ -489,7 +489,7 @@ const OUR_RENDER_FLAGS = ["--output-dir"];
  *
  * Returns the matched flag (for the message) or null.
  */
-export function rejectedOurInvocation(output) {
+function rejectedOurInvocation(output) {
     if (!output)
         return null;
     const complaint = /unknown flag|unknown shorthand flag|flag provided but not defined|unknown command|unrecognized (?:flag|option)/i.test(output);
@@ -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();
@@ -91,6 +86,7 @@ const oxlintRunner = {
         }
         // Run oxlint (or Vite+'s vp lint wrapper) on the file.
         const result = await safeSpawnAsync(cmd, args, {
+            cwd,
             timeout: 30000,
             maxOutputBytes: MAX_OXLINT_OUTPUT_BYTES,
         });

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

@@ -26,8 +26,10 @@ const PS_TIMEOUT_MS = 30000;
  * `spawnFailure.kind` and the errno on `error` — which is what lets the
  * availability policy tell a stall from an absence.
  */
-function spawnPs(cmd, args, timeoutMs = PS_TIMEOUT_MS) {
+function spawnPs(cmd, args, options = {}) {
+    const { timeoutMs = PS_TIMEOUT_MS, cwd } = options;
     return safeSpawnAsync(cmd, args, {
+        cwd,
         timeout: timeoutMs,
         resourceLabel: "psscriptanalyzer",
     });
@@ -124,6 +126,7 @@ async function resolvePowerShellCmd() {
     for (const candidate of ["pwsh", "powershell"]) {
         const sampler = startHostStallSampler();
         const startedAt = Date.now();
+        // cwd-exempt: global interpreter-presence probe ("is pwsh/powershell on PATH at all"), not tied to any project
         const result = await spawnPs(candidate, [
             "-NoProfile",
             "-NonInteractive",
@@ -175,6 +178,7 @@ async function checkModuleAvailable(cmd) {
         return memo;
     const sampler = startHostStallSampler();
     const startedAt = Date.now();
+    // cwd-exempt: global module-presence probe -- PSScriptAnalyzer resolves from PowerShell's module search path, not a per-project install
     const result = await spawnPs(cmd, [
         "-NoProfile",
         "-NonInteractive",
@@ -269,7 +273,6 @@ const psScriptAnalyzerRunner = {
     id: "psscriptanalyzer",
     appliesTo: ["powershell"],
     priority: PRIORITY.GENERAL_ANALYSIS,
-    enabledByDefault: true,
     skipTestFiles: false,
     async run(ctx) {
         const cmd = await resolvePowerShellCmd();
@@ -304,7 +307,7 @@ const psScriptAnalyzerRunner = {
                 tmpScript,
                 "-FilePath",
                 absPath,
-            ]);
+            ], { timeoutMs: PS_TIMEOUT_MS, cwd });
             const hostStallMs = sampler.stop();
             const elapsedMs = Date.now() - startedAt;
             // The two probes above use `-Command`, which PowerShell's execution

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);
@@ -64,7 +63,7 @@ const ruffRunner = {
         // builder, so the package-owned fallback config can never drift.
         const configArgs = ruffConfigArgs(cwd);
         // Step 1: Capture diagnostics (before fixing) — teaching signal for the agent
-        const checkResult = await safeSpawnAsync(cmd, ["check", "--output-format", "json", ...configArgs, ctx.filePath], { timeout: 30000 });
+        const checkResult = await safeSpawnAsync(cmd, ["check", "--output-format", "json", ...configArgs, ctx.filePath], { cwd, timeout: 30000 });
         const raw = stripAnsi(checkResult.stdout + checkResult.stderr);
         const diagnostics = parseRuffJson(checkResult.stdout || "", ctx.filePath);
         const parsedDiagnostics = diagnostics.length > 0 ? diagnostics : parseRuffOutput(raw, ctx.filePath);

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();
@@ -150,7 +149,7 @@ const shellcheckRunner = {
             args.push("--severity", "info");
         }
         args.push(ctx.filePath);
-        const result = await safeSpawnAsync(cmd, args, { timeout: 15000 });
+        const result = await safeSpawnAsync(cmd, args, { cwd, timeout: 15000 });
         // shellcheck exits with code 1 if issues found, 0 if clean
         if (result.status === 0 && !result.stdout?.trim()) {
             return { status: "succeeded", diagnostics: [], semantic: "none" };

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,18 +80,19 @@ const spellcheckRunner = {
     id: "spellcheck",
     appliesTo: ["markdown"],
     priority: PRIORITY.DOC_QUALITY,
-    enabledByDefault: true,
     skipTestFiles: false, // Check docs in test files too
     async run(ctx) {
+        const cwd = ctx.cwd || process.cwd();
         // Skip if typos-cli is not installed
-        if (!(await typos.isAvailableAsync(ctx.cwd || process.cwd()))) {
+        if (!(await typos.isAvailableAsync(cwd))) {
             return { status: "skipped", diagnostics: [], semantic: "none" };
         }
         // Run typos-cli with JSON output
         // --format json: Output JSON Lines
         // --exclude <pattern>: Could be used to exclude code blocks if needed
         const args = ["--format", "json", ctx.filePath];
-        const result = await safeSpawnAsync(typos.getCommand(ctx.cwd || process.cwd()), args, {
+        const result = await safeSpawnAsync(typos.getCommand(cwd), args, {
+            cwd,
             timeout: 15000,
         });
         // #1816: typos-cli exits 0 clean, 2 with typos found, and 1 on an ERROR

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,16 +4,17 @@
  * 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 { getAutofixCapability } from "../../../tool-policy.js";
 /**
  * Create a parser for line-based tool output.
  * Common format: file:line:col: message (with variations)
  */
-export function createLineParser(config) {
+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);
@@ -79,71 +80,6 @@ export const parseGoVetOutput = createLineParser({
     generateId: (m) => `go-vet-${m[2]}`,
     defectClass: "correctness",
 });
-/**
- * Parse Biome output: file:line:col message (category)
- * With autofix support for fix suggestions
- */
-export function createBiomeParser(_autofix = false) {
-    const biomeAutofix = getAutofixCapability("biome");
-    return createLineParser({
-        tool: "biome",
-        regex: /^(.+?):(\d+):(\d+)\s+(.+?)\s*\((.+?)\)/,
-        extractMessage: (m) => `${m[5]}: ${m[4]}`, // category: message
-        extractRule: (m) => m[5],
-        generateId: (m) => `biome-${m[2]}-${m[5]}`,
-        getSeverity: (line) => (line.includes("error") ? "error" : "warning"),
-        fixable: true,
-        autoFixAvailable: biomeAutofix?.safePipelineAutofix ?? false,
-        fixKind: biomeAutofix?.fixKind === "none" ? undefined : biomeAutofix?.fixKind,
-    });
-}
-// Backward-compatible default biome parser
-export const parseBiomeOutput = createBiomeParser(false);
 // =============================================================================
 // GENERIC PARSER FACTORY
 // =============================================================================
-/**
- * Create a simple parser for tools using standard file:line:col format.
- * Format variations: :line:col:, line:col, (line,col), etc.
- */
-export function createSimpleParser(tool, options = {}) {
-    const sep = options.separator ?? ":";
-    const severity = options.severity ?? "warning";
-    const fixable = options.fixable ?? false;
-    // Build regex based on separator type
-    const escapedSep = sep === " " ? "\\s+" : escapeRegExp(sep);
-    const regex = options.includesFileName
-        ? new RegExp(`^(.+?)${escapedSep}(\\d+)${escapedSep}(\\d+)${escapedSep}(.+)`)
-        : new RegExp(`^(\\d+)${escapedSep}(\\d+)${escapedSep}(.+)`);
-    return (raw, filePath) => {
-        const diagnostics = [];
-        const lines = raw.split("\n").filter((l) => l.trim());
-        for (const line of lines) {
-            const match = line.match(regex);
-            if (!match)
-                continue;
-            const lineNum = options.includesFileName
-                ? parseInt(match[2], 10)
-                : parseInt(match[1], 10);
-            const colNum = options.includesFileName
-                ? parseInt(match[3], 10)
-                : parseInt(match[2], 10);
-            const message = options.includesFileName ? match[4] : match[3];
-            diagnostics.push({
-                id: `${tool}-${lineNum}`,
-                message: message.trim(),
-                filePath,
-                line: lineNum,
-                column: colNum,
-                severity,
-                semantic: severity === "error" ? "blocking" : "warning",
-                tool,
-                fixable,
-            });
-        }
-        return diagnostics;
-    };
-}
-function escapeRegExp(string) {
-    return string.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-}