Skip to content

Fix current-window chart and Codex process race - #34

Merged
thrr87 merged 1 commit into
mainfrom
codex/fix-window-and-process-race
Jul 29, 2026
Merged

Fix current-window chart and Codex process race#34
thrr87 merged 1 commit into
mainfrom
codex/fix-window-and-process-race

Conversation

@thrr87

@thrr87 thrr87 commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • draw observed usage only from allowance windows inside the selected range, including the reset-boundary case
  • replace exception-raising FileHandle.availableData reads with an owned POSIX read descriptor
  • cover closed output, timeout reconnection, and prior-window filtering with regression tests

Verification

  • 93 affected tests pass: CodexClientTests, AnalyticsWorkspaceTests, and CodexAssistedInsightTests
  • Scripts/build-app.sh passes in Release
  • full-screen QA confirms Current window excludes prior-window data and three refreshes complete without a crash

@thrr87
thrr87 merged commit 27f0538 into main Jul 29, 2026
4 checks passed
@thrr87
thrr87 deleted the codex/fix-window-and-process-race branch July 29, 2026 15:16
thrr87 added a commit that referenced this pull request Jul 30, 2026
* perf: reuse timestamp parser during activity restore

* fix: bound usage chart and harden Codex reads (#34)

Co-authored-by: thrr87 <193831865+thrr87@users.noreply.github.com>

* perf: reuse timestamp parser during activity restore

* perf: bound refresh memory and local ingestion

* Load local analytics only when needed (#37)

* perf: load local analytics on demand

* Use App Server account data for the lightweight core (#42)

* perf: use account data for lightweight analytics core

* chore: prepare v0.2.4

---------

Co-authored-by: thrr87 <193831865+thrr87@users.noreply.github.com>

---------

Co-authored-by: thrr87 <193831865+thrr87@users.noreply.github.com>

---------

Co-authored-by: thrr87 <193831865+thrr87@users.noreply.github.com>
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