Keep pooled live snapshots cancellation-safe #138

Merged
rockachopa merged 1 commits from timmy/137-cancellation-safe-live-snapshot into main 2026-08-06 20:52:39 +00:00
Member

Summary

  • shield the pooled live snapshot from individual caller cancellation
  • give the shared snapshot task one bounded deadline for every attached waiter
  • cancel and await active snapshot work before closing the Gitea transport

TDD proof

  • RED: disconnect test failed because _live_snapshot_task became None
  • RED: shared deadline test failed because upstream work remained active
  • RED: shutdown test observed stop (active) before snapshot cancellation
  • GREEN: pytest -q118 passed in 2.38s
  • python3 -m compileall -q src tests
  • git diff --check

Closes #137

## Summary - shield the pooled live snapshot from individual caller cancellation - give the shared snapshot task one bounded deadline for every attached waiter - cancel and await active snapshot work before closing the Gitea transport ## TDD proof - RED: disconnect test failed because `_live_snapshot_task` became `None` - RED: shared deadline test failed because upstream work remained active - RED: shutdown test observed `stop (active)` before snapshot cancellation - GREEN: `pytest -q` → `118 passed in 2.38s` - `python3 -m compileall -q src tests` - `git diff --check` Closes #137
rockachopa added 1 commit 2026-08-06 20:52:04 +00:00
fix: keep pooled live snapshots cancellation-safe (#137)
All checks were successful
CI / lint (pull_request) Successful in 10s
CI / build-frontend (pull_request) Successful in 5s
0dc5c2c831
rockachopa merged commit 3807e7afd1 into main 2026-08-06 20:52:39 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: stackchain/stackchain-dashboard#138
No description provided.