Update build pipeline to support server.json #1104

conniey · 2025-11-06T23:40:38Z

What does this PR do?

Adds a server.json to Azure.Mcp.Server
Adds a template server.json to Fabric.Mcp.Server
Updates build pipeline:
- Update version information in server.json and publish as build_info
- Publish the updated server.json to either staging or public based on publish target internal or public.

GitHub issue number?

[Link to the GitHub issue this PR addresses]

Pre-merge Checklist

Copilot

Pull Request Overview

This PR adds support for publishing MCP server metadata to the official MCP repository registry. The changes introduce server.json files containing package metadata, update build scripts to use these files, and add deployment infrastructure.

Adds server.json files for Azure and Fabric MCP servers with registry metadata
Refactors packaging scripts to copy existing server.json files instead of generating them dynamically
Implements Deploy-ServerJson.ps1 script for publishing to the MCP registry
Updates version management to synchronize server.json versions with package versions

Reviewed Changes

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

Show a summary per file

File	Description
servers/Fabric.Mcp.Server/src/Fabric.Mcp.Server.csproj	Adds ServerJsonPath and commented McpRepositoryName properties for Fabric server
servers/Fabric.Mcp.Server/server.json	New server metadata file with placeholder values for description and package identifier
servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj	Adds ServerJsonPath and McpRepositoryName properties for Azure server
servers/Azure.Mcp.Server/server.json	New server metadata file with complete Azure MCP server configuration for NuGet and npm packages
servers/Azure.Mcp.Server/README.md	Adds MCPRepositoryMetadata insertion point for package processing
eng/scripts/Update-Version.ps1	Extends version update logic to synchronize versions in server.json files
eng/scripts/Pack-Nuget.ps1	Replaces dynamic server.json generation with file copy, adds MCPRepositoryMetadata to README processing
eng/scripts/Pack-Npm.ps1	Adds mcpName field to npm package.json from server properties
eng/scripts/Get-ProjectProperties.ps1	Adds ServerJsonPath and McpRepositoryName to list of extracted project properties
eng/scripts/Deploy-ServerJson.ps1	New script implementing MCP registry authentication and publication using Azure Key Vault and Go-based publisher tool
eng/pipelines/templates/jobs/update-mcp-repository.yml	New pipeline job template for building MCP publisher and updating registry
eng/pipelines/templates/jobs/release.yml	Integrates MCP repository update job into release pipeline

eng/scripts/Update-Version.ps1

servers/Fabric.Mcp.Server/src/Fabric.Mcp.Server.csproj

servers/Fabric.Mcp.Server/server.json

Copilot · 2025-11-06T23:45:21Z

eng/pipelines/templates/jobs/update-mcp-repository.yml

+        -TemporaryDirectory '$(Pipeline.TemporaryDirectory)'
+        -ReleaseType '${{ parameters.ReleaseType }}'
+        -KeyVaultName 'kv-azuresdk-codesign'
+        -KeyVaultKeyName ''


The KeyVaultKeyName parameter is set to an empty string. This is required for the Azure Key Vault authentication in the Deploy-ServerJson.ps1 script and should be set to a valid key name.

eng/pipelines/templates/jobs/release.yml

eng/scripts/Deploy-ServerJson.ps1

servers/Azure.Mcp.Server/server.json

joshfree · 2025-11-07T17:13:12Z

@joelverhagen PTAL too thanks

servers/Azure.Mcp.Server/server.json

joelverhagen · 2025-11-07T19:03:04Z

servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj

    <PackageIcon>$(MSBuildThisFileDirectory)../azureicon.png</PackageIcon>

+    <!-- Official MCP repository settings -->
+    <McpRepositoryName>com.microsoft/azure-mcp</McpRepositoryName>


Suggested change

<McpRepositoryName>com.microsoft/azure-mcp</McpRepositoryName>

<McpRepositoryName>com.microsoft/azure</McpRepositoryName>

I would argue mcp is redundant in this context.

@joshfree I have no opinions. If the wider team is OK with this name... the shorter the better. :D

joelverhagen · 2025-11-07T19:05:12Z

servers/Azure.Mcp.Server/server.json

+    "$schema": "https://static.modelcontextprotocol.io/schemas/2025-10-17/server.schema.json",
+    "name": "com.microsoft/azure-mcp",
+    "description": "All Azure MCP tools to create a seamless connection between AI agents and Azure services.",
+    "version": "2.0.0-beta.2",


Do you want icons? Probably yes, for parity with the existing.
It might be worth checking with Toby on how these icon URLs will be used for GitHub MCP Registry (direct linked, replicated, etc).

See modelcontextprotocol/registry#277 (comment) for issues...

You could supply icon in a future release, if you want to move fast.

joelverhagen · 2025-11-07T19:06:36Z

servers/Azure.Mcp.Server/server.json

+        {
+            "registryType": "nuget",
+            "identifier": "Azure.Mcp",
+            "version": "2.0.0-beta.2",


Do you think it would be worth putting explicit placeholder values here? So is someone wonders "hmm ... this looks old" they will rest assured something else will fix it up.

True. We could have placeholder values

eng/scripts/Deploy-ServerJson.ps1

hallipr · 2025-11-11T19:07:29Z

@chidozieononiwu, can you check the server.json content to make sure it's correct for npx packaging?

hallipr · 2025-11-11T20:32:02Z

Can you update servers/Template.Mcp.Server as well? I'd like to be able to test this as part of Template's release pipeline

…tegration to push to staging

hallipr · 2025-11-26T18:52:43Z

eng/pipelines/templates/jobs/integration.yml

        -BuildInfoPath '$(Pipeline.Workspace)/build_info/build_info.json'
        -NpmRegistry '${{ parameters.NpmRegistry }}'
+
+- template: /eng/pipelines/templates/jobs/update-mcp-repository.yml


Does this mean we'll be publishing prerelease updates to the official registry pointing to internal/public package feeds?

Integration in this repo should have been called "Release Unofficial" or "Release Internal". It doesn't do the same jobs as Integration in the sdk repos

Ahh makes sense. I realise that staging.mcp. only performs the check against the official NuGet.org repo.. so it doesn't make sense since we don't host our own internal mcp repo. Let me remove this.

nm. I see the staging repo work in Deploy-ServerJson.ps1

hallipr · 2025-11-26T20:06:20Z

eng/scripts/Deploy-ServerJson.ps1

+Write-Host "Installing Microsoft Go and building MCP publishing tool..."
+
+# This go-install.ps1 script could be checked into your source repository
+$goInstallScriptPath = "$PSScriptRoot/Install-Go.ps1"


I've opened an issue to see if all of this is really necessary. I suspect the publisher tool is just making an authenticated API call with either a token minted using our private key or with a body signed using our key. Either way, it could be all powershell with no git clone in the middle of a deployment step.

hallipr · 2025-11-26T20:17:47Z

eng/scripts/Verify-NuGetRelease.ps1

+
+<#
+.SYNOPSIS
+    Verifies the NuGet package for the specified server is available on nuget.org.


Any chance dotnet package search --exact-match would handle this for us reliably?

https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-package-search

This approach was documented because it checks the same artifact (embedded package README) that the MCP Registry uses. dotnet package search --exact-match checks the NuGet.org package metadata endpoint. There is certainly a relationship between these two things (NuGet.org publishes README strictly before search availability) but for a general purpose script it helps avoid false positives when a package has forgotten to embed the README.

It is not a bad suggestion, I'm just providing the context.

hallipr · 2025-11-26T20:43:39Z

servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj

    <Description>Azure MCP Server - Model Context Protocol implementation for Azure</Description>
    <ReadmeUrl>https://github.com/Microsoft/mcp/blob/main/servers/Azure.Mcp.Server#readme</ReadmeUrl>
    <ReadmePath>$(RepoRoot)/servers/Azure.Mcp.Server/README.md</ReadmePath>
+    <ServerJsonPath>servers/Azure.Mcp.Server/server.json</ServerJsonPath>


The paths should be absolute.

maybe

Suggested change

<ServerJsonPath>servers/Azure.Mcp.Server/server.json</ServerJsonPath>

<ServerJsonPath>$(RepoRoot)servers/Azure.Mcp.Server/server.json</ServerJsonPath>

or

Suggested change

<ServerJsonPath>servers/Azure.Mcp.Server/server.json</ServerJsonPath>

<ServerJsonPath>$(MSBuildThisFileDirectory)server.json</ServerJsonPath>

hallipr · 2025-11-26T20:47:37Z

servers/Azure.Mcp.Server/server.json

+    "title": "Azure MCP Server",
+    "packages": [
+        {
+            "registryType": "nuget",


Can this whole section be inject and not just the version. Otherwise, there's the possibility of conflict between the package name we see here and the one in the server's csproj

Copilot AI review requested due to automatic review settings November 6, 2025 23:40

conniey requested review from a team as code owners November 6, 2025 23:40

github-project-automation bot added this to Azure MCP Server Nov 6, 2025

conniey requested review from JasonYeMSFT, KarishmaGhiya, fanyang-mono, g2vinay, jongio, joshfree and wbreza November 6, 2025 23:40

github-project-automation bot moved this to Untriaged in Azure MCP Server Nov 6, 2025

Copilot AI reviewed Nov 6, 2025

View reviewed changes

joshfree assigned conniey Nov 7, 2025

joshfree moved this from Untriaged to In Progress in Azure MCP Server Nov 7, 2025

joshfree added the tools-McpRegistry MCP Server Registry / Repository / Marketplace label Nov 7, 2025

joshfree added this to the 2025-11 milestone Nov 7, 2025

joshfree reviewed Nov 7, 2025

View reviewed changes

eng/pipelines/templates/jobs/release.yml Outdated Show resolved Hide resolved

eng/scripts/Deploy-ServerJson.ps1 Show resolved Hide resolved

eng/scripts/Deploy-ServerJson.ps1 Show resolved Hide resolved

servers/Azure.Mcp.Server/server.json Show resolved Hide resolved

joshfree assigned joelverhagen Nov 7, 2025

weshaggard requested a review from hallipr November 7, 2025 18:00

joelverhagen reviewed Nov 7, 2025

View reviewed changes

servers/Azure.Mcp.Server/server.json Show resolved Hide resolved

joelverhagen reviewed Nov 7, 2025

View reviewed changes

eng/scripts/Deploy-ServerJson.ps1 Show resolved Hide resolved

conniey force-pushed the conniey/server-json branch from 2613183 to 9cb4b8b Compare November 10, 2025 18:15

conniey added 14 commits November 25, 2025 15:00

Use checked in Go-Install

07caa7d

Update CHANGELOG.

2a44e45

Fix path to install go

33e36ae

Only run when publish is not skipped

66a77ec

Add ServerJsonPath to Build-Info

030acea

Use placeholder version

4ec042b

Remove RepoRoot from server.json

12fc482

Only publish for matching ServerName

2524b6f

Add script to verify-Nugetrelease

cd1fe25

Add step to verify nuget package release

b7086fa

Remove duplicated New-BuildInfo McpRepositryName

1b135de

Add script to generate server.json

4e6f881

Add creation of server.json to initialize

63b27b1

Add path to server.json. Update script to use that path.

48b9c51

conniey force-pushed the conniey/server-json branch from 9036a2b to 48b9c51 Compare November 25, 2025 23:00

conniey added 7 commits November 25, 2025 15:02

Fixup order for build-info.sjon

597d68d

Add documentation and support for querying a nuget feed.

253380e

Add ServerName parameter to common.yml and integration.yml. Update in…

174005b

…tegration to push to staging

Add NugetFeedIndexUrl to update-mcp

c205743

Force create file and any children directories.

43ddb7a

Fix unknown condition

725319c

Fix unexpected condition

50910bd

hallipr reviewed Nov 26, 2025

View reviewed changes

Fix temporarily removed file directory.

c7ad105

hallipr reviewed Nov 26, 2025

View reviewed changes

conniey added 2 commits November 26, 2025 12:08

Move check for publish targets.

95a3efe

Remove step to integration.

a827fcd

hallipr reviewed Nov 26, 2025

View reviewed changes

	<McpRepositoryName>com.microsoft/azure-mcp</McpRepositoryName>
	<McpRepositoryName>com.microsoft/azure</McpRepositoryName>

	<ServerJsonPath>servers/Azure.Mcp.Server/server.json</ServerJsonPath>
	<ServerJsonPath>$(RepoRoot)servers/Azure.Mcp.Server/server.json</ServerJsonPath>

	<ServerJsonPath>servers/Azure.Mcp.Server/server.json</ServerJsonPath>
	<ServerJsonPath>$(MSBuildThisFileDirectory)server.json</ServerJsonPath>

Update build pipeline to support server.json #1104

Are you sure you want to change the base?

Update build pipeline to support server.json #1104

Uh oh!

Conversation

conniey commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What does this PR do?

GitHub issue number?

Pre-merge Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Nov 6, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

joshfree commented Nov 7, 2025

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

hallipr commented Nov 11, 2025

Uh oh!

hallipr commented Nov 11, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

conniey Nov 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

conniey commented Nov 6, 2025 •

edited

Loading

conniey Nov 26, 2025 •

edited

Loading