Skip to content

fix: honor stop before retry admission - #118

Merged
dmoliveira merged 1 commit into
mainfrom
improve/v0.2.8-reliability
Jul 30, 2026
Merged

dmoliveira merged 1 commit into
mainfrom
improve/v0.2.8-reliability

Conversation

@dmoliveira

Copy link
Copy Markdown
Owner

Summary

  • serialize retry admission with durable stop mutations
  • keep stop intent dominant over late pause requests while allowing explicit resume
  • preserve the failed attempt and avoid creating retry evidence after stop

Validation

  • uv run ruff check .
  • uv run ruff format --check src/ailoop/service.py tests/test_service.py
  • Python 3.11 targeted retry/stop/resume tests: 13 passed
  • full suite: 421 passed, 85% coverage
  • final high-risk review: GO, no critical/high/medium findings

Concurrency contract

Retry admission linearizes under the state mutation lock. A stop committed first suppresses the retry; a stop committed after admission follows normal active-run cancellation.

Runtime-Session: ses_057afdc3effemiOJ2zfmjJiDqe

Runtime-Session: ses_057afdc3effemiOJ2zfmjJiDqe
@dmoliveira
dmoliveira merged commit 7a6a35d into main Jul 30, 2026
2 checks passed
@dmoliveira
dmoliveira deleted the improve/v0.2.8-reliability branch July 30, 2026 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant