Repository navigation
Disable TaskAnalyzer code fixes for 18.13 - #15402
Merged
JanProvaznik merged 6 commits intoOct 27, 2026
Merged
Conversation
Remove MEF exports from both code fix providers while preserving implementations and analyzer behavior. Add a discovery contract regression and document temporary code fix unavailability. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Restore the original detailed code fix guidance and retain only brief notes about temporary 18.13 code fix unavailability. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Keep customer documentation focused on available diagnostics and suppressions; remove code fix availability notes, implementation details, examples, links and development references. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
AR-May
marked this pull request as ready for review
October 26, 2026 14:52
AR-May
temporarily deployed
to
copilot-pat-pool
October 26, 2026 14:52 — with
GitHub Actions
Inactive
AR-May
temporarily deployed
to
copilot-pat-pool
October 26, 2026 14:53 — with
GitHub Actions
Inactive
AR-May
had a problem deploying
to
copilot-pat-pool
October 26, 2026 14:53 — with
GitHub Actions
Failure
JanProvaznik
approved these changes
Oct 26, 2026
JanProvaznik
left a comment
Member
There was a problem hiding this comment.
discussed that fixers are broken, hard to get right, and low value even if fixed, feeding errors to agents is more flexible
AR-May
temporarily deployed
to
copilot-pat-pool
October 26, 2026 14:55 — with
GitHub Actions
Inactive
AR-May
temporarily deployed
to
copilot-pat-pool
October 26, 2026 14:56 — with
GitHub Actions
Inactive
AR-May
had a problem deploying
to
copilot-pat-pool
October 26, 2026 14:56 — with
GitHub Actions
Failure
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The narrowly scoped export removals disable unsafe fixes without changing diagnostic behavior, and the discovery regression covers the intended contract.
No open issues 🎉
What changed in this PR
Disables automatic TaskAnalyzer code fixes for the 18.13 release to avoid unsafe transformations, while preserving diagnostics and the existing fixer implementations and tests.
Changes:
- Removes discovery exports from both code-fix providers.
- Adds a regression test verifying that no code fixes are exported and all expected analyzers remain discoverable.
- Removes code-fix availability claims and documentation.
| File | Description |
|---|---|
src/TaskAnalyzer/README.md |
Removes automatic-fix documentation while retaining diagnostic guidance. |
src/TaskAnalyzer/PreferTypedParameterCodeFixProvider.cs |
Removes typed-parameter fixer exports. |
src/TaskAnalyzer/MultiThreadableTaskCodeFixProvider.cs |
Removes multithreaded-task fixer exports. |
src/TaskAnalyzer/AnalyzerReleases.Shipped.md |
Removes code-fix availability claims. |
src/TaskAnalyzer.Tests/PackageContractTests.cs |
Adds regression coverage for fixer and analyzer discovery. |
Review effort: Balanced
Co-authored-by: Jan Provazník <janpro@janpro.dev>
JanProvaznik
approved these changes
Oct 26, 2026
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
JanProvaznik
enabled auto-merge (squash)
October 27, 2026 17:16
This was referenced Oct 28, 2026
41 of 69 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related issue(s): #14971
Context
The current fixes can produce uncompilable code, change path/parsing behavior, or break callers when retyping public properties. Disable them for the first release rather than ship unsafe transformations.
Changes Made
Remove both providers’ MEF exports and code-fix documentation. Keep implementations/tests and add a discovery regression.
Testing
Unit tests