-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix(codex): keep native eligibility metadata off routed catalog rows #2862
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -105,6 +105,38 @@ them by ignoring `visibility`. See [Codex Desktop native-allowlist compatibility | |
| for the command, disable-key semantics, and safety constraints. | ||
|
|
||
| ## Integration path | ||
| ## Native quota fallback limitation | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Preserve heading boundaries and section order in the translated guides. Add a blank line before each newly added H2 so the documentation follows the expected heading spacing. In the zh-TW guide, also move the quota-fallback section so it remains in the intended section order. 📍 Affects 2 files
🤖 Prompt for AI AgentsSource: Linters/SAST tools |
||
|
|
||
| When the Codex app exhausts its native five-hour quota it can switch to a reserve | ||
| fallback model and grey out the other rows in its picker. Reported in | ||
| [#2813](https://github.com/lidge-jun/opencodex/issues/2813), that gating also hides routed | ||
| opencodex rows, even though those use unrelated provider credentials and consume none of the | ||
| ChatGPT quota. | ||
|
|
||
| This gate is applied by the client before a request reaches the proxy, so opencodex cannot lift | ||
| it. Routed rows are written with `visibility: "list"`, catalog filtering consults only | ||
| `disabledModels` and each provider's `selectedModels`, and no quota value takes part in routed | ||
| visibility. | ||
|
|
||
| Selecting a routed model explicitly does not go through the picker. Set the model in | ||
| `config.toml`: | ||
|
|
||
| ```toml | ||
| model = "anthropic/claude-sonnet-5" | ||
| ``` | ||
|
|
||
| or send it directly: | ||
|
|
||
| ```bash | ||
| ocx access test anthropic/claude-sonnet-5 --protocol responses | ||
| ``` | ||
|
|
||
| Both paths route correctly **once the request reaches the proxy** — that part is covered by | ||
| tests. What is not established is whether the app still sends the configured model while reserve | ||
| mode is active; if the client rewrites or refuses it before the request leaves, no proxy-side | ||
| setting changes that. Treat the explicit-selection route as worth trying rather than a confirmed | ||
| workaround. | ||
|
|
||
|
|
||
| `ocx init`, `ocx start`, and `ocx sync` wire the shared Codex config and catalog into the proxy; see | ||
| [Codex Integration](/guides/codex-integration/) for config injection, catalog sync, shims, WebSocket | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new H2 immediately follows
## Integration path, leaving that section empty and causing its existing content at lines 141–143 to render under “Native quota fallback limitation.” The same placement error appears in all seven translated pages, where the existing model-refresh sections become empty and their instructions are attached to the quota section. Move this block before the existing heading or after that section’s body so the page outline and navigation remain accurate.AGENTS.md reference: docs-site/AGENTS.md:L15-L16
Useful? React with 👍 / 👎.