Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 26, 2025

Adjusts MCP analyzer diagnostic severities and improves MCP001 reporting logic.

  • MCP001 (InvalidXmlDocumentation): Remains at DiagnosticSeverity.Warning, but now only reports when the method is declared as partial (since invalid XML only impacts generated code for partial methods)
  • MCP002 (McpMethodMustBePartial): Changed from DiagnosticSeverity.Warning to DiagnosticSeverity.Info

Updated corresponding test assertions to reflect these changes.

Original prompt

lower the analyzer diagnostics in diagnostics.cs from warning to info


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
@stephentoub stephentoub marked this pull request as ready for review November 26, 2025 11:19
Copilot AI changed the title [WIP] Lower analyzer diagnostics from warning to info Lower analyzer diagnostics from Warning to Info severity Nov 26, 2025
Copilot AI requested a review from stephentoub November 26, 2025 11:22
Copilot finished work on behalf of stephentoub November 26, 2025 11:22
Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
Copilot AI changed the title Lower analyzer diagnostics from Warning to Info severity Adjust analyzer diagnostic severities and improve MCP001 reporting Nov 26, 2025
Copilot AI requested a review from stephentoub November 26, 2025 14:10
Copilot finished work on behalf of stephentoub November 26, 2025 14:10
@stephentoub stephentoub merged commit b230b17 into main Nov 27, 2025
21 of 24 checks passed
@stephentoub stephentoub deleted the copilot/change-diagnostics-warning-to-info branch November 27, 2025 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants