fix: review findings — multi-turn history, race condition, collapse UI

- Core: toCoreMessages now includes tool calls and tool results in history
- Core: isError read from step tool results instead of hardcoded false
- API: status set synchronously before async generator to prevent race
- Frontend: DaisyUI collapse-open class applied dynamically on expanded state
- Frontend: removed duplicate isConnected update in wsClient.onEvent

1 files changed, 10 insertions, 2 deletions

diff --git a/packages/core/src/agent/agent.ts b/packages/core/src/agent/agent.ts
index a20a4ce..c28d691 100644
--- a/packages/core/src/agent/agent.ts
+++ b/packages/core/src/agent/agent.ts
@@ -17,7 +17,14 @@ function toCoreMessages(messages: ChatMessage[]): CoreMessage[] {
 		if (msg.role === "user") {
 			result.push({ role: "user", content: msg.content });
 		} else if (msg.role === "assistant") {
-			result.push({ role: "assistant", content: msg.content });
+			const parts: Array<{ type: "text"; text: string } | { type: "tool-call"; toolCallId: string; toolName: string; args: Record<string, unknown> }> = [{ type: "text", text: msg.content }];
+			for (const tc of msg.toolCalls ?? []) {
+				parts.push({ type: "tool-call", toolCallId: tc.id, toolName: tc.name, args: tc.arguments });
+			}
+			result.push({ role: "assistant", content: parts });
+			for (const tr of msg.toolResults ?? []) {
+				result.push({ role: "tool", content: [{ type: "tool-result", toolCallId: tr.toolCallId, toolName: "", result: tr.result }] });
+			}
 		}
 	}
 	return result;
@@ -108,12 +115,13 @@ export class Agent {
 				const stepToolResults = step.toolResults as unknown as Array<{
 					toolCallId: string;
 					result: unknown;
+					isError?: boolean;
 				}>;
 				for (const tr of stepToolResults) {
 					const toolResult: ToolResult = {
 						toolCallId: tr.toolCallId,
 						result: typeof tr.result === "string" ? tr.result : JSON.stringify(tr.result),
-						isError: false,
+						isError: tr.isError ?? false,
 					};
 					toolResults.push(toolResult);
 					yield { type: "tool-result", toolResult };