Skip to content

docs(coop): correct amq-keepalive reboot vs sleep mechanism - #264

Merged
avivsinai merged 1 commit into
mainfrom
docs/keepalive-wording-fix
Jul 21, 2026
Merged

docs(coop): correct amq-keepalive reboot vs sleep mechanism#264
avivsinai merged 1 commit into
mainfrom
docs/keepalive-wording-fix

Conversation

@avivsinai

Copy link
Copy Markdown
Owner

Follow-up accuracy fix to #263. Codex caught that the wording (inherited verbatim from Ohad's #244) conflated two distinct mechanisms — it credited the user LaunchAgent with reattaching wake after reboot.

Verified against the companion README's Boundaries section:

  • "The tool does not launch or resurrect terminal sessions."
  • "Reboot survival comes from reattaching on session start, not from assuming a terminal id survives … restart. The recreated session registers its current target."

So: the LaunchAgent supervises registered wakes through sleep; the SessionStart hooks handle reboot reattach. Split the claim accordingly.

Caught by codex; verified live against ohade/amq-keepalive README.

🤖 Generated with Claude Code

The #263 wording (inherited from #244) conflated two distinct mechanisms: it
credited the user LaunchAgent with reattaching wake after reboot. Per the
companion README's Boundaries, the tool does not resurrect terminal sessions —
the LaunchAgent supervises registered wakes through sleep, while reboot
survival comes from the SessionStart hooks reattaching the recreated session
(which registers its current target). Split the claim accordingly.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@avivsinai
avivsinai merged commit cec0336 into main Jul 21, 2026
5 checks passed
@avivsinai
avivsinai deleted the docs/keepalive-wording-fix branch July 21, 2026 18:27
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