FE-1808: Separate stock and Brunch assistant presentations - #9883
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
PR SummaryMedium Risk Overview
Tests are reorganized: voice-dock coverage moves next to those components, and many assertions run as paired stock/brunch parameterised suites. Tab attention tests now look for Reviewed by Cursor Bugbot for commit 560d796. Bugbot is set up for automated code reviews on this repo. Configure here. |
…unch presentations Restore main's stock transcript, tool rows, tabs, scrolling and composer behind the default presentation, and give the stock and Brunch transcripts, tool lists, composer and status row their own files. Stopped in-progress stock tools show Cancelled instead of a spinner. Co-authored-by: Cursor <cursoragent@cursor.com>
fe51762 to
f4454d0
Compare
Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit a822cee. Configure here.
A failed stream never ends its reasoning part, so the stock disclosure stayed open with a running timer after an error. Co-authored-by: Cursor <cursoragent@cursor.com>

🌟 What is the purpose of this PR?
The Brunch conversation stack (#9829) was meant to change only the Brunch assistant, but it also changed the stock assistant that every other host and npm consumer of
@hashintel/petrinautsees. Both presentations shared one large component, so Brunch changes leaked into stock.This PR restores the stock assistant and gives each presentation its own components.
🔗 Related links
🚫 Blocked by
Nothing. Targets
main. #9886 is stacked on top.🔍 What does this change?
presentation: "stock", the default) matchesmainfrom before FE-1793: Unify Chat and Voice conversation presentation #9829: message order, tool rows, styling, tabs, scrolling and composer. Two fixes stay: a tool stopped mid-run shows "Cancelled" instead of spinning, and reasoning stops looking live once its reply fails.Pre-Merge Checklist 🚀
🚢 Has this modified a publishable library?
This PR:
📜 Does this require a change to the docs?
The changes in this PR:
🕸️ Does this require a change to the Turbo Graph?
The changes in this PR:
isBrunchChatchecks remain in the container.🐾 Next steps
isBrunchChatchecks out of the container.ai-assistant-panel.tsxand the stories importing from the privateai-assistant-contents/folder (they do onmaintoo).🛡 What tests cover this?
❓ How to test this?
mainfrom before FE-1793: Unify Chat and Voice conversation presentation #9829 (4561ee8e7b^) next to it.main, and the stopped tool shows "Cancelled".main.