Skip to content

Commit babced6

Browse files
committed
fix: show server default model in /model display
Instead of just "Default", show "Default (claude-sonnet-4-6)" or "Default (CLI default)" so users can verify what model is active after resetting.
1 parent b7c00c5 commit babced6

File tree

2 files changed

+13
-3
lines changed

2 files changed

+13
-3
lines changed

src/bot/handlers/command.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1260,7 +1260,9 @@ def _current_model_label(context: ContextTypes.DEFAULT_TYPE) -> str:
12601260
label = short.capitalize()
12611261
break
12621262
if not override:
1263-
label = "Default"
1263+
settings = context.bot_data.get("settings")
1264+
server_model = getattr(settings, "claude_model", None) if settings else None
1265+
label = f"Default ({server_model or 'CLI default'})"
12641266
parts = [label]
12651267
if effort:
12661268
parts.append(f"effort={effort}")

tests/unit/test_bot/test_model_command.py

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,10 +206,18 @@ async def test_model_switch_clears_stale_effort(callback_query, context):
206206
# ---------------------------------------------------------------------------
207207

208208

209-
def test_label_default():
209+
def test_label_default_no_settings():
210210
ctx = MagicMock()
211211
ctx.user_data = {}
212-
assert _current_model_label(ctx) == "Default"
212+
ctx.bot_data = {}
213+
assert _current_model_label(ctx) == "Default (CLI default)"
214+
215+
216+
def test_label_default_with_server_model():
217+
ctx = MagicMock()
218+
ctx.user_data = {}
219+
ctx.bot_data = {"settings": MagicMock(claude_model="claude-sonnet-4-6")}
220+
assert _current_model_label(ctx) == "Default (claude-sonnet-4-6)"
213221

214222

215223
def test_label_with_model_and_effort():

0 commit comments

Comments
 (0)