Fix/issue 2189 focus lost alt tab

[MuriloFP]

PR Title: fix: restore focus to chat text box after Alt+Tab (#2189)

---

Related GitHub Issue

Closes: #2189

Roo Code Task Context (Optional)

N/A

Description

This PR addresses the issue where users lose focus from the Roo Code chat text box when switching away from VSCode using Alt+Tab and then switching back.

Key implementation details:

• Added a window focus event listener in ChatView.tsx that triggers when the window regains focus
• The listener checks if the text area should be focusable using existing conditions: !isHidden && !sendingDisabled && !enableButtons
• Used setTimeout with 0ms delay to ensure focus happens after the event loop cycle for better reliability across different platforms
• Added proper cleanup of the event listener on component unmount to prevent memory leaks

Design choices:

• Reused existing focus conditions to maintain consistency with current behavior
• The window focus event approach was chosen as it specifically targets the Alt+Tab scenario without interfering with other focus behaviors
• The setTimeout ensures the focus action happens after VSCode's webview has fully processed the window focus event

Test Procedure

Unit tests added:

• Added tests in ChatView.spec.tsx to verify window focus event listener setup and cleanup
• Added test to verify focus restoration when conditions are met
• Tests pass with npx vitest run src/components/chat/__tests__/ChatView.spec.tsx

Manual testing steps:

1. Open VSCode with Roo Code extension
2. Start typing in the Roo Code chat text box
3. Press Alt+Tab to switch to another application
4. Press Alt+Tab again to switch back to VSCode
5. Verify that the cursor is back in the chat text box and you can continue typing immediately

Testing environment: Windows 11, VSCode 1.95.0, Roo Code v3.11.3

Additional test scenarios:

• Verified focus is NOT restored when the chat view is hidden
• Verified focus is NOT restored when a tool is running (sending disabled)
• Verified focus is NOT restored when confirmation buttons are shown

Pre-Submission Checklist

• Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
• Scope: My changes are focused on the linked issue (one major feature/fix per PR).
• Self-Review: I have performed a thorough self-review of my code.
• Testing: New and/or updated tests have been added to cover my changes (if applicable).
• Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
• Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

N/A - No UI changes. This is a focus behavior fix that doesn't alter the visual appearance.

Documentation Updates

• No documentation updates are required.

Additional Notes

This fix specifically addresses the VSCode webview focus loss issue reported by multiple users. The implementation is minimal and follows React best practices for effect handling and focus management.

Get in Touch

@MuriloFP

---

Important

Adds window focus event listener in ChatView.tsx to restore focus to chat text box after Alt+Tab, with tests verifying functionality.

• Behavior:
  • Adds window focus event listener in ChatView.tsx to restore focus to chat text box after Alt+Tab.
  • Uses setTimeout to ensure focus restoration after event loop cycle.
  • Cleans up event listener on component unmount.
• Tests:
  • Adds tests in ChatView.spec.tsx to verify window focus event listener setup and cleanup.
  • Tests focus restoration when conditions are met.
  • Tests pass with npx vitest run src/components/chat/__tests__/ChatView.spec.tsx.

^{This description was created by for e63ed48. You can customize this summary. It will automatically update as commits are pushed.}

[daniel-lxs]

I'm unable to reproduce the issue on main, Would it be good to demonstrate the issue to check if this PR fixes it?

[MuriloFP]

Seems the issue was fixed on main somehow... Closing this PR.