Sync upstream and keep MongoDB-only memory - #1
Merged
Conversation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
atlas-local-previewandatlas-cloudprofile handling, restored ClawMongo identity/skills/template guidance, wired MongoDB memory tools into the current upstream tool registry, added a local Memongo-style bridge facade, added recall trace and memory job primitives, restored plugin/test compatibility needed by the upstream suite, and added a real E2E preflight script that requires MongoDB URI plus Voyage credentials.Change Type (select all)
Scope (select all touched areas)
Linked Issue/PR
Real behavior proof (required for external PRs)
pnpm check,pnpm build, targeted Mongo/plugin/gateway/command tests, repeated fullpnpm testattempts,pnpm clawmongo:e2epreflight.Root Cause (if applicable)
Regression Test Plan (if applicable)
src/memory/backend-config.test.ts,src/memory/mongodb-schema.test.ts,src/agents/tools/memory-tool-mongodb.test.ts,src/memory/mongodb-manager.test.ts,src/gateway/server-startup-memory.test.ts,src/plugins/memory-runtime.test.ts,src/plugins/uninstall.test.ts,src/commands/configure-memory.test.ts.User-visible / Behavior Changes
ClawMongo memory is explicitly MongoDB-only with
atlas-local-previewandatlas-clouddeployment profiles.community-mongotremains accepted as a compatibility alias foratlas-local-preview. Workspace templates now state MongoDB is durable runtime memory.pnpm clawmongo:e2evalidates that MongoDB and Voyage credentials are present before attempting a real memory proof.Diagram (if applicable)
Security Impact (required)
Yes/No) NoYes/No) YesYes/No) YesYes/No) YesYes/No) YesYes, explain risk + mitigation: MongoDB URI/profile handling expands to Atlas cloud. Secrets remain config/env supplied; docs use placeholders; runtime validates missing URI/profile/search/Voyage capability instead of falling back to non-Mongo memory.Repro + Verification
Environment
Steps
pnpm test src/memory/backend-config.test.ts src/memory/mongodb-schema.test.ts src/agents/tools/memory-tool-mongodb.test.ts src/memory/mongodb-manager.test.ts src/gateway/server-startup-memory.test.ts src/plugins/memory-runtime.test.ts src/plugins/uninstall.test.ts test/scripts/test-live-shard.test.tspnpm test src/gateway/server.auth.control-ui.test.ts src/gateway/server.auth.compat-baseline.test.ts src/gateway/server.auth.modes.test.ts src/gateway/server.health.test.ts src/gateway/server.node-invoke-approval-bypass.test.ts src/gateway/server.config-patch.test.ts src/gateway/server.chat.gateway-server-chat.test.tspnpm test extensions/bonjour/src/advertiser.test.tspnpm test src/commands/configure-memory.test.ts src/commands/doctor-memory-recall-diagnostic.test.ts src/commands/onboard-hooks.test.ts src/commands/status.scan.shared.test.ts src/commands/doctor/shared/stale-plugin-config.test.ts extensions/memory-core/index.test.ts extensions/memory-lancedb/config.test.ts extensions/nvidia/onboard.test.ts extensions/lmstudio/src/setup.test.ts src/cli/gateway-cli/run.option-collisions.test.ts src/gateway/server.startup-websocket-race.test.tspnpm test test/web-provider-boundary.test.ts test/scripts/lint-suppressions.test.ts ui/src/ui/controllers/chat.test.ts extensions/active-memory/index.test.ts src/auto-reply/reply/reply-state.test.ts extensions/acpx/src/config.test.ts extensions/telegram/src/bot.create-telegram-bot.test.ts src/agents/pi-auth-json.test.ts src/agents/pi-settings.test.ts src/agents/pi-tools.workspace-only-false.test.ts src/agents/workspace.test.tspnpm checkpnpm buildpnpm clawmongo:e2epreflightpnpm testattempted as broad gate.Expected
Actual
pnpm clawmongo:e2ecorrectly failed fast because no MongoDB URI was configured in this cloud environment.pnpm teststill did not pass. The latest full-suite attempt is much closer but still has additional unit-fast/plugin metadata and generated-contract failures that need a continued full-suite stabilization pass before main.Evidence
Human Verification (required)
atlas-cloud;community-mongotaliases toatlas-local-preview; test-minimal gateway mode binds loopback in containerized test environments.pnpm testgreen.Review Conversations
Compatibility / Migration
Yes/No) PartiallyYes/No) YesYes/No) Yesmemory.mongodb.uri; setmemory.mongodb.deploymentProfiletoatlas-local-previeworatlas-cloud; keepmemory.mongodb.embeddingModeasautomated; provide Voyage credentials to MongoDB Search/mongot; optionally setmemory.mongodb.autoEmbedModelfor the target cluster.Risks and Mitigations
autoEmbedModelconfig, and E2E preflight requiring MongoDB plus Voyage credentials.