[Experimental] Add MCP server for analysis and bulk-fixing#17
Draft
yaakov-h wants to merge 3 commits into
Draft
Conversation
|
|
||
| <PropertyGroup> | ||
| <TargetFramework>net472</TargetFramework> | ||
| <TargetFrameworks>net472;net10.0</TargetFrameworks> |
Contributor
There was a problem hiding this comment.
Do we still need net472? I would be tempted to drop it and just target net10.0.
Last I checked, the .NET Core version of Roslyn had a bug when adding new assembly references, but is that something we need to worry about here? Are we likely to run a code-fix that adds an assembly reference?
Member
Author
There was a problem hiding this comment.
I am also quite tempted to drop it, but we should cut over to modern .NET as a (series of) separate PR(s).
Have you ever come across a code-fix that adds assembly references?
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.
Warning
This PR is raw AI-generated code and is probably not yet fit for human consumption.
This PR adds a new executable, WTG.BulkAnalysisRunner.Mcp, which exposes a MCP server for LLM coding agents to find Roslyn Diagnostics in a workspace and apply the fixes.
The MCP server holds analysis sessions in-memory to avoid recomputing the whole project/solution on each request.
This is the result of a few back-and-forth sessions with Claude and includes a bunch of changes that should be their own outright prereqs, including upgrading dependencies, restructuring outputs, and changing target frameworks.
It's also quite possible that I mucked up line endings in this PR.