@agentshouse/kit 0.2.1-c7f53ff9eaf98 → 0.2.1-c845e172df1af

package/dist/conversations.js

@@ -4,13 +4,16 @@ import { mkdir } from 'node:fs/promises';
 import { join } from 'node:path';
 import { unlessAborted } from './abort.js';
 import { startAdapter } from './acp.js';
+import { finalRefusal, HouseRefusal } from './api.js';
 import { openBridge } from './bridge.js';
 import { CLIS } from './clis.js';
 import { answeredFailure, readerFor } from './events.js';
-import { placeFiles, uploadBytes } from './files.js';
+import { placeFiles, storeRefusal, uploaded } from './files.js';
 import { agentBase } from './home.js';
 import { instructions } from './instructions.js';
 import { Parts } from './parts.js';
+import { refusalOf } from './refusals.js';
+import { relayed } from './relay.js';
 import { killMarked, Scope } from './scope.js';
 import { holdSecretInput } from './secret-input.js';
 import { createHowWeWork } from './skills.js';
@@ -19,10 +22,17 @@ import { workFor } from './work.js';
 const SIGNED_OUT = RequestError.authRequired().code;
 const KILLED = 'the conversation was killed';
 const REFUSED = 'the CLI refused the prompt';
-const NO_CONNECTION = "the Kit could not open this conversation's connection";
+const NO_CONNECTION = "the Agent's computer could not open this conversation; try again";
+const GONE = 'House no longer holds this conversation; start a new one';
+const RETIRED = "this conversation's Agent is retired, so it cannot start";
+const ENDED = {
+    conversation_not_found: GONE,
+    conversation_historical: RETIRED,
+};
 const NO_START = "the Agent's CLI did not start";
 const NO_SESSION = "the Agent's CLI did not open a session";
 const NO_RESUME = "the Agent's CLI could not resume this conversation";
+const NOT_SIGNED_IN = "the Agent's CLI is not signed in; sign it in on the Agent's machine";
 const NO_SETTINGS = "the Agent's CLI refused the Agent's model, mode or effort";
 class NotStarted extends Error {
 }
@@ -31,9 +41,20 @@ async function step(reason, work) {
         return await work;
     }
     catch (error) {
-        throw new NotStarted(reason, { cause: error });
+        throw new NotStarted(typeof reason === 'string' ? reason : reason(error), { cause: error });
     }
 }
+function signedOut(error) {
+    return typeof error === 'object' && error !== null && error.code === SIGNED_OUT;
+}
+function unopened(reason) {
+    return (error) => (signedOut(error) ? NOT_SIGNED_IN : reason);
+}
+// A start-up step is shown only once it has lasted a second, so a quick start shows none of them.
+const STARTING_SHOWN_AFTER_MS = 1_000;
+function unbridged(error) {
+    return (finalRefusal(error) ? ENDED[refusalOf(error.text)?.code ?? ''] : undefined) ?? NO_CONNECTION;
+}
 const EXIT_PLAN_MODE = {
     parse(value) {
         if (typeof value !== 'object' || value === null || Array.isArray(value)) {
@@ -42,13 +63,12 @@ const EXIT_PLAN_MODE = {
         return value;
     },
 };
-export const CONVERSATION_INPUTS = new Set(['open', 'message', 'interrupt', 'kill', 'option', 'answer']);
+export const CONVERSATION_METHODS = new Set(['open', 'message', 'interrupt', 'kill', 'option', 'answer']);
 function causeOf(error) {
     return error instanceof Error ? error.message : String(error);
 }
 function answeredWith(error) {
-    const signedOut = typeof error === 'object' && error !== null && error.code === SIGNED_OUT;
-    return { failure: signedOut ? 'signed_out' : 'error', cause: causeOf(error) };
+    return { failure: signedOut(error) ? 'signed_out' : 'error', cause: causeOf(error) };
 }
 function exitedWith(running, cause) {
     return running.killed ? { failure: 'killed', cause: KILLED } : { failure: 'exited', cause };
@@ -56,6 +76,12 @@ function exitedWith(running, cause) {
 function answerMessage(request, response) {
     return `The User answered a question you asked earlier.\nQuestion: ${JSON.stringify(request)}\nAnswer: ${JSON.stringify(response)}`;
 }
+async function sentPart(url, bytes) {
+    const answer = await fetch(relayed(url), { method: 'PUT', body: bytes });
+    await answer.arrayBuffer();
+    if (!answer.ok)
+        throw storeRefusal('upload', answer.status);
+}
 function logged(error) {
     process.stderr.write(`kit: ${causeOf(error)}\n`);
 }
@@ -72,10 +98,10 @@ function marker(bridge) {
 class Turn {
     id = randomUUID();
     parts;
-    admitted = false;
     questions = 0;
     secrets = 0;
     prompted = false;
+    taken = null;
     ended = false;
     images = 0;
     uploads = [];
@@ -89,6 +115,8 @@ class Conversation {
     session = null;
     turn = null;
     commands = null;
+    options = null;
+    exitUntold = false;
     context = null;
     leftovers = new Set();
     waiting = [];
@@ -96,6 +124,7 @@ class Conversation {
     subagents = new Map();
     held = [];
     opening = null;
+    launch = null;
     prompted = null;
     kills = 0;
     killed = new AbortController();
@@ -104,6 +133,11 @@ class Conversation {
     open = 0;
     queue = Promise.resolve();
     reports = Promise.resolve();
+    starting = null;
+    preparing = null;
+    told = null;
+    untold = false;
+    shown = null;
     constructor(id) {
         this.id = id;
     }
@@ -119,9 +153,7 @@ function asksSecret(request) {
 export class Conversations {
     kit;
     conversations = new Map();
-    acks = new Map();
     questions = new Map();
-    carrying = 0;
     reporting = 0;
     watched = false;
     constructor(kit) {
@@ -129,38 +161,27 @@ export class Conversations {
         // A process or a durable schedule ends without a report, so the Kit looks at both again every two seconds.
         setInterval(() => this.look(), 2_000);
     }
-    input(input) {
-        this.carrying++;
-        const carried = this.acks.get(input.input_id) ??
-            (() => {
-                const conversation = this.conversation(input.conversation_id);
-                if (input.kind === 'kill') {
-                    conversation.kills++;
-                    conversation.killed.abort(new Error(KILLED));
-                    for (const waiting of conversation.waiting.splice(0))
-                        waiting.settle({ refused: KILLED });
-                    this.halt(conversation);
-                }
-                if (input.kind === 'interrupt') {
-                    conversation.interrupts++;
-                    conversation.interrupted.abort();
-                }
-                const handled = conversation.queue.then(() => this.carry(conversation, input));
-                conversation.queue = handled.catch(() => undefined);
-                const work = handled.then(({ ack }) => ack);
-                this.acks.set(input.input_id, work);
-                return work;
-            })();
-        carried
-            .then((body) => this.kit.house.deliver(`/kit/inputs/${input.input_id}/ack`, body))
-            .catch(logged)
-            .finally(() => {
-            this.carrying--;
-            this.kit.changed();
+    call(call) {
+        void this.kit.calls.carry(call.input_id, () => {
+            const conversation = this.conversation(call.conversation_id);
+            if (call.kind === 'kill') {
+                conversation.kills++;
+                conversation.killed.abort(new Error(KILLED));
+                for (const waiting of conversation.waiting.splice(0))
+                    waiting.settle({ refused: KILLED });
+                this.halt(conversation);
+            }
+            if (call.kind === 'interrupt') {
+                conversation.interrupts++;
+                conversation.interrupted.abort();
+            }
+            const handled = conversation.queue.then(() => this.carry(conversation, call));
+            conversation.queue = handled.catch(() => undefined);
+            return handled.then(async ({ result }) => ({ result: await result }));
         });
     }
     idle() {
-        if (this.carrying > 0 || this.reporting > 0)
+        if (this.reporting > 0)
             return false;
         for (const conversation of this.conversations.values()) {
             const turn = conversation.turn;
@@ -189,6 +210,14 @@ export class Conversations {
     }
     opened() {
         this.watched = false;
+        for (const conversation of this.conversations.values()) {
+            if (conversation.running !== null)
+                this.facts(conversation);
+            else if (conversation.exitUntold)
+                this.process(conversation, false);
+            if (conversation.untold || conversation.told !== null)
+                this.retell(conversation);
+        }
     }
     watching(watched) {
         const resend = watched && !this.watched;
@@ -208,50 +237,87 @@ export class Conversations {
         }
         return conversation;
     }
-    async carry(conversation, input) {
-        if (input.kind === 'open')
-            return { ack: Promise.resolve(await this.open(conversation, input)) };
-        if (input.kind === 'message')
-            return this.message(conversation, input);
-        if (input.kind === 'interrupt') {
+    async carry(conversation, call) {
+        if (call.kind === 'open')
+            return { result: this.open(conversation, call) };
+        if (call.kind === 'message')
+            return this.message(conversation, call);
+        await this.launched(conversation);
+        if (call.kind === 'interrupt') {
             if (--conversation.interrupts === 0)
                 conversation.interrupted = new AbortController();
-            await this.interrupt(conversation, String(input.turn_id));
+            await this.interrupt(conversation, String(call.turn_id));
         }
-        if (input.kind === 'kill') {
+        if (call.kind === 'kill') {
             await this.kill(conversation);
             await conversation.sending;
             if (--conversation.kills === 0)
                 conversation.killed = new AbortController();
         }
-        if (input.kind === 'option')
-            await this.option(conversation, String(input.option), input.value);
-        if (input.kind === 'answer') {
-            const question = this.questions.get(String(input.interaction_id));
+        if (call.kind === 'option')
+            await this.option(conversation, String(call.option), call.value);
+        if (call.kind === 'answer') {
+            const question = this.questions.get(String(call.interaction_id));
             if (question === undefined) {
-                const text = answerMessage(input.request, input.response);
-                return this.message(conversation, { ...input, text, files: [], first: false });
+                const text = answerMessage(call.request, call.response);
+                return this.message(conversation, { ...call, text, files: [], first: false });
             }
-            question.answer(input.response);
+            question.answer(call.response);
         }
-        return { ack: Promise.resolve({}) };
+        return { result: Promise.resolve({}) };
     }
-    async open(conversation, input) {
+    open(conversation, call) {
         if (conversation.running !== null)
-            return {};
-        try {
-            return { provider_session_id: await this.start(conversation, await this.route(conversation, input.agent_id), null, true) };
-        }
-        catch (error) {
-            return { refused: refusal(conversation, error) };
-        }
+            return Promise.resolve({});
+        const launch = conversation.launch ?? this.launch(conversation, this.route(conversation, call.agent_id), null, true);
+        return launch.session.then((session) => ({ provider_session_id: session }), (error) => ({ refused: refusal(conversation, error) }));
+    }
+    launch(conversation, route, sessionId, empty) {
+        let opened;
+        let failed;
+        const bridge = new Promise((resolve, reject) => {
+            opened = resolve;
+            failed = reject;
+        });
+        bridge.catch(() => undefined);
+        const session = this.start(conversation, route, sessionId, empty, opened);
+        session.catch(failed);
+        const launch = { bridge, session };
+        conversation.launch = launch;
+        void session
+            .catch(() => undefined)
+            .then(() => {
+            if (conversation.launch !== launch)
+                return;
+            conversation.launch = null;
+            this.next(conversation);
+        });
+        this.next(conversation);
+        return launch;
     }
-    async message(conversation, input) {
-        const content = [{ type: 'text', text: String(input.text) }];
+    async launched(conversation) {
+        while (conversation.launch !== null)
+            await conversation.launch.session.catch(() => undefined);
+    }
+    instructed(conversation, route) {
+        const bridge = conversation.running === null ? conversation.launch?.bridge : Promise.resolve(conversation.running.bridge);
+        if (bridge === undefined)
+            return null;
+        const read = Promise.all([bridge, this.kit.copies.mapping()]).then(([opened, copies]) => instructions(opened, route.base_instructions, copies));
+        read.catch(() => undefined);
+        return read;
+    }
+    async message(conversation, call) {
+        const content = [{ type: 'text', text: String(call.text) }];
         let route;
+        let instructing = null;
         try {
-            route = await this.route(conversation, input.agent_id);
-            const files = input.files;
+            route = await this.route(conversation, call.agent_id);
+            if (call.first === true)
+                instructing = this.instructed(conversation, route);
+            const files = call.files;
+            if (files.length > 0)
+                this.prepares(conversation, 'files');
             const paths = await placeFiles(this.kit.house, route.working_directory, files, conversation.killed.signal);
             const lines = await this.transcribed(conversation, files, paths);
             if (lines.length > 0)
@@ -260,11 +326,13 @@ export class Conversations {
                 throw new Error(KILLED);
         }
         catch (error) {
-            return { ack: Promise.resolve({ refused: refusal(conversation, error) }) };
+            this.prepares(conversation, null);
+            return { result: Promise.resolve({ refused: refusal(conversation, error) }) };
         }
+        conversation.preparing = null;
         return {
-            ack: new Promise((settle) => {
-                conversation.waiting.push({ input, route, content, settle });
+            result: new Promise((settle) => {
+                conversation.waiting.push({ call, route, content, instructing, settle });
                 this.next(conversation);
             }),
         };
@@ -273,38 +341,48 @@ export class Conversations {
         if (conversation.sending !== null || conversation.open > 0)
             return;
         const waiting = conversation.waiting.shift();
-        if (waiting === undefined)
+        if (waiting === undefined) {
+            const step = conversation.preparing ?? (conversation.launch === null ? null : 'session');
+            if (conversation.starting !== step)
+                this.starts(conversation, step);
             return;
+        }
         conversation.sending = this.send(conversation, waiting).finally(() => {
             conversation.sending = null;
             this.next(conversation);
         });
     }
     async send(conversation, waiting) {
-        const { input } = waiting;
-        let ack = {};
+        const { call } = waiting;
+        let result = {};
         let started = false;
+        let instructing = waiting.instructing;
         const prompt = [];
         try {
             if (conversation.running === null) {
-                const stored = conversation.session ?? input.provider_session_id;
-                const empty = input.first === true && conversation.prompted !== stored;
-                started = (await this.start(conversation, waiting.route, stored, empty)) !== stored;
+                this.starts(conversation, 'session');
+                await this.launched(conversation);
+            }
+            if (conversation.running === null) {
+                const stored = conversation.session ?? call.provider_session_id;
+                const empty = call.first === true && conversation.prompted !== stored;
+                const launch = this.launch(conversation, Promise.resolve(waiting.route), stored, empty);
+                if (call.first === true)
+                    instructing = this.instructed(conversation, waiting.route);
+                started = (await launch.session) !== stored;
             }
             const session = conversation.running.sessionId;
-            if (started || (input.first === true && session !== input.provider_session_id)) {
-                ack = { provider_session_id: session };
+            if (started || (call.first === true && session !== call.provider_session_id)) {
+                result = { provider_session_id: session };
             }
-            if (input.first === true) {
-                prompt.push({
-                    type: 'text',
-                    text: await instructions(conversation.running.bridge, waiting.route.base_instructions, await this.kit.copies.mapping()),
-                });
+            if (call.first === true) {
+                prompt.push({ type: 'text', text: await (instructing ?? this.instructed(conversation, waiting.route)) });
             }
             if (conversation.kills > 0)
                 throw new Error(KILLED);
         }
         catch (error) {
+            this.starts(conversation, null);
             if (started) {
                 conversation.session = null;
                 await this.kill(conversation);
@@ -312,8 +390,37 @@ export class Conversations {
             waiting.settle({ refused: refusal(conversation, error) });
             return;
         }
-        this.prompt(conversation, [...prompt, ...waiting.content]);
-        waiting.settle(ack);
+        this.starts(conversation, null);
+        this.prompt(conversation, [...prompt, ...waiting.content], () => this.kit.calls.keepResult(String(call.input_id), result));
+        waiting.settle(result);
+    }
+    starts(conversation, step) {
+        conversation.starting = step;
+        if (conversation.shown !== null)
+            clearTimeout(conversation.shown);
+        conversation.shown = null;
+        if (step === null || conversation.told !== null) {
+            this.tell(conversation, step);
+            return;
+        }
+        conversation.shown = setTimeout(() => {
+            conversation.shown = null;
+            if (conversation.starting === step)
+                this.tell(conversation, step);
+        }, STARTING_SHOWN_AFTER_MS);
+    }
+    prepares(conversation, step) {
+        conversation.preparing = step;
+        this.next(conversation);
+    }
+    tell(conversation, step) {
+        if (conversation.told === step && !conversation.untold)
+            return;
+        conversation.told = step;
+        this.retell(conversation);
+    }
+    retell(conversation) {
+        conversation.untold = !this.kit.send({ type: 'starting', conversation_id: conversation.id, step: conversation.told });
     }
     async transcribed(conversation, files, paths) {
         const lines = [];
@@ -321,6 +428,7 @@ export class Conversations {
             lines.push(path);
             if (files[index].media_type?.startsWith('audio/') !== true)
                 continue;
+            this.prepares(conversation, 'transcription');
             try {
                 const text = await transcribe(path, AbortSignal.any([conversation.killed.signal, conversation.interrupted.signal]));
                 if (text !== null && text !== '')
@@ -387,7 +495,8 @@ export class Conversations {
         }
         return route;
     }
-    async start(conversation, route, sessionId, empty) {
+    async start(conversation, routed, sessionId, empty, bridged) {
+        const route = await routed;
         const reader = readerFor(route.kind);
         reader.session = sessionId;
         const app = client({ name: '@agentshouse/kit' })
@@ -395,7 +504,7 @@ export class Conversations {
             .onRequest('elicitation/create', ({ params, signal }) => this.ask(conversation, 'elicitation/create', params, asksSecret(params), signal))
             .onRequest('_x.ai/exit_plan_mode', EXIT_PLAN_MODE, ({ params, signal }) => this.ask(conversation, '_x.ai/exit_plan_mode', params, false, signal));
         const cli = await unlessAborted(this.kit.agents.cli(route.kind), conversation.killed.signal);
-        const bridge = await step(NO_CONNECTION, openBridge(this.kit.house, this.kit.copies, conversation.id, conversation.killed.signal));
+        const bridge = await step(unbridged, openBridge(this.kit.house, this.kit.copies, conversation.id, conversation.killed.signal));
         if (conversation.kills > 0) {
             bridge.close();
             throw new Error(KILLED);
@@ -403,11 +512,13 @@ export class Conversations {
         conversation.opening = bridge;
         const work = workFor(route.kind, route.working_directory, () => this.kit.changed());
         const scope = new Scope(marker(bridge), randomUUID(), cli);
+        let adapter = null;
         try {
             await unlessAborted(createHowWeWork(bridge), conversation.killed.signal);
+            bridged(bridge);
             await CLIS[route.kind].allowHouse(bridge.env.PATH);
             const streamed = await step(NO_CONNECTION, work.open(cli));
-            const adapter = await step(NO_START, startAdapter(route.kind, cli, route.working_directory, app, (message) => {
+            adapter = await step(NO_START, startAdapter(route.kind, cli, route.working_directory, app, (message) => {
                 work.read(message);
                 this.read(conversation, reader, message);
             }, { ...bridge.env, ...streamed }, undefined, scope));
@@ -416,26 +527,24 @@ export class Conversations {
             const cwd = route.working_directory;
             const _meta = CLIS[route.kind].sessionMeta(bridge.env.HOUSE_BRIDGE, conversation.id);
             const resumed = sessionId === null ? null : await this.resume(conversation, route.kind, adapter, { sessionId, cwd, mcpServers: [], _meta }, empty);
-            const opened = resumed ?? (await step(NO_SESSION, agent.request('session/new', { cwd, mcpServers: [], _meta })));
+            const opened = resumed ?? (await step(unopened(NO_SESSION), agent.request('session/new', { cwd, mcpServers: [], _meta })));
             reader.session = opened.sessionId;
             if (opened.sessionId !== sessionId && CLIS[route.kind].renames) {
                 const prompt = [{ type: 'text', text: `/rename ${conversation.id}` }];
-                await step(NO_SESSION, agent.request('session/prompt', { sessionId: opened.sessionId, prompt }));
+                await step(unopened(NO_SESSION), agent.request('session/prompt', { sessionId: opened.sessionId, prompt }));
             }
             const running = { adapter, bridge, sessionId: opened.sessionId, killed: false, reader, work, scope };
             const options = await step(NO_SETTINGS, this.launchSettings(running, route, opened.configOptions ?? []));
             void work.settled().then(() => scope.record());
             conversation.running = running;
             conversation.session = running.sessionId;
-            this.kit.send({ type: 'process', conversation_id: conversation.id, running: true });
-            if (conversation.commands !== null)
-                this.commands(conversation, conversation.commands);
-            this.options(conversation, options);
-            void adapter.exited.then((cause) => this.exited(conversation, adapter, cause));
+            conversation.options = options;
+            this.facts(conversation);
+            void adapter.lost.then((cause) => this.exited(conversation, running, cause));
             return running.sessionId;
         }
         catch (error) {
-            scope.kill();
+            adapter?.retire();
             work.close();
             conversation.leftovers.add(scope);
             bridge.close();
@@ -451,7 +560,7 @@ export class Conversations {
         }
         catch (error) {
             if (!empty || conversation.kills > 0 || !CLIS[kind].unknownSession(error)) {
-                throw new NotStarted(NO_RESUME, { cause: error });
+                throw new NotStarted(unopened(NO_RESUME)(error), { cause: error });
             }
             logged(new Error(`conversation ${conversation.id} opens a new session: ${NO_RESUME}: ${causeOf(error)}`));
             return null;
@@ -478,10 +587,7 @@ export class Conversations {
         }
         return options;
     }
-    exited(conversation, adapter, cause) {
-        const running = conversation.running;
-        if (running?.adapter !== adapter)
-            return;
+    exited(conversation, running, cause) {
         this.detach(conversation, running);
         if (conversation.turn !== null) {
             this.end(conversation, conversation.turn, exitedWith(running, cause));
@@ -494,11 +600,22 @@ export class Conversations {
         conversation.leftovers.add(running.scope);
         conversation.running = null;
         conversation.commands = null;
+        conversation.options = null;
         for (const [id, question] of this.questions) {
             if (question.conversation === conversation)
                 this.questions.delete(id);
         }
-        this.kit.send({ type: 'process', conversation_id: conversation.id, running: false });
+        this.process(conversation, false);
+    }
+    process(conversation, running) {
+        conversation.exitUntold = !this.kit.send({ type: 'process', conversation_id: conversation.id, running }) && !running;
+    }
+    facts(conversation) {
+        this.process(conversation, true);
+        if (conversation.commands !== null)
+            this.commands(conversation, conversation.commands);
+        if (conversation.options !== null)
+            this.options(conversation, conversation.options);
     }
     commands(conversation, commands) {
         conversation.commands = commands;
@@ -509,6 +626,7 @@ export class Conversations {
     options(conversation, options) {
         if (conversation.running === null)
             return;
+        conversation.options = options;
         this.kit.send({ type: 'options', conversation_id: conversation.id, options });
     }
     deliver(conversation, turn, event, body) {
@@ -523,7 +641,7 @@ export class Conversations {
         return delivered;
     }
     write(conversation, turn, operation) {
-        if (!turn.admitted || turn.ended || !this.watched)
+        if (turn.ended || !this.watched)
             return;
         this.kit.send({ type: 'turn', conversation_id: conversation.id, turn_id: turn.id, ...operation });
     }
@@ -534,12 +652,7 @@ export class Conversations {
     begin(conversation) {
         const turn = new Turn((operation) => this.write(conversation, turn, operation));
         conversation.open++;
-        void this.deliver(conversation, turn, 'started', {})
-            .then(() => {
-            turn.admitted = true;
-            this.replay(conversation, turn);
-        })
-            .catch(logged);
+        void this.deliver(conversation, turn, 'started', {}).catch(logged);
         conversation.turn = turn;
         if (conversation.context !== null)
             turn.parts.measure(conversation.context);
@@ -572,12 +685,14 @@ export class Conversations {
         this.next(conversation);
         this.kit.changed();
     }
-    prompt(conversation, prompt) {
+    prompt(conversation, prompt, taken) {
         const running = conversation.running;
         const turn = this.begin(conversation);
         turn.prompted = true;
+        turn.taken = taken;
         conversation.prompted = running.sessionId;
         running.adapter.connection.agent.request('session/prompt', { sessionId: running.sessionId, prompt }).then((answer) => {
+            this.took(turn);
             const notice = answeredFailure(answer);
             if (notice !== null) {
                 this.end(conversation, turn, { failure: notice.limit ? 'limit' : 'error', cause: notice.title }, notice.title);
@@ -590,8 +705,9 @@ export class Conversations {
                 this.end(conversation, turn, exitedWith(running, await running.adapter.exited));
             }
             else {
+                this.took(turn);
                 if (conversation.open === 1 && conversation.running === running) {
-                    running.scope.kill();
+                    running.adapter.retire();
                     this.detach(conversation, running);
                 }
                 this.end(conversation, turn, answeredWith(error));
@@ -599,7 +715,9 @@ export class Conversations {
         });
     }
     started(conversation) {
-        return conversation.turn ?? this.begin(conversation);
+        const turn = conversation.turn ?? this.begin(conversation);
+        this.took(turn);
+        return turn;
     }
     ended(conversation) {
         const turn = conversation.turn;
@@ -694,6 +812,11 @@ export class Conversations {
         }
         this.apply(conversation, this.started(conversation), event);
     }
+    took(turn) {
+        const taken = turn.taken;
+        turn.taken = null;
+        taken?.();
+    }
     apply(conversation, turn, event) {
         switch (event.kind) {
             case 'text':
@@ -748,13 +871,16 @@ export class Conversations {
                 bytes: bytes.length,
                 sha256: createHash('sha256').update(bytes).digest('hex'),
             }));
-            const saved = declared.upload === undefined ? declared : await uploadBytes(declared.upload, bytes);
-            if (saved.save.status !== 'saved')
-                throw new Error(saved.save.failure ?? saved.save.status);
-            turn.parts.arrive({ type: 'image', file: saved.version, name, media_type: mediaType }, reserved);
+            const saved = await uploaded(declared.upload, {
+                bytes: bytes.length,
+                send: (url, start, end) => sentPart(url, bytes.subarray(start, end)),
+                finish: (upload) => this.kit.house.deliver('/kit/uploads/finish', { upload }),
+            });
+            turn.parts.arrive({ type: 'image', file: saved.path, name, media_type: mediaType }, reserved);
         }
         catch (error) {
-            turn.parts.arrive({ type: 'marker', marker: 'retry', text: `${name} was not saved: ${causeOf(error)}` }, reserved);
+            const cause = error instanceof HouseRefusal ? (refusalOf(error.text)?.text ?? causeOf(error)) : causeOf(error);
+            turn.parts.arrive({ type: 'marker', marker: 'retry', text: `${name} was not saved: ${cause}` }, reserved);
         }
     }
 }