Fix Codegen and Add MCPCode Generation

.config/dotnet-tools.json

@@ -2,6 +2,13 @@
   "version": 1,
   "isRoot": true,
   "tools": {
+    "csharpier": {
+      "version": "0.30.2",
+      "commands": [
+        "dotnet-csharpier"
+      ],
+      "rollForward": false
+    },
     "fantomas": {
       "version": "6.3.15",
       "commands": [

.editorconfig

@@ -12,7 +12,6 @@ dotnet_analyzer_diagnostic.category-Globalization.severity = error
 dotnet_analyzer_diagnostic.category-Documentation.severity = error
 dotnet_analyzer_diagnostic.category-Readability.severity = error
 dotnet_analyzer_diagnostic.category-Ordering.severity = error
-dotnet_analyzer_diagnostic.category-StyleCop.CSharp.OrderingRules.severity = none
 
 # Nullability
 # CS8602: Dereference of a possibly null reference.
@@ -41,7 +40,6 @@ dotnet_diagnostic.CA2000.severity = error
 dotnet_diagnostic.CA2201.severity = error
 dotnet_diagnostic.CS1591.severity = error
 dotnet_diagnostic.IDE0022.severity = error
-dotnet_diagnostic.CA1054.severity = error
 dotnet_diagnostic.CS8600.severity = error
 dotnet_diagnostic.CS8601.severity = error
 dotnet_diagnostic.CS8603.severity = error
@@ -373,9 +371,15 @@ dotnet_diagnostic.SA1400.severity = none
 dotnet_diagnostic.SA1114.severity = none
 dotnet_diagnostic.SA1118.severity = none
 dotnet_diagnostic.SA1649.severity = none
+dotnet_diagnostic.CA1054.severity = none
 
+dotnet_diagnostic.SA1200.severity = none
 
 
+# TODO: these are nice. Put back
+dotnet_diagnostic.SA1201.severity = none
+dotnet_diagnostic.SA1202.severity = none
+dotnet_diagnostic.SA1204.severity = none
 
 
 

.github/workflows/pr-build.yml

@@ -20,25 +20,36 @@ jobs:
             8.0.x
             9.0.x
 
+      - name: Restore .NET tools
+        run: dotnet tool restore
+
       - name: Restore dependencies
         run: dotnet restore RestClient.sln
 
       - name: Check code formatting with CSharpier
-        run: |
-          dotnet tool install --global csharpier
-          dotnet csharpier --check .
+        run: dotnet csharpier --check .
 
       - name: Build solution
         run: dotnet build RestClient.sln --configuration Release --no-restore /warnaserror
 
+      - name: Run code analysis
+        run: dotnet build RestClient.sln --configuration Release --no-restore /p:RunAnalyzers=true /p:TreatWarningsAsErrors=true
+
+      - name: Run Stryker Mutation Testing
+        working-directory: RestClient.Net.CsTest
+        run: dotnet stryker --break-at 100
+
+      - name: Verify Docker is available
+        run: |
+          docker --version
+          docker compose version
+
       - name: Run all tests with code coverage
         run: dotnet test RestClient.sln --configuration Release --no-build --verbosity normal --logger "console;verbosity=detailed" --collect:"XPlat Code Coverage" -- DataCollectionRunSettings.DataCollectors.DataCollector.Configuration.Threshold=100 DataCollectionRunSettings.DataCollectors.DataCollector.Configuration.ThresholdType=line,branch,method DataCollectionRunSettings.DataCollectors.DataCollector.Configuration.ThresholdStat=total
 
-      - name: Install Stryker Mutator
-        run: dotnet tool install --global dotnet-stryker
-
-      - name: Run Stryker Mutation Testing
-        run: dotnet stryker --break-at 100 --reporter "console" --reporter "html"
+      - name: Cleanup Docker containers
+        if: always()
+        run: |
+          cd Samples/NucliaDbClient
+          docker compose down -v --remove-orphans || true
 
-      - name: Run code analysis
-        run: dotnet build RestClient.sln --configuration Release --no-restore /p:RunAnalyzers=true /p:TreatWarningsAsErrors=true

.vscode/launch.json

@@ -33,7 +33,7 @@
                 "-u",
                 "${workspaceFolder}/Samples/NucliaDbClient/api.yaml",
                 "-o",
-                "${workspaceFolder}/Samples/RestClient.OpenApiGenerator.Sample.NucliaDB/Generated",
+                "${workspaceFolder}/Samples/NucliaDbClient/Generated",
                 "-n",
                 "NucliaDB.Generated",
                 "-c",

CLAUDE.md

@@ -4,18 +4,19 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 ## Code Rules
 
-- NO DUPLICATION - EVER!!!!
-- Reduce the AMOUNT of code wherever possible
+- NO DUPLICATION - EVER!!!! REMOVING DUPLICATION is the absolute HIGHEST PRIORITY!!!
+- YOU ARE NOT ALLOWED TO SKIP TESTS
 - No throwing exceptions, except for in tests
 - FP style code. Pure functions with immutable types.
-- 100% test coverage everywhere
-- Don't use Git unless I explicitly ask you to
 - Keep functions under 20 LOC
 - Keep files under 300 LOC
-- Use StyleCop.Analyzers and Microsoft.CodeAnalysis.NetAnalyzers for code quality
-- Ccode analysis warnings are always errors
-- Nullable reference types are enabled
 - NEVER copy files. Only MOVE files
+- Don't use Git unless I explicitly ask you to
+- Promote code analysis warnings to errors
+- EXHAUSTION001 is a critical error and must be turned on everywhere
+- Nullable reference types are enabled and MUST be obeyed
+- Do not back files up
+- Aggressively pursue these aims, even when it means taking more time on a task
 
 ## Build, Test, and Development Commands
 

RestClient.Net.CsTest/HttpClientFactoryExtensionsTests.cs

@@ -1686,4 +1686,216 @@ public async Task CreatePatch_CancellationToken_CancelsRequest()
 
         Assert.IsInstanceOfType<TaskCanceledException>(exception);
     }
+
+    [TestMethod]
+    public async Task CreateHead_ReturnsSuccessResult()
+    {
+        // Arrange
+        var expectedContent = "Head Success";
+        using var response = new HttpResponseMessage(HttpStatusCode.OK)
+        {
+            Content = new StringContent(expectedContent),
+        };
+        using var httpClient = CreateMockHttpClientFactory(response: response).CreateClient();
+
+        var head = CreateHead<string, MyErrorModel, int>(
+            url: "http://test.com".ToAbsoluteUrl(),
+            buildRequest: id => new HttpRequestParts(
+                RelativeUrl: new RelativeUrl($"/items/{id}"),
+                Body: null,
+                Headers: null
+            ),
+            deserializeSuccess: TestDeserializer.Deserialize<string>,
+            deserializeError: TestDeserializer.Deserialize<MyErrorModel>
+        );
+
+        // Act
+        var result = await head(httpClient, 123).ConfigureAwait(false);
+
+        // Assert
+        var successValue = +result;
+        Assert.AreEqual(expectedContent, successValue);
+    }
+
+    [TestMethod]
+    public async Task CreateHead_ErrorResponse_ReturnsFailureResult()
+    {
+        // Arrange
+        var expectedErrorContent = "Head Failed";
+        using var errorResponse = new HttpResponseMessage(HttpStatusCode.BadRequest)
+        {
+            Content = new StringContent( /*lang=json,strict*/
+                "{\"message\":\"Head Failed\"}"
+            ),
+        };
+        using var httpClient = CreateMockHttpClientFactory(response: errorResponse).CreateClient();
+
+        var head = CreateHead<string, MyErrorModel, int>(
+            url: "http://test.com".ToAbsoluteUrl(),
+            buildRequest: id => new HttpRequestParts(
+                RelativeUrl: new RelativeUrl($"/items/{id}"),
+                Body: null,
+                Headers: null
+            ),
+            deserializeSuccess: TestDeserializer.Deserialize<string>,
+            deserializeError: TestDeserializer.Deserialize<MyErrorModel>
+        );
+
+        // Act
+        var result = await head(httpClient, 123).ConfigureAwait(false);
+
+        // Assert
+        var httpError = !result;
+        if (httpError is not ResponseError(var body, var statusCode, var headers))
+        {
+            throw new InvalidOperationException("Expected error response");
+        }
+
+        Assert.AreEqual(HttpStatusCode.BadRequest, statusCode);
+        Assert.AreEqual(expectedErrorContent, body.Message);
+    }
+
+    [TestMethod]
+    public async Task CreateHead_CancellationToken_CancelsRequest()
+    {
+        // Arrange
+        using var cts = new CancellationTokenSource();
+        using var httpClient = CreateMockHttpClientFactory(
+                exceptionToThrow: new TaskCanceledException()
+            )
+            .CreateClient();
+
+        var head = CreateHead<string, MyErrorModel, int>(
+            url: "http://test.com".ToAbsoluteUrl(),
+            buildRequest: id => new HttpRequestParts(
+                RelativeUrl: new RelativeUrl($"/items/{id}"),
+                Body: null,
+                Headers: null
+            ),
+            deserializeSuccess: TestDeserializer.Deserialize<string>,
+            deserializeError: TestDeserializer.Deserialize<MyErrorModel>
+        );
+
+        await cts.CancelAsync().ConfigureAwait(false);
+
+        // Act
+        var result = await head(httpClient, 123, cts.Token).ConfigureAwait(false);
+
+        // Assert
+        var exception = !result switch
+        {
+            ExceptionError(var ex) => ex,
+            ResponseError(var b, var sc, var h) => throw new InvalidOperationException(
+                "Expected exception error"
+            ),
+        };
+
+        Assert.IsInstanceOfType<TaskCanceledException>(exception);
+    }
+
+    [TestMethod]
+    public async Task CreateOptions_ReturnsSuccessResult()
+    {
+        // Arrange
+        var expectedContent = "Options Success";
+        using var response = new HttpResponseMessage(HttpStatusCode.OK)
+        {
+            Content = new StringContent(expectedContent),
+        };
+        using var httpClient = CreateMockHttpClientFactory(response: response).CreateClient();
+
+        var options = CreateOptions<string, MyErrorModel, int>(
+            url: "http://test.com".ToAbsoluteUrl(),
+            buildRequest: id => new HttpRequestParts(
+                RelativeUrl: new RelativeUrl($"/items/{id}"),
+                Body: null,
+                Headers: null
+            ),
+            deserializeSuccess: TestDeserializer.Deserialize<string>,
+            deserializeError: TestDeserializer.Deserialize<MyErrorModel>
+        );
+
+        // Act
+        var result = await options(httpClient, 123).ConfigureAwait(false);
+
+        // Assert
+        var successValue = +result;
+        Assert.AreEqual(expectedContent, successValue);
+    }
+
+    [TestMethod]
+    public async Task CreateOptions_ErrorResponse_ReturnsFailureResult()
+    {
+        // Arrange
+        var expectedErrorContent = "Options Failed";
+        using var errorResponse = new HttpResponseMessage(HttpStatusCode.BadRequest)
+        {
+            Content = new StringContent( /*lang=json,strict*/
+                "{\"message\":\"Options Failed\"}"
+            ),
+        };
+        using var httpClient = CreateMockHttpClientFactory(response: errorResponse).CreateClient();
+
+        var options = CreateOptions<string, MyErrorModel, int>(
+            url: "http://test.com".ToAbsoluteUrl(),
+            buildRequest: id => new HttpRequestParts(
+                RelativeUrl: new RelativeUrl($"/items/{id}"),
+                Body: null,
+                Headers: null
+            ),
+            deserializeSuccess: TestDeserializer.Deserialize<string>,
+            deserializeError: TestDeserializer.Deserialize<MyErrorModel>
+        );
+
+        // Act
+        var result = await options(httpClient, 123).ConfigureAwait(false);
+
+        // Assert
+        var httpError = !result;
+        if (httpError is not ResponseError(var body, var statusCode, var headers))
+        {
+            throw new InvalidOperationException("Expected error response");
+        }
+
+        Assert.AreEqual(HttpStatusCode.BadRequest, statusCode);
+        Assert.AreEqual(expectedErrorContent, body.Message);
+    }
+
+    [TestMethod]
+    public async Task CreateOptions_CancellationToken_CancelsRequest()
+    {
+        // Arrange
+        using var cts = new CancellationTokenSource();
+        using var httpClient = CreateMockHttpClientFactory(
+                exceptionToThrow: new TaskCanceledException()
+            )
+            .CreateClient();
+
+        var options = CreateOptions<string, MyErrorModel, int>(
+            url: "http://test.com".ToAbsoluteUrl(),
+            buildRequest: id => new HttpRequestParts(
+                RelativeUrl: new RelativeUrl($"/items/{id}"),
+                Body: null,
+                Headers: null
+            ),
+            deserializeSuccess: TestDeserializer.Deserialize<string>,
+            deserializeError: TestDeserializer.Deserialize<MyErrorModel>
+        );
+
+        await cts.CancelAsync().ConfigureAwait(false);
+
+        // Act
+        var result = await options(httpClient, 123, cts.Token).ConfigureAwait(false);
+
+        // Assert
+        var exception = !result switch
+        {
+            ExceptionError(var ex) => ex,
+            ResponseError(var b, var sc, var h) => throw new InvalidOperationException(
+                "Expected exception error"
+            ),
+        };
+
+        Assert.IsInstanceOfType<TaskCanceledException>(exception);
+    }
 }