Agents: fix sessions workspace picker

[sandy081]

Fixes several issues with the sessions workspace picker in the Agents window.

Changes

Filter copilot worktrees from VS Code recents

Folders whose name starts with copilot- are filtered out from the VS Code recent folders list, as these are internal worktrees and not user-facing workspaces.

Preserve VS Code recents order

Removed the alphabetical sort that was previously applied to VS Code recent folders. Entries now retain the order VS Code stores them in (most recently used first).

Stable picker width

When maxWidth is set on the action list options, DOM-based width measurement is now skipped entirely and the fixed value is used directly. Previously, the measured width varied with scrollbar presence (which depended on list height which depended on window height), causing the picker width to fluctuate when the window was resized.

Scrollbar clearance

Row width is set to calc(100% - 10px) so content never extends under the 10px Monaco overlay scrollbar track.

Group title / toolbar spacing

In inline-description mode, the flex row gap is set to 0 with explicit margins on individual elements. This keeps the group title ("Folders") and remove button (×) flush with just a small readable gap, while still giving the group title margin-right breathing room when the toolbar is hidden.

Bottom gap

A 30px gap is maintained between the bottom of the picker and the window edge.

Document entry ordering

Added JSDoc on _buildItems() explaining the sorting: own recents first, VS Code recents after, groups ordered by first appearance, within-group order preserved.

[Copilot]

Pull request overview

This PR fixes several UX and ordering issues in the Agents window “sessions workspace picker”, primarily by stabilizing picker sizing/spacing and adjusting how recent workspaces are sourced and ordered.

Changes:

• Adds a fixed-width option for action lists and uses it in the sessions workspace picker to prevent resize-induced width fluctuations.
• Updates action widget styling to keep row content clear of the overlay scrollbar and refines inline-description spacing.
• Adjusts sessions workspace picker recents behavior: preserves VS Code recents order and filters out certain internal Copilot worktrees.

Show a summary per file

File	Description
src/vs/sessions/contrib/chat/browser/sessionWorkspacePicker.ts	Preserves recents ordering, filters Copilot worktree recents, adds picker JSDoc, and sets a fixed picker width via action list options.
src/vs/platform/actionWidget/browser/actionWidget.css	Tweaks action-widget row sizing and inline-description layout to improve scrollbar clearance and spacing.
src/vs/platform/actionWidget/browser/actionList.ts	Introduces an options-driven “fixed width” path (via maxWidth) and adjusts height calculations to maintain a window-edge gap.

Copilot's findings

Comments suppressed due to low confidence (2)

src/vs/platform/actionWidget/browser/actionList.ts:1773

• There’s test coverage for ActionListWidget filtering, but the newly introduced maxWidth/fixed-width behavior in ActionList.layout() (and the new bottom-gap height behavior) is untested. Adding a small browser test that asserts the returned width/layout behavior when maxWidth is set (including in a constrained container) would help prevent regressions.

		// When a fixed maxWidth is provided, skip DOM measurement entirely.
		// DOM-based measurement varies with scrollbar presence (which depends on
		// the list height), causing the width to fluctuate as the window is resized.
		let computedWidth: number;
		if (this._options?.maxWidth !== undefined) {
			computedWidth = this._options.maxWidth;
		} else {
			computedWidth = this._widget.computeMaxWidth(minWidth);
		}
		this._cachedMaxWidth = computedWidth;
		this._widget.layout(listHeight, this._cachedMaxWidth);

src/vs/platform/actionWidget/browser/actionList.ts:1771

• When options.maxWidth is set, layout() forces the widget width to that value without respecting minWidth or the widget’s CSS constraints (e.g. .action-widget { max-width: 80vw; }). In narrow windows this can cause a mismatch where the DOM clamps the widget width but the list is laid out as if it were wider, leading to clipped content/incorrect scrolling. Consider clamping the fixed width to the effective available width (e.g. viewport/CSS max) and also honoring the effective min width.

		if (this._options?.maxWidth !== undefined) {
			computedWidth = this._options.maxWidth;
		} else {
			computedWidth = this._widget.computeMaxWidth(minWidth);
		}

• Files reviewed: 3/3 changed files
• Comments generated: 3

[github-actions]

Screenshot Changes

Base: 677ee018 Current: 0cced04a

Changed (2)

editor/codeActionList/SimpleQuickFixes/Light

Before	After

editor/codeActionList/SimpleQuickFixes/Dark

Before	After

[hediet]

Can you confirm if the fixture change is intentional?

I believe it is caused by that:

/* Leave clearance on the right for the vertical scrollbar track */
width: calc(100% - 10px);

(disabled auto-merge as precautionary measure, feel free to merge nontheless)

[sandy081]

Hey - this is a great catch - I will fix it