Skip to content

Bump Meziantou.Analyzer from 3.0.98 to 3.0.114#181

Merged
ANcpLua merged 2 commits into
mainfrom
dependabot/nuget/Meziantou.Analyzer-3.0.113
Jun 30, 2026
Merged

Bump Meziantou.Analyzer from 3.0.98 to 3.0.114#181
ANcpLua merged 2 commits into
mainfrom
dependabot/nuget/Meziantou.Analyzer-3.0.113

Conversation

@ANcpLua

@ANcpLua ANcpLua commented Jun 30, 2026

Copy link
Copy Markdown
Owner

updated-dependencies:

  • dependency-name: Meziantou.Analyzer dependency-version: 3.0.113 dependency-type: direct:production update-type: version-update:semver-patch ...

---
updated-dependencies:
- dependency-name: Meziantou.Analyzer
  dependency-version: 3.0.113
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 0f6343b6-772b-4abc-a168-cc3c2b735107

📥 Commits

Reviewing files that changed from the base of the PR and between 9343663 and c382f47.

📒 Files selected for processing (1)
  • Version.props
📜 Recent review details
⏰ Context from checks skipped due to timeout. (2)
  • GitHub Check: aot-publish
  • GitHub Check: GitGuardian Security Checks
🧰 Additional context used
📓 Path-based instructions (3)
*

📄 CodeRabbit inference engine (.editorconfig)

*: Use spaces for indentation with 4-space indentation size
Use LF (Unix-style) line endings
Use UTF-8 charset for all files
Trim trailing whitespace from all lines
Insert a final newline at the end of all files

Files:

  • Version.props
**

⚙️ CodeRabbit configuration file

**: # ErrorOrX

Source generator converting ErrorOr<T> handlers into ASP.NET Core Minimal API endpoints with full Native AOT support.

Automatic Routing (for Claude)

Always invoke /working-in-erroror skill when starting work in this repo.

Task Use
Implementation Task tool → erroror-generator-specialist
Debugging Task tool → deep-debugger
Before completion Run dotnet build + dotnet test, show output
Cross-repo work Invoke /ancplua-ecosystem first

Quick Reference

dotnet build ErrorOrX.slnx
dotnet test --solution ErrorOrX.slnx # VERIFY
dotnet pack src/ErrorOrX/ErrorOrX.csproj -c Release
dotnet pack src/ErrorOrX.Generators/ErrorOrX.Generators.csproj -c Release

Note: The # VERIFY comment bypasses the MTP smart-test-filtering hook that blocks full suite runs. Always
include it for verification runs.

What the Generator Does

Convert ErrorOr<T> handlers into fully-wired ASP.NET endpoints:

User writes:                         Generator produces:
[Get("/todos/{id:guid}")]             app.MapGet("/todos/{id:guid}", (Delegate)Invoke_Ep1)
ErrorOr<Todo> GetById(Guid id)  ->       .WithName("TodoApi_GetById")
                                         .WithMetadata(new ProducesResponseTypeMetadata(...))
                                         .RequireAuthorization("Admin")
                                         ;

                                     static async Task<Results<Ok<Todo>, ...>> Invoke_Ep1(HttpContext ctx)
                                     {
                                         return await Invoke_Ep1_Core(ctx);
                                     }

                                     static Task<Results<Ok<Todo>, ...>> Invoke_Ep1_Core(...)
                                     {
        ...

Files:

  • Version.props
**/*.props

⚙️ CodeRabbit configuration file

MSBuild property files (Directory.Build.props, Directory.Packages.props, Version.props). Review for: Central Package Management correctness, version consistency, and that new packages are added with explicit version pins. Flag transitive dependency promotions that aren't justified. Verify TFM targeting is correct (.NET 10).

Files:

  • Version.props
🔇 Additional comments (1)
Version.props (1)

25-25: LGTM!


Summary by CodeRabbit

  • Chores
    • Updated the analyzer package version to a newer release, bringing the latest improvements and fixes for projects using it.

Walkthrough

Version.props bumps MeziantouAnalyzerVersion from 3.0.98 to 3.0.114. No other changes.

Analyzer Version Bump

Layer / File(s) Summary
MeziantouAnalyzerVersion update
Version.props
Increments MeziantouAnalyzerVersion from 3.0.98 to 3.0.114.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

area:infra


Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Title check ❌ Error The title matches the dependency bump, but it does not follow the required conventional-commit format with an allowed area scope. Use a conventional commit like chore(infra): bump Meziantou.Analyzer to 3.0.114 and keep it under 72 characters.
✅ Passed checks (7 passed)
Check name Status Explanation
Description check ✅ Passed The description clearly refers to the Meziantou.Analyzer dependency update, so it is relevant to the changeset.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Otel Instrumentation Required ✅ Passed Only Version.props changed, bumping MeziantouAnalyzerVersion; no new DI-registered classes or ActivitySource/Meter additions were introduced.
No Unbounded Mcp Responses ✅ Passed PASS: The PR only changes Version.props, and the repo contains no src/qyl.mcp/ path, so no MCP tool definitions were modified.
Duckdb Backpressure On Write Paths ✅ Passed Diff only bumps MeziantouAnalyzerVersion in Version.props; no code changes or DuckDB write paths were added.
Cancellationtoken Threading ✅ Passed No src/**/*.cs files changed; the PR only updates Version.props, so there are no new public async methods to audit.

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

@ANcpLua ANcpLua merged commit f04606c into main Jun 30, 2026
4 of 5 checks passed
@ANcpLua ANcpLua deleted the dependabot/nuget/Meziantou.Analyzer-3.0.113 branch June 30, 2026 11:58
ANcpLua added a commit that referenced this pull request Jun 30, 2026
…u 3.0.114 bump (#182)

Meziantou.Analyzer 3.0.98 -> 3.0.114 (#181) newly enforces MA0074 on the
five `string.Contains(string)` call sites in ErrorOrX.Generators, which
under TreatWarningsAsErrors broke `main` (AOT Publish Smoke, run 28442562600).

The netstandard2.0 BCL lacks `Contains(string, StringComparison)`; the
ANcpLua.Roslyn.Utilities.Sources polyfill supplies it (that overload's
existence is precisely what made MA0074 start firing). Rather than the BCL
overload, switch to the package's `ContainsOrdinal` helper — the same
ordinal-explicit idiom already used alongside (`StartsWithOrdinal`,
`EndsWithOrdinal`). Behavior is identical (default string.Contains is already
ordinal); the rule is satisfied by construction.

Verified: `dotnet build -c Release` of the generator project and full slnx
both 0 warnings / 0 errors; 156 generator tests pass.

Co-authored-by: ANcpLua <alex_nachtmann@yahoo.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant