Skip to content

Conversation

joelverhagen
Copy link
Member

@joelverhagen joelverhagen commented Oct 3, 2025

@joelverhagen joelverhagen requested a review from a team as a code owner October 3, 2025 20:00
@Copilot Copilot AI review requested due to automatic review settings October 3, 2025 20:00
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 updates the Model Context Protocol (MCP) server.json configuration files to adhere to the latest schema version released on 2025-09-29. The changes standardize property naming conventions from snake_case to camelCase as required by the updated schema specification.

  • Updated schema reference from 2025-07-09 to 2025-09-29 version
  • Changed property naming from snake_case to camelCase (registry_type → registryType, package_arguments → packageArguments, environment_variables → environmentVariables)

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/src/McpServer/McpServer-CSharp/.mcp/server.json Updated template file with new schema and camelCase properties
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/mcpserver.Basic.verified/mcpserver/.mcp/server.json Updated test snapshot for basic configuration
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/mcpserver.AotTrue.verified/mcpserver/.mcp/server.json Updated test snapshot for AOT-enabled configuration
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/mcpserver.SelfContainedFalse.verified/mcpserver/.mcp/server.json Updated test snapshot for non-self-contained configuration
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/mcpserver.net10.verified/mcpserver/.mcp/server.json Updated test snapshot for .NET 10 configuration

@github-actions github-actions bot added the area-ai-templates Microsoft.Extensions.AI.Templates label Oct 3, 2025
@joelverhagen
Copy link
Member Author

joelverhagen commented Oct 3, 2025

@jeffhandley / @MackinnonBuck - I would like to release this change in a package around the time NuGet.org updates to latest schema. Could we work offline to time things? I understand there is the merge to main, a release branch activity, then pushing the nuget package.

@jeffhandley jeffhandley changed the title Adhere to latest server.json schema Update MCP Server Template to adhere to latest server.json schema Oct 7, 2025
@jeffhandley jeffhandley changed the title Update MCP Server Template to adhere to latest server.json schema Update MCP Server Template to adhere to 2025-09-29 server.json schema Oct 7, 2025
Copy link
Member

@jeffhandley jeffhandley left a comment

Choose a reason for hiding this comment

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

Looks good; thanks for submitting this. As suggested, I will connect with you offline on timing before we merge into main.

@jeffhandley jeffhandley added * NO MERGE * Do not merge this PR as long as this label is present. and removed * NO MERGE * Do not merge this PR as long as this label is present. labels Oct 7, 2025
@jeffhandley
Copy link
Member

Discussed with @joelverhagen and we're good to merge this now, with the expectation of this being included in next week's release.

This was referenced Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-ai-templates Microsoft.Extensions.AI.Templates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants