Ben Villalobos

Automations: Management UI (create/edit dialog, list widget, ChatInputPart integration) (#323914) * feat(automations): add management UI — list widget, create/edit dialog, modal polish Adds the complete automations management interface: - AutomationsListWidget: WorkbenchList-based view with run status, dynamic row heights - Create/Edit dialog: ChatInputPart-hosted prompt editor with ghost text - WorkspacePicker integration for folder selection - Isolation mode dropdown (Worktree/Folder via ActionListWidget) - Static 'Copilot CLI' harness chip (future picker placeholder) - Schedule configuration (Manual/Hourly/Daily/Weekly with time/day pickers) - Permission level and model selection via ChatInputPart toolbar - CSS styling matching form field backgrounds to prompt editor - PlaceholderTextContribution registered on ChatInputPart editor - List widget tests Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(automations): polish isolation chip and fix its default label Add a hover highlight and remove the redundant chevron so the isolation chip matches the new-session picker. Fix the chip showing 'Folder' for an unset isolation mode: the label now derives from !== 'workspace', so an undefined mode reads as 'Worktree' to match the picker, provider, and run defaults (previously it lied and the session ran as a worktree). * fix(automations): replace unknown fontWeight-medium with semiBold The stylelint rule flags --vscode-agents-fontWeight-medium as unknown. Only regular and semiBold are registered in vscode-known-variables.json. Use semiBold for form labels — visually equivalent for this context. * fix(automations): address review findings — CCR patterns, test gaps, bug fixes - Remove 33 em dashes from comments - Compress 5 over-length JSDoc blocks to 1-2 sentences - Fix semicolons-as-conjunctions in comments - Track rAF via MutableDisposable (use-after-dispose fix) - Switch runNow to CancellationToken.None (phantom CTS removal) - Restore defensive typeof guard in getCategoryLabel - Re-add ILabelService for URI display in remote contexts - Add tests: openEditDialog error, openCreateDialog, runNow failure, resetLanguageModelToDefault - Add race-safety comment in createSessionTypeBinder * refactor(automations): replace querySelector toolbar injection with menu-driven actions Migrate the harness chip and isolation group from DOM surgery (querySelector on ChatInputPart internals) to the structured MenuId + actionViewItemProvider pattern used by the New Session Page. - Register OpenAutomationsHarnessChipAction on MenuId.ChatInputSecondary - Register OpenAutomationsIsolationGroupAction on MenuId.ChatInputSecondary - Gate both with ChatContextKeys.inAutomationsDialog context key - Add secondaryToolbarActionViewItemProvider to IChatInputPartOptions - Route custom items in ChatInputPart secondary toolbar provider - Remove querySelector and eslint-disable-next-line comments * fix(automations): honor storeSelection in setChatMode2, handle hidden saved modes - setChatMode2 now gates _syncInputStateToModel behind storeSelection, preventing unintended persistence of transient mode changes. - automationDialog detects when a saved mode is hidden by the hideCustomChatModes filter and falls back to default instead of setting up an infinite retry watcher. * feat(automations): restore accessibility help provider and fix a11y issues * Signing commit * fix: correct indentation in dialog.ts focus-out handler * Signing commit --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

automations: new management view (#328471) * feat(automations): automations custom view with cards UI and run history Implements the Automations page as an AbstractCustomView using the new custom view grid infrastructure. Registers via ICustomViewService and renders the cards widget (automation cards + run history with scroll). Adds ManageAutomations and MarkAllAutomationRunsRead actions. * feat(automations): sessions list sidebar shortcut with status indicator Adds an Automations shortcut section to the sessions list sidebar. Clicking it opens the automations custom view via ICustomViewService. Shows a blue dot status indicator for active/unread automation runs. * fix: automations UX polish — layout shift, focus outlines, icon color - Use visibility instead of display for toolbar show/hide to prevent blue dot layout shift on hover - Suppress focus outline on shortcut rows (focused/selected states) - Preserve bright icon/label color when shortcut is active * feat: migrate New Automation button to header, fix cards rendering and polish - Move 'New Automation' from inline widget header to custom view header action bar via Action2 + CustomViewAutomations MenuId - Register PrimaryButtonActionViewItem via IActionViewItemService so the button renders as primary (blue) instead of secondary - Fix cards not rendering: remove stale newButtonElement references that crashed renderCards silently - Add openCreateDialog back for empty state create button - Remove inner DomScrollableElement (outer custom view handles scroll) - Remove widget tabIndex and suppress .custom-view-content focus outline - Fix mark-read race: store read ID before opening session - Use pixel spinner grid animation for running/pending history status - Remove small: true from button for better sizing Apply the same pattern: override the content element to full width via CSS, and let the widget self-center with max-width and margin: 0 auto. * Remove redundant automation edit action * Derive automations shortcut active state * fix: improve automation card keyboard accessibility - Change card role from 'button' to 'group' since it contains nested interactive elements (buttons inside a button is invalid ARIA) - Add Enter/Space keyboard handler on the card to open edit dialog - Add role='toolbar' and aria-label to the actions container - Improve card aria-label to include schedule info for screen readers The action buttons were already keyboard-reachable (tabindex=0) and visible on :focus-within, but the card itself lacked keyboard activation and had an invalid ARIA role. * refactor: split AutomationsCardsWidget into section classes Extract two internal classes from the 480-line monolith: - AutomationCardsSection: card grid, empty state, card interactions, edit/create/delete dialogs - AutomationHistorySection: history list, date grouping, run cards, read state management The widget becomes a thin composition layer that creates both sections, wires up observables, and handles layout. Shared helpers (formatSchedule, groupRunsByDate, runStatusIcon) are promoted to module-level functions. All classes remain in the same file, following the sessionsList.ts convention of keeping related renderers together. * perf: split autorun so cards and history re-render independently Previously one autorun subscribed to automations, runs, and readStateVersion, causing both sections to fully rebuild when any observable fired. Now cards only re-render when automations change, and history only re-renders when runs, automations, or read state change. * Revert "perf: split autorun so cards and history re-render independently" * automations: unify unread run status * automations: refine view focus * automations: remove focus override

feat(automations): add foundation — types, storage, scheduler, leader election (#323745) * feat(automations): add foundation — types, storage service, scheduler, leader election Introduces the Automations subsystem core: - IAutomation types and IAutomationSchedule (interval-based: manual/hourly/daily/weekly) - DST-safe computeNextRunAt scheduling logic - IAutomationService with persistent JSON ledger (StorageScope.APPLICATION) - Optimistic concurrency via revision counter for cross-window safety - Leader election to ensure single scheduler instance across windows - AutomationScheduler: periodic tick loop checking due automations - Feature gate: chat.automations.enabled setting mirrored to context key - Unit tests for scheduling logic Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: address Copilot Code Review feedback on PR 1 - Add _unsupportedSchema guard to recordRunStart and updateRun - Localize CRASH_RECOVERY_REASON string via nls.localize() - Move imports above registerSingleton call in contribution file - Fix test comment referencing wrong helper name (buildLocalDate -> localDate) * fix: address CCR round 2 — schema guards, telemetry cleanup, scope fix - Add _unsupportedSchema guard to markStaleRunsFailed and advanceNextRunAt - Replace boolean telemetry flags with actual enum values (permissionLevel, isolationMode) - Remove hasProviderId, hasSessionTypeId, hasModelId, hasMode, hasPermissionLevel flags - Add scope: ConfigurationScope.MACHINE to runTimeoutMinutes setting * fix: gate scheduler construction on chat.automations.enabled AutomationSchedulerCore (and its leader election heartbeat/storage writes) are now only constructed when the feature setting is enabled. Uses MutableDisposable for clean teardown on runtime toggle. Addresses CCR feedback: avoid 30s heartbeat timer + storage writes when the feature is disabled. * fix: isolate per-automation errors in dispatch loop Wrap advanceNextRunAt + runOneWithTimeout in try/catch so a failure in one automation does not abort dispatch of remaining due automations. * fix: register automations path in i18n resources Adds vs/sessions/contrib/automations to the translation resources so nls.localize calls pass the eslint translation-remind rule. * fix: validate enum fields at write boundary and harden persist() - Normalize permissionLevel/isolationMode in createAutomation and mergeAutomation: reject unknown values to keep persisted data and GDPR-classified telemetry low-cardinality. - Wrap persist() storage read and write in try/catch so storage failures degrade gracefully instead of breaking the scheduler. * fix: scheduler to Eventually phase, simplify persist(), document serial dispatch - Move scheduler registration from AfterRestored to Eventually — background concern that doesn't need to block workbench restore. - Remove re-read-before-write in persist() — refreshFromStorage via onDidChangeValue already handles cross-window sync. Eliminates redundant JSON.parse on every mutation and the misleading optimistic concurrency check that overwrote anyway. - Document that _pendingRuns serializes dispatch intentionally. * style: remove em dashes and obscure acronyms from comments Replace all em dashes with periods or semicolons. Remove TOCTOU acronym. Replace multiplication sign with plain x. * Signing commit --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Automations: migrate to provider supplied automations client side (#329655) * Automations: support provider-owned storage * Automations: transfer storage ownership on retarget * Automations: guard storage transfers against concurrent changes * Automations: clarify storage snapshot APIs * Automations: recover stale runs for late providers * Automations: preserve conflicting storage snapshots * Automations: format provider storage tests Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Automations view polish (#328883) * Hide the 'New Automation' button when there are no automations * Fix caps * Fix context key setting for btn * Always show custom view header divider Previously the bottom border on the custom view header band was transparent by default and only became visible when the user scrolled down (via the .scrolled class toggled in customViewNode.ts). This meant the divider between the header and content was invisible at the top scroll position. Change the border color to always be visible so the header is consistently separated from the content regardless of scroll position. * Move automation dialog CSS to dedicated file and fix button padding The automation dialog CSS was still living in aiCustomizationManagement.css after the automations list widget was removed in #328654. Move it to a dedicated automationDialog.css file next to the dialog service and add the import there. Also fix padding issues where Create/Cancel buttons and the close button were too close to the dialog border (padding: 0 was inherited from the base dialog override): - dialog-toolbar-row (close button): 4px/6px → 8px/10px from top/right - dialog-buttons-row: add 8px top, 10px side, 10px bottom padding * Fix pointer cursor on automation dialog form labels Schedule, Time, Day of week, and Prompt labels were rendered as <label> elements without 'for' attributes. Chromium applies cursor: pointer to <label> elements regardless of author CSS, making them look like links. Change them to <span> elements since they are purely visual section headers with no programmatic association to their controls. * Fix extra spacing around automation dialog prompt input chat.css sets padding: 4px 0 on .interactive-input-part, designed for the full chat panel. In the compact dialog this adds visible dead space above the rounded input container (below the Prompt label) and below the secondary toolbar. Zero out both ends. Also reduce the 6px bottom padding inside .chat-input-container (between the picker chips and the container border) to 2px. * Truncate long automation card titles with ellipsis word-break: break-word on the card name container caused the Disabled badge to word-wrap when titles were long. Replace with ellipsis truncation on the title text so the badge always stays on one line. * Use editorWidget.border token for automation card outlines widget.border is transparent in many themes, making cards invisible. Switch to editorWidget.border which has concrete defaults across light and dark themes, and add a contrastBorder override for HC themes. * Fix ellipsis truncation on automation card titles and add context key test The title span needs flex: 1 1 0; min-width: 0 to actually shrink as a flex item — without it the min-width: auto default prevents ellipsis from engaging on long unbroken strings. Also add test coverage for the AutomationsHasItemsContext global context key fix: verifies the key transitions false → true → false as the automations observable changes through empty/non-empty/empty states. Export AutomationsCustomViewContribution to enable direct instantiation in the test. * Always show header divider for automations view Stamp data-view-id on the custom view node element so individual views can override the scroll-driven divider behavior via CSS. Use this to always show the header bottom border in the automations view, while keeping the scroll-driven behavior intact for any other custom views.

Automations: rename interface types and make run sessionResource a URI (#330302) * Rename IAutomation to IAutomationDescriptor and IAutomationSnapshot to IAutomation Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: aa7ca598-6aee-45b5-a7cf-3e203ba059f3 * Change automation run sessionResource to URI Store it as a string at the persistence boundary for backward compatibility. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: aa7ca598-6aee-45b5-a7cf-3e203ba059f3 * Add persistence test for string-to-URI sessionResource round-trip Seeds a schema-v3 ledger with a string sessionResource, verifies it loads as a URI, and confirms a subsequent write persists it as a string. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: aa7ca598-6aee-45b5-a7cf-3e203ba059f3

automations: add context menu actions for automation cards (#333226) * Agent Host changes for agents/automation-card-duplicate-option * sessions: handle duplicate dialog failures Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a713c12c-1a43-4f7f-9337-fca7f8dfe89f * Fix automation context menu test typing Copilot-Session: a713c12c-1a43-4f7f-9337-fca7f8dfe89f

Enforce workspace trust for unattended agent sessions (#327670) * sessions: enforce trust for unattended sends Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Validate workspace trust before selecting an automation workspace (#327724) * automations: validate workspace trust before selection Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Automations: Wire into customizations UI (#324167) Connects the Automations feature to existing VS Code surfaces: - AI Customizations sidebar entry gated on ChatAutomationsEnabledContext - Overview tile with automation count badge - Deep-link: sidebar click navigates directly to Automations tab - Feature gate when-clauses on all entry points - Allow a way to set model without persisting globally