Conversation
## Frontend - New Components - Add `sidebar-thread-list.tsx` - Extracted thread list from nav-agents for better modularity - Add `mode-indicator.tsx` - New model selection indicator (Basic/Advanced mode toggle) - Add `StreamingLoader.tsx` - Shared streaming loader with cycling phrases animation ## Frontend - UI Improvements - Refactor `kortix-loader.tsx` - Replace Lottie animation with lightweight CSS spinner - Enhance `FileBrowserView.tsx` - Add file type detection, image thumbnails, and hide internal folders - Update `PanelHeader.tsx` - Improved panel header styling - Improve `NavigationControls.tsx` - Better navigation UX ## Frontend - Component Refactoring - Refactor `nav-agents.tsx` and `nav-agents-view.tsx` - Extract thread list logic - Update `sidebar-left.tsx` - Integrate new thread list component - Clean up `chat-input.tsx`, `floating-tool-preview.tsx`, `unified-config-menu.tsx` - Update various tool views (Apify, Canvas, FileOperation, Presentation) ## Frontend - Fixes & Polish - Update `library/[projectId]/page.tsx` - Streamlined library page - Minor fixes in ThreadContent, ThreadSkeleton, MeshGradientLoader - Update stores (file-viewer, thread-navigation) with minor improvements ## Backend - Add `file_name_generator.py` - Smart LLM-powered filename generation for media - Update `sb_image_edit_tool.py` - Integrate smart filename generation - Update `prompt.py` - Prompt improvements ## Desktop & Mobile - Update `main.js` - Desktop app improvements - Clean up mobile KortixComputer component ## Database - Add user avatars bucket migration
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| setOpenMobile(false); | ||
| } | ||
| }, [pathname, searchParams, isMobile, setOpenMobile]); | ||
| }, [pathname, searchParams, isMobile, setOpenMobile, isLibraryChatSwitch]); |
There was a problem hiding this comment.
Sidebar closes on mobile despite library-chat switch flag
Medium Severity
The isLibraryChatSwitch flag intended to prevent sidebar collapse on mobile when switching between library and chat views doesn't work correctly. Because isLibraryChatSwitch is included in the useEffect dependency array, when the effect sets it to false and returns early, the effect immediately runs again (triggered by the state change). On this second run, isLibraryChatSwitch is now false, so setOpenMobile(false) gets called anyway, closing the sidebar. The flag should either be removed from the dependency array or converted to a ref to avoid re-triggering the effect.
| } | ||
| } | ||
| return; | ||
| } |
There was a problem hiding this comment.
Cmd+click in library mode opens duplicate tabs
Medium Severity
When Cmd/Ctrl+clicking on an expanded thread item in library mode, two browser tabs open instead of one. The handleItemClick function calls window.open(url, '_blank') for library mode but returns without calling e.preventDefault(). Since the expanded threads within the Collapsible use a Next.js Link component (lines 852-861), the browser's default Cmd+click behavior also fires, opening the Link's href (library URL) in a second tab. This doesn't affect the single-thread cards which correctly use a div wrapper for library mode.
Additional Locations (1)
1 participant
Frontend - New Components
sidebar-thread-list.tsx- Extracted thread list from nav-agents for better modularitymode-indicator.tsx- New model selection indicator (Basic/Advanced mode toggle)StreamingLoader.tsx- Shared streaming loader with cycling phrases animationFrontend - UI Improvements
kortix-loader.tsx- Replace Lottie animation with lightweight CSS spinnerFileBrowserView.tsx- Add file type detection, image thumbnails, and hide internal foldersPanelHeader.tsx- Improved panel header stylingNavigationControls.tsx- Better navigation UXFrontend - Component Refactoring
nav-agents.tsxandnav-agents-view.tsx- Extract thread list logicsidebar-left.tsx- Integrate new thread list componentchat-input.tsx,floating-tool-preview.tsx,unified-config-menu.tsxFrontend - Fixes & Polish
library/[projectId]/page.tsx- Streamlined library pageBackend
file_name_generator.py- Smart LLM-powered filename generation for mediasb_image_edit_tool.py- Integrate smart filename generationprompt.py- Prompt improvementsDesktop & Mobile
main.js- Desktop app improvementsDatabase
Note
SidebarThreadListconsolidates thread listing (grouping, pagination, deletion, new-chat) and powers bothnav-agentsandnav-agents-viewviamode(chats/library).sidebar-left.tsxnow path‑aware and switches betweenlibraryandchatroutes for the same project without collapsing the sidebar.library/[projectId]/page.tsxreplaced complex bespoke UI withFileBrowserView, resets store path to/workspace, and provides a handler to jump to the project’s chat.setting-up, maintenance, onboarding progress,ThreadContent,ThreadSkeleton, andMeshGradientLoaderfor consistent visuals.floating-tool-previewsimplified to compact status indicator;unified-config-menutrimmed to agent/worker controls;chat-inputremoves inline config dropdown usage.public/kortix-logomark-white.svg.Written by Cursor Bugbot for commit bc1fa4d. This will update automatically on new commits. Configure here.