lsm / neokai / 27696093944

82%

push

github

fix: strip all trailing [1m] suffixes from GLM model IDs to prevent 1M → 200K fallback (#2125)

GLM-5.2 sessions reported context window mismatch:
"SDK reports 1000000 tokens for model=glm-5.2[1m][1m] but metadata declares 200000 tokens (mismatch 80.0%)"

Root cause: glm-provider.ts buildSdkConfig() only stripped ONE trailing [1m] suffix.
When modelId was glm-5.2[1m][1m], it became glm-5.2[1m], then got another [1m] appended
for routing → glm-5.2[1m][1m] again, which wasn't in CONTEXT_WINDOW_BY_MODEL_ID map,
causing 200K fallback.

Fix:
- Changed regex from /\[1m\]$/ to /(\[1m\])+$/ to strip ALL trailing [1m] suffixes
- Added normalizeModelId() helper in context-fetcher.ts to collapse double suffixes
- Added tests for glm-5.2[1m][1m] and glm-5.2[1m][1m][1m] routing

Expected behavior after fix:
- glm-5.2 → displays 1M, routes to glm-5.2[1m], context window 1M ✓
- glm-5.2[1m] → displays 1M, routes to glm-5.2[1m], context window 1M ✓
- glm-5.2[1m][1m] → normalizes to glm-5.2[1m], displays 1M, context window 1M ✓

10693 of 15600 branches covered (68.54%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 2 files covered. (100.0%)

90831 of 108713 relevant lines covered (83.55%)

293.43 hits per line