fix(web): restore thread ID copy in the default sidebar - #4563
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
ApprovabilityVerdict: Approved 143da7b Restores a 'Copy thread ID' menu item using existing clipboard copy patterns. Self-contained UI change with proper test coverage and no security or schema implications. You can customize Macroscope's approvability policy. Learn more. |
165b14f to
3d0934c
Compare
Dismissing prior approval to re-evaluate 3d0934c
3d0934c to
143da7b
Compare
Dismissing prior approval to re-evaluate 143da7b
Summary
Copy Thread IDto the default sidebar thread context menuVerification
pnpm exec vp lint apps/web/src/components/Sidebar.tsx --report-unused-disable-directivespnpm exec vp fmt --check apps/web/src/components/Sidebar.tsxpnpm exec vp run --filter @t3tools/web typechecktest-t3-apppass: enabled default sidebar, opened a thread context menu, selectedCopy Thread ID, and verified the exact thread UUID was copied with a success toastNote
Low Risk
Clipboard-only UI with no server or auth changes; behavior mirrors existing copy-path/copy-branch actions.
Overview
Restores Copy thread ID on the shared thread context menu (
buildThreadActionMenuItems), so sidebar and chat-header menus match v1 again.Choosing the action copies the thread UUID via
useCopyToClipboardand shows the same success/error toasts as copy path/branch. Wiring is duplicated inSidebar.tsxanduseThreadActionMenu.tsso both entry points handle the new menu id.Reviewed by Cursor Bugbot for commit 143da7b. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Restore 'Copy thread ID' action in sidebar and chat header thread menus
Re-adds the
copy-thread-idmenu item tobuildThreadActionMenuItemsin threadActionMenu.logic.ts and wires up the clipboard action in both Sidebar.tsx and useThreadActionMenu.ts. Both handlers useuseCopyToClipboardwith success and error toasts.Macroscope summarized 143da7b.