[pull] main from microsoft:main#1330
Merged
Merged
Conversation
Remove CSS rule causing flicker focus
…he new-issue form (#319577) Fix issue reporter "Preview on GitHub" opening repo root instead of new-issue form When filing against VS Code (not an extension), the wizard issue reporter only appended `/issues/new` to the target URL for extension targets. For VS Code / Marketplace targets routed through a bare repo URL (e.g. a `data.uri` override without the `/issues/new` suffix), the preview URL became `https://github.com/microsoft/vscode?title=...`, which opens the repository landing page rather than the pre-filled new-issue form. Generalize the `/issues/new` handling in `getIssueUrlWithTitle` to apply to any GitHub target URL that does not already point at the new-issue form, and drop the now-redundant `fileOnExtension` argument. Non-GitHub trackers short-circuit as external links earlier and are left untouched. Fixes #319238 Fixes #318374
Fixes #319676 Co-authored-by: Philip Cox <neogeneva@gmail.com>
…owser-serverReadyAction Add openIntegratedBrowser option to serverReadyAction.action
…Error (fixes #319622) (#319625) The previousRequest promise chain stored rejected promises when CopilotCLIQuotaExceededError was thrown. Since nothing catches previousRequest itself (it's only used as a serialization chain), this produced unhandled promise rejections reaching error telemetry. Add .catch() to the serialization chain so rejections don't escape. The error still propagates to the caller via the returned promise. Co-authored-by: vs-code-engineering[bot] <122617954+vs-code-engineering[bot]@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
fix: disable copy breadcrumbs path without symbols Co-authored-by: vivekjm <90664309+vivekjm@users.noreply.github.com> Co-authored-by: Raymond Zhao <7199958+rzhao271@users.noreply.github.com>
* Do not swap CLI session if session fails due to quota limit * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
* Update Agent Host Skill complletions * Refactor AgentHostSkillCompletionProvider tests for improved clarity and structure * Filter skills by typed prefix and ignore disabled customizations in AgentHostSkillCompletionProvider
* Agents: session header command center and toolbar refactor Restructure the Agents window titlebar and session view header: - Add a VS Code editor style command center to the titlebar showing the session kind icon, session title and workspace name. The session title truncates while the workspace name stays fully visible. - Split the monolithic ChatCompositeBar into a dedicated SessionHeader (session identity, title, meta row and toolbars) and a tabs-only ChatCompositeBar (chat tab strip). - Move the Open in VS Code and Run actions next to the session header, introducing menu ids as needed; keep terminal/run actions in place. - Align the session header with the sessions list (status icon column + stacked title/meta) and show a folder/worktree/cloud icon before the workspace name. - Add an architecture rule discouraging direct DOM traversal helpers (closest, etc.). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Agents: update RunScriptContribution test for new menu location Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Agents: address PR review feedback - sessionsTitleBarWidget: drop status/changes from the re-render autorun (the widget only displays title + workspace, so tracking the extra observables caused unnecessary DOM rebuilds). - sessionHeader: guard the drag-start handler so drags don't initiate from inside the title-row toolbar or the meta-row inline toolbar, preventing button clicks from being swallowed by a session drag. - chatCompositeBar: mark the tabs container as role="tablist" with an aria-label so screen readers announce it as a set of tabs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
After adopting the sessions grid we lost the previous "restore on reload" behaviour. Restore the full visible grid (order, sticky/pinned state and the active session, including the empty new-session slot) from persisted workspace state instead of only the last active session. - Snapshot the live visibility model on save and rebuild it atomically via VisibleSessions.restoreGrid so multiple restored sessions appear in one shot without flicker; late-arriving sessions are inserted into their persisted position incrementally. - Suppress the empty new-session view while restore runs (view-owned restoring flag + grid hide) so the composer does not flash before sessions are laid out. - Show per-leaf load progress on each chat view's own ProgressBar (mirroring editor-group ScopedProgressIndicator) while its chat model loads. - Move keyboard focus into the active session by observing ISessionsManagementService.activeSession from the part, guarded so it never steals focus from another surface (e.g. the sessions list). - Use a dedicated restore cancellation token that is actually cancelled when the user opens a session, and bound provider waits with a timeout so a deleted session cannot keep restore (and its listeners) pending forever. - Add a coding guideline forbidding lazy invokeFunction/accessor.get service resolution. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…19793) * WIP - initial implementation * Refactor things * Use "baseline" as a special token for a turnId in order to be able to compute the "All Changes" changeset * Pull request feedback
…oved type definitions (#319799) * feat: enhance Copilot plugin integration with model handling and improved type definitions * fix: streamline model extraction logic in toSdkCustomAgents function * fix: remove unused import of IParsedPlugin in copilotAgentSession.ts
* Chat: reuse image attachment thumbnails * Chat: avoid forcing attachment image MIME type
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
10 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.
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 : )