@adecore/agents 0.31.3 → 0.31.5

package/src/chat/acp-backend.ts

@@ -1,4 +1,6 @@
-import { readFileSync } from 'node:fs';
+import { readFileSync, realpathSync, statSync } from 'node:fs';
+import { isAbsolute, relative, resolve } from 'node:path';
+import { pathToFileURL } from 'node:url';
 import { attachmentImageMime, type ChatAttachment, type ModelSelection } from '@adecore/agent-contracts';
 import { errorText } from '../error-text.ts';
 import { MODE_ORDER } from '../modes.ts';
@@ -14,10 +16,13 @@ import {
     resolveAcpMode,
     runtimeModeOf,
     sessionIdOf,
+    sessionUpdateOf,
+    thoughtLevelOf,
     type AcpAgentCapabilities,
     type AcpModeMap,
     type AcpPermissionRequest,
     type AcpSessionInfo,
+    type AcpThoughtLevel,
     type AcpTurnUsage
 } from './acp-protocol.ts';
 import type { ApprovalDecision, BackendEvent, BackendHost, BackendLaunch, ChatBackend, TurnInput } from './backend.ts';
@@ -54,6 +59,10 @@ export interface AcpAgentSpec {
     onSession?(info: AcpSessionInfo): void;
     // The tokens of every turn whose `session/prompt` response reports them, for a host that keeps a usage log.
     onUsage?(usage: AcpUsageReport): void;
+    // Called once per process, at the first `usage_update` that names the context window.
+    onContextReport?(): void;
+    // The slash commands a session offers (without the slash), each time the agent lists them.
+    onCommands?(commands: string[]): void;
     // A permission request the host answers itself, in any runtime mode: true allows it once without asking the
     // person, for a call the host already guards. A throw counts as false.
     autoAllow?(request: AcpPermissionRequest): boolean;
@@ -69,6 +78,15 @@ export interface AcpUsageReport extends AcpTurnUsage {
 // What `initialize` and a session request may take; a prompt and a login wait for as long as the agent runs.
 const REQUEST_TIMEOUT_MS = 60_000;
 
+// What a prompt carries of mentioned files; a larger file stays a path the agent opens with its own tools.
+const MAX_EMBEDDED_FILE_BYTES = 256 * 1024;
+const MAX_EMBEDDED_BYTES = 1024 * 1024;
+
+function inside(folder: string, path: string): boolean {
+    const rest = relative(folder, path);
+    return rest === '' || (!rest.startsWith('..') && !isAbsolute(rest));
+}
+
 function nameOf(client: AcpClientInfo): string {
     return client.title ?? client.name;
 }
@@ -93,6 +111,10 @@ export class AcpBackend implements ChatBackend {
     private contextPending: string | null = null;
     private prompt: { promptId: string | undefined; cancelled: boolean } | null = null;
     private model: string | null = null;
+    private contextReported = false;
+    private commands: string[] = [];
+    // An agent may send updates for a new session before it answers with the session's id.
+    private early: unknown[] = [];
 
     constructor(launch: BackendLaunch, host: BackendHost, spec: AcpAgentSpec) {
         this.launch = launch;
@@ -154,9 +176,9 @@ export class AcpBackend implements ChatBackend {
         this.capabilities = capabilities;
         const info = await this.openSession(transport, capabilities);
         const modeId = await this.applyMode(transport, info);
-        const model = await this.applyModel(transport, info);
+        const { model, thoughtLevel } = await this.applyModel(transport, info);
         this.model = model;
-        await this.applyThoughtLevel(transport, info);
+        await this.applyThoughtLevel(transport, thoughtLevel);
         this.spec.onSession?.(info);
         this.emit({
             type: 'session',
@@ -165,6 +187,9 @@ export class AcpBackend implements ChatBackend {
             effectiveRuntimeMode: this.launch.runtimeMode,
             ...(modeId === null ? {} : { permissionMode: modeId })
         });
+        for (const params of this.early.splice(0)) {
+            this.emitAll(this.protocol.update(this.sessionId, params));
+        }
     }
 
     private sessionParams(): Record<string, unknown> {
@@ -263,24 +288,29 @@ export class AcpBackend implements ChatBackend {
         return resolved.modeId;
     }
 
-    /* Switches to the selected model where the agent offers it; answers the model the session runs. */
-    private async applyModel(transport: JsonRpcTransport, info: AcpSessionInfo): Promise<string | null> {
+    /*
+     * Switches to the selected model where the agent offers it; answers the model the session runs and the
+     * reasoning effort it offers there, which may change with the model.
+     */
+    private async applyModel(transport: JsonRpcTransport, info: AcpSessionInfo): Promise<{ model: string | null; thoughtLevel: AcpThoughtLevel | null }> {
         const models = info.models;
+        const thoughtLevel = info.thoughtLevel;
         const wanted = this.spec.model ? this.spec.model(this.launch.selection) : this.launch.selection.model;
         if (models === null || wanted === null || wanted === models.current) {
-            return models?.current ?? wanted;
+            return { model: models?.current ?? wanted, thoughtLevel };
         }
         if (!models.models.some((model) => model.id === wanted)) {
             this.emit({ type: 'note', level: 'warning', text: `${this.spec.name} does not offer ${this.launch.modelName}. It runs ${models.current}.` });
-            return models.current;
+            return { model: models.current, thoughtLevel };
         }
         try {
             if (models.configId === null) {
                 await transport.request('session/set_model', { sessionId: this.sessionId, modelId: wanted });
-            } else {
-                await transport.request('session/set_config_option', { sessionId: this.sessionId, configId: models.configId, value: wanted });
+                return { model: wanted, thoughtLevel };
             }
-            return wanted;
+            const result = await transport.request('session/set_config_option', { sessionId: this.sessionId, configId: models.configId, value: wanted });
+            const after = thoughtLevelOf(result);
+            return { model: wanted, thoughtLevel: after === undefined ? thoughtLevel : after };
         } catch (error) {
             if (!(error instanceof JsonRpcError)) {
                 throw error;
@@ -290,21 +320,26 @@ export class AcpBackend implements ChatBackend {
                 level: 'warning',
                 text: `${this.spec.name} did not switch to ${this.launch.modelName} (${errorText(error)}). It runs ${models.current}.`
             });
-            return models.current;
+            return { model: models.current, thoughtLevel };
         }
     }
 
-    private async applyThoughtLevel(transport: JsonRpcTransport, info: AcpSessionInfo): Promise<void> {
-        const level = info.thoughtLevel;
+    /* Sets the selected effort where the session offers it; a model without one runs as it is. */
+    private async applyThoughtLevel(transport: JsonRpcTransport, level: AcpThoughtLevel | null): Promise<void> {
         const effort = this.launch.selection.options.effort;
-        if (level === null || typeof effort !== 'string' || effort === level.current || !level.values.includes(effort)) {
+        const choice = level?.choices.find((entry) => entry.id === effort);
+        if (level === null || choice === undefined || effort === level.current) {
             return;
         }
-        await transport.request('session/set_config_option', { sessionId: this.sessionId, configId: level.configId, value: effort }).catch((error: unknown) => {
+        try {
+            await transport.request('session/set_config_option', { sessionId: this.sessionId, configId: level.configId, value: choice.id });
+        } catch (error) {
             if (!(error instanceof JsonRpcError)) {
                 throw error;
             }
-        });
+            const current = level.choices.find((entry) => entry.id === level.current)?.name ?? level.current;
+            this.emit({ type: 'note', level: 'warning', text: `${this.spec.name} kept its reasoning at ${current} instead of ${choice.name}.` });
+        }
     }
 
     private promptBlocks(input: TurnInput): unknown[] {
@@ -322,7 +357,50 @@ export class AcpBackend implements ChatBackend {
         const named: ChatAttachment[] = input.attachments.filter((attachment) => !images.includes(attachment));
         const parts = [this.contextPending, input.preamble, input.text, attachmentNote(named)].filter((part): part is string => part !== null && part !== '');
         this.contextPending = null;
-        return [{ type: 'text', text: parts.join('\n\n') }, ...blocks];
+        return [{ type: 'text', text: parts.join('\n\n') }, ...blocks, ...this.mentionBlocks(input.mentions)];
+    }
+
+    /*
+     * The text of each mentioned file as a `resource` block, for an agent that takes embedded context. Only a text file
+     * inside the chat's folders goes along, since a mention is a path anyone who may send could name; the rest stays
+     * in the prompt as the path it is.
+     */
+    private mentionBlocks(mentions: readonly string[]): unknown[] {
+        if (this.capabilities?.embeddedContext !== true || mentions.length === 0) {
+            return [];
+        }
+        const real = (path: string): string | null => {
+            try {
+                return realpathSync(path);
+            } catch {
+                return null;
+            }
+        };
+        const folders = [this.launch.cwd, ...(this.launch.folders ?? [])].flatMap((folder) => real(folder) ?? []);
+        const blocks: unknown[] = [];
+        let budget = MAX_EMBEDDED_BYTES;
+        for (const mention of new Set(mentions)) {
+            const path = real(resolve(this.launch.cwd, mention));
+            if (path === null || !folders.some((folder) => inside(folder, path))) {
+                continue;
+            }
+            let text: string;
+            try {
+                const stat = statSync(path);
+                if (!stat.isFile() || stat.size > MAX_EMBEDDED_FILE_BYTES || stat.size > budget) {
+                    continue;
+                }
+                text = readFileSync(path, 'utf8');
+            } catch {
+                continue;
+            }
+            if (text.includes('\u0000')) {
+                continue;
+            }
+            budget -= Buffer.byteLength(text);
+            blocks.push({ type: 'resource', resource: { uri: pathToFileURL(path).href, text } });
+        }
+        return blocks;
     }
 
     sendTurn(input: TurnInput): void {
@@ -352,8 +430,12 @@ export class AcpBackend implements ChatBackend {
             });
     }
 
-    /* ACP has no call to fold the context; a host that offers it sends the agent's own command as a prompt. */
+    /* ACP has no call to fold the context, so an agent that lists a `compact` command gets it as a prompt. */
     compact(): void {
+        if (this.commands.includes('compact')) {
+            this.sendTurn({ text: '/compact', preamble: null, attachments: [], mentions: [], skills: [] });
+            return;
+        }
         this.emit({ type: 'note', level: 'warning', text: `${this.spec.name} cannot fold its context on request.` });
         this.emit({ type: 'turn.done', state: 'done', costUsd: 0 });
     }
@@ -431,9 +513,11 @@ export class AcpBackend implements ChatBackend {
         }
         const id = typeof frame.id === 'number' || typeof frame.id === 'string' ? frame.id : null;
         if (frame.method === 'session/update' && id === null) {
-            if (!this.replaying) {
-                this.emitAll(this.protocol.update(this.sessionId, frame.params));
+            if (this.sessionId === '' || this.replaying) {
+                this.holdEarly(frame.params);
+                return;
             }
+            this.emitAll(this.protocol.update(this.sessionId, frame.params));
             return;
         }
         if (id === null) {
@@ -467,6 +551,14 @@ export class AcpBackend implements ChatBackend {
         this.emitAll(permission.events);
     }
 
+    /* Holds an update until the session is known; of a history a load replays, only the commands, since the thread has the rest. */
+    private holdEarly(params: unknown): void {
+        const update = sessionUpdateOf(params);
+        if (update !== null && (!this.replaying || update.commands !== null)) {
+            this.early.push(params);
+        }
+    }
+
     private hostAllows(request: AcpPermissionRequest): boolean {
         try {
             return this.spec.autoAllow?.(request) === true;
@@ -504,6 +596,14 @@ export class AcpBackend implements ChatBackend {
         if (event.type === 'model') {
             this.model = event.model;
         }
+        if (event.type === 'session' && event.slashCommands !== undefined) {
+            this.commands = event.slashCommands;
+            this.spec.onCommands?.(event.slashCommands);
+        }
+        if (event.type === 'usage' && event.contextWindow !== undefined && !this.contextReported) {
+            this.contextReported = true;
+            this.spec.onContextReport?.();
+        }
         this.host.onEvent(event);
     }
 }

package/src/chat/acp-protocol.ts

@@ -107,7 +107,7 @@ const ModeStateSchema = z.looseObject({
 
 const ModeSchema = z.looseObject({ id: z.string(), name: z.string() });
 
-const SelectValueSchema = z.looseObject({ value: z.string(), name: z.string() });
+const SelectValueSchema = z.looseObject({ value: z.string(), name: z.string(), description: z.string().nullish(), _meta: z.unknown().optional() });
 
 const ConfigOptionSchema = z.looseObject({
     id: z.string(),
@@ -124,7 +124,13 @@ const ModelStateSchema = z.looseObject({
     availableModels: z.array(z.unknown())
 });
 
-const ModelEntrySchema = z.looseObject({ modelId: z.string(), name: z.string(), description: z.string().nullish() });
+const ModelEntrySchema = z.looseObject({ modelId: z.string(), name: z.string(), description: z.string().nullish(), _meta: z.unknown().optional() });
+
+/* The context window an agent names for a model in the model's `_meta` (`totalContextTokens`, not part of the protocol); null where it names none. */
+function contextWindowOf(meta: unknown): number | null {
+    const tokens = isRecord(meta) ? meta.totalContextTokens : undefined;
+    return typeof tokens === 'number' && Number.isSafeInteger(tokens) && tokens > 0 ? tokens : null;
+}
 
 const AuthMethodSchema = z.looseObject({ id: z.string(), name: z.string() });
 
@@ -133,7 +139,7 @@ const InitializeResponseSchema = z.looseObject({
     agentCapabilities: z
         .looseObject({
             loadSession: z.boolean().optional(),
-            promptCapabilities: z.looseObject({ image: z.boolean().optional() }).optional(),
+            promptCapabilities: z.looseObject({ image: z.boolean().optional(), embeddedContext: z.boolean().optional() }).optional(),
             sessionCapabilities: z
                 .looseObject({
                     resume: z.unknown().optional(),
@@ -172,6 +178,8 @@ export interface AcpAgentCapabilities {
     // Whether the agent takes `session/close`, which ends a session it would otherwise keep.
     closeSession: boolean;
     images: boolean;
+    // Whether a prompt may carry a file's text as a `resource` block.
+    embeddedContext: boolean;
     authMethods: string[];
     agent: AcpAgentInfo | null;
 }
@@ -191,6 +199,7 @@ export function parseInitialize(result: unknown): AcpAgentCapabilities {
         additionalDirectories: sessions?.additionalDirectories !== undefined && sessions.additionalDirectories !== null,
         closeSession: sessions?.close !== undefined && sessions.close !== null,
         images: capabilities?.promptCapabilities?.image === true,
+        embeddedContext: capabilities?.promptCapabilities?.embeddedContext === true,
         authMethods: validItems(AuthMethodSchema, parsed.data.authMethods).map((method) => method.id),
         agent: agent ? { name: agent.name, title: agent.title ?? null, version: agent.version } : null
     };
@@ -200,6 +209,8 @@ export interface AcpModel {
     id: string;
     name: string;
     description: string | null;
+    // In tokens, where the agent names it.
+    contextWindow: number | null;
 }
 
 /* The models a session offers. `configId` is the config option that picks one, or null where the agent takes `session/set_model`. */
@@ -221,14 +232,27 @@ export interface AcpModeState {
     configId: string | null;
 }
 
+export interface AcpChoice {
+    id: string;
+    name: string;
+    description: string | null;
+}
+
+/* The reasoning effort a session offers: the config option that sets it, the value it runs at and the values it takes. */
+export interface AcpThoughtLevel {
+    configId: string;
+    current: string;
+    choices: AcpChoice[];
+}
+
 /* What `session/new`, `session/load` or `session/resume` said about the session, for a host that fills its catalog from it. */
 export interface AcpSessionInfo {
     sessionId: string;
     agent: AcpAgentInfo | null;
     models: AcpModelState | null;
     modes: AcpModeState | null;
-    // The reasoning effort a session offers, as a config option with the `thought_level` category.
-    thoughtLevel: { configId: string; current: string; values: string[] } | null;
+    // A config option of the `thought_level` category, or one with the id `reasoning_effort` where an agent sets no category.
+    thoughtLevel: AcpThoughtLevel | null;
 }
 
 type ConfigOption = z.infer<typeof ConfigOptionSchema>;
@@ -243,6 +267,22 @@ function selectOption(options: ConfigOption[], category: string): ConfigOption |
     return options.find((option) => option.type === 'select' && option.category === category && typeof option.currentValue === 'string') ?? null;
 }
 
+function thoughtOption(options: ConfigOption[]): ConfigOption | null {
+    return (
+        selectOption(options, 'thought_level') ??
+        options.find((option) => option.type === 'select' && option.id === 'reasoning_effort' && typeof option.currentValue === 'string') ??
+        null
+    );
+}
+
+/* The reasoning effort in a response that carries the session's config options; undefined when it carries none. */
+export function thoughtLevelOf(result: unknown): AcpThoughtLevel | null | undefined {
+    if (!isRecord(result) || !Array.isArray(result.configOptions)) {
+        return undefined;
+    }
+    return sessionState(validItems(ConfigOptionSchema, result.configOptions), null, null).thoughtLevel;
+}
+
 export function parseSessionInfo(result: unknown, sessionId: string, agent: AcpAgentInfo | null): AcpSessionInfo {
     const record = isRecord(result) ? result : {};
     const options = validItems(ConfigOptionSchema, Array.isArray(record.configOptions) ? record.configOptions : []);
@@ -252,14 +292,19 @@ export function parseSessionInfo(result: unknown, sessionId: string, agent: AcpA
 function sessionState(options: ConfigOption[], modes: unknown, models: unknown): Omit<AcpSessionInfo, 'sessionId' | 'agent'> {
     const modelOption = selectOption(options, 'model');
     const modeOption = selectOption(options, 'mode');
-    const thoughtOption = selectOption(options, 'thought_level');
+    const thought = thoughtOption(options);
     const modeState = ModeStateSchema.safeParse(modes);
     const modelState = ModelStateSchema.safeParse(models);
     return {
         models: modelOption
             ? {
                   current: modelOption.currentValue as string,
-                  models: selectValues(modelOption).map((value) => ({ id: value.value, name: value.name, description: null })),
+                  models: selectValues(modelOption).map((value) => ({
+                      id: value.value,
+                      name: value.name,
+                      description: null,
+                      contextWindow: contextWindowOf(value._meta)
+                  })),
                   configId: modelOption.id
               }
             : modelState.success
@@ -268,7 +313,8 @@ function sessionState(options: ConfigOption[], modes: unknown, models: unknown):
                     models: validItems(ModelEntrySchema, modelState.data.availableModels).map((model) => ({
                         id: model.modelId,
                         name: model.name,
-                        description: model.description ?? null
+                        description: model.description ?? null,
+                        contextWindow: contextWindowOf(model._meta)
                     })),
                     configId: null
                 }
@@ -286,12 +332,33 @@ function sessionState(options: ConfigOption[], modes: unknown, models: unknown):
                     configId: null
                 }
               : null,
-        thoughtLevel: thoughtOption
-            ? { configId: thoughtOption.id, current: thoughtOption.currentValue as string, values: selectValues(thoughtOption).map((value) => value.value) }
+        thoughtLevel: thought
+            ? {
+                  configId: thought.id,
+                  current: thought.currentValue as string,
+                  choices: selectValues(thought).map((value) => ({ id: value.value, name: value.name, description: value.description ?? null }))
+              }
             : null
     };
 }
 
+function commandNames(commands: unknown): string[] {
+    return validItems(CommandSchema, Array.isArray(commands) ? commands : []).map((command) => command.name);
+}
+
+/* The session a `session/update` notification is for, and the commands it lists when it is an `available_commands_update`; null when it does not parse. */
+export function sessionUpdateOf(params: unknown): { sessionId: string; commands: string[] | null } | null {
+    const notification = SessionNotificationSchema.safeParse(params);
+    if (!notification.success) {
+        return null;
+    }
+    const update = notification.data.update;
+    return {
+        sessionId: notification.data.sessionId,
+        commands: update.sessionUpdate === 'available_commands_update' ? commandNames(update.availableCommands) : null
+    };
+}
+
 export function sessionIdOf(result: unknown): string | null {
     const parsed = SessionResponseSchema.safeParse(result);
     return parsed.success ? (parsed.data.sessionId ?? null) : null;
@@ -557,10 +624,8 @@ export class AcpProtocol {
             }
             case 'plan':
                 return this.plan(update);
-            case 'available_commands_update': {
-                const commands = validItems(CommandSchema, Array.isArray(update.availableCommands) ? update.availableCommands : []);
-                return [{ type: 'session', agentSessionId: null, model: null, slashCommands: commands.map((command) => command.name) }];
-            }
+            case 'available_commands_update':
+                return [{ type: 'session', agentSessionId: null, model: null, slashCommands: commandNames(update.availableCommands) }];
             case 'current_mode_update':
                 return typeof update.currentModeId === 'string' ? [this.modeEvent(update.currentModeId)] : [];
             case 'config_option_update': {

package/src/chat/index.ts

@@ -9,6 +9,8 @@ export type {
     AcpPermissionRequest,
     AcpSessionInfo,
     AcpModeMap,
+    AcpChoice,
+    AcpThoughtLevel,
     AcpTurnUsage
 } from './acp-protocol.ts';
 export { AttachmentStore } from './attachment-store.ts';

package/src/providers/accounts/acp-account.ts

@@ -2,7 +2,15 @@ import { mkdtemp, rm } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { DEFAULT_ACP_CLIENT, type AcpClientInfo } from '../../chat/acp-backend.ts';
-import { ACP_AUTH_REQUIRED, ACP_PROTOCOL_VERSION, parseInitialize, parseSessionInfo, sessionIdOf, type AcpSessionInfo } from '../../chat/acp-protocol.ts';
+import {
+    ACP_AUTH_REQUIRED,
+    ACP_PROTOCOL_VERSION,
+    parseInitialize,
+    parseSessionInfo,
+    sessionIdOf,
+    sessionUpdateOf,
+    type AcpSessionInfo
+} from '../../chat/acp-protocol.ts';
 import type { SpawnChatProcess } from '../../chat/chat-process.ts';
 import { JsonRpcError, JsonRpcTransport, type JsonRpcFrame } from '../../chat/json-rpc-transport.ts';
 import { ASK_TIMEOUT_MS, type AccountReading } from './status.ts';
@@ -24,6 +32,8 @@ export interface AcpAccountProbe {
     timer?: ProbeTimer;
     // What the probe's session offered, for a host that fills its model catalog from the agent.
     onSession?(info: AcpSessionInfo): void;
+    // The slash commands the probe's session offered, when the agent listed them before the session closed.
+    onCommands?(commands: string[]): void;
 }
 
 const SIGNED_OUT: AccountReading = { signedIn: false, email: null, plan: null, organization: null };
@@ -63,6 +73,20 @@ export async function askAcpAccount(probe: AcpAccountProbe): Promise<AccountRead
     const folder = await mkdtemp(join(tmpdir(), 'acp-account-'));
     const client = probe.client ?? DEFAULT_ACP_CLIENT;
     let transport: JsonRpcTransport | null = null;
+    // An agent may list its commands before it answers with the session's id.
+    const listed = new Map<string, string[]>();
+    let sessionId: string | null = null;
+    const noteCommands = (frame: JsonRpcFrame): void => {
+        const update = frame.method === 'session/update' && (frame.id === undefined || frame.id === null) ? sessionUpdateOf(frame.params) : null;
+        if (update === null || update.commands === null) {
+            return;
+        }
+        if (sessionId === null) {
+            listed.set(update.sessionId, update.commands);
+        } else if (update.sessionId === sessionId) {
+            probe.onCommands?.(update.commands);
+        }
+    };
     try {
         const started = new JsonRpcTransport({
             command: [...probe.command],
@@ -74,7 +98,10 @@ export async function askAcpAccount(probe: AcpAccountProbe): Promise<AccountRead
             // The probe's own deadline covers every request.
             requestTimeoutMs: null,
             versionField: true,
-            onFrame: (frame) => refuseRequest(started, frame),
+            onFrame: (frame) => {
+                noteCommands(frame);
+                refuseRequest(started, frame);
+            },
             onExit: () => undefined
         });
         transport = started;
@@ -98,11 +125,15 @@ export async function askAcpAccount(probe: AcpAccountProbe): Promise<AccountRead
                 }
                 throw error;
             }
-            const sessionId = sessionIdOf(result);
+            sessionId = sessionIdOf(result);
             if (sessionId === null) {
                 throw new Error(`${probe.name} opened a session without an id`);
             }
             probe.onSession?.(parseSessionInfo(result, sessionId, capabilities.agent));
+            const commands = listed.get(sessionId);
+            if (commands !== undefined) {
+                probe.onCommands?.(commands);
+            }
             if (capabilities.closeSession) {
                 // The answer is known by now; an agent that cannot close it keeps an empty session, nothing worse.
                 await started.request('session/close', { sessionId }).catch(() => undefined);