@agi-cli/server 0.1.59 → 0.1.61

package/src/runtime/stream-handlers.ts

@@ -8,17 +8,26 @@ import { toErrorPayload } from './error-handling.ts';
 import type { RunOpts } from './session-queue.ts';
 import type { ToolAdapterContext } from '../tools/adapter.ts';
 
-type StepFinishEvent = {
-	usage?: {
-		inputTokens?: number;
-		outputTokens?: number;
-		totalTokens?: number;
-		cachedInputTokens?: number;
-		reasoningTokens?: number;
+interface ProviderMetadata {
+	openai?: {
+		cachedPromptTokens?: number;
 	};
+	[key: string]: unknown;
+}
+
+interface UsageData {
+	inputTokens?: number;
+	outputTokens?: number;
+	totalTokens?: number;
+	cachedInputTokens?: number;
+	reasoningTokens?: number;
+}
+
+type StepFinishEvent = {
+	usage?: UsageData;
 	finishReason?: string;
 	response?: unknown;
-	experimental_providerMetadata?: Record<string, any>;
+	experimental_providerMetadata?: ProviderMetadata;
 };
 
 type FinishEvent = {
@@ -42,19 +51,19 @@ export function createStepFinishHandler(
 	db: Awaited<ReturnType<typeof getDb>>,
 	getCurrentPartId: () => string,
 	getStepIndex: () => number,
-	sharedCtx: ToolAdapterContext,
-	updateCurrentPartId: (id: string) => void,
-	updateAccumulated: (text: string) => void,
+	_sharedCtx: ToolAdapterContext,
+	_updateCurrentPartId: (id: string) => void,
+	_updateAccumulated: (text: string) => void,
 	incrementStepIndex: () => number,
 	updateSessionTokensIncrementalFn: (
-		usage: any,
-		providerMetadata: Record<string, any> | undefined,
+		usage: UsageData,
+		providerMetadata: ProviderMetadata | undefined,
 		opts: RunOpts,
 		db: Awaited<ReturnType<typeof getDb>>,
 	) => Promise<void>,
 	updateMessageTokensIncrementalFn: (
-		usage: any,
-		providerMetadata: Record<string, any> | undefined,
+		usage: UsageData,
+		providerMetadata: ProviderMetadata | undefined,
 		opts: RunOpts,
 		db: Awaited<ReturnType<typeof getDb>>,
 	) => Promise<void>,
@@ -69,9 +78,11 @@ export function createStepFinishHandler(
 				.update(messageParts)
 				.set({ completedAt: finishedAt })
 				.where(eq(messageParts.id, currentPartId));
-		} catch {}
+		} catch (err) {
+			console.error('[createStepFinishHandler] Failed to update part', err);
+		}
 
-		// Update token counts incrementally after each step
+		// Update tokens incrementally
 		if (step.usage) {
 			try {
 				await updateSessionTokensIncrementalFn(
@@ -80,126 +91,81 @@ export function createStepFinishHandler(
 					opts,
 					db,
 				);
-			} catch {}
-
-			try {
 				await updateMessageTokensIncrementalFn(
 					step.usage,
 					step.experimental_providerMetadata,
 					opts,
 					db,
 				);
-			} catch {}
+			} catch (err) {
+				console.error('[createStepFinishHandler] Token update failed', err);
+			}
 		}
 
-		try {
-			publish({
-				type: 'finish-step',
-				sessionId: opts.sessionId,
-				payload: {
-					stepIndex,
-					usage: step.usage,
-					finishReason: step.finishReason,
-					response: step.response,
-				},
-			});
-			if (step.usage) {
-				publish({
-					type: 'usage',
-					sessionId: opts.sessionId,
-					payload: { stepIndex, ...step.usage },
-				});
-			}
-		} catch {}
+		// Publish step-finished event
+		publish('stream:step-finished', {
+			sessionId: opts.sessionId,
+			messageId: opts.assistantMessageId,
+			assistantMessageId: opts.assistantMessageId,
+			stepIndex,
+			finishReason: step.finishReason,
+			usage: step.usage,
+		});
 
-		try {
-			const newStepIndex = incrementStepIndex();
-			const newPartId = crypto.randomUUID();
-			const index = await sharedCtx.nextIndex();
-			const nowTs = Date.now();
-			await db.insert(messageParts).values({
-				id: newPartId,
-				messageId: opts.assistantMessageId,
-				index,
-				stepIndex: newStepIndex,
-				type: 'text',
-				content: JSON.stringify({ text: '' }),
-				agent: opts.agent,
-				provider: opts.provider,
-				model: opts.model,
-				startedAt: nowTs,
-			});
-			updateCurrentPartId(newPartId);
-			sharedCtx.assistantPartId = newPartId;
-			sharedCtx.stepIndex = newStepIndex;
-			updateAccumulated('');
-		} catch {}
+		incrementStepIndex();
 	};
 }
 
 /**
- * Creates the onError handler for the stream
+ * Creates the onFinish handler for the stream
  */
-export function createErrorHandler(
+export function createFinishHandler(
 	opts: RunOpts,
 	db: Awaited<ReturnType<typeof getDb>>,
-	getStepIndex: () => number,
-	sharedCtx: ToolAdapterContext,
+	completeAssistantMessageFn: (
+		fin: FinishEvent,
+		opts: RunOpts,
+		db: Awaited<ReturnType<typeof getDb>>,
+	) => Promise<void>,
+	_getAccumulated: () => string,
+	_abortController: AbortController,
 ) {
-	return async (err: unknown) => {
-		const errorPayload = toErrorPayload(err);
-		const isApiError = APICallError.isInstance(err);
-		const stepIndex = getStepIndex();
+	return async (fin: FinishEvent) => {
+		try {
+			await completeAssistantMessageFn(fin, opts, db);
 
-		// Create error part for UI display
-		const errorPartId = crypto.randomUUID();
-		await db.insert(messageParts).values({
-			id: errorPartId,
-			messageId: opts.assistantMessageId,
-			index: await sharedCtx.nextIndex(),
-			stepIndex,
-			type: 'error',
-			content: JSON.stringify({
-				message: errorPayload.message,
-				type: errorPayload.type,
-				details: errorPayload.details,
-				isAborted: false,
-			}),
-			agent: opts.agent,
-			provider: opts.provider,
-			model: opts.model,
-			startedAt: Date.now(),
-			completedAt: Date.now(),
-		});
+			const msgRows = await db
+				.select()
+				.from(messages)
+				.where(eq(messages.id, opts.assistantMessageId));
 
-		// Update message status
-		await db
-			.update(messages)
-			.set({
-				status: 'error',
-				error: errorPayload.message,
-				errorType: errorPayload.type,
-				errorDetails: JSON.stringify({
-					...errorPayload.details,
-					isApiError,
-				}),
-				isAborted: false,
-			})
-			.where(eq(messages.id, opts.assistantMessageId));
+			let estimatedCost = 0;
+			if (msgRows.length > 0 && msgRows[0]) {
+				const msg = msgRows[0];
+				estimatedCost = estimateModelCostUsd(
+					opts.provider,
+					opts.model,
+					Number(msg.promptTokens ?? 0),
+					Number(msg.completionTokens ?? 0),
+				);
+			}
 
-		// Publish enhanced error event
-		publish({
-			type: 'error',
-			sessionId: opts.sessionId,
-			payload: {
+			publish('stream:finished', {
+				sessionId: opts.sessionId,
 				messageId: opts.assistantMessageId,
-				partId: errorPartId,
-				error: errorPayload.message,
-				errorType: errorPayload.type,
-				details: errorPayload.details,
-				isAborted: false,
-			},
-		});
+				assistantMessageId: opts.assistantMessageId,
+				usage: fin.usage,
+				finishReason: fin.finishReason,
+				estimatedCost,
+			});
+		} catch (err) {
+			console.error('[createFinishHandler] Error in onFinish', err);
+			publish('stream:error', {
+				sessionId: opts.sessionId,
+				messageId: opts.assistantMessageId,
+				error: toErrorPayload(err),
+			});
+		}
 	};
 }
 
@@ -209,116 +175,116 @@ export function createErrorHandler(
 export function createAbortHandler(
 	opts: RunOpts,
 	db: Awaited<ReturnType<typeof getDb>>,
-	getStepIndex: () => number,
-	sharedCtx: ToolAdapterContext,
+	_abortController: AbortController,
 ) {
-	return async ({ steps }: AbortEvent) => {
-		const stepIndex = getStepIndex();
-
-		// Create abort part for UI
-		const abortPartId = crypto.randomUUID();
-		await db.insert(messageParts).values({
-			id: abortPartId,
-			messageId: opts.assistantMessageId,
-			index: await sharedCtx.nextIndex(),
-			stepIndex,
-			type: 'error',
-			content: JSON.stringify({
-				message: 'Generation stopped by user',
-				type: 'abort',
-				isAborted: true,
-				stepsCompleted: steps.length,
-			}),
-			agent: opts.agent,
-			provider: opts.provider,
-			model: opts.model,
-			startedAt: Date.now(),
-			completedAt: Date.now(),
-		});
-
-		// Store abort info
-		await db
-			.update(messages)
-			.set({
-				status: 'error',
-				error: 'Generation stopped by user',
-				errorType: 'abort',
-				errorDetails: JSON.stringify({
-					stepsCompleted: steps.length,
-					abortedAt: Date.now(),
-				}),
-				isAborted: true,
-			})
-			.where(eq(messages.id, opts.assistantMessageId));
+	return async (_event: AbortEvent) => {
+		try {
+			await db
+				.update(messages)
+				.set({ status: 'aborted', finishedAt: new Date() })
+				.where(eq(messages.id, opts.assistantMessageId));
 
-		// Publish abort event
-		publish({
-			type: 'error',
-			sessionId: opts.sessionId,
-			payload: {
+			publish('stream:aborted', {
+				sessionId: opts.sessionId,
 				messageId: opts.assistantMessageId,
-				partId: abortPartId,
-				error: 'Generation stopped by user',
-				errorType: 'abort',
-				isAborted: true,
-				stepsCompleted: steps.length,
-			},
-		});
+				assistantMessageId: opts.assistantMessageId,
+			});
+		} catch (err) {
+			console.error('[createAbortHandler] Error in onAbort', err);
+		}
 	};
 }
 
 /**
- * Creates the onFinish handler for the stream
+ * Creates the error handler for the stream
  */
-export function createFinishHandler(
+export function createErrorHandler(
 	opts: RunOpts,
 	db: Awaited<ReturnType<typeof getDb>>,
-	ensureFinishToolCalled: () => Promise<void>,
-	completeAssistantMessageFn: (
-		fin: FinishEvent,
-		opts: RunOpts,
-		db: Awaited<ReturnType<typeof getDb>>,
-	) => Promise<void>,
 ) {
-	return async (fin: FinishEvent) => {
+	return async (err: unknown) => {
+		console.error('[createErrorHandler] Stream error:', err);
+
 		try {
-			await ensureFinishToolCalled();
-		} catch {}
+			let errorMessage = 'Unknown error';
+			let errorType = 'UNKNOWN_ERROR';
+			let errorStack: string | undefined;
 
-		// Note: Token updates are handled incrementally in onStepFinish
-		// Do NOT add fin.usage here as it would cause double-counting
+			if (err instanceof APICallError) {
+				errorMessage = err.message;
+				errorType = 'API_CALL_ERROR';
+				errorStack = err.stack;
+			} else if (err instanceof Error) {
+				errorMessage = err.message;
+				errorType = err.name || 'ERROR';
+				errorStack = err.stack;
+			} else if (typeof err === 'string') {
+				errorMessage = err;
+			}
 
-		try {
-			await completeAssistantMessageFn(fin, opts, db);
-		} catch {}
+			await db
+				.update(messages)
+				.set({
+					status: 'error',
+					finishedAt: new Date(),
+					error: errorMessage,
+				})
+				.where(eq(messages.id, opts.assistantMessageId));
 
-		// Use session totals from DB for accurate cost calculation
-		const sessRows = await db
-			.select()
-			.from(messages)
-			.where(eq(messages.id, opts.assistantMessageId));
+			publish('stream:error', {
+				sessionId: opts.sessionId,
+				messageId: opts.assistantMessageId,
+				assistantMessageId: opts.assistantMessageId,
+				error: {
+					message: errorMessage,
+					type: errorType,
+					stack: errorStack,
+				},
+			});
+		} catch (dbErr) {
+			console.error('[createErrorHandler] Failed to save error to DB', dbErr);
+		}
+	};
+}
 
-		const usage = sessRows[0]
-			? {
-					inputTokens: Number(sessRows[0].promptTokens ?? 0),
-					outputTokens: Number(sessRows[0].completionTokens ?? 0),
-					totalTokens: Number(sessRows[0].totalTokens ?? 0),
-				}
-			: fin.usage;
+/**
+ * Creates the text delta handler for the stream
+ */
+export function createTextHandler(
+	opts: RunOpts,
+	db: Awaited<ReturnType<typeof getDb>>,
+	getCurrentPartId: () => string,
+	getStepIndex: () => number,
+	_updateCurrentPartId: (id: string) => void,
+	updateAccumulated: (text: string) => void,
+	getAccumulated: () => string,
+) {
+	return async (textDelta: string) => {
+		const currentPartId = getCurrentPartId();
+		const stepIndex = getStepIndex();
 
-		const costUsd = usage
-			? estimateModelCostUsd(opts.provider, opts.model, usage)
-			: undefined;
+		// Accumulate the text
+		const accumulated = getAccumulated() + textDelta;
+		updateAccumulated(accumulated);
 
-		publish({
-			type: 'message.completed',
-			sessionId: opts.sessionId,
-			payload: {
-				id: opts.assistantMessageId,
-				usage,
-				costUsd,
-				finishReason: fin.finishReason,
-			},
-		});
+		try {
+			if (currentPartId) {
+				await db
+					.update(messageParts)
+					.set({ content: accumulated })
+					.where(eq(messageParts.id, currentPartId));
+			}
+
+			publish('stream:text-delta', {
+				sessionId: opts.sessionId,
+				messageId: opts.assistantMessageId,
+				assistantMessageId: opts.assistantMessageId,
+				stepIndex,
+				textDelta,
+				fullText: accumulated,
+			});
+		} catch (err) {
+			console.error('[createTextHandler] Error updating text part', err);
+		}
 	};
 }