Skip to content

feat: session resume with conflict detection + Windows PTY fix#2

Merged
jmanuelcorral merged 2 commits into
mainfrom
feat/session-resume-and-windows-pty
Apr 29, 2026
Merged

feat: session resume with conflict detection + Windows PTY fix#2
jmanuelcorral merged 2 commits into
mainfrom
feat/session-resume-and-windows-pty

Conversation

@jmanuelcorral
Copy link
Copy Markdown
Owner

Summary

Adds the ability to resume previous Copilot CLI sessions from the session history panel, plus a Windows PTY launch fix for Volta/.cmd shims.

Features

  • Resume button (Play icon, emerald) on each session card in SessionHistoryPanel
  • Conflict detection: if an active session exists, a confirmation dialog offers to stop it first
  • Two-step IPC flow: \sessions:resume\ (returns conflict info) + \sessions:forceResume\ (stops active + resumes)
  • spawnCopilotPty() helper shared by both start and resume paths

Fixes

  • Windows PTY launch: routes \copilot\ through \cmd.exe /d /c\ so Volta .cmd\ shims resolve (fixes \�rror code: 2)
  • Hooks server URL: replaced hardcoded \localhost:3001\ with dynamic \getHooksServerUrl()\

Testing

  • New E2E test (\

Copilot and others added 2 commits April 29, 2026 08:23
- Add resumeCopilotSession() with --resume flag and conflict detection
- Add sessions:resume and sessions:forceResume IPC channels
- Add resume button (Play icon) on SessionHistoryPanel session cards
- Add conflict confirmation dialog (stop active + resume selected)
- Fix Windows PTY launch: route copilot through cmd.exe /d /c for .cmd shims
- Extract spawnCopilotPty() helper shared by start and resume paths
- Fix hardcoded hooks server URL; use getHooksServerUrl() dynamically
- Add E2E test for session launch with mock copilot CLI
- Add squad skills docs and .gitignore for test artifacts

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Narrow resume conflicts before reading resumed session id
- Log resume failures instead of swallowing errors silently
- Ignore local Copilot hook config generated during session launch tests

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jmanuelcorral jmanuelcorral merged commit c84baca into main Apr 29, 2026
1 check passed
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