Skip to content

fix(processor): handle abort signal during cleanup (#11)#41

Merged
randomm merged 1 commit intodevfrom
fix/abort-cleanup
Jan 26, 2026
Merged

fix(processor): handle abort signal during cleanup (#11)#41
randomm merged 1 commit intodevfrom
fix/abort-cleanup

Conversation

@randomm
Copy link
Owner

@randomm randomm commented Jan 26, 2026

Fixes #11

Changes

  • Add abort check after retry sleep to prevent infinite loop
  • Check abort before and after snapshot patch operation
  • Log warning when cleanup interrupted mid-tool-loop
  • Move completed time assignment inside abort check with try/catch
  • Add logging for aborted sleep

Testing

  • Typecheck passes
  • Addresses all 5 critical issues from adversarial review

- Add abort check after retry sleep to prevent infinite loop
- Check abort before and after snapshot patch
- Log warning when cleanup interrupted mid-loop
- Move completed time inside abort check
- Add logging for aborted sleep
@github-actions
Copy link

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@randomm randomm merged commit 9209301 into dev Jan 26, 2026
1 of 3 checks passed
@randomm randomm deleted the fix/abort-cleanup branch January 26, 2026 13:42
randomm added a commit that referenced this pull request Feb 3, 2026
- Add abort check after retry sleep to prevent infinite loop
- Check abort before and after snapshot patch
- Log warning when cleanup interrupted mid-loop
- Move completed time inside abort check
- Add logging for aborted sleep
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant