Fixed LLM multi message responses

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ztimson/ai-utils",
-	"version": "0.1.18",
+	"version": "0.1.22",
 	"description": "AI Utility library",
 	"author": "Zak Timson",
 	"license": "MIT",

src/antrhopic.ts

@@ -1,5 +1,5 @@
 import {Anthropic as anthropic} from '@anthropic-ai/sdk';
-import {findByProp, objectMap, JSONSanitize, JSONAttemptParse} from '@ztimson/utils';
+import {findByProp, objectMap, JSONSanitize, JSONAttemptParse, deepCopy} from '@ztimson/utils';
 import {Ai} from './ai.ts';
 import {LLMMessage, LLMRequest} from './llm.ts';
 import {AbortablePromise, LLMProvider} from './provider.ts';
@@ -39,19 +39,20 @@ export class Anthropic extends LLMProvider {
 			if(history[i].role == 'tool') {
 				const h: any = history[i];
 				history.splice(i, 1,
-					{role: 'assistant', content: [{type: 'tool_use', id: h.id, name: h.name, input: h.args}], timestamp: h.timestamp},
-					{role: 'user', content: [{type: 'tool_result', tool_use_id: h.id, is_error: !!h.error, content:  h.error || h.content}], timestamp: Date.now()}
+					{role: 'assistant', content: [{type: 'tool_use', id: h.id, name: h.name, input: h.args}]},
+					{role: 'user', content: [{type: 'tool_result', tool_use_id: h.id, is_error: !!h.error, content:  h.error || h.content}]}
 				)
 				i++;
 			}
 		}
-		return history;
+		return history.map(({timestamp, ...h}) => h);
 	}
 
 	ask(message: string, options: LLMRequest = {}): AbortablePromise<LLMMessage[]> {
 		const controller = new AbortController();
 		const response = new Promise<any>(async (res, rej) => {
 			let history = this.fromStandard([...options.history || [], {role: 'user', content: message, timestamp: Date.now()}]);
+			const original = deepCopy(history);
 			if(options.compress) history = await this.ai.llm.compress(<any>history, options.compress.max, options.compress.min, options);
 			const requestParams: any = {
 				model: options.model || this.model,
@@ -72,12 +73,15 @@ export class Anthropic extends LLMProvider {
 				stream: !!options.stream,
 			};
 
-			let resp: any;
-			const loopMessages: any[] = [];
+			let resp: any, isFirstMessage = true;
+			const assistantMessages: string[] = [];
 			do {
 				resp = await this.client.messages.create(requestParams);
+
+				// Streaming mode
 				if(options.stream) {
-					if(loopMessages.length) options.stream({text: '\n\n'});
+					if(!isFirstMessage) options.stream({text: '\n\n'});
+					else isFirstMessage = false;
 					resp.content = [];
 					for await (const chunk of resp) {
 						if(controller.signal.aborted) break;
@@ -104,10 +108,11 @@ export class Anthropic extends LLMProvider {
 					}
 				}
 
-				loopMessages.push({role: 'assistant', content: resp.content, timestamp: Date.now()});
+				// Run tools
 				const toolCalls = resp.content.filter((c: any) => c.type === 'tool_use');
 				if(toolCalls.length && !controller.signal.aborted) {
-					history.push({role: 'assistant', content: resp.content, timestamp: Date.now()});
+					history.push({role: 'assistant', content: resp.content});
+					original.push({role: 'assistant', content: resp.content});
 					const results = await Promise.all(toolCalls.map(async (toolCall: any) => {
 						const tool = options.tools?.find(findByProp('name', toolCall.name));
 						if(!tool) return {tool_use_id: toolCall.id, is_error: true, content: 'Tool not found'};
@@ -118,18 +123,13 @@ export class Anthropic extends LLMProvider {
 							return {type: 'tool_result', tool_use_id: toolCall.id, is_error: true, content: err?.message || err?.toString() || 'Unknown'};
 						}
 					}));
-					const userMsg = {role: 'user', content: results, timestamp: Date.now()};
-					history.push(userMsg);
-					loopMessages.push(userMsg);
+					history.push({role: 'user', content: results});
 					requestParams.messages = history;
 				}
 			} while (!controller.signal.aborted && resp.content.some((c: any) => c.type === 'tool_use'));
 
-			const combinedContent = loopMessages.filter(m => m.role === 'assistant')
-				.map(m => m.content.filter((c: any) => c.type == 'text').map((c: any) => c.text).join('\n\n'))
-				.filter(c => c).join('\n\n');
 			if(options.stream) options.stream({done: true});
-			res(this.toStandard([...history, {role: 'assistant', content: combinedContent, timestamp: Date.now()}]));
+			res(this.toStandard([...history, {role: 'assistant', content: resp.content.filter((c: any) => c.type == 'text').map((c: any) => c.text).join('\n\n')}]));
 		});
 
 		return Object.assign(response, {abort: () => controller.abort()});

src/llm.ts

@@ -12,7 +12,7 @@ export type LLMMessage = {
 	/** Message content */
 	content: string | any;
 	/** Timestamp */
-	timestamp: number;
+	timestamp?: number;
 } | {
 	/** Tool call */
 	role: 'tool';
@@ -27,7 +27,7 @@ export type LLMMessage = {
 	/** Tool error */
 	error: undefined | string;
 	/** Timestamp */
-	timestamp: number;
+	timestamp?: number;
 }
 
 export type LLMOptions = {

src/ollama.ts

@@ -31,8 +31,9 @@ export class Ollama extends LLMProvider {
 
 	private fromStandard(history: LLMMessage[]): any[] {
 		return history.map((h: any) => {
-			if(h.role != 'tool') return h;
-			return {role: 'tool', tool_name: h.name, content: h.error || h.content, timestamp: h.timestamp}
+			const {timestamp, ...rest} = h;
+			if(h.role != 'tool') return rest;
+			return {role: 'tool', tool_name: h.name, content: h.error || h.content}
 		});
 	}
 
@@ -71,12 +72,12 @@ export class Ollama extends LLMProvider {
 				}))
 			}
 
-			let resp: any;
-			const loopMessages: any[] = [];
+			let resp: any, isFirstMessage = true;
 			do {
 				resp = await this.client.chat(requestParams);
 				if(options.stream) {
-					if(loopMessages.length) options.stream({text: '\n\n'});
+					if(!isFirstMessage) options.stream({text: '\n\n'});
+					else isFirstMessage = false;
 					resp.message = {role: 'assistant', content: '', tool_calls: []};
 					for await (const chunk of resp) {
 						if(controller.signal.aborted) break;
@@ -89,31 +90,26 @@ export class Ollama extends LLMProvider {
 					}
 				}
 
-				loopMessages.push({role: 'assistant', content: resp.message?.content, timestamp: Date.now()});
-
 				if(resp.message?.tool_calls?.length && !controller.signal.aborted) {
-					history.push({...resp.message, timestamp: Date.now()});
+					history.push(resp.message);
 					const results = await Promise.all(resp.message.tool_calls.map(async (toolCall: any) => {
 						const tool = (options.tools || this.ai.options.tools)?.find(findByProp('name', toolCall.function.name));
-						if(!tool) return {role: 'tool', tool_name: toolCall.function.name, content: '{"error": "Tool not found"}', timestamp: Date.now()};
+						if(!tool) return {role: 'tool', tool_name: toolCall.function.name, content: '{"error": "Tool not found"}'};
 						const args = typeof toolCall.function.arguments === 'string' ? JSONAttemptParse(toolCall.function.arguments, {}) : toolCall.function.arguments;
 						try {
 							const result = await tool.fn(args, this.ai);
-							return {role: 'tool', tool_name: toolCall.function.name, args, content: JSONSanitize(result), timestamp: Date.now()};
+							return {role: 'tool', tool_name: toolCall.function.name, args, content: JSONSanitize(result)};
 						} catch (err: any) {
-							return {role: 'tool', tool_name: toolCall.function.name, args, content: JSONSanitize({error: err?.message || err?.toString() || 'Unknown'}), timestamp: Date.now()};
+							return {role: 'tool', tool_name: toolCall.function.name, args, content: JSONSanitize({error: err?.message || err?.toString() || 'Unknown'})};
 						}
 					}));
 					history.push(...results);
-					loopMessages.push(...results);
 					requestParams.messages = history;
 				}
 			} while (!controller.signal.aborted && resp.message?.tool_calls?.length);
 
-			const combinedContent = loopMessages.filter(m => m.role === 'assistant')
-				.map(m => m.content).filter(c => c).join('\n\n');
 			if(options.stream) options.stream({done: true});
-			res(this.toStandard([...history, {role: 'assistant', content: combinedContent, timestamp: Date.now()}]));
+			res(this.toStandard([...history, {role: 'assistant', content: resp.message?.content}]));
 		});
 
 		return Object.assign(response, {abort: () => controller.abort()});

src/open-ai.ts

@@ -47,16 +47,15 @@ export class OpenAi extends LLMProvider {
 					content: null,
 					tool_calls: [{ id: h.id, type: 'function', function: { name: h.name, arguments: JSON.stringify(h.args) } }],
 					refusal: null,
-					annotations: [],
-					timestamp: h.timestamp
+					annotations: []
 				}, {
 					role: 'tool',
 					tool_call_id: h.id,
-					content: h.error || h.content,
-					timestamp: Date.now()
+					content: h.error || h.content
 				});
 			} else {
-				result.push(h);
+				const {timestamp, ...rest} = h;
+				result.push(rest);
 			}
 			return result;
 		}, [] as any[]);
@@ -88,12 +87,12 @@ export class OpenAi extends LLMProvider {
 				}))
 			};
 
-			let resp: any;
-			const loopMessages: any[] = [];
+			let resp: any, isFirstMessage = true;
 			do {
 				resp = await this.client.chat.completions.create(requestParams);
 				if(options.stream) {
-					if(loopMessages.length) options.stream({text: '\n\n'});
+					if(!isFirstMessage) options.stream({text: '\n\n'});
+					else isFirstMessage = false;
 					resp.choices = [{message: {content: '', tool_calls: []}}];
 					for await (const chunk of resp) {
 						if(controller.signal.aborted) break;
@@ -107,34 +106,28 @@ export class OpenAi extends LLMProvider {
 					}
 				}
 
-				loopMessages.push({role: 'assistant', content: resp.choices[0].message.content || '', timestamp: Date.now()});
-
 				const toolCalls = resp.choices[0].message.tool_calls || [];
 				if(toolCalls.length && !controller.signal.aborted) {
-					history.push({...resp.choices[0].message, timestamp: Date.now()});
+					history.push(resp.choices[0].message);
 					const results = await Promise.all(toolCalls.map(async (toolCall: any) => {
 						const tool = options.tools?.find(findByProp('name', toolCall.function.name));
-						if(!tool) return {role: 'tool', tool_call_id: toolCall.id, content: '{"error": "Tool not found"}', timestamp: Date.now()};
+						if(!tool) return {role: 'tool', tool_call_id: toolCall.id, content: '{"error": "Tool not found"}'};
 						try {
 							const args = JSONAttemptParse(toolCall.function.arguments, {});
 							const result = await tool.fn(args, this.ai);
-							return {role: 'tool', tool_call_id: toolCall.id, content: JSONSanitize(result), timestamp: Date.now()};
+							return {role: 'tool', tool_call_id: toolCall.id, content: JSONSanitize(result)};
 						} catch (err: any) {
-							return {role: 'tool', tool_call_id: toolCall.id, content: JSONSanitize({error: err?.message || err?.toString() || 'Unknown'}), timestamp: Date.now()};
+							return {role: 'tool', tool_call_id: toolCall.id, content: JSONSanitize({error: err?.message || err?.toString() || 'Unknown'})};
 						}
 					}));
 					history.push(...results);
-					loopMessages.push(...results);
 					requestParams.messages = history;
 				}
 			} while (!controller.signal.aborted && resp.choices?.[0]?.message?.tool_calls?.length);
 
-			const combinedContent = loopMessages.filter(m => m.role === 'assistant')
-				.map(m => m.content).filter(c => c).join('\n\n');
 			if(options.stream) options.stream({done: true});
-			res(this.toStandard([...history, {role: 'assistant', content: combinedContent, timestamp: Date.now()}]));
+			res(this.toStandard([...history, {role: 'assistant', content: resp.choices[0].message.content || ''}]));
 		});
-
 		return Object.assign(response, {abort: () => controller.abort()});
 	}
 }