Update repo to use net10.0 #1221

hallipr · 2025-11-20T01:34:43Z

What does this PR do?

Replace the mixed net9.0, net10-preview dependency with the new net10.0 GA version.

Pre-merge Checklist

Copilot

Pull Request Overview

This PR updates the repository from a mixed .NET 9.0 and .NET 10.0-preview dependency to the new .NET 10.0 GA version. The changes comprehensively update SDK versions, target frameworks, build paths, and documentation across the codebase.

Key Changes

Updated global.json SDK version from preview (10.0.100-preview.7) to GA (10.0.100)
Changed TargetFramework from net9.0 to net10.0 in Directory.Build.props
Removed preview-specific NuGet packages (System.Linq.AsyncEnumerable, System.Net.ServerSentEvents)
Updated all documentation and build scripts to reference net10.0 paths instead of net9.0
Removed redundant .NET 9.0.x SDK installation steps from CI/CD pipelines

Reviewed Changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
global.json	Updated SDK version from preview to GA release (10.0.100)
Directory.Build.props	Changed TargetFramework from net9.0 to net10.0
Directory.Packages.props	Removed preview package versions for System.Linq.AsyncEnumerable and System.Net.ServerSentEvents
core/Azure.Mcp.Core/src/Azure.Mcp.Core.csproj	Removed System.Linq.AsyncEnumerable package reference
eng/pipelines/templates/jobs/*.yml	Removed .NET 9.0.x SDK installation tasks (build.yml, analyze.yml, live-test.yml)
eng/scripts/Update-AzCommandsMetadata.ps1	Updated default azmcp path from net9.0 to net10.0, improved path resolution
eng/scripts/Test-Code.ps1	Reordered parameters to make Paths positional
servers/Azure.Mcp.Server/README.md	Updated .NET version reference from preview to GA
servers/Azure.Mcp.Server/TROUBLESHOOTING.md	Updated all binary paths from net9.0 to net10.0, enhanced table of contents
servers/Azure.Mcp.Server/docs/new-command.md	Updated TargetFramework references to net10.0, cleaned up trailing whitespace
servers/Azure.Mcp.Server/tests/.../CommandMetadataSyncTests.cs	Updated executable path from net9.0 to net10.0
servers/Fabric.Mcp.Server/README.md	Updated SDK version reference from 9.x to 10.x
servers/Fabric.Mcp.Server/SUPPORT.md	Updated .NET version references from 9 to 10
servers/Fabric.Mcp.Server/TROUBLESHOOTING.md	Updated SDK version and binary paths from net9.0 to net10.0
CONTRIBUTING.md	Updated all azmcp executable paths from net9.0 to net10.0
AGENTS.md	Updated example mcp.json path from net9.0 to net10.0

servers/Azure.Mcp.Server/docs/new-command.md

eng/scripts/Test-Code.ps1

alzimmermsft

FYI @conniey , since you have this issue assigned to you #1198

alzimmermsft · 2025-11-20T14:46:36Z

Directory.Packages.props

Should the following also be updated:

System.Formats.Asn1 -> 10.0.0
Microsoft.Extensions.AI.Abstractions -> 10.0.0
Microsoft.Extensions.AI.Evaluation.Quality -> 10.0.0
Microsoft.AspNetCore.SignalR.Client -> 10.0.0
Microsoft.Extensions.Caching.Memory -> 10.0.0
Microsoft.Extensions.Configuration -> 10.0.0
Microsoft.Extensions.Configuration.EnvironmentVariables -> 10.0.0
Microsoft.Extensions.Configuration.Json -> 10.0.0
Microsoft.Extensions.DependencyInjection -> 10.0.0
Microsoft.Extensions.Hosting -> 10.0.0
System.CommandLine -> 2.0.0
System.Numerics.Tensors -> 10.0.0
System.Text.Json -> 10.0.0

Updated, but now we get some new AOT failures:
https://dev.azure.com/azure-sdk/public/public%20Team/_build/results?buildId=5603241&view=logs&j=1e679852-4871-5dae-dd38-eb073eb853d4&t=3639d9f6-46e5-5c59-1fa3-9c261e132d8a&l=105

ILLink : Trim analysis error IL2026: Azure.AI.OpenAI.Files.AzureFileExpirationOptions.PersistableModelWriteCore(ModelReaderWriterOptions): Using member 'System.ClientModel.Primitives.ModelReaderWriter.Write(!!0, ModelReaderWriterOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. This method uses reflection. Use the overload that takes a ModelReaderWriterContext to be AOT compatible. [/mnt/vss/_work/1/s/servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj] ILLink : Trim analysis error IL2026: Azure.AI.OpenAI.CustomSerializationHelpers.SerializeInstance<TOutput,UInstanceInput>(UInstanceInput, ModelReaderWriterOptions): Using member 'System.ClientModel.Primitives.ModelReaderWriter.Write(Object, ModelReaderWriterOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. This method uses reflection. Use the overload that takes a ModelReaderWriterContext to be AOT compatible. [/mnt/vss/_work/1/s/servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj] ILLink : Trim analysis error IL2026: Azure.ResourceManager.HDInsight.Models.HDInsightLocalizedName.System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.HDInsight.Models.HDInsightLocalizedName>.Write(ModelReaderWriterOptions): Using member 'System.ClientModel.Primitives.ModelReaderWriter.Write(!!0, ModelReaderWriterOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. This method uses reflection. Use the overload that takes a ModelReaderWriterContext to be AOT compatible. [/mnt/vss/_work/1/s/servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj] ILLink : Trim analysis error IL2026: Azure.ResourceManager.HDInsight.Models.HDInsightUsage.System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.HDInsight.Models.HDInsightUsage>.Write(ModelReaderWriterOptions): Using member 'System.ClientModel.Primitives.ModelReaderWriter.Write(!!0, ModelReaderWriterOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. This method uses reflection. Use the overload that takes a ModelReaderWriterContext to be AOT compatible. [/mnt/vss/_work/1/s/servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj]

@hallipr, could you update Azure.ResourceManager.HDInsight to 1.2.0-beta.5? I'll look into Azure.AI.OpenAI

Checking with the .NET SDK team on Microsoft.Extensions.AI.OpenAI. If this PR is urgent, we can keep the current version of Microsoft.Extensions.AI.OpenAI and just update Azure.ResourceManager.HDInsight to 1.2.0-beta.5.

alzimmermsft · 2025-11-20T14:49:26Z

eng/pipelines/templates/jobs/analyze.yml

-  - task: UseDotNet@2
-    displayName: "Use .NET SDK 9.0.x"
-    retryCountOnTaskFailure: 3
-    inputs:
-      packageType: sdk
-      version: 9.0.x


Depending how soon we want this PR in may still need this but be updated to 10.0.x as not all hosted images have updated to include .NET Core 10.0 yet

There's a task just above these that installs the version in global.json

We were having to also install 9.0 after 10-preview because we wanted 10.0-preview tooling, but a 9.0 dependency

...Azure.Mcp.Server/tests/Azure.Mcp.Server.UnitTests/Infrastructure/CommandMetadataSyncTests.cs

weshaggard · 2025-11-20T17:59:27Z

servers/Azure.Mcp.Server/TROUBLESHOOTING.md

    "Azure Storage": {
      "type": "stdio",
-      "command": "<absolute-path-to>/azure-mcp/core/src/AzureMcp.Cli/bin/Debug/net9.0/azmcp[.exe]",
+      "command": "<absolute-path-to>/mcp/servers/Azure.Mcp.Server/src/Debug/net10.0/azmcp[.exe]",


What are these examples for? Do we need to really hard-code the full path?

These are examples of using a local build of Azure MCP in an mcp.json. Ex, I have this locally for testing end-to-end:

{ "servers": { "azure-mcp-local-debug": { "type": "stdio", "command": "npx", "args": [ "-y", "D:\\GitHub\\mcp\\servers\\Azure.Mcp.Server\\src\\bin\\Debug\\net9.0\\azmcp.exe", "server", "start", "--mode", "namespace" ] } } }

Update repo to use net10.0

96bcffb

Copilot AI review requested due to automatic review settings November 20, 2025 01:34

hallipr requested review from a team as code owners November 20, 2025 01:34

hallipr requested review from JasonYeMSFT, alzimmermsft, chidozieononiwu, g2vinay, jongio, joshfree and msalaman November 20, 2025 01:34

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

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

Copilot started reviewing on behalf of hallipr November 20, 2025 01:35 View session

Copilot finished reviewing on behalf of hallipr November 20, 2025 01:38

Copilot AI reviewed Nov 20, 2025

View reviewed changes

servers/Azure.Mcp.Server/docs/new-command.md Show resolved Hide resolved

servers/Azure.Mcp.Server/docs/new-command.md Show resolved Hide resolved

eng/scripts/Test-Code.ps1 Show resolved Hide resolved

Remove EndOfStream use in async

76a050b

alzimmermsft reviewed Nov 20, 2025

View reviewed changes

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

weshaggard reviewed Nov 20, 2025

View reviewed changes

...Azure.Mcp.Server/tests/Azure.Mcp.Server.UnitTests/Infrastructure/CommandMetadataSyncTests.cs Outdated Show resolved Hide resolved

weshaggard reviewed Nov 20, 2025

View reviewed changes

hallipr added 3 commits November 20, 2025 12:44

Use BaseDirectory to find exe

dc28934

Better path usage

b248361

TEmp debug output

7cce402

hallipr marked this pull request as draft November 20, 2025 21:20

hallipr added 4 commits November 20, 2025 13:46

Use slnx for test-code.ps1

b4cb0cf

Fix a few rwar projects

7d31759

Update dependencies to 10.0 versions

41b2e98

Update system.commandline

969cebf

joshfree assigned hallipr and anuchandy Nov 24, 2025

joshfree added the Engineering Excellence Items required to be resolved before onboarding the "third wave" of azure RPs label Nov 24, 2025

joshfree modified the milestones: 2025-12, 2026-01 Nov 24, 2025

stephentoub mentioned this pull request Nov 28, 2025

Update ModelContextProtocol to latest #1255

Open

Update repo to use net10.0 #1221

Are you sure you want to change the base?

Update repo to use net10.0 #1221

Uh oh!

Conversation

hallipr commented Nov 20, 2025

What does this PR do?

Pre-merge Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Key Changes

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

alzimmermsft left a comment

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!

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!

6 participants