Skip to content

feat: add mcp-ssh-tool#1089

Closed
oaslananka wants to merge 2 commits intomodelcontextprotocol:mainfrom
oaslananka:feat/add-mcp-ssh-tool
Closed

feat: add mcp-ssh-tool#1089
oaslananka wants to merge 2 commits intomodelcontextprotocol:mainfrom
oaslananka:feat/add-mcp-ssh-tool

Conversation

@oaslananka
Copy link
Copy Markdown

SSH automation MCP server that enables Claude and ChatGPT to execute commands, manage files, install packages, and control services on remote servers over SSH. Published on npm: https://www.npmjs.com/package/mcp-ssh-tool

Copilot AI review requested due to automatic review settings March 22, 2026 03:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new MCP server descriptor for the mcp-ssh-tool package, intended to document/distribute an SSH automation MCP server via npm.

Changes:

  • Introduces a new server descriptor JSON file under servers/.
  • Provides basic metadata (author/repo/homepage) and an npm package entry for installation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

},
"homepage": "https://github.com/oaslananka/mcp-ssh-tool",
"repository": {
"type": "git",
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

repository objects in server.json require both url and source (e.g., source: "github"). This entry provides type: "git" and omits source, which will fail schema validation if repository is present.

Suggested change
"type": "git",
"type": "git",
"source": "github",

Copilot uses AI. Check for mistakes.
Comment on lines +18 to +20
"name": "mcp-ssh-tool",
"version": "latest",
"runtimeHint": "npx"
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

The packages[0] object doesn’t match the schema: it must use identifier (not name) and must include a transport object (required). Also, version: "latest" is explicitly disallowed/reserved by the schema/validators; use a specific published version (or omit version if appropriate for your registryType).

Suggested change
"name": "mcp-ssh-tool",
"version": "latest",
"runtimeHint": "npx"
"identifier": "mcp-ssh-tool",
"runtimeHint": "npx",
"transport": {
"type": "node",
"command": "npx",
"args": ["mcp-ssh-tool"]
}

Copilot uses AI. Check for mistakes.
@@ -0,0 +1,23 @@
{
"name": "mcp-ssh-tool",
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

This JSON doesn’t conform to the registry’s server.json schema: the top-level version field is required, and name must be in reverse-DNS format with exactly one / (e.g., io.github.<owner>/mcp-ssh-tool). As written ("name": "mcp-ssh-tool" and no version), it will fail validation/import.

Suggested change
"name": "mcp-ssh-tool",
"version": "1.0.0",
"name": "io.github.oaslananka/mcp-ssh-tool",

Copilot uses AI. Check for mistakes.
Comment on lines +4 to +9
"license": "MIT",
"author": {
"name": "Osman Aslan",
"url": "https://github.com/oaslananka"
},
"homepage": "https://github.com/oaslananka/mcp-ssh-tool",
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

Fields like license, author, and homepage are not part of pkg/api/v0.ServerJSON and will be ignored/dropped when unmarshaled/stored by the registry. If you want this information available to clients, map it to supported fields (e.g., websiteUrl, repository, or _meta.io.modelcontextprotocol.registry/publisher-provided).

Copilot uses AI. Check for mistakes.
@oaslananka
Copy link
Copy Markdown
Author

Closing this PR because registry publication for mcp-ssh-tool should go through the documented mcp-publisher flow rather than a manual server file PR. I have updated the server metadata in the source repo and will republish using the official workflow.

@oaslananka oaslananka closed this Mar 24, 2026
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