Skip to content

[codex] Add query trace logging - #2637

Merged
Widthdom merged 2 commits into
mainfrom
codex/fix-issue1901
May 24, 2026
Merged

[codex] Add query trace logging#2637
Widthdom merged 2 commits into
mainfrom
codex/fix-issue1901

Conversation

@Widthdom

@Widthdom Widthdom commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add opt-in --trace=stderr|file|none handling for CLI query commands.
  • Emit one sanitized JSON trace line with elapsed time, exit code, error status, and result count when it can be derived from --count/NDJSON output.
  • Document the query trace option and add a bilingual changelog fragment.

Documentation and changelog

  • Updated README.md with the query trace contract.
  • Added changelog.d/unreleased/1901.added.md.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~ProgramRunnerTests|FullyQualifiedName~CliFlagSchemaTests" -p:UseSharedCompilation=false
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json

Review

  • In-thread adversarial review found no blocking/actionable issues.
  • codex exec adversarial review attempt was blocked by the local Codex usage limit before producing review output.

Follow-up candidates

  • None.

Fixes #1901

@Widthdom
Widthdom marked this pull request as ready for review May 24, 2026 14:03
@Widthdom
Widthdom merged commit 54ee483 into main May 24, 2026
11 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1901 branch May 25, 2026 22:16
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.

QueryCommandRunner: CLI query commands lack structured logging

1 participant