feat(quota): auto-expire stale cooldown data on read #87

Merged

lz merged 8 commits from feat/cooldown-reset into main

2026-07-22 13:02:54 +02:00

lz commented

2026-07-22 12:46:33 +02:00

Owner

Summary

The Nexus header badge and worker terminal statusline both used to keep showing a rate-limit window's last-known percentage and reset countdown forever, even after resets_at had already passed — claude-code only refreshes rate_limits on a real inference call, so an idle session never self-corrects.
getAccountQuota() (nexus/src/lib/server/quota/store.ts) now prunes an expired window as a pure read-time filter (not a mutation) instead of fabricating a 0% or leaving stale data in place.
worker/nexus-statusline.sh does the equivalent inline on every render, via a new expired() POSIX helper comparing resets_at against date +%s.
No frontend changes: an expired window serializes as null, identical to "never reported" — QuotaBadge.svelte already renders that as the dashed "unknown" state.
Write path (the POST to /api/agent/quota) is deliberately untouched — expiry is a display-layer concern on both sides, not a storage concern.
AGENTS.md fact #19 documents the behavior for future readers.

Test plan

pnpm test (nexus/) — 843 tests passing
sh worker/nexus-statusline.test.sh — 30 tests passing (7 original + 3 more closing gaps found in review)
Per-task spec-compliance + code-quality review during implementation (subagent-driven-development)
Whole-diff final review before this PR
Independent 4-agent review pass (general code quality, test coverage, comment accuracy, silent-failure hunting) — all gaps found (two test-coverage gaps, one doc-precision nit) closed before opening this PR

Design doc: docs/superpowers/specs/2026-07-16-quota-statusline-design.md ("Addendum (2026-07-21)"). Plan: docs/superpowers/plans/2026-07-21-cooldown-reset.md.

## Summary - The Nexus header badge and worker terminal statusline both used to keep showing a rate-limit window's last-known percentage and reset countdown forever, even after `resets_at` had already passed — claude-code only refreshes `rate_limits` on a real inference call, so an idle session never self-corrects. - `getAccountQuota()` (`nexus/src/lib/server/quota/store.ts`) now prunes an expired window as a pure read-time filter (not a mutation) instead of fabricating a 0% or leaving stale data in place. - `worker/nexus-statusline.sh` does the equivalent inline on every render, via a new `expired()` POSIX helper comparing `resets_at` against `date +%s`. - No frontend changes: an expired window serializes as `null`, identical to "never reported" — `QuotaBadge.svelte` already renders that as the dashed "unknown" state. - Write path (the POST to `/api/agent/quota`) is deliberately untouched — expiry is a display-layer concern on both sides, not a storage concern. - `AGENTS.md` fact #19 documents the behavior for future readers. ## Test plan - [x] `pnpm test` (nexus/) — 843 tests passing - [x] `sh worker/nexus-statusline.test.sh` — 30 tests passing (7 original + 3 more closing gaps found in review) - [x] Per-task spec-compliance + code-quality review during implementation (subagent-driven-development) - [x] Whole-diff final review before this PR - [x] Independent 4-agent review pass (general code quality, test coverage, comment accuracy, silent-failure hunting) — all gaps found (two test-coverage gaps, one doc-precision nit) closed before opening this PR Design doc: `docs/superpowers/specs/2026-07-16-quota-statusline-design.md` ("Addendum (2026-07-21)"). Plan: `docs/superpowers/plans/2026-07-21-cooldown-reset.md`.

lz added 8 commits

2026-07-22 12:46:33 +02:00

docs(quota): spec addendum for cooldown auto-expiry-on-read a669864687

docs(quota): implementation plan for cooldown auto-expiry-on-read 6d7ea81152

feat(quota): prune expired rate-limit windows on read d2dc84a814

getAccountQuota() now drops a window once its resets_at has passed,
instead of returning a stale percentage indefinitely until the next
session happens to report. Pure read-time filter, not a mutation.

test(quota): strengthen both-windows-expired test to actually set both windows ed86127d04

feat(quota): blank an expired rate-limit window in the statusline dd6fc75294

Mirrors the Nexus-side store fix: a resets_at that has already
passed falls back to the existing 'quota —' display instead of an
increasingly wrong percentage, checked inline on every render.

docs(agents): record cooldown auto-expiry-on-read behavior 82fe7d1d7d

docs(quota): tighten getAccountQuota's purity/single-reader comment 08738dc5cf

test(quota): close statusline fail-safe and wire-format test gaps found in PR review

ci / nexus (pull_request) Successful in 5m37s

Details

ci / images (pull_request) Successful in 16m13s

Details

pr-image-cleanup / delete-pr-images (pull_request) Successful in 8s

Details

3d40eee464

lz merged commit 119e930325 into main

2026-07-22 13:02:54 +02:00