fix(sync): refresh catalog for side profiles when Codex injection is OFF - #1931
fix(sync): refresh catalog for side profiles when Codex injection is OFF#1931harryzhou2000 wants to merge 1 commit into
Conversation
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
Review readiness checklist
0/4 boxes ticked. This pull request was already a draft. Its draft status will be preserved after every issue above is resolved. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Explicit ocx sync (and ocx sync-cache) now refresh the catalog and models cache even when the Codex integration toggle is OFF or an external model_provider owns config.toml.
Rationale: users may consume the OpenCodex catalog through a custom side profile without injection. The refresh is harmless in those modes because config, journal, and history are never touched (new catalog-only sync status). POST /api/sync, startup, and ensure keep the existing OFF skip.
Tests: unit tests for the catalog-only and external-provider paths, models-cache override test, and updated composed-acceptance coverage.