Skip to content

Fix hotspot counts and grouped path caps - #3335

Merged
Widthdom merged 6 commits into
mainfrom
codex/fix-issue3153-3159-2991
Jun 6, 2026
Merged

Fix hotspot counts and grouped path caps#3335
Widthdom merged 6 commits into
mainfrom
codex/fix-issue3153-3159-2991

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keep C# property hotspots on the conservative in-target counting path so common property names are not inflated by unrelated bare-name references.
  • Make hotspots --count use total-count queries for symbol, file, and grouped-name modes so --limit no longer truncates count metadata.
  • Cap hotspots --group-by-name --json path samples per group and expose paths_truncated.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~ConsoleUiTests -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~RunHotspots -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~GetSymbolHotspots -p:UseSharedCompilation=false
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • git diff --check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review: No blocking/actionable issues found.

Fixes #2991
Fixes #3153
Fixes #3159

@Widthdom
Widthdom merged commit d2baa3b into main Jun 6, 2026
10 of 16 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue3153-3159-2991 branch June 6, 2026 15: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

1 participant