Skip to content

Conversation

@benbp
Copy link
Member

@benbp benbp commented Oct 30, 2025

Adding the corresponding config to pair with Azure/azure-sdk-tools#12696

Specifically, the dotnet language check service from azsdk calls the CodeChecks.ps1 script, so this config is required to support Azure/azure-sdk-tools#12696

FYI @m-redding

@benbp benbp requested a review from hallipr as a code owner October 30, 2025 22:40
Copilot AI review requested due to automatic review settings October 30, 2025 22:40
@benbp benbp requested a review from weshaggard as a code owner October 30, 2025 22:40
@benbp benbp requested a review from m-redding October 30, 2025 22:40
@benbp benbp self-assigned this Oct 30, 2025
@benbp benbp added the Central-EngSys This issue is owned by the Engineering System team. label Oct 30, 2025
@benbp benbp moved this to 🔬 Dev in PR in Azure SDK EngSys 🍔🌭 Oct 30, 2025
@benbp benbp changed the title Add script override config for azsdk cli/mcp [azsdk-cli] Add script override config for azsdk cli/mcp Oct 30, 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 introduces a configuration file for the azsdk CLI/MCP tool to enable script overrides for repository operations. The configuration maps specific operations to custom PowerShell scripts that should be executed instead of default behavior.

  • Adds a new JSON configuration file that defines command overrides for the azsdk CLI tool
  • Configures the "CodeChecks" operation to use a custom PowerShell script located at eng/scripts/CodeChecks.ps1

// See https://github.com/azure/azure-sdk-tools/blob/main/tools/azsdk-cli/docs/specs/7-repo-tooling-contract.md
[
{
"tags": [ "CodeChecks" ],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was not expecting at this level. I'm expecting this to be "UpdateMetada" or another MCP tool name.

Copy link
Member

@m-redding m-redding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, I don't have a preference on the tag name

Copy link
Member

@weshaggard weshaggard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's hold on this until we settle on the design.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team.

Projects

Status: 🔬 Dev in PR

Development

Successfully merging this pull request may close these issues.

3 participants