Skip to content

Fix allowance-window reset timestamp jitter - #68

Merged
thrr87 merged 1 commit into
mainfrom
codex/fix-reset-time-jitter
Aug 5, 2026
Merged

Fix allowance-window reset timestamp jitter#68
thrr87 merged 1 commit into
mainfrom
codex/fix-reset-time-jitter

Conversation

@thrr87

@thrr87 thrr87 commented Aug 5, 2026

Copy link
Copy Markdown
Owner

What changed

  • align near-identical reset timestamps once at the analytics entrypoint
  • anchor the current window to the live account reset and historical windows to their latest observation
  • keep persisted observations unchanged
  • add a regression test for the fake second Usage Remaining line

Root cause

Synced Macs can receive reset timestamps that differ by seconds for the same weekly allowance window. Exact Date equality split those readings into separate chart and analytics windows.

Validation

  • focused regression failed before the fix and passes after it
  • full Swift suite: 565 tests passed
  • existing known-reset coverage confirms separate weekly windows remain separate

Closes #67

@thrr87
thrr87 marked this pull request as ready for review August 5, 2026 11:40
@thrr87
thrr87 merged commit 8ba0472 into main Aug 5, 2026
4 checks passed
@thrr87
thrr87 deleted the codex/fix-reset-time-jitter branch August 5, 2026 11:46
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.

Treat near-identical reset timestamps as one allowance window

1 participant