docs: fix drift in Memory Rules & Patterns - #302
Draft
jack-arturo wants to merge 1 commit into
Draft
Conversation
Corrects five claims that no longer match mcp-automem@2816beb:
- src/index.ts expand-parameter anchor 764-811 -> permalinked L784-L831
- removed fabricated {{CURRENT_MONTH}} template variable
- tagging guidance recommended banned platform + YYYY-MM date tags
- Preferences importance 0.6-0.8 -> 0.9 (contradicted the page's own table)
- Cursor "3-phase lifecycle" -> two-phase recall + three storage triggers
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019Y3LbXh9Nn2xMybf9qx6ky
Deploying automem-website with
|
| Latest commit: |
55b9cf0
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fe7f674b.automem-website.pages.dev |
| Branch Preview URL: | https://docs-audit-best-practices-me-fi45.automem-website.pages.dev |
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.
First review of
best-practices/memory-rules— no prior review-state memory existed for this page. Automated docs-accuracy audit.src/index.ts:764-811"expand_min_importance(L818) andexpand_min_strength(L825) — two of the six the list documents{{CURRENT_MONTH}}listed as a common template variable resolving to2026-02YYYY-MMmonth tag — in the mermaid diagram, both JSON examples, an "Always include the current month as a tag" instruction, and the Best Practices summarylang/prefixes, platform tags, or date-stamped tags" and "NO platform tag, NO [YYYY-MM]"; date-scoped facts uset_valid/t_invalidinsteadtime_query+ validity-window guidance, added a bare-tags caution, fixed the summary bullet0.6–0.80.9, which falls outside the documented range. The page also contradicted itself — the Memory Type Taxonomy table above lists Preference at0.75–0.900.9and corrected the rationaleautomem.mdc.templateis built around two-phase session-start recall plus exactly three mid-conversation storage triggers, and its Storage Discipline section forbids storing session summaries — so the "conversation end" phase contradicts the templateVerified against: mcp-automem@2816beb7bb81d87bb8d592e8712fd8b4a7cc4f95
Questions
project-x/auth/jwt) and thetag_match: "prefix"example sit awkwardly against the policy line "No namespace prefixes (project/*,lang/*)" — but the same policy file elsewhere recommends "precise, namespace-style tags (e.g.slack/channel-ops)". The source is genuinely ambiguous here, so I left both sections alone. Worth settling upstream inshared.tsand then reflecting here.Unverified
Follow-ups
Beyond the 5-fix cap:
cursor/andcodex/.templates/now also containsantigravity/,claude-code/,copilot/,hermes/,openclaw/,COPILOT_INSTRUCTIONS_MEMORY_RULES.md,claude_desktop_config.json, andcursor_mcp.json;cursor/holds three files (automem.mdc.template,rule-evals.md,user-rules.md), not one. Nothing listed is wrong, so this is a completeness gap rather than an error.tag_match: "prefix"framed as opt-in. "Settag_match: \"prefix\"to support hierarchical tag queries" reads as though prefix matching must be enabled; the tool schema documents prefix as the default. Minor wording.0.85–0.95vs0.85-0.9; Pattern0.70–0.85vs0.8; Insight0.75–0.85vs bug-fix0.75). Each documented range contains the policy value, so none is provably wrong — but the two tables on this page could be reconciled to the policy's concrete numbers.Generated by Claude Code