Skip to content

fix(models): price Hermes lowercase glm-5.2 the same as GLM-5.2#545

Merged
iamtoruk merged 1 commit into
getagentseal:mainfrom
ozymandiashh:fix/hermes-glm-5.2-lowercase-pricing
Jun 21, 2026
Merged

fix(models): price Hermes lowercase glm-5.2 the same as GLM-5.2#545
iamtoruk merged 1 commit into
getagentseal:mainfrom
ozymandiashh:fix/hermes-glm-5.2-lowercase-pricing

Conversation

@ozymandiashh

Copy link
Copy Markdown
Contributor

Follow-up to the maintainer's note on #543. The merged Hermes provider (#544) reads sessions.model and passes it straight to calculateCost, but Hermes stores GLM-5.2 lowercased as glm-5.2, while BUILTIN_ALIASES only carries the capitalized GLM-5.2 (added for ZCode). As a result GLM-5.2 sessions logged by Hermes resolve to no pricing and report €0.

This adds the lowercase glm-5.2 alias pointing to the same glm-5p1 sibling, plus a regression test asserting the lowercase and capitalized spellings price identically. Verified against a real ~/.hermes/state.db whose only GLM model id is glm-5.2.

@iamtoruk iamtoruk merged commit 4dcb7e6 into getagentseal:main Jun 21, 2026
3 checks passed
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.

2 participants