Skip to content

Conversation

@connor4312
Copy link
Member

Copilot AI review requested due to automatic review settings September 29, 2025 15:57
@connor4312 connor4312 enabled auto-merge (squash) September 29, 2025 15:58
@connor4312 connor4312 self-assigned this Sep 29, 2025
Copy link
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 adds support for custom icons in MCP (Model Context Protocol) servers, tools, and resources. The changes implement the icon specification from the MCP protocol, allowing servers to provide custom icons that are displayed in the VS Code UI.

Key Changes

  • Adds icon parsing and validation logic with security constraints based on transport type
  • Updates MCP data models to include icon support for servers, tools, and resources
  • Integrates custom icons into the UI for tool picker, resource quick access, and language model tools

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/vs/workbench/contrib/mcp/common/mcpIcons.ts New module implementing icon parsing, validation, and caching logic
src/vs/workbench/contrib/mcp/test/common/mcpIcons.test.ts Comprehensive test suite for icon validation and parsing functionality
src/vs/workbench/contrib/mcp/common/mcpTypes.ts Updates type definitions to include IMcpIcons interface and icon properties
src/vs/workbench/contrib/mcp/common/mcpServer.ts Integrates icon parsing into server metadata, tools, prompts, and resources
src/vs/workbench/contrib/mcp/common/mcpLanguageModelToolContribution.ts Updates tool UI to display custom icons when available
src/vs/workbench/contrib/mcp/browser/mcpResourceQuickAccess.ts Adds icon support to resource quick access picker
src/vs/workbench/contrib/chat/browser/actions/chatToolPicker.ts Updates chat tool picker to show server custom icons

@vs-code-engineering vs-code-engineering bot added this to the September 2025 milestone Sep 29, 2025
@connor4312 connor4312 merged commit 6eb0010 into main Sep 29, 2025
28 checks passed
@connor4312 connor4312 deleted the connor4312/mcp-metadata branch September 29, 2025 16:13
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Nov 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MCP: adopt SEP-973 for icons/metadata

3 participants