Skip to content

Fix Go generic symbol extraction - #2656

Merged
Widthdom merged 5 commits into
mainfrom
codex/fix-issue2047-2048-2049-2050
May 24, 2026
Merged

Fix Go generic symbol extraction#2656
Widthdom merged 5 commits into
mainfrom
codex/fix-issue2047-2048-2049-2050

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Index Go embedded generic struct types, while avoiding ordinary named fields.
  • Skip Go blank identifiers in var/const declarations.
  • Preserve type-parameterized Go interface method signatures.
  • Surface Go build directives as annotations and classify import "C" as cgo.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~SymbolExtractorTests.Extract_Go" -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter SymbolExtractorTests
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Adversarial review: No blocking/actionable issues found.

Documentation and Changelog

  • Added bilingual changelog fragments:
    • changelog.d/unreleased/2047.fixed.md
    • changelog.d/unreleased/2048.fixed.md
    • changelog.d/unreleased/2049.fixed.md
    • changelog.d/unreleased/2050.fixed.md
  • No AGENT.md, CLAUDE.md, or AGENT_GUIDE.md changes.

Fixes #2047
Fixes #2048
Fixes #2049
Fixes #2050

@Widthdom
Widthdom marked this pull request as ready for review May 24, 2026 16:27
@Widthdom
Widthdom merged commit 672b257 into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue2047-2048-2049-2050 branch May 24, 2026 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment