Azure / Azure/azure-sdk-tools

[MCP/CODEOWNERS] Final Refactor

Open
#11,940 1 comment 0 reactions 0 assignees View on GitHub
needs-triage
Dominant language
C#
Stars
135
Forks
260
Avg merge
3d 1h
Merged PRs (30d)
143

Description

The current UpdateCodeowners tool can be split into both Add and Remove Codeowners tools. Something like:
```
[McpServerTool(Name = "azsdk_engsys_add_codeowners"), Description("Adds codeowners for a given service label or path in a repo.")]
public async Task AddCodeowners(
string repo,
bool isMgmtPlane,
string path = "",
string serviceLabel = "",
List serviceOwners = null,
List sourceOwners = null,
string workingBranch = ""
)

[McpServerTool(Name = "azsdk_engsys_add_codeowners"), Description("Removes codeowners for a given service label or path in a repo.")]
public async Task RemoveCodeowners(
string repo,
bool isMgmtPlane,
string path = "",
string serviceLabel = "",
List serviceOwners = null,
List sourceOwners = null,
string workingBranch = ""
)
```

Both of these methods would call the existing UpdateCodeowners method as Ben mentioned in PR https://github.com/Azure/azure-sdk-tools/pull/11841

Contributor guide

Open the contributing guide

Research direction

Read the existing UpdateCodeowners entry point and PR #11841 first; trace how its inputs are used. Done means exposing separate AddCodeowners and RemoveCodeowners MCP tools that delegate to the existing update behavior, with their descriptions and parameters matching the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.