fix: write-path review findings (held lease cap, live reserve check, validation) #90

Open
mariusz.pacek wants to merge 0 commits from mariusz.pacek/fix/write-path-review AGit into master
pull from: mariusz.pacek/fix/write-path-review
Owner

Caps a held lease after a failed later boost, decides a raised reserve on the live table, validates reserve.json, treats incomplete settings readings as stale, shares one token throttle.

Caps a held lease after a failed later boost, decides a raised reserve on the live table, validates reserve.json, treats incomplete settings readings as stale, shares one token throttle.
mariusz.pacek added 4 commits 2026-10-24 15:45:24 +00:00
A boost that failed after its write could leave a raised value under the
held change's far lease. The restore now keeps both values but runs
within one lease (ADR-012).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The cached reading can be 30 minutes old. A reading missing fields now
counts as no usable settings instead of failing with a 500.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
fix(http): share one failed-token throttle between endpoints
CI / Check (gateway, backend, frontend) (push) Successful in 1m23s
f57c3adc58
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
All checks were successful
CI / Check (gateway, backend, frontend) (push) Successful in 1m23s
This branch is already included in the target branch. There is nothing to merge.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin refs/pull/90/head:mariusz.pacek/fix/write-path-review
git checkout mariusz.pacek/fix/write-path-review
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mariusz.pacek/home-pump#90