Fix scaffold copy and better error handling for create app (#202)

src/ipc/handlers/app_handlers.ts

@@ -12,7 +12,10 @@ import { promises as fsPromises } from "node:fs";
 
 // Import our utility modules
 import { withLock } from "../utils/lock_utils";
-import { getFilesRecursively } from "../utils/file_utils";
+import {
+  copyDirectoryRecursive,
+  getFilesRecursively,
+} from "../utils/file_utils";
 import {
   runningApps,
   processCounter,
@@ -187,17 +190,13 @@ export function registerAppHandlers() {
         })
         .returning();
 
-      // Start async operations in background
-      try {
-        await fsPromises.cp(
+      // Why do we not use fs.cp here?
+      // Because scaffold is inside ASAR and it does NOT
+      // behave like a regular directory if you use fs.cp
+      // https://www.electronjs.org/docs/latest/tutorial/asar-archives#limitations-of-the-node-api
+      await copyDirectoryRecursive(
         path.join(__dirname, "..", "..", "scaffold"),
         fullAppPath,
-          {
-            recursive: true,
-            // Scaffold should *not* have node_modules anyways, but
-            // just in case, we filter it out.
-            filter: (source) => !source.includes("node_modules"),
-          },
       );
       // Initialize git repo and create first commit
       await git.init({
@@ -228,9 +227,6 @@ export function registerAppHandlers() {
           initialCommitHash: commitHash,
         })
         .where(eq(chats.id, chat.id));
-      } catch (error) {
-        logger.error("Error in background app initialization:", error);
-      }
 
       return { app, chatId: chat.id };
     },

src/ipc/utils/file_utils.ts

 import fs from "node:fs";
+import { promises as fsPromises } from "node:fs";
 import path from "node:path";
 
 /**
@@ -31,3 +32,22 @@ export function getFilesRecursively(dir: string, baseDir: string): string[] {
 
   return files;
 }
+
+export async function copyDirectoryRecursive(
+  source: string,
+  destination: string,
+) {
+  await fsPromises.mkdir(destination, { recursive: true });
+  const entries = await fsPromises.readdir(source, { withFileTypes: true });
+
+  for (const entry of entries) {
+    const srcPath = path.join(source, entry.name);
+    const destPath = path.join(destination, entry.name);
+
+    if (entry.isDirectory()) {
+      await copyDirectoryRecursive(srcPath, destPath);
+    } else {
+      await fsPromises.copyFile(srcPath, destPath);
+    }
+  }
+}

src/pages/home.tsx

@@ -25,6 +25,7 @@ import { useTheme } from "@/contexts/ThemeContext";
 import { Button } from "@/components/ui/button";
 import { ExternalLink } from "lucide-react";
 import { ImportAppButton } from "@/components/ImportAppButton";
+import { showError } from "@/lib/toast";
 
 // Adding an export for attachments
 export interface HomeSubmitOptions {
@@ -133,6 +134,7 @@ export default function HomePage() {
       navigate({ to: "/chat", search: { id: result.chatId } });
     } catch (error) {
       console.error("Failed to create chat:", error);
+      showError("Failed to create app. " + (error as any).toString());
       setIsLoading(false); // Ensure loading state is reset on error
     }
     // No finally block needed for setIsLoading(false) here if navigation happens on success