Skip to content

sonar(tools/corpus/seeder/SyntaxMutator.cs): 4 findings - #287

Open
Zafnok wants to merge 1 commit into
mainfrom
sonar/syntax-mutator
Open

Zafnok wants to merge 1 commit into
mainfrom
sonar/syntax-mutator

Conversation

@Zafnok

@Zafnok Zafnok commented Sep 29, 2026

Copy link
Copy Markdown
Owner

Closes #273

All four findings reproduced on HEAD; none had evaporated.

  • Every finding fixed or accepted. S125 (x2): the two section comments now describe the transform in prose instead of pseudo-code. IDE0300: new StatementSyntax[] { declare, rewritten } is now [declare, rewritten]. S3267: AdjacentPairCandidates iterates .Select(block => block.Statements). Nothing accepted in policy.jsonc.
  • IDE0300 pinned. dotnet_diagnostic.IDE0300.severity = error added to .editorconfig. It was the only remaining IDE0300 site in the seeder.
  • Gate. Left to CI (per equiv-quality-gates); no local run.
  • No pragma, coverage exclusion or lowered gate. No behaviour change.

🤖 Generated with Claude Code

Rewords two comments that read as code (S125), uses a collection expression
(IDE0300) and iterates block.Statements directly (S3267). Pins
dotnet_diagnostic.IDE0300.severity = error.

Ticket: GH-273

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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.

sonar(tools/corpus/seeder/SyntaxMutator.cs): 4 findings

1 participant