Skip to content

Conversation

SteveL-MSFT
Copy link
Member

PR Summary

As discussed in #1093 (comment) add an optional parameter to list_dsc_resource([adapter]) instead of having a separate tool for listing adapted resources. Add show_dsc_resource() tool to get detailed information including schema for a dsc resource.

PR Context

Part of #1093

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 consolidates DSC resource listing functionality by adding an optional adapter parameter to the list_dsc_resources tool and introduces a new show_dsc_resource tool for retrieving detailed resource information including schemas. The changes eliminate the separate list_adapted_resources tool in favor of a unified approach.

  • Replaces list_adapted_resources with an optional adapter parameter in list_dsc_resources
  • Adds show_dsc_resource tool to get detailed information including schema for specific DSC resources
  • Updates tests to validate the consolidated functionality and new tool

Reviewed Changes

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

Show a summary per file
File Description
dsc/tests/dsc_mcp.tests.ps1 Updates test cases to validate consolidated resource listing and new show resource functionality
dsc/src/mcp/show_dsc_resource.rs Implements new tool for getting detailed DSC resource information including schema
dsc/src/mcp/mod.rs Removes list_adapted_resources module and adds show_dsc_resource module
dsc/src/mcp/mcp_server.rs Updates tool router to use new consolidated tools
dsc/src/mcp/list_dsc_resources.rs Adds optional adapter parameter to support both regular and adapted resource listing
dsc/locales/en-us.toml Adds localized error messages for the new functionality

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 15, 2025
Merged via the queue into PowerShell:main with commit 9e650e7 Sep 15, 2025
4 checks passed
@SteveL-MSFT SteveL-MSFT deleted the mcp-show-resource branch September 15, 2025 18:57
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