Skip to content

Commit e847bfc

Browse files
LaZzyManclaude
andcommitted
fix(worktree): add showWorktreeExitDialog to dialogsVisible
Phase C task 8 introduced showWorktreeExitDialog state and the dialog render in DialogManager, but missed adding the flag to the dialogsVisible OR expression. DefaultAppLayout only renders DialogManager when dialogsVisible is true, so the dialog was never shown — second Ctrl+C in a worktree silently absorbed instead of triggering the prompt. Caught by Group E E2E tests. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 97afde7 commit e847bfc

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

packages/cli/src/ui/AppContainer.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2132,7 +2132,8 @@ export const AppContainer = (props: AppContainerProps) => {
21322132
isHelpDialogOpen ||
21332133
isExtensionsManagerDialogOpen ||
21342134
isRewindSelectorOpen ||
2135-
bgTasksDialogOpen;
2135+
bgTasksDialogOpen ||
2136+
showWorktreeExitDialog;
21362137
dialogsVisibleRef.current = dialogsVisible;
21372138
const shouldShowStickyTodos =
21382139
stickyTodos !== null &&

0 commit comments

Comments
 (0)