Repository navigation
feat(morph): refresh curated models - #7314
diegosouzapw merged 2 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the Morph provider catalog by adding the GLM-5.2 744B and MiniMax M3 models, while removing the deprecated MiniMax M2.7 model. It also introduces a new unit test file to verify the presence of the new models and ensure ID uniqueness. The reviewer suggested adding an assertion to the test suite to explicitly verify that the deprecated MiniMax M2.7 model has been successfully removed from the catalog.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
96356f3 to
6a79ca1
Compare
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
2186ed0 to
479b0b3
Compare
|
Thanks for the Morph catalog refresh — the GLM-5.2 744B / MiniMax M3 additions and the MiniMax M2.7 removal look correct and consistent with the existing naming/versioning pattern in One thing before merge: the PR description lists |
807fb8f to
8f0e0ce
Compare
087abac to
f99bfa2
Compare
The Morph curated-model refresh (morph-glm52-744b, morph-minimax3-428b added; morph-minimax27-230b removed) shipped with a PR body claiming a tests/unit/morph-provider-catalog.test.ts that was never actually committed. Add that test for real: asserts the two new ids/metadata are present, the retired MiniMax M2.7 id is gone, the untouched models are still there, and there are no duplicate ids in CHAT_OPENAI_COMPAT_MODELS.morph. Verified red against the pre-refresh catalog (3/6 assertions fail: glm52 missing, minimax3 missing, minimax27 still present) and green against this branch's shared.ts (6/6). Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
f99bfa2 to
dfc385d
Compare
aa0b56d
into
diegosouzapw:release/v3.8.49
|
Merged into |
* feat(morph): refresh curated models * test(providers): add regression guard for Morph catalog refresh The Morph curated-model refresh (morph-glm52-744b, morph-minimax3-428b added; morph-minimax27-230b removed) shipped with a PR body claiming a tests/unit/morph-provider-catalog.test.ts that was never actually committed. Add that test for real: asserts the two new ids/metadata are present, the retired MiniMax M2.7 id is gone, the untouched models are still there, and there are no duplicate ids in CHAT_OPENAI_COMPAT_MODELS.morph. Verified red against the pre-refresh catalog (3/6 assertions fail: glm52 missing, minimax3 missing, minimax27 still present) and green against this branch's shared.ts (6/6). Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com> --------- Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
* feat(morph): refresh curated models * test(providers): add regression guard for Morph catalog refresh The Morph curated-model refresh (morph-glm52-744b, morph-minimax3-428b added; morph-minimax27-230b removed) shipped with a PR body claiming a tests/unit/morph-provider-catalog.test.ts that was never actually committed. Add that test for real: asserts the two new ids/metadata are present, the retired MiniMax M2.7 id is gone, the untouched models are still there, and there are no duplicate ids in CHAT_OPENAI_COMPAT_MODELS.morph. Verified red against the pre-refresh catalog (3/6 assertions fail: glm52 missing, minimax3 missing, minimax27 still present) and green against this branch's shared.ts (6/6). Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com> --------- Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
Summary
Related Issues
Validation
npm run typecheck:corebun scripts/check/check-provider-consistency.tsTests Added Or Updated
Coverage Notes
Reviewer Notes
preservation.tsdiff is the shared CI regression fix described below; no migration or feature flag is required.CI Follow-up (2026-07-17)
Root Refresh (2026-07-17,
93e217e76)release/v3.8.49root at93e217e76375d1e250708ea0d0ce85618d346a89.git diff --check, full ESLint, core typecheck, complexity ratchets, and provider-consistency validation passed during the refresh.Root Refresh (2026-07-18,
ed4944e77)release/v3.8.49root ated4944e771d3e5dba944534414cf470c85598f22.git diff --check, core typecheck, and provider consistency passed (190 registry entries, 267 canonical providers, 0 exceptions).Recovery Audit (2026-07-18,
e6f81d827)release/v3.8.49root ate6f81d827e5d92f13fd4ac157f82dd1e6bc7bd1a.tests/unit/microsoft-designer-web-6672.test.tsfrom Strykertap.testFiles; the newfreetheaialias collides with the test fixture prefixfta, the translate-path golden does not yet containfreetheai, and structural complexity is 2058 against the committed 2056 baseline. Cognitive complexity is 890 / 890.