Skip to content

fix: resume opens the right session and banner Resume checks the prompt (INC-487) - #540

Merged
arzafran merged 2 commits into
mainfrom
franco/inc-487-resume
Nov 4, 2026
Merged

arzafran merged 2 commits into
mainfrom
franco/inc-487-resume

Conversation

@arzafran

@arzafran arzafran commented Nov 4, 2026

Copy link
Copy Markdown
Member

Resume could type into the wrong place or reopen the wrong session. This fixes three causes.

  • The interrupted-agent banner's Resume typed the resume command into the pane without checking that the shell was at its prompt, so it could land in vim or ssh. It now checks, like the command palette does, and beeps otherwise.
  • Two panes in the same folder overwrote each other's resume history, so resume could reopen the other pane's session. Entries are now kept per pane and agent.
  • A session-end hook with an unknown id cleared the newest live session on the pane. Now only that id's record is removed.

Two new Python regression tests in tests/test_cli_agent_resume.py. The first commit adds them alone, so CI shows them failing; the second commit has the fix.

Not seen in a running app.

Fixes INC-487

@arzafran
arzafran merged commit da48903 into main Nov 4, 2026
12 checks passed
@arzafran
arzafran deleted the franco/inc-487-resume branch November 4, 2026 21:56
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