Skip to content

V10.1.6/service update - #34

Merged
gimlichael merged 6 commits into
mainfrom
v10.1.6/service-update
Jul 22, 2026
Merged

V10.1.6/service update#34
gimlichael merged 6 commits into
mainfrom
v10.1.6/service-update

Conversation

@codebelt-aicia

Copy link
Copy Markdown
Contributor

This is a service update that focuses on package dependencies.

Automated changes:

  • Codebelt/Cuemon package versions bumped to latest compatible
  • PackageReleaseNotes.txt updated for v10.1.6
  • CHANGELOG.md entry added for v10.1.6

Note: Third-party packages (Microsoft.Extensions.*, BenchmarkDotNet, etc.) are not auto-updated.
Use Dependabot or manual updates for those.

Generated by codebelt-aicia
Triggered by: xunit @ 11.1.2

@greptile-apps

greptile-apps Bot commented Jul 18, 2026

Copy link
Copy Markdown

Greptile Summary

This is a routine service update (v10.1.6) that bumps Codebelt/Cuemon package versions, upgrades xunit and Microsoft.NET.Test.Sdk, and refactors the HttpExceptionDescriptor YAML serialization into extracted private helper methods. Build configuration is tightened with stricter analyzer settings for source projects.

  • Dependency bumps: Codebelt.Extensions.Xunit → 11.1.2, all Cuemon.* packages → 10.5.5, Microsoft.NET.Test.Sdk → 18.8.1.
  • Directory.Build.props refinement: conditions on NuGet/publisher settings switched from IsTestProject == false to the more precise IsSourceProject == true; AnalysisLevel=latest + AnalysisMode=Recommended added for source projects; AnalysisLevel=none in the test project block replaced with AnalysisMode=None (analyzers are already disabled via RunAnalyzers=false, so the effect is equivalent).
  • Refactoring: AddHttpExceptionDescriptorConverter now delegates to extracted WriteError, WriteEvidence, WriteOptionalString, and WriteOptionalUri helpers; a new unit test covers the serialized output.

Confidence Score: 5/5

Safe to merge — all changes are dependency bumps, build configuration tightening, and a pure serialization refactor backed by a new unit test.

The dependency updates are patch/minor increments within the same major version. The build property changes are precise and internally consistent (analyzer suppression in test projects is still enforced via RunAnalyzers=false regardless of the AnalysisLevel→AnalysisMode swap). The YAML serialization refactoring preserves the exact writer open/close nesting of the original code, and the new test validates the output structure end-to-end.

No files require special attention.

Important Files Changed

Filename Overview
Directory.Build.props Build configuration refined: new IsSourceProject/IsBenchmarkProject/IsToolingProject predicates, publisher settings scoped to source projects only, AnalysisLevel+AnalysisMode added for source, AnalysisLevel→AnalysisMode swap in test block (safe because RunAnalyzers=false already disables analyzer execution).
Directory.Packages.props Package versions bumped: Codebelt.Extensions.Xunit 11.1.1→11.1.2, Cuemon.* 10.5.4→10.5.5, Microsoft.NET.Test.Sdk 18.7.0→18.8.1; no structural changes.
src/Codebelt.Extensions.AspNetCore.Text.Yaml/Converters/YamlConverterExtensions.cs HttpExceptionDescriptor serialization refactored into private helpers (WriteError, WriteEvidence, WriteOptionalString, WriteOptionalUri); logic and object open/close nesting are preserved correctly.
test/Codebelt.Extensions.AspNetCore.Text.Yaml.Tests/Converters/YamlConverterExtensionsTest.cs New unit test added covering the refactored AddHttpExceptionDescriptorConverter path, asserting YAML structure, optional field presence, and absence of empty Evidence block.
src/Codebelt.Extensions.YamlDotNet/Converters/ExceptionDescriptorConverter.cs Documentation-only fix: XML cref corrected from T:System.Type to Type for consistent doc generation.
.docfx/Dockerfile.docfx Nginx base image tag changed from 1.31.2-alpine (pinned patch) to 1.31-alpine (floating minor tag); intentional per CHANGELOG entry.
CHANGELOG.md v10.1.6 entry added with accurate description of all changes; comparison links updated correctly.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[AddHttpExceptionDescriptorConverter] --> B[WriteHttpExceptionDescriptor]
    B --> C[WriteStartObject - root]
    B --> D[WritePropertyName Error]
    B --> E[WriteError]
    E --> E1[WriteStartObject]
    E --> E2[WriteOptionalUri - Instance]
    E --> E3[WriteString - Status/Code/Message]
    E --> E4[WriteOptionalUri - HelpLink]
    E --> E5[WriteFailure]
    E5 -->|HasFlag Failure| E5a[ExceptionConverter.WriteYaml]
    E --> E6[WriteEndObject]
    B --> F[WriteEvidence]
    F -->|HasFlag Evidence AND Any| F1[WriteStartObject / forEach / WriteEndObject]
    B --> G[WriteOptionalString - CorrelationId]
    B --> H[WriteOptionalString - RequestId]
    B --> I[WriteOptionalString - TraceId]
    B --> J[WriteEndObject - root]
Loading

Reviews (2): Last reviewed commit: "🔧 update changelog for version 10.1.6 w..." | Re-trigger Greptile

Comment thread Directory.Packages.props Outdated
Comment on lines 25 to 26
</ItemGroup>
</Project> No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 The automated update removed the trailing newline from this file (the diff shows No newline at end of file). Most editors, linters, and POSIX tools expect files to end with a newline — this was not an issue in the prior version.

Suggested change
</ItemGroup>
</Project>
</ItemGroup>
</Project>
Prompt To Fix With AI
This is a comment left during a code review.
Path: Directory.Packages.props
Line: 25-26

Comment:
The automated update removed the trailing newline from this file (the diff shows `No newline at end of file`). Most editors, linters, and POSIX tools expect files to end with a newline — this was not an issue in the prior version.

```suggestion
  </ItemGroup>
</Project>
```

How can I resolve this? If you propose a fix, please make it concise.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.54%. Comparing base (9abf797) to head (0e15d49).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   97.10%   97.54%   +0.44%     
==========================================
  Files          18       18              
  Lines         621      612       -9     
  Branches       57       54       -3     
==========================================
- Hits          603      597       -6     
+ Misses         18       15       -3     

☔ View full report in Codecov by Harness.
📢 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.

aicia-bot and others added 5 commits July 22, 2026 23:36
Update the shared test SDK baseline to 18.8.1 so the repository stays on the latest stable runner package.
Scope shared package metadata to source projects, enable the newer analyzer defaults, and keep the DocFX nginx base image on the maintained 1.31 line.
Break the HttpExceptionDescriptor YAML serialization path into focused helpers and add coverage that locks the emitted shape during the refactor.
Replace the legacy framework type reference in the XML docs with current cref syntax so generated documentation resolves it consistently.
@gimlichael
gimlichael merged commit 091753f into main Jul 22, 2026
24 checks passed
@gimlichael
gimlichael deleted the v10.1.6/service-update branch July 22, 2026 21:56
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.

2 participants