docs: fix drift in OpenClaw - #299
Draft
jack-arturo wants to merge 1 commit into
Draft
Conversation
Verified against verygoodplugins/mcp-automem@2816beb. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DGd3AJwGfwXzgH1ip6Jrut
Deploying automem-website with
|
| 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated docs-accuracy audit of
platforms/openclaw. Every fix below is anchored to current source inverygoodplugins/mcp-automem.["platform/openclaw", "project/my-project"], and "Tags use namespace-style formatting" (platform/openclawauto-added,project/<name>derived)buildDefaultProjectTags()returns a single bare slug — lowercased, non-alphanumeric runs collapsed to-, truncated to 64 chars. No platform tag, noproject/prefix. Ambiguous slugs (api,app,test,video) return[]. Both shipped skills say verbatim: "Use bare tags only, and avoid platform tags likeopenclaw."["my-project"]; Tags section rewritten to describe the real derivation + ambiguous-slug caseenv.AUTOMEM_API_KEY; prose said the installer "configuresskills.entries.automem.envwith endpoint and API key"buildSkillConfigEntry()writes the key as a top-levelapiKeyon the skill entry.envreceivesAUTOMEM_API_URLand, when a project tag resolves,AUTOMEM_DEFAULT_TAGS.apiKeyto top level and addedAUTOMEM_DEFAULT_TAGS; prose corrected"tags": ["openclaw"]["project-slug", "decision"], and its rules forbid exactly this platform tag--plugin-sourcedefault is@verygoodplugins/mcp-automemresolvePluginSource()prefers the bundleddist/openclaw-plugin-packagedirectory when it exists, then this package's ownname, and only then the npm spec--endpointdefaulthttp://127.0.0.1:8001;--api-keydefault "None"resolveEndpoint()falls back throughAUTOMEM_API_URL→AUTOMEM_ENDPOINT→ the literal;resolveApiKey()readsAUTOMEM_API_KEY→AUTOMEM_API_TOKENVerified against: verygoodplugins/mcp-automem@2816beb
Questions
--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 shareddrops 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
openclaw plugins list,mcporter list,openclaw skills info automem) and thememory-core/memory-lancedbclaims describe OpenClaw's own CLI and plugins, which are outside this repo set. Left untouched.exposurevalues, tool names, slash-command behavior, and workspace-detection order were all checked and are correct as written.Follow-ups
?query=...&limit=5, while the shipped legacy skill's own example useslimit=20&format=detailed. Not clearly wrong (it is illustrative), so left alone.Generated by Claude Code