Skip to content

fix(ux): correct reconnect command suggestion from "spawn connect" to "spawn last"#3311

Merged
louisgv merged 1 commit intomainfrom
fix/ux-reconnect-command-suggestion
Apr 16, 2026
Merged

fix(ux): correct reconnect command suggestion from "spawn connect" to "spawn last"#3311
louisgv merged 1 commit intomainfrom
fix/ux-reconnect-command-suggestion

Conversation

@la14-1
Copy link
Copy Markdown
Member

@la14-1 la14-1 commented Apr 16, 2026

Why: After exhausting SSH reconnect attempts, the CLI tells users "Reconnect manually: spawn connect" — but spawn connect is not a valid top-level command. Users following this guidance see "Unknown agent or cloud: connect", a confusing dead-end. spawn last is the correct command to reconnect to the most recent spawn.

Changes

  • packages/cli/src/shared/orchestrate.ts: Replace spawn connect with spawn last in reconnect failure message
  • packages/cli/package.json: Bump version 1.0.14 → 1.0.15

… "spawn last"

"spawn connect" is not a valid top-level CLI command — users following
this guidance after SSH reconnect failure would see "Unknown agent or
cloud: connect". Replace with "spawn last" which correctly reconnects
to the most recent spawn.

Agent: ux-engineer
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@la14-1 la14-1 force-pushed the fix/ux-reconnect-command-suggestion branch from 9ab0785 to 02ff629 Compare April 16, 2026 07:14
Copy link
Copy Markdown
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security Review

Verdict: APPROVED — Clean UX fix
Commit: 02ff629

Changes

  • Fixed incorrect reconnect command suggestion from "spawn connect" to "spawn last"
  • Version bump: 1.0.14 → 1.0.15

Security Analysis

  • No injection risks: Simple string literal change
  • No credential handling: User-facing message only
  • No command execution: Log message, not shell invocation
  • No data validation issues: Hardcoded strings

Tests

  • bun test: ✅ PASS (2070 tests, 0 failures)
  • bash -n: N/A (no shell scripts modified)

Recommendation

Safe to merge — This is a trivial UX improvement with zero security implications.


-- security/pr-reviewer

@louisgv louisgv added the security-approved Security review approved label Apr 16, 2026
@louisgv louisgv merged commit 513d344 into main Apr 16, 2026
5 checks passed
@louisgv louisgv deleted the fix/ux-reconnect-command-suggestion branch April 16, 2026 08:39
la14-1 pushed a commit that referenced this pull request Apr 16, 2026
Two user-facing reconnect hints missed by #3311 still showed
'spawn connect <name>', which is not a registered command. Users
following the hint get 'Unknown agent or cloud: connect'.

Agent: code-health
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AhmedTMM added a commit that referenced this pull request Apr 19, 2026
Two user-facing reconnect hints missed by #3311 still showed
'spawn connect <name>', which is not a registered command. Users
following the hint get 'Unknown agent or cloud: connect'.

Agent: code-health

Co-authored-by: B <6723574+louisgv@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-authored-by: Ahmed Abushagur <ahmed@abushagur.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

security-approved Security review approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants