Add login and port override flags to wsh ssh

[Copilot]

wsh ssh already accepted user@host[:port], but there was no way to override the parsed user or port from flags. This change adds -l/--login and -p/--port, with flag values taking precedence over the target string.

• CLI surface

  • Adds -l, --login to set the remote user
  • Adds -p, --port to set the remote port

• Override behavior

  • Normalizes the final SSH target before connect/block metadata updates
  • Reuses the existing shared SSH target parser/formatter, so overrides apply consistently to:
    • the connection request
    • the stored connection string shown in block metadata

• Coverage

  • Adds focused unit tests for:
    • login override
    • port override
    • combined login + port override
    • bare host inputs
    • invalid target handling when overrides are requested

Example:

wsh ssh -l foo root@bar.com
# => connects/stores foo@bar.com

wsh ssh -p 2222 root@bar.com:2022
# => connects/stores root@bar.com:2222

wsh ssh -l foo -p 2200 bar.com
# => connects/stores foo@bar.com:2200

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)

• cmd/wsh/cmd/wshcmd-ssh.go - Added --login and --port flags for SSH connection override
• cmd/wsh/cmd/wshcmd-ssh_test.go - New test file with comprehensive test coverage

The implementation adds two new CLI flags (--login / -l and --port / -p) to the wsh ssh command, allowing users to override the login username and port when connecting to remote hosts. The code follows proper patterns:

• Uses cobra's ExactArgs(1) validation to ensure the SSH target argument is provided
• Applies overrides before using the argument for connection setup
• Properly handles errors from remote.ParseOpts()
• Includes comprehensive test coverage for various edge cases (no overrides, login only, port only, both, bare hosts, invalid targets)