Skip to content

Comments

Disable free GLM-5#420

Open
kiloconnect[bot] wants to merge 1 commit intomainfrom
disable-free-glm-5
Open

Disable free GLM-5#420
kiloconnect[bot] wants to merge 1 commit intomainfrom
disable-free-glm-5

Conversation

@kiloconnect
Copy link
Contributor

@kiloconnect kiloconnect bot commented Feb 20, 2026

Disable the free GLM-5 (glm-5) model.

Changes

  • Set is_enabled: false in src/lib/providers/zai.ts
  • Removed the GLM-5 free notification from src/lib/notifications.ts
  • Removed the GLM-5 entry from the approval test src/tests/openrouter-models-sorting.approved.json

This PR should be merged around Feb 22nd 23:59 Beijing Time (UTC+8) / Feb 22nd 16:59 Amsterdam Time (CET, UTC+1).


Built for Christiaan by Kilo for Slack

- Set is_enabled to false in zai.ts
- Remove GLM-5 free notification
- Remove GLM-5 entry from approval test
@chrarnoldus chrarnoldus marked this pull request as ready for review February 20, 2026 21:03
@kiloconnect
Copy link
Contributor Author

kiloconnect bot commented Feb 20, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Notes

  • The is_enabled: false approach is well-designed — it properly gates all free model behavior via isFreeModel(), isRateLimitedModel(), isDataCollectionRequiredOnKiloCodeOnly(), and the preferredModels fallback.
  • The as KiloFreeModelsatisfies KiloFreeModel change in zai.ts is a nice improvement, consistent with the project's coding style rules.
Other Observations (not in diff)
File Line Observation
src/lib/models.ts 67 kiloFreeModels array still uses as KiloFreeModel[] — could be updated to satisfies KiloFreeModel[] for consistency with the satisfies change in this PR (not blocking).
Files Reviewed (3 files)
  • src/lib/notifications.ts — notification entry removed
  • src/lib/providers/zai.ts — model disabled, assatisfies
  • src/tests/openrouter-models-sorting.approved.json — test snapshot updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants