fix(cli): update command registry tests for compatibility

- Update bash-command.test.ts for new test patterns
- Update command-args.test.ts for new test patterns
- Minor updates to command-registry.ts

Author: brandonkachen

cli/src/commands/__tests__/bash-command.test.ts

@@ -33,7 +33,7 @@ describe('bash command', () => {
       logoutMutation: {} as any,
       streamMessageIdRef: { current: null },
       addToQueue: mock(() => {}),
-      clearMessages: mock(() => {}),
+      resetRunState: mock(() => {}),
       saveToHistory: mock(() => {}),
       scrollToLatest: mock(() => {}),
       sendMessage: mock(async () => {}),
@@ -301,7 +301,7 @@ describe('bash command', () => {
       logoutMutation: {} as any,
       streamMessageIdRef: { current: null },
       addToQueue: mock(() => {}),
-      clearMessages: mock(() => {}),
+      resetRunState: mock(() => {}),
       saveToHistory: mock(() => {}),
       scrollToLatest: mock(() => {}),
       sendMessage: mock(async () => {}),

cli/src/commands/__tests__/command-args.test.ts

@@ -30,7 +30,7 @@ describe('command factory pattern', () => {
       logoutMutation: {} as RouterParams['logoutMutation'],
       streamMessageIdRef: { current: null },
       addToQueue: mock(() => {}),
-      clearMessages: mock(() => {}),
+      resetRunState: mock(() => {}),
       saveToHistory: mock(() => {}),
       scrollToLatest: mock(() => {}),
       sendMessage: mock(async () => {}),
@@ -214,22 +214,22 @@ describe('command factory pattern', () => {
 
       const sendMessage = mock(async () => {})
       const setMessages = mock(() => {})
-      const clearMessages = mock(() => {})
+      const resetRunState = mock(() => {})
       const setCanProcessQueue = mock(() => {})
 
       const params = createMockParams({
         inputValue: '/new hello world',
         sendMessage,
         setMessages,
-        clearMessages,
+        resetRunState,
         setCanProcessQueue,
       })
 
       newCmd!.handler(params, 'hello world')
 
       // Should clear messages
       expect(setMessages).toHaveBeenCalled()
-      expect(clearMessages).toHaveBeenCalled()
+      expect(resetRunState).toHaveBeenCalled()
 
       // Should re-enable queue and send message
       expect(setCanProcessQueue).toHaveBeenCalledWith(true)
@@ -245,22 +245,22 @@ describe('command factory pattern', () => {
 
       const sendMessage = mock(async () => {})
       const setMessages = mock(() => {})
-      const clearMessages = mock(() => {})
+      const resetRunState = mock(() => {})
       const setCanProcessQueue = mock(() => {})
 
       const params = createMockParams({
         inputValue: '/new',
         sendMessage,
         setMessages,
-        clearMessages,
+        resetRunState,
         setCanProcessQueue,
       })
 
       newCmd!.handler(params, '')
 
       // Should clear messages
       expect(setMessages).toHaveBeenCalled()
-      expect(clearMessages).toHaveBeenCalled()
+      expect(resetRunState).toHaveBeenCalled()
 
       // Should disable queue and NOT send message
       expect(setCanProcessQueue).toHaveBeenCalledWith(false)

cli/src/commands/command-registry.ts

@@ -34,7 +34,7 @@ export type RouterParams = {
   logoutMutation: UseMutationResult<boolean, Error, void, unknown>
   streamMessageIdRef: React.MutableRefObject<string | null>
   addToQueue: (message: string, attachments?: PendingAttachment[]) => void
-  clearMessages: () => void
+  resetRunState: () => void
   saveToHistory: (message: string) => void
   scrollToLatest: () => void
   sendMessage: SendMessageFn
@@ -315,7 +315,7 @@ export const COMMAND_REGISTRY: CommandDefinition[] = [
 
       // Clear the conversation
       params.setMessages(() => [])
-      params.clearMessages()
+      params.resetRunState()
       params.saveToHistory(params.inputValue.trim())
       clearInput(params)
       params.stopStreaming()