Conversation
Exposes MEM0_LLM_BASE_URL, MEM0_LLM_API_KEY, MEM0_EMBED_BASE_URL, MEM0_EMBED_API_KEY, MEM0_GRAPH_LLM_BASE_URL, MEM0_GRAPH_LLM_API_KEY, and MEM0_GRAPH_EXTRACTION_STRATEGY across all 4 compose files, .env.example, and README. All vars are optional and fall back to OPENAI_BASE_URL / OPENAI_API_KEY. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 992392c002
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| from pathlib import Path | ||
| p=Path("$COMPOSE_FILE") | ||
| s=p.read_text() | ||
| s=s.replace('NEO4J_AUTH: neo4j/${NEO4J_PASSWORD}','NEO4J_AUTH: none') |
There was a problem hiding this comment.
The auth toggle targets NEO4J_AUTH: neo4j/${NEO4J_PASSWORD} here, but the default compose file uses NEO4J_AUTH: ${NEO4J_AUTH:-neo4j/changeme} (compose.graph.yml), so this replacement is a no-op in the shipped topology. That leaves auth enabled and makes the subsequent unauthenticated cypher-shell ALTER USER ... step fail, so neo4j-auth-rotate.sh cannot complete successfully out of the box.
Useful? React with 👍 / 👎.
| if grep -q '^NEO4J_PASSWORD=' "$ENV_FILE"; then | ||
| sed -i.bak "s#^NEO4J_PASSWORD=.*#NEO4J_PASSWORD=${NEW_PASS}#" "$ENV_FILE" |
There was a problem hiding this comment.
This script rewrites NEO4J_PASSWORD in .env before attempting any restart or password change in Neo4j. If a later step fails (container name mismatch, startup timing, cypher error), the file now contains a password that was never applied, causing auth drift and breaking subsequent preflight/restart flows until someone manually repairs .env.
Useful? React with 👍 / 👎.
1 participant
Summary
.env.example, and READMENew variables (all optional, all fall back to shared
OPENAI_*)MEM0_LLM_BASE_URL/MEM0_LLM_API_KEYMEM0_EMBED_BASE_URL/MEM0_EMBED_API_KEYMEM0_GRAPH_LLM_BASE_URL/MEM0_GRAPH_LLM_API_KEYMEM0_GRAPH_EXTRACTION_STRATEGYtool_calling(default) orjson_promptingFiles changed
.env.example— documented with fallback commentscompose.one.yml/compose.split.yml/compose.external.yml/compose.graph.yml— env vars added to store serviceREADME.md— new "Per-purpose inference" tableTest plan
docker compose -f compose.one.yml configvalidates without errorsOPENAI_*)🤖 Generated with Claude Code