From 030b4bdbf3657863e5e6e533455a1e248cb861fe Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 30 Jan 2026 22:08:18 +0000 Subject: [PATCH 1/3] Initial plan From 57de0aa75a1b7896470fd68724ae224325803f1e Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 30 Jan 2026 22:14:48 +0000 Subject: [PATCH 2/3] Skip .NET Framework tests on non-Windows platforms - Configure TargetFrameworks to conditionally include net472 only on Windows - Remove mono setup steps from CI workflow Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> --- .github/workflows/ci-build-test.yml | 9 --------- .../TestServerWithHosting/TestServerWithHosting.csproj | 3 ++- .../ModelContextProtocol.TestServer.csproj | 3 ++- .../ModelContextProtocol.Tests.csproj | 3 ++- 4 files changed, 6 insertions(+), 12 deletions(-) diff --git a/.github/workflows/ci-build-test.yml b/.github/workflows/ci-build-test.yml index 37ee6ab83..70e768761 100644 --- a/.github/workflows/ci-build-test.yml +++ b/.github/workflows/ci-build-test.yml @@ -47,15 +47,6 @@ jobs: 10.0.x 9.0.x - # NetFX testing on non-Windows requires mono - - name: 🔧 Setup Mono - if: runner.os == 'Linux' - run: sudo apt-get install -y mono-devel - - - name: 🔧 Setup Mono on macOS - if: runner.os == 'macOS' - run: brew install mono - - name: 🔧 Set up Node.js uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6.2.0 with: diff --git a/samples/TestServerWithHosting/TestServerWithHosting.csproj b/samples/TestServerWithHosting/TestServerWithHosting.csproj index 7967395c0..3d2fd870b 100644 --- a/samples/TestServerWithHosting/TestServerWithHosting.csproj +++ b/samples/TestServerWithHosting/TestServerWithHosting.csproj @@ -2,7 +2,8 @@ Exe - net10.0;net9.0;net8.0;net472 + net10.0;net9.0;net8.0 + $(TargetFrameworks);net472 enable enable true diff --git a/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj b/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj index c4b39bb54..b874c0385 100644 --- a/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj +++ b/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj @@ -2,7 +2,8 @@ Exe - net10.0;net9.0;net8.0;net472 + net10.0;net9.0;net8.0 + $(TargetFrameworks);net472 enable enable TestServer diff --git a/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj b/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj index e0fb3d1fa..8f44ba5f6 100644 --- a/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj +++ b/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj @@ -2,7 +2,8 @@ Exe - net10.0;net9.0;net8.0;net472 + net10.0;net9.0;net8.0 + $(TargetFrameworks);net472 enable enable From 6a2bb115cdd94f60e6f2539df448a6277745c775 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 30 Jan 2026 22:28:38 +0000 Subject: [PATCH 3/3] Consolidate test target frameworks into Directory.Build.props Define DefaultTestTargetFrameworks in root Directory.Build.props with conditional net472 on Windows, then reference it from the test projects. Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> --- Directory.Build.props | 6 ++++++ samples/TestServerWithHosting/TestServerWithHosting.csproj | 3 +-- .../ModelContextProtocol.TestServer.csproj | 3 +-- .../ModelContextProtocol.Tests.csproj | 3 +-- 4 files changed, 9 insertions(+), 6 deletions(-) diff --git a/Directory.Build.props b/Directory.Build.props index 390be36e7..f5cdd3aad 100644 --- a/Directory.Build.props +++ b/Directory.Build.props @@ -45,4 +45,10 @@ true + + + + net10.0;net9.0;net8.0 + $(DefaultTestTargetFrameworks);net472 + diff --git a/samples/TestServerWithHosting/TestServerWithHosting.csproj b/samples/TestServerWithHosting/TestServerWithHosting.csproj index 3d2fd870b..9eca173cf 100644 --- a/samples/TestServerWithHosting/TestServerWithHosting.csproj +++ b/samples/TestServerWithHosting/TestServerWithHosting.csproj @@ -2,8 +2,7 @@ Exe - net10.0;net9.0;net8.0 - $(TargetFrameworks);net472 + $(DefaultTestTargetFrameworks) enable enable true diff --git a/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj b/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj index b874c0385..229c57d60 100644 --- a/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj +++ b/tests/ModelContextProtocol.TestServer/ModelContextProtocol.TestServer.csproj @@ -2,8 +2,7 @@ Exe - net10.0;net9.0;net8.0 - $(TargetFrameworks);net472 + $(DefaultTestTargetFrameworks) enable enable TestServer diff --git a/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj b/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj index 8f44ba5f6..efc3527b4 100644 --- a/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj +++ b/tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj @@ -2,8 +2,7 @@ Exe - net10.0;net9.0;net8.0 - $(TargetFrameworks);net472 + $(DefaultTestTargetFrameworks) enable enable