fix(providers): restore glm-5.3 to both Alibaba Token Plan catalogs - #5010
Conversation
The 260909 refresh removed glm-5.3 as a phantom (the gateway 404'd the id then). It is real now: /models lists it for global Team, global Personal, and CN Team from 260917, and it answers completions on a Personal key (probed 260918). Contract on the plan gateway: effort low/high/max (default max), thinking always-on (enable_thinking:false rejected with 400), 1M context, 131,072 max output, text-only input, strict json_schema accepted. glm-5.3-flash stays out: still never served by the plan. Evidence: https://github.com/oliver-mee/alibaba-token-plan-wiki (machine-readable catalogue, data/models.json, updated 2026-09-18).
|
✅ Deterministic PR hygiene checks passed. |
✅ READY
Review readiness checklist
✅ 4/4 boxes ticked. This pull request is already Ready for Review. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (4)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughGLM-5.3 is restored to the Beijing and International Alibaba Token Plan registries with its model metadata. Provider roster and parity tests now expect GLM-5.3 and continue to exclude GLM-5.3-flash. ChangesAlibaba GLM-5.3 restoration
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The restored model catalog matches the intended provider metadata and roster behavior, so no merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
리뷰 · 우선순위 74 / 80이 PR은 Alibaba Token Plan 두 카탈로그에서 빠진 작성자 설명은 이렇게 이어진다. 260909 삭제는 그때 기준으로 맞았다(모든 plan 키에서 404). 그런데 260917부터 게이트웨이 코드 변경은 네 파일로 좁다. 현재 스냅샷 방향(프로바이더/카탈로그 충실도, 릴리스 규율)과도 잘 맞는다. types.ts/config.ts 분할 캠페인에 걸려 무효화될 종류의 패치가 아니다. 닫고 리베이스 말라는 케이스가 아니다. 다만 PR은 아직 draft이고 readiness 체크리스트가 비어 있으며, 브랜치 부모는 한 가지 계약 구멍도 있다. PR 본문은 plan 게이트웨이에서 effort 기본값이 max이고 thinking이 always-on( 라인 825/874 (entries-extended.ts 메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
Summary
glm-5.3returns to both Alibaba Token Plan catalogs (alibaba-token-planBeijing andalibaba-token-plan-intlSingapore). The 260909 refresh removed it as a phantom, which was correct at the time: the id 404'd on every plan key. The gateway started serving it on 260917:/modelsnow listsglm-5.3for global Team, global Personal, and CN Team, and it answers completions on a Personal key (probed 260918).low|high|max(reuses the existingZAI_GLM_53_REASONING_EFFORTSconst), thinking always-on (the gateway rejectsenable_thinking:falsewith 400restricted to True, unlike glm-5.2), 1M context, 131,072 max output, text-only input, strictjson_schemaaccepted so nonoJsonSchemaModelswiring needed.glm-5.3-flashstays out of both rosters: it is still never served by the Token Plan gateway (Z.ai VLM id, not plan entitlement). The parity golden test and the catalog assertions are updated in this same commit, per the repo convention that catalog PRs keep golden parity current.Verification
bun test tests/gui/alibaba-intl-token-plan.test.ts- 16 pass, 0 fail.bun test tests/providers/provider-registry-parity.test.ts- 53 pass, 0 fail.bun test tests/claude-integration/claude-desktop-1m.test.ts- 2 pass, 0 fail.bun run typecheck- clean.bun run privacy:scan- passed.git diff --check- clean.bun testlanes unreliable; focused lanes above cover every file this diff touches).AI-assisted: prepared with Hermes agent on behalf of the account owner; catalogue facts and the gateway probes above were performed and dated by the maintainer of https://github.com/oliver-mee/alibaba-token-plan-wiki (this repo's cited evidence source for Token Plan rows).
Review readiness checklist
This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:
All CI tests are green on my local testing.
I pushed my PR to the latest dev commit.
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.