@codingame/monaco-vscode-workbench-service-override 37.6.0 → 38.0.0

package/vscode/src/vs/sessions/contrib/automations/browser/automationDialogService.d.ts

@@ -2,11 +2,16 @@ import { IConfigurationService } from "@codingame/monaco-vscode-api/vscode/vs/pl
 import { IContextKeyService } from "@codingame/monaco-vscode-api/vscode/vs/platform/contextkey/common/contextkey.service";
 import { IContextViewService } from "@codingame/monaco-vscode-api/vscode/vs/platform/contextview/browser/contextView.service";
 import { IInstantiationService } from "@codingame/monaco-vscode-api/vscode/vs/platform/instantiation/common/instantiation";
+import { IHoverService } from "@codingame/monaco-vscode-api/vscode/vs/platform/hover/browser/hover.service";
 import { IKeybindingService } from "@codingame/monaco-vscode-api/vscode/vs/platform/keybinding/common/keybinding.service";
 import { ILogService } from "@codingame/monaco-vscode-api/vscode/vs/platform/log/common/log.service";
+import { INotificationService } from "@codingame/monaco-vscode-api/vscode/vs/platform/notification/common/notification.service";
+import { IQuickInputService } from "@codingame/monaco-vscode-api/vscode/vs/platform/quickinput/common/quickInput.service";
+import { ITelemetryService } from "@codingame/monaco-vscode-api/vscode/vs/platform/telemetry/common/telemetry.service";
 import { IWorkspaceTrustRequestService } from "@codingame/monaco-vscode-api/vscode/vs/platform/workspace/common/workspaceTrust.service";
 import { IAutomationDialogResult, IShowAutomationDialogOptions } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automationDialogService";
 import { IAutomationDialogService } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automationDialogService.service";
+import { IAutomationService } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automationService.service";
 import { IHostService } from "@codingame/monaco-vscode-api/vscode/vs/workbench/services/host/browser/host.service";
 import { IWorkbenchLayoutService } from "@codingame/monaco-vscode-api/vscode/vs/workbench/services/layout/browser/layoutService.service";
 import { ISessionsManagementService } from "@codingame/monaco-vscode-api/vscode/vs/sessions/services/sessions/common/sessionsManagement.service";
@@ -26,7 +31,12 @@ export declare class AutomationDialogService implements IAutomationDialogService
     private readonly hostService;
     private readonly sessionsManagementService;
     private readonly workspaceTrustRequestService;
+    private readonly automationService;
+    private readonly telemetryService;
+    private readonly notificationService;
+    private readonly hoverService;
+    private readonly quickInputService;
     readonly _serviceBrand: undefined;
-    constructor(instantiationService: IInstantiationService, contextKeyService: IContextKeyService, contextViewService: IContextViewService, configurationService: IConfigurationService, keybindingService: IKeybindingService, layoutService: IWorkbenchLayoutService, logService: ILogService, hostService: IHostService, sessionsManagementService: ISessionsManagementService, workspaceTrustRequestService: IWorkspaceTrustRequestService);
+    constructor(instantiationService: IInstantiationService, contextKeyService: IContextKeyService, contextViewService: IContextViewService, configurationService: IConfigurationService, keybindingService: IKeybindingService, layoutService: IWorkbenchLayoutService, logService: ILogService, hostService: IHostService, sessionsManagementService: ISessionsManagementService, workspaceTrustRequestService: IWorkspaceTrustRequestService, automationService: IAutomationService, telemetryService: ITelemetryService, notificationService: INotificationService, hoverService: IHoverService, quickInputService: IQuickInputService);
     showAutomationDialog(options: IShowAutomationDialogOptions): Promise<IAutomationDialogResult | undefined>;
 }

package/vscode/src/vs/sessions/contrib/automations/browser/automationDialogService.js

@@ -2,28 +2,38 @@
 import { registerCss } from '@codingame/monaco-vscode-api/css';
 import { __decorate, __param } from '@codingame/monaco-vscode-api/external/tslib/tslib.es6';
 import * as automationDialog from './media/automationDialog.css';
-import { $ as $$1, append, getWindow } from '@codingame/monaco-vscode-api/vscode/vs/base/browser/dom';
+import { $ as $$1, append, getWindow, EventType, addDisposableListener, EventHelper, isHTMLElement } from '@codingame/monaco-vscode-api/vscode/vs/base/browser/dom';
 import { ButtonBar } from '@codingame/monaco-vscode-api/vscode/vs/base/browser/ui/button/button';
 import { Dialog } from '@codingame/monaco-vscode-api/vscode/vs/base/browser/ui/dialog/dialog';
+import { ProgressBar } from '@codingame/monaco-vscode-api/vscode/vs/base/browser/ui/progressbar/progressbar';
 import { DeferredPromise } from '@codingame/monaco-vscode-api/vscode/vs/base/common/async';
 import { CancellationTokenSource } from '@codingame/monaco-vscode-api/vscode/vs/base/common/cancellation';
-import { isCancellationError } from '@codingame/monaco-vscode-api/vscode/vs/base/common/errors';
+import { isCancellationError, getErrorMessage } from '@codingame/monaco-vscode-api/vscode/vs/base/common/errors';
 import { DisposableStore, MutableDisposable, toDisposable } from '@codingame/monaco-vscode-api/vscode/vs/base/common/lifecycle';
+import '@codingame/monaco-vscode-api/vscode/vs/base/common/observableInternal/index';
 import { isWindows } from '@codingame/monaco-vscode-api/vscode/vs/base/common/platform';
 import { localize } from '@codingame/monaco-vscode-api/vscode/vs/nls';
 import { IConfigurationService } from '@codingame/monaco-vscode-api/vscode/vs/platform/configuration/common/configuration.service';
 import { IContextKeyService } from '@codingame/monaco-vscode-api/vscode/vs/platform/contextkey/common/contextkey.service';
 import { IContextViewService } from '@codingame/monaco-vscode-api/vscode/vs/platform/contextview/browser/contextView.service';
 import { IInstantiationService } from '@codingame/monaco-vscode-api/vscode/vs/platform/instantiation/common/instantiation';
+import { IHoverService } from '@codingame/monaco-vscode-api/vscode/vs/platform/hover/browser/hover.service';
 import { IKeybindingService } from '@codingame/monaco-vscode-api/vscode/vs/platform/keybinding/common/keybinding.service';
 import { ILogService } from '@codingame/monaco-vscode-api/vscode/vs/platform/log/common/log.service';
+import { INotificationService } from '@codingame/monaco-vscode-api/vscode/vs/platform/notification/common/notification.service';
+import { IQuickInputService } from '@codingame/monaco-vscode-api/vscode/vs/platform/quickinput/common/quickInput.service';
+import { ITelemetryService } from '@codingame/monaco-vscode-api/vscode/vs/platform/telemetry/common/telemetry.service';
 import { IWorkspaceTrustRequestService } from '@codingame/monaco-vscode-api/vscode/vs/platform/workspace/common/workspaceTrust.service';
-import { defaultButtonStyles, defaultDialogStyles } from '@codingame/monaco-vscode-api/vscode/vs/platform/theme/browser/defaultStyles';
+import { defaultProgressBarStyles, defaultButtonStyles, defaultDialogStyles } from '@codingame/monaco-vscode-api/vscode/vs/platform/theme/browser/defaultStyles';
 import { createWorkbenchDialogOptions } from '@codingame/monaco-vscode-api/vscode/vs/workbench/browser/parts/dialogs/dialog';
+import { automationScheduleToLocal, automationScheduleToUTC } from '../../../../workbench/contrib/chat/common/automations/schedule.js';
+import { IAutomationService } from '@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automationService.service';
 import { IHostService } from '@codingame/monaco-vscode-api/vscode/vs/workbench/services/host/browser/host.service';
 import { IWorkbenchLayoutService } from '@codingame/monaco-vscode-api/vscode/vs/workbench/services/layout/browser/layoutService.service';
 import { ISessionsManagementService } from '@codingame/monaco-vscode-api/vscode/vs/sessions/services/sessions/common/sessionsManagement.service';
-import { renderForm, registerAutomationDialogKeyboardNavigation, isAutomationDialogPopupTarget, updateSaveButtonState, shouldPassThroughAutomationDialogCommand } from './automationDialog.js';
+import { getAutomationDialogProviders, createAutomationTarget, renderForm, registerAutomationDialogKeyboardNavigation, isAutomationDialogPopupTarget, updateSaveButtonState, shouldPassThroughAutomationDialogCommand } from './automationDialog.js';
+import { AutomationDialogTelemetry } from './automationTelemetry.js';
+import { autorun } from '@codingame/monaco-vscode-api/vscode/vs/base/common/observableInternal/reactions/autorun';
 
 registerCss(automationDialog, 1);
 const $ = $$1;
@@ -49,7 +59,8 @@ const automationDialogAllowableCommands = ( new Set([
     "quickInput.next",
     "quickInput.previous",
     "quickInput.accept",
-    "quickInput.hide"
+    "quickInput.hide",
+    "workbench.action.closeQuickOpen"
 ]));
 let AutomationDialogService = class AutomationDialogService {
     constructor(
@@ -62,7 +73,12 @@ let AutomationDialogService = class AutomationDialogService {
         logService,
         hostService,
         sessionsManagementService,
-        workspaceTrustRequestService
+        workspaceTrustRequestService,
+        automationService,
+        telemetryService,
+        notificationService,
+        hoverService,
+        quickInputService
     ) {
         this.instantiationService = instantiationService;
         this.contextKeyService = contextKeyService;
@@ -74,12 +90,21 @@ let AutomationDialogService = class AutomationDialogService {
         this.hostService = hostService;
         this.sessionsManagementService = sessionsManagementService;
         this.workspaceTrustRequestService = workspaceTrustRequestService;
+        this.automationService = automationService;
+        this.telemetryService = telemetryService;
+        this.notificationService = notificationService;
+        this.hoverService = hoverService;
+        this.quickInputService = quickInputService;
     }
     async showAutomationDialog(options) {
         const disposables = ( new DisposableStore());
         const existing = options.existing;
+        const allowedProviders = getAutomationDialogProviders(this.automationService, existing);
         const initial = existing ?? options.initialValues;
+        const timezoneOffset = ( new Date()).getTimezoneOffset();
+        const initialSchedule = initial && automationScheduleToLocal(initial.schedule, timezoneOffset);
         const isEdit = !!existing;
+        const dialogTelemetry = ( new AutomationDialogTelemetry(this.telemetryService, isEdit ? "update" : "create"));
         const initialTarget = initial?.target;
         const initialWorkspaceTarget = initialTarget?.kind === "workspace" ? initialTarget : undefined;
         const initialSessionConfiguration = initial ? {
@@ -90,17 +115,19 @@ let AutomationDialogService = class AutomationDialogService {
         } : undefined;
         const state = {
             name: initial?.name ?? "",
-            interval: initial?.schedule.interval ?? "daily",
-            hour: initial?.schedule.scheduleHour ?? 9,
-            minute: initial?.schedule.scheduleMinute ?? 0,
-            day: initial?.schedule.scheduleDay ?? 1,
-            isQuickChat: initialTarget?.kind === "quickChat",
+            interval: initialSchedule?.interval ?? "daily",
+            hour: initialSchedule?.scheduleHour ?? 9,
+            minute: initialSchedule?.scheduleMinute ?? 0,
+            day: initialSchedule?.scheduleDay ?? 1,
+            isQuickChat: initialTarget === undefined || initialTarget.kind === "quickChat",
             folderUri: initialWorkspaceTarget?.folderUri,
             providerId: initialTarget?.providerId,
             sessionTypeId: initialTarget?.sessionTypeId,
             isolationMode: initialWorkspaceTarget?.isolation.kind === "default" ? undefined : initialWorkspaceTarget?.isolation.kind === "worktree" ? "worktree" : "workspace",
             branch: initialWorkspaceTarget?.isolation.kind === "worktree" ? initialWorkspaceTarget.isolation.branch : undefined,
-            enabled: initial?.enabled ?? true
+            enabled: initial?.enabled ?? true,
+            timeZone: initial?.schedule.timeZone,
+            timezoneOffset
         };
         const validation = {
             nameError: undefined,
@@ -120,32 +147,41 @@ let AutomationDialogService = class AutomationDialogService {
         let getBranch = () => initialWorkspaceTarget?.isolation.kind === "worktree" ? initialWorkspaceTarget.isolation.branch : undefined;
         let waitForAutomationSessionSync = async () => {};
         let setSaving = () => {};
-        let showSessionConfigurationError = () => {};
-        let focusSessionConfigurationError = () => {};
+        let getCustomizationIds = () => undefined;
+        let waitForCustomizationChoices = async () => {};
+        let progressBar;
+        let dialogElement;
+        let footerNote;
+        let closeToolbar;
+        let commitInProgress = false;
+        let showSaveError = () => {};
+        let focusSaveError = () => {};
         let getFocusableElements = () => [];
         let focusFirst = () => {};
         let saveInProgress = false;
         const saveCancellation = disposables.add(( new MutableDisposable()));
         const completion = ( new DeferredPromise());
-        const title = isEdit ? ( localize(3351, "Edit automation")) : ( localize(3352, "New automation"));
-        const saveButtonLabel = isEdit ? ( localize(3353, "Save")) : ( localize(3354, "Create"));
-        const cancelButtonLabel = ( localize(3355, "Cancel"));
-        const savingButtonLabel = ( localize(3356, "Saving…"));
+        const title = isEdit ? ( localize(3756, "Edit automation")) : ( localize(3757, "New automation"));
+        const saveButtonLabel = isEdit ? ( localize(3758, "Save")) : ( localize(3759, "Create"));
+        const cancelButtonLabel = ( localize(3760, "Cancel"));
+        const savingButtonLabel = ( localize(3761, "Saving…"));
         const captureErrorMessage = ( localize(
-            3357,
+            3762,
             "The automation wasn't saved because its session configuration couldn't be captured. Check the provider connection and try again."
         ));
         const buildResult = sessionConfigurationCapture => {
-            const schedule = {
+            const localSchedule = {
                 interval: state.interval,
                 scheduleHour: state.hour,
                 scheduleMinute: state.minute,
                 scheduleDay: state.day
             };
+            const schedule = state.timeZone === "UTC" ? automationScheduleToUTC(localSchedule, timezoneOffset) : localSchedule;
             const prompt = getPrompt();
             const sessionConfiguration = sessionConfigurationCapture.configuration;
             const sessionTemplate = sessionConfiguration?.sessionTemplate;
             const target = createAutomationTarget(state, getBranch());
+            const customizationIds = getCustomizationIds();
             if (!target) {
                 return undefined;
             }
@@ -158,7 +194,10 @@ let AutomationDialogService = class AutomationDialogService {
                     ...(sessionConfigurationCapture.kind === "captured" ? {
                         sessionTemplate: sessionTemplate ?? null
                     } : {}),
-                    enabled: state.enabled
+                    enabled: state.enabled,
+                    ...(customizationIds !== undefined ? {
+                        customizationIds
+                    } : {})
                 };
                 return {
                     kind: "update",
@@ -167,7 +206,7 @@ let AutomationDialogService = class AutomationDialogService {
                 };
             }
             const create = {
-                name: state.name,
+                name: state.name.trim() ? state.name : deriveAutomationName(prompt),
                 prompt,
                 schedule,
                 target,
@@ -184,7 +223,10 @@ let AutomationDialogService = class AutomationDialogService {
                         permissionLevel: sessionConfiguration.permissionLevel
                     } : {})
                 } : {}),
-                enabled: state.enabled
+                enabled: state.enabled,
+                ...(customizationIds !== undefined ? {
+                    customizationIds
+                } : {})
             };
             return {
                 kind: "create",
@@ -192,9 +234,10 @@ let AutomationDialogService = class AutomationDialogService {
             };
         };
         const closeDialog = result => {
-            if (completion.isSettled) {
+            if (completion.isSettled || (commitInProgress && result === undefined)) {
                 return;
             }
+            dialogTelemetry.complete(result !== undefined);
             saveCancellation.value?.cancel();
             void completion.complete(result);
             dialog.dispose();
@@ -204,15 +247,21 @@ let AutomationDialogService = class AutomationDialogService {
                 return;
             }
             revalidate();
-            if (validation.nameError || validation.promptError || validation.folderError || validation.sessionTypeError || validation.branchError) {
+            if (state.targetPending) {
+                return;
+            }
+            if (validation.nameError || validation.promptError || validation.folderError || validation.sessionTypeError || validation.branchError || validation.scheduleError) {
+                dialogTelemetry.validationFailed();
                 return;
             }
             if ((!state.isQuickChat && !state.folderUri) || !state.sessionTypeId || (state.isQuickChat && !state.providerId)) {
+                dialogTelemetry.validationFailed();
                 return;
             }
             saveInProgress = true;
-            showSessionConfigurationError(undefined);
+            showSaveError(undefined);
             setSaving(true);
+            progressBar?.infinite().show();
             if (saveButton) {
                 saveButton.enabled = false;
                 saveButton.label = savingButtonLabel;
@@ -224,24 +273,42 @@ let AutomationDialogService = class AutomationDialogService {
             let shouldFocusError = false;
             try {
                 await waitForAutomationSessionSync(cancellation.token);
+                await waitForCustomizationChoices(cancellation.token);
                 const sessionConfigurationCapture = await getSessionConfiguration(cancellation.token);
                 if (sessionConfigurationCapture.kind === "failed") {
-                    showSessionConfigurationError(captureErrorMessage);
+                    dialogTelemetry.captureFailed();
+                    showSaveError(captureErrorMessage);
                     shouldFocusError = true;
                     return;
                 }
+                revalidate();
+                if (state.targetPending || validation.sessionTypeError || validation.scheduleError) {
+                    return;
+                }
                 const result = buildResult(sessionConfigurationCapture);
                 if (result) {
+                    if (options.commit) {
+                        commitInProgress = true;
+                        setSaving(true, true);
+                        if (cancelButton) {
+                            cancelButton.enabled = false;
+                        }
+                        dialogElement?.classList.add("committing");
+                        closeToolbar?.setAttribute("inert", "");
+                        dialogElement?.focus();
+                        await options.commit(result);
+                        commitInProgress = false;
+                    }
                     shouldClose = true;
                     closeDialog(result);
                 }
             } catch (error) {
-                if (!isCancellationError(error) && !cancellation.token.isCancellationRequested) {
-                    this.logService.error(
-                        "[AutomationDialog] Failed to save the automation session configuration.",
-                        error
-                    );
-                    showSessionConfigurationError(captureErrorMessage);
+                if (commitInProgress || (!isCancellationError(error) && !cancellation.token.isCancellationRequested)) {
+                    this.logService.error("[AutomationDialog] Failed to save automation.", error);
+                    if (!commitInProgress) {
+                        dialogTelemetry.captureFailed();
+                    }
+                    showSaveError(commitInProgress ? getErrorMessage(error) : captureErrorMessage);
                     shouldFocusError = true;
                 }
             } finally {
@@ -249,14 +316,21 @@ let AutomationDialogService = class AutomationDialogService {
                     saveCancellation.clear();
                 }
                 saveInProgress = false;
+                commitInProgress = false;
                 if (!shouldClose && !completion.isSettled) {
+                    dialogElement?.classList.remove("committing");
+                    closeToolbar?.removeAttribute("inert");
+                    if (cancelButton) {
+                        cancelButton.enabled = true;
+                    }
+                    progressBar?.stop().hide();
                     setSaving(false);
                     if (saveButton) {
                         saveButton.label = saveButtonLabel;
                     }
                     revalidate();
                     if (shouldFocusError) {
-                        focusSessionConfigurationError();
+                        focusSaveError();
                     }
                 }
             }
@@ -274,6 +348,10 @@ let AutomationDialogService = class AutomationDialogService {
             renderFooter: container => {
                 container.classList.add("dialog-buttons", "automation-dialog-footer-actions");
                 container.parentElement?.classList.add("dialog-buttons-row", "automation-dialog-footer-row");
+                footerNote = $("p.automation-dialog-footer-note", {
+                    id: "automation-dialog-footer-note"
+                });
+                container.before(footerNote);
                 const buttonBar = disposables.add(( new ButtonBar(container)));
                 const createSaveButton = () => {
                     saveButton = buttonBar.addButton(defaultButtonStyles);
@@ -298,15 +376,20 @@ let AutomationDialogService = class AutomationDialogService {
             },
             renderBody: container => {
                 container.classList.add("automation-dialog-body");
+                dialogElement = container.closest(".monaco-dialog-box") ?? undefined;
+                const progressHost = append(container, $(".automation-dialog-progress"));
+                progressBar = disposables.add(( new ProgressBar(progressHost, defaultProgressBarStyles)));
+                progressBar.hide();
                 const titlebar = append(container, $(".automation-titlebar"));
                 titlebar.setAttribute("aria-hidden", "true");
                 titlebar.textContent = title;
-                const description = append(container, $(".automation-description"));
-                description.textContent = isEdit ? ( localize(3358, "Update the schedule, prompt, or run target for this automation.")) : ( localize(
-                    3359,
+                const scrollable = append(container, $(".automation-dialog-scrollable"));
+                const description = append(scrollable, $(".automation-description"));
+                description.textContent = isEdit ? ( localize(3763, "Update the schedule, prompt, or run target for this automation.")) : ( localize(
+                    3764,
                     "Define a prompt that will run on a schedule against the selected target."
                 ));
-                const formPane = append(container, $(".automation-form-pane"));
+                const formPane = append(scrollable, $(".automation-form-pane"));
                 const form = append(formPane, $(".automation-form"));
                 const handle = renderForm(
                     form,
@@ -324,26 +407,79 @@ let AutomationDialogService = class AutomationDialogService {
                     this.workspaceTrustRequestService,
                     initial?.prompt ?? "",
                     initialTarget,
-                    initialSessionConfiguration
+                    initialSessionConfiguration,
+                    allowedProviders,
+                    providerId => this.automationService.getProviderConfiguration?.(providerId),
+                    isEdit,
+                    {
+                        service: this.automationService,
+                        hoverService: this.hoverService,
+                        existingId: existing?.id
+                    },
+                    error => this.notificationService.error(error),
+                    this.quickInputService
                 );
+                disposables.add(autorun(reader => {
+                    const description = handle.providerDescription.read(reader);
+                    if (!footerNote) {
+                        return;
+                    }
+                    footerNote.textContent = description ?? "";
+                    footerNote.style.display = description ? "" : "none";
+                    if (description) {
+                        saveButton?.element.setAttribute("aria-describedby", footerNote.id);
+                    } else {
+                        saveButton?.element.removeAttribute("aria-describedby");
+                    }
+                }));
                 getPrompt = handle.getPrompt;
                 getSessionConfiguration = handle.getSessionConfiguration;
                 getBranch = handle.getBranch;
                 waitForAutomationSessionSync = handle.waitForAutomationSessionSync;
                 setSaving = handle.setSaving;
-                showSessionConfigurationError = handle.showSessionConfigurationError;
-                focusSessionConfigurationError = handle.focusSessionConfigurationError;
+                getCustomizationIds = handle.getCustomizationIds;
+                waitForCustomizationChoices = handle.waitForCustomizationChoices;
+                showSaveError = handle.showSaveError;
+                focusSaveError = handle.focusSaveError;
                 getFocusableElements = handle.getFocusableElements;
                 const keyboardNavigation = disposables.add(
                     registerAutomationDialogKeyboardNavigation(getWindow(container), () => [
                         ...getFocusableElements(),
                         ...(saveButton ? [saveButton.element] : []),
                         ...(cancelButton ? [cancelButton.element] : [])
-                    ], isAutomationDialogPopupTarget, handle.acceptPromptSuggestion, handle.cancelPromptSuggestion)
+                    ], isAutomationDialogPopupTarget, () => !saveInProgress && handle.acceptPromptSuggestion(), () => !saveInProgress && handle.cancelPromptSuggestion())
                 );
                 focusFirst = keyboardNavigation.focusFirst;
+                for (const type of [EventType.KEY_DOWN, EventType.KEY_UP]) {
+                    disposables.add(addDisposableListener(getWindow(container), type, event => {
+                        if (commitInProgress && event.key === "Escape") {
+                            EventHelper.stop(event, true);
+                            event.stopImmediatePropagation();
+                        }
+                    }, true));
+                }
+                disposables.add(addDisposableListener(activeContainer, EventType.CLICK, event => {
+                    if (commitInProgress && isHTMLElement(event.target) && closeToolbar?.contains(event.target)) {
+                        EventHelper.stop(event, true);
+                        event.stopImmediatePropagation();
+                    }
+                }, true));
                 revalidate = () => {
-                    updateSaveButtonState(saveButton, state, validation, form, getPrompt, getBranch);
+                    const providerAvailable = state.providerId !== undefined && allowedProviders.get().includes(state.providerId);
+                    updateSaveButtonState(
+                        saveButton,
+                        state,
+                        validation,
+                        form,
+                        getPrompt,
+                        getBranch,
+                        this.sessionsManagementService,
+                        providerAvailable,
+                        existing?.target.providerId,
+                        isEdit
+                    );
+                    handle.showTargetValidationError(validation.sessionTypeError);
+                    handle.showScheduleValidationError(validation.scheduleError);
                     if (saveInProgress && saveButton) {
                         saveButton.enabled = false;
                     }
@@ -357,6 +493,7 @@ let AutomationDialogService = class AutomationDialogService {
         );
         try {
             void dialog.show().then(() => closeDialog(undefined));
+            closeToolbar = dialogElement?.querySelector(".dialog-toolbar") ?? undefined;
             focusFirst();
             return await completion.p;
         } finally {
@@ -364,33 +501,17 @@ let AutomationDialogService = class AutomationDialogService {
         }
     }
 };
-AutomationDialogService = ( __decorate([( __param(0, IInstantiationService)), ( __param(1, IContextKeyService)), ( __param(2, IContextViewService)), ( __param(3, IConfigurationService)), ( __param(4, IKeybindingService)), ( __param(5, IWorkbenchLayoutService)), ( __param(6, ILogService)), ( __param(7, IHostService)), ( __param(8, ISessionsManagementService)), ( __param(9, IWorkspaceTrustRequestService))], AutomationDialogService));
-function createAutomationTarget(state, branch) {
-    if (state.isQuickChat) {
-        return state.providerId && state.sessionTypeId ? {
-            kind: "quickChat",
-            providerId: state.providerId,
-            sessionTypeId: state.sessionTypeId
-        } : undefined;
-    }
-    if (!state.folderUri) {
-        return undefined;
+AutomationDialogService = ( __decorate([( __param(0, IInstantiationService)), ( __param(1, IContextKeyService)), ( __param(2, IContextViewService)), ( __param(3, IConfigurationService)), ( __param(4, IKeybindingService)), ( __param(5, IWorkbenchLayoutService)), ( __param(6, ILogService)), ( __param(7, IHostService)), ( __param(8, ISessionsManagementService)), ( __param(9, IWorkspaceTrustRequestService)), ( __param(10, IAutomationService)), ( __param(11, ITelemetryService)), ( __param(12, INotificationService)), ( __param(13, IHoverService)), ( __param(14, IQuickInputService))], AutomationDialogService));
+function deriveAutomationName(prompt) {
+    const text = prompt.trim().replace(/\s+/g, " ");
+    const maxLength = 50;
+    const characters = Array.from(text);
+    if (characters.length <= maxLength) {
+        return text;
     }
-    const isolation = state.isolationMode === "worktree" ? (branch ? {
-        kind: "worktree",
-        branch
-    } : undefined) : state.isolationMode === "workspace" ? {
-        kind: "folder"
-    } : {
-        kind: "default"
-    };
-    return isolation ? {
-        kind: "workspace",
-        folderUri: state.folderUri,
-        providerId: state.providerId,
-        sessionTypeId: state.sessionTypeId,
-        isolation
-    } : undefined;
+    const prefix = characters.slice(0, maxLength).join("");
+    const wordBoundary = text.lastIndexOf(" ", prefix.length);
+    return wordBoundary > 0 ? text.slice(0, wordBoundary) : prefix;
 }
 
 export { AutomationDialogService };

package/vscode/src/vs/sessions/contrib/automations/browser/automationRunner.d.ts

@@ -1,21 +1,23 @@
 import { CancellationToken } from "@codingame/monaco-vscode-api/vscode/vs/base/common/cancellation";
 import { ILogService } from "@codingame/monaco-vscode-api/vscode/vs/platform/log/common/log.service";
 import { INotificationService } from "@codingame/monaco-vscode-api/vscode/vs/platform/notification/common/notification.service";
-import { AutomationRunTrigger, IAutomationDescriptor } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automation";
+import { IAutomationDescriptor } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automation";
 import { IAutomationRunOperation } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automationRunner";
 import { IAutomationRunner } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automationRunner.service";
 import { IAutomationService } from "@codingame/monaco-vscode-api/vscode/vs/workbench/contrib/chat/common/automations/automationService.service";
-import { ISessionsManagementService } from "@codingame/monaco-vscode-api/vscode/vs/sessions/services/sessions/common/sessionsManagement.service";
-/** Sessions-layer runner. Never throws; failures are recorded on the run row. */
+import { ISessionsProvidersService } from "@codingame/monaco-vscode-api/vscode/vs/sessions/services/sessions/browser/sessionsProvidersService.service";
+/**
+ * Client-side Run Now coordinator that requests execution through IAutomationService and observes the host result.
+ * Reports dispatch failures and forwards supported cancellation without creating run sessions.
+ */
 export declare class AutomationRunner implements IAutomationRunner {
     private readonly automationService;
-    private readonly sessionsManagementService;
+    private readonly sessionsProvidersService;
     private readonly logService;
     private readonly notificationService;
     readonly _serviceBrand: undefined;
-    constructor(automationService: IAutomationService, sessionsManagementService: ISessionsManagementService, logService: ILogService, notificationService: INotificationService);
-    runOnce(automation: IAutomationDescriptor, trigger: AutomationRunTrigger, leaderWindowId: number, token?: CancellationToken): IAutomationRunOperation;
-    private _runOnce;
-    private _runOnceInner;
-    private _markCancelled;
+    constructor(automationService: IAutomationService, sessionsProvidersService: ISessionsProvidersService, logService: ILogService, notificationService: INotificationService);
+    runOnce(automation: IAutomationDescriptor, token?: CancellationToken): IAutomationRunOperation;
+    private run;
+    private assertProviderAvailable;
 }