build(deps): bump ANcpLua.Analyzers 1.27.4 → 1.29.0

[ANcpLua]

Summary

• Bump ANcpLua.Analyzers from 1.27.4 → 1.29.0
• AL0038 (GetOrNull/GetOrDefault suggestion) removed — BCL GetValueOrDefault covers the use case without the dual-overload ambiguity bug

Test plan

• Version pin change only — no code changes

🤖 Generated with Claude Code

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 739f2fcc-92db-4c94-b5fe-dd12e58b89e1

📥 Commits

Reviewing files that changed from the base of the PR and between a3d252f and d05d056.

📒 Files selected for processing (1)

• src/Build/Common/Version.props

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting

---

Summary by CodeRabbit

• Chores
  • Updated analyzer package version to the latest release to improve overall stability, reliability, and maintenance of the development tooling. This change helps catch issues earlier and contributes to a smoother build and testing experience for the team.

Walkthrough

Updated the ANcpLuaAnalyzersVersion property in src/Build/Common/Version.props from 1.27.4 to 1.29.0. No other properties changed.

Changes

Cohort / File(s)	Summary
Build Configuration src/Build/Common/Version.props	Bumped ANcpLuaAnalyzersVersion from 1.27.4 to 1.29.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

area:infra

🚥 Pre-merge checks | ✅ 8

✅ Passed checks (8 passed)

Check name	Status	Explanation
Title check	✅ Passed	Title follows conventional commits format with 'build(deps)' prefix, clearly describes the dependency bump, and is 51 characters—well under the 72-character limit.
Description check	✅ Passed	Description is directly related to the changeset, documenting the version bump, rationale for the analyzer rule removal, and test plan.
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	PR updates only a dependency version pin without adding new injectable services, making OpenTelemetry instrumentation verification inapplicable.
No Unbounded Mcp Responses	✅ Passed	PR only modifies Version.props to bump ANcpLua.Analyzers; MCP tool directory does not exist and is unmodified.
Duckdb Backpressure On Write Paths	✅ Passed	Version pin update to ANcpLuaAnalyzersVersion in build configuration—no new code paths, DuckDB interactions, or write operations.
Cancellationtoken Threading	✅ Passed	PR only modifies build configuration file; no new public async methods added.

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

---

_{Review rate limit: 1/5 review remaining, refill in 45 minutes and 4 seconds.}

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

[gemini-code-assist]

Code Review

This pull request updates the ANcpLuaAnalyzersVersion from 1.27.4 to 1.29.0 in the Version.props file. I have no feedback to provide as there are no review comments.

[Copilot]

Pull request overview

Updates the centralized analyzer version pin used by the build so the repo consumes ANcpLua.Analyzers v1.29.0.

Changes:

• Bump ANcpLuaAnalyzersVersion from 1.27.4 to 1.29.0.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.