Skip to content

Fix agent termination cleanup and Codex slash command UX#24

Merged
blipinsk merged 1 commit intomainfrom
fix-agent-termination-and-codex-ux
Mar 26, 2026
Merged

Fix agent termination cleanup and Codex slash command UX#24
blipinsk merged 1 commit intomainfrom
fix-agent-termination-and-codex-ux

Conversation

@blipinsk
Copy link
Copy Markdown
Member

Summary

  • Use SIGINT instead of SIGTERM when stopping agents via queue actions, so agents (especially Claude Code) run their full cleanup path - hooks, terminal state restore, session save
  • Drain kitty keyboard protocol stack after agent exit to prevent terminal corruption where Ctrl+C shows as raw ^[[99;5u escape sequences
  • Scope slash command body guard to the triggering message only, fixing Codex rejecting follow-up confirmations like "y" or "yes" because they don't contain "mob"/"codemob"
  • Note Codex slash command rough edges in README

Test plan

  • Verified kitty protocol fix - terminal stays clean after queue-based switch
  • Verified SIGINT - Claude exits gracefully on queue action
  • Verified Codex - slash commands accept follow-up confirmations
  • Verify on a terminal without kitty KB protocol support (escape sequence should be silently ignored)

- Use SIGINT instead of SIGTERM when stopping agents via queue actions,
  so agents run their full cleanup path (hooks, terminal state, etc.)
- Drain kitty keyboard protocol stack after agent exit to prevent
  terminal corruption (Ctrl+C showing as raw escape sequences)
- Scope slash command body guard to the triggering message only, fixing
  Codex rejecting follow-up confirmations like "y" or "yes"
- Note Codex slash command rough edges in README
@blipinsk blipinsk merged commit edf2a28 into main Mar 26, 2026
1 check passed
@blipinsk blipinsk deleted the fix-agent-termination-and-codex-ux branch March 26, 2026 18:23
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