Add Agents Window taskbar shortcut and icons - #335333
Merged
Dmitriy Vasyura (dmitrivMS) merged 9 commits intoSep 10, 2026
Merged
Conversation
Add an Agents Window task to the Windows jump list using the existing --agents launch mode and branded icon.\n\nRefs #335321 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot started reviewing on behalf of
Dmitriy Vasyura (dmitrivMS)
September 9, 2026 18:56
View session
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The task uses the existing launch path, and the packaged icon path and frames are valid.
Review tier: Balanced
Findings: None
What changed in this PR
Adds direct Agents Window access from the Windows taskbar jump list.
Changes:
- Adds a localized Agents Window task using
--agents. - Packages a validated multi-resolution Agents icon.
| File | Description |
|---|---|
workspacesHistoryMainService.ts |
Registers the jump-list task. |
gulpfile.vscode.ts |
Packages the icon on Windows. |
resources/win32/agents.ico |
Provides the branded task icon. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Give the Windows taskbar button the same visual identity as the jump-list task. Refs #335321 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use the branded Agents icon for Agents Window entries while preserving existing file and workspace icons for other windows. Fixes #315734 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Dmitriy Vasyura (dmitrivMS)
changed the title
Carry the authoritative Agents window flag into the window picker instead of comparing incompatible native and renderer workspace URI schemes. Reuse the distro-mixed sessions.ico path so native Windows icons match each product quality. Fixes #315734 Refs #335321 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Dmitriy Vasyura (dmitrivMS)
added
windows
Copilot started reviewing on behalf of
Dmitriy Vasyura (dmitrivMS)
September 10, 2026 07:46
View session
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The Switch Window implementation introduces a prohibited dependency from vs/workbench onto the higher vs/sessions layer.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review tier: Balanced
Findings: 1
New issues introduced by this change (1)
| Severity | Finding |
|---|---|
 |
src/vs/workbench/electron-browser/actions/windowActions.ts — This makes the lower vs/workbench layer resolve an asset owned by vs/sessions, contrary to the… |
Copilot started reviewing on behalf of
Dmitriy Vasyura (dmitrivMS)
September 10, 2026 09:01
View session
Contributor
|
Base:
|
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The Windows integration, icon packaging, metadata propagation, and fallback behavior are consistent and complete.
Review tier: Balanced
Findings: None
Issues resolved since last review (1)
| Severity | Finding |
|---|---|
|
src/vs/workbench/electron-browser/actions/windowActions.ts — This makes the lower vs/workbench layer resolve an asset owned by vs/sessions, contrary to the… View resolved comment |
Keep the internal Agents workspace in shared recent history while omitting it from the Windows jump list now that a dedicated task is available. Refresh the CI-provided component fixture hashes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot started reviewing on behalf of
Dmitriy Vasyura (dmitrivMS)
September 10, 2026 19:23
View session
Expose native icon metadata only for the explicit Agents Window override so regular workspace, folder, and file entries continue using their resource icons. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Dmitriy Vasyura (dmitrivMS)
added
the
workbench-os-integration
Dmitriy Vasyura (dmitrivMS)
changed the title
Dmitriy Vasyura (dmitrivMS)
marked this pull request as ready for review
September 10, 2026 21:33
Dmitriy Vasyura (dmitrivMS)
enabled auto-merge (squash)
September 10, 2026 21:34
Raymond Zhao (rzhao271)
approved these changes
Sep 10, 2026
Dmitriy Vasyura (dmitrivMS)
deleted the
agents/add-agents-window-shortcut
branch
September 10, 2026 21:43
Dmitriy Vasyura (dmitrivMS)
added a commit
that referenced
this pull request
Sep 11, 2026
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.
--agentsmodesessions.icoasset for the jump-list task and resulting Windows taskbar button, so Stable, Insiders, and Exploration receive matching product-quality brandingCodeWindowicon path through generic opened-window metadataScreenshots
Related issues
Fixes #335321
Fixes #315734
Fixes #315935