[pull] main from microsoft:main - #1620
Merged
Merged
Conversation
test: pin complete Copilot request bodies Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* sessions: move diff preference to overflow Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: simplify inline diff toggle Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: move collapse diffs to overflow Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: move code review to editor title Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: enable code review in editor title Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: move changes overflow to editor title Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: gate editor title actions by visibility Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: keep single-tab add action with title Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: balance editor action separator spacing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: keep group header actions minimal Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: separate add tab from editor actions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * editor: hide orphan layout action separator Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Revert "editor: hide orphan layout action separator" This reverts commit cf817b1. * editor: condition tab action separator on layout actions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * editor: require adjacent actions for separator Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: balance single-tab separator spacing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: align Add Tab hover target Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: address editor action review feedback Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* sessions: disable unsupported editor splits Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: address split guard feedback Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test: clean up restored editor state Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: isolate single-pane split guards Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Initial plan * Hide the session Changes pill for folderless chats Co-authored-by: benibenj <44439583+benibenj@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: benibenj <44439583+benibenj@users.noreply.github.com>
External sessions whose provider surfaces them without a title showed up unnamed in the sessions list. Generate one from the user's first prompt, reusing the existing rename infrastructure. Adds a deferred-work lane on AgentService: `markStartupComplete()` is signalled by the process mains, and work queued through `_runWhenStartupSettled` runs once that and the first served session listing have both happened. Background maintenance therefore never competes with startup, and the service owns no ambient timer. The stale external-session prune moves onto the same lane, replacing its 60s timer. Discovery and legacy migration queue newly registered external sessions that have no provider title. The job keeps the 2 most recently updated candidates, reads the first user prompt from the default chat, and hands it to `AgentHostSessionTitleController.generateExternalSessionTitle`, which reuses the same utility-model prompt, cleaning, persistence, and cancellation as first-message titling. Sessions that already carry a persisted title are left alone, and a rename during generation cancels it. Since these sessions are surfaced but not live, `updateSurfacedSessionTitle` pushes the title onto the surfaced summary so clients update in place. 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 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 : )