feat: Major refactor to v1.0.0 - Redesigned MCP tool surface and persistence layer #5
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…istence layer - Complete redesign of MCP tools: replaced create_connection/list_connections/close_connection with streamlined set: list_servers, save_server, remove_server, execute_command, upload_file, download_file - Moved persistence from ~/.remoteShell/config.json to ~/.config/remoteshell/hosts.json - Added connection status caching with last_connected timestamp tracking - Enhanced error messages with machine-readable codes (e.g., auth_failed) for better LLM recovery - Implemented automatic local path generation for file transfer tools - Simplified configuration to single uvx setup method - Updated to FastMCP 2.x features for richer tool descriptions - Updated version to 1.0.0
chouzz
force-pushed
the
feature/v1.0.0-refactor
branch
from
413522e to
6be2303
Compare
chouzz
added a commit
that referenced
this pull request
Dec 23, 2025
…istence layer (#5) - Complete redesign of MCP tools: replaced create_connection/list_connections/close_connection with streamlined set: list_servers, save_server, remove_server, execute_command, upload_file, download_file - Moved persistence from ~/.remoteShell/config.json to ~/.config/remoteshell/hosts.json - Added connection status caching with last_connected timestamp tracking - Enhanced error messages with machine-readable codes (e.g., auth_failed) for better LLM recovery - Implemented automatic local path generation for file transfer tools - Simplified configuration to single uvx setup method - Updated to FastMCP 2.x features for richer tool descriptions - Updated version to 1.0.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Major refactor to v1.0.0 - Redesigned MCP tool surface and persistence layer
This PR implements a comprehensive refactor of the RemoteShell MCP server according to the task.md requirements:
Key Changes
Complete redesign of MCP tools: Replaced the old 6 tools with a new streamlined set:
list_servers- Get all saved server configurations with statussave_server- Persist server connection inforemove_server- Delete saved server configurationsexecute_command- Execute non-interactive shell commandsupload_file- Upload files to remote serversdownload_file- Download files from remote serversNew persistence layer: Moved from
~/.remoteShell/config.jsonto~/.config/remoteshell/hosts.jsonwith enhanced schemaConnection status caching: Added
last_connectedtimestamp tracking to help LLMs prioritize recently used serversEnhanced error handling: Added machine-readable error codes (e.g.,
auth_failed) to enable better LLM recovery and user guidanceAutomatic path handling: File transfer tools now auto-generate local paths when omitted and return them in responses
Simplified configuration: Single
uvxsetup method only, removed complex multi-config approachesFastMCP 2.x features: Leveraged new decorator capabilities for richer tool descriptions and parameter validation
Breaking Changes
Testing
All tests pass and the server builds correctly with the new architecture.