pi-lens 3.6.3 → 3.6.4

package/clients/file-kinds.test.js

@@ -1,169 +0,0 @@
-/**
- * Tests for file-kinds.ts
- * Centralized file type detection
- */
-import { describe, expect, it } from "vitest";
-import { detectFileKind, getExtensionsForKind, getFileKindLabel, getLanguageId, isCodeKind, isConfigKind, isFileKind, isScannableFile, } from "./file-kinds.js";
-describe("detectFileKind", () => {
-    it("should detect JavaScript/TypeScript files", () => {
-        expect(detectFileKind("src/app.ts")).toBe("jsts");
-        expect(detectFileKind("src/app.tsx")).toBe("jsts");
-        expect(detectFileKind("src/app.js")).toBe("jsts");
-        expect(detectFileKind("src/app.jsx")).toBe("jsts");
-        expect(detectFileKind("src/app.mjs")).toBe("jsts");
-        expect(detectFileKind("src/app.cjs")).toBe("jsts");
-    });
-    it("should detect Python files", () => {
-        expect(detectFileKind("app.py")).toBe("python");
-        expect(detectFileKind("tests/test_app.py")).toBe("python");
-    });
-    it("should detect Go files", () => {
-        expect(detectFileKind("main.go")).toBe("go");
-        expect(detectFileKind("pkg/utils.go")).toBe("go");
-    });
-    it("should detect Rust files", () => {
-        expect(detectFileKind("main.rs")).toBe("rust");
-        expect(detectFileKind("lib/app.rs")).toBe("rust");
-    });
-    it("should detect C++ files", () => {
-        expect(detectFileKind("main.cpp")).toBe("cxx");
-        expect(detectFileKind("header.hpp")).toBe("cxx");
-        expect(detectFileKind("file.cc")).toBe("cxx");
-        expect(detectFileKind("file.hxx")).toBe("cxx");
-    });
-    it("should detect CMake files", () => {
-        expect(detectFileKind("CMakeLists.txt")).toBe("cmake");
-        expect(detectFileKind("build.cmake")).toBe("cmake");
-    });
-    it("should detect Shell files", () => {
-        expect(detectFileKind("script.sh")).toBe("shell");
-        expect(detectFileKind("script.bash")).toBe("shell");
-        expect(detectFileKind("Makefile")).toBe("shell");
-    });
-    it("should detect JSON files", () => {
-        expect(detectFileKind("config.json")).toBe("json");
-        expect(detectFileKind("package.json")).toBe("json");
-    });
-    it("should detect Markdown files", () => {
-        expect(detectFileKind("README.md")).toBe("markdown");
-        expect(detectFileKind("docs/guide.mdx")).toBe("markdown");
-    });
-    it("should detect CSS files", () => {
-        expect(detectFileKind("style.css")).toBe("css");
-        expect(detectFileKind("style.scss")).toBe("css");
-        expect(detectFileKind("style.less")).toBe("css");
-    });
-    it("should detect YAML files", () => {
-        expect(detectFileKind("config.yaml")).toBe("yaml");
-        expect(detectFileKind("config.yml")).toBe("yaml");
-    });
-    it("should return undefined for unknown extensions", () => {
-        expect(detectFileKind("file.xyz")).toBeUndefined();
-        expect(detectFileKind("file")).toBeUndefined();
-    });
-    it("should handle case-insensitive extensions", () => {
-        expect(detectFileKind("file.TS")).toBe("jsts");
-        expect(detectFileKind("file.PY")).toBe("python");
-    });
-    it("should handle paths with special characters", () => {
-        expect(detectFileKind("/path/to/file.ts")).toBe("jsts");
-        expect(detectFileKind("C:\\path\\to\\file.py")).toBe("python");
-    });
-});
-describe("isFileKind", () => {
-    it("should check single file kind", () => {
-        expect(isFileKind("app.ts", "jsts")).toBe(true);
-        expect(isFileKind("app.py", "jsts")).toBe(false);
-    });
-    it("should check multiple file kinds", () => {
-        expect(isFileKind("app.ts", ["jsts", "python"])).toBe(true);
-        expect(isFileKind("app.py", ["jsts", "python"])).toBe(true);
-        expect(isFileKind("app.go", ["jsts", "python"])).toBe(false);
-    });
-    it("should return false for undefined file kind", () => {
-        expect(isFileKind("file.xyz", "jsts")).toBe(false);
-        expect(isFileKind("file.xyz", ["jsts", "python"])).toBe(false);
-    });
-});
-describe("isCodeKind", () => {
-    it("should identify code file kinds", () => {
-        expect(isCodeKind("jsts")).toBe(true);
-        expect(isCodeKind("python")).toBe(true);
-        expect(isCodeKind("go")).toBe(true);
-        expect(isCodeKind("rust")).toBe(true);
-        expect(isCodeKind("cxx")).toBe(true);
-        expect(isCodeKind("shell")).toBe(true);
-    });
-    it("should reject non-code file kinds", () => {
-        expect(isCodeKind("json")).toBe(false);
-        expect(isCodeKind("markdown")).toBe(false);
-        expect(isCodeKind("css")).toBe(false);
-    });
-});
-describe("isConfigKind", () => {
-    it("should identify config file kinds", () => {
-        expect(isConfigKind("json")).toBe(true);
-        expect(isConfigKind("yaml")).toBe(true);
-        expect(isConfigKind("markdown")).toBe(true);
-        expect(isConfigKind("css")).toBe(true);
-    });
-    it("should reject non-config file kinds", () => {
-        expect(isConfigKind("jsts")).toBe(false);
-        expect(isConfigKind("python")).toBe(false);
-        expect(isConfigKind("go")).toBe(false);
-    });
-});
-describe("isScannableFile", () => {
-    it("should return true for code files", () => {
-        expect(isScannableFile("app.ts")).toBe(true);
-        expect(isScannableFile("app.py")).toBe(true);
-    });
-    it("should return true for config files", () => {
-        expect(isScannableFile("config.json")).toBe(true);
-        expect(isScannableFile("README.md")).toBe(true);
-    });
-    it("should return false for test files", () => {
-        expect(isScannableFile("app.test.ts")).toBe(false);
-        expect(isScannableFile("app.spec.ts")).toBe(false);
-        expect(isScannableFile("test-app.ts")).toBe(false);
-    });
-    it("should return false for unknown extensions", () => {
-        expect(isScannableFile("file.xyz")).toBe(false);
-    });
-});
-describe("getLanguageId", () => {
-    it("should return correct language IDs", () => {
-        expect(getLanguageId("jsts")).toBe("typescript");
-        expect(getLanguageId("python")).toBe("python");
-        expect(getLanguageId("go")).toBe("go");
-        expect(getLanguageId("rust")).toBe("rust");
-        expect(getLanguageId("cxx")).toBe("cpp");
-        expect(getLanguageId("json")).toBe("json");
-    });
-    it("should return plaintext for unknown kinds", () => {
-        expect(getLanguageId("unknown")).toBe("plaintext");
-    });
-});
-describe("getExtensionsForKind", () => {
-    it("should return extensions for jsts", () => {
-        const exts = getExtensionsForKind("jsts");
-        expect(exts).toContain(".ts");
-        expect(exts).toContain(".tsx");
-        expect(exts).toContain(".js");
-        expect(exts).toContain(".jsx");
-    });
-    it("should return extensions for python", () => {
-        const exts = getExtensionsForKind("python");
-        expect(exts).toEqual([".py"]);
-    });
-});
-describe("getFileKindLabel", () => {
-    it("should return human-readable labels", () => {
-        expect(getFileKindLabel("jsts")).toBe("JavaScript/TypeScript");
-        expect(getFileKindLabel("python")).toBe("Python");
-        expect(getFileKindLabel("cxx")).toBe("C/C++");
-    });
-    it("should return kind as fallback", () => {
-        expect(getFileKindLabel("unknown")).toBe("unknown");
-    });
-});

package/clients/file-kinds.test.ts

@@ -1,210 +0,0 @@
-/**
- * Tests for file-kinds.ts
- * Centralized file type detection
- */
-
-import { describe, expect, it } from "vitest";
-import {
-	detectFileKind,
-	getExtensionsForKind,
-	getFileKindLabel,
-	getLanguageId,
-	isCodeKind,
-	isConfigKind,
-	isFileKind,
-	isScannableFile,
-} from "./file-kinds.js";
-
-describe("detectFileKind", () => {
-	it("should detect JavaScript/TypeScript files", () => {
-		expect(detectFileKind("src/app.ts")).toBe("jsts");
-		expect(detectFileKind("src/app.tsx")).toBe("jsts");
-		expect(detectFileKind("src/app.js")).toBe("jsts");
-		expect(detectFileKind("src/app.jsx")).toBe("jsts");
-		expect(detectFileKind("src/app.mjs")).toBe("jsts");
-		expect(detectFileKind("src/app.cjs")).toBe("jsts");
-	});
-
-	it("should detect Python files", () => {
-		expect(detectFileKind("app.py")).toBe("python");
-		expect(detectFileKind("tests/test_app.py")).toBe("python");
-	});
-
-	it("should detect Go files", () => {
-		expect(detectFileKind("main.go")).toBe("go");
-		expect(detectFileKind("pkg/utils.go")).toBe("go");
-	});
-
-	it("should detect Rust files", () => {
-		expect(detectFileKind("main.rs")).toBe("rust");
-		expect(detectFileKind("lib/app.rs")).toBe("rust");
-	});
-
-	it("should detect C++ files", () => {
-		expect(detectFileKind("main.cpp")).toBe("cxx");
-		expect(detectFileKind("header.hpp")).toBe("cxx");
-		expect(detectFileKind("file.cc")).toBe("cxx");
-		expect(detectFileKind("file.hxx")).toBe("cxx");
-	});
-
-	it("should detect CMake files", () => {
-		expect(detectFileKind("CMakeLists.txt")).toBe("cmake");
-		expect(detectFileKind("build.cmake")).toBe("cmake");
-	});
-
-	it("should detect Shell files", () => {
-		expect(detectFileKind("script.sh")).toBe("shell");
-		expect(detectFileKind("script.bash")).toBe("shell");
-		expect(detectFileKind("Makefile")).toBe("shell");
-	});
-
-	it("should detect JSON files", () => {
-		expect(detectFileKind("config.json")).toBe("json");
-		expect(detectFileKind("package.json")).toBe("json");
-	});
-
-	it("should detect Markdown files", () => {
-		expect(detectFileKind("README.md")).toBe("markdown");
-		expect(detectFileKind("docs/guide.mdx")).toBe("markdown");
-	});
-
-	it("should detect CSS files", () => {
-		expect(detectFileKind("style.css")).toBe("css");
-		expect(detectFileKind("style.scss")).toBe("css");
-		expect(detectFileKind("style.less")).toBe("css");
-	});
-
-	it("should detect YAML files", () => {
-		expect(detectFileKind("config.yaml")).toBe("yaml");
-		expect(detectFileKind("config.yml")).toBe("yaml");
-	});
-
-	it("should return undefined for unknown extensions", () => {
-		expect(detectFileKind("file.xyz")).toBeUndefined();
-		expect(detectFileKind("file")).toBeUndefined();
-	});
-
-	it("should handle case-insensitive extensions", () => {
-		expect(detectFileKind("file.TS")).toBe("jsts");
-		expect(detectFileKind("file.PY")).toBe("python");
-	});
-
-	it("should handle paths with special characters", () => {
-		expect(detectFileKind("/path/to/file.ts")).toBe("jsts");
-		expect(detectFileKind("C:\\path\\to\\file.py")).toBe("python");
-	});
-});
-
-describe("isFileKind", () => {
-	it("should check single file kind", () => {
-		expect(isFileKind("app.ts", "jsts")).toBe(true);
-		expect(isFileKind("app.py", "jsts")).toBe(false);
-	});
-
-	it("should check multiple file kinds", () => {
-		expect(isFileKind("app.ts", ["jsts", "python"])).toBe(true);
-		expect(isFileKind("app.py", ["jsts", "python"])).toBe(true);
-		expect(isFileKind("app.go", ["jsts", "python"])).toBe(false);
-	});
-
-	it("should return false for undefined file kind", () => {
-		expect(isFileKind("file.xyz", "jsts")).toBe(false);
-		expect(isFileKind("file.xyz", ["jsts", "python"])).toBe(false);
-	});
-});
-
-describe("isCodeKind", () => {
-	it("should identify code file kinds", () => {
-		expect(isCodeKind("jsts")).toBe(true);
-		expect(isCodeKind("python")).toBe(true);
-		expect(isCodeKind("go")).toBe(true);
-		expect(isCodeKind("rust")).toBe(true);
-		expect(isCodeKind("cxx")).toBe(true);
-		expect(isCodeKind("shell")).toBe(true);
-	});
-
-	it("should reject non-code file kinds", () => {
-		expect(isCodeKind("json")).toBe(false);
-		expect(isCodeKind("markdown")).toBe(false);
-		expect(isCodeKind("css")).toBe(false);
-	});
-});
-
-describe("isConfigKind", () => {
-	it("should identify config file kinds", () => {
-		expect(isConfigKind("json")).toBe(true);
-		expect(isConfigKind("yaml")).toBe(true);
-		expect(isConfigKind("markdown")).toBe(true);
-		expect(isConfigKind("css")).toBe(true);
-	});
-
-	it("should reject non-config file kinds", () => {
-		expect(isConfigKind("jsts")).toBe(false);
-		expect(isConfigKind("python")).toBe(false);
-		expect(isConfigKind("go")).toBe(false);
-	});
-});
-
-describe("isScannableFile", () => {
-	it("should return true for code files", () => {
-		expect(isScannableFile("app.ts")).toBe(true);
-		expect(isScannableFile("app.py")).toBe(true);
-	});
-
-	it("should return true for config files", () => {
-		expect(isScannableFile("config.json")).toBe(true);
-		expect(isScannableFile("README.md")).toBe(true);
-	});
-
-	it("should return false for test files", () => {
-		expect(isScannableFile("app.test.ts")).toBe(false);
-		expect(isScannableFile("app.spec.ts")).toBe(false);
-		expect(isScannableFile("test-app.ts")).toBe(false);
-	});
-
-	it("should return false for unknown extensions", () => {
-		expect(isScannableFile("file.xyz")).toBe(false);
-	});
-});
-
-describe("getLanguageId", () => {
-	it("should return correct language IDs", () => {
-		expect(getLanguageId("jsts")).toBe("typescript");
-		expect(getLanguageId("python")).toBe("python");
-		expect(getLanguageId("go")).toBe("go");
-		expect(getLanguageId("rust")).toBe("rust");
-		expect(getLanguageId("cxx")).toBe("cpp");
-		expect(getLanguageId("json")).toBe("json");
-	});
-
-	it("should return plaintext for unknown kinds", () => {
-		expect(getLanguageId("unknown" as any)).toBe("plaintext");
-	});
-});
-
-describe("getExtensionsForKind", () => {
-	it("should return extensions for jsts", () => {
-		const exts = getExtensionsForKind("jsts");
-		expect(exts).toContain(".ts");
-		expect(exts).toContain(".tsx");
-		expect(exts).toContain(".js");
-		expect(exts).toContain(".jsx");
-	});
-
-	it("should return extensions for python", () => {
-		const exts = getExtensionsForKind("python");
-		expect(exts).toEqual([".py"]);
-	});
-});
-
-describe("getFileKindLabel", () => {
-	it("should return human-readable labels", () => {
-		expect(getFileKindLabel("jsts")).toBe("JavaScript/TypeScript");
-		expect(getFileKindLabel("python")).toBe("Python");
-		expect(getFileKindLabel("cxx")).toBe("C/C++");
-	});
-
-	it("should return kind as fallback", () => {
-		expect(getFileKindLabel("unknown" as any)).toBe("unknown");
-	});
-});

package/clients/file-time.js

@@ -1,152 +0,0 @@
-/**
- * FileTime Tracking for pi-lens
- *
- * Prevents race conditions when auto-formatting or external tools modify files.
- * Tracks file modification times and sizes to detect external changes.
- *
- * Inspired by OpenCode's FileTime system - ensures agents re-read files
- * that have been modified externally (including by formatters).
- */
-import * as fs from "node:fs";
-import * as path from "node:path";
-// --- Singleton State ---
-const globalState = {
-    reads: new Map(),
-    locks: new Map(),
-};
-// --- Public API ---
-export class FileTime {
-    constructor(sessionID) {
-        this.sessionID = sessionID;
-    }
-    /**
-     * Record a file read with current stats
-     * Call this after ANY file modification (including formatting)
-     */
-    read(filePath) {
-        const absolutePath = path.resolve(filePath);
-        const stamp = createStamp(absolutePath);
-        let sessionReads = globalState.reads.get(this.sessionID);
-        if (!sessionReads) {
-            sessionReads = new Map();
-            globalState.reads.set(this.sessionID, sessionReads);
-        }
-        sessionReads.set(absolutePath, stamp);
-        return stamp;
-    }
-    /**
-     * Get last recorded stamp for a file
-     */
-    get(filePath) {
-        const absolutePath = path.resolve(filePath);
-        const sessionReads = globalState.reads.get(this.sessionID);
-        return sessionReads?.get(absolutePath);
-    }
-    /**
-     * Assert file hasn't changed since last read
-     * Throws error if file modified externally - forces agent to re-read
-     */
-    assert(filePath) {
-        const absolutePath = path.resolve(filePath);
-        const sessionReads = globalState.reads.get(this.sessionID);
-        const recorded = sessionReads?.get(absolutePath);
-        if (!recorded) {
-            throw new FileTimeError(`You must read file ${absolutePath} before modifying it. Use the read tool first.`, absolutePath, "not-read");
-        }
-        const current = createStamp(absolutePath);
-        const changed = current.mtime !== recorded.mtime ||
-            current.ctime !== recorded.ctime ||
-            current.size !== recorded.size;
-        if (changed) {
-            throw new FileTimeError(`File ${absolutePath} has been modified since it was last read.\n` +
-                `Last modification: ${new Date(current.mtime ?? Date.now()).toISOString()}\n` +
-                `Last read: ${recorded.readAt.toISOString()}\n\n` +
-                `Please read the file again before modifying it.`, absolutePath, "modified");
-        }
-    }
-    /**
-     * Check if file has changed (non-throwing version of assert)
-     */
-    hasChanged(filePath) {
-        const absolutePath = path.resolve(filePath);
-        const sessionReads = globalState.reads.get(this.sessionID);
-        const recorded = sessionReads?.get(absolutePath);
-        if (!recorded)
-            return true; // Never read = changed
-        const current = createStamp(absolutePath);
-        return (current.mtime !== recorded.mtime ||
-            current.ctime !== recorded.ctime ||
-            current.size !== recorded.size);
-    }
-    /**
-     * Acquire exclusive lock on file
-     * Prevents concurrent modifications to same file
-     */
-    async withLock(filePath, fn) {
-        const absolutePath = path.resolve(filePath);
-        // Wait for existing lock
-        while (globalState.locks.has(absolutePath)) {
-            const existing = globalState.locks.get(absolutePath);
-            if (existing)
-                await existing;
-        }
-        // Create new lock
-        const lockPromise = fn().finally(() => {
-            globalState.locks.delete(absolutePath);
-        });
-        globalState.locks.set(absolutePath, lockPromise.then(() => { }));
-        return lockPromise;
-    }
-    /**
-     * Clear all tracked files for this session
-     */
-    clear() {
-        globalState.reads.delete(this.sessionID);
-    }
-    /**
-     * Clear specific file tracking
-     */
-    clearFile(filePath) {
-        const absolutePath = path.resolve(filePath);
-        const sessionReads = globalState.reads.get(this.sessionID);
-        sessionReads?.delete(absolutePath);
-    }
-}
-// --- Error Type ---
-export class FileTimeError extends Error {
-    constructor(message, filePath, reason) {
-        super(message);
-        this.name = "FileTimeError";
-        this.filePath = filePath;
-        this.reason = reason;
-    }
-}
-// --- Utilities ---
-function createStamp(filePath) {
-    try {
-        const stats = fs.statSync(filePath);
-        return {
-            readAt: new Date(),
-            mtime: stats.mtime.getTime(),
-            ctime: stats.ctime.getTime(),
-            size: stats.size,
-        };
-    }
-    catch {
-        // File doesn't exist - return empty stamp
-        return {
-            readAt: new Date(),
-            mtime: undefined,
-            ctime: undefined,
-            size: undefined,
-        };
-    }
-}
-// --- Global Helpers ---
-export function createFileTime(sessionID) {
-    return new FileTime(sessionID);
-}
-export function clearAllSessions() {
-    globalState.reads.clear();
-    globalState.locks.clear();
-}

package/clients/file-utils.js

@@ -1,40 +0,0 @@
-/**
- * Shared file path utilities for 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",
-    ".next",
-    ".pi-lens",
-    ".pi", // pi agent directory
-    ".ruff_cache", // Python linter cache
-    "venv",
-    ".venv",
-    "coverage",
-    "__pycache__",
-    ".tox",
-    ".pytest_cache",
-];
-/**
- * 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).
- */
-export function isTestFile(filePath) {
-    const normalized = filePath.replace(/\\/g, "/");
-    return (normalized.includes(".test.") ||
-        normalized.includes(".spec.") ||
-        normalized.includes("/test/") ||
-        normalized.includes("/tests/") ||
-        normalized.includes("__tests__/") ||
-        normalized.includes("test-utils") ||
-        normalized.startsWith("test-") ||
-        normalized.includes(".fixture.") ||
-        normalized.includes(".mock."));
-}