tets

Author: KSemenenko

.github/workflows/ci.yml

@@ -26,19 +26,47 @@ jobs:
         with:
           dotnet-version: ${{ env.DOTNET_VERSION }}
 
+      - name: Restore local tools
+        run: dotnet tool restore
+
       - name: Restore
         run: dotnet restore ManagedCode.MCPGateway.slnx
 
       - name: Build
         run: dotnet build ManagedCode.MCPGateway.slnx --configuration Release --no-restore
 
+      - name: Analyze
+        run: dotnet build ManagedCode.MCPGateway.slnx --configuration Release --no-restore -p:RunAnalyzers=true
+
+      - name: Verify formatting
+        run: dotnet format ManagedCode.MCPGateway.slnx --verify-no-changes
+
+      - name: Roslynator
+        run: dotnet tool run roslynator analyze src/ManagedCode.MCPGateway/ManagedCode.MCPGateway.csproj tests/ManagedCode.MCPGateway.Tests/ManagedCode.MCPGateway.Tests.csproj
+
       - name: Test
         run: dotnet test --solution ManagedCode.MCPGateway.slnx --configuration Release --no-build --verbosity normal --report-trx --results-directory ./artifacts/test-results
 
+      - name: Collect coverage
+        run: dotnet tool run coverlet tests/ManagedCode.MCPGateway.Tests/bin/Release/net10.0/ManagedCode.MCPGateway.Tests.dll --target "dotnet" --targetargs "test --solution ManagedCode.MCPGateway.slnx -c Release --no-build" --format cobertura --output artifacts/coverage/coverage.cobertura.xml
+
+      - name: Render coverage report
+        run: dotnet tool run reportgenerator -reports:"artifacts/coverage/coverage.cobertura.xml" -targetdir:"artifacts/coverage-report" -reporttypes:"HtmlSummary;MarkdownSummaryGithub"
+
       - name: Upload test artifacts
         if: always()
         uses: actions/upload-artifact@v4
         with:
           name: test-results
           path: ./artifacts/test-results/**
           retention-days: 5
+
+      - name: Upload coverage artifacts
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-report
+          path: |
+            ./artifacts/coverage/**
+            ./artifacts/coverage-report/**
+          retention-days: 5

.github/workflows/release.yml

@@ -28,6 +28,9 @@ jobs:
         with:
           dotnet-version: ${{ env.DOTNET_VERSION }}
 
+      - name: Restore local tools
+        run: dotnet tool restore
+
       - name: Extract version from Directory.Build.props
         id: version
         shell: bash
@@ -57,6 +60,15 @@ jobs:
       - name: Build
         run: dotnet build ManagedCode.MCPGateway.slnx --configuration Release --no-restore
 
+      - name: Analyze
+        run: dotnet build ManagedCode.MCPGateway.slnx --configuration Release --no-restore -p:RunAnalyzers=true
+
+      - name: Verify formatting
+        run: dotnet format ManagedCode.MCPGateway.slnx --verify-no-changes
+
+      - name: Roslynator
+        run: dotnet tool run roslynator analyze src/ManagedCode.MCPGateway/ManagedCode.MCPGateway.csproj tests/ManagedCode.MCPGateway.Tests/ManagedCode.MCPGateway.Tests.csproj
+
       - name: Test
         run: dotnet test --solution ManagedCode.MCPGateway.slnx --configuration Release --no-build --verbosity normal
 

AGENTS.md

@@ -80,6 +80,7 @@ If no new rule is detected -> do not update the file.
 ### Commands
 
 - restore: `dotnet restore ManagedCode.MCPGateway.slnx`
+- tool-restore: `dotnet tool restore`
 - build: `dotnet build ManagedCode.MCPGateway.slnx -c Release --no-restore`
 - analyze: `dotnet build ManagedCode.MCPGateway.slnx -c Release --no-restore -p:RunAnalyzers=true`
 - test: `dotnet test --solution ManagedCode.MCPGateway.slnx -c Release --no-build`
@@ -89,7 +90,9 @@ If no new rule is detected -> do not update the file.
 - test-runner-help: `tests/ManagedCode.MCPGateway.Tests/bin/Release/net10.0/ManagedCode.MCPGateway.Tests --help`
 - format: `dotnet format ManagedCode.MCPGateway.slnx`
 - format-check: `dotnet format ManagedCode.MCPGateway.slnx --verify-no-changes`
-- coverage: `dotnet test --solution ManagedCode.MCPGateway.slnx -c Release --no-build --coverage --coverage-output-format cobertura --results-directory ./artifacts/coverage`
+- roslynator-analyze: `dotnet tool run roslynator analyze src/ManagedCode.MCPGateway/ManagedCode.MCPGateway.csproj tests/ManagedCode.MCPGateway.Tests/ManagedCode.MCPGateway.Tests.csproj`
+- coverage: `dotnet tool run coverlet tests/ManagedCode.MCPGateway.Tests/bin/Release/net10.0/ManagedCode.MCPGateway.Tests.dll --target "dotnet" --targetargs "test --solution ManagedCode.MCPGateway.slnx -c Release --no-build" --format cobertura --output artifacts/coverage/coverage.cobertura.xml`
+- coverage-report: `dotnet tool run reportgenerator -reports:"artifacts/coverage/coverage.cobertura.xml" -targetdir:"artifacts/coverage-report" -reporttypes:"HtmlSummary;MarkdownSummaryGithub"`
 
 ### Rule Precedence
 
@@ -109,7 +112,7 @@ If no new rule is detected -> do not update the file.
 
 - Keep the standardized workflow skills first; use the extra installed inventory only when the repository actually wires that tool into commands, CI, docs, or an explicit user request.
 - Core .NET routing: `mcaf-dotnet`, `mcaf-dotnet-features`, `mcaf-testing`, `mcaf-dotnet-tunit`
-- Standardized .NET toolchain: `mcaf-dotnet-analyzer-config`, `mcaf-dotnet-code-analysis`, `mcaf-dotnet-format`, `mcaf-dotnet-roslynator`, `mcaf-dotnet-codeql`
+- Standardized .NET toolchain: `mcaf-dotnet-analyzer-config`, `mcaf-dotnet-code-analysis`, `mcaf-dotnet-format`, `mcaf-dotnet-roslynator`, `mcaf-dotnet-coverlet`, `mcaf-dotnet-reportgenerator`, `mcaf-dotnet-codeql`
 - Extended .NET catalog: `mcaf-dotnet-archunitnet`, `mcaf-dotnet-coverlet`, `mcaf-dotnet-csharpier`, `mcaf-dotnet-meziantou-analyzer`, `mcaf-dotnet-mstest`, `mcaf-dotnet-netarchtest`, `mcaf-dotnet-reportgenerator`, `mcaf-dotnet-semgrep`, `mcaf-dotnet-stryker`, `mcaf-dotnet-stylecop-analyzers`, `mcaf-dotnet-xunit`
 - Quality and maintainability: `mcaf-dotnet-quality-ci`, `mcaf-dotnet-complexity`, `mcaf-solid-maintainability`
 - Governance and docs: `mcaf-solution-governance`, `mcaf-architecture-overview`, `mcaf-adr-writing`, `mcaf-feature-spec`, `mcaf-ci-cd`
@@ -199,6 +202,7 @@ If no new rule is detected -> do not update the file.
 - Test framework in this repository is TUnit. Never add or keep xUnit here.
 - This repository uses `TUnit` on `Microsoft.Testing.Platform`; never use VSTest-only flags such as `--filter` or `--logger`, because they are not supported here.
 - For TUnit solution runs, always invoke `dotnet test --solution ...`; do not pass the solution path positionally.
+- Coverage in this repository uses the local `coverlet.console` tool against the built test assembly, then renders human-readable output with the local `reportgenerator` tool.
 - Every behavior change must include or update tests in `tests/ManagedCode.MCPGateway.Tests/`.
 - Add tests only when they close a meaningful behavior or regression gap; avoid low-signal tests that only increase count without improving confidence.
 - Keep tests focused on real gateway behavior:
@@ -211,7 +215,9 @@ If no new rule is detected -> do not update the file.
 - Do not remove tests to get green builds.
 - Keep `global.json` configured for `Microsoft.Testing.Platform` when TUnit is used.
 - At the end of implementation work, run code-size and quality verification with `cloc`, `roslynator`, and the repository's strict .NET build/test checks, then fix actionable findings so oversized files and quality drift do not accumulate.
+- `CSharpier` and `Stryker.NET` are installed as opt-in local tools for focused checks; they are not default formatter or default fast-path CI gates in this repository.
 - Run verification in this order:
+  - tool-restore
   - restore
   - build
   - test

Directory.Packages.props

@@ -3,7 +3,6 @@
     <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
   </PropertyGroup>
   <ItemGroup>
-    <PackageVersion Include="coverlet.collector" Version="8.0.0" />
     <PackageVersion Include="DotNet.ReproducibleBuilds" Version="2.0.2" />
     <PackageVersion Include="Microsoft.Agents.AI" Version="1.0.0-rc3" />
     <PackageVersion Include="Microsoft.Extensions.AI" Version="10.3.0" />

README.md

@@ -494,6 +494,7 @@ Use these when you need design details rather than package onboarding:
 ## Local Development
 
 ```bash
+dotnet tool restore
 dotnet restore ManagedCode.MCPGateway.slnx
 dotnet build ManagedCode.MCPGateway.slnx -c Release --no-restore
 dotnet test --solution ManagedCode.MCPGateway.slnx -c Release --no-build
@@ -503,4 +504,19 @@ Analyzer pass:
 
 ```bash
 dotnet build ManagedCode.MCPGateway.slnx -c Release --no-restore -p:RunAnalyzers=true
+dotnet tool run roslynator analyze src/ManagedCode.MCPGateway/ManagedCode.MCPGateway.csproj tests/ManagedCode.MCPGateway.Tests/ManagedCode.MCPGateway.Tests.csproj
+dotnet format ManagedCode.MCPGateway.slnx --verify-no-changes
 ```
+
+Coverage and human-readable report:
+
+```bash
+dotnet tool run coverlet tests/ManagedCode.MCPGateway.Tests/bin/Release/net10.0/ManagedCode.MCPGateway.Tests.dll --target "dotnet" --targetargs "test --solution ManagedCode.MCPGateway.slnx -c Release --no-build" --format cobertura --output artifacts/coverage/coverage.cobertura.xml
+dotnet tool run reportgenerator -reports:"artifacts/coverage/coverage.cobertura.xml" -targetdir:"artifacts/coverage-report" -reporttypes:"HtmlSummary;MarkdownSummaryGithub"
+```
+
+The local tool manifest currently owns `roslynator`, `coverlet.console`, `reportgenerator`, `dotnet-stryker`, and `csharpier`.
+
+- `dotnet format` remains the repository's formatter of record.
+- `csharpier` is installed for opt-in checks only and is not part of the default CI path.
+- `dotnet-stryker` is installed for focused mutation runs only and is not part of the default fast-path CI checks.

src/ManagedCode.MCPGateway/AGENTS.md

@@ -44,8 +44,10 @@ For this .NET project:
 - `mcaf-dotnet-analyzer-config`
 - `mcaf-dotnet-code-analysis`
 - `mcaf-dotnet-codeql`
+- `mcaf-dotnet-coverlet`
 - `mcaf-dotnet-format`
 - `mcaf-dotnet-features`
+- `mcaf-dotnet-reportgenerator`
 - `mcaf-dotnet-roslynator`
 - `mcaf-testing`
 - `mcaf-dotnet-tunit`

tests/ManagedCode.MCPGateway.Tests/AGENTS.md

@@ -29,6 +29,8 @@ Parent: `../../AGENTS.md`
 
 - `build`: `dotnet build tests/ManagedCode.MCPGateway.Tests/ManagedCode.MCPGateway.Tests.csproj -c Release --no-restore`
 - `test`: `dotnet test --solution ManagedCode.MCPGateway.slnx -c Release --no-build`
+- `coverage`: `dotnet tool run coverlet tests/ManagedCode.MCPGateway.Tests/bin/Release/net10.0/ManagedCode.MCPGateway.Tests.dll --target "dotnet" --targetargs "test --solution ManagedCode.MCPGateway.slnx -c Release --no-build" --format cobertura --output artifacts/coverage/coverage.cobertura.xml`
+- `coverage-report`: `dotnet tool run reportgenerator -reports:"artifacts/coverage/coverage.cobertura.xml" -targetdir:"artifacts/coverage-report" -reporttypes:"HtmlSummary;MarkdownSummaryGithub"`
 - `format`: `dotnet format ManagedCode.MCPGateway.slnx`
 - `analyze`: `dotnet build tests/ManagedCode.MCPGateway.Tests/ManagedCode.MCPGateway.Tests.csproj -c Release --no-restore -p:RunAnalyzers=true`
 
@@ -42,8 +44,10 @@ For this .NET project:
 
 - `mcaf-dotnet-analyzer-config`
 - `mcaf-dotnet-code-analysis`
+- `mcaf-dotnet-coverlet`
 - `mcaf-dotnet-format`
 - `mcaf-testing`
+- `mcaf-dotnet-reportgenerator`
 - `mcaf-dotnet-roslynator`
 - `mcaf-dotnet-tunit`
 - `mcaf-dotnet-quality-ci`

tests/ManagedCode.MCPGateway.Tests/ManagedCode.MCPGateway.Tests.csproj

@@ -12,10 +12,6 @@
   <ItemGroup>
     <PackageReference Include="Microsoft.Agents.AI" />
     <PackageReference Include="TUnit" />
-    <PackageReference Include="coverlet.collector">
-      <PrivateAssets>all</PrivateAssets>
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-    </PackageReference>
     <PackageReference Include="Microsoft.Extensions.AI" />
     <PackageReference Include="Microsoft.Extensions.DependencyInjection" />
     <PackageReference Include="Microsoft.Extensions.Logging" />