Repository navigation
fix: bug-PR merge train batch 3 (seven carried fixes) #5904
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
c96e711
1c4d779
5cedfcb
70d6ac6
4e77139
6522b6a
edecfbf
d8d18ea
0f120fd
41deb03
1543df1
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 |
|---|---|---|
|
|
@@ -174,3 +174,17 @@ the id `grok-4.5`. Generated aliases avoid dots entirely for this reason. | |
| document on every reload. | ||
| - **Catalog updates:** the fenced block reflects the catalog at injection time. After | ||
| adding providers or models, run `ocx ensure` (or restart the proxy) to refresh it. | ||
|
|
||
| ## Devin rate limits | ||
|
|
||
| If Devin rejects a direct request with a rate limit before producing output or performing | ||
| an action, OpenCodex returns HTTP 429 with a JSON error. Grok can then recognize the limit | ||
| instead of displaying a generic 500 error from a failed HTTP 200 stream. If the response | ||
| includes `Retry-After`, follow that delay before retrying. Exhausted quotas without a reset | ||
| hint do not receive a default delay. Existing OAuth account failover still applies. This | ||
| HTTP 429 handling also applies to non-streaming Responses requests. | ||
|
|
||
| For streaming requests, OpenCodex waits up to the configured stall timeout for this early | ||
| refusal. If that wait | ||
| expires, it opens the SSE response and lets the existing stream watchdog handle stalls. | ||
| After HTTP headers are sent, a later failure stays in the stream as `response.failed`. | ||
|
Comment on lines
+187
to
+190
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. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Document preflight when the stream watchdog is disabled. When
As per coding guidelines, “Document current shipped or intentionally pending behavior.” As per path instructions, “Check that user-facing docs stay in sync with actual CLI/API behavior.” 📍 Affects 3 files
🤖 Prompt for AI AgentsSources: Coding guidelines, Path instructions |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -498,6 +498,7 @@ region-pinned EU routes, is at [opper.ai/models](https://opper.ai/models). Opper | |
| | Umans AI · Neuralwatt | `https://api.code.umans.ai` · `https://api.neuralwatt.com/v1` | | ||
| | Mistral | `https://api.mistral.ai/v1` | | ||
| | MiniMax · MiniMax (CN) | `https://api.minimax.io/v1` · `https://api.minimaxi.com/v1` | | ||
|
|
||
|
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. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Remove blank lines that split the provider URL tables. A blank line terminates a Markdown table. These additions leave subsequent provider rows outside the table in both locales. (github.github.com)
As per coding guidelines, “Reuse the existing Astro and Starlight structure, navigation, components, and style.” 📍 Affects 2 files
🤖 Prompt for AI AgentsSource: Coding guidelines |
||
| | DeepSeek | `https://api.deepseek.com` | | ||
| | Cerebras | `https://api.cerebras.ai/v1` | | ||
| | Chutes | `https://llm.chutes.ai/v1` | | ||
|
|
@@ -538,6 +539,10 @@ region-pinned EU routes, is at [opper.ai/models](https://opper.ai/models). Opper | |
| | Cloudflare AI Gateway | `https://gateway.ai.cloudflare.com/v1/{account-id}/{gateway}/anthropic` | | ||
| | …and more | opencode zen, Vercel AI Gateway, Venice, NanoGPT, Synthetic, Qianfan, Alibaba, Parallel, ZenMux, LiteLLM | | ||
|
|
||
| The MiniMax and MiniMax (CN) provider cards can also show Coding Plan quota when the configured | ||
| key has an active plan. The dashboard reads the plan's 5-hour window and, when present, weekly | ||
| window; these are display observations and do not change model routing. | ||
|
|
||
| **OpenCode Go** requires a stable session identifier for routing. OpenCodex derives | ||
| its Go session header from Codex thread/session headers, or from a client's | ||
| `x-opencode-session` header when Codex headers are absent. This applies to direct | ||
|
|
||
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.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Translate the existing-profile migration steps.
The English guide explains that older managed profiles can retain date-shaped IDs and directs users to reapply the profile and fully quit and reopen Desktop after upgrading. The translated guides do not identify this migration case, so users may keep an old profile that can suppress model switches.
docs-site/src/content/docs/fr/guides/claude-code.md#L256-L258: Add the French instructions to reapply the profile and fully quit and reopen Desktop after upgrading.docs-site/src/content/docs/ja/guides/claude-code.md#L246-L246: Add the Japanese instructions to reapply the profile and fully quit and reopen Desktop after upgrading.docs-site/src/content/docs/ko/guides/claude-code.md#L281-L281: Add the Korean instructions to reapply the profile and fully quit and reopen Desktop after upgrading.docs-site/src/content/docs/ru/guides/claude-code.md#L259-L259: Add the Russian instructions to reapply the profile and fully quit and reopen Desktop after upgrading.docs-site/src/content/docs/tr/guides/claude-code.md#L251-L255: Add the Turkish instructions to reapply the profile and fully quit and reopen Desktop after upgrading.docs-site/src/content/docs/zh-cn/guides/claude-code.md#L238-L238: Add the Simplified Chinese instructions to reapply the profile and fully quit and reopen Desktop after upgrading.docs-site/src/content/docs/zh-tw/guides/claude-code.md#L187-L189: Add the Traditional Chinese instructions to reapply the profile and fully quit and reopen Desktop after upgrading.As per coding guidelines, “Update all directly affected pages when a user workflow changes.” The path instructions also require translated pages to stay aligned with English.
🧰 Tools
🪛 LanguageTool
[typographical] ~256-~256: Caractère d’apostrophe incorrect.
Context: ...ropic reçoivent des alias stables comme
claude-opus-4-8-p01q, avec un code de quatre caractères préf...(APOS_INCORRECT)
[typographical] ~256-~256: Caractère d’apostrophe incorrect.
Context: ...n code de quatre caractères préfixé par
p. OpenCodex conserve un emplacement synt...(APOS_INCORRECT)
[style] ~257-~257: Une phrase longue peut apporter une perte de sens pour le lecteur. Celle-ci peut donc être divisée afin d’apporter de la clarté et du rythme.
Context: ...ur stabiliser les affectations du profil, mais n'expose pas cette date comme identifia...
(POINT_MAIS)
📍 Affects 7 files
docs-site/src/content/docs/fr/guides/claude-code.md#L256-L258(this comment)docs-site/src/content/docs/ja/guides/claude-code.md#L246-L246docs-site/src/content/docs/ko/guides/claude-code.md#L281-L281docs-site/src/content/docs/ru/guides/claude-code.md#L259-L259docs-site/src/content/docs/tr/guides/claude-code.md#L251-L255docs-site/src/content/docs/zh-cn/guides/claude-code.md#L238-L238docs-site/src/content/docs/zh-tw/guides/claude-code.md#L187-L189🤖 Prompt for AI Agents
Sources: Coding guidelines, Path instructions