Ruben FiszelandClaude Opus 4.8 8ffbe773fb fix(debounce): atomic claim+persist, GC only non-queued rows; reword comment ...

Address review findings:

- [P1] Claim and accumulated-args persist are now in one transaction. Before,
  a crash between stamping batch rows consumed_by=self and the `UPDATE v2_job
  SET args` could let a zombie re-pull see its own prior claim and keep only its
  own args (dropping the siblings it had claimed). Wrapping claim + accumulate +
  persist in a tx makes them commit together or roll back together (re-pull then
  re-claims cleanly).

- [P1] GC of consumed batch rows now also requires the job to no longer be in
  v2_job_queue. A consumed sibling can stay queued well past any time grace under
  a concurrency limit / backlog; reclaiming its marker by age alone let its
  eventual pull treat it as never-batched and re-run its item (a duplicate).
  Keeping the row until the job leaves the queue preserves the "already consumed"
  signal. Test extended with a still-queued consumed row that must survive GC.

- [P2] Drop "Customer" attribution from a test doc comment (AGENTS.md).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

2026-06-25 15:56:18 +00:00

..

fixtures

feat: store hashed tokens instead of plaintext (#8217)

2026-03-17 01:15:38 +00:00

concurrency_counter_test.rs

fix(concurrency): two-phase admit to skip FOR UPDATE on over-limit pulls (#9064)

2026-05-07 09:43:33 +00:00

debounce_test.rs

fix(debounce): atomic claim+persist, GC only non-queued rows; reword comment

2026-06-25 15:56:18 +00:00

job_cleanup_test.rs

refactor: move integration tests to subcrates to reduce recompilation

2026-02-11 14:04:54 +00:00

schedule_push.rs

fix: handle singlestepflow zombies and stop filtering them from runs page (#9055)

2026-05-07 08:00:38 +00:00