[edit] simplify CodeQL workflow, clean up code style

- Switch CodeQL runner from windows-2025-vs2026 to ubuntu-latest
- Replace manual dotnet build with build-mode: none in CodeQL init
- Update codeql-action from v3 to v4, revert checkout from v7 to v6
- Remove boilerplate comments from codeql-analysis.yml
- Convert FooModel.cs to file-scoped namespace
- Remove redundant _statusCode backing field in Json.cs, use primary constructor param directly

Author: i4004

.github/workflows/codeql-analysis.yml

@@ -1,14 +1,3 @@
-# For most projects, this workflow file will not need changing; you simply need
-# to commit it to your repository.
-#
-# You may wish to alter this file to override the set of languages analyzed,
-# or to provide custom queries or build logic.
-#
-# ******** NOTE ********
-# We have attempted to detect the languages in your repository. Please check
-# the `language` matrix defined below to confirm you have the correct set of
-# supported CodeQL languages.
-#
 name: "CodeQL"
 
 on:
@@ -22,7 +11,7 @@ on:
 jobs:
   analyze:
     name: Analyze
-    runs-on: windows-2025-vs2026
+    runs-on: ubuntu-latest
     permissions:
       actions: read
       contents: read
@@ -32,31 +21,16 @@ jobs:
       fail-fast: false
       matrix:
         language: ["csharp"]
-        # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python' ]
-        # Learn more:
-        # https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#changing-the-languages-that-are-analyzed
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
 
-      # Initializes the CodeQL tools for scanning.
       - name: Initialize CodeQL
-        uses: github/codeql-action/init@v3
+        uses: github/codeql-action/init@v4
         with:
           languages: ${{ matrix.language }}
-          # If you wish to specify custom queries, you can do so here or in a config file.
-          # By default, queries listed here will override any specified in a config file.
-          # Prefix the list here with "+" to use these queries and those in the config file.
-          # queries: ./path/to/local/query, your-org/your-repo/queries@main
-
-      - name: Setup .NET
-        uses: actions/setup-dotnet@v5
-        with:
-          dotnet-version: 9.0.x
-
-      # Build
-      - run: dotnet build src/Simplify.Web.slnx -v minimal
+          build-mode: none
 
       - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@v3
+        uses: github/codeql-action/analyze@v4

src/Simplify.Web.Tests/Model/Binding/Binders/TestTypes/FooModel.cs

@@ -1,8 +1,7 @@
-namespace Simplify.Web.Tests.Model.Binding.Binders.TestTypes
+namespace Simplify.Web.Tests.Model.Binding.Binders.TestTypes;
+
+public class FooModel
 {
-	public class FooModel
-	{
-		public int ID { get; set; }
-		public string? Name { get; set; }
-	}
-}
+	public int ID { get; set; }
+	public string? Name { get; set; }
+}

src/Simplify.Web/Responses/Json.cs

@@ -14,14 +14,6 @@ namespace Simplify.Web.Responses;
 /// <param name="statusCode">The HTTP response status code.</param>
 public class Json(object objectToConvert, int statusCode = 200) : ControllerResponse
 {
-	/// <summary>
-	/// Gets the HTTP response status code.
-	/// </summary>
-	/// <value>
-	/// The HTTP response status code.
-	/// </value>
-	private readonly int _statusCode = statusCode;
-
 	/// <summary>
 	/// Gets or sets the default JsonSerializerOptions
 	/// </summary>
@@ -36,7 +28,7 @@ public class Json(object objectToConvert, int statusCode = 200) : ControllerResp
 	public override async Task<ResponseBehavior> ExecuteAsync()
 	{
 		Context.Response.ContentType = "application/json";
-		Context.Response.StatusCode = _statusCode;
+		Context.Response.StatusCode = statusCode;
 
 		await ResponseWriter.WriteAsync(Context.Response, JsonSerializer.Serialize(objectToConvert, DefaultOptions));