pi-lens 4.3.0 → 4.4.0

package/dist/clients/file-utils.js

@@ -1,1032 +0,0 @@
-/**
- * Shared file path utilities for pi-lens
- */
-import * as fs from "node:fs";
-import { createHash } from "node:crypto";
-import * as os from "node:os";
-import * as path from "node:path";
-import { Minimatch } from "./deps/minimatch.js";
-import { detectFileRole } from "./file-role.js";
-import { FRESHNESS_CADENCE_MS } from "./freshness-cadence.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 { homeRelativePath, isExternalOrVendorFile, isUnderDir, normalizeEphemeralMapKey, normalizeFilePath, toProjectRelativePath, } from "./path-utils.js";
-import { findPiLensConfigInDir, findPiLensProjectConfig, loadPiLensConfigInDir, loadPiLensProjectConfig, } from "./project-lens-config.js";
-import { safeSpawnAsync } from "./safe-spawn.js";
-/**
- * Return the directory where pi-lens stores project-specific data
- * (caches, indexes, worklogs, etc.).
- *
- * Default: reuse <project>/.pi-lens if it already exists, otherwise use
- * ~/.pi-lens/projects/<project-slug>
- *
- * Override: set PILENS_DATA_DIR=/some/path — each project gets its own
- * subdirectory named after a sanitized form of its absolute path, e.g.
- *   PILENS_DATA_DIR=~/.pi-lens/projects
- *   → ~/.pi-lens/projects/home-user-myapp-<8-hex-hash>/
- *
- * This keeps project folders clean and avoids creating .pi-lens folders
- * inside user projects.
- */
-export function getProjectDataDir(cwd) {
-    const legacyProjectDir = path.join(cwd, ".pi-lens");
-    const configuredBase = process.env.PILENS_DATA_DIR?.trim();
-    if (!configuredBase && fs.existsSync(legacyProjectDir)) {
-        return legacyProjectDir;
-    }
-    const base = configuredBase || path.join(getGlobalPiLensDir(), "projects");
-    const resolvedBase = base.trim();
-    const memoKey = `${resolvedBase}\0${path.resolve(cwd)}`;
-    const cached = settledDataDirs.get(memoKey);
-    if (cached !== undefined)
-        return cached;
-    const canonical = canonicalProjectRoot(cwd);
-    const readable = projectDataDirReadableSlug(canonical.root);
-    const hash = projectDataDirRootHash(canonical.root);
-    const slug = `${readable || "default"}-${hash}`;
-    const dir = path.join(resolvedBase, slug);
-    const settled = settleProjectDataDir(resolvedBase, readable || "default", dir, memoKey, canonical.fallback);
-    return settled;
-}
-/**
- * The human-readable half of the project data-dir slug: the absolute root
- * with separators folded to dashes and anything else stripped.
- *
- * #2874: this form ALONE is not an identity — `src/pi-lens` and
- * `src/pi/lens` fold to the same string. It is only ever combined with
- * {@link projectDataDirRootHash}. Never use it as a directory name alone.
- */
-function projectDataDirReadableSlug(canonicalRoot) {
-    const normalized = normalizeFilePath(canonicalRoot);
-    return normalized
-        .replace(/^[a-z]:/i, "") // strip Windows drive letter
-        .replace(/\/+/g, "-") // separators → dashes
-        .replace(/[^A-Za-z0-9-]/g, "") // strip anything else
-        .replace(/^-+/, "") // trim leading dashes
-        .replace(/-+$/, ""); // trim trailing dashes
-}
-/**
- * The canonical root is the resolved absolute path for both slug halves.
- * `realpathSync` is probed only to expose a fallback boundary failure; using
- * its result would let one root spelling change identity when the boundary
- * recovers. The hash is the first 8 hex chars of SHA-256 over that root. A
- * collision needs the same readable slug and the same 32-bit prefix, so the
- * practical collision population is readable-slug twin pairs. Pinned on
- * purpose because changing it renames every directory.
- */
-function canonicalProjectRoot(cwd) {
-    const resolved = path.resolve(cwd);
-    try {
-        // Probe the root, but keep the resolved spelling as the identity in both
-        // success and fallback cases. Resolving only on success would let a
-        // transient boundary failure choose a second data directory.
-        fs.realpathSync(resolved);
-    }
-    catch {
-        // Best-effort: the root may not exist yet.
-        return { root: resolved, fallback: true };
-    }
-    return { root: resolved, fallback: false };
-}
-function projectDataDirRootHash(canonicalRoot) {
-    return createHash("sha256").update(canonicalRoot).digest("hex").slice(0, 8);
-}
-const pendingDataDirMigrations = [];
-const settledDataDirs = new Map();
-export function resetProjectDataDirSessionState() {
-    pendingDataDirMigrations.splice(0);
-}
-/**
- * One-time upgrade from a pre-#2874 slug directory to its hashed name.
- * When the old directory exists and the new one does not, the state moves
- * with one atomic `renameSync` (same parent, so same filesystem). When both
- * exist the new one wins and nothing moves. Either case queues one
- * {@link ProjectDataDirMigration} for the session-start drain, which emits
- * the `data_dir_migrated` record. Best-effort throughout: on any failure the
- * caller gets the pre-existing directory and behavior is unchanged.
- *
- * No ledger import here on purpose: `probe-home-state.ts` documents that a
- * direct edge between this module and `degradation-ledger.ts` in either
- * direction adds `no-client-cycles` violations, so the record is emitted by
- * the drain site in `runtime-session.ts` instead.
- */
-function settleProjectDataDir(base, oldSlug, dir, memoKey, identityFallback) {
-    const oldDir = path.join(base, oldSlug);
-    const newExists = fs.existsSync(dir);
-    const oldExists = fs.existsSync(oldDir);
-    if (!oldExists) {
-        if (identityFallback && pendingDataDirMigrations.length < 32) {
-            pendingDataDirMigrations.push({
-                to: dir,
-                outcome: "identity-fallback",
-            });
-        }
-        // Steady state: nothing to migrate from.
-        settledDataDirs.set(memoKey, dir);
-        return dir;
-    }
-    if (!newExists) {
-        try {
-            fs.renameSync(oldDir, dir);
-        }
-        catch (error) {
-            if (error.code === "ENOENT") {
-                if (fs.existsSync(dir)) {
-                    settledDataDirs.set(memoKey, dir);
-                    return dir;
-                }
-                if (fs.existsSync(oldDir))
-                    return oldDir;
-                return dir;
-            }
-            // Best-effort: retain the old directory and publish one bounded
-            // failure notice for this process's memo key.
-            if (pendingDataDirMigrations.length < 32) {
-                pendingDataDirMigrations.push({
-                    to: dir,
-                    outcome: "rename-failed",
-                });
-            }
-            settledDataDirs.set(memoKey, oldDir);
-            return oldDir;
-        }
-    }
-    // Either the rename just moved the state, or both directories already
-    // existed and the new one wins. Queue one migration for the drain.
-    settledDataDirs.set(memoKey, dir);
-    if (pendingDataDirMigrations.length < 32) {
-        pendingDataDirMigrations.push({
-            to: dir,
-            outcome: newExists ? "coexisting" : "renamed",
-        });
-    }
-    return dir;
-}
-/**
- * Drain the queued {@link ProjectDataDirMigration}s, clearing the queue.
- * Called once per session start by `handleSessionStart`; each queued outcome is
- * therefore recorded at most once per session. Test-only callers must not rely
- * on queue depth across drains.
- */
-export function drainProjectDataDirMigrations() {
-    return pendingDataDirMigrations.splice(0);
-}
-/**
- * Test-only reset for the settled-directory memo. The memo is a pure
- * performance cache (one `existsSync` pair saved per latched directory), so
- * clearing it changes no observable behavior — it lets a test stage an
- * old-slug directory AFTER learning the new name, then resolve again.
- */
-export function _resetProjectDataDirMemoForTests() {
-    settledDataDirs.clear();
-}
-/**
- * The project-data path to SHOW an agent or a human, for a file that
- * {@link getProjectDataDir} owns.
- *
- * #2521: the turn-end actionable-warnings advisory told the agent to read a
- * hardcoded `.pi-lens/cache/actionable-warnings.json`, but that spelling is
- * only correct in the LEGACY arm of `getProjectDataDir` — a project with no
- * `.pi-lens/` directory (the common case) stores the report under
- * `~/.pi-lens/projects/<slug>/cache/`, and a `PILENS_DATA_DIR` project stores
- * it somewhere else again. Agents followed the advisory literally and got
- * `cat: .pi-lens/cache/actionable-warnings.json: No such file or directory`.
- *
- * Any instructional string that NAMES a project-data location must be built
- * here, so the displayed path and the written path can never disagree. Guarded
- * by `tests/clients/data-dir-display-path-sweep.test.ts`.
- *
- * Two output shapes, matching `toRunnerDisplayPath`'s convention
- * (`clients/dispatch/runner-context.ts`) so one advisory never mixes styles:
- *   - under `cwd` → a `/`-separated path relative to it
- *     (`.pi-lens/cache/actionable-warnings.json`), which is what the legacy
- *     arm produces and what an agent can paste straight into a shell.
- *   - anywhere else → absolute, `/`-separated, and `~`-folded via
- *     {@link homeRelativePath} so the default `~/.pi-lens/projects/...` store
- *     does not leak the account name into agent context (#2440 F5).
- *
- * Pure string work on top of `getProjectDataDir`'s own resolution: it never
- * stats the target, so it is safe to call for a file that has not been written
- * yet and cannot throw on a surface whose whole job is to render a hint.
- *
- * The relative-or-absolute decision reuses `toProjectRelativePath`
- * (`clients/path-utils.ts`) — the same shape-aware, non-stat'ing primitive
- * `toRunnerDisplayPath` (`clients/dispatch/runner-context.ts`) composes for
- * the analogous "relative path if under this root, else something else"
- * decision — rather than a second hand-rolled `path.relative` fold. It is NOT
- * `toRunnerDisplayPath` itself: that helper routes its result through
- * `normalizeMapKey`, which on win32 calls `realpathSync.native()` (or, for a
- * not-yet-existing path, lowercases the unresolved tail) — exactly the
- * stat-and-mutate-casing behavior this function's contract above rules out.
- */
-export function displayProjectDataPath(cwd, ...segments) {
-    const absolute = path.resolve(getProjectDataDir(cwd), ...segments);
-    const absolutePosix = absolute.replace(/\\/g, "/");
-    const relative = toProjectRelativePath(absolute, cwd);
-    if (relative !== absolutePosix) {
-        return relative;
-    }
-    // `toProjectRelativePath` fell back to the (posix-ified) absolute path —
-    // `absolute` is not under `cwd`. Fold `~` so the default
-    // `~/.pi-lens/projects/...` store does not leak the account name into
-    // agent context (#2440 F5). Posix-ify BEFORE the fold: `homeRelativePath`
-    // deliberately returns a path that is not under `$HOME` unchanged,
-    // separators included, so a `PILENS_DATA_DIR=D:\lens-data` store would
-    // otherwise render with backslashes while the legacy arm above renders
-    // with forward ones — `absolutePosix` is already folded, so this is a
-    // no-op fold, not a second one.
-    return homeRelativePath(absolutePosix);
-}
-/**
- * Machine-global pi-lens directory: `~/.pi-lens/` by default.
- *
- * Used for tool binaries (`~/.pi-lens/tools/`, `~/.pi-lens/bin/`), the
- * cross-process instance registry (`instances.json`, #449/#525) and the
- * orphan-backstop lease beside it, the auto-install probe cache, the
- * canonical global config (`config.json`), LSP server storage (intelephense's
- * index, PSES's per-PID session dir) and JVM runtimes — state that is
- * intentionally NOT project-scoped, because it spans every project pi-lens
- * has touched.
- *
- * Override: set `PI_LENS_HOME=/some/path` to relocate this ENTIRE root (every
- * caller routes through this one function, so one env var covers all of
- * them — see #525). Tests MUST set this to a per-worker temp dir in
- * `tests/support/vitest-setup.ts` rather than mocking each caller separately;
- * otherwise a test that exercises `registerInstance`/`sweepOrphans` or any
- * logger writes into the developer's REAL `~/.pi-lens` (dogfooded live: a
- * test-fixture instance survived in the real `instances.json` for 17h).
- *
- * DELIBERATELY cwd-INDEPENDENT (#2506 round 3). Its sibling
- * `getGlobalPiLensLogDir()` (`probe-home-state.ts`) carries the probe-home
- * redirect; this one must not, because the things resolved here are shared
- * machine state a session needs to KEEP: a pi session running from an agent
- * worktree or a throwaway temp project must still find the tools it already
- * installed and must still register in the one `instances.json` every other
- * pi-lens process on the box reads. Redirecting THIS function (as round 2 of
- * #2506 did) would silently give every worktree its own empty tool tree and
- * its own private registry, so the reaper could never see across them — a
- * worse defect than the telemetry pollution the redirect exists to stop.
- *
- * Distinct from `getProjectDataDir(cwd)`, which respects `PILENS_DATA_DIR`
- * (project-scoped) and produces per-project subdirectories. Callers writing
- * project caches, snapshots, or worklogs should use `getProjectDataDir(cwd)`
- * instead — `PI_LENS_HOME` is the MACHINE-scoped sibling of that override.
- * `getProjectDataDir` composes through THIS function, not the log one: a
- * probe's project caches are already isolated by its own path slug, and a
- * worktree session keeping its warm caches is the same "do not break the
- * session" argument as the tools above.
- */
-export function getGlobalPiLensDir() {
-    const override = process.env.PI_LENS_HOME?.trim();
-    if (override)
-        return path.resolve(override);
-    return path.join(os.homedir(), ".pi-lens");
-}
-/**
- * Directories to exclude from all scans (build outputs, dependencies, caches).
- * Used consistently across all scanners to avoid noise from generated files.
- */
-export const EXCLUDED_DIRS = [
-    "node_modules",
-    ".git",
-    "dist",
-    "build",
-    ".turbo",
-    ".cache",
-    "target",
-    "out",
-    ".parcel-cache",
-    ".svelte-kit",
-    ".nuxt",
-    ".yarn",
-    ".pnpm-store",
-    ".gradle",
-    ".next",
-    ".pi-lens",
-    ".pi", // pi agent directory
-    // The SAME agent, rebranded: pi derives its config-dir name as
-    // `pkg.piConfig?.configDir || ".pi"` (pi-coding-agent 0.85.1,
-    // `dist/bundle/chunks/chunk-JVUZSMYM.js`), so a redistribution spells its
-    // `<configDir>/agent/sessions` history `.omp/agent/sessions` instead.
-    // #3112: a project-local one was walked entry-by-entry and spent the whole
-    // startup-scan entry budget, so a five-file project reported
-    // `too-many-entries` and never warmed its caches.
-    ".omp",
-    ".ruff_cache", // Python linter cache
-    ".worktrees",
-    ".claude",
-    ".codex",
-    ".rescue",
-    ".agents",
-    ".gstack",
-    ".superpowers",
-    ".guardrails",
-    ".playwright-cli",
-    ".playwright-mcp",
-    ".vscode",
-    "venv",
-    ".venv",
-    "coverage",
-    "__pycache__",
-    ".tox",
-    ".pytest_cache",
-    "*.dSYM",
-    // Vendored upstream source conventions — universally too large to scan
-    "vendor", // Go modules, PHP Composer, Ruby Bundler
-    "third_party", // Chromium/Google convention (llama.cpp, sherpa-onnx, gRPC, TF)
-    "third-party",
-    "vendors",
-];
-function resolveGitIgnoreRoot(startDir) {
-    const fallback = path.resolve(startDir);
-    let current = fallback;
-    while (true) {
-        if (fs.existsSync(path.join(current, ".git")))
-            return current;
-        const parent = path.dirname(current);
-        if (parent === current)
-            return fallback;
-        current = parent;
-    }
-}
-function collapseSlashes(value) {
-    let out = "";
-    let previousWasSlash = false;
-    for (const ch of value) {
-        if (ch === "/") {
-            if (!previousWasSlash)
-                out += ch;
-            previousWasSlash = true;
-            continue;
-        }
-        out += ch === "\\" ? "/" : ch;
-        previousWasSlash = false;
-    }
-    return out;
-}
-function stripLeadingDotSlash(value) {
-    return value.startsWith("./") ? value.slice(2) : value;
-}
-function stripTrailingSlashes(value) {
-    let end = value.length;
-    while (end > 0 && value[end - 1] === "/")
-        end -= 1;
-    return value.slice(0, end);
-}
-function stripLeadingSlashes(value) {
-    let start = 0;
-    while (start < value.length && value[start] === "/")
-        start += 1;
-    return value.slice(start);
-}
-function normalizeIgnorePath(value) {
-    return collapseSlashes(stripLeadingDotSlash(value));
-}
-function stripTrailingSpaces(value) {
-    // Good-enough gitignore whitespace handling: unescaped trailing spaces are ignored.
-    let end = value.length;
-    while (end > 0 && value[end - 1] === " " && value[end - 2] !== "\\")
-        end -= 1;
-    return value.slice(0, end).replace(/\\ /g, " ");
-}
-function parseGitignoreContent(content, layer) {
-    const patterns = [];
-    for (const rawLine of content.split(/\r?\n/)) {
-        let line = stripTrailingSpaces(rawLine.trimStart());
-        if (!line || line.startsWith("#"))
-            continue;
-        let negated = false;
-        if (line.startsWith("!")) {
-            negated = true;
-            line = line.slice(1);
-        }
-        line = normalizeIgnorePath(line);
-        if (!line)
-            continue;
-        const directoryOnly = line.endsWith("/");
-        if (directoryOnly)
-            line = stripTrailingSlashes(line);
-        const rooted = line.startsWith("/");
-        if (rooted)
-            line = stripLeadingSlashes(line);
-        if (!line)
-            continue;
-        patterns.push({
-            pattern: line,
-            negated,
-            directoryOnly,
-            rooted,
-            hasSlash: line.includes("/"),
-            layer,
-        });
-    }
-    return patterns;
-}
-function expandGitignorePattern(pattern) {
-    const body = pattern.pattern;
-    if (pattern.directoryOnly) {
-        if (pattern.rooted || pattern.hasSlash)
-            return [body, `${body}/**`];
-        return [body, `${body}/**`, `**/${body}`, `**/${body}/**`];
-    }
-    if (pattern.rooted || pattern.hasSlash)
-        return [body];
-    return [body, `**/${body}`];
-}
-function matchesGitignorePattern(pattern, relativePath, isDirectory, matchExpanded) {
-    const candidate = stripLeadingSlashes(normalizeIgnorePath(relativePath));
-    if (!candidate)
-        return false;
-    const candidates = isDirectory ? [candidate, `${candidate}/`] : [candidate];
-    return expandGitignorePattern(pattern).some((expanded) => {
-        if (isDirectory && expanded.endsWith("/**")) {
-            const prefix = expanded.slice(0, -3);
-            if (candidate === prefix || candidate.startsWith(`${prefix}/`))
-                return true;
-        }
-        return candidates.some((value) => matchExpanded(value, expanded));
-    });
-}
-function readGitignorePatterns(rootDir, layer = "gitignore") {
-    const gitignorePath = path.join(rootDir, ".gitignore");
-    try {
-        return parseGitignoreContent(fs.readFileSync(gitignorePath, "utf-8"), layer);
-    }
-    catch {
-        return [];
-    }
-}
-function ancestorDirsBetween(rootDir, targetDir) {
-    const relative = path.relative(rootDir, targetDir);
-    if (relative.startsWith("..") || path.isAbsolute(relative))
-        return [];
-    const dirs = [rootDir];
-    if (!relative)
-        return dirs;
-    let current = rootDir;
-    for (const segment of relative.split(path.sep).filter(Boolean)) {
-        current = path.join(current, segment);
-        dirs.push(current);
-    }
-    return dirs;
-}
-function buildProjectIgnoreMatcher(resolvedRoot, patterns) {
-    const consumedIgnoreSources = new Map();
-    // `signature` lets a caller that already statted the file reuse that one
-    // stat (#2016's class on this seam: never re-derive a value you just
-    // computed). Callers with no signature in hand still pay their own stat.
-    const rememberIgnoreSource = (filePath, signature = fileFreshnessSignature(filePath)) => {
-        consumedIgnoreSources.set(filePath, { path: filePath, ...signature });
-    };
-    rememberIgnoreSource(path.join(resolvedRoot, ".gitignore"));
-    const nestedCache = new Map();
-    // #783: layer a NESTED `.pi-lens.json`'s `ignore` field the same way a
-    // nested `.gitignore` is already layered — each ancestor directory between
-    // `resolvedRoot` and the target is checked for its OWN config file (no
-    // upward walk; `findPiLensConfigInDir`/`loadPiLensConfigInDir` look only
-    // directly in `dir`), and its `ignore` patterns are anchored to `dir` (same
-    // anchoring semantics as a `.gitignore` living in that directory) and
-    // tagged `"pilens"` so #703's tracked-file-rescue rule still skips them.
-    // `loadPiLensConfigInDir` reuses `project-lens-config.ts`'s own
-    // path+mtime cache, so this never re-reads/re-parses JSON that some other
-    // caller (or the root-config lookup above) already loaded.
-    const patternsForDir = (dir) => {
-        if (dir === resolvedRoot)
-            return patterns;
-        const cached = nestedCache.get(dir);
-        const now = Date.now();
-        // #2071: inside the cadence window this directory's rules are frozen, so
-        // no stat runs and every verdict already memoized under `dir` stays
-        // derivable from exactly these patterns. Before this gate the freshness
-        // check ran per call while `patternMemo` did not, which is what let a
-        // fresh path and a memoized path in the same directory return opposite
-        // verdicts for the same rule. Externally edited nested sources are
-        // therefore picked up within PROJECT_IGNORE_FRESHNESS_CADENCE_MS, the
-        // same bound #2159 already accepted for the root sources.
-        // pi-authored `.gitignore` writes keep their instant path through
-        // `invalidateProjectIgnoreMatcherForPath`.
-        if (cached !== undefined &&
-            now - cached.checkedAtMs < PROJECT_IGNORE_FRESHNESS_CADENCE_MS) {
-            return cached.patterns;
-        }
-        const gitignoreSig = gitignoreSignature(dir);
-        // One stat, two consumers. `rememberIgnoreSource` used to stat the same
-        // file a second time on the line below this one.
-        rememberIgnoreSource(path.join(dir, ".gitignore"), gitignoreSig);
-        // #1105: gate on size too. `findPiLensConfigInDir` returns `size` alongside
-        // `mtimeMs` (both from one stat), and `gitignoreSignature` reads both for
-        // the nested `.gitignore`, so an mtime-preserving, length-changing edit to
-        // EITHER nested source can no longer replay stale patterns for this subtree.
-        const pilensInfo = findPiLensConfigInDir(dir);
-        const pilensMtime = pilensInfo?.mtimeMs ?? -1;
-        const pilensSize = pilensInfo?.size ?? -1;
-        if (cached?.gitignoreMtimeMs === gitignoreSig.mtimeMs &&
-            cached?.gitignoreSize === gitignoreSig.size &&
-            cached?.pilensMtimeMs === pilensMtime &&
-            cached?.pilensSize === pilensSize) {
-            // Unchanged: re-arm the shared clock so the next window is measured
-            // from this check, not from the build that first populated the entry.
-            cached.checkedAtMs = now;
-            return cached.patterns;
-        }
-        const nestedConfig = loadPiLensConfigInDir(dir);
-        const nextPatterns = [
-            ...readGitignorePatterns(dir),
-            ...parseGitignoreContent(nestedConfig.ignore.join("\n"), "pilens"),
-        ];
-        nestedCache.set(dir, {
-            checkedAtMs: now,
-            gitignoreMtimeMs: gitignoreSig.mtimeMs,
-            gitignoreSize: gitignoreSig.size,
-            pilensMtimeMs: pilensMtime,
-            pilensSize: pilensSize,
-            patterns: nextPatterns,
-        });
-        // #2071: drift is the SINGLE trigger. The rules under `dir` just changed,
-        // so every verdict computed under the superseded rules dies in the same
-        // step that supersedes them. `cached === undefined` is a first build, not
-        // a change, and has no verdicts to drop.
-        if (cached !== undefined)
-            dropVerdictsUnder(dir);
-        return nextPatterns;
-    };
-    // Per-matcher path → pattern-verdict memo. The matcher itself is cached by
-    // `getProjectIgnoreMatcher` keyed on `.gitignore` size+mtime (#1105), so this
-    // Map's lifetime is bounded to a single set of ignore rules — when any
-    // `.gitignore` changes, the matcher is rebuilt and the memo is dropped
-    // with it. Without this memo, every background scan (comment scan, knip,
-    // jscpd, call-graph, source-filter, pipeline) recomputes O(ancestorDirs ×
-    // patterns) per file, multiplying into 2-3s of pure CPU on a 2k-file
-    // project. With it, the second visitor of the same path is O(1).
-    //
-    // #703: this memoizes only the PATTERN verdict (which is deterministic —
-    // it never changes for a given matcher instance), NOT the tracked-aware
-    // final verdict. The tracked-files set can transition from "not yet
-    // primed" to "primed" mid-process (a walker calls `ensureTrackedIndex()`
-    // partway through the matcher's lifetime), so baking the tracked check
-    // into this memo would let an early, pre-priming call poison every later
-    // lookup of the same path for this matcher's entire cache lifetime. The
-    // tracked-set lookup itself is a cheap Set#has over syntactically-folded
-    // keys (see `isTrackedAndRescued` — no `realpathSync` anywhere in this
-    // function), and — critically — it's only ever paid for paths a pattern
-    // already flagged as ignored, so it doesn't reintroduce the per-file cost
-    // this memo exists to avoid for the common (not-ignored) case.
-    const patternMemo = new Map();
-    /**
-     * Drop every memoized verdict at or below `subtree`, leaving `nestedCache`
-     * alone. `patternsForDir` calls this the moment it rebuilds a directory's
-     * patterns, which is why it must NOT evict the entry that rebuild just
-     * wrote. `invalidateSubtree` layers the eviction on top for the write-hook
-     * path, where the caller knows the source changed but has not re-read it.
-     */
-    function dropVerdictsUnder(subtree) {
-        const resolvedSubtree = path.resolve(subtree);
-        for (const key of patternMemo.keys()) {
-            const memoPath = key.slice(2);
-            const relative = path.relative(resolvedSubtree, memoPath);
-            if (!relative ||
-                (!relative.startsWith("..") && !path.isAbsolute(relative))) {
-                patternMemo.delete(key);
-            }
-        }
-    }
-    const invalidateSubtree = (subtree) => {
-        dropVerdictsUnder(subtree);
-        nestedCache.delete(path.resolve(subtree));
-    };
-    // Compile expanded gitignore globs once per matcher instance. Relative-path
-    // resolution remains outside this cache, so nested ignore scopes cannot leak
-    // verdicts between sibling trees. The instance lifetime bounds this map.
-    const gitignoreGlobOptions = {
-        dot: true,
-        nocase: process.platform === "win32",
-    };
-    const compiledGlobs = new Map();
-    const matchExpanded = (value, expanded) => {
-        let compiled = compiledGlobs.get(expanded);
-        if (compiled === undefined) {
-            compiled = new Minimatch(expanded, gitignoreGlobOptions);
-            compiledGlobs.set(expanded, compiled);
-        }
-        return compiled.match(value);
-    };
-    // #703 perf follow-up: `normalizeEphemeralMapKey` (cheap slash-fold +
-    // Windows-lowercase, zero fs I/O), NOT `normalizeMapKey` (realpath-backed).
-    // This runs on every `isIgnored` call that reaches this branch — walks
-    // over this repo alone visit thousands of pattern-ignored compiled
-    // `*.js`/`*.d.ts` files, and `dispatch/integration.ts`'s per-edit cascade
-    // check hits it too — so a `realpathSync` here would violate the
-    // event-loop/slow-FS discipline `isIgnored` is required to keep (#462: 75x
-    // slower on 9p/drvfs). Both `resolved` (this matcher's own `path.resolve`,
-    // never realpath'd) and the tracked-set's keys (`git-tracked-ignore.ts`,
-    // realpath'd ONCE per fetch on the shared root, not per file) are
-    // self-consistent within one process/session, which is exactly
-    // `normalizeEphemeralMapKey`'s contract — see that function's doc and
-    // `git-tracked-ignore.ts`'s module doc for the full reasoning. Accepted
-    // edge case: a symlinked or 8.3-short-name project root can make the cheap
-    // fold miss even after the fetch side's one realpath — the file then stays
-    // pattern-ignored, i.e. degrades to today's (pre-#703) behavior, which is
-    // consistent with this whole feature's fail-open contract.
-    function isTrackedAndRescued(resolved) {
-        const snapshot = getTrackedFilesSnapshot(resolvedRoot);
-        if (!snapshot)
-            return false; // never primed / git unavailable: fail-open to pattern-only
-        return snapshot.has(normalizeEphemeralMapKey(resolved));
-    }
-    return {
-        rootDir: resolvedRoot,
-        patterns,
-        getConsumedIgnoreSources: () => [...consumedIgnoreSources.values()],
-        refreshConsumedIgnoreSource(filePath) {
-            // The freshness sweep refreshes this baseline after invalidation.
-            if (consumedIgnoreSources.has(filePath))
-                rememberIgnoreSource(filePath);
-        },
-        invalidateSubtree,
-        ensureTrackedIndex() {
-            return collectTrackedFiles(resolvedRoot).then(() => undefined);
-        },
-        isIgnored(filePath, isDirectory = false) {
-            const resolved = path.resolve(filePath);
-            // Two namespaces (D: for directory queries, F: for file queries)
-            // because gitignore semantics differ for trailing-slash patterns.
-            const memoKey = (isDirectory ? "D:" : "F:") + resolved;
-            let verdict = patternMemo.get(memoKey);
-            if (verdict === undefined) {
-                const rootRelative = path.relative(resolvedRoot, resolved);
-                if (!rootRelative ||
-                    rootRelative.startsWith("..") ||
-                    path.isAbsolute(rootRelative)) {
-                    verdict = { ignored: false, layer: undefined };
-                }
-                else {
-                    let ignored = false;
-                    let layer;
-                    const patternDirs = ancestorDirsBetween(resolvedRoot, path.dirname(resolved));
-                    for (const dir of patternDirs) {
-                        const dirPatterns = patternsForDir(dir);
-                        if (dirPatterns.length === 0)
-                            continue;
-                        const relative = path.relative(dir, resolved);
-                        const normalized = normalizeIgnorePath(relative);
-                        for (const pattern of dirPatterns) {
-                            if (!matchesGitignorePattern(pattern, normalized, isDirectory, matchExpanded))
-                                continue;
-                            ignored = !pattern.negated;
-                            layer = pattern.layer;
-                        }
-                    }
-                    verdict = { ignored, layer };
-                }
-                patternMemo.set(memoKey, verdict);
-            }
-            if (!verdict.ignored)
-                return false;
-            // #703 layer semantics: a winning positive match from `global` or
-            // `gitignore` emulates git, so it inherits git's "a tracked file is
-            // never ignored" rule. A winning match from `pilens` is pi-lens-native
-            // intent and stays excluded regardless of tracked status. Directory
-            // queries are never tracked-rescued — the tracked set is a file-id
-            // set, not a directory set.
-            if (!isDirectory &&
-                verdict.layer !== "pilens" &&
-                isTrackedAndRescued(resolved)) {
-                return false;
-            }
-            return true;
-        },
-    };
-}
-export function createProjectIgnoreMatcher(rootDir, extraPatterns = [], globalPatterns = []) {
-    const resolvedRoot = resolveGitIgnoreRoot(rootDir);
-    // Precedence is gitignore order: LATER patterns override earlier ones. So
-    // global (lowest) → project .gitignore → project .pi-lens.json (highest),
-    // which lets a project `!negation` re-include a globally-ignored path (#252).
-    // Each layer is tagged (#703) so `isIgnored` can tell a git-emulating match
-    // (`global`/`gitignore` — subject to "a tracked file is never ignored")
-    // apart from pi-lens-native intent (`pilens` — excludes regardless).
-    const patterns = [
-        ...parseGitignoreContent(globalPatterns.join("\n"), "global"),
-        ...readGitignorePatterns(resolvedRoot, "gitignore"),
-        ...parseGitignoreContent(extraPatterns.join("\n"), "pilens"),
-    ];
-    return buildProjectIgnoreMatcher(resolvedRoot, patterns);
-}
-const projectIgnoreMatcherCache = new Map();
-/**
- * Nested ignore sources are checked at most once per root per cadence window.
- * Sourced from the shared leaf (`freshness-cadence.ts`) rather than owning the
- * value: `project-lens-config.ts`'s no-config discovery cache (#2483 round 2)
- * needs the same cadence and cannot import it FROM here, because this module
- * already imports `project-lens-config.ts` (the `clients/` acyclic-imports
- * rule would reject the reverse edge). Re-exported under this name so the
- * existing test imports (`tests/clients/project-ignore-freshness.test.ts` and
- * siblings) are unaffected.
- */
-export const PROJECT_IGNORE_FRESHNESS_CADENCE_MS = FRESHNESS_CADENCE_MS;
-function fileFreshnessSignature(filePath) {
-    try {
-        const stat = fs.statSync(filePath);
-        return { mtimeMs: stat.mtimeMs, size: stat.size };
-    }
-    catch {
-        return { mtimeMs: -1, size: -1 };
-    }
-}
-function globalConfigSignature() {
-    return fileFreshnessSignature(getPiLensGlobalConfigPath());
-}
-function gitignoreSignature(rootDir) {
-    return fileFreshnessSignature(path.join(rootDir, ".gitignore"));
-}
-function hasIgnoreSourceDrift(sources) {
-    return sources.some((source) => {
-        const current = fileFreshnessSignature(source.path);
-        return current.mtimeMs !== source.mtimeMs || current.size !== source.size;
-    });
-}
-/**
- * The project config file found by the same upward walk as the loader. Cache
- * invalidation must track the actual file found, not only a file directly under
- * the git root: nested worktrees/submodules can legitimately inherit a
- * `.pi-lens.json` from a parent directory.
- */
-function lensConfigInfo(rootDir) {
-    const info = findPiLensProjectConfig(rootDir);
-    return info
-        ? { info, path: info.path, mtimeMs: info.mtimeMs, size: info.size }
-        : { info, path: undefined, mtimeMs: -1, size: -1 };
-}
-export function getProjectIgnoreMatcher(rootDir) {
-    const resolvedRoot = resolveGitIgnoreRoot(rootDir);
-    const gitignoreSig = gitignoreSignature(resolvedRoot);
-    const lensConfig = lensConfigInfo(resolvedRoot);
-    const globalSig = globalConfigSignature();
-    const cached = projectIgnoreMatcherCache.get(resolvedRoot);
-    // A path can be discovered during the previous matcher lookup. Publish only
-    // previously unseen sources so a walk cannot replace a pre-edit baseline.
-    //
-    // This runs BEFORE the sweep below, and the order is load-bearing. A nested
-    // source can only be published on a call AFTER the one that consumed it.
-    // With the publish second, the sweep never saw a source during the window in
-    // which it was discovered, yet still reset `lastIgnoreFreshnessCheckMs`, so
-    // the first sweep that could see it ran a FULL cadence later. Pickup was up
-    // to 2x cadence, not 1x: measured stale at 2 s and 3 s, fresh at 4 s. The
-    // baseline is protected by the `known` set, not by the ordering, so moving
-    // the publish earlier cannot let a walk overwrite a pre-edit baseline.
-    if (cached) {
-        const known = new Set(cached.ignoreSources.map((source) => source.path));
-        for (const source of cached.matcher.getConsumedIgnoreSources()) {
-            if (!known.has(source.path))
-                cached.ignoreSources.push(source);
-        }
-    }
-    if (cached &&
-        Date.now() - cached.lastIgnoreFreshnessCheckMs >=
-            PROJECT_IGNORE_FRESHNESS_CADENCE_MS) {
-        cached.lastIgnoreFreshnessCheckMs = Date.now();
-        if (hasIgnoreSourceDrift(cached.ignoreSources)) {
-            for (const source of cached.ignoreSources) {
-                if (hasIgnoreSourceDrift([source])) {
-                    const sourceIndex = cached.ignoreSources.findIndex((cachedSource) => cachedSource.path === source.path);
-                    invalidateProjectIgnoreMatcherForPath(source.path);
-                    cached.matcher.refreshConsumedIgnoreSource(source.path);
-                    const refreshedSource = cached.matcher
-                        .getConsumedIgnoreSources()
-                        .find((current) => current.path === source.path);
-                    if (sourceIndex !== -1 && refreshedSource !== undefined) {
-                        cached.ignoreSources.splice(sourceIndex, 1, refreshedSource);
-                    }
-                }
-            }
-        }
-    }
-    if (cached?.gitignoreMtimeMs === gitignoreSig.mtimeMs &&
-        cached?.gitignoreSize === gitignoreSig.size &&
-        cached?.lensConfigPath === lensConfig.path &&
-        cached?.lensConfigMtimeMs === lensConfig.mtimeMs &&
-        cached?.lensConfigSize === lensConfig.size &&
-        cached?.globalConfigMtimeMs === globalSig.mtimeMs &&
-        cached?.globalConfigSize === globalSig.size) {
-        return cached.matcher;
-    }
-    // Load both configs fresh on cache miss. On a cache HIT (the common case)
-    // none of this runs — the only per-call cost is the size:mtimeMs stats above
-    // (size is free from the same stat). The project loader is itself
-    // size:mtimeMs-cached; the global loader re-parses, but only here on miss
-    // (when some tracked signature changed).
-    const projectConfig = loadPiLensProjectConfig(resolvedRoot, lensConfig.info);
-    const matcher = createProjectIgnoreMatcher(resolvedRoot, projectConfig.ignore, getGlobalIgnorePatterns());
-    projectIgnoreMatcherCache.set(resolvedRoot, {
-        ignoreSources: [...matcher.getConsumedIgnoreSources()],
-        lastIgnoreFreshnessCheckMs: Date.now(),
-        gitignoreMtimeMs: gitignoreSig.mtimeMs,
-        gitignoreSize: gitignoreSig.size,
-        lensConfigPath: lensConfig.path,
-        lensConfigMtimeMs: lensConfig.mtimeMs,
-        lensConfigSize: lensConfig.size,
-        globalConfigMtimeMs: globalSig.mtimeMs,
-        globalConfigSize: globalSig.size,
-        matcher,
-    });
-    return matcher;
-}
-/**
- * Invalidate the cached matcher state affected by a written `.gitignore`.
- *
- * Root changes discard the whole matcher because they change its base pattern
- * set. Nested changes evict only verdicts in that directory's subtree; the
- * matcher and compiled-glob memo remain reusable for unrelated trees.
- */
-export function invalidateProjectIgnoreMatcherForPath(filePath) {
-    const resolvedPath = path.resolve(filePath);
-    if (path.basename(resolvedPath).toLowerCase() !== ".gitignore")
-        return;
-    for (const [cachedRoot, cached] of projectIgnoreMatcherCache) {
-        if (!isUnderDir(resolvedPath, cachedRoot))
-            continue;
-        if (normalizeFilePath(resolvedPath) ===
-            normalizeFilePath(path.join(cachedRoot, ".gitignore"))) {
-            projectIgnoreMatcherCache.delete(cachedRoot);
-            continue;
-        }
-        cached.matcher.invalidateSubtree(path.dirname(resolvedPath));
-    }
-}
-export function isPathIgnoredByProject(filePath, rootDir, isDirectory = false) {
-    return getProjectIgnoreMatcher(rootDir).isIgnored(filePath, isDirectory);
-}
-/**
- * "Is this a project-source path pi-lens's bookkeeping should care about" —
- * the `isPathIgnoredByProject` + `isExternalOrVendorFile` pair, previously
- * hand-duplicated at each of its 2 call sites (`registerReadBridge` and
- * `registerMutationBridge` in `index.ts`, both #2423). Consolidated here
- * (#2450 review round 2, F4) when a THIRD call site — `tools/lsp-navigation.ts`
- * threading the same gate onto its directly-threaded `LspMutationContext`, so
- * the direct LSP-mutation path and the mutation-bridge fallback path apply
- * the identical gate instead of the fallback silently being narrower. A
- * caller still layers its own `no-read-guard`-style flag check on top; that
- * flag has no file path to check against, so it stays outside this helper.
- */
-export function isRecordableProjectPath(filePath, projectRoot) {
-    if (isPathIgnoredByProject(filePath, projectRoot, false))
-        return false;
-    if (isExternalOrVendorFile(filePath, projectRoot))
-        return false;
-    return true;
-}
-const projectIgnoreGlobsCache = new Map();
-export function getProjectIgnoreGlobs(rootDir) {
-    const resolvedRoot = path.resolve(rootDir);
-    const signature = gitignoreSignature(resolvedRoot);
-    const cached = projectIgnoreGlobsCache.get(resolvedRoot);
-    if (cached &&
-        cached.mtimeMs === signature.mtimeMs &&
-        cached.size === signature.size) {
-        return cached.globs;
-    }
-    const globs = readGitignorePatterns(resolvedRoot)
-        .filter((pattern) => !pattern.negated)
-        .flatMap((pattern) => expandGitignorePattern(pattern));
-    projectIgnoreGlobsCache.set(resolvedRoot, { ...signature, globs });
-    return globs;
-}
-/**
- * Read simple directory-name entries from a root .gitignore.
- *
- * Prefer createProjectIgnoreMatcher() for path-aware gitignore matching. This
- * helper is kept for callers/tests that only need simple directory names.
- */
-export function readGitignoreDirs(rootDir) {
-    return readGitignorePatterns(rootDir)
-        .filter((entry) => !entry.negated &&
-        !entry.pattern.includes("*") &&
-        !entry.pattern.includes("?") &&
-        !entry.pattern.includes("[") &&
-        !entry.pattern.includes("/"))
-        .map((entry) => entry.pattern);
-}
-function globToRegExp(glob) {
-    // Directory names use the same `*`-only dialect as read-guard exemptions.
-    // Collapse adjacent stars before compiling to avoid nullable-group
-    // backtracking on a non-matching name (#2622).
-    const escaped = glob
-        .replace(/\*+/g, "*")
-        .replace(/[.+^${}()|[\]\\]/g, "\\$&")
-        .replace(/\*/g, ".*")
-        .replace(/\?/g, ".");
-    return new RegExp(`^${escaped}$`, "i");
-}
-/**
- * Match directory name against exclusion patterns.
- * Supports exact names and lightweight glob patterns (for example `*.dSYM`).
- */
-export function isExcludedDirName(dirName, extraPatterns = []) {
-    const candidate = dirName.trim();
-    if (!candidate)
-        return false;
-    const patterns = [...EXCLUDED_DIRS, ...extraPatterns]
-        .map((p) => p.trim())
-        .filter((p) => p.length > 0);
-    const candidateLower = candidate.toLowerCase();
-    for (const pattern of patterns) {
-        const patLower = pattern.toLowerCase();
-        if (!patLower.includes("*") && !patLower.includes("?")) {
-            if (candidateLower === patLower)
-                return true;
-            continue;
-        }
-        if (globToRegExp(pattern).test(candidate))
-            return true;
-    }
-    return false;
-}
-/**
- * Convert excluded directory names into glob patterns used by scanners.
- */
-export function getExcludedDirGlobs() {
-    return EXCLUDED_DIRS.map((dir) => `**/${dir}/**`);
-}
-/**
- * Shared Knip ignore patterns derived from central exclusions.
- */
-export function getKnipIgnorePatterns() {
-    return [
-        ...getExcludedDirGlobs(),
-        "**/*.test.ts",
-        "**/*.test.tsx",
-        "**/*.test.js",
-        "**/*.test.jsx",
-        "**/*.spec.ts",
-        "**/*.spec.tsx",
-        "**/*.spec.js",
-        "**/*.spec.jsx",
-        "**/*.poc.test.ts",
-        "**/*.poc.test.tsx",
-        "**/__tests__/**",
-        "**/tests/**",
-    ];
-}
-/**
- * Spawn a command and detect whether it modified a file on disk.
- * Returns 1 if the file content changed after the command ran, 0 otherwise.
- * Useful for auto-fix tools (ESLint, Stylelint, RuboCop, etc.).
- */
-export async function detectFileChangedAfterCommand(filePath, command, args, cwd, ignoreStatuses = []) {
-    let before = "";
-    try {
-        before = fs.readFileSync(filePath, "utf-8");
-    }
-    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;
-    }
-    try {
-        const after = fs.readFileSync(filePath, "utf-8");
-        return before !== after ? 1 : 0;
-    }
-    catch {
-        return 0;
-    }
-}
-/**
- * Check if file path is a test/fixture/mock file.
- * Used by secrets scanner, rate command, and dispatch runners
- * to skip these files (false positives on fake credentials, etc).
- *
- * Delegate naming/directory classification to file-role (#2928). Fixtures,
- * mocks and test helpers remain this skip gate's separate policy.
- */
-export function isTestFile(filePath) {
-    if (detectFileRole(filePath) === "test")
-        return true;
-    const normalized = filePath.replace(/\\/g, "/");
-    return (normalized.includes("test-utils") ||
-        normalized.startsWith("test-") ||
-        normalized.includes(".fixture.") ||
-        normalized.includes(".mock."));
-}