Skip to content

Conversation

SteveL-MSFT
Copy link
Member

PR Summary

Add list_adapted_resources(<adapter>) tool to MCP server
Refactored the code so that the definition of McpServer is in own file and tools are in their own

PR Context

Part of #1093

@SteveL-MSFT SteveL-MSFT requested a review from Copilot September 12, 2025 05:33
Copy link
Contributor

@Copilot 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 a new list_adapted_resources tool to the MCP server to enumerate DSC resources that require an adapter, complementing the existing list_dsc_resources tool. The refactoring also improves code organization by separating the MCP server definition and tools into dedicated modules.

  • Add list_adapted_resources tool that filters resources by adapter type
  • Refactor MCP server structure into separate modules for better organization
  • Update tests to validate the new tool functionality

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
dsc/tests/dsc_mcp.tests.ps1 Updates test expectations and adds comprehensive tests for the new list_adapted_resources tool
dsc/src/mcp/mod.rs Refactors module structure by moving MCP server definition to separate file and adding new tool module
dsc/src/mcp/mcp_server.rs New file containing the core MCP server implementation with tool router configuration
dsc/src/mcp/list_dsc_resources.rs Updates import path and tool router visibility for the existing DSC resources tool
dsc/src/mcp/list_adapted_resources.rs New tool implementation for listing adapted resources with adapter filtering

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@Copilot 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

Copilot reviewed 5 out of 6 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@SteveL-MSFT SteveL-MSFT added this pull request to the merge queue Sep 12, 2025
Merged via the queue into PowerShell:main with commit 42b0a0f Sep 12, 2025
4 checks passed
@SteveL-MSFT SteveL-MSFT deleted the mcp-adapted branch September 12, 2025 16:07
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.

2 participants