Skip to content

feat(framework): Add app supervisor for *app lifecycle management on posix systems without orchestration#7062

Draft
msheller wants to merge 46 commits into
mainfrom
codex/supervisor-app
Draft

feat(framework): Add app supervisor for *app lifecycle management on posix systems without orchestration#7062
msheller wants to merge 46 commits into
mainfrom
codex/supervisor-app

Conversation

@msheller
Copy link
Copy Markdown
Member

Description

SuperExec needs a trusted host-side lifecycle primitive for future local app supervision. The existing app-side --parent-pid model depends on host PID visibility, which is not a good fit for PID namespace isolation.

Related issues/PRs

Related to future SuperExec local subprocess sandboxing work.

Explanation

Add a POSIX app supervisor primitive for SuperExec. The supervisor launches a final app command in a new process group, monitors a parent-owned lifeline FD, and terminates the app process group if the lifeline closes. The helper also uses config and status pipes to keep app command details out of supervisor argv and to report app launch failures before returning.

This PR adds the primitive, tests, and an RFC. It does not wire existing SuperExec plugins to use the supervisor yet.

Checklist

  • Implement proposed change
  • Write tests
  • Update documentation
  • Address LLM-reviewer comments, if applicable (e.g., GitHub Copilot)
  • Make CI checks pass
  • Ping maintainers on Slack (channel #contributions)

Any other comments?

Follow-up PR will wire SuperExec app launches through this supervisor.

@github-actions github-actions Bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Apr 29, 2026
@msheller msheller changed the base branch from report-clientapp-appio-communication-failures-as-errors to main May 4, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainer Used to determine what PRs (mainly) come from Flower maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants