From 79d36ac0a54c17a877264eb4d532854eb51e2e42 Mon Sep 17 00:00:00 2001 From: Abhi <43648792+abhipatel12@users.noreply.github.com> Date: Thu, 17 Jul 2025 12:07:10 -0400 Subject: [PATCH] (fix) - regression to quit shortcut from command migration (#4374) --- packages/cli/src/ui/App.tsx | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/packages/cli/src/ui/App.tsx b/packages/cli/src/ui/App.tsx index 5e16b449..e2856199 100644 --- a/packages/cli/src/ui/App.tsx +++ b/packages/cli/src/ui/App.tsx @@ -432,15 +432,8 @@ const App = ({ config, settings, startupWarnings = [], version }: AppProps) => { if (timerRef.current) { clearTimeout(timerRef.current); } - const quitCommand = slashCommands.find( - (cmd) => cmd.name === 'quit' || cmd.altName === 'exit', - ); - if (quitCommand && quitCommand.action) { - quitCommand.action(commandContext, ''); - } else { - // This is unlikely to be needed but added for an additional fallback. - process.exit(0); - } + // Directly invoke the central command handler. + handleSlashCommand('/quit'); } else { setPressedOnce(true); timerRef.current = setTimeout(() => { @@ -449,8 +442,7 @@ const App = ({ config, settings, startupWarnings = [], version }: AppProps) => { }, CTRL_EXIT_PROMPT_DURATION_MS); } }, - // Add commandContext to the dependency array here! - [slashCommands, commandContext], + [handleSlashCommand], ); useInput((input: string, key: InkKeyType) => {