-
Notifications
You must be signed in to change notification settings - Fork 2.4k
feat: add GLM-4.5V model support with image capabilities for z.ai provider #8416
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
Conversation
|
Acknowledged. I will add the full model definitions you outlined (including glm-4.5v with image support) to packages/types/src/providers/zai.ts, and extend tests in src/api/providers/tests/zai.spec.ts to cover them. |
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.
I found some issues that need attention. Summarized below.
| expect(model.info.contextWindow).toBe(204_800) | ||
| }) | ||
|
|
||
| it("should return GLM-4.5V China model with image support", () => { |
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.
P3: Optional: add a quick tiers assertion for mainland GLM-4.5V (e.g., expect(model.info.tiers?.length).toBe(3)) to prevent accidental tier removal.
|
full list no ? import type { ModelInfo } from '../model'; // Z AI export type InternationalZAiModelId = keyof typeof internationalZAiModels; export const ZAI_DEFAULT_TEMPERATURE = 0; |
|
Acknowledged. I’ve implemented the requested Z.AI model additions and verified everything end-to-end. Summary of changes:
Schema alignment:
Type compatibility:
Verification:
Files touched:
No behavior change beyond the expanded model catalog and image-capable glm-4.5v support. |
thanks @roomote |
|
Acknowledged. I reviewed the PR context and the Z.AI model additions, including glm-4.5v with image support. I will monitor checks and address any follow-ups. |
|
Update: Expanded International Z.AI model catalog and added glm-4.6 link in packages/types/src/providers/zai.ts. Adjusted provider handler typing in src/api/providers/zai.ts to satisfy BaseOpenAiCompatibleProvider when switching between international and mainland maps at runtime. Test coverage in src/api/providers/tests/zai.spec.ts passes locally; type checks pass locally. Pushed 3241255 and 365e6a9 to re-trigger CI; checks are pending and will be monitored. |
nice improvement now we don't need to use mcp vision to share image with coding plan and can just switch to GLM-4.5V if Pro/Max Plan then back to GM-4.6 for current usage |
|
@cobra91 do we have any updates? |
no :( |
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.
I found some issues that need attention @roomote, please take a look and fix the mwith PR fixer
| type MainlandZAiModelId, | ||
| ZAI_DEFAULT_TEMPERATURE, | ||
| zaiApiLineConfigs, | ||
| } from "@roo-code/types" |
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.
P2: Re-introduce type-only imports so the class can use the precise union type without any-casts (e.g., type InternationalZAiModelId, type MainlandZAiModelId).
Review Complete ✓All previously identified issues have been addressed in subsequent commits. The PR is ready for approval. Summary
No new issues identified. |
|
PR fixes applied:
Local status:
Commits pushed:
Re-triggered CI on latest fork head. I will monitor until all checks pass. |
…ntl tiers comment
…eral for zaiApiLineConfigs
…a to align with provider and tests
80a0b5c to
fb9f4ee
Compare
|
creating a new PR for this |
|
New PR #8745 |
Add glm-4.5v model to both international and mainland Z.AI configurations
Related GitHub Issue
Closes: #
Roo Code Task Context (Optional)
Description
Test Procedure
Pre-Submission Checklist
Screenshots / Videos
Documentation Updates
Additional Notes
Get in Touch
Important
Add GLM-4.5V model with image support to Z.AI provider, updating configurations and tests.
glm-4.5vmodel tointernationalZAiModelsandmainlandZAiModelsinzai.tswith image support and specific pricing tiers.zai.spec.tsto verifyglm-4.5vmodel integration for both international and China configurations, ensuring image support and correct context window.zai.tsto includeglm-4.5vmodel description and pricing details.This description was created by
for 8f7d39b. You can customize this summary. It will automatically update as commits are pushed.