[pull] main from microsoft:main#1334
Merged
Merged
Conversation
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>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
A disabled `Toggle` only set `aria-disabled` and the `disabled` class but kept `tabIndex=0`, so it stayed reachable via Tab even though clicking or pressing it is a no-op while disabled. This is most visible in the editor find widget: after using Find once, the Match Case / Match Whole Word / Use Regular Expression / Preserve Case toggles remain in the tab order even while the widget is hidden (their owning input is disabled when hidden). Make `enable()`/`disable()` manage `tabIndex` so a disabled toggle is taken out of the tab order, matching the behaviour of `SimpleButton` in the find widget. Toggles created with `notFocusable` (e.g. toolbar action items that use a roving tabindex) keep their externally managed `tabIndex`. Fixes #147299
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
fix: update font size and weight variables for agents UI components Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.com> Co-authored-by: Copilot <copilot@github.com>
) The `sessions.goBack`/`sessions.goForward` actions registered Ctrl+Shift+Tab and Ctrl+Tab as secondary keybindings at `WorkbenchContrib` the sameweight chords and weight as the editor quick-open actions. With equal weight the editor actions won the conflict in the Agents window, so the chords never triggered session navigation. Bump the weight to `WorkbenchContrib + 1` so the session bindings win. The the Agents window, leaving the main workbench Ctrl+Tab behavior untouched. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ent (#319854) * Improve skill completion filtering and avoid duplicates in Copilot agent * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Remove unnecessary return statement in AgentHostSkillCompletionProvider * Filter out plugins with non-file scheme directories in CopilotAgent to prevent duplicates --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Bump built-in dompurify
Fix css var name
Fix two bugs causing the Codebase Semantic Index to show a perpetual
"Indexing..." spinner when the API returns a 401 authentication error:
1. Fix `codeSearchRepo.ts`: Change `if (!triggerSuccess)` to
`if (triggerSuccess.isError())`. The Result object is always truthy,
so the error handling code was never executed — errors fell through
to set BuildingIndex state and return ok.
2. Fix `githubCodeSearchService.ts`: Return `{ type: 'not-authorized' }`
for 401/403 HTTP responses in both `triggerIndexing` and
`getRemoteIndexState`, matching the pattern in `adoCodeSearchService.ts`.
This allows callers to properly differentiate auth failures.
Co-authored-by: mjbvz <12821956+mjbvz@users.noreply.github.com>
…-telemetry Emit restricted image metadata for Copilot telemetry
fix(docs): correct grammar and typo issues in documentation
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…19962) * Copilot AH: Avoid reading and parsing custom agents on the client * update
feat: enhance session item styling for unread and needs input states Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.com>
Bumps [hono](https://github.com/honojs/hono) from 4.12.18 to 4.12.23. - [Release notes](https://github.com/honojs/hono/releases) - [Commits](honojs/hono@v4.12.18...v4.12.23) --- updated-dependencies: - dependency-name: hono dependency-version: 4.12.23 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…pping (#319998) When clipping context with an even split between code above and below the cursor (the `!prioritizeAboveCursor` path of `expandRangeToPageRange`), any budget left unused by the above-cursor pass was discarded instead of being given to the below-cursor pass. Add an opt-in `useLeftoverBudgetFromAbove` flag that donates the unused upper half to the lower half. Because `expandRangeToPageRange` is shared by both the current file (`clipPreservingRange`) and other/recently-viewed files (`clipAroundFocalRanges`), expose it as two independent team-internal settings: - chat.advanced.inlineEdits.xtabProvider.currentFile.useLeftoverBudgetFromAbove - chat.advanced.inlineEdits.xtabProvider.recentlyViewedDocuments.useLeftoverBudgetFromAbove Both default to false, preserving existing behavior. Adds unit coverage for the donate-on vs donate-off behavior. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…319987) * sessions: add editor part chevron to toggle the secondary side bar Add a chevron toggle to the editor part title bar (shown when the editor is in the grid, not as a modal) that expands the editor by collapsing the auxiliary bar (secondary side bar). When the auxiliary bar is visible a chevron-right "Push Editor Right" hides it; once hidden the same slot shows a chevron-left "Show Secondary Side Bar" that restores it. The flip is driven by AuxiliaryBarVisibleContext. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: refine editor part secondary side bar toggle - Move the auxiliary bar chevron to the right of maximize/restore since it changes the right-hand side of the layout (order 99.5, before close). - Restore the editor's right border and corner radii via a .noauxiliarybar rule so it keeps its card appearance when the auxiliary bar is hidden. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* fix(ci): Copilot build uses public registry * Add Windows-specific steps
Add max width to session header
Syncs the agent-host-protocol types to upstream 740f6cf and adopts the breaking changes from PR 188, while reverting PR 191 (Changeset/ChangesSummary rename) which we are not ready to adopt yet.
- Replaces `toolClientId` on tool-call actions with `contributor: { kind, clientId }` so the renderer/server can distinguish client vs MCP-provided tools
- Plumbs the new `ChangesetOperationStatus` field through commit-operation provider, tests, and reducer fixtures
- Adds the required `enabled`/`state` fields to McpServerCustomization stubs across tests and pluginParsers
- Narrows `ISyncedCustomization.customization` and related helpers to `PluginCustomization` so per-plugin `children`/`load` access is type-safe
- Reverts PR 191 changes in the synced protocol files: keeps `ChangesetSummary` (with `additions`/`deletions`/`files`) and drops `ChangesSummary` from `SessionSummary`
(Commit message generated by Copilot)
…19995) The session header showed a static codicon for in-progress / needs-input sessions while the sessions list showed the animated pixel spinner. Make the header render the same spinner, and extract the shared status-icon logic into sessions/browser/sessionStatusIcon.ts so both surfaces stay in sync without the core browser layer importing from contrib. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…319815) * feat: enhance language model chat provider to support auto model insertion * Updates * feat: improve auto model handling in Copilot CLI by refining model filtering logic * feat: add support for auto model tracking in request details and persistence * feat: update persistCopilotCLIResponseModelId to include model type in parameters * feat: enable auto model feature in CLI configuration * fix: update persistCopilotCLIResponseModelId to accept boolean for model type parameter * feat: add buildAutoModel function for dynamic model selection and update tests to include auto model flag
Use stable position for markdown reopen with
* fix(chat): remove fallback for inputState when loading remote session * Skip for now * refactor(chat): enhance logging in state model changes with session ID * fix(chat): enhance state model logging with session ID
agentHost: adopt AHP protocol updates (PR 188)
* feat: implement inline session title editing functionality * feat: enhance session title input with placeholder and improved mouse event handling * grow when editing
…nfocused (#320005) sessions: scope nav keybindings to editor area focus The Go Back (Ctrl+Shift+Tab) and Go Forward (Ctrl+Tab) session navigation keybindings were gated on EditorContextKeys.editorTextFocus.toNegated(), which also suppressed them in cases where the editor part is open but not focused. Switch to EditorAreaFocusContext.toNegated() so the chords work whenever keyboard focus is outside the editor area, and only defer to the editor's Ctrl+Tab quick-open when the editor area actually has focus. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Fixes #313084 Already did this for the copy in core
…ssue Fix GithubCodeSearchService.triggerIndexing 401 auth error handling
Bump dompurify versions to 3.4.8
Move context menu into session header
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 : )