Add chat mode controls and thread search UI to repoless chat shell - #155
Conversation
- Display chat mode controls in discuss shell when visibility hook allows it - Add thread search dialog state wiring for opening/selecting threads - Trigger new-thread flow from control actions without changing thread behavior
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
More reviews will be available in 53 minutes and 26 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate. For paid Pro and Pro+ PR reviews, CodeRabbit uses rolling per-developer review limits. Reviews become available again as older review attempts age out of the rolling limit window. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough
ChangesChat Navigation Controls Integration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/components/repoless-chat-shell.tsx (1)
239-246: 🚀 Performance & Scalability | 🔵 Trivial | ⚡ Quick winAvoid always mounting
ThreadSearchDialogto prevent background thread-list queries.Because this dialog is rendered unconditionally,
listRepolessThreadscan subscribe/fetch even when closed. Conditionally mount it whenisThreadSearchOpenis true.♻️ Proposed change
- <ThreadSearchDialog - open={isThreadSearchOpen} - onOpenChange={setIsThreadSearchOpen} - repositoryId={null} - mode={chatMode} - selectedThreadId={urlThreadId} - onSelectThread={handleSelectThread} - /> + {isThreadSearchOpen ? ( + <ThreadSearchDialog + open={isThreadSearchOpen} + onOpenChange={setIsThreadSearchOpen} + repositoryId={null} + mode={chatMode} + selectedThreadId={urlThreadId} + onSelectThread={handleSelectThread} + /> + ) : null}🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/components/repoless-chat-shell.tsx` around lines 239 - 246, The ThreadSearchDialog component is being mounted unconditionally, which causes unnecessary background queries like listRepolessThreads to execute even when the dialog is closed. Wrap the ThreadSearchDialog component in a conditional render that only mounts it when isThreadSearchOpen is true, using standard React conditional rendering with the && operator or a ternary statement.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@src/components/repoless-chat-shell.tsx`:
- Around line 239-246: The ThreadSearchDialog component is being mounted
unconditionally, which causes unnecessary background queries like
listRepolessThreads to execute even when the dialog is closed. Wrap the
ThreadSearchDialog component in a conditional render that only mounts it when
isThreadSearchOpen is true, using standard React conditional rendering with the
&& operator or a ternary statement.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: b9c32337-e531-4134-9fbd-d85a10f6ab60
📒 Files selected for processing (1)
src/components/repoless-chat-shell.tsx
- Mount `ThreadSearchDialog` only when `isThreadSearchOpen` is true - Avoid unnecessary rendering/effects when the thread search modal is closed
1 participant
Summary
ChatModeControlsintoRepolessChatShellso navigation controls render for discuss mode when visibility rules allow.ThreadSearchDialogwithmode={chatMode},repositoryId={null}, and current thread selection viaurlThreadId.onSearchThreadsto open the dialog andonNewThreadto reuse the existing thread-creation flow.handleSelectThreadpath so thread selection from search continues through the normal navigation flow.Testing
Summary by CodeRabbit
Release Notes