Skip to content

fix(agentos-sidecar): harden the close_session already-exited short-circuit#1658

Merged
NathanFlurry merged 1 commit into
mainfrom
stack/fix-agentos-sidecar-harden-the-close_session-already-exited-short-circuit-mkkuuouw
Jul 8, 2026
Merged

fix(agentos-sidecar): harden the close_session already-exited short-circuit#1658
NathanFlurry merged 1 commit into
mainfrom
stack/fix-agentos-sidecar-harden-the-close_session-already-exited-short-circuit-mkkuuouw

Conversation

@NathanFlurry

Copy link
Copy Markdown
Member

@NathanFlurry

Copy link
Copy Markdown
Member Author

Stack for rivet-dev/agentos

Get stack: forklift get 1658
Push local edits: forklift submit
Merge when ready: forklift merge 1658

@railway-app

railway-app Bot commented Jul 8, 2026

Copy link
Copy Markdown

🚅 Deployed to the agentos-pr-1658 environment in agentos

Service Status Web Updated (UTC)
agentos 😴 Sleeping (View Logs) Web Jul 8, 2026 at 5:23 am

🚅 Environment agentos-pr-1658 in rivet-frontend has no services deployed.

…ircuit

- Builds on #1640: also short-circuit on the session record already marked closed (covers exits the kill RPC does not report), and skip the second wait when SIGKILL itself reports the process gone.
- Mirror the short-circuit in the agentos-sidecar-core sync engine, with regression tests proving zero dead waits.
@NathanFlurry NathanFlurry force-pushed the stack/fix-agentos-sidecar-harden-the-close_session-already-exited-short-circuit-mkkuuouw branch from 0edfcb2 to b461a0d Compare July 8, 2026 06:17
@railway-app railway-app Bot temporarily deployed to agentos / agentos-pr-1658 July 8, 2026 06:17 Destroyed
@NathanFlurry NathanFlurry merged commit b461a0d into main Jul 8, 2026
3 of 4 checks passed
@NathanFlurry NathanFlurry deleted the stack/fix-agentos-sidecar-harden-the-close_session-already-exited-short-circuit-mkkuuouw branch July 8, 2026 06:18
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