Skip to content

Remove heartbeat from exec-based Claude execution#109

Draft
mcintyre94 wants to merge 1 commit intomainfrom
i-suspect-that-with-the-recent-change-from-using-a-b0b4dfec
Draft

Remove heartbeat from exec-based Claude execution#109
mcintyre94 wants to merge 1 commit intomainfrom
i-suspect-that-with-the-recent-change-from-using-a-b0b4dfec

Conversation

@mcintyre94
Copy link
Owner

Summary

  • Removes the 20s stderr heartbeat that was added to keep sprites alive while Claude waits for API responses
  • Exec sessions stay alive as long as we're connected and for up to an hour after disconnect, so it's no longer needed
  • The heartbeat was specifically a workaround for the service-based approach where output inactivity could cause the sprite to pause

Test plan

  • Send a chat message and verify Claude responds normally
  • Let Claude do a slow tool call (e.g. a long bash command) and confirm the session doesn't drop
  • Verify the exec session stays alive across a longer idle period

🤖 Generated with Claude Code

Exec sessions stay alive as long as we're connected and for up to an
hour after disconnect, so the 20s stderr heartbeat that was needed to
keep service-based sprites alive is no longer necessary.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
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