@sean.holung/minicode 0.3.8 → 0.3.10

package/dist/tests/permission-gate-cli.test.js

@@ -0,0 +1,126 @@
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { handlePermissionsSlashCommand } from "../src/cli/permissions-slash-command.js";
+import { createPermissionGate } from "../src/ui/permission-gate.js";
+import { UiStore } from "../src/ui/state/ui-store.js";
+test("/permissions status reports the current mode", () => {
+    const store = new UiStore();
+    const off = handlePermissionsSlashCommand("/permissions status", store);
+    assert.equal(off.handled, true);
+    assert.match(off.message ?? "", /OFF/);
+    store.setAutoAllowMode("writes");
+    const writes = handlePermissionsSlashCommand("/permissions status", store);
+    assert.match(writes.message ?? "", /WRITES/);
+    store.setAutoAllowMode("all");
+    const all = handlePermissionsSlashCommand("/permissions status", store);
+    assert.match(all.message ?? "", /ALL/);
+});
+test("/permissions with no args defaults to status", () => {
+    const store = new UiStore();
+    const result = handlePermissionsSlashCommand("/permissions", store);
+    assert.equal(result.handled, true);
+    assert.match(result.message ?? "", /OFF/);
+});
+test("/permissions auto MODE sets each mode", () => {
+    const store = new UiStore();
+    for (const mode of ["none", "writes", "commands", "all"]) {
+        const result = handlePermissionsSlashCommand(`/permissions auto ${mode}`, store);
+        assert.equal(result.handled, true);
+        assert.equal(store.getAutoAllowMode(), mode);
+    }
+});
+test("/permissions auto with bad value shows usage", () => {
+    const store = new UiStore();
+    store.setAutoAllowMode("writes");
+    const result = handlePermissionsSlashCommand("/permissions auto maybe", store);
+    assert.equal(result.handled, true);
+    assert.match(result.message ?? "", /Usage/i);
+    assert.equal(store.getAutoAllowMode(), "writes", "bad value should not change state");
+});
+test("non-permissions input is not handled", () => {
+    const store = new UiStore();
+    assert.equal(handlePermissionsSlashCommand("hello", store).handled, false);
+    assert.equal(handlePermissionsSlashCommand("/help", store).handled, false);
+    assert.equal(handlePermissionsSlashCommand("/permissionsXYZ", store).handled, false, "prefix-only matches must include space or be exact");
+});
+test("createPermissionGate: read-only tools allow without parking a prompt", async () => {
+    const store = new UiStore();
+    const gate = createPermissionGate(store);
+    const decision = await gate({ name: "read_file", input: { path: "x" } });
+    assert.deepEqual(decision, { outcome: "allow" });
+    assert.equal(store.getState().pendingPermission, null);
+});
+test("createPermissionGate: mode 'all' short-circuits every mutating tool", async () => {
+    const store = new UiStore();
+    store.setAutoAllowMode("all");
+    const gate = createPermissionGate(store);
+    for (const name of ["write_file", "edit_file", "run_command"]) {
+        const decision = await gate({ name, input: {} });
+        assert.deepEqual(decision, { outcome: "allow" }, `${name} should auto-allow under 'all'`);
+    }
+    assert.equal(store.getState().pendingPermission, null);
+});
+test("createPermissionGate: mode 'writes' auto-allows write_file/edit_file but prompts run_command", async () => {
+    const store = new UiStore();
+    store.setAutoAllowMode("writes");
+    const gate = createPermissionGate(store);
+    for (const name of ["write_file", "edit_file"]) {
+        const d = await gate({ name, input: {} });
+        assert.deepEqual(d, { outcome: "allow" });
+    }
+    assert.equal(store.getState().pendingPermission, null);
+    void gate({ name: "run_command", input: { command: "ls" } });
+    await new Promise((resolve) => setImmediate(resolve));
+    assert.ok(store.getState().pendingPermission, "run_command should still prompt");
+});
+test("createPermissionGate: mode 'commands' auto-allows run_command but prompts writes", async () => {
+    const store = new UiStore();
+    store.setAutoAllowMode("commands");
+    const gate = createPermissionGate(store);
+    const cmd = await gate({ name: "run_command", input: { command: "ls" } });
+    assert.deepEqual(cmd, { outcome: "allow" });
+    assert.equal(store.getState().pendingPermission, null);
+    void gate({ name: "write_file", input: {} });
+    await new Promise((resolve) => setImmediate(resolve));
+    assert.ok(store.getState().pendingPermission);
+});
+test("createPermissionGate: parks a pending prompt and resolves on user allow", async () => {
+    const store = new UiStore();
+    const gate = createPermissionGate(store);
+    const pending = gate({ name: "edit_file", input: { path: "src/x.ts" } });
+    await new Promise((resolve) => setImmediate(resolve));
+    const prompt = store.getState().pendingPermission;
+    assert.ok(prompt, "prompt should be parked on the store");
+    assert.equal(prompt.toolName, "edit_file");
+    prompt.resolve({ decision: "allow" });
+    const decision = await pending;
+    assert.deepEqual(decision, { outcome: "allow" });
+    assert.equal(store.getState().pendingPermission, null, "prompt cleared after resolve");
+    assert.equal(store.getAutoAllowMode(), "none", "mode unchanged when no setMode is sent");
+});
+test("createPermissionGate: deny resolves with a reason", async () => {
+    const store = new UiStore();
+    const gate = createPermissionGate(store);
+    const pending = gate({ name: "run_command", input: { command: "ls" } });
+    await new Promise((resolve) => setImmediate(resolve));
+    const prompt = store.getState().pendingPermission;
+    assert.ok(prompt);
+    prompt.resolve({ decision: "deny" });
+    const decision = await pending;
+    assert.equal(decision.outcome, "deny");
+    if (decision.outcome === "deny") {
+        assert.match(decision.reason, /declined/i);
+    }
+    assert.equal(store.getState().pendingPermission, null);
+});
+test("createPermissionGate: setMode='all' on allow flips the mode (used by [a] shortcut)", async () => {
+    const store = new UiStore();
+    const gate = createPermissionGate(store);
+    const pending = gate({ name: "write_file", input: { path: "x" } });
+    await new Promise((resolve) => setImmediate(resolve));
+    const prompt = store.getState().pendingPermission;
+    assert.ok(prompt);
+    prompt.resolve({ decision: "allow", setMode: "all" });
+    await pending;
+    assert.equal(store.getAutoAllowMode(), "all");
+});

package/dist/tests/permission-gate.test.js

@@ -0,0 +1,196 @@
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { AgentBridge } from "../src/serve/agent-bridge.js";
+function captureBridge() {
+    const events = [];
+    const bridge = new AgentBridge((msg) => events.push(msg), false);
+    return { bridge, events };
+}
+test("permission gate: read-only tools bypass entirely (no event, immediate allow)", async () => {
+    const { bridge, events } = captureBridge();
+    const decision = await bridge.invokeToolPermissionGateForTesting({
+        name: "read_file",
+        input: { path: "src/foo.ts" },
+    });
+    assert.deepEqual(decision, { outcome: "allow" });
+    assert.equal(events.length, 0, "no event should be emitted for read_file");
+});
+test("permission gate: search and other read-only tools bypass too", async () => {
+    const { bridge, events } = captureBridge();
+    for (const tool of ["search", "list_files", "find_references", "get_dependencies"]) {
+        const decision = await bridge.invokeToolPermissionGateForTesting({
+            name: tool,
+            input: {},
+        });
+        assert.equal(decision.outcome, "allow", `${tool} should allow without prompting`);
+    }
+    assert.equal(events.length, 0);
+});
+test("permission gate: mode 'all' short-circuits every gated tool", async () => {
+    const { bridge, events } = captureBridge();
+    bridge.setAutoAllowMode("all");
+    events.length = 0;
+    for (const tool of ["write_file", "edit_file", "run_command"]) {
+        const decision = await bridge.invokeToolPermissionGateForTesting({
+            name: tool,
+            input: { path: "x" },
+        });
+        assert.deepEqual(decision, { outcome: "allow" }, `${tool} should allow without prompting when mode is 'all'`);
+    }
+    assert.equal(events.length, 0, "no permission_required events should be emitted");
+});
+test("permission gate: mode 'writes' auto-allows write_file/edit_file but still prompts run_command", async () => {
+    const { bridge, events } = captureBridge();
+    bridge.setAutoAllowMode("writes");
+    events.length = 0;
+    for (const tool of ["write_file", "edit_file"]) {
+        const decision = await bridge.invokeToolPermissionGateForTesting({
+            name: tool,
+            input: {},
+        });
+        assert.deepEqual(decision, { outcome: "allow" }, `${tool} should auto-allow under 'writes'`);
+    }
+    assert.equal(events.length, 0, "no events for auto-allowed tools");
+    // run_command should still prompt.
+    void bridge.invokeToolPermissionGateForTesting({
+        name: "run_command",
+        input: { command: "ls" },
+    });
+    await new Promise((resolve) => setImmediate(resolve));
+    const required = events.find((e) => e.type === "permission_required");
+    assert.ok(required, "run_command should still prompt under 'writes'");
+});
+test("permission gate: mode 'commands' auto-allows run_command but still prompts write_file/edit_file", async () => {
+    const { bridge, events } = captureBridge();
+    bridge.setAutoAllowMode("commands");
+    events.length = 0;
+    const cmd = await bridge.invokeToolPermissionGateForTesting({
+        name: "run_command",
+        input: { command: "ls" },
+    });
+    assert.deepEqual(cmd, { outcome: "allow" });
+    void bridge.invokeToolPermissionGateForTesting({
+        name: "write_file",
+        input: { path: "x" },
+    });
+    await new Promise((resolve) => setImmediate(resolve));
+    const required = events.find((e) => e.type === "permission_required");
+    assert.ok(required, "write_file should still prompt under 'commands'");
+});
+test("permission gate: emits permission_required and resolves on allow", async () => {
+    const { bridge, events } = captureBridge();
+    const pending = bridge.invokeToolPermissionGateForTesting({
+        name: "write_file",
+        input: { path: "src/new.ts", content: "hello" },
+    });
+    await new Promise((resolve) => setImmediate(resolve));
+    const required = events.find((e) => e.type === "permission_required");
+    assert.ok(required, "permission_required event should be emitted");
+    assert.equal(required.toolName, "write_file");
+    assert.deepEqual(required.input, { path: "src/new.ts", content: "hello" });
+    bridge.resolvePermissionRequest(required.requestId, { decision: "allow" });
+    const decision = await pending;
+    assert.deepEqual(decision, { outcome: "allow" });
+    assert.equal(bridge.getAutoAllowMode(), "none", "mode should still be 'none' when no setAutoAllowMode was sent");
+});
+test("permission gate: deny resolves with reason fed back to the model", async () => {
+    const { bridge, events } = captureBridge();
+    const pending = bridge.invokeToolPermissionGateForTesting({
+        name: "run_command",
+        input: { command: "rm -rf /tmp/x" },
+    });
+    await new Promise((resolve) => setImmediate(resolve));
+    const required = events.find((e) => e.type === "permission_required");
+    assert.ok(required);
+    bridge.resolvePermissionRequest(required.requestId, { decision: "deny" });
+    const decision = await pending;
+    assert.equal(decision.outcome, "deny");
+    if (decision.outcome === "deny") {
+        assert.match(decision.reason, /declined/i);
+    }
+});
+test("permission gate: setAutoAllowMode on allow flips the mode and broadcasts", async () => {
+    const { bridge, events } = captureBridge();
+    const pending = bridge.invokeToolPermissionGateForTesting({
+        name: "write_file",
+        input: { path: "x" },
+    });
+    await new Promise((resolve) => setImmediate(resolve));
+    const required = events.find((e) => e.type === "permission_required");
+    assert.ok(required);
+    bridge.resolvePermissionRequest(required.requestId, {
+        decision: "allow",
+        setAutoAllowMode: "all",
+    });
+    await pending;
+    assert.equal(bridge.getAutoAllowMode(), "all");
+    const broadcast = events.find((e) => e.type === "auto_allow_mode_changed");
+    assert.ok(broadcast, "auto_allow_mode_changed should be broadcast");
+    assert.equal(broadcast.mode, "all");
+});
+test("permission gate: setAutoAllowMode is idempotent and broadcasts only on change", () => {
+    const { bridge, events } = captureBridge();
+    bridge.setAutoAllowMode("all");
+    bridge.setAutoAllowMode("all");
+    bridge.setAutoAllowMode("all");
+    const broadcasts = events.filter((e) => e.type === "auto_allow_mode_changed");
+    assert.equal(broadcasts.length, 1, "duplicate sets should not re-broadcast");
+    assert.equal(bridge.getAutoAllowMode(), "all");
+    bridge.setAutoAllowMode("writes");
+    const allBroadcasts = events.filter((e) => e.type === "auto_allow_mode_changed");
+    assert.equal(allBroadcasts.length, 2);
+    assert.equal(bridge.getAutoAllowMode(), "writes");
+});
+test("permission gate: resolving an unknown requestId is silently ignored (no throw)", () => {
+    const { bridge } = captureBridge();
+    bridge.resolvePermissionRequest("nonexistent-id", { decision: "allow" });
+});
+test("permission gate: API bypass allows mutating tools without prompting (used by /v1/chat/completions)", async () => {
+    const { bridge, events } = captureBridge();
+    // Mode is 'none' — every gated tool would normally prompt. The API
+    // bypass overrides this for programmatic API turns since there's no UI
+    // to answer the prompt.
+    bridge.setApiBypassForTesting(true);
+    for (const tool of ["write_file", "edit_file", "run_command"]) {
+        const decision = await bridge.invokeToolPermissionGateForTesting({
+            name: tool,
+            input: { path: "x" },
+        });
+        assert.deepEqual(decision, { outcome: "allow" }, `${tool} should auto-allow under API bypass even with mode='none'`);
+    }
+    assert.equal(events.length, 0, "no permission_required events should be emitted during API bypass");
+});
+test("permission gate: clearing the API bypass restores normal gating", async () => {
+    const { bridge, events } = captureBridge();
+    bridge.setApiBypassForTesting(true);
+    // First call: bypass on → allow without event.
+    await bridge.invokeToolPermissionGateForTesting({
+        name: "write_file",
+        input: { path: "x" },
+    });
+    assert.equal(events.length, 0);
+    bridge.setApiBypassForTesting(false);
+    // Second call: bypass off, mode='none' → must prompt.
+    void bridge.invokeToolPermissionGateForTesting({
+        name: "write_file",
+        input: { path: "y" },
+    });
+    await new Promise((resolve) => setImmediate(resolve));
+    const required = events.find((e) => e.type === "permission_required");
+    assert.ok(required, "after the bypass clears, mutating tools should prompt again");
+});
+test("permission gate: duplicate response for same requestId is ignored", async () => {
+    const { bridge, events } = captureBridge();
+    const pending = bridge.invokeToolPermissionGateForTesting({
+        name: "write_file",
+        input: { path: "x" },
+    });
+    await new Promise((resolve) => setImmediate(resolve));
+    const required = events.find((e) => e.type === "permission_required");
+    assert.ok(required);
+    bridge.resolvePermissionRequest(required.requestId, { decision: "allow" });
+    // Second call should be ignored — promise already resolved.
+    bridge.resolvePermissionRequest(required.requestId, { decision: "deny" });
+    const decision = await pending;
+    assert.deepEqual(decision, { outcome: "allow" });
+});