[pull] main from microsoft:main#1510
Merged
Merged
Conversation
* fix: update workingDirectory to workingDirectories in ListSessionsResult * fix: update workingDirectories to be undefined instead of an empty array in ListSessionsResult
* AgentHost - auto-commit changes when archiving a session * Remove test that is not needed
* Enhance Agent Sessions UI: Update item and section heights, improve styling for better visibility and interaction Co-authored-by: Copilot <copilot@github.com> * Add tests for AgentSessionsListDelegate to verify height calculations and UI behavior Co-authored-by: Copilot <copilot@github.com> * Fix missing newline at end of file in agentSessionsListDelegate test * Add itemHeight and sectionHeight options to AgentSessionsControl and WelcomePage Co-authored-by: Copilot <copilot@github.com> * Refactor AgentSessionsControl to use a dedicated list delegate for improved height management Co-authored-by: Copilot <copilot@github.com> --------- Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.com> Co-authored-by: Copilot <copilot@github.com>
createModalEditorPart awaited the ModalEditorPart creation before assigning the singleton reference, so concurrent callers could each pass the "already exists" check and create duplicate modal parts/groups. Track the in-flight creation promise and have concurrent callers await and reuse it, applying their own options to the shared instance once it resolves. Adds a regression test that invokes createModalEditorPart concurrently via Promise.all and asserts a single instance/group is returned and only one group-registration event fires. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* correctly set up discover tests * attempt to fix windows tests
Await Copilot session teardown before refresh Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6a07710e-3e11-4636-999a-11986503b6cb
* Stream output for SDK built-in shell tool * better todo comment * Bump Copilot SDK to 1.0.8 preview Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 5083fb1f-34d7-44bf-82a1-c4582e57683b * Exclude unused Koffi native modules Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 5083fb1f-34d7-44bf-82a1-c4582e57683b * Adopt AHP terminal completion changes and stream built-in shell output over the terminal channel - Re-vendor agent-host-protocol: terminalComplete removed, exitCode/preview/truncated fold into terminal.result (TerminalCommandResult), isPty added to ToolResultTerminalContent and TerminalState - shell_exit now lands on the tool call's terminal block; history replay synthesizes a non-pty block keyed by toolCallId - Stream tool.execution_partial_result into an output-only agenthost-terminal channel (isPty false) instead of replacing Text content - Normalize LF to CRLF in AgentHostPty when the channel is plain text - Re-apply local CompletionItem.label extension clobbered by the protocol sync (still needs upstreaming) * Track last emitted snapshot for non-pty shell streaming The runtime emits cumulative plain-text snapshots (ANSI-stripped, throttled) that are capped to the leading ~10KB with a growing truncation marker, so a length-based delta goes stale once output exceeds the cap. Track the last emitted snapshot and prefix-check it: extend in place while the snapshot grows, reset the channel and rewrite when it was rewritten. * Normalize output terminal data to CRLF in the terminal manager Pty-backed channels already carry CRLF because the pty converts output line endings; do the same for output-only channels at the point data enters the channel, so terminal frontends render both identically and AgentHostPty needs no isPty-specific handling. Reverts the client-side normalization. * no need to complicate too much with lf crlf * Fix output-only shell command lifecycle rendering * keep non pty output handling scoped to agent hsot * fix output only terminal completion state * simplify agent host output terminal lifecycle * leave unnncessary background stuff out * scope non pty shell terminals by session * change ahp version to be proper * agentHost: regenerate AHP completion types --------- Copilot-Session: 5083fb1f-34d7-44bf-82a1-c4582e57683b
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.
8 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 : )