chat: show agent host model pricing in turn details#315159
Merged
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
|
Base:
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR enhances Agent Host chat session history so each response can display the selected model’s pricing/multiplier (when available) in the response footer details, including for restored “active turn” placeholder responses after reconnect/restore. It also extends the Agent Host chat history tests to validate the per-turn model + pricing details behavior.
Changes:
- Populate
detailsfor restored active-turn placeholder responses using the per-turn model lookup. - Extend model display formatting to append
pricing(e.g.Opus 4.7 · 15x) when present. - Update Agent Host chat history tests to stub language model metadata and assert per-response
detailsvalues.
Show a summary per file
| File | Description |
|---|---|
| src/vs/workbench/contrib/chat/browser/agentSessions/agentHost/agentHostSessionHandler.ts | Adds model pricing to the per-response details display string and ensures restored active-turn placeholder responses include details. |
| src/vs/workbench/contrib/chat/test/browser/agentSessions/agentHostChatContribution.test.ts | Injects stub model metadata via ILanguageModelsService.lookupLanguageModel and asserts response details includes model name + pricing across history and active turn. |
Copilot's findings
- Files reviewed: 2/2 changed files
- Comments generated: 0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…917f8f26384' into agents/944c6bd9-457f-4a9d-9f2b-9917f8f26384 # Conflicts: # src/vs/workbench/contrib/chat/browser/agentSessions/agentHost/stateToProgressAdapter.ts
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Member
Author
|
/requires-eval-assessment vscbench,terminalbench2 gpt-5.4,claude-opus-4.6 |
roblourens
added
the
~requires-eval-assessment
Contributor
|
⏳ Queued vscode build for
|
vs-code-engineering
Bot
removed
the
~requires-eval-assessment
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bhavyaus
approved these changes
May 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix #315160
Summary
Validation