From ebca062175a628feba7b8394bd8930cc2cb5eb77 Mon Sep 17 00:00:00 2001 From: Mathews-Tom Date: Sun, 19 Jul 2026 15:15:58 +0530 Subject: [PATCH] fix(coding-agent): expect allowAskReopen in merged tree-summary navigateTree assertions Upstream's shift+enter summarize-and-switch test file (f33465a97, brought in by the upstream/main merge) asserts navigateTree's options literally, but this PR's ask-reanswer flow always passes allowAskReopen: true. Update the NavigateTree mock type and the four call assertions to match. --- .../test/selector-controller-tree-summary.test.ts | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/packages/coding-agent/test/selector-controller-tree-summary.test.ts b/packages/coding-agent/test/selector-controller-tree-summary.test.ts index 325c20200..4e457b012 100644 --- a/packages/coding-agent/test/selector-controller-tree-summary.test.ts +++ b/packages/coding-agent/test/selector-controller-tree-summary.test.ts @@ -44,7 +44,7 @@ function userNode(id: string, parentId: string | null, text: string): SessionTre type NavigateTree = ( entryId: string, - options: { summarize: boolean; customInstructions: string | undefined }, + options: { summarize: boolean; customInstructions: string | undefined; allowAskReopen: boolean }, ) => Promise<{ cancelled: boolean }>; type ShowHookSelector = (title: string, options: string[]) => Promise; @@ -131,6 +131,7 @@ describe("SelectorController tree branch summaries", () => { expect(harness.navigateTree).toHaveBeenCalledWith("root", { summarize: false, customInstructions: undefined, + allowAskReopen: true, }); }); @@ -145,6 +146,7 @@ describe("SelectorController tree branch summaries", () => { expect(harness.navigateTree).toHaveBeenCalledWith("root", { summarize: true, customInstructions: undefined, + allowAskReopen: true, }); }); @@ -160,6 +162,7 @@ describe("SelectorController tree branch summaries", () => { expect(harness.navigateTree).toHaveBeenCalledWith("root", { summarize: true, customInstructions: undefined, + allowAskReopen: true, }); }); @@ -179,6 +182,7 @@ describe("SelectorController tree branch summaries", () => { expect(harness.navigateTree).toHaveBeenCalledWith("root", { summarize: true, customInstructions: undefined, + allowAskReopen: true, }); }); });