From 78069369e2253c9788c09b7a71478d140c9741f2 Mon Sep 17 00:00:00 2001 From: Adam <2363879+adamdotdevin@users.noreply.github.com> Date: Mon, 2 Mar 2026 13:10:37 -0600 Subject: fix(app): default auto-respond to false --- packages/app/src/context/permission-auto-respond.test.ts | 4 ++-- packages/app/src/context/permission-auto-respond.ts | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'packages/app/src/context') diff --git a/packages/app/src/context/permission-auto-respond.test.ts b/packages/app/src/context/permission-auto-respond.test.ts index 8657427d7..2e4cf4faf 100644 --- a/packages/app/src/context/permission-auto-respond.test.ts +++ b/packages/app/src/context/permission-auto-respond.test.ts @@ -31,13 +31,13 @@ describe("autoRespondsPermission", () => { expect(autoRespondsPermission({ root: true }, sessions, permission("child"), "/tmp/project")).toBe(true) }) - test("defaults to auto-accept when no lineage override exists", () => { + test("defaults to requiring approval when no lineage override exists", () => { const sessions = [session({ id: "root" }), session({ id: "child", parentID: "root" }), session({ id: "other" })] const autoAccept = { other: true, } - expect(autoRespondsPermission(autoAccept, sessions, permission("child"), "/tmp/project")).toBe(true) + expect(autoRespondsPermission(autoAccept, sessions, permission("child"), "/tmp/project")).toBe(false) }) test("inherits a parent session's false override", () => { diff --git a/packages/app/src/context/permission-auto-respond.ts b/packages/app/src/context/permission-auto-respond.ts index cabd514e7..727ccc937 100644 --- a/packages/app/src/context/permission-auto-respond.ts +++ b/packages/app/src/context/permission-auto-respond.ts @@ -37,5 +37,5 @@ export function autoRespondsPermission( const value = sessionLineage(session, permission.sessionID) .map((id) => accepted(autoAccept, id, directory)) .find((item): item is boolean => item !== undefined) - return value ?? true + return value ?? false } -- cgit v1.2.3