Skip comments in regex brace matching

[tristanmanchester]

Summary

• teach the regex-based brace matchers to skip /* ... */ and // ... comment regions before counting braces
• apply the fix to the affected regex extractors: C#, C++, Dart (via the shared C# helper), and Rust
• add regressions proving comment braces no longer truncate extracted function bodies

Testing

• uv run --with pytest python -m pytest desloppify/languages/csharp/tests/test_csharp_parse_helpers.py desloppify/languages/cxx/tests/test_extractors.py desloppify/languages/rust/tests/test_extractors.py desloppify/languages/dart/tests/test_extractors.py

[peteromallet]

Thanks for the fix. I accepted the underlying issue, but implemented it with changes on this branch in cfaf256 rather than merging this PR directly.

Decision: IMPLEMENT_WITH_CHANGES. The fallback brace matchers now skip line and block comments for C#/C++/Dart, and Rust uses a nested block-comment depth counter so braces after an inner */ but before the outer */ are still ignored. I added regressions for the affected fallback extractors, including nested Rust comments.

Validation: PYENV_VERSION=3.11.11 python -m pytest desloppify/languages/csharp/tests/test_csharp_parse_helpers.py desloppify/languages/cxx/tests/test_extractors.py desloppify/languages/dart/tests/test_extractors.py desloppify/languages/rust/tests/test_extractors.py desloppify/languages/typescript/tests/test_ts_fixers.py -q

[peteromallet]

Closing as superseded by cfaf256 on the review branch.

[peteromallet]

Thanks for the contribution. This is included in v1.0: https://github.com/peteromallet/desloppify/releases/tag/v1.0