chore(deps): update dependency polly to 8.6.4

[thomhurst]

This PR contains the following updates:

Package	Type	Update	Change
Polly	nuget	patch	8.6.3 -> 8.6.4

---

Release Notes

App-vNext/Polly (Polly)

v8.6.4

Compare Source

• Fix after-release and update CHANGELOG by @​martincostello in #​2714
• Use collection expressions by @​martincostello in #​2715
• Fix flaky mutants by @​martincostello in #​2717
• fix documentation for ExecuteOutcomeAsync by @​carvex21 in #​2710
• Bump MartinCostello.WaitForNuGetPackage by @​martincostello in #​2721
• Bump sign tool by @​martincostello in #​2722
• xunit v3 preparation by @​martincostello in #​2723
• Add summary NuGet badges by @​martincostello in #​2724
• Delete actionlint config by @​martincostello in #​2730
• RateLimiting: add typed builder/guard tests and external limiter lifetime test by @​mohammed-saalim in #​2711
• Update Dependabot auto-merge conditions by @​martincostello in #​2736
• Bump NuGet packages by @​martincostello in #​2738
• Bump dotnet-stryker to 4.8.1 by @​martincostello in #​2739
• Simplify code by @​martincostello in #​2740
• Update zizmor by @​martincostello in #​2746
• Use NuGet Trusted Publishing by @​martincostello in #​2751
• Respect override for PipelineExecuting and PipelineExecuted by @​peter-csala in #​2752

---

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR has been generated by Renovate Bot.

[claude]

Claude finished @thomhurst's task —— View job

---

Code Review for Polly Dependency Update

Todo List:

• Read the changed file (Directory.Packages.props)
• Analyze the Polly usage in TUnit codebase
• Review Polly 8.6.4 changelog for security/breaking changes
• Assess compatibility and impact
• Provide comprehensive review feedback

---

📋 Review Summary

This is a safe and recommended dependency update from Polly 8.6.3 to 8.6.4.

🔍 Code Quality & Changes

File Modified: Directory.Packages.props:60

• Simple version bump: 8.6.3 → 8.6.4
• Follows centralized package management pattern correctly

🏗️ Architecture & Usage Analysis

Polly usage in TUnit is minimal and low-risk:

1. Primary Usage (TUnit.Pipeline/Modules/TestNugetPackageModule.cs:8):

   • Uses Polly.Retry.AsyncRetryPolicy<T> for test pipeline retry logic
   • Critical for build reliability during CI/CD operations

2. Test Usage (Multiple test files):

   • Uses Polly.CircuitBreaker.CircuitState enum as test data
   • No actual circuit breaker functionality, just enum values for testing

🛡️ Security Assessment

✅ No Security Concerns:

• Patch version update (8.6.3 → 8.6.4)
• No breaking changes or security vulnerabilities reported
• Changes are primarily bug fixes and code improvements

📈 Performance Considerations

✅ Performance Improvements Expected:

• v8.6.4 includes "Use collection expressions" optimizations
• "Fix flaky mutants" suggests improved reliability
• No performance regressions expected for a patch update

🔧 Compatibility & Impact

✅ Fully Compatible:

• Patch version ensures API compatibility
• TUnit's minimal usage (retry policies + enum values) unaffected
• No breaking changes in this release

🧪 Test Coverage

✅ Adequate Coverage:

• Existing tests will validate the retry behavior still works
• CircuitState enum usage in tests provides regression coverage
• No additional tests needed for this patch update

✅ Recommendations

1. Proceed with merge - This is a safe dependency update
2. Monitor CI/CD - Verify retry logic continues working as expected
3. Consider auto-merge - Future Polly patch updates can likely be auto-merged

🚀 Changelog Highlights (8.6.4)

Key improvements that benefit TUnit:

• Collection expressions optimization (performance)
• Documentation fixes for ExecuteOutcomeAsync
• Flaky test fixes (reliability)
• General code simplifications

---

Overall Assessment: ✅ APPROVED - Safe patch update with improvements and no risks.