@thevinci/web 1.3.0 → 1.3.2

package/server/lib/fs/routes.js

@@ -746,15 +746,25 @@ export const registerFsRoutes = (app, dependencies) => {
     const Busboy = (await import('busboy')).default;
     const busboy = Busboy({ headers: req.headers, limits: { files: 50 } });
 
-    const targetDir = req.body?.path || '';
+    let targetDir = '';
     const uploadedFiles = [];
     const errors = [];
+    let responseSent = false;
+
+    const sendError = (status, error) => {
+      if (responseSent) return;
+      responseSent = true;
+      return res.status(status).json({ error: error || 'Upload failed' });
+    };
+
+    busboy.on('field', (fieldname, value) => {
+      if (fieldname === 'path' && typeof value === 'string') {
+        targetDir = value;
+      }
+    });
 
     busboy.on('file', (fieldname, file, info) => {
       const filename = info?.filename || fieldname;
-      const encoding = info?.encoding || '7bit';
-      const mimeType = info?.mimeType || 'application/octet-stream';
-
       const resolvedTarget = targetDir ? `${targetDir}/${filename}` : filename;
 
       const writePromise = (async () => {
@@ -785,10 +795,15 @@ export const registerFsRoutes = (app, dependencies) => {
 
       writePromise
         .then((result) => uploadedFiles.push(result))
-        .catch((err) => errors.push({ filename, error: err.message }));
+        .catch((err) => {
+          console.error(`[upload] Failed to write ${filename}:`, err.message);
+          errors.push({ filename, error: err.message });
+        });
     });
 
     busboy.on('finish', () => {
+      if (responseSent) return;
+      responseSent = true;
       if (errors.length > 0 && uploadedFiles.length === 0) {
         return res.status(500).json({ error: errors[0].error || 'Upload failed', errors });
       }
@@ -796,8 +811,12 @@ export const registerFsRoutes = (app, dependencies) => {
     });
 
     busboy.on('error', (err) => {
-      console.error('Busboy error:', err);
-      return res.status(500).json({ error: 'Upload failed' });
+      console.error('[upload] Busboy error:', err);
+      return sendError(500, 'Upload failed');
+    });
+
+    req.on('aborted', () => {
+      console.error('[upload] Request aborted');
     });
 
     req.pipe(busboy);