Skip to content

fix(llm): MiniMaxモデル名表記をMiniMax-M3(PascalCase)に統一 - #115

Merged
fukukei23 merged 1 commit into
mainfrom
feature/minimax-model-name-consistency
Jul 27, 2026
Merged

fix(llm): MiniMaxモデル名表記をMiniMax-M3(PascalCase)に統一#115
fukukei23 merged 1 commit into
mainfrom
feature/minimax-model-name-consistency

Conversation

@fukukei23

Copy link
Copy Markdown
Owner

Summary

MiniMaxモデル名の表記揺れ(minimax-m3 小文字 / MiniMax-M3 PascalCase)を MiniMax-M3 に統一。

  • llm_profiles.py: PROFILE_REGISTRY の minimax_default/analytical を minimax-m3MiniMax-M3
  • test_generator.py: _call_minimax デフォルトを MiniMax-M2.7(旧モデル)→MiniMax-M3
  • 上記テスト追従 + test_call_minimax_sends_correct_payloadMINIMAX_MODEL env を明示セットして .env 流入を防御(従来は env 未隔離でフルスイート時のみ1件FAIL)

背景

実API(.envMINIMAX_MODEL=MiniMax-M3)と整合しない小文字表記が残存。_call_minimax が env 未設定時に小文字を送り実APIで弾かれるリスクがあった(PR #114 の test plan にも 1 failed(MiniMax-M2.7→MiniMax-M3不一致) と記載)。

Test plan

  • LLM系 tests/llm/ + test_generator — 681 passed / 0 failed
  • フルスイート tests/ — 4905 passed / 0 failed(従来 4904 passed, 1 failed → 失敗ゼロ化)

🤖 Generated with Claude Code

- llm_profiles.py: minimax_default/analytical のmodelをminimax-m3→MiniMax-M3
- test_generator.py: _call_minimaxデフォルトをMiniMax-M2.7(旧)→MiniMax-M3
- 上記テスト追従 + test_call_minimax_sends_correct_payload をMINIMAX_MODEL
  env明示セットで.env流入防御
- 実API(.env MINIMAX_MODEL=MiniMax-M3)と整合・小文字送信によるAPI弾きリスク解消
- 検証: LLM系681 passed / フル4905 passed・0 failed(従来1 failed→ゼロ化)

Co-Authored-By: Claude <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@github-actions

Copy link
Copy Markdown

STIT Pipeline Result ✅

Branch: feature/minimax-model-name-consistency

All STIT checks have completed. Check the workflow run for details.

@github-actions

Copy link
Copy Markdown

Phase3 Coverage (graph_builder / unified_analyzer / test_generator / tree_sitter_checker)

Module Stmts Miss Coverage
analyzer.unified_analyzer 128 8 93.8%
analyzer.graph_builder 46 4 91.3%
utils.test_generator 208 116 44.2%
utils.tree_sitter_checker 36 24 33.3%
TOTAL 418 152 63.6%

@fukukei23
fukukei23 merged commit 578ce34 into main Jul 27, 2026
14 checks passed
@fukukei23
fukukei23 deleted the feature/minimax-model-name-consistency branch July 27, 2026 19:11
@github-actions

Copy link
Copy Markdown

STIT Pipeline Result ✅

Branch: feature/minimax-model-name-consistency

All STIT checks have completed. Check the workflow run for details.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant