Skip to content

fix(gateway): keep launchd KeepAlive unconditional for macOS resilience - #38837

Closed
drzeast-png wants to merge 1 commit into
NousResearch:mainfrom
drzeast-png:pr/keepalive-unconditional
Closed

fix(gateway): keep launchd KeepAlive unconditional for macOS resilience#38837
drzeast-png wants to merge 1 commit into
NousResearch:mainfrom
drzeast-png:pr/keepalive-unconditional

Conversation

@drzeast-png

Copy link
Copy Markdown

Cherry-picked onto latest NousResearch:main (c2ca3f0).

Replaces #35668 (closed — head branch was force-pushed/recreated, GitHub refuses reopen).

@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists labels Jun 4, 2026
@drzeast-png
drzeast-png force-pushed the pr/keepalive-unconditional branch from 57afd73 to 334bd43 Compare June 4, 2026 11:37
@drzeast-png
drzeast-png requested a review from a team June 4, 2026 11:37
@drzeast-png
drzeast-png force-pushed the pr/keepalive-unconditional branch 2 times, most recently from ae3d76e to 5064819 Compare June 4, 2026 12:37
…resilience

Changes KeepAlive from {SuccessfulExit: false} to unconditional <true/>
so launchd restarts the gateway process on every exit, not just abnormal
exits.  This prevents edge cases where a clean shutdown (exit code 0)
leaves the gateway dead until manually restarted.
@drzeast-png
drzeast-png force-pushed the pr/keepalive-unconditional branch from 5064819 to e1974da Compare June 4, 2026 12:39
@drzeast-png drzeast-png closed this Jun 4, 2026
@drzeast-png
drzeast-png deleted the pr/keepalive-unconditional branch June 4, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants