[mirror] microsoft/vscode#307742 Use provider ordering in chat model picker#1063
Open
austindyoung wants to merge 3241 commits intomainfrom
Open
[mirror] microsoft/vscode#307742 Use provider ordering in chat model picker#1063austindyoung wants to merge 3241 commits intomainfrom
austindyoung wants to merge 3241 commits intomainfrom
Conversation
…tomato-hornet Update CI status widget labels and styles for improved clarity
* Sessions: remove weird description
* Add chat model reference inspection (Written by Copilot) * Add archive state and age to chat model inspection (Written by Copilot) * fix import * Cleanups * test
* Refactor padding and line-height in AI Customization Management styles for improved layout Co-authored-by: Copilot <copilot@github.com> * Update src/vs/workbench/contrib/chat/browser/aiCustomization/media/aiCustomizationManagement.css Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.com> Co-authored-by: Copilot <copilot@github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Michael Lively <12552271+Yoyokrazy@users.noreply.github.com>
…irefly Bump distro
…d consistency (microsoft#306858) * style: update agent feedback widget styles for code review and adjust border radius * style: adjust agent feedback widget colors for better visibility and consistency * style: update agent feedback widget suggestion styles for improved layout and visual clarity * style: enhance suggestion border for code review items with color-mix for better visibility * style: update agent feedback widget suggestion header for improved clarity and consistency Co-authored-by: Copilot <copilot@github.com> * style: refactor suggestion rendering and improve CSS for consistency and clarity Co-authored-by: Copilot <copilot@github.com> * Update src/vs/sessions/contrib/agentFeedback/browser/media/agentFeedbackEditorWidget.css Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.com> Co-authored-by: Copilot <copilot@github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Michael Lively <12552271+Yoyokrazy@users.noreply.github.com>
…6902) Adjust header height and padding in CIStatusWidget; refine button styles in changes view Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.com>
Replace the temporary chat.defaultMode experiment with a proper configuration setting chat.newSession.defaultMode whose default is driven by TAS treatment chatDefaultNewSessionMode. - Add DefaultNewSessionMode to ChatConfiguration enum - Register setting dynamically via ChatAgentSettingContribution following the registerMaxRequestsSetting pattern - Rewrite _setEmptyModelState to sync read the config setting with case-insensitive custom mode name matching - Remove unused validateChatMode, isChatMode, and getDefaultModeExperimentStorageKey - Remove IWorkbenchAssignmentService from ChatInputPart (moved to ChatAgentSettingContribution) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Sessions: Update naming (fix microsoft#306461) * .
* Update distro commit to 1992012c * Update distro commit to 084798d4 --------- Co-authored-by: vs-code-engineering[bot] <122617954+vs-code-engineering[bot]@users.noreply.github.com>
* Link to browser editors in tool calls * a11y, feedback
* Show browser tab icons in editor quick access pickers * feedback
…crosoft#306940) share auto approval logic bw runInTerminal and
…ction (microsoft#307585) * chore: use protocol from product configuration for openAgentsWindow action * chore: address review feedback Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * chore: fix build --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…idget.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
fix: improve test coverage filter quickpick readability
… explorer (microsoft#307555) When typing text like "@smoke" in the test explorer filter, the @-prefix was consumed by the tag regex but did not match any known filter term or tag syntax. This caused the text to silently disappear from the filter, making it impossible to search for tests with "@" in their names. Now, @-prefixed text that is neither a known filter term (@failed, @doc, etc.) nor a tag (@ctrlId:tagId) is preserved as regular glob filter text. Closes microsoft#159708
* feat: add ChatHookProvider to chatPromptFiles API Wire hooks through the same chatPromptFiles proposed API surface used by agents, instructions, and skills: - Add ChatHookProvider interface with onDidChangeHooks/provideHooks - Add chat.hooks getter and chat.onDidChangeHooks event - Add chat.registerHookProvider() registration - Add IHookDto and $acceptHooks to ext host protocol - Add onDidChangeHooks to IPromptsService interface and impl - Push hook files from mainThread to ext host on change - Fix MockPromptsService for new interface member * polish: expand provider hook items into individual entries The provider path was showing hooks as file-level items (e.g. 'hooks.json', 'settings.json'), while the core path expands them into individual hook entries with type labels and command descriptions. Add _expandProviderHookItems() that reads and parses each hook file from the provider, expanding into the same individual entries the core path produces (e.g. 'Session Start', 'Pre-Tool Use' with truncated command text).
Wire installed agent plugins through the chatPromptFiles proposed API: - Add ChatPluginResource interface with uri + label - Add chat.plugins getter and chat.onDidChangePlugins event - Add IPluginDto and $acceptPlugins to ext host protocol - Push plugins from mainThread via autorun on IAgentPluginService - Inject IAgentPluginService into MainThreadChatAgents2
…ss when condition specifies chatSessionType (microsoft#307530)
) * chat: add startup telemetry for live chat model counts After reviving sessions with pending edits at startup, log telemetry with the number of live chat models, how many are open in widgets, how many are background-only, and how many are kept alive solely because they have unaccepted edits. - Expose whenSessionsRevived promise on IChatService - Add browser-layer contribution that awaits revival + uses IChatWidgetService to classify models without hard-coded strings Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * address review: error handling, void fire-and-forget, referenceCount check - Wrap reviveSessionsWithEdits with .catch() to avoid unhandled rejection - Mark logTelemetry() as void fire-and-forget - Check referenceCount === 1 for modelsKeptAliveOnlyForEdits Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove chat.statusWidget.anonymous setting
…ions (microsoft#307677) * agentHost: archive and store archive state for remote agent host sessions * stub IStorageService in remote agent host sessions provider tests
* agentPlugins: clone locally when in a remote The git extension runs on the remote side, so it's not suitable to use to clone. This just does it directly. I would kind of like to just take this path in all cases since I previously had to add some internal commands in the git extension. But for now going into next release this only affects the remote case which was previously broken. Closes microsoft#298701 * cleanup * comments
* Sessions: preserve cancelled session content in list When a session is stopped before the agent commits a worktree, keep it in the sessions list with Completed status instead of removing it. This lets the user review whatever content the agent produced before cancellation. - Use CancellationError in _waitForCommittedSession to distinguish cancellation from unexpected failures - On CancellationError in _sendFirstChat/_sendSubsequentChat, set Completed status and fire a changed event instead of removing - Add tests for committed-then-stopped and cancelled-before-commit flows Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Address review: defer CancellationError until timeout, remove unused variable - Move isCanceled check from the early-exit path to after the 5s safety timeout so a late commit event (user stops after worktree was initiated but before IPC finishes) is not missed. - Remove unused changes variable in committed-session test. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Sessions - move state computation to the view model * Remove debug message * Add back the missing context key
* sessions: disable branch picker in folder mode Keep the new chat branch picker visible when isolation switches to Folder, but disable it until Worktree is selected again. Also add regression coverage for the folder/worktree toggle and keep the disabled picker cursor consistent on hover. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: remove branch picker hover cursor tweak Keep the branch picker behavior change, but drop the extra disabled hover cursor styling so the PR only contains the intended picker visibility/disabled-state update. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Append repository detail metadata, or fall back to the worktree folder name, in the sessions titlebar widget so active titles render as 'Title · repo (branch/worktree)'. Also update the sessions layout spec to document the new title format. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Lower the sessions sidebar resize floor from 270px to 170px and keep the layout specification in sync with the final sessions app behavior. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Some old bad logic here caught up to us
Add vendorPriority field to the language model metadata API surface, allowing extensions to specify a numeric priority for vendor-based ordering in the chat model picker. When set, models are sorted by vendorPriority before falling back to the existing alphabetical vendor/name ordering. This applies to the promoted section, other models section, and flat view sort in chatModelPicker.ts. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
When models have vendorPriority set, they are automatically placed in the promoted section of the picker (above Other Models) and sorted by their priority value. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove vendorPriority-based auto-promotion so the picker relies on VS Code's existing models control manifest for featured models, while still using vendorPriority for ordering within promoted and other model lists. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Drop the accidental product.json change from the branch so the PR only contains the model picker ordering work. The product.json override was only needed for local OSS testing and should not ship in the PR. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
20 participants
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.
Automated mirror PR for DriftFence counterfactual telemetry.
Source PR: microsoft#307742
Source URL: microsoft#307742
Mirror branch: driftfence/mirror/pr-307742/latest
This PR is maintained by automation for telemetry and review links.