Skip to content

fix: use native Windows process liveness checks - #49

Merged
Pimpmuckl merged 1 commit into
mainfrom
fix/native-process-liveness
Aug 9, 2026
Merged

fix: use native Windows process liveness checks#49
Pimpmuckl merged 1 commit into
mainfrom
fix/native-process-liveness

Conversation

@Pimpmuckl

Copy link
Copy Markdown
Owner

Summary

  • replace per-poll tasklist.exe process checks with kernel32 OpenProcess and WaitForSingleObject
  • preserve deliberate live, exited, missing, invalid, and access-denied behavior
  • close every acquired Windows handle and leave the non-Windows path unchanged

Validation

  • uv run pytest plugins/review-suite/tests/test_review_suite_local.py -q: 89 passed
  • uv run pytest plugins/review-suite/tests -q: 644 passed
  • uv run ruff check on the changed script and test file: passed
  • Review Suite fast rvw_8fd40a6a: two reviewers clean

@Pimpmuckl
Pimpmuckl force-pushed the fix/native-process-liveness branch from 233f30a to a7c3fb5 Compare August 9, 2026 02:09
@Pimpmuckl
Pimpmuckl merged commit 7e12e0e into main Aug 9, 2026
3 checks passed
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