perf(platform): optimize dev server and UI improvements#255
Merged
Conversation
- Add optimizeDeps.include for heavy dependencies to reduce dev server module requests from 236 to ~15 bundled chunks - Add title header to automations layout for consistency with other pages - Remove key prop from ConversationsClient that caused unnecessary remounts Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Update ConversationsClientSkeleton to match the actual two-panel layout with proper list and detail panel skeletons for better loading UX. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Contributor
📝 WalkthroughWalkthroughThis pull request introduces structural and configuration changes across four files. The conversations client skeleton component is restructured to display a two-panel layout during loading with a sticky header, message list, and editor skeleton. The automations route is enhanced with a StickyHeader, AdaptiveHeader, and LayoutErrorBoundary composition wrapping an Outlet. A React key prop is removed from the ConversationsClient component in the status route. The Vite configuration is updated to explicitly include dependencies in the optimization pre-bundling step. Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Comment |
6 tasks
4 tasks
yannickmonney
pushed a commit
that referenced
this pull request
Apr 8, 2026
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
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
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
optimizeDeps.includeto Vite config for heavy dependencies, reducing dev server module requests from 236 individual files to ~15 bundled chunks (massive dev performance improvement)keyprop from ConversationsClient that caused unnecessary full remounts on filter changesTest plan
🤖 Generated with Claude Code
Summary by CodeRabbit
Improvements
Chores
✏️ Tip: You can customize this high-level summary in your review settings.