Skip to content

[codex] Fix inspect flags and help coverage - #3844

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue3786-3666-3665-3663
Jun 20, 2026
Merged

[codex] Fix inspect flags and help coverage#3844
Widthdom merged 4 commits into
mainfrom
fix-issue3786-3666-3665-3663

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Add inspect source excerpt support for --line, range, context, and path/line-only usage.
  • Let inspect --kind filter definition candidates and prefer production source definitions over matching test definitions.
  • Add --body-line-count as an alias for inspect --body-lines.
  • List every implemented command in the top-level cdidx --help command summary.

Issues

Fixes #3786
Fixes #3666
Fixes #3665
Fixes #3663

Validation

  • dotnet run --project tools/CodeIndex.Changelog -- check
  • git diff --check origin/main..HEAD
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~Issue
  • dotnet build CodeIndex.sln
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll inspect RunInspect --line 5363 --context 1 --json --fields source_excerpt --exact-name
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll inspect --path src/CodeIndex/Cli/QueryCommandRunner.cs --line 5363 --json --fields source_excerpt

Review

Adversarial review of origin/main..HEAD: no blocking/actionable issues found.

Notes

@Widthdom
Widthdom marked this pull request as ready for review June 20, 2026 11:35
@Widthdom
Widthdom merged commit af4f3c5 into main Jun 20, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3786-3666-3665-3663 branch June 20, 2026 12:05
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