fix(status): show actual routed model in /status after fallback#26876
fix(status): show actual routed model in /status after fallback#26876xinhuagu wants to merge 1 commit into
Conversation
|
@xinhuagu The Could we split this into focused PRs (status fix vs Copilot cache hardening)? It would make review, regression triage, and rollback safer. |
|
@bmendonca3 Done — I removed the unrelated Copilot cache changes from this PR and pushed an update. #26876 is now scoped to the /status routed-model fix (plus its status regression coverage) only. |
5e371b9 to
221c95f
Compare
|
Status note: latest remaining red appears to be the shared Windows Feishu proxy test path, which looks unrelated to this /status routed-model fix. Keeping this PR up to date while waiting for green upstream test state. |
fd0c910 to
ef17d47
Compare
ef17d47 to
72e21c2
Compare
|
Closing — #23866 addressed the same issue earlier. |
Summary
Fixes /status showing only the configured model after runtime fallback by using the latest transcript provider/model metadata as runtime truth.
Closes #23830
What changed
/statusnow enables transcript-tail inspection (includeTranscriptUsage: true) so it can resolve runtime model metadata.readUsageFromSessionLognow captures bothproviderandmodelfrom transcript lines and normalizes them into a fullprovider/modelref.buildStatusMessagenow applies transcript-derived runtime model even whensessionEntry.modelexists, so fallback routing is reflected in status output.Test
pnpm -s vitest src/auto-reply/status.test.tsGreptile Summary
Fixed
/statuscommand to display the actual routed model after runtime fallback by enabling transcript inspection and using provider/model metadata from the transcript as the source of truth.includeTranscriptUsage: trueincommands-status.tsso transcript metadata is availablereadUsageFromSessionLogto capture bothproviderandmodelfrom transcript lines and normalize them into a full provider/model referenceConfidence Score: 5/5
Last reviewed commit: 6e4b502