-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix(responses): sanitize Codex metadata and route vision by backend capability #4528
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
2668cb6
8a96f86
84f89dd
d1d8d45
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 |
|---|---|---|
|
|
@@ -404,3 +404,9 @@ Hata hedefe özgü olmaktan ziyade uç (terminal) bir hataydı. Geçersiz girdiy | |
| düzeltin, aşırı büyük bir bağlamı azaltın, bir politika reddini işleyin veya | ||
| reddedilen istek kaynağını düzeltin. Kombolar bu durumlar için atlama yapmaz. | ||
|
|
||
|
|
||
| ## İsteğe bağlı parametre uyumluluğu | ||
|
|
||
| Sonlandırıcı 400 hatalarının dar bir istisnası vardır: `user` alanını açıkça reddeden, `reasoning.effort`/`reasoning_effort` için desteklenmeyen değer bildiren veya modele özgü görüntü girdisini reddeden (`param: input`) yapılandırılmış hata, çıktı başlamadan sonraki uygun hedefe geçebilir. Bu uyumsuzluk için bekleme süresi kaydedilmez. Güvenlik politikası reddi, iptal ve başlamış çıktı yeniden yürütülmez. | ||
|
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 Clarify the Turkish pre-output boundary. At 🤖 Prompt for AI Agents |
||
|
|
||
| [Canonical compatibility details](/guides/combos/#request-local-target-compatibility). | ||
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.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Remove unsupported image-input failover claims from all documentation.
src/combos/failover.ts:isOptionalControlIncompatibility()recognizes onlyUnsupported parameter: userand unsupported reasoning-effort values.comboFailureDecision()calls this helper for hop decisions, then stopsinvalid_request_errorfailures atsrc/combos/failover.ts:508-510. Image-input rejections therefore do not receive the documented request-local hop.Remove the image-input case from all three claims:
docs-site/src/content/docs/guides/combos.md:217docs-site/src/content/docs/guides/combos.md:262structure/runtime.md:358Also remove the image-specific null-provider-code exception from
structure/runtime.md:358.Proposed correction
📝 Committable suggestion
🤖 Prompt for AI Agents