docs: document cloud review and MCP app - #38
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2524885c66
ℹ️ 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".
|
|
||
| Your assistant calls `open_basic_memory` and opens the requested view. The default workspace and project resolve automatically, so you do not need to list or choose a project first unless you want a different location. | ||
|
|
||
| ::theme-image{light="/screenshots/mcp-app/mcp-app-open-light.png" dark="/screenshots/mcp-app/mcp-app-open-dark.png" alt="The Basic Memory MCP App open to the Notes gallery inside an AI conversation"} |
There was a problem hiding this comment.
Redact internal operations from the public screenshot
Both referenced mcp-app-open-*.png captures include internal engineering activity beneath the app, including the missing POLAR_RETENTION_DISCOUNT_ID production configuration, retention-discount provisioning, retry/stall investigations, and an invite bug. Publishing /cloud/mcp-app therefore discloses those operational details publicly; replace, crop, or redact these captures before using them in the documentation.
Useful? React with 👍 / 👎.
| | Parameter | Type | Required | Notes | | ||
| |---|---|---|---| | ||
| | `view` | `notes`, `note`, `folders`, `search`, or `graph` | No | Default `notes`. Destination screen | | ||
| | `workspace` | string | No | Workspace slug, name, or tenant ID. Omit for the default | |
There was a problem hiding this comment.
Document open_basic_memory as a workspace exception
Adding this workspace selector contradicts the common-parameter guidance at line 26, which says that workspace appears only on note-writing and cloud-project create/delete tools. A reader relying on that guidance can conclude that a non-default workspace cannot be selected here despite this table supporting it, so include open_basic_memory in the documented exceptions.
Useful? React with 👍 / 👎.
No description provided.