Osvaldo Ortega

Agents web: Host filtering improvements and mobile improvements (#313062) * sessions: mobile UI improvements — filter chips, terminal view, diff view, tool cards - Feature 1: Add horizontally scrollable filter chips below session list header on phone layout. Chips for Completed, In Progress, and Failed status filters; active state highlights chip. - Feature 2: Improve touch targets for collapsible tool cards in chat on mobile. Minimum 44px height for tool card toggle buttons and always-visible chevron. - Feature 3: Add dedicated full-screen mobile terminal view overlay. Tapping a terminal tool invocation on phone opens a full-screen output view with back navigation, monospace output, and copy/search footer actions. - Feature 4: Add dedicated full-screen mobile diff view overlay. Tapping a file change on phone opens a Diff/File segmented control view with inline unified diff (colored +/- gutters) and full file content tab. All features are phone-only (IsPhoneLayoutContext). Desktop behavior is unchanged. Layer rules preserved: sessions/browser imports only vs/workbench/services; contrib-layer command handlers bridge to local-interface view functions. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * MVP Co-authored-by: Copilot <copilot@github.com> * Review * Clean up * Filter fixes * Search fix Co-authored-by: Copilot <copilot@github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <copilot@github.com>

Mobile agents: home screen improvements (#314226) * Mobile home screen tweaks * Full * Update * Updates * Updates

Prompt timeline rail for the Agents window (#324131) * Add prompt timeline rail to the Agents window Introduce a right-edge "prompt timeline" rail in the Agents window that lets users scan and jump between the prompts they have sent in a chat session, adapted from GitHub's "jump to your messages" affordance. - Self-contained contrib under src/vs/sessions/contrib/promptTimeline, attached per-widget via IChatWidgetContrib (no vs/workbench changes). - Recency-bucketed ticks (pure, unit-tested budgetBucketPrompts) capped so each tick keeps a >=24px hit target (WCAG 2.5.8). - Dense marks at rest that expand on hover/focus, with green/red per-turn diff magnitude and a solid-blue active mark. - Interactive hover card: prompt text, clickable +/- diff stats that open the per-request diff, and per-file drill-down rows. - Per-request diffs via provider-agnostic IChatResponseFileChangesService (agent-host server-computed turn changeset) with editing-session fallback; diff sides are probed so missing checkpoint blobs render as a pure add/delete instead of crashing the multi-diff editor. - Keyboard-first commands: Go to Next/Previous Prompt, Go to Prompt... (quick pick), Review Changes for Prompt, with ARIA announcements. - Gated behind the new sessions.promptTimeline.enabled setting and ChatContextKeys.enabled; rail is created/disposed reactively. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: quiet gray-dense rail with single-mark accordion Replace the whole-rail expand-on-engage behavior with a per-mark accordion. The rail now stays quiet, gray and dense at all times; only the mark under the pointer (or keyboard focus) expands to a >=24px pill and reveals its green/red diff, so the rail no longer becomes a column of slim pills with large gaps when interacted with. - CSS: drop the `.engaged` whole-rail slot expansion and the transform fisheye; expand only `:hover`/`:focus-visible` marks (8px -> 24px slot, fat pill, diff colors). Active mark stays solid blue in every state. - Rail: remove the fisheye magnify, the engaged-state tracking, and the magnitude width buckets; capacity now reserves headroom for a single expanded mark over the dense 8px rhythm, so more prompts fit. - Drop the now-unused --tick-scale/--tick-scale-y custom properties. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: open per-file diff inside the multi-diff, revealed at that file Clicking a file row in the hover card previously opened a standalone single-file diff. Route it through the same multi-file diff we already build for "Review Changes", passing viewState.revealData so it opens revealed at the clicked file. Per-file and whole-prompt review now share one multi-diff experience (and open in the modal editor when enabled), reusing the existing MultiDiffEditorInput without any new UI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: show per-turn diffs against the frozen after-turn snapshot The per-prompt diff was rendering before-turn vs the live working file, so it showed the combined changes of all later turns. The per-turn changeset already carries a frozen "after" snapshot; surface it and diff against it. - Add optional `modifiedContentURI` to IEditSessionEntryDiff: the frozen after-state RHS content, distinct from `modifiedURI` (the live file used for identity / go-to-file). Opt-in, so existing consumers (e.g. the chat "Changed N files" summary) keep their current behavior unchanged. - Populate it in AgentHostResponseFileChangesProvider from the changeset's after-content; extend its test to assert the mapping. - PromptTimelineModel now diffs originalURI (frozen before) against the frozen after snapshot, so review shows only that turn's changes, while go-to-file still opens the live working file. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: add an overview-ruler style selectable by setting Introduce a second rail style and let users choose via the new sessions.promptTimeline.style setting ('compact' | 'overview'). - Extract IPromptTimelineRail as the seam the contrib consumes, and a shared PromptTimelineCard (hover preview) used by both styles. - Rename the existing rail to PromptTimelinePillRail (dense pills). - Add PromptTimelineRulerRail: the session compressed into the rail like the editor overview ruler — proportional marks at their transcript scroll positions, a scrollbar-slider you-are-here thumb, one green "changed code" signal (editorOverviewRuler.addedForeground), focusBorder active, >=24px hit targets. Detail stays in the shared hover card. - Model exposes getScrollLayout()/onDidChangeScrollLayout for proportional positions (per-request offsets via currentRenderedHeight + scrollTop). - Contrib picks the rail class reactively and reuses the enablement swap pattern, wiring setScrollLayout only for rails that support it. All colors come from real theme tokens. Validated: typecheck, eslint, layers, hygiene, unit tests (12 passing). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: position overview-ruler marks from the list layout model The overview-ruler rail placed each mark by accumulating chat items' `currentRenderedHeight`, which the virtualized tree only sets for rows it has actually rendered. Off-screen prompts reported `undefined` (treated as 0), so every not-yet-rendered mark collapsed onto the same position and only spread out as the user scrolled and rows rendered. Read each prompt's top from the list's layout height model instead, the same virtualization-safe source the scrollbar uses (all items have a height via the delegate's `currentRenderedHeight ?? defaultElementHeight`). Threaded a read-only accessor through the layers, all in the list's content-pixel space so the viewport thumb stays aligned: - AbstractTree.getElementTop(location): absolute top (works off-screen), sibling of getRelativeTop. - ChatListWidget.getElementTop / ChatWidget.getElementTop + ChatWidget.scrollHeight. - PromptTimelineModel.getScrollLayout() and _updateActive() now use these instead of accumulating currentRenderedHeight. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: color ruler marks by the turn's add/remove split Ruler marks were a single green "edited" signal, so a delete-heavy turn still showed green. Render the mark as a two-tone bar instead (a green added segment and a red removed segment sized by flexGrow from the turn's diff stat), reusing the pill rail's seg-add/seg-del pattern. Only the non-zero side is appended, so a pure-add turn is fully green and a pure-delete turn fully red, and a 1px min-width keeps the minority color visible on a lopsided split. Segments go transparent while the mark is active so the focusBorder you-are-here color wins. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: fall back to the live file when a turn snapshot is unreadable Per-turn review diffed the frozen before/after turn-checkpoint snapshots so only that turn's changes show. But those checkpoint blobs can be absent (an added file's original, or a pruned/restored session where whole turn checkpoints are gone). When both sides were unreadable every item was dropped and reviewChanges opened nothing at all. _readableSides now still prefers the frozen snapshots but falls back to the live working file for the modified side when the checkpoint blob is unreadable, so review always opens with the best available fidelity. An unreadable side is still dropped so the file renders as a clean add/delete instead of crashing the diff editor. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: drop the pill style, keep only the overview ruler The rail shipped with two selectable styles (dense pills vs overview ruler) behind sessions.promptTimeline.style. We're standardizing on the overview-ruler UX, so remove the pill style and the setting entirely (the on/off sessions.promptTimeline.enabled setting stays). - Delete promptTimelinePillRail.ts and the PROMPT_TIMELINE_STYLE_SETTING / PromptTimelineStyle enum. - The contribution always builds the ruler rail; drop the style-based swap, the capacity->display-budget wiring, and the optional setScrollLayout indirection (now required on the interface). - Model no longer tracks a dynamic display budget; ticks bucket against the constant MAX_TICKS (still caps very long sessions). - Remove the pill-only CSS; the overflow rule now hides the ruler marks. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: address review feedback on grouped-tick diffs and read probes Two fixes from a council review: - getRequestFiles: when a grouped tick edits the same file in more than one prompt, advance the merged entry's diffModifiedURI to the later prompt's after-snapshot (keeping the earliest originalURI) so the opened multi-diff spans the whole tick instead of only the first edit. - _canRead: probe availability with readFile(resource, { length: 1 }) instead of reading whole (potentially large) file contents just to test whether a diff side is readable. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: refresh comments left over from earlier iterations Comment-only cleanup after removing the pill style and the display-budget mechanism: drop the stale "or budget", "(like the pill rail)", and "visual density" references so the docs match the single overview-ruler implementation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: gate the enabled setting behind an experiment Put sessions.promptTimeline.enabled behind the experimentation service via the `experiment: { mode: 'startup' }` property (the sanctioned way for core TS-registered settings; the registry auto-adds the onExP tag). Default is now false so the rail is off until the experiment (or the user) turns it on. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: address PR review feedback and fix class-field init order Review fixes: - Remove the redundant GoToNext/GoToPrevious commands: they duplicated and collided with the built-in chat nextUserPrompt/previousUserPrompt keybindings (same chord, weight, when). Drop the two actions, their command ids, contrib.navigate(), and model.getSiblingTick(). GoToPrompt and ReviewChanges stay. - Fix the active-mark fallback: when scrolled above the oldest prompt, highlight the oldest tick (ticks.at(0)), not the newest (ticks.at(-1)). - Replace the getComputedStyle probe + unreverted host `position` mutation with a lifecycle-scoped `.prompt-timeline-host` class removed on teardown. - Give the overview-ruler toolbar a proper keyboard model: aria-orientation, roving tabindex (a single Tab stop) and Arrow/Home/End navigation between marks. Also assign `_sessionResource` in the constructor instead of a field initializer: it reads `this.widget`, which under useDefineForClassFields is not yet assigned when field initializers run (fixes the define-class-fields-check / TS2729). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: stop showing the unreliable per-prompt timestamp Agent-host sessions don't record per-turn timestamps; the transcript is reconstructed via ChatModel.addRequest(), which stamps Date.now(), so every prompt showed ~the current time. Since the real send times aren't available to recover, drop the absolute time from the hover card and the "Go to Prompt" picker. Grouped ticks still show their prompt count; the prompt text and diff stats (both reliable) are unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: act on PR review (remove ReviewChanges action; rename modifiedContentURI) - Remove the ReviewPromptChangesAction command: per-prompt review is already available from the hover card, so the palette/keyboard entry point was redundant. Drops the command id and the now-unused contrib.getActiveTick() / contrib.reviewChanges() bridge (model.reviewChanges stays, wired to the card). - Rename IEditSessionEntryDiff.modifiedContentURI -> modifiedSnapshotURI: it's the frozen after-turn snapshot content, and "snapshot" reads clearer next to modifiedURI (the live file) than "content". - Clarify in comments that this field is distinct from the agent-host checkpoint- ref readability fix (#323932): that made the snapshot blobs readable; this carries which snapshot to diff against so a per-turn review shows only that turn's changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

agents mobile: redesign changes/diff overlays with master-detail navigation and syntax highlighting (#314433) * Improve sessions list rendering on phone layout - Phone-aware row height (76px) in SessionsTreeDelegate, refreshed on IsPhoneLayoutContext changes. - Larger title/details fonts and roomier padding under .phone-layout. - Card-style row margin/border-radius and centered icon alignment. - Per-row toolbar always visible on touch; pulled out of the title-row flow with absolute positioning so 44px tap targets do not shift the details row. - Stop pointer/click propagation at the toolbar container so action taps do not trigger the row open handler. * Address PR review - Stop Monaco Gesture tap from reaching the list when the user taps an inline action on touch (DOM stopPropagation only covers mouse). - Drop unsafe vertical margin on the absolutely positioned list row; any inter-row spacing lives inside the row via padding/border. - Refresh stale doc comments (row geometry, iteration cost, no-op '2-line wrap' description). * agents mobile: redesign changes/diff overlays with master-detail nav and syntax highlighting - Add MobileChangesView: full-screen overlay listing all changed files with codicon file-type icons, A/M/D pills, colored +N/-N counters and reactive live updates via autorun over sessionsManagementService.activeSession.changes - Rewrite MobileDiffView: prev/next navigation between sibling files (chevrons + horizontal swipe gesture), Monaco tokenization with regex fallback (4 token kinds: comment/string/keyword/number), 3px colored left-edge bar on added/removed lines, sticky hunk headers, horizontal scroll fix - Wire title-bar Changes pill to MOBILE_OPEN_CHANGES_VIEW_COMMAND_ID; single- file sessions skip the list and open the diff directly (fewer taps) - Add mobileDiffColors.ts: mobile-specific registerColor registrations for agentsMobileDiff.{added,modified,deleted}Foreground; imported as side-effect from mobileChangesView.ts so colors are available before any view renders - Fix disposable leak: per-row gesture/click listeners use a rowsStore that clears on every renderList call instead of accumulating in viewStore - Fix merged-hunk rendering: track sub-change list per group, emit context rows between adjacent sub-changes so unchanged lines aren't misclassified - Update vscode-known-variables.json with new --vscode-agentsMobileDiff-* theme tokens and --mobile-diff-{tok,added,modified,deleted} CSS vars Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * agents mobile: address Copilot reviewer feedback on PR #314433 - Fix deletion bug in single-file shortcut: reuse toRow/rowToDiffData (now exported) from mobileChangesView.ts so modifiedURI stays undefined for IChatSessionFileChange2 deletions instead of incorrectly falling back to the v2 uri field - Localize Action2 titles in mobileOverlayContribution.ts via localize2() - Correct MOBILE.md: MobileChangesView uses codicons not ResourceLabels; MobileDiffView uses async tokenizeToString + injected colorMap <style>, with regex fallback emitting CSS-class spans (not inline style spans) - Rewrite regex tokenizer comment to accurately describe the class-based approach and per-theme CSS variable layering in mobileOverlayViews.css Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fix new cloud session disappearing and misgrouping after commit (#326230) * Fix new cloud session disappearing and misgrouping after commit The Agents window (vs/sessions) drove a new cloud (RemoteNewSession) agent session through the wrong commit path, causing it to disappear ~5s after delegating, and then group under "Unknown" / flip its repo label once it did appear. Three related fixes: 1. Wait for the committed resource on cloud sessions. A cloud session commits a new resource mid-request (untitled -> /task/<id>), like Copilot CLI, so _sendFirstChat now routes RemoteNewSession through _waitForCommittedSession instead of waiting on the stale untitled resource (which timed out and removed the session). The wait is also deferred (skips the response-complete race, longer timeout) because the cloud commit is delayed by a confirmation round-trip and network delegation. 2. Attach owner/name metadata to PR-less task cards so a freshly created task groups under its repo instead of "Unknown"/"Other" until a PR resolves. Repo identity is hoisted from PullArtifactRef.repo to CloudSessionData.repo (a task has a repo regardless of a pull artifact); resolvePullArtifact now takes repo as an explicit argument. 3. Consolidate the repo label. Both the new-session workspace and the committed session adapter now derive an owner/repo label via a shared githubRemoteRepoLabel helper, so a cloud session stays in the same group before and after commit (no more microsoft/vscode -> vscode flip). Adds a regression test for the cloud commit-swap. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Comments --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

sessions: prompt timeline UX improvements (#326023) * sessions: prompt timeline rail becomes one scrollbar that fans in on scroll/hover Reshape the prompt timeline into a single scrollbar lane instead of a pill column sitting next to the transcript's native scrollbar. At rest it is a plain scrollbar (the rail draws its own thumb and the transcript's native vertical slider is hidden while the rail is active). On a deliberate gesture the thumb recedes and the prompt pills fan in over the same lane with a macOS-dock style fisheye: - Hard/fast scroll (wheel-velocity gated, capture phase so it works mid-content not just at the scroll limits) blooms the fan centred on where you are, and it glides continuously with the viewport as you keep scrolling. The reveal is gated on a real scrollTop change, so flicking against the top/bottom limit - or programmatic virtualization nudges - never opens it. - Hovering the lane blooms the fan and lets it follow the cursor; dragging the lane scrubs the transcript; clicking a pill still jumps; keyboard focus reveals the pills. It quietly collapses back to a plain scrollbar after a short linger once you stop scrolling and are not hovering. - Pills are laid out as an evenly-spaced dock centred in the lane (stable under virtualization) rather than scattered by content position; the two-tone green/red diff decoration and the hover card are preserved. Also reduces forced reflows on the hover/scroll paths (position the card and lane from cached geometry instead of getBoundingClientRect) and avoids churning the linger timer every scroll frame. Reduced-motion disables the fisheye but still reveals the calm dock. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * sessions: address prompt timeline PR feedback - normalize wheel deltas via StandardWheelEvent so line-mode devices trigger the fan - keep the native scrollbar when the rail is too narrow (below MIN_HOST_WIDTH) to replace it - use generic mouse listeners for lane scrub + mark guard so touch/iOS works - track keyboard focus: reveal a calm dock (:focus-within) with the fisheye suppressed - scale scrollTop into the estimated coordinate space before interpolating the fan focus - map the thumb/scrub math onto the list viewport height (ScrollbarState-style) - use --vscode-cornerRadius-circle for the thumb radius - extract the hard-wheel detector into a named helper Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * distance update --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Cloud agent: Create/Open pull request toolbar actions for settled tasks (#321520) * Cloud agent: Create/Open pull request toolbar actions for settled tasks Adds chat-input toolbar actions for v2 (Task API) cloud agent sessions: - "Create pull request" for a settled, PR-less task; the backend resolves the repo from the task (html_url, falling back to a GitHub repo-by-id lookup) and creates the PR, then re-applies the toolbar gates in place. - "Open pull request" (icon-only) once the task has a PR, opening it in the browser; resolves both task- and PR-keyed session resources. Also: compare-based changed files for settled PR-less tasks, reactive gate re-evaluation on task settle / PR creation, and resolver-path telemetry. Core: icon-only rendering for the open-PR action in the session changes toolbar. * Address review: fix create-PR response test shape, stale wording, telemetry name - Test FakeTaskApiClient now returns the real AgentTaskCreatePullRequestResponse shape ({ id, number, repository_id }) instead of the stale { pull_request }. - Reword the no-reflect warning and create-handler doc that wrongly implied a refresh (the handler updates the toolbar gates in place). - Rename the resolver telemetry event back to copilotcloud.pullArtifactResolve. - Drop the 'browser-open glyph' wording now that the open-PR icon is git-pull-request.

Cloud Agent: Tasks API experimental setting and backend (#317206) * Cloud Agent: Tasks API experimental setting and backend Introduces a CloudAgentBackend seam with two implementations: - JobsApiBackend wraps the existing sweagentd Jobs API (default, no behavior change). - TaskApiBackend implements the new Mission Control Task API and currently uses a StubTaskApiClient pending CAPI routing. Selection is controlled by the new 'github.copilot.chat.cloudAgentBackend.version' setting ('v1' = Jobs API, 'v2' = Task API). The setting is tagged experimental and defaults to 'v1', preserving existing behavior for legacy users. * Review comments * Address PR review feedback - TaskArtifactPullData: add optional 'number' field separate from db id - Add ListTaskEventsOptions for event-specific pagination/filters - Relax sendFollowUp success check (only undefined is failure) - Drop unused _configurationService field; read config locally in ctor - Document that cloudAgentBackend.version requires reload

Cloud agents on task API - Initial part (#317659) * Update * Review comments * Refactor * lock revert * Update * cleanup

Sessions: chat timeline variant (#327065) * Agent Host changes for osortega/agents/chat-session-timeline-ui-proposal * sessions: add dock rail variant to prompt timeline Adds a 'dock' style to sessions.promptTimeline.rail: a minimal left-edge handle (one dot per prompt, capped at 50 with an overflow marker) that opens an accessible flyout listing every prompt. Changes the setting to a string enum (off|ruler|dock), feeds the dock an unbucketed per-prompt list so entries are individually selectable, and wires the handle as a touch/keyboard disclosure (Gesture + click/tap, aria-haspopup/expanded/ controls) that closes on activation or Escape and restores focus. Adds aria-current to the active row, pairs the selection colours, snaps to design-system radius/spacing tokens, and documents the surface in the Sessions chat accessibility help. The sticky header label now navigates to the pinned prompt instead of opening Go to Symbol. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Live streaming for Task API cloud agent sessions (#319077) * Live streaming for Task API cloud agent sessions Adds TaskTurnStreamer to incrementally push v2 Task API events into vscode.ChatResponseStream for both initial open (mode: 'current') and follow-up sends (mode: 'next'), mirroring the v1 JobsApiBackend SSE flow. Key design (modelled after github-ui/agent-sessions): - Eager tool rendering from assistant.message.toolRequests, since tool.execution_complete does not fire for many agent-host-synthesised setup ops (run_setup, run_custom_setup_step). - Per-messageId content dedup: Task API sends progressive snapshots, not deltas; only the suffix is fed to the renderer. - Intermediate assistant.message.content (content + toolRequests in the same event) is suppressed; only pure-text assistant messages (the final reply of the turn) are rendered as markdown. - Default 'Working...' progress label until the first assistant.intent arrives. - Bounded poll loop: exits on terminal state, cancellation, or MAX_CONSECUTIVE_FETCH_FAILURES. - Follow-up (mode: 'next') only advances when task.sessions.length grows, to avoid premature exit when trailing prior-turn events arrive between the snapshot and sendFollowUpToTask resolving. ChatSessionContentBuilder updates: - buildTaskHistory: single-pass turn split + bootstrap event suppression. - buildTaskResponseTurn: mirrors the live rules (eager tool rendering, intermediate-narration suppression, dedup with live cards) so the refresh() after streaming is idempotent. - parseToolCallDetails: friendly cards for run_setup, run_custom_setup_step, report_intent. - toFileLabel: handles v1 (/home/runner/work/...) and v2 (/tmp/workspace/..., /workspace/...) layouts. * Address PR review: parallelize fetches, fix stale comment, add Task API history tests - Parallelize fetchTask/fetchEvents with Promise.all in both the main poll loop and _waitForTurnStart so each poll only pays the slower of the two requests. - Update StreamBaseline JSDoc to reflect that phase 1 (mode: 'next') unblocks only on turnCount > priorTurnCount, not 'any unseen event'. - Add unit tests covering the new Task API history rules: bootstrap suppression, turn-boundary split, eager tool rendering with execution_complete dedup, intermediate-narration suppression, and the synthesised-turn fallback when no user.message has arrived. * Readme update

Instrument Cloud Agent backend v1/v2 rollout behind an experiment (#321818) * Instrument Cloud Agent backend v1/v2 rollout Put the `chat.cloudAgentBackend.version` toggle (v1 = Jobs API, v2 = Task API) behind an experiment so the rollout can be ramped and rolled back remotely via ExP, and add an arm-tagged telemetry surface so the two backends emit identical funnel and guardrail signals for apples-to-apples monitoring. Previously v1 was partly instrumented and v2 emitted nothing, making any A/B comparison impossible. A shared `ICloudBackendInstrumentation` is now injected into both backends and stamps every event with `backendVersion`, covering session creation, activation, follow-up, and per-operation errors, plus matching OTel metrics. v2's silent catch sites now report through it, and `TaskApiError` carries the HTTP status for error classification. Legacy v1 backend events are consolidated into the shared surface and kept for dashboard continuity. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Rename "arm" to "backend version" in cloud telemetry comments "arm" is A/B-experiment jargon not used elsewhere in the codebase. Swap it for "backend version" in comments, docstrings, and GDPR comment text to match the existing `backendVersion` / v1 / v2 vocabulary. Comment-only change; no identifiers or behavior affected. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Address cloud backend rollout PR review comments - Move backend_version attribute to the canonical github.copilot.* namespace - Derive numeric HTTP status inside operationFailed so the status measurement is populated - Carry HTTP status on v1 (Jobs API) invalid-job errors via JobsApiError so v1 create failures classify by status - Only emit cloud pr_ready.count for v1 (Task API activation is a first turn, not a PR) - Add unit tests for the new cloud metrics - Document the new cloud operation/error metrics and backend_version attribute Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix TaskApiBackend construction in fetchSessionList specs Three fetchSessionList test cases constructed TaskApiBackend without the required instrumentation argument, failing the CI typecheck (TS2554). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Support Go to Symbol (⇧⌘O) for chat sessions (#326739) * Support Go to Symbol (⇧⌘O) for chat sessions Adds a chat outline so "Go to Symbol in Editor" lists each user request as a navigable symbol, jumping to it in the chat. Works for chat editors (via IOutlineService) and for a focused chat widget including the Agents window. - New ChatOutline / ChatOutlineCreator registered with IOutlineService so chat editors light up Go to Symbol, the Outline pane and Breadcrumbs. - GotoSymbolQuickAccessProvider prioritizes a focused chat widget in provide() so it wins over an active file editor opened side by side, and detects focus via isAncestorOfActiveElement. - Request labels are derived like the chat list renders them (followup message, else parsed request parts), not messageText which agent-host sessions leave empty. Codicons are embedded inline in the quick pick label (iconClasses can't render codicons in this path). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Address PR feedback for chat Go to Symbol - Fall back to an attachment summary (matching the chat list) for attachment-only requests before the numbered label. - Move ChatOutlineCreator into its own file and re-export ChatOutline through chat.ts so codeEditor consumes the contribution's API file instead of importing chatOutline.ts directly. - Only refresh the outline when request entries actually change, so streamed response updates don't rebuild it repeatedly. - Add a chatOutline unit test suite (label derivation incl. empty text and attachment-only requests, icon escaping, change detection, reveal/preview). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add account indicator to mobile titlebar in agents workbench (#312437) * Add account indicator to mobile titlebar in agents workbench The mobile titlebar now shows a contextual right slot: - Welcome / new session screen: [☰] [title] [account] - In a chat session: [☰] [title] [+] The account indicator mirrors the desktop titlebar account widget: - Shows GitHub avatar when signed in, codicon fallback otherwise - Displays dot badge for quota warnings/errors - Tapping opens a panel with account info, copilot status dashboard, and sign-in/sign-out actions - Touch-friendly: 44px minimum touch targets, touch-action: manipulation Also extracts the AccountMenu MenuId to Menus.AccountMenu for sharing between the desktop and mobile implementations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix account panel immediately dismissing on mobile tap The hover service registers a document-level mousedown listener for sticky hovers that dismisses the hover when clicking outside it. On mobile, the mousedown fires in the same event cycle as the click that opens the panel, so the hover was created and then immediately dismissed. Defer the showInstantHover call to the next animation frame so the mousedown event has completed before the sticky listener is registered. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Use DOM overlay for mobile account panel instead of hover service The hover service's sticky mousedown/mouseout handling doesn't work reliably on mobile touch devices. Replace it with a simple DOM overlay pattern: a full-screen backdrop that catches taps to dismiss, with the panel absolutely positioned below the top bar. This is more appropriate for mobile UX — touch interactions work natively without fighting the desktop hover infrastructure. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix account panel styling by appending inside workbench container The panel was appended to document.body, outside the .agent-sessions-workbench container, so none of the existing .sessions-account-titlebar-panel-* CSS selectors matched. Append the backdrop and panel inside the workbench container so the desktop panel styles (header, actions, dashboard, separators) apply correctly on mobile. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Redesign mobile account panel as full-screen bottom sheet Replace the desktop-style dropdown with a native mobile bottom sheet: - Full-screen sheet with header (title + close button) - Profile section with large avatar + name + provider - Copilot status dashboard when signed in - Large touch-friendly action rows (52px height, 16px font) with icons for Sign In/Out and Settings - Proper safe area insets for notched devices - Scrollable content area with overscroll containment - All new CSS classes (mobile-account-sheet-*) purpose-built for mobile — no reuse of desktop hover panel styles Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix scopeListOrRequest is not iterable in signIn When product.json defaultChatAgent.providerScopes is empty, scopes[0] evaluates to undefined. The spread in createSession then fails with is not iterable. Default to an empty array. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add OAuth callback support to sessions web dev server The sessions dev server was missing two things needed for GitHub OAuth sign-in to work: 1. A /callback route serving the OAuth redirect page (callback.html) that writes the auth code to localStorage 2. A urlCallbackProvider passed to the workbench create() options that polls localStorage for the auth code and delivers it back to the GitHub authentication extension Without these, the OAuth flow would complete on GitHub's side but the auth code was never delivered back to the workbench. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Revert "Add OAuth callback support to sessions web dev server" This reverts commit 4034dd5b0b8add4e94e47c38c08ee0fcb22ffcfe. * Fix web walkthrough auth scope mismatch and token expiry Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Use product config scopes for walkthrough sign-in instead of hardcoded The previous fix used defaultAccountService.signIn() which blocks on the init barrier — the DefaultAccountProvider may not be set yet when the walkthrough runs on first launch. Instead, keep using authenticationService.createSession() directly (which doesn't depend on the provider lifecycle) but read the scopes from productService.defaultChatAgent.providerScopes[0] instead of hardcoding ['repo', 'user:email', 'read:user']. This ensures the session has the 'workflow' scope that the account provider needs while avoiding the provider initialization race. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add fallback scopes when product config is unavailable If productService.defaultChatAgent.providerScopes is undefined (e.g. in vscode-dev web context where product config may not include it), fall back to the full scope set including workflow. Without this, an empty scopes array would create a session that the tunnel discovery code cannot find (it filters for user:email). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Listen to auth session changes directly for mobile account indicator The mobile account indicator relied solely on defaultAccountService.onDidChangeDefaultAccount, which depends on the DefaultAccountProvider completing its async initialization (extension registration, entitlement resolution). On mobile web, the walkthrough sign-in creates the session before the provider is fully initialized, so the event is missed. Also listen to authenticationService.onDidChangeSessions to pick up new GitHub sessions immediately when they're created, regardless of the DefaultAccountProvider lifecycle. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix account indicator showing Sign In after walkthrough on localhost Two fixes for the mobile account indicator not reflecting auth state after the walkthrough completes: 1. Fall back to reading GitHub sessions directly from IAuthenticationService when DefaultAccountService returns null. This covers the window between session creation (walkthrough) and DefaultAccountProvider initialization (extension registration). 2. When we have an account name from auth sessions but entitlement is still Unknown (not yet resolved), treat it as Unresolved rather than Unknown. This prevents getAccountTitleBarState() from returning 'Agents Signed Out' when the user is actually signed in but the entitlement service hasn't caught up yet. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Hide Copilot setup dashboard in mobile account sheet when unresolved The ChatStatusDashboard shows 'Set up Copilot to use AI features' when the entitlement is Unknown or Available (new user). This setup flow doesn't apply in the agents app which has its own walkthrough. Only show the dashboard when entitlements have fully resolved to a known plan (Free, Pro, etc). When still resolving, the profile section already shows the account name which is sufficient. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Apply same auth state fixes to desktop TitleBarAccountWidget The desktop account widget had the same issue as the mobile one: after walkthrough sign-in, it showed 'Agents Signed Out' because the DefaultAccountProvider hadn't initialized yet. Apply the same three fixes from the mobile indicator: 1. Listen to authenticationService.onDidChangeSessions directly 2. Fall back to reading GitHub sessions when defaultAccountService returns null 3. Override Unknown -> Unresolved when an account name is present to prevent showing signed-out state during entitlement resolution Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add INFO-level logging to DefaultAccountProvider for debugging Upgrade key log messages from debug to info level to trace the account resolution flow in the browser console: - Session count and matching results in findMatchingProviderSession - Account name and scopes when sessions are found - Final account state after initialization completes Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Use console.log for DefaultAccount debug logging Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Accept any session when providerScopes is empty On vscode.dev the product config may not include providerScopes, resulting in an empty allScopes array. The scope matching loop never executes, so no sessions match even though valid sessions exist. When no scopes are configured, accept any available session. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Hide 'Use AI Features' button when AI features are disabled When users disable AI features via chat.disableAIFeatures, the account section in the Agents window should not show the 'Set up Copilot to use AI features' button. This change checks sentiment.hidden in addition to the other disabled states to properly hide this UI element. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Hide Copilot setup dashboard in desktop account panel when unresolved Same fix as the mobile account sheet — don't show the ChatStatusDashboard when entitlement is Unknown or Available, as it renders a 'Set up Copilot to use AI features' prompt that doesn't apply in the agents app. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update * Clean up Co-authored-by: Copilot <copilot@github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <copilot@github.com>

feat(sessions): mobile bottom-sheet picker for opened-chat input (#314571) * feat(sessions): mobile bottom-sheet picker for opened-chat input Mirrors the new-chat empty-state mobile experience for the workbench ChatInputPart used in vscode.dev/agents on phone-classified viewports: the desktop Mode + Model pickers are replaced with a single chip that opens a unified bottom sheet (Agent Mode + Model rows). The bottom-sheet primitive (showMobilePickerSheet) stays in vs/sessions. A new workbench service (IChatPhoneInputPresenter, no-op default singleton) lets sessions register a phone-only impl that opens the sheet; ChatInputPart consults the presenter from its inline actionViewItemProvider to swap in a chip when active. Desktop behavior is unchanged. Files added: - src/vs/workbench/contrib/chat/browser/widget/input/chatPhoneInputPresenter.ts - src/vs/workbench/contrib/chat/browser/widget/input/media/chatPhoneInputPresenter.css - src/vs/sessions/contrib/chat/browser/mobileChatPhoneInputPresenter.ts Files modified: - src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts - src/vs/sessions/sessions.web.main.ts Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update * Address council review: fix stale session refs, wrong-widget target, dedupe storage key * Error when merging * Refactor: use MobileAgentHostModePicker subclass for phone path * re-structure * Fixture * Review comments --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add inline 'Create pull request' confirmation for completed tasks without PRs (#319629) * Add inline "Create pull request" confirmation for completed tasks without PRs * Update * Revert monaco change * Review comments * Fix * Revert

Move agent host picker to sessions sidebar (web desktop) (#313619) * Move agent host picker to sessions sidebar (web desktop) On web desktop (vscode.dev/agents), surface the agent host picker in a new toolbar at the bottom of the sessions sidebar instead of the titlebar's left layout. The new widget is modeled on AICustomizationShortcutsWidget but is always expanded -- no collapse chevron, no storage key. Scope: - Web desktop: picker moves to the sidebar (new SidebarAgentHost menu). - Web phone: unchanged -- still rendered in MobileTitlebarPart. - Electron: unchanged -- no host picker, never had one. Visual alignment: HostFilterActionViewItem gains a HostFilterAppearance ('titlebar' | 'sidebar') parameter so it can render the same Monaco Button + .sidebar-action-button shell used by CustomizationLinkViewItem. The sidebar appearance lays out the picker button as [remote-icon] [host-name (flex:1)] [chevron-down], with the connect / disconnect / re-discover indicator as an independent 26x26 sibling control to the right. When there are no hosts the right slot becomes a refresh button that triggers rediscover(). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update src/vs/sessions/contrib/remoteAgentHost/browser/hostFilterActionViewItem.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update src/vs/sessions/contrib/sessions/test/browser/agentHostShortcutsWidget.test.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Address PR #313619 review feedback - Drop the icon-by-state enumeration from the HostFilterActionViewItem class JSDoc. - Sidebar picker no longer no-ops with 0 hosts: pass the click event through to `_showMenu`, which already routes to `rediscover()` when no hosts are known. Same affordance as the dedicated refresh slot. - Widen `_showMenu(e)` to accept `Event`; this avoids fabricating a synthetic `MouseEvent('click')` that would anchor the context menu at (0,0) on touch/gesture activations. `dom.isMouseEvent(e)` keeps the runtime check. - Gate the `AgentHostShortcutsWidget` mount on `IsSessionsWindowContext && !IsAuxiliaryWindowContext && !IsPhoneLayoutContext` instead of `isWeb && !isPhoneLayout(...)`. Prevents an empty toolbar shell in auxiliary windows and aligns the runtime gate with the agents-window context the menu items target. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Avoid querySelector in HostFilterActionViewItem (sidebar) Hold stable references to the leading host icon and trailing chevron as `_sidebarLeadingIcon` / `_sidebarTrailingIcon` fields, created once in `_renderSidebar()`. `_renderSidebarButtonAffordances` now attaches/detaches the chevron via `appendChild` / `remove` on the stored reference instead of `querySelector(':scope > ...)`, fixing the `no-restricted-syntax` ESLint warnings reported by CI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Prompt timeline: reduce scroll jitter and polish the rail (selection, spacing, performance) (#325041) * Prompt timeline: reduce ruler scroll jitter with adaptive height estimate The ruler positioned marks from the chat list's height model, where every un-rendered row is guessed at a flat 200px default. Real turns vary widely (short prompts, tall/variable responses), so as rows render and get measured the list's tops snap around and the marks visibly drift while scrolling a fresh session. Compute the marks' positions from our own per-item height model instead: measured rows use their real currentRenderedHeight; un-measured rows use a running average of measured heights of the same kind (prompt vs response, seeded with priors). Marks land near their final position immediately and barely move, converging exactly with the list once every row is measured. Keep the viewport thumb in the list's own scrollTop/scrollHeight space so it stays aligned with the native scrollbar (the marks' estimated space would separate from it while heights settle and look like a second scrollbar). Add a short 'glide' transition, enabled one frame after each structural rebuild, so any residual drift animates smoothly without new marks sliding in from the top; disabled under prefers-reduced-motion. Also improve the keyboard focus indicator: grow the focused mark to a rounded pill hugged by the focus ring instead of a bare outline on the 2px bar. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: polish rail selection, spacing, and scroll performance Builds on the scroll-jitter work with a round of UX and performance fixes: - Selection: replace the glowy box-shadow/backing-lane with a crisp border on the pill, and give keyboard focus a distinct dashed ring so a focused mark is never mistaken for a second selection. There is only ever one selected mark (it follows the viewport and click-reveal). - No !important: the workbench outlines any focused [tabindex=0] button, which boxed the mark; suppress it by qualifying our rule through the marks container to win on specificity instead. - Scrollbar: reserve a gutter (derived from the real scrollbar width) so the marks sit left of the native scrollbar and it stays grabbable. - Spacing: reserve room on the transcript's right edge so message content keeps a guaranteed minimum gap from the marks, derived from the rail width via CSS variables rather than a magic number. - De-cluster marks: prompts that sit close in content space could let the 24px hit targets overlap; space adjacent mark centres apart (new pure promptTimelineLayout.ts, unit-tested). - Performance: the rail relayout read clientHeight on every scroll event (a forced reflow -> ~19% of active CPU while scrolling). Cache the height (refreshed only on resize) and coalesce scroll relayouts to one per animation frame; a re-profile shows relayout self-time drop from ~968ms to ~2ms. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: define the scrollbar gutter in CSS instead of JS Review feedback: the gutter was set from JS (DEFAULT_SCROLLBAR_SIZE + gap), but the list scrollbar is a stable 10px and the rail's other geometry already lives as CSS variables on .prompt-timeline-host. Move it there as --prompt-timeline-scrollbar-gutter and drop the runtime setProperty, the constant, and the base import. Not related to the focus-outline specificity fix. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Prompt timeline: simplify spaceMarkCenters even-distribution branch Review feedback: the even-distribution branch is only reached when (n - 1) * minGap > hi - lo, and hi > lo is already guaranteed, so n >= 2 there. The n === 1 guards were dead code; remove them. Single-mark inputs go through the forward/backward clamp pass, which the unit tests still cover. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Improve Copilot Cloud Sessions error handling and messaging (#325196) Agent Host changes for osortega/agents/copilot-cloud-sessions-v2-backend

Mobile chat (1/10): infra primitives — bottom sheet body, visualViewport, longPress, pulldown, edge-swipe (#318647) * Mobile infra: bottom-sheet body, visualViewport, longPress, pulldown, edge-swipe Foundation primitives used by the rest of the mobile-chat stack: - mobilePickerSheet.ts: extract shared sheet shell + add showMobileContentSheet for arbitrary-body overlays (used by confirmation/plan-review/anchor-peek/code-block sheets in follow-ups). - mobileVisualViewport.ts: IMobileVisualViewport service publishing keyboardHeight observable + --vscode-keyboard-height CSS var + KeyboardVisibleContext. Replaces the inline visualViewport block in workbench.ts. - longPress.ts: reusable installLongPress helper (powers chat-row action sheets in a follow-up). - mobilePulldownDismiss.ts: pull-down-to-dismiss gesture used by the mobile diff/changes/code-block overlays in follow-ups. - mobileEdgeSwipe.ts: left-edge swipe gesture that opens the sidebar drawer; integration with workbench.ts ships in the overlays PR. - vscode-known-variables.json: register --vscode-keyboard-height so stylelint accepts CSS that consumes it. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix long-press click suppression and restore session-view stylelint vars * Restore activeSessionView/inactiveSessionView stylelint vars * Address PR review: pointer capture for gestures, fix keyboard-height comment * Use const shell with forward reference in mobile sheet helpers --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>