Skip to content

[codex] Improve Codex log coverage#51

Merged
aymenfurter merged 1 commit into
microsoft:mainfrom
rusliksu:ruslan/codex-log-coverage
May 24, 2026
Merged

[codex] Improve Codex log coverage#51
aymenfurter merged 1 commit into
microsoft:mainfrom
rusliksu:ruslan/codex-log-coverage

Conversation

@rusliksu
Copy link
Copy Markdown
Contributor

Summary

  • Stream Codex JSONL parsing so sessions larger than the 50 MB safe-read limit are included.
  • Discover .codex/archived_sessions and .codex/archived-sessions alongside active Codex sessions.
  • Preserve Codex meta.cwd as the workspace root while keeping the displayed session location unchanged.

Why

Real Codex installations can have large JSONL session files and archived session directories. The previous parser undercounted sessions and workspaces, which made config health and agentic readiness scores misleading.

Validation

  • npm test passed: 995 tests.
  • npm run check passed.
  • npm run package passed.
  • Windows preflight over local logs: 506 sessions, 8694 requests, including 11 Codex JSONL files over 50 MB.

@aymenfurter aymenfurter self-requested a review May 24, 2026 13:01
@aymenfurter
Copy link
Copy Markdown
Contributor

Thanks for the contribution! Happy to merge once you've agreed to the CLA.

@aymenfurter aymenfurter added the enhancement New feature or request label May 24, 2026
@rusliksu
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@rusliksu rusliksu marked this pull request as ready for review May 24, 2026 14:41
@aymenfurter aymenfurter merged commit 8562a6b into microsoft:main May 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants