Skip to content

Fix SQL quoted qualified name matching - #2601

Merged
Widthdom merged 2 commits into
mainfrom
fix-issue2101
May 24, 2026
Merged

Fix SQL quoted qualified name matching#2601
Widthdom merged 2 commits into
mainfrom
fix-issue2101

Conversation

@Widthdom

@Widthdom Widthdom commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Preserve case-sensitive matching for PostgreSQL double-quoted SQL qualified names while keeping MySQL backtick and T-SQL bracket matching case-insensitive.
  • Use the same dialect-aware resolver path for SQL unused-symbol exact matching.
  • Add regression coverage and a bilingual changelog fragment.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedNameReferenceExtractorTests.Extract_Sql|FullyQualifiedNameReferenceExtractorTests.Extract_SQL|FullyQualifiedNameReferenceExtractorTests.SqlNameResolver|FullyQualifiedNameSymbolExtractorTests.Extract_RustTraitAssociatedTypeDefaults_RecordsPropertySymbols"
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedNameDbReaderTests.SqlQualifiedNames|FullyQualifiedNameReferenceExtractorTests.SqlNameResolver_QuotedQualifiedNames"
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • codex adversarial review: No blocking/actionable issues found.

Documentation / Changelog

  • Added changelog fragment: changelog.d/unreleased/2101.fixed.md
  • No README/guide changes required; this is a bug fix to existing SQL matching behavior.

Follow-up candidates

  • None.

Fixes #2101

@Widthdom
Widthdom merged commit affe441 into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue2101 branch May 24, 2026 12:31
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.

SQL dialect-specific identifier quoting asymmetry breaks search

1 participant