fix(clio-probes): rethrow rails in the stale-etag catch; assert billed ownership

↗ view on GitHub · sloth-ninja · 2026-08-14 · b22a0df2

Two one-line review fixes in the write phase, where the stakes are highest.

- The stale-If-Match catch was the one remaining rail-swallowing site: a blocked
  DB access or a rate-limit abort would have been filed as "Clio did not enforce
  concurrency" AND left the run issuing further writes. It now rethrows first,
  like every other inner catch.
- The billed candidate's ownership no longer rests solely on Clio honouring
  `user_id=` server-side; the row's own `user.id` must match the caller's stored
  Clio user id. If that param were ever ignored or loosened, the "no-op" patch
  would otherwise have landed on a colleague's billed time.

Verified: tsc both projects clean; prettier clean; --dry-run clean.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Repository Sloth-ninja/JessicaOSS
Author sloth-ninja <ezanahaddis@gmail.com>
Authored
Committed
Parents 711ea9b3
Stats 1 file changed , +10 , -1
Part of Guard-railed Clio live-probe runner

Capture this commit into my fork

Download a Markdown prompt that tells Claude how to port this exact commit into your working tree. Run it via claude -p < capture-commit-b22a0df2.md from inside the repo you want the change in.

⬇ Download capture-commit-b22a0df2.md