Skip to content

V10.2.1/service update#144

Merged
gimlichael merged 7 commits intomainfrom
v10.2.1/service-update
Feb 12, 2026
Merged

V10.2.1/service update#144
gimlichael merged 7 commits intomainfrom
v10.2.1/service-update

Conversation

@gimlichael
Copy link
Member

@gimlichael gimlichael commented Feb 11, 2026

This pull request primarily updates package dependencies across multiple .nuget release notes files to ensure compatibility with the latest supported target frameworks. Additionally, it bumps the NGINX version in the .docfx/Dockerfile.docfx. These changes help maintain stability and security by keeping dependencies current.

Dependency upgrades for supported frameworks:

  • Updated dependencies to the latest compatible versions for all supported target frameworks in the following release notes files:
    • .nuget/Cuemon.AspNetCore.App/PackageReleaseNotes.txt
    • .nuget/Cuemon.AspNetCore.Authentication/PackageReleaseNotes.txt
    • .nuget/Cuemon.AspNetCore.Mvc/PackageReleaseNotes.txt
    • .nuget/Cuemon.AspNetCore.Razor.TagHelpers/PackageReleaseNotes.txt
    • .nuget/Cuemon.AspNetCore/PackageReleaseNotes.txt
    • .nuget/Cuemon.Core.App/PackageReleaseNotes.txt
    • .nuget/Cuemon.Core/PackageReleaseNotes.txt
    • .nuget/Cuemon.Data.Integrity/PackageReleaseNotes.txt
    • .nuget/Cuemon.Data.SqlClient/PackageReleaseNotes.txt
    • .nuget/Cuemon.Data/PackageReleaseNotes.txt
    • .nuget/Cuemon.Diagnostics/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore.Authentication/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore.Mvc.Formatters.Text.Json/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore.Mvc.Formatters.Xml/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore.Mvc.RazorPages/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore.Mvc/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore.Text.Json/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore.Xml/PackageReleaseNotes.txt
    • .nuget/Cuemon.Extensions.AspNetCore/PackageReleaseNotes.txt

Docker image update:

  • Bumped the NGINX version in .docfx/Dockerfile.docfx from 1.29.4-alpine to 1.29.5-alpine for improved security and stability.

Summary by CodeRabbit

  • New Features

    • Performance improvements for stream operations (reduced allocations and faster byte-array conversion)
  • Chores

    • Released version 10.2.1 across packages with .NET 10/.NET 9 support
    • Upgraded package dependencies to latest patch versions
    • Updated build/test container images and CI/test environment artifacts
  • Tests / Benchmarks

    • Added unit tests and benchmark reports validating stream behavior and performance

@gimlichael gimlichael self-assigned this Feb 11, 2026
Copilot AI review requested due to automatic review settings February 11, 2026 00:13
@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

Adds release notes for v10.2.1 across many packages, bumps central package versions, updates Docker/test environment images and docfx nginx tag, introduces stream performance optimizations with tests and benchmarks, removes a null-check in UnM49DataContainer, and updates CHANGELOG.md.

Changes

Cohort / File(s) Summary
Package release notes
.nuget/*/PackageReleaseNotes.txt
Inserted new Version 10.2.1 release block (Availability for .NET 10/.NET 9/NET Standard 2.0) and ALM note about dependency upgrades across ~39 package release notes; some files show minor formatting changes or relocation/removal of a prior 10.2.0 ALM line.
Central package versions
Directory.Packages.props
Bumped multiple package patch versions (Microsoft.Data.Sqlite, Microsoft.Extensions.*, System.Text.Json, Microsoft.Bcl.AsyncInterfaces, etc.) across target-framework-specific ItemGroups.
Build/configuration & docs
.docfx/Dockerfile.docfx, testenvironments.json, CHANGELOG.md
Updated ARG NGINX_VERSION from 1.29.4-alpine1.29.5-alpine; switched test runner image reference to codebeltnet/ubuntu-testrunner:...; added [10.2.1] entry to CHANGELOG.md.
Core IO implementation
src/Cuemon.Core/Extensions/IO/StreamDecoratorExtensions.cs
Refactored CopyStream and InvokeToByteArray to reduce allocations, use seekability checks, avoid redundant position calls, and return optimized buffers for zero-length/seekable streams.
Tests & benchmarks
test/Cuemon.Core.Tests/.../StreamDecoratorExtensionsTest.cs, tuning/.../StreamDecoratorExtensionsBenchmark.cs, reports/tuning/...-report-github.md
Added comprehensive unit tests for CopyStream/InvokeToByteArray behaviors, introduced BenchmarkDotNet benchmark class and generated benchmark report markdown.
Globalization tweak
src/Cuemon.Core/Globalization/UnM49DataContainer.cs
Removed explicit null-check/InvalidOperationException for missing embedded resource stream; condensed using expression.
Project references
test/Cuemon.Core.Tests/Cuemon.Core.Tests.csproj
Removed a ProjectReference to Cuemon.IO.Cuemon.IO.csproj.

Sequence Diagram(s)

(omitted — changes are not multi-component sequential flows requiring diagram)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 I nibbled bytes and chased a leak,
Patched the crates and warmed the creek,
Docker hummed a newer tune,
Streams now sprint beneath the moon,
Hop, the release is out—huzzah, quick peek! 🚀

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 8.70% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'V10.2.1/service update' is partially related to the changeset. It references the version number (10.2.1) which appears in the release notes, but 'service update' is a vague descriptor that doesn't clearly convey the main changes: dependency upgrades, stream performance optimizations, Docker image bumps, and package release notes updates. Consider a more specific title that highlights the primary changes, such as 'Bump dependencies to latest compatible versions and optimize stream operations' or 'V10.2.1: Update package dependencies and StreamDecoratorExtensions performance'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch v10.2.1/service-update

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
test/Cuemon.Core.Tests/Extensions/IO/StreamDecoratorExtensionsTest.cs (1)

174-182: Consider adding a test for the zero-length non-MemoryStream path.

InvokeToByteArray_ShouldHandleEmptyStream uses a MemoryStream, which hits the is MemoryStream early-return (calling s.ToArray()). It doesn't exercise the new canSeek && length == 0 → Array.Empty<byte>() fast path (Line 75-79 in StreamDecoratorExtensions.cs). A test wrapping an empty MemoryStream in a BufferedStream would cover that branch.

🧪 Proposed additional test
[Fact]
public void InvokeToByteArray_ShouldReturnEmptyArray_ForZeroLengthNonMemoryStream()
{
    using var stream = new BufferedStream(new MemoryStream());

    var result = Decorator.Enclose(stream).InvokeToByteArray(leaveOpen: true);

    Assert.Empty(result);
    Assert.Same(Array.Empty<byte>(), result);
}

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request is a service update (version 10.2.1) that primarily focuses on updating package dependencies to their latest compatible versions and updating container images for improved security and stability. The changes are straightforward maintenance updates with no functional changes to the codebase, aside from a minor code simplification.

Changes:

  • Updated Microsoft package dependencies from 10.0.2 to 10.0.3 for .NET 10/netstandard2, and from 9.0.12 to 9.0.13 for .NET 9
  • Updated Docker images: NGINX from 1.29.4-alpine to 1.29.5-alpine, and ubuntu-testrunner to latest SDK versions
  • Added version 10.2.1 release notes to 42+ package release note files with consistent messaging about dependency upgrades
  • Minor code cleanup in UnM49DataContainer.cs removing redundant null check

Reviewed changes

Copilot reviewed 47 out of 47 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Directory.Packages.props Updated Microsoft package versions to 10.0.3 (net10/netstandard2) and 9.0.13 (net9)
CHANGELOG.md Added version 10.2.1 entry with TBD placeholder date
.docfx/Dockerfile.docfx Bumped NGINX version from 1.29.4-alpine to 1.29.5-alpine
testenvironments.json Updated Docker test runner image to latest SDK versions (8.0.418, 9.0.311, 10.0.103)
src/Cuemon.Core/Globalization/UnM49DataContainer.cs Simplified code by removing redundant null check and line formatting
.nuget/*/PackageReleaseNotes.txt (42 files) Added consistent 10.2.1 release notes documenting dependency upgrades for all packages

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

❌ Patch coverage is 89.65517% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.53%. Comparing base (f95659e) to head (44acb7b).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...on.Core/Extensions/IO/StreamDecoratorExtensions.cs 89.28% 3 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #144   +/-   ##
=======================================
  Coverage   80.53%   80.53%           
=======================================
  Files         598      598           
  Lines       18826    18839   +13     
  Branches     1934     1936    +2     
=======================================
+ Hits        15161    15172   +11     
- Misses       3599     3601    +2     
  Partials       66       66           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sonarqubecloud
Copy link

@gimlichael gimlichael merged commit 80260fe into main Feb 12, 2026
305 checks passed
@gimlichael gimlichael deleted the v10.2.1/service-update branch February 12, 2026 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant