docs(cli): fix setup output claims - #353
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b67e65a6c5
ℹ️ 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".
| **For Claude Code:** environment export lines for the current endpoint plus a placeholder API key (`your-auto-mem-api-key`), not your real secret. | ||
|
|
||
| **For Hermes:** YAML examples for the generated `mcp_servers` or `memory.provider` setup. | ||
| **For Hermes:** an `mcp_servers` YAML snippet for `~/.hermes/config.yaml`. For native provider setup (`memory.provider`), run `npx @verygoodplugins/mcp-automem hermes`. |
There was a problem hiding this comment.
Pass provider mode to the Hermes installer
When a reader follows this command specifically to configure memory.provider, the bare hermes invocation uses the documented MCP-only default (src/content/docs/docs/cli/platform-installers.md:217), so it installs mcp_servers.automem instead of the native provider. Both new provider-mode references should show npx @verygoodplugins/mcp-automem hermes --mode provider (or explicitly mention --mode both as the alternative).
Useful? React with 👍 / 👎.
Deploying automem-website with
|
| Latest commit: |
b67e65a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4233cc7a.automem-website.pages.dev |
| Branch Preview URL: | https://docs-mcp-automem-cli-setup-o.automem-website.pages.dev |
Summary
setup/configdocs to say Claude Code output uses the endpoint plus a placeholder API keymcp_serversYAML snippet and direct native provider setup to thehermescommandVerification
/Users/jgarturo/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin/node --test tests/cli-setup-docs.test.mjs/Users/jgarturo/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin/node --test tests/*.test.mjsPATH="/Users/jgarturo/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin:/Users/jgarturo/Projects/OpenAI/automem-website/node_modules/.bin:$PATH" /Users/jgarturo/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin/node scripts/build-pages.mjsgit diff --checkCLEAR
Follow-up to #352