docs: fix drift in Direct API vs MCP Tools#175
Merged
Conversation
- Update source note SHAs b81c63ae → 34fcfe2b (both src/index.ts and server.json links were pointing to a stale commit) - Fix update_memory metadata param description: "merged with existing" → "replaces existing" (PATCH /memory/:id uses replacement semantics, confirmed in PR #169; MCP tool passes through to the same endpoint) - Fix tools array line reference: 178-620 → 177-667 - Fix ListToolsRequestSchema handler line reference: 622-624 → 669 - Fix CallToolRequestSchema handler line reference: 626-783 → 671-826 Follow-ups (not fixed here — exceed the 5-fix cap): - Process Lifecycle table line ranges need full update at 34fcfe2b: Entry 1-57→1-52, Mode Detection 58-62→53-68, CLI Routing 63-118→88-157, Configuration 119-177→158-176, Server Setup 178-783→177-826, Main Loop 785-795→~828-834 - installStdioErrorGuards() "lines 120-130" is now at L106 Verified against: mcp-automem@34fcfe2b7bdac6a99829c64cc74611e29af69a38
Deploying automem-website with
|
| Latest commit: |
b06ebb3
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://efa1eea4.automem-website.pages.dev |
| Branch Preview URL: | https://docs-audit-direct-vs-mcp-202.automem-website.pages.dev |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the “Direct API vs MCP Tools” documentation to eliminate drift against the current mcp-automem implementation, keeping GitHub permalinks and referenced source line ranges accurate for readers.
Changes:
- Updated
mcp-automemsource permalinks (src/index.ts,server.json) to commit34fcfe2b…. - Corrected
update_memorymetadata semantics wording to indicate replacement (not merge) behavior. - Refreshed referenced line ranges for the
toolsarray and request-schema handlers to match the pinned commit.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | `tags` | array[string] | No | — | New tags (replaces existing) | | ||
| | `importance` | number | No | 0–1 | New importance score | | ||
| | `metadata` | object | No | — | Metadata (merged with existing) | | ||
| | `metadata` | object | No | — | Metadata (replaces existing) | |
Member
Author
There was a problem hiding this comment.
Good catch. memory-operations.md line 533 has the same "merged with existing" wording and is already queued for correction in the open draft PR #169 (which fixes several issues on that page, including this one). Once #169 merges, both pages will agree on replacement semantics. No change needed here.
Generated by Claude Code
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
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.
Automated docs audit — 2026-06-12. Never-previously-reviewed page.
b81c63ae(bothsrc/index.tsandserver.jsonlinks)34fcfe2b34fcfe2bupdate_memorymetadata param: "Metadata (merged with existing)"/memory/:iduses replacement semantics — confirmed in PR #169; MCP tool passes through to the same endpointtoolsarray reference: "lines 178–620"34fcfe2b, tools array is lines 177–667ListToolsRequestSchemahandler: "lines 622–624"34fcfe2b, handler is at line 669CallToolRequestSchemahandler: "lines 626–783"34fcfe2b, handler is lines 671–826Follow-ups (not fixed — exceed 5-fix cap):
34fcfe2b: Entry 1–57→1–52, Mode Detection 58–62→53–68, CLI Routing 63–118→88–157, Configuration 119–177→158–176, Server Setup 178–783→177–826, Main Loop 785–795→~828–834installStdioErrorGuards()comment "lines 120–130" is now at L106Verified against:
mcp-automem@34fcfe2b7bdac6a99829c64cc74611e29af69a38Generated by Claude Code