Skip to content

feat: add mcp-ssh-tool#1089

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

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

Conversation

@oaslananka
Copy link

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
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.
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