Skip to content

Port upstream 0.62.0: persist and record Codex quota reset observations - #743

Closed
Finesssee wants to merge 1 commit into
port/upstream-0.62.0from
port/micro-0.62.0-codex-reset-observations
Closed

Finesssee wants to merge 1 commit into
port/upstream-0.62.0from
port/micro-0.62.0-codex-reset-observations

Conversation

@Finesssee

Copy link
Copy Markdown
Collaborator

Summary

Ports the 0.62.0 gap: persist and record Codex quota-window reset observations (upstream steipete#3358), so Codex weekly boundaries are recovered from recorded observations across restarts instead of being estimated from the live window alone.

  • New rust/src/providers/codex/reset_observations.rs: durable, DPAPI-wrapped observation store modeled on the Claude reset_observations pattern (single codex scope; observations deduped by resets_at keeping the newest capture, sorted by reset time, bounded at 512 entries; staged atomic publish with failure cleanup).
  • chart.rs: the Codex chart history now loads the persisted observations and records the live window's reset on every refresh, then passes them to codex_quota_windows_from_cache as observed_next_resets (previously &[] — the gap). Read/merge failures degrade to the previously stored observations; the chart never breaks on a store error.

Validation

  • cargo +1.98.0 fmt --all -- --check: clean
  • cargo +1.98.0 clippy --workspace --all-targets -- -D warnings: clean
  • cargo +1.98.0 test -p codexbar --lib: 2173 passed, 0 failed (1 ignored), including 7 new reset-observations tests (empty/foreign scope rejection, missing store, dedupe/capture-regression, sort, bounded log, unsupported version, DPAPI-wrapped payload)
  • cargo +1.98.0 test -p codexbar-desktop-tauri: 461 passed / 1 failed (documented Isolate bootstrap payload test from real settings #684 bootstrap_payload_exposes_every_provider_variant baseline; backend-only diff, no frontend files)

Upstream reference

steipete/CodexBar 0.62.0, quota-window reset observations (steipete#3358) — tag-pinned read-only.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Finesssee

Copy link
Copy Markdown
Collaborator Author

Shipped in v0.70.0: this PR's head is included in main via #735 (merge commit 9d0a37a). Closing as integrated.

@Finesssee Finesssee closed this Oct 3, 2026
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.

1 participant