Skip to content

Agents - remove branch name from explorer view - #343289

Merged
Ladislau Szomoru (lszomoru) merged 2 commits into
mainfrom
lszomoru/agents/remove-branch-name-from-explorer-view
Oct 21, 2026
Merged

Ladislau Szomoru (lszomoru) merged 2 commits into
mainfrom
lszomoru/agents/remove-branch-name-from-explorer-view

Conversation

@lszomoru

Copy link
Copy Markdown
Member

This pull request removes the branch name from the root folder display in the Agents window Explorer, addressing user confusion since the branch name is already shown in the title bar.

Changes made:

  • Updated workspaceFolderManagement.ts to request the short form of the folder label instead of the long form that included the branch name.
  • Modified the test in workspaceFolderManagement.test.ts to reflect the change in folder naming, ensuring it now uses repository names instead of session-branch names.

Validation:

  • The changes have been validated with Node 24.22.0, and all relevant unit tests and ESLint checks have passed successfully.

Copilot AI balanced review requested due to automatic review settings October 21, 2026 18:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

3 open issues
What changed in this PR

Removes the branch name suffix from the Agents Explorer root folder label by switching to the “short” workspace-folder label, and updates the corresponding unit test expectations.

Changes:

  • Request a non-verbose (short) workspace folder label when constructing workspace folder entries.
  • Update unit test to expect short labels (no (<branch>) suffix) and corresponding update events.
File Description
src/​vs/​sessions/​contrib/​workspace/​browser/​workspaceFolderManagement.ts Stops requesting the verbose/long workspace folder label to avoid including branch names.
src/​vs/​sessions/​contrib/​workspace/​test/​browser/​workspaceFolderManagement.test.ts Updates expectations to reflect short labels (without branch names).

Review effort: Lite

Make the workspace folder label stub append the branch name for verbose requests, mirroring the real service, so the test fails if the explorer root starts showing the branch name again.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@lszomoru
Ladislau Szomoru (lszomoru) merged commit a875c77 into main Oct 21, 2026
58 checks passed
@lszomoru
Ladislau Szomoru (lszomoru) deleted the lszomoru/agents/remove-branch-name-from-explorer-view branch October 21, 2026 18:56
@vs-code-engineering vs-code-engineering Bot added this to the 1.144.0 milestone Oct 21, 2026
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.

4 participants