fix(brainbar): title injection bursts from chunk content#337
Conversation
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.
|
Warning Review limit reached
More reviews will be available in 38 minutes and 23 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ 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 |
|
@codex review |
|
@coderabbitai review |
|
You need to increase your spend limit or enable usage-based billing to run background agents. Go to Cursor |
✅ Actions performedReview triggered.
|
|
Codex Review: Didn't find any major issues. Delightful! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Verification
Follow-up to Item F Fix 2 visual verification; PR #336 exposed that burst titles could still display source prompt text in live BrainBar.
Note
Low Risk
Presentation-only change in BrainBar injection burst titles; grouping and data paths are unchanged.
Overview
Injection burst card titles now show the first retrieved chunk’s summary/content (via existing
previewChunks/displayText) instead of repeating the source/trigger prompt. When no chunk text is available, the fallback is shortened to “N chunk(s) injected” without quotingtopicOrSource.Burst grouping still keys on session + query (
topicOrSource); only the visible title string changed. A regression test covers the case where the query is a long orchestrator-style prompt but chunk content should drive the title.Reviewed by Cursor Bugbot for commit 48177f1. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix
Burst.summaryTitleto show first chunk's display text instead of source/promptBurst.summaryTitlein InjectionPresentation.swift now returns the first preview chunk'sdisplayTextwhen it is non-empty.'N chunk(s) injected'instead of'N chunk(s) injected from "<source>"'.Macroscope summarized 48177f1.