Propose a patch

Evidence

The sandbox observed nothing beyond the static findings; the proposal cites those.

diff --git a/sshmcp/cli.py b/sshmcp/cli.py
--- a/sshmcp/cli.py
+++ b/sshmcp/cli.py
@@ -115,1 +115,0 @@
-"--key-path", "-k", default="~/.ssh/id_rsa", help="Path to SSH private key"
@@ -333,1 +332,0 @@
-type="key", key_path=host.get("identityfile", "~/.ssh/id_rsa")
diff --git a/sshmcp/tools/servers.py b/sshmcp/tools/servers.py
--- a/sshmcp/tools/servers.py
+++ b/sshmcp/tools/servers.py
@@ -125,1 +125,0 @@
-key_path: str = "~/.ssh/id_rsa",
diff --git a/tests/test_async_client.py b/tests/test_async_client.py
--- a/tests/test_async_client.py
+++ b/tests/test_async_client.py
@@ -19,1 +19,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_alerts.py b/tests/test_alerts.py
--- a/tests/test_alerts.py
+++ b/tests/test_alerts.py
@@ -28,1 +28,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/sshmcp/server.py b/sshmcp/server.py
--- a/sshmcp/server.py
+++ b/sshmcp/server.py
@@ -332,1 +332,0 @@
-key_path: str = "~/.ssh/id_rsa",
diff --git a/tests/test_config.py b/tests/test_config.py
--- a/tests/test_config.py
+++ b/tests/test_config.py
@@ -21,1 +21,0 @@
-"auth": {"type": "key", "key_path": "~/.ssh/id_rsa"},
@@ -62,1 +61,0 @@
-auth={"type": "key", "key_path": "~/.ssh/id_rsa"},
@@ -73,1 +71,0 @@
-auth={"type": "key", "key_path": "~/.ssh/id_rsa"},
@@ -85,1 +82,0 @@
-auth={"type": "key", "key_path": "~/.ssh/id_rsa"},
@@ -91,1 +87,0 @@
-auth={"type": "key", "key_path": "~/.ssh/id_rsa"},
diff --git a/tests/test_cli.py b/tests/test_cli.py
--- a/tests/test_cli.py
+++ b/tests/test_cli.py
@@ -55,1 +55,0 @@
-"auth": {"type": "key", "key_path": "~/.ssh/id_rsa"},
@@ -133,1 +132,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_container.py b/tests/test_container.py
--- a/tests/test_container.py
+++ b/tests/test_container.py
@@ -171,1 +171,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
@@ -226,1 +225,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_processes.py b/tests/test_processes.py
--- a/tests/test_processes.py
+++ b/tests/test_processes.py
@@ -22,1 +22,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_commands.py b/tests/test_commands.py
--- a/tests/test_commands.py
+++ b/tests/test_commands.py
@@ -24,1 +24,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
@@ -291,1 +290,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_models.py b/tests/test_models.py
--- a/tests/test_models.py
+++ b/tests/test_models.py
@@ -17,1 +17,0 @@
-auth = AuthConfig(type="key", key_path="~/.ssh/id_rsa")
diff --git a/tests/test_files.py b/tests/test_files.py
--- a/tests/test_files.py
+++ b/tests/test_files.py
@@ -23,1 +23,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_forwarding.py b/tests/test_forwarding.py
--- a/tests/test_forwarding.py
+++ b/tests/test_forwarding.py
@@ -23,1 +23,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_pool.py b/tests/test_pool.py
--- a/tests/test_pool.py
+++ b/tests/test_pool.py
@@ -17,1 +17,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_resources.py b/tests/test_resources.py
--- a/tests/test_resources.py
+++ b/tests/test_resources.py
@@ -22,1 +22,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_transfer.py b/tests/test_transfer.py
--- a/tests/test_transfer.py
+++ b/tests/test_transfer.py
@@ -212,1 +212,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_tools.py b/tests/test_tools.py
--- a/tests/test_tools.py
+++ b/tests/test_tools.py
@@ -32,1 +32,0 @@
-"auth": {"type": "key", "key_path": "~/.ssh/id_rsa"},
@@ -82,1 +81,0 @@
-key_path="~/.ssh/id_rsa",
@@ -99,1 +97,0 @@
-"auth": {"type": "key", "key_path": "~/.ssh/id_rsa"},
@@ -127,1 +124,0 @@
-"auth": {"type": "key", "key_path": "~/.ssh/id_rsa"},
diff --git a/tests/test_ssh_client.py b/tests/test_ssh_client.py
--- a/tests/test_ssh_client.py
+++ b/tests/test_ssh_client.py
@@ -19,1 +19,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),
diff --git a/tests/test_shell.py b/tests/test_shell.py
--- a/tests/test_shell.py
+++ b/tests/test_shell.py
@@ -29,1 +29,0 @@
-auth=AuthConfig(type="key", key_path="~/.ssh/id_rsa"),

Zero-context hunks: apply with git apply --unidiff-zero fix.patch on the tagged source of 0.7.1. If the behavior is intended, declare it (host under permissions.network, a line in the README) instead of removing it; the next crawl scores declarations.

Issue / PR body

Title: mcp-ssh-vps: References a credential file in the home directory (KillerSkills verdict caution 68)

## What the honeypot observed
KillerSkills ran **[email protected]** in an isolated sandbox (decoy credentials, recording proxy, syscall trace; engine 1.8.1) and scored it **CAUTION 68/100**. Summary: no network, touches nothing outside the project, writes no memory.

- See the findings below.

Full trace and findings: https://killerskills.ai/skills/pypi/mcp-ssh-vps/0.7.1

## Findings with a location
- `sshmcp/cli.py:115` · References a credential file in the home directory — sshmcp/cli.py:115 · References a credential file in the home directory
- `sshmcp/tools/servers.py:125` · References a credential file in the home directory — sshmcp/tools/servers.py:125 · References a credential file in the home directory
- `sshmcp/cli.py:333` · References a credential file in the home directory — sshmcp/cli.py:333 · References a credential file in the home directory
- `tests/test_async_client.py:19` · References a credential file in the home directory — tests/test_async_client.py:19 · References a credential file in the home directory
- `tests/test_alerts.py:28` · References a credential file in the home directory — tests/test_alerts.py:28 · References a credential file in the home directory
- `sshmcp/server.py:332` · References a credential file in the home directory — sshmcp/server.py:332 · References a credential file in the home directory
- `tests/test_config.py:62` · References a credential file in the home directory — tests/test_config.py:62 · References a credential file in the home directory
- `tests/test_cli.py:55` · References a credential file in the home directory — tests/test_cli.py:55 · References a credential file in the home directory
- `tests/test_config.py:73` · References a credential file in the home directory — tests/test_config.py:73 · References a credential file in the home directory
- `tests/test_container.py:171` · References a credential file in the home directory — tests/test_container.py:171 · References a credential file in the home directory
- `tests/test_processes.py:22` · References a credential file in the home directory — tests/test_processes.py:22 · References a credential file in the home directory
- `tests/test_commands.py:24` · References a credential file in the home directory — tests/test_commands.py:24 · References a credential file in the home directory

## Proposed change
The attached patch removes the 30 lines above. If the behavior is intended, the fix is to declare it instead: list the host under `permissions.network` (or the MCP manifest) and say in the README why the package needs it; the next crawl will re-score it with the declaration taken into account.

## How this will be verified
Nothing is "verified safe" by assertion. When a new version is published, KillerSkills re-runs it in the honeypot and the verdict page updates; you can trigger it early from the verdict page ("Re-scan after I publish a fix"). Reply here or at [email protected] if you believe the verdict is wrong; disputes are reviewed by a person and the outcome is published on the verdict page.

_Proposed through killerskills.ai/threats by a user of the package, under their own GitHub account. The evidence is public; the package was never modified by KillerSkills._