You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Replace the brace scanner in src/csharp.ts with a C# parser (such as tree-sitter-c-sharp) if conditional-compilation branches with unbalanced braces make the api-compat floor fail closed too often; it needs a new dependency, which #112's plan avoided.
all 9 come from #if branches that each open a brace
Conditional compilation never makes the api-compat floor fail closed on NCodeGroup code, so a parser dependency isn't justified, and the scanner stays.
The measurement did find one real bug: a verbatim string whose text starts with an escaped quote (@"""", @"""Hi"" there") was read as a raw string literal, masked to the end of the file, and failed closed. #140 fixes it with a regression test.
Activity
polewskm commentedon Nov 1, 2026
Measurement for the "if it fails closed too often" condition:
declarationSpansfromsrc/csharp.tswas run over every*.csfile outsideobj/andbin/.#if#ifbranches that each open a braceConditional compilation never makes the api-compat floor fail closed on NCodeGroup code, so a parser dependency isn't justified, and the scanner stays.
The measurement did find one real bug: a verbatim string whose text starts with an escaped quote (
@"""",@"""Hi"" there") was read as a raw string literal, masked to the end of the file, and failed closed. #140 fixes it with a regression test.