Skip to content

Agents - Changes view pickers should work with multiple chats in a session#309479

Merged
lszomoru merged 2 commits intomainfrom
lszomoru/joyous-turtle
Apr 13, 2026
Merged

Agents - Changes view pickers should work with multiple chats in a session#309479
lszomoru merged 2 commits intomainfrom
lszomoru/joyous-turtle

Conversation

@lszomoru
Copy link
Copy Markdown
Member

No description provided.

@lszomoru lszomoru enabled auto-merge (squash) April 13, 2026 14:44
Copilot AI review requested due to automatic review settings April 13, 2026 14:44
@lszomoru lszomoru self-assigned this Apr 13, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 13, 2026

Screenshot Changes

Base: 8f57c3e1 Current: e2dbd22e

Changed (2)

editor/inlineChatAffordance/InlineChatOverlay/Light
Before After
before after
agentSessionsViewer/NeedsInput/Light
Before After
before after

@lszomoru lszomoru added this to the 1.116.0 milestone Apr 13, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Sessions “Changes” view model to better support session groups containing multiple chats, so that change/version pickers can derive the correct checkpoint references from the most recently completed chat rather than assuming a single chat per session.

Changes:

  • Add chat sorting by lastTurnEnd to select the most recently completed chat when deriving the active session’s lastCheckpointRef.
  • Switch local repository diff computation to use diffBetweenWithStats2() with a single range string (<ref1>..<ref2>).
Show a summary per file
File Description
src/vs/sessions/contrib/changes/browser/changesViewModel.ts Picks the “last checkpoint” based on the latest-finished chat in multi-chat sessions; updates git diff call to use the newer diffBetweenWithStats2 API.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 2

@lszomoru lszomoru disabled auto-merge April 13, 2026 14:57
@lszomoru lszomoru merged commit 90c4da9 into main Apr 13, 2026
26 checks passed
@lszomoru lszomoru deleted the lszomoru/joyous-turtle branch April 13, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants