Skip to content

fix(mcp): inject default namespace for memwal_remember_bulk calls - #667

Open
ducnmm wants to merge 1 commit into
devfrom
fix/gh-625-mcp-remember-bulk-namespace
Open

fix(mcp): inject default namespace for memwal_remember_bulk calls#667
ducnmm wants to merge 1 commit into
devfrom
fix/gh-625-mcp-remember-bulk-namespace

Conversation

@ducnmm

@ducnmm ducnmm commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Resolves #625 (and companion #611)

Summary

Includes memwal_remember_bulk in the set of NAMESPACE_TOOLS so the configured default namespace (--namespace / MEMWAL_NAMESPACE) is injected when an agent calls bulk remember without an explicit namespace.

Problem

Previously, NAMESPACE_TOOLS in packages/mcp/src/bridge.ts only included memwal_remember, memwal_recall, memwal_analyze, and memwal_restore. When an agent invoked memwal_remember_bulk without an explicit namespace, no namespace argument was injected and the batch was written into the relayer's fallback default namespace, making it unrecallable from the configured project namespace.

Changes

  • Added "memwal_remember_bulk" to NAMESPACE_TOOLS in packages/mcp/src/bridge.ts.
  • Added unit tests in packages/mcp/test/default-namespace.test.mjs to verify default namespace injection behavior for memwal_remember_bulk and other tools.

Verification

  • pnpm --filter @mysten-incubation/memwal-mcp test (14/14 tests pass)

@ducnmm
ducnmm requested review from HoangDucBach and nikola0x0 and removed request for HoangDucBach August 17, 2026 12:49
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.

memwal_remember_bulk bypasses --namespace injection, silently writing to the relayer default

1 participant