Automations PR 2 (Runner)

[benvillalobos]

Context

Second PR in the 4-part Automations stack. Builds on PR 1 (Foundation) which has been merged to main.

This PR implements the AutomationRunner, the execution engine that bridges the automation data layer (PR 1) to the sessions layer. When the scheduler decides an automation is due, the runner creates a chat session, configures it, sends the prompt, and records the outcome.

Architecture

classDiagram
    class AutomationRunner {
        +runOnce(automation, trigger, windowId, token)
        -_runOnceInner()
        -_markCancelled()
    }

    class ISessionsManagementService {
        +createAndSendNewChatRequest(folderUri, options, createOptions)
        -_resolveProviderForNewSession()
        -_sendNewChatRequestInBackground()
    }

    class ISessionsProvider {
        +createNewSession(folderUri, sessionTypeId)
        +setModel(sessionId, modelId)
        +setMode?(sessionId, modeId)
        +setPermissionLevel?(sessionId, level)
        +setIsolationMode?(sessionId, mode)
        +setBranch?(sessionId, branch)
        +sendRequest(sessionId, chatResource, options)
    }

    class AutomationService {
        +recordRunStart()
        +updateRun()
        Data layer (from PR 1)
    }

    AutomationRunner --> ISessionsManagementService : creates session + sends prompt
    AutomationRunner --> AutomationService : records run start/completion/failure
    ISessionsManagementService --> ISessionsProvider : configures + dispatches

Loading

Execution Flow

sequenceDiagram
    participant Sch as Scheduler
    participant R as AutomationRunner
    participant AS as AutomationService
    participant SM as SessionsManagementService
    participant P as Provider

    Sch->>R: runOnce(automation, trigger, token)
    R->>AS: recordRunStart(automationId, trigger)
    R->>SM: createAndSendNewChatRequest(folderUri, options, createOptions)
    SM->>P: createNewSession(folderUri, sessionTypeId)
    SM->>P: setModel / setMode / setPermissionLevel / setIsolationMode / setBranch
    SM->>P: sendRequest(sessionId, chatResource, options)
    P-->>SM: session
    SM-->>R: session
    alt cancelled mid-flight
        R->>AS: updateRun(failed, "Cancelled")
    else success
        R->>AS: updateRun(completed, sessionId)
    else error
        R->>AS: updateRun(failed, errorMessage)
        R->>Notification: show error to user
    end

Loading

What's New

New file: src/vs/sessions/contrib/automations/browser/automationRunner.ts

• Implements IAutomationRunner (interface from PR 1).
• Creates sessions programmatically via createAndSendNewChatRequest.
• Handles mid-flight cancellation by re-checking the token post-send.
• Reports telemetry on success and failure.

Sessions layer extensions:

• ISessionsManagementService.createAndSendNewChatRequest() for programmatic session creation without the composer UI. Cleans up stranded sessions on failure.
• ISessionsProvider gains 4 optional setters: setMode?, setPermissionLevel?, setIsolationMode?, setBranch?. These were previously only callable through the concrete provider class from UI widgets.
• ISendRequestOptions.title lets callers pre-name the session so it shows a meaningful title during launch.

Telemetry: Refactored to report actual enum values (permissionLevel, isolationMode) instead of boolean flags.

[Copilot]

Pull request overview

This PR adds the sessions-layer execution engine for Chat Automations by introducing an AutomationRunner that creates and configures a new chat session via ISessionsManagementService, sends the automation prompt in the background, and records run outcomes. It also extends the sessions management/provider APIs to support programmatic session configuration (model/mode/permission/isolation/branch) and pre-titling sessions for better UX during launch.

Changes:

• Add AutomationRunner (sessions layer) plus unit tests; wire it into the automations contribution.
• Extend ISessionsManagementService.createAndSendNewChatRequest to return the started ISession | undefined and accept richer ICreateNewSessionOptions for session configuration.
• Extend ISessionsProvider request options with an optional title, and add optional provider setters (setMode, setPermissionLevel, setIsolationMode, setBranch).

Show a summary per file

File	Description
src/vs/sessions/services/sessions/test/browser/sessionNavigation.test.ts	Updates mock signature for createAndSendNewChatRequest return type.
src/vs/sessions/services/sessions/common/sessionsProvider.ts	Adds ISendRequestOptions.title and optional provider configuration setter APIs.
src/vs/sessions/services/sessions/common/sessionsManagement.ts	Extends ICreateNewSessionOptions and updates createAndSendNewChatRequest return type.
src/vs/sessions/services/sessions/browser/sessionsManagementService.ts	Implements configuration application before send; returns committed session (or undefined).
src/vs/sessions/contrib/providers/copilotChatSessions/browser/copilotChatSessionsProvider.ts	Uses options.title when setting initial session title.
src/vs/sessions/contrib/automations/test/browser/automationRunner.test.ts	Adds coverage for runner success/failure/cancellation and option pass-through.
src/vs/sessions/contrib/automations/browser/automations.contribution.ts	Registers the real AutomationRunner instead of a stub.
src/vs/sessions/contrib/automations/browser/automationRunner.ts	New runner implementation bridging automations to sessions + telemetry/notifications.

Review details

• Files reviewed: 8/8 changed files
• Comments generated: 9
• Review effort level: Low

[Copilot]

Review details

• Files reviewed: 8/8 changed files
• Comments generated: 7
• Review effort level: Low

[Copilot]

Review details

• Files reviewed: 9/9 changed files
• Comments generated: 6
• Review effort level: Low

[vs-code-engineering]

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@sandy081

Matched files:

• src/vs/sessions/services/sessions/browser/sessionsManagementService.ts
• src/vs/sessions/services/sessions/common/sessionsManagement.ts
• src/vs/sessions/services/sessions/common/sessionsProvider.ts
• src/vs/sessions/services/sessions/test/browser/sessionNavigation.test.ts
• src/vs/sessions/services/sessions/test/browser/sessionsManagementService.test.ts

@lszomoru

Matched files:

• src/vs/sessions/services/sessions/browser/sessionsManagementService.ts
• src/vs/sessions/services/sessions/common/sessionsManagement.ts
• src/vs/sessions/services/sessions/common/sessionsProvider.ts
• src/vs/sessions/services/sessions/test/browser/sessionNavigation.test.ts
• src/vs/sessions/services/sessions/test/browser/sessionsManagementService.test.ts