Skip to content

Commit e658e44

Browse files
committed
fix: surface selected model in setup summary
Read nested process payload model metadata from summary.model_used so successful setup runs show which model handled the source.
1 parent 5c2a632 commit e658e44

2 files changed

Lines changed: 9 additions & 0 deletions

File tree

tests/test_setup_studio_contract.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,3 +259,11 @@ def test_setup_views_expose_common_fixes_and_runtime_readiness_copy() -> None:
259259
assert '["Available locally", `${localModels.length} ready`],' in views_js.text
260260
assert "runtimeView?.next_action ||" in views_js.text
261261
assert "renderCommonFixes," in views_js.text
262+
263+
264+
def test_setup_process_summary_view_reads_nested_model_used() -> None:
265+
view_models_js = client.get("/static/setup_view_models.js")
266+
267+
assert view_models_js.status_code == 200
268+
assert "function collectProcessModel(result)" in view_models_js.text
269+
assert "result.summary?.model_used" in view_models_js.text

video_rss_aggregator/static/setup_view_models.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ function collectProcessModel(result) {
6060
result.model ||
6161
result.model_name ||
6262
result.summary_model ||
63+
result.summary?.model_used ||
6364
result.selected_model ||
6465
"Model not reported"
6566
);

0 commit comments

Comments
 (0)