Skip to content

[codex] Fix product guide storage paths#238

Merged
0xmrpeter merged 1 commit intoOpen-ACP:mainfrom
cloudyun888:codex/fix-product-guide-storage-paths
Apr 25, 2026
Merged

[codex] Fix product guide storage paths#238
0xmrpeter merged 1 commit intoOpen-ACP:mainfrom
cloudyun888:codex/fix-product-guide-storage-paths

Conversation

@cloudyun888
Copy link
Copy Markdown
Contributor

Summary

  • Update the assistant product guide to describe per-instance storage under <workspace>/.openacp/.
  • Keep shared global paths only for shared downloads/caches such as ~/.openacp/agents/ and ~/.openacp/cache/registry-cache.json.
  • Correct the config, agent store, log, and data-storage path references in the guide.

Why

The product guide is injected into the Assistant prompt, so stale paths can make the Assistant give users outdated configuration and storage advice. The current instance model stores config, sessions, logs, plugins, and instance agent definitions in the instance root, while only shared downloads/caches live under ~/.openacp/.

Validation

  • node --check src/data/product-guide.ts
  • git diff --check
  • rg -n "Config file:|Installed agent definitions|logging\\.logDir|Session-specific logs|Data & Storage|OpenACP stores per-instance|~/.openacp/config\\.json|~/.openacp/logs|~/.openacp/agents\\.json|All data is stored in" src/data/product-guide.ts
  • Cross-checked against src/core/instance/instance-context.ts, src/core/instance/__tests__/instance-context.test.ts, and the existing GitBook storage/config docs.

Note: I did not run the full test suite because this checkout does not have node_modules installed; this change is limited to the static product-guide string and was syntax-checked with Node.

@0xmrpeter 0xmrpeter marked this pull request as ready for review April 25, 2026 13:44
@0xmrpeter 0xmrpeter merged commit 3e641c2 into Open-ACP:main Apr 25, 2026
4 checks passed
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