docs: fix drift in System Overview - #312
Draft
jack-arturo wants to merge 1 commit into
Draft
Conversation
- list all 11 api/ route blueprints (entity, backup, stream were missing) - _serialize_node/_summarize_relation_node live in automem/utils/graph.py - enrichment retry uses a flat backoff, not 5s/10s/15s exponential - correct ENRICHMENT_*/CONSOLIDATION_*/SEARCH_WEIGHT_* variable counts - correct .env load precedence (project .env is loaded first, so it wins) Verified against verygoodplugins/automem@42ba8b6 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Tnorev9PMn8gL7uTyZzege
Deploying automem-website with
|
| Latest commit: |
272449d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://0054864a.automem-website.pages.dev |
| Branch Preview URL: | https://docs-audit-architecture-over-c4u8.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.
Routine docs-accuracy audit. First review of this page — no prior review state existed for
architecture/overview.automem/api/blueprints as memory, recall, graph, admin, health, enrichment, consolidation, viewerentity.py,backup.py, andstream.pyblueprintsautomem/stores/graph_store.py… Key functions:_build_graph_tag_predicate,_serialize_node,_summarize_relation_node"_build_graph_tag_predicateis ingraph_store.py._serialize_nodeand_summarize_relation_nodeare defined inautomem/utils/graph.py; the latter's parameter isdata, notrelautomem/utils/graph.pyand correcting the parameter name5s, 10s, 15senrichment_failure_backoff_seconds(ENRICHMENT_FAILURE_BACKOFF_SECONDS, default5) before re-enqueueing — the value never scales with attempt countENRICHMENT_*(12 variables),CONSOLIDATION_*(15),SEARCH_WEIGHT_*(8)automem/+app.py)~/.config/automem/.env→ project.env→ defaultsconfig.pycallsload_dotenv()(project.env) beforeload_dotenv(~/.config/automem/.env), andload_dotenvdoes not override names already set — so the project file actually outranks the user-level oneVerified against: verygoodplugins/automem@42ba8b6
Checked and found accurate, left unchanged: every path in the "Key Initialization Functions" table; all ten scoring weights and their defaults; the
ServiceStateworker/lock/tracking-set field names; the 14 relationship types (11 authorable + 3 system); the admin-token table for/admin/reembed,/enrichment/reprocess,/consolidate,/consolidate/status; the five embedding provider modules; the embedding batch accumulator's 20-item / 2s defaults (EMBEDDING_BATCH_SIZE,EMBEDDING_BATCH_TIMEOUT_SECONDSinapp.py); themcp-sse-server/server.jsport 8080 node.Questions
/var/lib/falkordb/data. That matchesdocs/RAILWAY_DEPLOYMENT.mdin the automem repo (the diagram is Railway-scoped), butdocker-compose.ymluses/datafor local Docker. Left as-is since the diagram is explicitly a Railway topology — flagging because the same string was previously corrected on the Docker Deployment page and the two pages now disagree by design.SEARCH_WEIGHT_*variables.SEARCH_WEIGHT_TEMPORALonly applies during the optionalrecency_biasre-rank, so excluding it from the ten looks deliberate — not touched.Unverified
None — every code-anchored claim on this page resolved to current source.
Follow-ups
POST /admin/sync, theautomem/api/backup.pyroutes, and twoautomem/api/entity.pyroutes also gate onrequire_admin_token. The table does not claim to be exhaustive, so this was left alone rather than expanded.automem@4b5eaafandautomem@ed36b98. They still resolve, and only links actually touched by this PR were re-pinned; a wholesale refresh to42ba8b6would be a separate, larger diff.Generated by Claude Code