Skip to content

Fix SQL MERGE column references - #2612

Merged
Widthdom merged 3 commits into
mainfrom
codex/fix-issue2100-clean
May 24, 2026
Merged

Fix SQL MERGE column references#2612
Widthdom merged 3 commits into
mainfrom
codex/fix-issue2100-clean

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Add SQL MERGE action parsing for UPDATE SET, INSERT (...), VALUES (...), and ON predicates.
  • Emit column_reference and join_condition_reference edges so MERGE-driven lineage is visible.
  • Add focused SQL reference extractor coverage and a bilingual changelog fragment.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release -p:UseSharedCompilation=false --filter "FullyQualifiedName~ReferenceExtractorTests"
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet build src/CodeIndex/CodeIndex.csproj -p:UseSharedCompilation=false
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --commits 8a89bae1cfab976c0fb25b1273205778551254f1 --json
  • codex exec adversarial review: No blocking/actionable issues found.

Full test note:

Documentation And Changelog

  • Added changelog.d/unreleased/2100.fixed.md.
  • No README/user guide changes were needed; this preserves the existing SQL reference extraction surface while filling missing MERGE edges.

Follow-Up Candidates

Fixes #2100

@Widthdom
Widthdom marked this pull request as ready for review May 24, 2026 10:49
@Widthdom
Widthdom merged commit 921aff5 into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue2100-clean branch May 24, 2026 12:33
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 MERGE statement target column lists missing reference edges

1 participant