refactor: .NET 10 update (#355)

* chore: update project SDKs and target frameworks; remove obsolete Windows-specific converters

- Changed global.json to use new SDK versioning format.
- Updated Directory.Build.props to enable nullable reference types and continuous integration build settings.
- Refactored MADE.Collections project to target net8.0 and net10.0, switching to Microsoft.NET.Sdk.
- Deleted obsolete Windows-specific BooleanToStringValueConverter and DateTimeToStringValueConverter files.
- Updated MADE.Data.Converters to target net8.0 and net10.0, refactoring converters to remove partial class definitions.
- Updated MADE.Data.EFCore project to target net8.0 and net10.0, updating package references for EF Core.
- Updated all MADE.* projects to use Microsoft.NET.Sdk and target net8.0 and net10.0.
- Updated test projects to target net8.0 and net10.0, updating package references to latest versions.

* feat: migrate to System.Text.Json and drop third-party dependencies

Replace Newtonsoft.Json with System.Text.Json across Networking, Web, Web.Mvc, and Data.Serialization libraries. Replace JsonTypeMigrationSerializationBinder with JsonTypeMigrationConverter using System.Text.Json's JsonConverter API.

Remove Z.EntityFramework.Plus.EFCore from Data.EFCore, reimplementing OrderBy using expression trees. Add QueryableExtensions tests for OrderBy and Page.

Add CHANGELOG.md documenting all v3 breaking changes with migration guidance.

* refactor: modernize codebase with file-scoped namespaces, nullable annotations, and async improvements

- Convert all source and test files to file-scoped namespace declarations
- Add ConfigureAwait(false) to all 52 await expressions in library code
- Replace manual null checks with ArgumentNullException.ThrowIfNull (22 locations)
- Add nullable reference type annotations to public APIs (13 fixes across 11 files)
- Change IEventLogger methods from void to Task for proper async support
- Simplify JsonTypeMigrationConverter: rename AddTypeMigrationAsync to AddTypeMigration, replace SemaphoreSlim with lock
- Fix flaky EntityBaseTests by capturing timestamps around save operations
- Add comprehensive .editorconfig with modern .NET analysis rules
- Update CHANGELOG.md with v3 breaking changes and QOL improvements

* refactor: add nullable annotations to improve code safety and clarity

* refactor: enhance nullability annotations and improve code safety across multiple files

* refactor: enhance nullability annotations and improve code safety across multiple files

* docs: enhance documentation and features across multiple articles

- Updated threading.md to include ITimer interface and TaskExtensions for better async task handling.
- Added JsonResult for returning JSON with custom status codes in web-mvc.md.
- Introduced AuthenticatedUser and API versioning support in web.md.
- Revised intro.md to reflect updated target frameworks and added new features in various packages.
- Removed obsolete Media Image section from the table of contents.
- Updated docfx.json to target net10.0 and modified templates for a modern look.
- Created a new custom CSS file for improved styling on the landing page.
- Deleted outdated material template files to streamline the documentation structure.
- Adjusted table of contents to point to articles instead of docs.

* feat: add assertion helpers, async primitives, and EFCore extensions

New features across multiple packages:
- Testing: assertion extensions for objects, booleans, comparables,
  strings, and exceptions (ShouldBeNull, ShouldBeTrue, ShouldThrow, etc.)
- Threading: AsyncLazy<T>, Debouncer, and Throttler
- Data.EFCore: soft-delete (ISoftDeletable) and audit trail
  (IAuditableEntity) with DbContext extensions
- Data.Validation: IAsyncValidator and AsyncValidatorCollection
- Networking: MultipartFormDataPostNetworkRequest and
  RetryDelegatingHandler with exponential backoff
- Web.Mvc: ForbiddenObjectResult (403) with controller extensions

Also adds CancellationToken support to async methods that were
missing it, and updates documentation for all new and existing
features.

* feat: add value converters and extensions for DateTime, string, and file size; enhance length and math conversions

* test: add tests for new features across all packages

* fix(tests): use mock handler for error tests, keep real service for success

The httpbin.org error tests were flaky in CI because the external
service can return transient errors (502) instead of the expected
status code (405). Success tests still hit httpbin.org to prove
real HTTP plumbing works. Error tests now use a MockHttpMessageHandler
for deterministic assertions.

* feat(networking): add INetworkRequestFactory for DI-friendly request creation

Wraps IHttpClientFactory to eliminate manual HttpClient management.
Supports named clients via WithClient() and one-line DI registration
with services.AddNetworkRequestFactory(). Documentation updated to
use the factory as the primary recommended approach.

Author: jamesmcroft

.editorconfig

@@ -1,7 +1,42 @@
 version: 2
 updates:
-- package-ecosystem: nuget
-  directory: "/"
-  schedule:
-    interval: daily
-  open-pull-requests-limit: 10
+  - package-ecosystem: nuget
+    directory: "/"
+    schedule:
+      interval: weekly
+      day: monday
+    open-pull-requests-limit: 10
+    commit-message:
+      prefix: "chore"
+      include: "scope"
+    groups:
+      testing:
+        patterns:
+          - "coverlet*"
+          - "Microsoft.NET.Test*"
+          - "NUnit*"
+          - "Shouldly"
+          - "Bogus"
+          - "Moq"
+      entity-framework:
+        patterns:
+          - "Microsoft.EntityFrameworkCore*"
+      aspnet:
+        patterns:
+          - "Asp.Versioning*"
+      analyzers:
+        patterns:
+          - "Microsoft.SourceLink*"
+      validation:
+        patterns:
+          - "FluentValidation*"
+
+  - package-ecosystem: github-actions
+    directory: "/"
+    schedule:
+      interval: weekly
+      day: monday
+    open-pull-requests-limit: 5
+    commit-message:
+      prefix: "ci"
+      include: "scope"

.github/dependabot.yml

@@ -8,34 +8,39 @@ on:
     - main
     paths:
     - src/**
-    - samples/**
     - tests/**
     - build/**
     - .github/workflows/ci.yml
+    - Directory.Build.props
     - global.json
   pull_request:
     branches:
     - main
     paths:
     - src/**
-    - samples/**
     - tests/**
     - build/**
     - .github/workflows/ci.yml
+    - Directory.Build.props
     - global.json
   workflow_dispatch:
 
+permissions:
+  contents: read
+  pull-requests: write
+
 jobs:
-  build:
+  build-test:
+    runs-on: ubuntu-latest
 
     env:
       BUILD_CONFIG: 'Release'
       SOLUTION: 'MADE.NET.sln'
 
-    runs-on: windows-latest
-
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v6
+      with:
+        fetch-depth: 0
 
     - name: Get Build Version
       run: |
@@ -45,33 +50,88 @@ jobs:
         echo "BUILD_VERSION=$version" | Out-File -FilePath $Env:GITHUB_ENV -Encoding utf-8 -Append
       shell: pwsh
 
-    - name: Setup .NET 6.0
-      uses: actions/setup-dotnet@v1
+    - name: Setup .NET 8.0
+      uses: actions/setup-dotnet@v5
+      with:
+        dotnet-version: 8.0.x
+
+    - name: Setup .NET 10.0
+      uses: actions/setup-dotnet@v5
       with:
-        dotnet-version: 6.0.x
+        dotnet-version: 10.0.3xx
+
+    - name: Restore dependencies
+      run: dotnet restore $SOLUTION
 
-    - name: Setup .NET 7.0
-      uses: actions/setup-dotnet@v1
+    - name: Build
+      run: dotnet build $SOLUTION --configuration $BUILD_CONFIG --no-restore -p:Version=$BUILD_VERSION
+
+    - name: Run tests
+      run: >
+        dotnet test
+        --configuration $BUILD_CONFIG
+        --no-restore
+        --no-build
+        --collect:"XPlat Code Coverage"
+
+    - name: Generate coverage report
+      uses: danielpalme/ReportGenerator-GitHub-Action@5
       with:
-        dotnet-version: 7.0.x
+        reports: '**/coverage.cobertura.xml'
+        targetdir: 'coverage-report'
+        reporttypes: 'MarkdownSummaryGithub;Cobertura'
+
+    - name: Publish coverage to workflow summary
+      run: cat coverage-report/SummaryGithub.md >> $GITHUB_STEP_SUMMARY
+
+    - name: Post coverage to PR
+      if: github.event_name == 'pull_request'
+      run: gh pr comment $PR_NUMBER --edit-last --create-if-none --body-file coverage-report/SummaryGithub.md
+      env:
+        GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        PR_NUMBER: ${{ github.event.number }}
+
+  publish:
+    needs: build-test
+    if: startsWith(github.ref, 'refs/tags/v')
+    runs-on: ubuntu-latest
+
+    env:
+      BUILD_CONFIG: 'Release'
+      SOLUTION: 'MADE.NET.sln'
+
+    steps:
+    - uses: actions/checkout@v6
+      with:
+        fetch-depth: 0
+
+    - name: Get Build Version
+      run: |
+        Import-Module .\build\GetBuildVersion.psm1
+        Write-Host $Env:GITHUB_REF
+        $version = GetBuildVersion -VersionString $Env:GITHUB_REF
+        echo "BUILD_VERSION=$version" | Out-File -FilePath $Env:GITHUB_ENV -Encoding utf-8 -Append
+      shell: pwsh
 
     - name: Setup .NET 8.0
-      uses: actions/setup-dotnet@v1
+      uses: actions/setup-dotnet@v5
       with:
         dotnet-version: 8.0.x
 
+    - name: Setup .NET 10.0
+      uses: actions/setup-dotnet@v5
+      with:
+        dotnet-version: 10.0.3xx
+
     - name: Restore dependencies
       run: dotnet restore $SOLUTION
 
-    - name: Setup MSBuild
-      uses: microsoft/setup-msbuild@v1.0.2
-
     - name: Build
-      run: msbuild $env:SOLUTION /p:Configuration=$env:BUILD_CONFIG /p:Platform="Any CPU" -p:Version=$env:BUILD_VERSION
+      run: dotnet build $SOLUTION --configuration $BUILD_CONFIG --no-restore -p:Version=$BUILD_VERSION
 
-    - name: Run tests
-      run: dotnet test /p:Configuration=$env:BUILD_CONFIG /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura --no-restore --no-build --verbosity normal
-
-    - name: Publish
-      if: startsWith(github.ref, 'refs/tags/v')
-      run: nuget push **\*.nupkg -Source 'https://api.nuget.org/v3/index.json' -ApiKey ${{secrets.NUGET_API_KEY}}
+    - name: Publish to NuGet.org
+      run: |
+        Get-ChildItem . -Recurse -Filter *.nupkg | ForEach-Object {
+          dotnet nuget push $_.FullName --source 'https://api.nuget.org/v3/index.json' --api-key '${{ secrets.NUGET_API_KEY }}' --skip-duplicate
+        }
+      shell: pwsh

.github/workflows/ci.yml

@@ -16,50 +16,41 @@ on:
     - .github/workflows/docs.yml
   workflow_dispatch:
 
+permissions:
+  contents: write
+
 jobs:
   generate-docs:
 
-    runs-on: windows-latest
+    runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v2
-
-    - name: Setup .NET 6.0
-      uses: actions/setup-dotnet@v1
-      with:
-        dotnet-version: 6.0.x
-
-    - name: Setup .NET 7.0
-      uses: actions/setup-dotnet@v1
-      with:
-        dotnet-version: 7.0.x
+    - uses: actions/checkout@v6
 
     - name: Setup .NET 8.0
-      uses: actions/setup-dotnet@v1
+      uses: actions/setup-dotnet@v5
       with:
         dotnet-version: 8.0.x
 
-    - name: Setup MSBuild
-      uses: microsoft/setup-msbuild@v1.0.2
-
-    - name: Setup DocFX
-      uses: crazy-max/ghaction-chocolatey@v1
+    - name: Setup .NET 10.0
+      uses: actions/setup-dotnet@v5
       with:
-        args: install docfx
+        dotnet-version: 10.0.3xx
 
-    - name: Generate DocFX metadata
-      working-directory: docs
-      run: docfx metadata .\docfx.json
-      continue-on-error: false
+    - name: Install DocFX
+      run: dotnet tool install -g docfx
+
+    - name: Restore projects
+      run: dotnet restore MADE.NET.sln
 
     - name: Build DocFX site
       working-directory: docs
-      run: docfx .\docfx.json
+      run: docfx docfx.json
       continue-on-error: false
 
     - name: Publish DocFX site
       if: github.event_name == 'push'
-      uses: peaceiris/actions-gh-pages@v3
+      uses: peaceiris/actions-gh-pages@v4
       with:
         github_token: ${{ secrets.GITHUB_TOKEN }}
         publish_dir: docs/_site

.github/workflows/docs.yml

@@ -0,0 +1,112 @@
+# Changelog
+
+## v3.0.0
+
+### Breaking Changes
+
+#### Target Framework Updates
+
+- All libraries now target `net8.0` and `net10.0`. Previous target frameworks have been removed.
+
+#### Newtonsoft.Json Replaced with System.Text.Json
+
+The following libraries have migrated from `Newtonsoft.Json` to `System.Text.Json`. All public APIs that previously accepted `Newtonsoft.Json.JsonSerializerSettings` now accept `System.Text.Json.JsonSerializerOptions`.
+
+**MADE.Web.Mvc**
+
+- `JsonResult` constructor parameter type changed from `JsonSerializerSettings` to `JsonSerializerOptions`.
+- `JsonResult.SerializerOptions` property type changed from `JsonSerializerSettings` to `JsonSerializerOptions`.
+- `ControllerBaseExtensions.Json()` parameter type changed from `JsonSerializerSettings` to `JsonSerializerOptions`.
+
+**MADE.Web**
+
+- All `HttpResponseExtensions.WriteJsonAsync()` overloads that accepted `JsonSerializerSettings` now accept `JsonSerializerOptions`.
+
+**MADE.Networking**
+
+- Internal serialization switched from `Newtonsoft.Json` to `System.Text.Json`. All deserialization uses `PropertyNameCaseInsensitive = true` to maintain behavioral compatibility.
+- No public API signature changes.
+
+**MADE.Data.Serialization**
+
+- `JsonTypeMigrationSerializationBinder` has been removed. Use `JsonTypeMigrationConverter` instead (see migration guide below).
+
+### Removed Dependencies
+
+| Library | Removed Dependency | Replacement |
+| --- | --- | --- |
+| MADE.Networking | `Newtonsoft.Json` | `System.Text.Json` (built-in) |
+| MADE.Web | `Newtonsoft.Json` | `System.Text.Json` (built-in) |
+| MADE.Web.Mvc | `Newtonsoft.Json` | `System.Text.Json` (built-in) |
+| MADE.Data.Serialization | `Newtonsoft.Json` | `System.Text.Json` (built-in) |
+| MADE.Data.EFCore | `Z.EntityFramework.Plus.EFCore` | Custom implementation using Expression trees |
+
+### Migration Guide
+
+#### Newtonsoft.Json to System.Text.Json
+
+Replace `using Newtonsoft.Json` with `using System.Text.Json` and update any `JsonSerializerSettings` references to `JsonSerializerOptions`.
+
+```csharp
+// Before (v2)
+using Newtonsoft.Json;
+
+var result = controller.Json(value, HttpStatusCode.OK, new JsonSerializerSettings
+{
+    NullValueHandling = NullValueHandling.Ignore
+});
+
+// After (v3)
+using System.Text.Json;
+
+var result = controller.Json(value, HttpStatusCode.OK, new JsonSerializerOptions
+{
+    DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull
+});
+```
+
+#### JsonTypeMigrationSerializationBinder to JsonTypeMigrationConverter
+
+The Newtonsoft.Json-based `JsonTypeMigrationSerializationBinder` in the `MADE.Data.Serialization.Json.Binders` namespace has been replaced with `JsonTypeMigrationConverter` in the `MADE.Data.Serialization.Json.Converters` namespace.
+
+```csharp
+// Before (v2)
+using MADE.Data.Serialization.Json.Binders;
+
+var binder = new JsonTypeMigrationSerializationBinder();
+binder.AddTypeMigration(new JsonTypeMigration("OldAssembly", "OldNamespace.OldType", typeof(NewType)));
+
+var settings = new JsonSerializerSettings
+{
+    TypeNameHandling = TypeNameHandling.All,
+    SerializationBinder = binder
+};
+var result = JsonConvert.DeserializeObject<object>(json, settings);
+
+// After (v3)
+using MADE.Data.Serialization.Json.Converters;
+
+var converter = new JsonTypeMigrationConverter();
+converter.AddTypeMigration(new JsonTypeMigration("OldAssembly", "OldNamespace.OldType", typeof(NewType)));
+
+var options = new JsonSerializerOptions();
+options.Converters.Add(converter);
+var result = JsonSerializer.Deserialize<object>(json, options);
+```
+
+#### IEventLogger Methods Changed from void to Task
+
+All 15 methods on `IEventLogger` (`WriteDebug`, `WriteInfo`, `WriteWarning`, `WriteError`, `WriteCritical` and their overloads) now return `Task` instead of `void`. Implementations must be updated accordingly.
+
+#### JsonTypeMigrationConverter Simplified
+
+- `AddTypeMigrationAsync` has been renamed to `AddTypeMigration` and is now synchronous (uses `lock` instead of `SemaphoreSlim`).
+
+### Code Quality Improvements
+
+- **File-scoped namespaces**: All source files converted to file-scoped namespace declarations.
+- **ConfigureAwait(false)**: Added to all `await` expressions in library code (52 locations across 17 files) to prevent deadlocks in synchronization-context-bound environments.
+- **ArgumentNullException.ThrowIfNull**: Replaced manual null-check-and-throw patterns with `ArgumentNullException.ThrowIfNull()` (22 locations across 7 files).
+- **Nullable reference type annotations**: Added `?` annotations to parameters, return types, fields, and properties that accept or return `null` (17 fixes across 10 files).
+- **Async correctness**: `FileEventLogger` and `AppDiagnostics` rewritten for proper async patterns, removing `async void` methods.
+- **Comprehensive .editorconfig**: Added modern .NET analysis rules including CA2007, CA1822, CA1849, and async naming conventions.