@gotgenes/pi-permission-system 18.1.1 → 18.1.2

package/test/builtin-tool-input-formatters.test.ts

@@ -1,109 +0,0 @@
-import { describe, expect, test } from "vitest";
-
-import {
-  formatMcpInputForPrompt,
-  registerBuiltinToolInputFormatters,
-} from "#src/builtin-tool-input-formatters";
-import { ToolInputFormatterRegistry } from "#src/tool-input-formatter-registry";
-
-// ── formatMcpInputForPrompt ───────────────────────────────────────────────
-
-describe("formatMcpInputForPrompt", () => {
-  test("returns undefined when arguments is absent", () => {
-    expect(formatMcpInputForPrompt({ tool: "exa:search" })).toBeUndefined();
-  });
-
-  test("returns undefined when arguments is an empty object", () => {
-    expect(
-      formatMcpInputForPrompt({ tool: "exa:search", arguments: {} }),
-    ).toBeUndefined();
-  });
-
-  test("returns a summary for a single string argument", () => {
-    const result = formatMcpInputForPrompt({
-      tool: "exa:search",
-      arguments: { query: "typescript generics" },
-    });
-    expect(result).toBeDefined();
-    expect(result).toContain("query");
-    expect(result).toContain("typescript generics");
-    expect(result).toMatch(/^with /);
-  });
-
-  test("returns a comma-separated summary for multiple arguments", () => {
-    const result = formatMcpInputForPrompt({
-      tool: "exa:search",
-      arguments: { query: "test", numResults: 5 },
-    });
-    expect(result).toContain("query");
-    expect(result).toContain("numResults");
-    expect(result).toContain("5");
-  });
-
-  test("renders number arguments without quotes", () => {
-    const result = formatMcpInputForPrompt({
-      arguments: { count: 42 },
-    });
-    expect(result).toContain("42");
-    expect(result).not.toContain('"42"');
-  });
-
-  test("renders boolean arguments without quotes", () => {
-    const result = formatMcpInputForPrompt({
-      arguments: { verbose: true },
-    });
-    expect(result).toContain("true");
-  });
-
-  test("renders array arguments as '[N items]'", () => {
-    const result = formatMcpInputForPrompt({
-      arguments: { ids: [1, 2, 3] },
-    });
-    expect(result).toContain("[3 items]");
-  });
-
-  test("renders nested object arguments as '{…}'", () => {
-    const result = formatMcpInputForPrompt({
-      arguments: { filter: { type: "file" } },
-    });
-    expect(result).toContain("{…}");
-  });
-
-  test("truncates the full summary when it exceeds the limit", () => {
-    // Need multiple long-valued args so the joined summary exceeds 160 chars
-    const result = formatMcpInputForPrompt({
-      arguments: {
-        first: "x".repeat(80),
-        second: "y".repeat(80),
-        third: "z".repeat(80),
-      },
-    });
-    expect(result).toBeDefined();
-    expect(result!.endsWith("…")).toBe(true);
-  });
-
-  test("truncates long string argument values", () => {
-    const result = formatMcpInputForPrompt({
-      arguments: { query: "x".repeat(100) },
-    });
-    expect(result).toBeDefined();
-    // Should not include the full 100-char string verbatim
-    expect(result).not.toContain("x".repeat(100));
-  });
-});
-
-// ── registerBuiltinToolInputFormatters ────────────────────────────────────
-
-describe("registerBuiltinToolInputFormatters", () => {
-  test("registers the mcp formatter in the registry", () => {
-    const registry = new ToolInputFormatterRegistry();
-    registerBuiltinToolInputFormatters(registry);
-    expect(registry.get("mcp")).toBe(formatMcpInputForPrompt);
-  });
-
-  test("throws if called twice (duplicate registration guard)", () => {
-    const registry = new ToolInputFormatterRegistry();
-    registerBuiltinToolInputFormatters(registry);
-    expect(() => registerBuiltinToolInputFormatters(registry)).toThrow("mcp");
-  });
-});

package/test/canonicalize-path.test.ts

@@ -1,119 +0,0 @@
-import { beforeEach, describe, expect, test, vi } from "vitest";
-
-const realpathSync = vi.hoisted(() => vi.fn<(path: string) => string>());
-
-vi.mock("node:fs", () => ({
-  realpathSync,
-  default: { realpathSync },
-}));
-
-import { canonicalizePath } from "#src/canonicalize-path";
-
-function enoent(p: string): NodeJS.ErrnoException {
-  return Object.assign(new Error(`ENOENT: no such file or directory '${p}'`), {
-    code: "ENOENT",
-  });
-}
-
-describe("canonicalizePath", () => {
-  beforeEach(() => {
-    realpathSync.mockReset();
-  });
-
-  test("returns empty string for empty input", () => {
-    expect(canonicalizePath("", "linux")).toBe("");
-  });
-
-  test("returns realpathSync result when path exists", () => {
-    realpathSync.mockReturnValueOnce("/real/projects/app");
-    expect(canonicalizePath("/projects/link", "linux")).toBe(
-      "/real/projects/app",
-    );
-  });
-
-  test("re-appends a non-existent leaf to the canonical parent", () => {
-    realpathSync
-      .mockImplementationOnce(() => {
-        throw enoent("/projects/app/new-file.ts");
-      })
-      .mockReturnValueOnce("/canonical/app");
-    expect(canonicalizePath("/projects/app/new-file.ts", "linux")).toBe(
-      "/canonical/app/new-file.ts",
-    );
-  });
-
-  test("walks up multiple levels for a deeply non-existent path", () => {
-    realpathSync
-      .mockImplementationOnce(() => {
-        throw enoent("/projects/app/src/new-file.ts");
-      })
-      .mockImplementationOnce(() => {
-        throw enoent("/projects/app/src");
-      })
-      .mockImplementationOnce(() => {
-        throw enoent("/projects/app");
-      })
-      .mockReturnValueOnce("/canonical/projects");
-    expect(canonicalizePath("/projects/app/src/new-file.ts", "linux")).toBe(
-      "/canonical/projects/app/src/new-file.ts",
-    );
-  });
-
-  test("returns input unchanged when walk reaches filesystem root (all ENOENT)", () => {
-    realpathSync.mockImplementation(() => {
-      throw enoent("");
-    });
-    expect(canonicalizePath("/nonexistent/path/file.ts", "linux")).toBe(
-      "/nonexistent/path/file.ts",
-    );
-  });
-
-  test("returns input unchanged on ELOOP (symlink loop)", () => {
-    realpathSync.mockImplementation(() => {
-      throw Object.assign(new Error("ELOOP"), { code: "ELOOP" });
-    });
-    expect(canonicalizePath("/some/looping/path", "linux")).toBe(
-      "/some/looping/path",
-    );
-  });
-
-  test("returns input unchanged on EACCES (permission denied)", () => {
-    realpathSync.mockImplementation(() => {
-      throw Object.assign(new Error("EACCES"), { code: "EACCES" });
-    });
-    expect(canonicalizePath("/restricted/path", "linux")).toBe(
-      "/restricted/path",
-    );
-  });
-
-  test("handles ENOTDIR by walking up (like ENOENT)", () => {
-    realpathSync
-      .mockImplementationOnce(() => {
-        throw Object.assign(new Error("ENOTDIR"), { code: "ENOTDIR" });
-      })
-      .mockReturnValueOnce("/real/parent");
-    expect(canonicalizePath("/real/parent/not-a-dir", "linux")).toBe(
-      "/real/parent/not-a-dir",
-    );
-  });
-
-  // ── injected platform flavor (win32-separator splitting) ──────────────
-
-  test("win32: splits and rejoins on the backslash separator", () => {
-    realpathSync
-      .mockImplementationOnce(() => {
-        throw enoent("C:\\projects\\link\\file.ts");
-      })
-      .mockReturnValueOnce("C:\\real\\app");
-    expect(canonicalizePath("C:\\projects\\link\\file.ts", "win32")).toBe(
-      "C:\\real\\app\\file.ts",
-    );
-  });
-
-  test("win32: resolves an existing path via realpathSync", () => {
-    realpathSync.mockReturnValueOnce("C:\\real\\app");
-    expect(canonicalizePath("C:\\projects\\link", "win32")).toBe(
-      "C:\\real\\app",
-    );
-  });
-});