Skip to content

Fix issue #363: add exact regression coverage for C# phantom string symbols - #504

Merged
Widthdom merged 1 commit into
mainfrom
fix-issue363
Apr 21, 2026
Merged

Fix issue #363: add exact regression coverage for C# phantom string symbols#504
Widthdom merged 1 commit into
mainfrom
fix-issue363

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

Fixes #363.

This PR adds exact regression coverage for the user-reported C# phantom-symbol fixture involving raw strings, interpolated raw strings, and multi-line verbatim strings.

What changed

Why

The underlying masking behavior is already present on current main, but issue #363's exact dogfood fixture was not pinned directly in the test suite. This PR prevents silent regression of that specific report.

Validation

  • dotnet test
  • Local repro with dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index ... and symbols --db ... --json
  • Adversarial Codex review on origin/main...HEAD with no blocking/actionable findings

@Widthdom
Widthdom merged commit fbc8d12 into main Apr 21, 2026
6 checks passed
@Widthdom
Widthdom deleted the fix-issue363 branch April 21, 2026 03:32
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# raw string phantom symbols persist: """...""", $"""...""", and @"..." content still captured as real symbols (#177 regression)

1 participant