Skip to content

fix(codex): support upstream marketplace install and robust recall#2879

Merged
t0saki merged 1 commit into
mainfrom
codex/openviking-codex-marketplace-upstream
Jun 30, 2026
Merged

fix(codex): support upstream marketplace install and robust recall#2879
t0saki merged 1 commit into
mainfrom
codex/openviking-codex-marketplace-upstream

Conversation

@LinQiang391

Copy link
Copy Markdown
Collaborator

Summary

  • add the Codex marketplace catalog so users can install OpenViking Memory from volcengine/OpenViking
  • use Codex's local marketplace source object shape so the desktop plugin UI renders the plugin
  • make Codex auto-recall robust across explicit user paths, shorthand user paths, and session-aware search planner no-result cases

Verification

  • node --test examples/codex-memory-plugin/scripts/marketplace.test.mjs examples/codex-memory-plugin/scripts/auto-recall.test.mjs examples/codex-memory-plugin/scripts/ov-credentials.test.mjs
  • verified codex plugin marketplace add volcengine/OpenViking --ref codex/openviking-codex-marketplace-upstream in a fresh temporary CODEX_HOME
  • verified current Codex config can use https://github.com/volcengine/OpenViking.git#codex/openviking-codex-marketplace-upstream

@LinQiang391 LinQiang391 changed the title fix(codex): support upstream marketplace install and robust recall fix(codex): [WIP]support upstream marketplace install and robust recall Jun 28, 2026
@LinQiang391 LinQiang391 force-pushed the codex/openviking-codex-marketplace-upstream branch 2 times, most recently from 62904ca to 76e809d Compare June 28, 2026 17:44
@LinQiang391 LinQiang391 changed the title fix(codex): [WIP]support upstream marketplace install and robust recall fix(codex): support upstream marketplace install and robust recall Jun 29, 2026
@LinQiang391 LinQiang391 force-pushed the codex/openviking-codex-marketplace-upstream branch from 76e809d to bf45d3d Compare June 29, 2026 12:11
@LinQiang391 LinQiang391 force-pushed the codex/openviking-codex-marketplace-upstream branch from bf45d3d to 8c9e581 Compare June 29, 2026 15:30
@Mijamind719 Mijamind719 self-assigned this Jun 30, 2026
@t0saki t0saki merged commit 8708deb into main Jun 30, 2026
5 checks passed
@t0saki t0saki deleted the codex/openviking-codex-marketplace-upstream branch June 30, 2026 03:22
@github-project-automation github-project-automation Bot moved this from Backlog to Done in OpenViking project Jun 30, 2026
t0saki added a commit to t0saki/OpenViking that referenced this pull request Jun 30, 2026
…tall local-only

volcengine#2879 added a Codex marketplace section to the English integration docs. Revert
those docs-only additions so docs/ matches the pre-volcengine#2879 state — no marketplace
content in the agent-integration pages.

Keep the marketplace path in the plugin README, but reframe it: it is local-only
(unauthenticated http://127.0.0.1:1933), does not support authenticated or
remote/cloud servers, and is not recommended. The one-line installer becomes the
recommended path for everyone, including remote/cloud.
t0saki added a commit to t0saki/OpenViking that referenced this pull request Jun 30, 2026
…o local-only

volcengine#2879 added a Codex marketplace section to the English integration docs. Revert
those docs-only additions so docs/ matches the pre-volcengine#2879 state — no marketplace
content in the agent-integration pages.

Keep the marketplace path in the plugin README, but demote it: it is local-only
(unauthenticated http://127.0.0.1:1933), does not support authenticated or
remote/cloud servers, and is not recommended. Reorder the Quick Start so the
one-line installer is path A (recommended, supports remote/cloud) and the
marketplace install is path B (local-only, not recommended).
ZaynJarvis pushed a commit that referenced this pull request Jun 30, 2026
…ly (#2901)

#2879 added a Codex marketplace section to the English integration docs. Revert
those docs-only additions so docs/ matches the pre-#2879 state — no marketplace
content in the agent-integration pages.

Keep the marketplace path in the plugin README, but demote it: it is local-only
(unauthenticated http://127.0.0.1:1933), does not support authenticated or
remote/cloud servers, and is not recommended. Reorder the Quick Start so the
one-line installer is path A (recommended, supports remote/cloud) and the
marketplace install is path B (local-only, not recommended).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants