Select latest Codex session by record time - #393
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
ceiling | dcbd8ce | Commit Preview URL Branch Preview URL |
Aug 23 2026, 06:43 PM |
|
Warning Review limit reached
Next review available in: 25 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_4d23f453-006e-4919-96fc-bf3862062556) |
# Conflicts: # CHANGELOG.md
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_38cc736b-dd35-4207-a488-ffc0bb88bd36) |
# Conflicts: # CHANGELOG.md
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_cc711901-c7b5-44bc-b4b3-bce405ebf8ba) |
Fixes #271.
Codex latest-session summaries now rank each rollout by its newest included record timestamp. File modification time remains the fallback, and equal timestamps use a stable path tie-break.
Checks:
cargo fmt --all -- --checkcargo test --manifest-path rust/Cargo.toml codex_latest_sessionNote
Low Risk
Narrow cost-scanner ranking change with tests; it only affects which session is labeled latest, not totals or auth.
Overview
Codex local cost summaries now treat the latest session as the rollout with the newest transcript timestamp, not the file with the newest mtime. Copying or touching an older rollout no longer displaces a more recent session.
CodexReportRollupsranks by the newest in-rangerecord.timestamp, falls back to file mtime when a transcript has no timestamps, and breaks equal times with a stable path comparison. Tests cover both the mtime-vs-record case and the path tie-break.Reviewed by Cursor Bugbot for commit dcbd8ce. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Select latest Codex session by transcript record time, not file mtime
Previously, touching or copying an older rollout file could overwrite a newer session in local cost summaries because selection used the file's modified time. The
latestfield ofCodexReportRollupsnow storesDateTime<Utc>alongside the sourcePathBufandCostSummary, andingest_parsedpicks the most recentrecord.timestampfrom the transcript records instead of the filesystem mtime (which becomes a fallback). When two rollouts share the same record timestamp, the lexicographically larger path wins for deterministic ordering.CodexReportRollups.latestmust handle the new tuple shape(DateTime<Utc>, PathBuf, CostSummary); the in-tree consumer infinishis already updated.Macroscope summarized dcbd8ce.