Skip to content

Conversation

@tufantunc
Copy link
Owner

docs: add Claude Code setup instructions to README

  • Add comprehensive installation guide for Claude Code (Windsurf)
  • Include examples for password/SSH key authentication and custom timeouts
  • Document installation scopes (local, project, user) and environment variables
  • Update client setup introduction to mention various IDEs and LLMs
  • Add verification steps and link to official MCP documentation

- Added SSHConnectionManager class to maintain a single SSH connection
- Connection is automatically established on first command and reused for subsequent commands
- Implements automatic reconnection if connection is lost
- Added graceful shutdown handling to properly close connections
- Kept old execSshCommand function for backward compatibility with tests
- Reduces connection overhead and improves performance for multiple commands
- Created 26 new tests covering all aspects of SSHConnectionManager
- Unit tests: initialization, connection state management, cleanup
- Integration tests: connection reuse, reconnection, concurrent commands
- Connection lifecycle tests: multiple managers, proper cleanup
- Error handling tests: invalid config, timeouts, command errors
- Tests verify connection reuse across multiple commands
- Tests verify auto-reconnection after connection loss
- Tests verify concurrent command execution
- All 39 tests in the suite now pass successfully
- Add comprehensive installation guide for Claude Code (Windsurf)
- Include examples for password/SSH key authentication and custom timeouts
- Document installation scopes (local, project, user) and environment variables
- Update client setup introduction to mention various IDEs and LLMs
- Add verification steps and link to official MCP documentation
@tufantunc tufantunc self-assigned this Oct 21, 2025
@tufantunc tufantunc merged commit b85fd72 into main Oct 21, 2025
2 checks passed
@tufantunc tufantunc deleted the feature/enhance-readme-to-claude-code branch October 21, 2025 04:43
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.

2 participants