move cursor clamping into setter

Author: charleslien

cli/src/components/multiline-input.tsx

@@ -140,13 +140,6 @@ export const MultilineInput = forwardRef<
     [],
   )
 
-  // Sync cursor when value changes externally
-  useEffect(() => {
-    onChange((prev) => ({
-      ...prev,
-      cursorPosition: clamp(cursorPosition, 0, value.length),
-    }))
-  }, [value.length, cursorPosition])
 
   useOpentuiPaste(
     useCallback(

cli/src/state/chat-store.ts

@@ -2,6 +2,8 @@ import { enableMapSet } from 'immer'
 import { create } from 'zustand'
 import { immer } from 'zustand/middleware/immer'
 
+import { clamp } from '../utils/math'
+
 import type { ChatMessage } from '../types/chat'
 import type { AgentMode } from '../utils/constants'
 
@@ -120,7 +122,7 @@ export const useChatStore = create<ChatStore>()(
               })
             : value
         state.inputValue = text
-        state.cursorPosition = cursorPosition
+        state.cursorPosition = clamp(cursorPosition, 0, text.length)
         state.lastEditDueToNav = lastEditDueToNav
       }),