[matt-edmondson]

Fixes #151

What changed

• WriteAutoGeneratedHeader used to write // {copyright} once, so only the first line of a multi-line notice was commented. It now splits the copyright into lines and writes // {line} for each one, or a bare // for an empty line. The split happens at every terminator the C# compiler recognises: LF, CRLF, CR, NEL (U+0085), LINE SEPARATOR (U+2028) and PARAGRAPH SEPARATOR (U+2029). A single-line copyright gives the same output as before.
• RegionScope now throws ArgumentException when the name contains any of those terminators. It throws before anything is written, the same validate-first approach as Unknown TypeKind or AccessorKind writes half a declaration before throwing, and an Expression accessor with no BodyFactory emits get => ; #142, so the rest of the name can no longer end up outside the #region directive.
• New LineTerminators helper (internal). DocComment's private SplitLines has moved here, so all three callers use the same list of terminators. DocComment's behaviour is unchanged.
• The copyright parameter docs now describe how a multi-line notice is written.

The null check on copyright is now copyright is { Length: > 0 } instead of !string.IsNullOrEmpty(...). On netstandard2.0, IsNullOrEmpty has no nullable annotations, so the old form raised CS8604 when the value was passed to the splitter.

Tests

• AutoGeneratedHeaderCommentsEveryLineOfAMultiLineCopyright: one data row for each of the six terminators.
• AutoGeneratedHeaderWritesABlankCopyrightLineAsABareComment: an SPDX line, a blank line, then a holder line.
• RegionScopeRejectsANameThatSpansLinesBeforeWriting: one row for each terminator. Each row checks that the exception is thrown and that nothing was written.

With the CodeBlockerExtensions.cs and Scopes.cs changes reverted, all 13 new cases fail. With them in place, the full suite passes (236/236, net10.0, Linux). The local Sonar run (-p:CustomAfterMicrosoftCommonProps=.sonarlint/sonar-local.props) reports no new findings in the library.

Notes

• None of the open CodeBlocker PRs (Add a SessionStart hook for Claude Code cloud sessions #145, Leave the pending indentation alone on an empty Write [patch] #147–Reject an unknown type or accessor kind, or a bodiless expression accessor, before writing [patch] #150) touches these methods.
• A copyright containing U+2028, U+2029 or U+0085 makes WriteFileHeader emit code that fails to compile (CS1003/CS1002) SourceGeneratorToolkit#39 is the same bug class in WriteFileHeader. That fix lives in its own repository and is not part of this PR.

---

Generated by Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud