CI: Add Windows matrix leg with platform-specific test filtering

[Copilot]

• Create .github/workflows/ directory
• Create ci.yml workflow file with matrix strategy for ubuntu-latest and windows-latest
• Add build step for the solution
• Add conditional test step: run Magma.NetMap.Facts only on Linux
• Add conditional test step: run Magma.AF_XDP projects only on Linux (when tests exist)
• Add conditional test step: run WinTun tests only on Windows (when tests exist)
• Add conditional build step: verify Magma.WinTun.TcpHost sample builds on Windows
• Test CI workflow locally (validation on Linux)
• Add explicit permissions to workflow for security best practices
• Fix YAML indentation (steps properly indented under steps:)
• Use global.json for SDK version instead of hardcoded version

Original prompt

---

This section details on the original issue you should resolve

<issue_title>CI: Windows matrix leg</issue_title>
<issue_description>## Problem

The solution targets both Linux and Windows. WinTun only compiles on Windows. CI must cover both platforms.

Sub-tasks

• Add windows-latest to the matrix in ci.yml
• Conditional step: skip NetMap/AF_XDP test projects on Windows
• Conditional step: skip WinTun test projects on Linux
• Verify Magma.WinTun.TcpHost sample builds on Windows

Dependencies

Depends on: #99
Unlocks: #115 (WinTun native P/Invoke), #118 (WinTun tests)

Phase

Phase 0 (Foundation)</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes CI: Windows matrix leg #100

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull request overview

Adds a cross-platform GitHub Actions CI workflow to cover both Linux and Windows builds/tests for Magma, with OS-specific steps for platform-dependent components.

Changes:

• Introduces an OS matrix (ubuntu-latest, windows-latest) for the main build-and-test job.
• Runs platform-agnostic test projects on both OSes, while gating NetMap tests to Linux.
• Adds a Windows-only build verification step for the Magma.WinTun.TcpHost sample and sets explicit contents: read permissions.

[Copilot]

The steps: list is not indented under steps: (the - uses: line is aligned with steps:). This makes the workflow YAML invalid and will prevent the job from running. Indent all step items so they are children of steps:.

Suggested change

	- uses: actions/checkout@v4
	- name: Setup .NET
	uses: actions/setup-dotnet@v4
	with:
	dotnet-version: '10.0.x'
	- name: Restore dependencies
	run: dotnet restore
	- name: Build
	run: dotnet build --no-restore
	- name: Test (Common)
	run: dotnet test test/Magma.Common.Facts/Magma.Common.Facts.csproj --no-build --verbosity normal
	- name: Test (Link)
	run: dotnet test test/Magma.Link.Facts/Magma.Link.Facts.csproj --no-build --verbosity normal
	- name: Test (Internet.Ip)
	run: dotnet test test/Magma.Internet.Ip.Facts/Magma.Internet.Ip.Facts.csproj --no-build --verbosity normal
	- name: Test (NetMap) - Linux only
	if: runner.os == 'Linux'
	run: dotnet test test/Magma.NetMap.Facts/Magma.NetMap.Facts.csproj --no-build --verbosity normal
	- name: Verify WinTun.TcpHost builds - Windows only
	if: runner.os == 'Windows'
	run: dotnet build sample/Magma.WinTun.TcpHost/Magma.WinTun.TcpHost.csproj --no-restore
	- uses: actions/checkout@v4
	- name: Setup .NET
	uses: actions/setup-dotnet@v4
	with:
	dotnet-version: '10.0.x'
	- name: Restore dependencies
	run: dotnet restore
	- name: Build
	run: dotnet build --no-restore
	- name: Test (Common)
	run: dotnet test test/Magma.Common.Facts/Magma.Common.Facts.csproj --no-build --verbosity normal
	- name: Test (Link)
	run: dotnet test test/Magma.Link.Facts/Magma.Link.Facts.csproj --no-build --verbosity normal
	- name: Test (Internet.Ip)
	run: dotnet test test/Magma.Internet.Ip.Facts/Magma.Internet.Ip.Facts.csproj --no-build --verbosity normal
	- name: Test (NetMap) - Linux only
	if: runner.os == 'Linux'
	run: dotnet test test/Magma.NetMap.Facts/Magma.NetMap.Facts.csproj --no-build --verbosity normal
	- name: Verify WinTun.TcpHost builds - Windows only
	if: runner.os == 'Windows'
	run: dotnet build sample/Magma.WinTun.TcpHost/Magma.WinTun.TcpHost.csproj --no-restore

[Copilot]

Fixed in 38faebe. All step items are now properly indented with 6 spaces under steps:.

[Copilot]

actions/setup-dotnet is configured with dotnet-version: '10.0.x', but the repo pins 10.0.102 via global.json. Using the global.json file (or pinning the exact version) helps ensure CI uses the same SDK as local/dev builds and avoids subtle version drift.

Suggested change

	dotnet-version: '10.0.x'
	global-json-file: global.json

[Copilot]

Fixed in 38faebe. Now using global-json-file: global.json to ensure CI uses the pinned SDK version 10.0.102.

[benaadams]

@copilot rebase to master

[benaadams]

Closing as superseded — the CI workflow with Windows matrix support is already in master via PR #143.