🤖 fix: surface pre-stream errors and persist model on workspace creation

Two issues fixed:

1. Pre-stream error surfacing:
   - When sendMessage fails before streaming starts (e.g., API key not
     configured), the error was silently ignored
   - Modified StreamingMessageAggregator.handleStreamError to create a
     synthetic error message when no active stream exists
   - Added logging in workspaceService.completeWorkspaceCreation

2. Model persistence on workspace creation:
   - Model selected for creation was persisted at project scope but not
     synced to workspace scope
   - On restart, getSendOptionsFromStorage would fall back to default model
   - Added model sync in syncCreationPreferences alongside mode/thinking

_Generated with mux_

Author: ibetitsmike

src/browser/components/ChatInput/useCreationWorkspace.ts

@@ -9,6 +9,7 @@ import { readPersistedState, updatePersistedState } from "@/browser/hooks/usePer
 import { useSendMessageOptions } from "@/browser/hooks/useSendMessageOptions";
 import {
   getInputKey,
+  getModelKey,
   getModeKey,
   getPendingScopeId,
   getProjectScopeId,
@@ -27,6 +28,13 @@ interface UseCreationWorkspaceOptions {
 function syncCreationPreferences(projectPath: string, workspaceId: string): void {
   const projectScopeId = getProjectScopeId(projectPath);
 
+  // Sync model from project scope to workspace scope
+  // This ensures the model used for creation is persisted for future resumes
+  const projectModel = readPersistedState<string | null>(getModelKey(projectScopeId), null);
+  if (projectModel) {
+    updatePersistedState(getModelKey(workspaceId), projectModel);
+  }
+
   const projectMode = readPersistedState<UIMode | null>(getModeKey(projectScopeId), null);
   if (projectMode) {
     updatePersistedState(getModeKey(workspaceId), projectMode);

src/browser/utils/messages/StreamingMessageAggregator.ts

@@ -564,6 +564,28 @@ export class StreamingMessageAggregator {
       // Clean up stream-scoped state (active stream tracking, TODOs)
       this.cleanupStreamState(data.messageId);
       this.invalidateCache();
+    } else {
+      // Pre-stream error (e.g., API key not configured before streaming starts)
+      // Create a synthetic error message since there's no active stream to attach to
+      // Get the highest historySequence from existing messages so this appears at the end
+      const maxSequence = Math.max(
+        0,
+        ...Array.from(this.messages.values()).map((m) => m.metadata?.historySequence ?? 0)
+      );
+      const errorMessage: MuxMessage = {
+        id: data.messageId,
+        role: "assistant",
+        parts: [],
+        metadata: {
+          partial: true,
+          error: data.error,
+          errorType: data.errorType,
+          timestamp: Date.now(),
+          historySequence: maxSequence + 1,
+        },
+      };
+      this.messages.set(data.messageId, errorMessage);
+      this.invalidateCache();
     }
   }
 

src/node/services/workspaceService.ts

@@ -610,6 +610,11 @@ export class WorkspaceService extends EventEmitter {
       // Send the first message, surfacing errors to the chat UI
       void session.sendMessage(message, options).then((result) => {
         if (!result.success) {
+          log.error("sendMessage failed during workspace creation", {
+            workspaceId,
+            errorType: result.error.type,
+            error: result.error,
+          });
           const { message: errorMessage, errorType } = formatSendMessageError(result.error);
           const streamError: StreamErrorMessage = {
             type: "stream-error",