Skip to content

docs: clarify env file setup#321

Open
aqilaziz wants to merge 1 commit into
rohitg00:mainfrom
aqilaziz:docs/env-file-setup
Open

docs: clarify env file setup#321
aqilaziz wants to merge 1 commit into
rohitg00:mainfrom
aqilaziz:docs/env-file-setup

Conversation

@aqilaziz
Copy link
Copy Markdown

@aqilaziz aqilaziz commented May 13, 2026

Summary

  • clarify that agentmemory runtime config can live in ~/.agentmemory/.env
  • explain how to copy viewer export ... hints into the env file and note process env precedence
  • add the Windows path plus a minimal Claude Code subscription fallback example

Fixes #293

Tests

  • git diff --check

Summary by CodeRabbit

  • Documentation
    • Clarified how LLM providers are auto-detected and when Claude subscription fallback is available
    • Added "Config File" section documenting how to configure runtime settings via environment file (~/.agentmemory/.env or Windows equivalent)
    • Documented configuration options for enabling Claude subscription fallback and graph/consolidation features

Review Change Stack

Signed-off-by: aqilaziz <gonzes7@gmail.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 13, 2026

@aqilaziz is attempting to deploy a commit to the rohitg00's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2b0004c8-854c-4a33-a846-8bac87497a90

📥 Commits

Reviewing files that changed from the base of the PR and between 25dddc4 and f1c082c.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The README is updated to clarify that agentmemory disables LLM calls by default and adds a new Config File section documenting how to place runtime settings in ~/.agentmemory/.env (or the Windows equivalent), including environment variable precedence and example configurations for Claude subscription fallback and feature toggles.

Changes

Documentation – Configuration and LLM Provider Defaults

Layer / File(s) Summary
Config file and LLM provider clarification
README.md
LLM Providers intro is clarified to state LLM calls are disabled by default unless a provider is configured or AGENTMEMORY_ALLOW_AGENT_SDK is explicitly set. A new Config File subsection documents placing runtime settings in ~/.agentmemory/.env (or Windows %USERPROFILE%\.agentmemory\.env), explains environment variable precedence, and provides example entries for Claude subscription opt-in and enabling graph extraction and consolidation features.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • rohitg00/agentmemory#190: Both PRs update README.md's LLM provider defaults and explicitly document the opt-in behavior of Claude subscription fallback via AGENTMEMORY_ALLOW_AGENT_SDK=true instead of making it the default.

Poem

🐰 A config file so neat, so divine,
No more scattered env vars to align,
Just ~/.agentmemory/.env with care,
And features wake up without a flare!
Thanks for clarity, your docs declare—
Configuration's now a hoppy affair! 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs: clarify env file setup' is concise and directly related to the main change: clarifying environment file configuration in the README documentation.
Linked Issues check ✅ Passed The PR fully addresses the requirements in issue #293 by documenting the config file location (~/.agentmemory/.env), explaining how to place environment variables there, and providing Claude subscription fallback setup guidance.
Out of Scope Changes check ✅ Passed All changes are in-scope: the README updates directly address the linked issue's request for config file documentation and Claude subscription setup instructions without introducing unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Is there a config file to specify all environment variables at once?

1 participant