Fix keyboard input conflict between feedback modal and chat input

- Disable chat input focus when feedback modal is open
- Add disabled state to keyboard handlers to prevent chat shortcuts from firing
- Ensure modal has exclusive keyboard control when active

Author: brandonkachen

cli/src/chat.tsx

@@ -549,6 +549,7 @@ export const Chat = ({
     },
     historyNavUpEnabled,
     historyNavDownEnabled,
+    disabled: isFeedbackOpen,
   })
 
   const { tree: messageTree, topLevelMessages } = useMemo(

cli/src/hooks/use-keyboard-handlers.ts

@@ -19,6 +19,7 @@ interface KeyboardHandlersConfig {
   onInterrupt: () => void
   historyNavUpEnabled: boolean
   historyNavDownEnabled: boolean
+  disabled?: boolean
 }
 
 export const useKeyboardHandlers = ({
@@ -37,10 +38,13 @@ export const useKeyboardHandlers = ({
   onInterrupt,
   historyNavUpEnabled,
   historyNavDownEnabled,
+  disabled = false,
 }: KeyboardHandlersConfig) => {
   useKeyboard(
     useCallback(
       (key) => {
+        if (disabled) return
+
         const isEscape = key.name === 'escape'
         const isCtrlC = key.ctrl && key.name === 'c'
 
@@ -71,13 +75,14 @@ export const useKeyboardHandlers = ({
           }
         }
       },
-      [isStreaming, isWaitingForResponse, abortControllerRef, onCtrlC, onInterrupt],
+      [isStreaming, isWaitingForResponse, abortControllerRef, onCtrlC, onInterrupt, disabled],
     ),
   )
 
   useKeyboard(
     useCallback(
       (key) => {
+        if (disabled) return
         if (!focusedAgentId) return
 
         const isSpace =
@@ -125,13 +130,14 @@ export const useKeyboardHandlers = ({
           })
         }
       },
-      [focusedAgentId, setCollapsedAgents],
+      [focusedAgentId, setCollapsedAgents, disabled],
     ),
   )
 
   useKeyboard(
     useCallback(
       (key) => {
+        if (disabled) return
         if (key.name === 'escape' && focusedAgentId) {
           if (
             'preventDefault' in key &&
@@ -144,14 +150,16 @@ export const useKeyboardHandlers = ({
           inputRef.current?.focus()
         }
       },
-      [focusedAgentId, setFocusedAgentId, setInputFocused, inputRef],
+      [focusedAgentId, setFocusedAgentId, setInputFocused, inputRef, disabled],
     ),
   )
 
   // Handle chat history navigation
   useKeyboard(
     useCallback(
       (key) => {
+        if (disabled) return
+
         const isUpArrow =
           key.name === 'up' && !key.ctrl && !key.meta && !key.shift
         const isDownArrow =
@@ -174,13 +182,15 @@ export const useKeyboardHandlers = ({
           navigateDown()
         }
       },
-      [historyNavUpEnabled, historyNavDownEnabled, navigateUp, navigateDown],
+      [historyNavUpEnabled, historyNavDownEnabled, navigateUp, navigateDown, disabled],
     ),
   )
 
   useKeyboard(
     useCallback(
       (key) => {
+        if (disabled) return
+
         const isShiftTab =
           key.shift && key.name === 'tab' && !key.ctrl && !key.meta
 
@@ -195,7 +205,7 @@ export const useKeyboardHandlers = ({
 
         toggleAgentMode()
       },
-      [toggleAgentMode],
+      [toggleAgentMode, disabled],
     ),
   )
 }