Fixes #366: Add chat deletion controls - #401
Conversation
🦋 Changeset detectedLatest commit: d17add2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit ce273ee. Configure here.
|
@rohanmalhotracodes Thanks for raising the PR but only a few changes were required to make the delete work.
|
Screen.Recording.2026-08-24.at.5.25.57.PM.mov |
That's a good idea @rohanmalhotracodes, we are already working on this. I got the designs reviewed by our senior product designer and he is trying to keep things simple
|
|
@govindavashishtha |
|
Moreover, when the chat sidebar is minimized the arrow moves below the logo shouldn't it be something like the logo emains there and hovering on it show a arrow to open the sidebar |
I will not be adding it to the header as mentioned by @govindavashishtha although I am open to adding it in the future when you guys allow me to. Also I have kept the fix minimal as I was too |






Summary
Closes #366
Adds delete controls for persisted chat sessions from both the active chat header and chat history, with confirmation before deletion.

ss attached
Changes
DeleteChatButtonfor the current persisted chat.How was this tested?
pnpm buildpnpm testpnpm typecheckpnpm lint:cipnpm format:checkhttp://localhost:3000/Checklist
pnpm build,pnpm test,pnpm typecheck,pnpm lint:ci, andpnpm format:checkpass locallypackages/trueforge-sdk,.github/fern/openapi/openapi.json,docs/openapi.json) — fork PRs omit SDK regen; maintainers regenerate after merge.env.exampleupdated if configuration or behavior changedNote
Medium Risk
Introduces a destructive delete path to persisted sessions; impact is limited to explicit user actions and is gated on
server.deleteSessionbeing defined.Overview
Implements remote chat session deletion in the TrueForge harness server adapter by adding
deleteSession, which calls the SDKclient.sessions.deleteAPI so history delete actions (via assistant-uiThreadListItemPrimitive.Delete) actually remove persisted sessions when the server supports it.Chat history UI tweaks in
ThreadListContainer: refreshes delete menu styling (rounded corners, medium weight, failure-tinted hover states), portals the desktop “more” menu into the nearest.aui-theme-rootfor correct theming, and appliesfont-sans-flexon the dropdown panel.Adds a unit test that asserts the adapter issues
DELETE /api/v1/sessions/{id}. Includes a patch changeset for@truefoundry/trueforge-ui.Reviewed by Cursor Bugbot for commit d17add2. Bugbot is set up for automated code reviews on this repo. Configure here.