chore(deps): update dependency microsoft.visualstudio.threading.analyzers to 17.14.15 by thomhurst · Pull Request #6134 · thomhurst/TUnit (original) (raw)

Choose a reason for hiding this comment

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

Dependency Update Review: Microsoft.VisualStudio.Threading.Analyzers 17.13.61 → 17.14.15

This is a clean, minimal bump — single line in Directory.Packages.props, analyzer-only (no runtime impact).

What's in this update

Bug fixes worth noting:

• VSTHRD110 false negative in local functions of async methods is now correctly caught. Since TUnit explicitly prohibits .Result/.GetAwaiter().GetResult() (per CLAUDE.md), this tightening is beneficial — it may surface previously-missed violations in local async functions across the codebase.
• A race condition fix that could cause hangs.

Behavioral change to be aware of:

• The package switches from build to buildTransitive targets. This means analyzer rules will now flow transitively to projects that reference this package indirectly. In practice this is unlikely to cause issues since the analyzer is a dev-time tool, but worth knowing if any downstream consumers of TUnit packages start seeing new VSTHRD warnings.

Assessment

Safe to merge. The VSTHRD110 improvement aligns well with TUnit's strict no-blocking-async policy and may catch issues that were previously silently missed in local functions.