@gotgenes/pi-permission-system 18.1.1 → 18.2.0

package/test/yaml-frontmatter.test.ts

@@ -1,91 +0,0 @@
-import { describe, expect, test } from "vitest";
-
-import { extractFrontmatter, parseSimpleYamlMap } from "#src/yaml-frontmatter";
-
-describe("extractFrontmatter", () => {
-  test("returns empty string when no frontmatter delimiter", () => {
-    expect(extractFrontmatter("# Hello\nSome content")).toBe("");
-  });
-
-  test("returns empty string when only opening delimiter with no closing", () => {
-    expect(extractFrontmatter("---\nkey: value")).toBe("");
-  });
-
-  test("returns frontmatter body between delimiters", () => {
-    const markdown = "---\nissue: 1\ntitle: Test\n---\n# Content";
-    expect(extractFrontmatter(markdown)).toBe("issue: 1\ntitle: Test");
-  });
-
-  test("returns empty string when file does not start with ---", () => {
-    expect(extractFrontmatter("content\n---\nkey: val\n---")).toBe("");
-  });
-
-  test("handles CRLF line endings", () => {
-    const markdown = "---\r\nissue: 5\r\n---\r\n# Content";
-    expect(extractFrontmatter(markdown)).toBe("issue: 5");
-  });
-
-  test("returns empty string for empty string input", () => {
-    expect(extractFrontmatter("")).toBe("");
-  });
-
-  test("returns empty frontmatter for --- \\n--- with nothing between", () => {
-    const markdown = "---\n---\n# Content";
-    expect(extractFrontmatter(markdown)).toBe("");
-  });
-});
-
-describe("parseSimpleYamlMap", () => {
-  test("returns empty object for empty string", () => {
-    expect(parseSimpleYamlMap("")).toEqual({});
-  });
-
-  test("parses simple key-value pairs", () => {
-    const yaml = "issue: 21\ntitle: Test";
-    expect(parseSimpleYamlMap(yaml)).toEqual({ issue: "21", title: "Test" });
-  });
-
-  test("strips surrounding quotes from values", () => {
-    const yaml = 'title: "My Title"';
-    expect(parseSimpleYamlMap(yaml)).toEqual({ title: "My Title" });
-
-    const yaml2 = "title: 'My Title'";
-    expect(parseSimpleYamlMap(yaml2)).toEqual({ title: "My Title" });
-  });
-
-  test("skips lines without colon or with colon at position 0", () => {
-    const yaml = "no separator here\n:starts-with-colon: val\nkey: val";
-    const result = parseSimpleYamlMap(yaml);
-    expect(result.key).toBe("val");
-    expect(result["no separator here"]).toBeUndefined();
-  });
-
-  test("skips comment lines", () => {
-    const yaml = "# This is a comment\nkey: value";
-    expect(parseSimpleYamlMap(yaml)).toEqual({ key: "value" });
-  });
-
-  test("skips blank lines", () => {
-    const yaml = "\n\nkey: value\n\n";
-    expect(parseSimpleYamlMap(yaml)).toEqual({ key: "value" });
-  });
-
-  test("parses nested map (child indented under parent)", () => {
-    const yaml = "parent:\n  child: nested_value";
-    const result = parseSimpleYamlMap(yaml);
-    expect(result.parent).toEqual({ child: "nested_value" });
-  });
-
-  test("handles multi-line values correctly (second line is new key)", () => {
-    const yaml = "key1: val1\nkey2: val2";
-    const result = parseSimpleYamlMap(yaml);
-    expect(result.key1).toBe("val1");
-    expect(result.key2).toBe("val2");
-  });
-
-  test("strips quotes from keys", () => {
-    const yaml = '"quoted-key": value';
-    const result = parseSimpleYamlMap(yaml);
-    expect(result["quoted-key"]).toBe("value");
-  });
-});

package/test/yolo-mode.test.ts

@@ -1,188 +0,0 @@
-import { afterEach, describe, expect, test, vi } from "vitest";
-import type { PermissionSystemExtensionConfig } from "#src/extension-config";
-import { DEFAULT_EXTENSION_CONFIG } from "#src/extension-config";
-import { resolvePermissionForwardingTargetSessionId } from "#src/permission-forwarding";
-import {
-  canResolveAskPermissionRequest,
-  shouldAutoApprovePermissionState,
-} from "#src/yolo-mode";
-
-afterEach(() => {
-  vi.restoreAllMocks();
-});
-
-function makeConfig(
-  yoloMode: boolean | undefined,
-): PermissionSystemExtensionConfig {
-  return { yoloMode } as PermissionSystemExtensionConfig;
-}
-
-describe("shouldAutoApprovePermissionState", () => {
-  test("returns true for 'ask' when yolo mode is on", () => {
-    expect(shouldAutoApprovePermissionState("ask", makeConfig(true))).toBe(
-      true,
-    );
-  });
-
-  test("returns false for 'ask' when yolo mode is off", () => {
-    expect(shouldAutoApprovePermissionState("ask", makeConfig(false))).toBe(
-      false,
-    );
-  });
-
-  test("returns false for 'ask' when yolo mode is undefined", () => {
-    expect(shouldAutoApprovePermissionState("ask", makeConfig(undefined))).toBe(
-      false,
-    );
-  });
-
-  test("returns false for 'allow' even when yolo mode is on", () => {
-    expect(shouldAutoApprovePermissionState("allow", makeConfig(true))).toBe(
-      false,
-    );
-  });
-
-  test("returns false for 'deny' even when yolo mode is on", () => {
-    expect(shouldAutoApprovePermissionState("deny", makeConfig(true))).toBe(
-      false,
-    );
-  });
-});
-
-describe("canResolveAskPermissionRequest", () => {
-  test("returns true when hasUI is true regardless of other flags", () => {
-    expect(
-      canResolveAskPermissionRequest({
-        config: makeConfig(false),
-        hasUI: true,
-        isSubagent: false,
-      }),
-    ).toBe(true);
-  });
-
-  test("returns true when isSubagent is true regardless of other flags", () => {
-    expect(
-      canResolveAskPermissionRequest({
-        config: makeConfig(false),
-        hasUI: false,
-        isSubagent: true,
-      }),
-    ).toBe(true);
-  });
-
-  test("returns true when yolo mode is on regardless of UI/subagent flags", () => {
-    expect(
-      canResolveAskPermissionRequest({
-        config: makeConfig(true),
-        hasUI: false,
-        isSubagent: false,
-      }),
-    ).toBe(true);
-  });
-
-  test("returns false when no UI, not a subagent, and yolo mode is off", () => {
-    expect(
-      canResolveAskPermissionRequest({
-        config: makeConfig(false),
-        hasUI: false,
-        isSubagent: false,
-      }),
-    ).toBe(false);
-  });
-
-  test("returns false when no UI, not a subagent, and yolo mode is undefined", () => {
-    expect(
-      canResolveAskPermissionRequest({
-        config: makeConfig(undefined),
-        hasUI: false,
-        isSubagent: false,
-      }),
-    ).toBe(false);
-  });
-
-  test("returns true when all three conditions are true", () => {
-    expect(
-      canResolveAskPermissionRequest({
-        config: makeConfig(true),
-        hasUI: true,
-        isSubagent: true,
-      }),
-    ).toBe(true);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Moved from permission-system.test.ts catch-all (#342)
-// ---------------------------------------------------------------------------
-
-test("Yolo mode only auto-approves ask-state permissions", () => {
-  expect(
-    shouldAutoApprovePermissionState("ask", DEFAULT_EXTENSION_CONFIG),
-  ).toBe(false);
-  expect(
-    shouldAutoApprovePermissionState("ask", {
-      ...DEFAULT_EXTENSION_CONFIG,
-      yoloMode: true,
-    }),
-  ).toBe(true);
-  expect(
-    shouldAutoApprovePermissionState("deny", {
-      ...DEFAULT_EXTENSION_CONFIG,
-      yoloMode: true,
-    }),
-  ).toBe(false);
-  expect(
-    shouldAutoApprovePermissionState("allow", {
-      ...DEFAULT_EXTENSION_CONFIG,
-      yoloMode: true,
-    }),
-  ).toBe(false);
-});
-
-test("Yolo mode resolves ask permissions without UI or delegation forwarding", () => {
-  expect(
-    canResolveAskPermissionRequest({
-      config: DEFAULT_EXTENSION_CONFIG,
-      hasUI: false,
-      isSubagent: false,
-    }),
-  ).toBe(false);
-  expect(
-    canResolveAskPermissionRequest({
-      config: { ...DEFAULT_EXTENSION_CONFIG, yoloMode: true },
-      hasUI: false,
-      isSubagent: false,
-    }),
-  ).toBe(true);
-  expect(
-    canResolveAskPermissionRequest({
-      config: DEFAULT_EXTENSION_CONFIG,
-      hasUI: false,
-      isSubagent: true,
-    }),
-  ).toBe(true);
-});
-
-test("Yolo mode bypasses delegated ask routing when no parent forwarding target is available", () => {
-  const targetSessionId = resolvePermissionForwardingTargetSessionId({
-    hasUI: false,
-    isSubagent: true,
-    currentSessionId: "child-session",
-    env: {},
-  });
-
-  expect(targetSessionId).toBe(null);
-  expect(
-    canResolveAskPermissionRequest({
-      config: { ...DEFAULT_EXTENSION_CONFIG, yoloMode: true },
-      hasUI: false,
-      isSubagent: true,
-    }),
-  ).toBe(true);
-  expect(
-    shouldAutoApprovePermissionState("ask", {
-      ...DEFAULT_EXTENSION_CONFIG,
-      yoloMode: true,
-    }),
-  ).toBe(true);
-});