Render chat turn changes as a checkpoint-style summary - #324929
Merged
Conversation
Restyle the chat turn pills part to match the checkpoint file changes summary: a 'N files changed +ins -del' header with a View All File Changes action, an optional inline 'preview <file>' action for the first previewable file, and an expandable list of changed files. Each markdown/HTML row in the list gets a labelless 'Preview' action that opens the file in the Markdown preview or the integrated browser. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
lszomoru
previously approved these changes
Jul 8, 2026
Contributor
Screenshot ChangesBase: Added (22)
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR restyles the per-turn “changes/preview” area in Chat to match the checkpoint-style file changes summary, and extracts shared rendering so both summaries use the same structure and file list.
Changes:
- Reworked
ChatTurnPillsContentPartto render a checkpoint-style header + optional preview action + expandable changed-files list with per-row preview actions. - Extracted the collapsible changed-files list into a shared
renderChangesSummaryFileListhelper, reused by both checkpoint summaries and turn summaries. - Expanded component fixtures to cover the new standalone and in-chat states for the turn changes summary.
Show a summary per file
| File | Description |
|---|---|
| src/vs/workbench/test/browser/componentFixtures/chat/chatWidget.fixture.ts | Adds fixture plumbing for per-turn file diffs and enables turn-status-pill rendering in full chat fixtures. |
| src/vs/workbench/test/browser/componentFixtures/chat/chatTurnPills.fixture.ts | Adds standalone + in-chat fixtures for the new checkpoint-style turn summary across multiple states. |
| src/vs/workbench/contrib/chat/browser/widget/media/chat.css | Adds styling for the new turn-summary header preview action and per-row action bar in the file list. |
| src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatTurnPillsPart.ts | Replaces “floating pills” with checkpoint-style summary markup, integrates preview + per-row preview actions, and reuses shared file list rendering. |
| src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatChangesSummaryPart.ts | Extracts shared renderChangesSummaryFileList and adds optional per-row actions via an ActionBar. |
Review details
- Files reviewed: 5/5 changed files
- Comments generated: 3
- Review effort level: Low
- Stub IAgentHostService.getSubscription in chat fixtures so agent-host chat widgets (turn changes summary) render without crashing (the generic config chips lane opens a session subscription). - Use the canonical agent-host session scheme for the in-chat fixtures. - Reference CHANGES_SUMMARY_ELEMENT_HEIGHT from the list delegate. - Hide the preview separator in preview-only mode. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
TylerLeonhardt
approved these changes
Jul 8, 2026
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Restyles the chat turn pills part so it no longer looks like floating pills and instead matches the checkpoint file changes summary.
What changed
N files changed +ins -delheader with a View All File Changes action, an optional inline preview action for the first previewable (Markdown/HTML) file the turn produced, and an expandable list of changed files. Reuses the checkpoint summary's structure/classes so the two look identical.renderChangesSummaryFileListhelper used by both the checkpoint summary and the turn summary.Previewaction that opens the file in the Markdown preview or the integrated browser.entire chatfixtures rendering the summary in-situ via the real chat list renderer.Validation
typecheck-client, ESLint, and repo stylelint all pass on the changed files.--no-verifyper author approval.Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com