Skip to content

docs: fix drift in OpenClaw - #299

Draft
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-platforms-openclaw-20260812
Draft

docs: fix drift in OpenClaw#299
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-platforms-openclaw-20260812

Conversation

@jack-arturo

Copy link
Copy Markdown
Member

Automated docs-accuracy audit of platforms/openclaw. Every fix below is anchored to current source in verygoodplugins/mcp-automem.

Claim Current state Fix Evidence
Default tags are ["platform/openclaw", "project/my-project"], and "Tags use namespace-style formatting" (platform/openclaw auto-added, project/<name> derived) buildDefaultProjectTags() returns a single bare slug — lowercased, non-alphanumeric runs collapsed to -, truncated to 64 chars. No platform tag, no project/ prefix. Ambiguous slugs (api, app, test, video) return []. Both shipped skills say verbatim: "Use bare tags only, and avoid platform tags like openclaw." JSON example now ["my-project"]; Tags section rewritten to describe the real derivation + ambiguous-slug case shared.ts#L100-L107, openclaw.ts#L281-L283, skill-mcp/SKILL.md#L32
MCP/skill config example nests the token as env.AUTOMEM_API_KEY; prose said the installer "configures skills.entries.automem.env with endpoint and API key" buildSkillConfigEntry() writes the key as a top-level apiKey on the skill entry. env receives AUTOMEM_API_URL and, when a project tag resolves, AUTOMEM_DEFAULT_TAGS. Example moved apiKey to top level and added AUTOMEM_DEFAULT_TAGS; prose corrected openclaw.ts#L528-L551
Legacy-skill store example uses "tags": ["openclaw"] The shipped legacy skill's own example uses ["project-slug", "decision"], and its rules forbid exactly this platform tag Example tags corrected skill-legacy/SKILL.md#L30, #L63
--plugin-source default is @verygoodplugins/mcp-automem resolvePluginSource() prefers the bundled dist/openclaw-plugin-package directory when it exists, then this package's own name, and only then the npm spec Default column describes the real three-step resolution openclaw.ts#L254-L259, #L47-L50
--endpoint default http://127.0.0.1:8001; --api-key default "None" resolveEndpoint() falls back through AUTOMEM_API_URLAUTOMEM_ENDPOINT → the literal; resolveApiKey() reads AUTOMEM_API_KEYAUTOMEM_API_TOKEN Both Default cells now list the env fallbacks openclaw.ts#L1069-L1080, env.ts#L12-L24

Verified against: verygoodplugins/mcp-automem@2816beb

Questions

  • The CLI table describes --scope <workspace\|shared> as "Install scope for mcp/skill modes", but scope also changes plugin mode: requiresWorkspace = mode !== 'plugin' || scope === 'workspace', so --mode plugin --scope shared drops the workspace requirement entirely (openclaw.ts#L1088). Left as-is in case the mcp/skill framing is a deliberate simplification — worth a maintainer call.

Unverified

  • The "Quick verification commands" table (openclaw plugins list, mcporter list, openclaw skills info automem) and the memory-core / memory-lancedb claims describe OpenClaw's own CLI and plugins, which are outside this repo set. Left untouched.
  • Config-field ranges, defaults, exposure values, tool names, slash-command behavior, and workspace-detection order were all checked and are correct as written.

Follow-ups

  • The legacy recall example uses ?query=...&limit=5, while the shipped legacy skill's own example uses limit=20&format=detailed. Not clearly wrong (it is illustrative), so left alone.
  • This page carries no source-note permalinks at all, unlike the other platform pages. Adding a source-note block would make future audits anchorable.

Generated by Claude Code

Verified against verygoodplugins/mcp-automem@2816beb.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DGd3AJwGfwXzgH1ip6Jrut
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying automem-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8c713a9
Status: ✅  Deploy successful!
Preview URL: https://a9857803.automem-website.pages.dev
Branch Preview URL: https://docs-audit-platforms-opencla-ycnh.automem-website.pages.dev

View logs

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.

2 participants