🤖 fix: graceful fallback and better logging for workspace name generation

When workspace name generation fails (e.g., proxy returns invalid JSON):
- Fall back to placeholder name instead of blocking workspace creation
- Show warning toast with error details
- Log raw response body for debugging proxy issues

Also:
- Add warning toast type with theme-aware colors
- Explicitly disable extended thinking for generateObject calls

Author: ammar-agent

src/browser/components/ChatInput/useCreationWorkspace.ts

@@ -131,6 +131,19 @@ export function useCreationWorkspace({
             const pendingInputKey = getInputKey(getPendingScopeId(projectPath));
             updatePersistedState(pendingInputKey, "");
           }
+
+          // Show warning toast if name generation had issues (e.g., proxy errors)
+          // Workspace was still created with placeholder name
+          const warning = "warning" in result ? (result.warning as string | undefined) : undefined;
+          if (warning) {
+            setToast({
+              id: Date.now().toString(),
+              type: "warning",
+              title: "Workspace Created",
+              message: warning,
+            });
+          }
+
           // Settings are already persisted via useDraftWorkspaceSettings
           // Notify parent to switch workspace (clears input via parent unmount)
           onWorkspaceCreated(result.metadata);

src/browser/components/ChatInputToast.tsx

@@ -2,14 +2,15 @@ import type { ReactNode } from "react";
 import React, { useEffect, useCallback } from "react";
 import { cn } from "@/common/lib/utils";
 
-const toastTypeStyles: Record<"success" | "error", string> = {
+const toastTypeStyles: Record<"success" | "error" | "warning", string> = {
   success: "bg-toast-success-bg border border-accent-dark text-toast-success-text",
   error: "bg-toast-error-bg border border-toast-error-border text-toast-error-text",
+  warning: "bg-toast-warning-bg border border-toast-warning-border text-toast-warning-text",
 };
 
 export interface Toast {
   id: string;
-  type: "success" | "error";
+  type: "success" | "error" | "warning";
   title?: string;
   message: string;
   solution?: ReactNode;

src/browser/styles/globals.css

@@ -223,6 +223,9 @@
     --color-toast-error-bg: hsl(5 89% 60% / 0.15); /* #f14836 with 15% opacity */
     --color-toast-error-text: hsl(5 89% 60%); /* #f14836 */
     --color-toast-error-border: hsl(5 89% 60%); /* #f14836 */
+    --color-toast-warning-bg: hsl(45 100% 51% / 0.15); /* warning yellow with 15% opacity */
+    --color-toast-warning-text: hsl(45 100% 51%); /* warning yellow */
+    --color-toast-warning-border: hsl(45 100% 51%); /* warning yellow */
     --color-toast-fatal-bg: hsl(0 33% 18%); /* #2d1f1f - fatal error bg */
     --color-toast-fatal-border: hsl(0 36% 26%); /* #5a2c2c - fatal error border */
 
@@ -442,6 +445,9 @@
   --color-toast-error-bg: hsl(5 80% 55% / 0.18);
   --color-toast-error-text: hsl(5 78% 46%);
   --color-toast-error-border: hsl(5 78% 46%);
+  --color-toast-warning-bg: hsl(45 100% 50% / 0.18);
+  --color-toast-warning-text: hsl(45 100% 35%);
+  --color-toast-warning-border: hsl(45 100% 35%);
   --color-toast-fatal-bg: hsl(0 72% 94%);
   --color-toast-fatal-border: hsl(0 74% 82%);
 
@@ -662,6 +668,9 @@
   --color-toast-error-bg: hsla(1 79% 53% / 0.18);
   --color-toast-error-text: #dc322f;
   --color-toast-error-border: #dc322f;
+  --color-toast-warning-bg: hsla(45 100% 35% / 0.18);
+  --color-toast-warning-text: #b58900;
+  --color-toast-warning-border: #b58900;
   --color-toast-fatal-bg: #fce8e7;
   --color-toast-fatal-border: #e8a5a3;
 
@@ -867,6 +876,9 @@
   --color-toast-error-bg: hsla(1 79% 53% / 0.15);
   --color-toast-error-text: #dc322f;
   --color-toast-error-border: #dc322f;
+  --color-toast-warning-bg: hsla(45 100% 35% / 0.15);
+  --color-toast-warning-text: #b58900;
+  --color-toast-warning-border: #b58900;
   --color-toast-fatal-bg: #2d1f1f;
   --color-toast-fatal-border: #5a2c2c;
 

src/node/services/ipcMain.ts

@@ -301,7 +301,7 @@ export class IpcMain {
       trunkBranch?: string;
     }
   ): Promise<
-    | { success: true; workspaceId: string; metadata: FrontendWorkspaceMetadata }
+    | { success: true; workspaceId: string; metadata: FrontendWorkspaceMetadata; warning?: string }
     | Result<void, SendMessageError>
   > {
     // Generate IDs and placeholder upfront for immediate UI feedback
@@ -333,33 +333,26 @@ export class IpcMain {
 
     try {
       // 1. Generate workspace branch name using AI (SLOW - but user sees pending state)
+      //    Falls back to placeholder name if AI fails - don't block workspace creation
       let branchName: string;
+      let nameGenerationWarning: string | undefined;
       {
-        const isErrLike = (v: unknown): v is { type: string } =>
-          typeof v === "object" && v !== null && "type" in v;
         const nameResult = await generateWorkspaceName(message, options.model, this.aiService);
-        if (!nameResult.success) {
-          const err = nameResult.error;
-          // Clear pending state on error
-          session.emitMetadata(null);
-          if (isErrLike(err)) {
-            return Err(err);
-          }
-          const toSafeString = (v: unknown): string => {
-            if (v instanceof Error) return v.message;
-            try {
-              return JSON.stringify(v);
-            } catch {
-              return String(v);
-            }
-          };
-          const msg = toSafeString(err);
-          return Err({ type: "unknown", raw: `Failed to generate workspace name: ${msg}` });
+        if (nameResult.success) {
+          branchName = nameResult.data;
+        } else {
+          // Fall back to placeholder name - don't block workspace creation
+          branchName = generatePlaceholderName(message);
+          const errMsg =
+            nameResult.error.type === "unknown" && "raw" in nameResult.error
+              ? nameResult.error.raw
+              : `Name generation failed: ${nameResult.error.type}`;
+          nameGenerationWarning = errMsg;
+          log.info("Falling back to placeholder name", { branchName, error: nameResult.error });
         }
-        branchName = nameResult.data;
       }
 
-      log.debug("Generated workspace name", { branchName });
+      log.debug("Generated workspace name", { branchName, warning: nameGenerationWarning });
 
       // 2. Get trunk branch (use provided trunkBranch or auto-detect)
       const branches = await listLocalBranches(projectPath);
@@ -478,6 +471,7 @@ export class IpcMain {
         success: true,
         workspaceId,
         metadata: completeMetadata,
+        warning: nameGenerationWarning,
       };
     } catch (error) {
       const errorMessage = error instanceof Error ? error.message : String(error);

src/node/services/workspaceTitleGenerator.ts

@@ -54,7 +54,17 @@ export async function generateWorkspaceName(
     return Ok(validateBranchName(result.object.name));
   } catch (error) {
     const messageText = error instanceof Error ? error.message : String(error);
+
+    // Log the raw response body if available (helps debug proxy issues)
+    const responseBody =
+      error && typeof error === "object" && "responseBody" in error
+        ? (error as { responseBody?: unknown }).responseBody
+        : undefined;
+    if (responseBody) {
+      log.error("Failed to generate workspace name - raw response:", responseBody);
+    }
     log.error("Failed to generate workspace name with AI", error);
+
     return Err({ type: "unknown", raw: `Failed to generate workspace name: ${messageText}` });
   }
 }