Skip to content

Agents - fix resource order in the multi-file diff editor#311639

Merged
lszomoru merged 1 commit intomainfrom
lszomoru/tense-armadillo
Apr 21, 2026
Merged

Agents - fix resource order in the multi-file diff editor#311639
lszomoru merged 1 commit intomainfrom
lszomoru/tense-armadillo

Conversation

@lszomoru
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 21, 2026 09:18
@lszomoru lszomoru enabled auto-merge (squash) April 21, 2026 09:18
@lszomoru lszomoru self-assigned this Apr 21, 2026
@lszomoru lszomoru added this to the 1.118.0 milestone Apr 21, 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 Agents Sessions “Changes” view to open the multi-file diff editor with a deterministic, path-sorted resource order.

Changes:

  • Add a path-based sort for the resources passed to _workbench.openMultiDiffEditor.
  • Add validation to only pass reveal when the target resource is present in the multi-diff resources list.
Show a summary per file
File Description
src/vs/sessions/contrib/changes/browser/changesView.ts Sorts multi-diff resources by path before opening the multi-file diff editor and adjusts reveal handling.

Copilot's findings

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

Comment thread src/vs/sessions/contrib/changes/browser/changesView.ts
@lszomoru lszomoru merged commit 1f3b864 into main Apr 21, 2026
30 checks passed
@lszomoru lszomoru deleted the lszomoru/tense-armadillo branch April 21, 2026 10:10
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