[mirror] microsoft/vscode#307810 Migrate terminal shell integration to AsyncIterableProducer#1080
Open
austindyoung wants to merge 3251 commits intomainfrom
Open
[mirror] microsoft/vscode#307810 Migrate terminal shell integration to AsyncIterableProducer#1080austindyoung wants to merge 3251 commits intomainfrom
austindyoung wants to merge 3251 commits intomainfrom
Conversation
* Refactor padding and line-height in AI Customization Management styles for improved layout Co-authored-by: Copilot <[email protected]> * Update src/vs/workbench/contrib/chat/browser/aiCustomization/media/aiCustomizationManagement.css Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: mrleemurray <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Michael Lively <[email protected]>
…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 <[email protected]> * style: refactor suggestion rendering and improve CSS for consistency and clarity Co-authored-by: Copilot <[email protected]> * Update src/vs/sessions/contrib/agentFeedback/browser/media/agentFeedbackEditorWidget.css Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: mrleemurray <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Michael Lively <[email protected]>
…6902) Adjust header height and padding in CIStatusWidget; refine button styles in changes view Co-authored-by: mrleemurray <[email protected]>
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 <[email protected]>
* 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
* promptService: cache IHookDiscoveryInfo * Update src/vs/workbench/contrib/chat/common/promptSyntax/service/promptsServiceImpl.ts Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Paul <[email protected]>
* Sessions - move more code into the view model * Hide the path in the tree
…06953) * Sessions: Add Copilot status indicator to sidebar footer Add a Copilot status button to the AccountWidget in the sessions app sidebar footer. The button shows the current Copilot status icon and opens a ChatStatusDashboard hover popup on click. - Add per-section disable options to ChatStatusDashboard to control which sections render (disableInlineSuggestionsSettings, disableModelSelection, disableProviderOptions, disableCompletionsSnooze, disableContributions) - Copilot status button is hidden when the update button is visible - Status icon updates based on entitlement, sessions, and quota state - Scoped CSS overrides for dashboard popup in sessions context Co-authored-by: Copilot <[email protected]> * Fix Copilot status button accessibility and disposable leak - Set aria-label on the Copilot status button so screen readers announce the current status tooltip - Track dashboard DisposableStore via MutableDisposable registered on the widget, ensuring cleanup on widget disposal or re-click Co-authored-by: Copilot <[email protected]> * Remove native tooltip from Copilot status button Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
* Add readable element labels in browser tool calls * feedback * detail * browser
This enables restore/undo checkpoint for agent host sessions. It also does some initial work for forking although this is not yet fully implemented. The copilot SDK does not actually support these yet, so to do these we shut down the session, rewrite copilot SDK's disk storage, and start it back up again. It actually works. We'll need to make sure it works when we upgrade the SDK, but I don't expect it to break terribly, as the Copilot SDK folks must already be backwards-compatible to arbitrary old SDK versions that exist on the user's device, and we'd essentially just be an 'old SDK' with some dependency on internals. Of course that should all be swapped out when they eventually add proper support for it. I just flagged the specific scheme of agent host sessions thus far while developing, but will clean up prior to merging.
…zations agenthost: customization syncing data flow
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 <[email protected]> * 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 <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
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 <[email protected]> * 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 <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
* 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 <[email protected]> * 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 <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
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 <[email protected]>
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 <[email protected]>
Some old bad logic here caught up to us
…oggles the checkbox (microsoft#298074) * Initial plan * Fix clicking on settings link in bool setting toggles the checkbox When a settings link is clicked in the description of a bool setting, the MOUSE_DOWN handler was checking if `e.target.tagName === 'a'` to avoid toggling the checkbox. However, settings links rendered by the markdown renderer contain an SVG icon inside the `<a>` tag. Clicking on the SVG makes `e.target` the SVG element (not the `<a>`), causing the check to fail and incorrectly toggling the checkbox. Fix by using `targetElement.closest('a')` which traverses up the DOM tree to detect clicks within any ancestor `<a>` element. Co-authored-by: rzhao271 <[email protected]> * Fix type safety of e.target cast in bool setting MOUSE_DOWN handler Use `instanceof Element` guard instead of casting to HTMLElement, since clicks can originate from SVGElement/SVGPathElement. Also guard against null before calling closest('a'). Co-authored-by: rzhao271 <[email protected]> Agent-Logs-Url: https://github.com/microsoft/vscode/sessions/102850fa-98da-4c19-b23a-414cbe95323b --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: rzhao271 <[email protected]>
…imizations (microsoft#307715) * Agent Debug Panel: pagination, incremental filtering, and service optimizations * update * feedback update
Remember last browser URL before disposal
Co-authored-by: Copilot <[email protected]>
* agentHost: Enable local agent host in sessions app Co-authored-by: Copilot <[email protected]> * comments Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
From microsoft#304603 since it leads to not rendering parallel edits from Copilot CLI correctly Co-authored-by: Copilot <[email protected]>
…icrosoft#307736) Co-authored-by: Copilot <[email protected]>
* sessions: use generic sign out confirmation copy (microsoft#307465) Allow the shared account sign-out action to accept dialog copy overrides so the Agents app can show a generic confirmation instead of listing extension usage. Adds a focused test for the override path while preserving the default workbench confirmation behavior. Co-authored-by: Copilot <[email protected]> * sessions: scope sign out confirmation to Agents app (microsoft#307465) Keep the Agents-specific sign-out confirmation flow inside the sessions account menu instead of extending the shared authentication sign-out action. This restores the core workbench auth dialog behavior and limits the copy change to the sessions app surface. Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
* Ignore file-like domain suffixes in terminal sandbox * Handle file-like suffixes for URL and SSH domains
…oft#307783) * Clarify sandbox-first guidance for terminal tool * Refine terminal sandbox schema wording
…terableProducer Addresses part of microsoft#256854. The terminal shell execution data stream was using AsyncIterableObject which retains all emitted values in an internal _results array, causing a potential memory leak for long-running terminal sessions with large output. Migrated ShellExecutionDataStream to use AsyncIterableProducer which does not store values, and replaced the toPromise()-based flush with DeferredPromise tracking for completion signaling. Co-Authored-By: Claude Opus 4.6 <[email protected]>
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#307810
Source URL: microsoft#307810
Mirror branch: driftfence/mirror/pr-307810/latest
This PR is maintained by automation for telemetry and review links.