Add new MCP tool to update language exclusion justification#12417
Merged
praveenkuttappan merged 5 commits intoAzure:mainfrom Oct 8, 2025
Merged
Add new MCP tool to update language exclusion justification#12417praveenkuttappan merged 5 commits intoAzure:mainfrom
praveenkuttappan merged 5 commits intoAzure:mainfrom
Conversation
github-actions
bot
added
the
azsdk-cli
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds language exclusion management functionality to the release plan tool, enabling tracking and justification of SDK languages that are excluded from releases. The tool can automatically detect missing languages when updating SDK details and provides a mechanism to request justification for exclusions.
- Separates supported languages by data plane vs management plane
- Automatically marks excluded languages as "Requested" when updating SDK details
- Adds new MCP tool to update language exclusion justification
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| ReleasePlanTool.cs | Added language separation and new MCP tool for updating exclusion justification |
| DevOpsService.cs | Updated method name to async convention and made language mapping methods public |
| ReleasePlanDetails.cs | Added fields for tracking language exclusion notes and status |
| ReleasePlanToolTests.cs | Added unit tests for language exclusion detection scenarios |
| ReleasePlanManualTests.cs | Added manual integration tests for exclusion justification functionality |
| MockDevOpsService.cs | Updated mock to support new async method name and work item ID handling |
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
benbp
approved these changes
Oct 8, 2025
maririos
reviewed
Oct 9, 2025
| { | ||
| var workItem = new WorkItem | ||
| { | ||
| Id = 1, |
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs
Show resolved
Hide resolved
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.
This PR has changes to:
exclusion requestedif a required language is missing when updating SDK details.