Skip to content

fix: preserve Codex rows on refresh errors#25

Merged
aqua5230 merged 1 commit into
aqua5230:mainfrom
ericweichun:main
Jun 2, 2026
Merged

fix: preserve Codex rows on refresh errors#25
aqua5230 merged 1 commit into
aqua5230:mainfrom
ericweichun:main

Conversation

@ericweichun
Copy link
Copy Markdown
Contributor

Summary

  • keep the early Codex quota rows when the later full refresh path fails
  • preserve Codex session / weekly rows and stale metadata in the error state instead of clearing the Codex section to missing
  • add a regression test for a failed full refresh after the early Codex quota update

Context

#24 preserved the menu-bar Codex percentage during refresh failures, but the popover state still came from _error_state(), which reset the Codex rows to missing. This follow-up keeps the Codex rows that were already loaded at the start of the refresh.

Tests

  • .venv/bin/ruff check .
  • .venv/bin/mypy .
  • .venv/bin/python -m pytest tests/ -q

@aqua5230 aqua5230 merged commit cb86f96 into aqua5230:main Jun 2, 2026
1 check passed
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.

2 participants