forked from microsoft/vscode
-
Notifications
You must be signed in to change notification settings - Fork 0
[pull] main from microsoft:main #840
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
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
Tracing chat session status
* Expose MCP server definitions to ext host * Fixes * Update src/vs/workbench/api/common/extHostTypeConverters.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update src/vs/workbench/api/browser/mainThreadMcp.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Address review comments * address review comments * Reuse McpServerDefinition.Serialized instead of custom DTOs (#289165) * Initial plan * Reuse McpServerDefinition.Serialized instead of custom DTO interfaces Co-authored-by: DonJayamanne <1948812+DonJayamanne@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: DonJayamanne <1948812+DonJayamanne@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Use session resource for edit file tool
Removes the 500ms timeout fallback that was a hack to wait for focus transfer to auxiliary windows. Instead, only waits for focus transfer when preserveFocus is false, since preserveFocus: true means the window won't get focus and the layout service won't fire onDidChangeActiveContainer. - Remove unused raceCancellablePromises import - Add preserveFocus check to avoid waiting when focus won't transfer - Remove timeout(500) that could cause UI to appear in wrong location Fixes #279738 (Commit message generated by Copilot)
For #274403 Replacing chatSessionId in a few API proposals. Keeping around the old fields for now until we can adopt this everywhere
Adopt hover for session type picker and mode picker
Add `sessionResource` to proposed apis
* make fixed scrolling scrollable * set default back to fixedscrolling * fix comments * address some comments * fix scroll when done
* Workbench - update editor content menu design * Pull request feedback
…anel-dispose vscode-dts: Fix typedoc for WebviewPanel.dispose()
…get-updates 'Agent Status' widget tweaks
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…erns Co-authored-by: osortega <48293249+osortega@users.noreply.github.com>
Co-authored-by: osortega <48293249+osortega@users.noreply.github.com>
Use MarkdownString for privacy footer localization with embedded links
Combines adjacent text code blocks in ChatToolOutputContentSubPart to reduce the number of editor models and listeners created when tools return multiple separate output parts. When an MCP tool outputs many lines (e.g., 100 lines), they are now merged into a single code block instead of creating 100 separate blocks, which significantly improves responsiveness and prevents listener leak warnings. - Modified createOutputContents() to group consecutive code parts - Modified addCodeBlock() to accept an array of parts and combine their text content with newline separators - Reduces model/listener creation for high-volume MCP tool outputs Fixes #279624 (Commit message generated by Copilot)
Do not open the chat widget if welcome page is agent sessions
Privacy footer for agents welcome page
* fix: memory leak in folder configuration * pass resourcemap
Tracing for sessions timing
* Minor chat optimizations and fixes * Remove updateItemHeightOnRender in favor of a ResizeObserver * Remove other unneeded onDidChangeHeight emitters * Fix autoscroll- maybe this fixes our autoscroll issues. But it does fix the flickering I get when expanding items, due to waiting for the resize observer then waiting multiple animation frames for the scroll event to be handled * Clean up * More onDidChangeHeights cleanup * Cleanup * Fixes * Restore onDidChangeHeight for thinking part
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )