fix(codex): add invocation-scoped monitor lifecycle - #907
Open
GigiTiti-Kai wants to merge 12 commits into
Open
Conversation
- cleanup失敗時もTUI終了statusとlease解放を保持 - supervisorへのTERMでcaptured TUIとscope子processをreap - lifecycle testにfailure injectionとready barrierを追加 🤖 Generated with Codex
テストteardownでpidfile由来のTUI PIDをsignal対象にせず、fixture所有の gateとexit markerを使って失敗時にも安全にfake TUIを終了する。 🤖 Generated with Codex
No-key duplicate dispatchers now keep the legacy immediate CAS-exit behavior. Scoped dispatchers continue bounded standby and takeover while their exact server remains alive. 🤖 Generated with Codex
Canonicalize project paths physically before scoped keys and dispatcher routing, and clear inherited scoped server routing for legacy launches. Document the scope option and cover symlink aliases and nested no-scope sessions with regressions. 🤖 Generated with Codex
GigiTiti-Kai
marked this pull request as ready for review
August 20, 2026 22:47
Author
|
Ready for review. The final head is The second CI run exposed a macOS-only fixture mismatch ( Could you review and squash-merge this when ready? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
--invocation-scope TOKENmonitor mode with a fresh app-server record and CAS lease per physical project/scopeSIGKILLlimitationRelated to #149.
Safety boundaries
$!are signalledVerification
test_remote_engine_start_refusal.bats.config.lockrace, which also reproduced in isolation (8/9). Required Linux/macOS sharded CI remains the merge gate.Review