Skip to content

Fix C# phantom symbol extraction suffix-guard regressions - #787

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue496-782-785
Apr 23, 2026
Merged

Fix C# phantom symbol extraction suffix-guard regressions#787
Widthdom merged 4 commits into
mainfrom
fix-issue496-782-785

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • extend the C# type-body gate so field-like function rows for const / static readonly do not leak method-local symbols
  • keep the phantom qualified call-site filter for cases like elapsed < TimeSpan.FromSeconds(...) while preserving legal return types such as @new, await, and yield
  • add focused regressions plus changelog / developer guide updates in both English and Japanese

Test plan

  • dotnet test --filter "Extract_CSharp_ConstLocalsAndQualifiedCallArguments_DoNotLeakPhantomFunctions|Extract_CSharp_VerbatimReturnTypeIdentifiers_AreNotRejectedBySuffixGuard|Extract_CSharp_ContextualKeywordReturnTypeIdentifiers_AreNotRejectedBySuffixGuard"
  • dotnet test --filter FullyQualifiedName~SymbolExtractorTests
  • dotnet test
  • dotnet build
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --commits HEAD --json
  • local cdidx.dll smoke repros for TimeSpan.FromSeconds(...), public @new Make(), and public await MakeAwait() / public yield MakeYield()

Closes #496.
Closes #782.
Closes #785.

@Widthdom
Widthdom merged commit 5ba3e1f into main Apr 23, 2026
@Widthdom
Widthdom deleted the fix-issue496-782-785 branch April 23, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment