Conversation
Selectively brings over what the CommandOSSLabs/walrus-memory-plugin repo added on top of this repo's existing bundle: slash commands, the setup skill, and per-client usage docs. Marks the hosted custom-connector doc and setup-skill section as experimental and not production-ready, since the remote MCP OAuth work it depends on (MystenLabs/MemWal PR 584) is still under security review. Leaves the pre-existing hooks.json duplication and the branding attribution question unresolved, per the open items noted in the plugin README.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Summary
Selectively ports what
CommandOSSLabs/walrus-memory-pluginadded on top of theplugin bundle already present in this repo (
plugin/): slash commands, the setupskill, and per-client usage docs. Scripts, hooks, and the MCP config were already
byte-identical between the two repos, so this only adds/updates the parts that were
missing here.
Requested by the task owner in the "Add Claude custom-connector compatibility to
remote MCP" Notion task, following the plan to establish this repo as the single
canonical source and archive the standalone plugin repo once this lands.
What's added
plugin/commands/*.md(7 slash commands: setup, health, remember, recall, analyze, restore, logout)plugin/skills/setup/SKILL.md,plugin/skills/setup/SETUP.mddocs/usage/claude-code.md,docs/usage/codex.md,docs/usage/other-clients.md,docs/usage/hosted-connector.mdplugin/README.md— documents the plugin bundle's contents and known open itemsplugin/.claude-plugin/plugin.json— bumped to0.0.7, addsmcpServers/skills/commands/hooksmanifest pointers (needed now thatplugin/commands/andplugin/skills/exist), fixesrepositoryto point at this repo instead of the oldMystenLabs/MemWalmonorepo pathplugin/.mcp.json— adds--label "Claude Code Plugin"/MEMWAL_CLIENT_LABELso the plugin's delegate key is labeled distinctly from other clients in the dashboardDeliberately not changed
plugin/hooks/,plugin/scripts/— already identical between the two source repos, nothing to port.README.md— documents the standalone stdio package, out of scope for a plugin-bundle port.Marked as experimental — do not remove without checking upstream first
docs/usage/hosted-connector.mdand the hosted-connector section ofplugin/skills/setup/SETUP.mdnow carry an explicit warning that the hosted custom-connector OAuth flow is not
production-ready. That flow depends on MystenLabs/MemWal#584,
which is still under active security review (see the review comments there) — this PR
does not claim that path works end to end. Once #584 is merged and the live smoke test
is confirmed, these warnings should be removed/updated.
Known open items not resolved by this PR
plugin/hooks.json(top level) andplugin/hooks/hooks.jsonare two slightlydifferent hook manifests already present in this repo before this PR — establishing
one canonical file is tracked separately, not attempted here.
author: "Mysten Labs") in aCommandOSSLabs-owned repohas not been formally confirmed with Mysten Labs branding/ownership approval — this
PR carries forward the attribution that was already present in this repo's manifest
before this change, it does not introduce it.
Test plan
plugin/.claude-plugin/plugin.jsonandplugin/.mcp.jsonparse as valid JSON.mdfiles read without encoding/syntax issuesclaude plugin validate . --strictfromplugin/(not run in this environment — please confirm in CI or locally)claude --plugin-dir plugin/)