The cross-runner cancellation guard (molecule_ci_guard) was killing healthy molecule slots when unhealthy slots failed on high-disk runners, defeating fail-fast: false. With fail-fast: false and no guard, slots on healthy runners complete independently.
Changes
Deleted src/devx/molecule/molecule_ci_guard.py and its test file
Moved get_running_jobs, any_other_runner_failed, and poll_for_other_failures inline into integration_guard.py
Removed molecule guard CLI command registration
Removed guard from doc_coverage.py REQUIRED_SCRIPTS
Removed 5 dead translation keys, added 1 missing key
Updated all tests, docs, and README
Test Results
1977 passed, 100% coverage. All pre-commit hooks passed.
## Summary
The cross-runner cancellation guard (`molecule_ci_guard`) was killing healthy molecule slots when unhealthy slots failed on high-disk runners, defeating `fail-fast: false`. With `fail-fast: false` and no guard, slots on healthy runners complete independently.
## Changes
- Deleted `src/devx/molecule/molecule_ci_guard.py` and its test file
- Moved `get_running_jobs`, `any_other_runner_failed`, and `poll_for_other_failures` inline into `integration_guard.py`
- Removed `molecule guard` CLI command registration
- Removed guard from `doc_coverage.py` REQUIRED_SCRIPTS
- Removed 5 dead translation keys, added 1 missing key
- Updated all tests, docs, and README
## Test Results
1977 passed, 100% coverage. All pre-commit hooks passed.
Generated with [Devin](https://devin.ai)
The cross-runner cancellation guard was killing healthy molecule slots
when unhealthy slots failed, defeating fail-fast: false. With fail-fast:
false and no guard, slots on healthy runners complete independently.
Generated with [Devin](https://devin.ai)
Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Remove 5 dead translation keys left over from molecule_ci_guard
removal. Add the missing "Another runner failed" key that
integration_guard.py references.
Generated with [Devin](https://devin.ai)
Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: OK
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emo
approved these changes 2026-08-09 00:59:50 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
emo
Summary
The cross-runner cancellation guard (
molecule_ci_guard) was killing healthy molecule slots when unhealthy slots failed on high-disk runners, defeatingfail-fast: false. Withfail-fast: falseand no guard, slots on healthy runners complete independently.Changes
src/devx/molecule/molecule_ci_guard.pyand its test fileget_running_jobs,any_other_runner_failed, andpoll_for_other_failuresinline intointegration_guard.pymolecule guardCLI command registrationdoc_coverage.pyREQUIRED_SCRIPTSTest Results
1977 passed, 100% coverage. All pre-commit hooks passed.
Generated with Devin
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
Auto-approved: all CI checks passed (validate job).