feat(plugin): add mempalace-remote — SSH-proxied palace for remote clients

[messelink]

Summary

Adds a sibling plugin, mempalace-remote, that mirrors the existing mempalace plugin's MCP tools and auto-save hooks but proxies them to a central host over SSH instead of running locally. Lets a fleet of client machines (VPS, second laptop, dev container, etc.) share one canonical palace without each running its own mempalace install.

The marketplace plugins array gains a second entry, so the same MemPalace/mempalace repo serves both plugins. Existing users are unaffected — claude /plugin install mempalace@mempalace still installs only the local-install variant.

Why

I run mempalace on a home server and use Claude Code from several other machines. The hand-rolled approach was a pair of ~/.local/bin/{mempalace,mempalace-mcp} shell wrappers that ssh ... to the central install. That works for the MCP server but skips the auto-save Stop and PreCompact hooks entirely, since the upstream plugin's hook scripts call the local mempalace CLI directly. Packaging the wrappers + hook proxies as a real plugin makes the central-palace pattern install-and-go for anyone running the same setup.

Discussed (lightly) in #1049 and adjacent threads about non-local mempalace deployments.

Layout

Mirrors the existing .claude-plugin/ structure:

.claude-plugin-remote/
├── plugin.json                       # plugin manifest
├── .mcp.json                         # ${CLAUDE_PLUGIN_ROOT}/bin/mempalace-mcp-ssh.sh
├── README.md
├── bin/
│   └── mempalace-mcp-ssh.sh          # thin SSH wrapper for the MCP server
└── hooks/
    ├── hooks.json                    # registers Stop + PreCompact
    ├── mempal-stop-hook.sh           # ssh-pipe stdin to remote `mempalace hook run`
    └── mempal-precompact-hook.sh     # same for precompact

.claude-plugin/marketplace.json gains a single appended entry pointing at ./.claude-plugin-remote.

Configuration

Env var	Required	Default	Description
MEMPALACE_REMOTE_HOST	yes	—	SSH target — alias from ~/.ssh/config or fully qualified hostname
MEMPALACE_REMOTE_BIN	no	mempalace	Path to the mempalace CLI on the remote (used by Stop / PreCompact hooks)
MEMPALACE_REMOTE_MCP_BIN	no	mempalace-mcp	Path to the mempalace-mcp server on the remote (used by the MCP client)

The _BIN overrides cover the SSH non-interactive PATH gotcha where ~/.local/bin isn't on PATH for ssh host command shells. Bare defaults work for system-wide installs.

How it works

• MCP server — claude spawns ${CLAUDE_PLUGIN_ROOT}/bin/mempalace-mcp-ssh.sh, which validates the env vars and execs ssh -- $HOST mempalace-mcp. Stdio JSON-RPC rides the SSH channel transparently — the MCP client doesn't know it's remote.
• Stop / PreCompact hooks — fire on the client, validate env vars, pipe Claude Code's hook JSON to ssh -- $HOST mempalace hook run --hook {stop,precompact} --harness claude-code. The remote mempalace CLI handles all the actual logic; the bash wrapper is intentionally thin.

Coexistence

Don't enable both mempalace and mempalace-remote on the same machine — both register an MCP server named mempalace and the second one to load shadows the first. The README calls this out.

Tested

End-to-end test on a real two-machine setup (VPS client → home-server central palace):

• claude /plugin marketplace add → claude /plugin install mempalace-remote@mempalace → set env vars → restart Claude Code → working
• mempalace_status MCP call returned real wing/room counts from the central palace ✓
• Stop hook fired and landed in ~/.mempalace/hook_state/hook.log on the central host with the correct client session ID ✓
• PreCompact hook (/compact) fired and landed in the same log ✓
• Latency: sub-300ms per MCP call without SSH ControlMaster; ControlMaster optional, documented in README.

claude plugin validate passes on both .claude-plugin/plugin.json and .claude-plugin-remote/plugin.json.

Security

Two issues caught in review and fixed:

• SSH option-parsing of host — MEMPALACE_REMOTE_HOST is now passed after -- so a value starting with - can't be interpreted as ssh options (e.g. -oProxyCommand=... local code execution). Caught by Qodo review.
• Remote command injection via the _BIN env vars — MEMPALACE_REMOTE_BIN and MEMPALACE_REMOTE_MCP_BIN are validated against an allow-list regex (^[A-Za-z0-9_./-]+$) before being concatenated into the SSH remote command string, blocking the BIN='...; rm -rf ~ #' class. The MCP server config goes through the bin/mempalace-mcp-ssh.sh wrapper to apply the same defense the JSON config can't express directly.

Notes for review

• New env vars use the MEMPALACE_REMOTE_* namespace; no clash with existing MEMPAL_* env vars used by the local plugin.
• The author field on the new marketplace entry is messelink (mine). Happy to change to whatever attribution convention you prefer.
• Drift watching: if upstream changes .claude-plugin/hooks/hooks.json (e.g. adds SessionStart), the remote variant should mirror the addition. Same protocol, same CLI signature. Adjacent: Consolidate hooks/mempal_*_hook.sh into thin wrappers delegating to mempalace hook run #1069's proposed consolidation of the legacy hooks/mempal_*_hook.sh scripts uses the same thin-wrapper-to-mempalace hook run pattern this PR follows.
• Codex variant: not included here — same pattern would work for .codex-plugin-remote/ if there's interest.

[Qodo-Free-For-OSS]

Hi, MEMPALACE_REMOTE_HOST is passed as the first positional argument to ssh without an option terminator, so a value starting with "-" will be parsed as ssh options (e.g., ProxyCommand), enabling unintended local command execution under the plugin process. This affects both the MCP server spawn and the Stop/PreCompact hook invocations.

Severity: action required | Category: security

How to fix: Terminate ssh options with --

Agent prompt to fix - you can give this to your LLM of choice:

Issue description

MEMPALACE_REMOTE_HOST is passed to ssh as the destination without terminating option parsing. If it starts with -, OpenSSH will interpret it as options (e.g. -o ProxyCommand=...), which can lead to unexpected local command execution.

Issue Context

This affects both:

• the MCP server spawn (plugin.json / .mcp.json), and
• the Stop/PreCompact hook scripts.

Fix Focus Areas

• .claude-plugin-remote/hooks/mempal-stop-hook.sh[1-4]
• .claude-plugin-remote/hooks/mempal-precompact-hook.sh[1-4]
• .claude-plugin-remote/plugin.json[10-15]
• .claude-plugin-remote/.mcp.json[1-6]

Implementation notes

• Update shell scripts to ssh -- "${MEMPALACE_REMOTE_HOST:?…}" ....
• Update JSON args to include "--" before the host: "args": ["--", "${MEMPALACE_REMOTE_HOST}", ...].
• (Optional hardening) Reject host values beginning with - explicitly before invoking ssh.

We noticed a couple of other issues in this PR as well - happy to share if helpful.

---

Spotted by Qodo code review - free for open-source projects.

[messelink]

Thanks @qodo-ai-reviewer — good catch. Fixed in 3844bbe: added -- before the host in both .mcp.json (and the equivalent block in plugin.json) and both hook scripts.

Yes please do share the other issues you spotted.

[messelink]

Found a related second issue while looking for more — the same env-var injection vector applies to the bin path arguments:

• MEMPALACE_REMOTE_BIN (in the hook scripts) and MEMPALACE_REMOTE_MCP_BIN (in the MCP server config) are concatenated into the SSH remote command string, which the central host's shell parses. A value like mempalace; touch /tmp/X # becomes sh -c "mempalace; touch /tmp/X # hook run ..." on the remote — RCE on the central host as the SSH user.

Fixed in d6e728b:

• Hook scripts validate HOST and BIN against allow-list regexes before invoking ssh, fail with a clear message otherwise.
• MCP server entry now goes through a thin wrapper at bin/mempalace-mcp-ssh.sh that does the same validation, called via ${CLAUDE_PLUGIN_ROOT}/bin/... — lets us apply the same defense in the JSON config without trying to shell-quote inside JSON.
• The -- option terminator stays as belt-and-suspenders against the host-as-option case from your first finding.

Smoke-tested both: legitimate values still produce {} from the Stop hook end-to-end; injection attempts are rejected before any ssh call.

If you have more findings still queued, please share — happy to keep iterating.