Skip to content

[pull] main from microsoft:main#1071

Merged
pull[bot] merged 6 commits intocode:mainfrom
microsoft:main
Mar 25, 2026
Merged

[pull] main from microsoft:main#1071
pull[bot] merged 6 commits intocode:mainfrom
microsoft:main

Conversation

@pull
Copy link

@pull pull bot commented Mar 25, 2026

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 : )

joshspicer and others added 6 commits March 24, 2026 17:05
* fix #304030: show custom agent names as-is without title-casing

formatDisplayName was converting dashes/underscores to spaces and
applying title case (e.g. nls-code-helper → Nls Code Helper). This
made names inconsistent between the customization view and the agent
dropdown. Now only strips .md extension, showing names exactly as
defined in frontmatter headers.

* fix #304131: show workspace-relative paths in customization tooltips

Workspace items now show paths relative to the project root (e.g.
.github/instructions/coding-standards.instructions.md) instead of
absolute paths (/Users/.../workspace/.github/...).

* fix #304133: show extension name instead of full path in tooltips

Extension items now show 'Extension: github.copilot-chat' in the
tooltip instead of the full filesystem path into the extension
directory. The extensionLabel is populated from the extensionIdByUri
map during item post-processing.

* fix #304178: use save icon instead of ambiguous checkmark

The editor save indicator now uses Codicon.save (floppy disk) instead
of Codicon.check (bare checkmark), making it clear that the indicator
is about save status rather than an enable/disable toggle.

* fix #304183: show user-scoped hooks in sessions window

HOOKS_FILTER excluded PromptsStorage.user, so hooks in
~/.copilot/hooks/ were filtered out in the sessions window (and
any harness using the restricted filter). User hooks are now
supported in CLI, so add user storage to the allowed sources.

* fix: improve list layout footer fallback and remove extension tooltip

- Use 80px footer height fallback instead of 0 when sectionHeader
  has not rendered yet, preventing list from overlapping the footer
  on first paint
- Wait for async section load in fixture before final layout
- Remove extensionLabel tooltip (implementation detail, not relevant
  to users)

* fix #304178: show loading spinner before saved icon, remove green color

The save indicator now shows a spinning loading animation while the
file is being saved (onDidChangeDirty dirty→clean transition), then
switches to the save icon when the save completes (onDidSave). The
special green color was removed to use the default description
foreground color instead.

* fix: simplify list layout to use CSS flex instead of manual height calculation

All three list widgets (AICustomizationListWidget, McpListWidget,
PluginListWidget) now rely on CSS flex layout to distribute space
between search bar, list container, and footer. The layout() method
sets the widget height, clears any manual list container height,
and reads clientHeight (forcing a synchronous reflow) to get the
flex-computed height for the WorkbenchList.

This eliminates hardcoded fallback values, requestAnimationFrame
correction loops, and manual offsetHeight arithmetic that caused
the list to overlap the footer on first render.

* revert: drop save indicator icon changes (#304178)

Reverts the checkmark→save icon change, the loading spinner, and the
green color removal. The save indicator is back to the original
Codicon.check with green color — this will be addressed separately.

* fix: show Built-in/Extension name in tooltips, fix MCP scroll, hide copy path for built-in

- Built-in items show 'Built-in' in tooltip instead of file path
- Non-built-in extension items show 'Extension: {displayName}'
- Copy path context menu hidden for built-in items
- Fix MCP/Plugin scroll: fall back to passed height when container
  is hidden (display:none → clientHeight=0), and re-layout when
  section becomes visible via selectSectionById
- Remove unused resolveExtensionGroupKey method

* fix: use explicit height calculation for list layout instead of flex clientHeight

The flex-based clientHeight approach was unreliable — reflow timing
varied between the component explorer and the real product, causing
the list to clip behind the footer at certain window sizes.

Reverts to explicit height calculation by measuring sibling elements
(search bar, footer, back link) via offsetHeight and subtracting
from total height. For hidden widgets (MCP/Plugin when not the
active tab), layout is skipped since offsetHeight returns 0; the
selectSectionById re-layout handles showing them later.

Removes the rAF wait from the fixture since explicit measurement
doesn't depend on paint timing.

* Stabilize AI customizations editor fixtures

* Add narrow viewport fixture variants and fix widget overflow clipping

* fix: defer layout when offsetHeight returns 0 during display:none → visible transition

When switching sections in the Chat Customizations editor, the
container transitions from display:none to visible. The layout
method fires synchronously before the browser has reflowed, causing
offsetHeight to return 0 for sibling elements. Previously, layout
bailed out entirely, leaving the virtual list with no viewport
height — resulting in missing rows and empty space.

Now defers layout to the next animation frame so measurements
are accurate after the reflow completes.

* skill: add live debugging workflow to chat-customizations-editor skill

* skill: reference launch skill instead of duplicating agent-browser docs

* skill: add fixture gotchas — built-in grouping, editor contribution mocks, screenshot stability

* ci: add blocksCi labels to key customization fixtures for automated regression gating

* refactor: replace type assertion mutations with immutable map in applyBuiltinGroupKeys

* fix: show extension display name instead of ID for agent extension source labels
* Let browser pages handle key events first

* Shortcuts while paused in debugging

* move preload to session

* Remove smoke test (api tests already cover)
* Enhance image carousel to support video playback

* 💄

* 💄 2
@pull pull bot locked and limited conversation to collaborators Mar 25, 2026
@pull pull bot added the ⤵️ pull label Mar 25, 2026
@pull pull bot merged commit 16ea249 into code:main Mar 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants