Skip to content

refactor(orchestrator): guard detached launch ownership - #484

Open
LukeMathWalker wants to merge 2 commits into
mainfrom
refactor/orchestrator-detached-launch-guard
Open

refactor(orchestrator): guard detached launch ownership#484
LukeMathWalker wants to merge 2 commits into
mainfrom
refactor/orchestrator-detached-launch-guard

Conversation

@LukeMathWalker

Copy link
Copy Markdown
Contributor

Why

Detached startup repeated supervisor termination and generation rollback across several failure branches. Collector startup also consumed the supervisor child on failure, preventing the launcher from retaining a continuous collection capability.

What Changed

  • add an armed detached-launch guard that owns the supervisor through handoff
  • centralize explicit generation rollback for all handoff failures
  • return child ownership when background collector creation fails
  • recover poisoned collector state instead of dropping child capabilities
  • preserve collection through synchronous fallback paths

Risks / Notes

The two-phase filesystem handoff remains procedural and unchanged. The guard only makes ownership and rollback obligations structural. This is the second PR in the lifecycle-ownership stack and targets #483.

AI Assistance

Implementation assisted by OpenAI GPT-5.4.

@LukeMathWalker
LukeMathWalker force-pushed the refactor/orchestrator-detached-launch-guard branch from 012749a to d11f567 Compare August 10, 2026 06:42
@LukeMathWalker
LukeMathWalker force-pushed the refactor/orchestrator-detached-launch-guard branch from d11f567 to dc0857d Compare August 10, 2026 06:55
@LukeMathWalker
LukeMathWalker force-pushed the refactor/orchestrator-detached-launch-guard branch from 16a7bd3 to 18bc25a Compare August 10, 2026 08:43
@LukeMathWalker
LukeMathWalker force-pushed the refactor/orchestrator-detached-launch-guard branch from 18bc25a to 8820d38 Compare August 10, 2026 14:59
@LukeMathWalker
LukeMathWalker marked this pull request as ready for review August 10, 2026 15:15
@LukeMathWalker
LukeMathWalker added this pull request to the merge queue Aug 10, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to invalid changes in the merge commit Aug 10, 2026
Base automatically changed from refactor/orchestrator-owned-spawn to main August 10, 2026 15:21
@LukeMathWalker
LukeMathWalker force-pushed the refactor/orchestrator-detached-launch-guard branch from 8820d38 to 26d6a20 Compare August 10, 2026 15:21
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