perf(frontend): comprehensive performance optimizations #252
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… performance Replace multiple independent `useStore` calls in `UnifiedTimeline` with a combined selector, reducing subscription count and improving efficiency. Add rigorous performance tests to verify stable references and memoization across selectors. Update async tests for cross-session isolation.
…te handling Introduce `agentStreamingBuffer` for efficient text delta management and reduce redundant string concatenation in agent streaming. Refactor `processedToolRequests` to track requests per session. Add tests for streaming buffer, session-specific tool tracking, and unified input callback stability.
- Add throttled resize handlers to prevent excessive state updates during drag - Implement lazy loading for heavy components (SyntaxHighlighter, CodeMirror languages) - Optimize timeline selectors with shallow comparison to prevent unnecessary re-renders - Add settings caching to avoid redundant Tauri calls - Extract keyboard handler to reduce App.tsx effect dependencies - Refactor PaneLeaf with granular selectors for better memoization - Optimize SessionBrowser with debounced search and proper memoization - Add WebGL context loss recovery handling for Terminal - Batch ThemeManager terminal updates to reduce re-renders - Configure Vite manual chunks for better bundle splitting - Extract isMockBrowserMode to separate module for tree-shaking - Add comprehensive test coverage for new utilities and optimizations
- Fix unused imports and variables in test files - Add missing required properties (sessionId, startedAt, warnings, workingDirectory) - Update Zustand v5 usage to use useShallow instead of shallow as second argument - Fix non-null assertions to use conditional checks - Remove unused messagesContainerRef from SessionBrowser - Change div with role="button" to semantic button element - Fix hook being called after early return in VirtualizedTimeline
xlyk
force-pushed
the
frontend-improvements
branch
from
c48123d to
27927a0
Compare
itsmostafa
approved these changes
Feb 4, 2026
- Mark 33 of 82 issues as FIXED (40% complete) - All 5 SEVERE issues resolved (100%) - 25 of 26 HIGH priority issues resolved (96%) - Document new files created (11 selectors/hooks) - List major architectural improvements - Identify remaining critical path and quick wins
Add sound_enabled setting for in-app notification sounds independent of native OS notifications. Sounds are generated programmatically using Web Audio API for reliable cross-platform playback. Also refactors provider state management into useProviderSettings hook and consolidates AgentMessage derived state to reduce cascading recalculations during streaming.
…tence Import provider configuration utilities, API key getters, session initialization, and notification helpers to support upcoming model selection functionality.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
isMockBrowserModeto separate module to prevent mock code inclusion in productionTest plan
pnpm test:coverageto verify new tests pass