fix(bigmodel): restore Responses preset quota visibility - #4210
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks 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 |
|
✅ Deterministic PR hygiene checks passed. |
|
@lidge-jun Exact-head Cross-platform CI 34491624277 is now successful at b9109a1. I checked the actual macOS shard-2 log: both domestic preset cases and the custom-host/PAYG no-dispatch controls passed. Linux/macOS shards, typecheck/privacy, Docker, keyring and npm-global smoke also passed. Full Windows shards and macOS control were skipped by the workflow, not run. This is only the confirmed BigModel Responses quota-reader omission from #4201. No model roster, Flash entitlement, endpoint expansion, or live account/config change is included. The PR stays Draft for the remaining docs build and independent review; no merge was performed. |
리뷰 · 우선순위 68 / 80이 PR은 열려 있는 이슈 #4201의 쿼터 리더 누락만 고칩니다. 지금 국제 엔드포인트 쪽 #4174와는 독립입니다. 이 패치는 국내 src/providers/quota.ts 약 2909라인 근처 - 지금 HEAD는 zhipu-bigmodel-responses가 빠져 있어 Responses 프리셋이면 providerApiKeyQuotaMode가 unsupported처럼 보입니다. PR이 이름을 배열에 넣는 방향은 맞습니다. 메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
|
Closing this duplicate in favor of #4231, which is already on dev as df65a25. I checked the landed name admission, unchanged canonical destination/redirect guard, and dedicated positive/negative quota tests against this PR's scope. No second quota implementation or rebase is needed. #4201 remains open for the separate Flash/catalog evidence. The public providers guide still omits the Responses preset from its quota-name list at babb764; #4231's lane notes assign that documentation followup separately, so closing this PR does not claim that prose has already shipped. Thanks for retaining the attribution. No merge or local runtime change was performed by me. |
Summary
Refs #4201 — quota-reader omission only; the Flash model-support request remains open.
zhipu-bigmodel-responsespreset to the existing canonical Z.ai/BigModel quota reader.The installed-source report is confirmed in dev at
6d3ad12e3fbfd8423f1515a467db4d3349732886: the URL is already admitted, but the provider-name selector omits this preset. This is independent of #4174's international endpoint fix.Verification
git diff --check: passed.scripts/privacy-scan.ts: passed using fresh HOME, OPENCODEX_HOME, CODEX_HOME and transpiler cache.b9109a151aef864cdfe4d22bdac6623164a4ca0a. Linux/macOS shards, typecheck/privacy, Docker, keyring and npm-global smoke passed. Full Windows shards and macOS control were skipped, not executed.Review must preserve the exact destination guard and domestic authorization format. Do not broaden to custom gateways or infer Flash support from the Chat preset.
Checklist
Review readiness