[codex] fix(desktop): preserve chat cursor position

[Kitenite]

Summary

This change fixes the desktop chat composer moving the caret to the start of the prompt while typing.

What Changed

• updated the chat prompt focus hook to depend on the stable textInput.focus callback instead of the full textInput controller object
• kept the existing behavior of focusing the prompt when the pane becomes active, without re-focusing on ordinary input rerenders

Root Cause

useFocusPromptOnPane depended on the entire textInput object from the prompt controller. That object is recreated whenever the controlled prompt value changes, so the effect ran again on each keystroke while the pane was focused. Re-focusing the textarea during typing reset the selection and made the cursor jump.

Impact

Typing in the desktop chat composer should now preserve the caret position across rerenders, including edits in the middle of existing text.

Validation

• cd apps/desktop && bun run typecheck

---

Summary by cubic

Fixes the desktop chat composer cursor jumping to the start while typing. The focus hook now uses the stable textInput.focus callback instead of the full controller, so the prompt keeps its caret on rerenders and only auto-focuses when the pane becomes active.

^{Written for commit c939232. Summary will update on new commits.}

Summary by CodeRabbit

• Refactor
  • Improved chat input focus handling efficiency in the interface.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dac16e1a-5985-4e5b-ac7d-9170adf36d56

📥 Commits

Reviewing files that changed from the base of the PR and between 1ac52d4 and c939232.

📒 Files selected for processing (1)

• apps/desktop/src/renderer/components/Chat/ChatInterface/hooks/useFocusPromptOnPane.ts

---

📝 Walkthrough

Walkthrough

A React hook was refactored to cache the textInput.focus method into a local variable before invoking it. The useEffect dependency array was updated to reference the cached focus function instead of the entire textInput object, changing from [isFocused, textInput] to [focusPrompt, isFocused].

Changes

Cohort / File(s)	Summary
Focus Method Caching apps/desktop/src/renderer/components/Chat/ChatInterface/hooks/useFocusPromptOnPane.ts	Extracted textInput.focus into a focusPrompt variable and updated the useEffect dependency array from [isFocused, textInput] to [focusPrompt, isFocused].

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hopped through code so neat,
Cached the focus, made it sweet!
One small change, a methodly treat,
Dependencies now compete. 🐰✨

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-chat-gui-cursor-position-issue-on-rerender

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ⚠️ Neon database branch
• ⚠️ Electric Fly.io app

Thank you for your contribution! 🎉

[cubic-dev-ai]

No issues found across 1 file