diff options
Diffstat (limited to 'packages/cli/src/ui/hooks')
| -rw-r--r-- | packages/cli/src/ui/hooks/slashCommandProcessor.test.ts | 30 | ||||
| -rw-r--r-- | packages/cli/src/ui/hooks/slashCommandProcessor.ts | 30 |
2 files changed, 42 insertions, 18 deletions
diff --git a/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts b/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts index 0c12d855..971d7aac 100644 --- a/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts +++ b/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts @@ -98,6 +98,7 @@ describe('useSlashCommandProcessor', () => { let mockOnDebugMessage: ReturnType<typeof vi.fn>; let mockOpenThemeDialog: ReturnType<typeof vi.fn>; let mockPerformMemoryRefresh: ReturnType<typeof vi.fn>; + let mockSetQuittingMessages: ReturnType<typeof vi.fn>; let mockConfig: Config; let mockCorgiMode: ReturnType<typeof vi.fn>; const mockUseSessionStats = useSessionStats as Mock; @@ -111,6 +112,7 @@ describe('useSlashCommandProcessor', () => { mockOnDebugMessage = vi.fn(); mockOpenThemeDialog = vi.fn(); mockPerformMemoryRefresh = vi.fn().mockResolvedValue(undefined); + mockSetQuittingMessages = vi.fn(); mockConfig = { getDebugMode: vi.fn(() => false), getSandbox: vi.fn(() => 'test-sandbox'), @@ -156,6 +158,7 @@ describe('useSlashCommandProcessor', () => { mockPerformMemoryRefresh, mockCorgiMode, showToolDescriptions, + mockSetQuittingMessages, ), ); return result.current; @@ -406,7 +409,7 @@ Add any other context about the problem here. }); it.each([['/quit'], ['/exit']])( - 'should handle %s, add a quit message, and exit the process', + 'should handle %s, set quitting messages, and exit the process', async (command) => { const { handleSlashCommand } = getProcessor(); const mockDate = new Date('2025-01-01T01:02:03.000Z'); @@ -416,18 +419,25 @@ Add any other context about the problem here. handleSlashCommand(command); }); - expect(mockAddItem).toHaveBeenCalledTimes(2); - expect(mockAddItem).toHaveBeenNthCalledWith( - 2, - expect.objectContaining({ - type: MessageType.QUIT, + expect(mockAddItem).not.toHaveBeenCalled(); + expect(mockSetQuittingMessages).toHaveBeenCalledWith([ + { + type: 'user', + text: command, + id: expect.any(Number), + }, + { + type: 'quit', + stats: expect.any(Object), duration: '1h 2m 3s', - }), - expect.any(Number), - ); + id: expect.any(Number), + }, + ]); // Fast-forward timers to trigger process.exit - vi.advanceTimersByTime(100); + await act(async () => { + vi.advanceTimersByTime(100); + }); expect(mockProcessExit).toHaveBeenCalledWith(0); }, ); diff --git a/packages/cli/src/ui/hooks/slashCommandProcessor.ts b/packages/cli/src/ui/hooks/slashCommandProcessor.ts index 478a62a3..d343c6ff 100644 --- a/packages/cli/src/ui/hooks/slashCommandProcessor.ts +++ b/packages/cli/src/ui/hooks/slashCommandProcessor.ts @@ -69,6 +69,7 @@ export const useSlashCommandProcessor = ( performMemoryRefresh: () => Promise<void>, toggleCorgiMode: () => void, showToolDescriptions: boolean = false, + setQuittingMessages: (message: HistoryItem[]) => void, ) => { const session = useSessionStats(); const gitService = useMemo(() => { @@ -608,17 +609,24 @@ Add any other context about the problem here. name: 'quit', altName: 'exit', description: 'exit the cli', - action: async (_mainCommand, _subCommand, _args) => { + action: async (mainCommand, _subCommand, _args) => { const now = new Date(); const { sessionStartTime, cumulative } = session.stats; const wallDuration = now.getTime() - sessionStartTime.getTime(); - addMessage({ - type: MessageType.QUIT, - stats: cumulative, - duration: formatDuration(wallDuration), - timestamp: new Date(), - }); + setQuittingMessages([ + { + type: 'user', + text: `/${mainCommand}`, + id: now.getTime() - 1, + }, + { + type: 'quit', + stats: cumulative, + duration: formatDuration(wallDuration), + id: now.getTime(), + }, + ]); setTimeout(() => { process.exit(0); @@ -749,6 +757,7 @@ Add any other context about the problem here. gitService, loadHistory, addItem, + setQuittingMessages, ]); const handleSlashCommand = useCallback( @@ -763,7 +772,12 @@ Add any other context about the problem here. return false; } const userMessageTimestamp = Date.now(); - addItem({ type: MessageType.USER, text: trimmed }, userMessageTimestamp); + if (trimmed !== '/quit' && trimmed !== '/exit') { + addItem( + { type: MessageType.USER, text: trimmed }, + userMessageTimestamp, + ); + } let subCommand: string | undefined; let args: string | undefined; |
