Skip to content

Comments

Sync desktop/ from omi-desktop (474e1e1f)#4952

Open
m13v wants to merge 1 commit intomainfrom
sync/desktop-20260222-171223
Open

Sync desktop/ from omi-desktop (474e1e1f)#4952
m13v wants to merge 1 commit intomainfrom
sync/desktop-20260222-171223

Conversation

@m13v
Copy link
Collaborator

@m13v m13v commented Feb 22, 2026

Automated sync from omi-desktop at 474e1e1f.

This PR was created by the sync pipeline.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request is an automated sync that brings in a substantial set of improvements across the application. The changes are of high quality and address several key areas.

Key improvements include:

  • Architecture: The chat functionality has been significantly refactored by unifying the agent-bridge and acp-bridge into a single, more robust ACPBridge. This includes a much-improved OAuth flow for user authentication.
  • Reliability and Performance: There are numerous critical fixes that enhance stability. The move to asynchronous audio capture prevents main thread blocking. The VideoChunkEncoder is now more robust with watchdogs and improved termination logic for the ffmpeg process, preventing hangs under memory pressure. The ResourceMonitor now includes a crucial auto-restart mechanism for extreme memory usage, which will prevent system-wide unresponsiveness. Error handling has also been improved in the Gemini client and Sentry reporting.
  • Features and UX: The Tasks feature has been massively overhauled with drag-and-drop reordering (including cross-category moves), inline task creation, and a new background "Investigate" feature. The onboarding flow has also been redesigned for a better user experience. Many smaller bugs related to UI state, window resizing, and scrolling have also been addressed.

Overall, this is an excellent batch of changes that significantly improves the application's architecture, reliability, and user experience. I have not found any issues of high or critical severity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant