Skip to content

[5.0] Fix RCS1263 and RCS1139 for C# 14 extension block XmlDoc - #1799

Merged
josefpihrt merged 4 commits into
mainfrom
fix/analyzers/extension-block-xml-doc
Aug 16, 2026
Merged

[5.0] Fix RCS1263 and RCS1139 for C# 14 extension block XmlDoc#1799
josefpihrt merged 4 commits into
mainfrom
fix/analyzers/extension-block-xml-doc

Conversation

@josefpihrt

Copy link
Copy Markdown
Collaborator

Summary

  • Resolve extension block <param> and <typeparam> references in RCS1263 by handling SyntaxKind.ExtensionBlockDeclaration in CSharpUtility.GetParameterList / GetTypeParameterList
  • Skip RCS1139 when documentation is attached to an extension block (summary is not valid there)
  • Add analyzer tests for extension block documentation

Stacked on #1787 (feature/roslyn-5).

Fixes #1752

Test plan

  • dotnet test Tests/Analyzers.Tests/Analyzers.Tests.csproj --filter "ExtensionBlock"
  • dotnet build with RoslynVersion=roslyn5.0, roslyn4.7, and roslyn3.8 NuGet slices

Made with Cursor

@josefpihrt josefpihrt left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Review notes for the C# 14 extension-block XmlDoc fixes. Looks correct for #1752; one small follow-up on coverage.

@josefpihrt josefpihrt changed the title Fix RCS1263 and RCS1139 for C# 14 extension block XmlDoc [5.0] Fix RCS1263 and RCS1139 for C# 14 extension block XmlDoc Aug 15, 2026
Base automatically changed from feature/roslyn-5 to main August 16, 2026 13:49
josefpihrt and others added 3 commits August 16, 2026 15:53
Fixes #1752

Handle ExtensionBlockDeclaration in CSharpUtility parameter and
type-parameter resolution, and skip RCS1139 summary requirement on
extension block documentation comments.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@josefpihrt
josefpihrt force-pushed the fix/analyzers/extension-block-xml-doc branch from c16035b to 60453e7 Compare August 16, 2026 13:54
Co-authored-by: Cursor <cursoragent@cursor.com>
@josefpihrt
josefpihrt merged commit 2c35335 into main Aug 16, 2026
17 checks passed
@josefpihrt
josefpihrt deleted the fix/analyzers/extension-block-xml-doc branch August 16, 2026 14:13
This was referenced Aug 21, 2026
This was referenced Aug 24, 2026
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.

c# 14 extension members not understood properly for XmlDoc

1 participant