Fix file upload

server/document-routes.ts

@@ -10,12 +10,73 @@ const router = Router();
 /**
  * Upload documents (multiple files supported)
  */
-router.post('/upload', upload.array('files', 10), validateUpload, async (req, res) => {
+router.post('/upload', (req, res, next) => {
+  upload.array('files', 10)(req, res, (err) => {
+    if (err) {
+      console.error('Multer upload error:', err);
+      
+      // Handle specific multer errors
+      if (err.code === 'LIMIT_FILE_SIZE') {
+        return res.status(400).json({
+          success: false,
+          error: 'File too large',
+          message: `File size exceeds the maximum limit of 50MB. File: ${err.field}`
+        });
+      }
+      
+      if (err.code === 'LIMIT_FILE_COUNT') {
+        return res.status(400).json({
+          success: false,
+          error: 'Too many files',
+          message: 'Maximum 10 files allowed per upload'
+        });
+      }
+      
+      if (err.code === 'LIMIT_UNEXPECTED_FILE') {
+        return res.status(400).json({
+          success: false,
+          error: 'Unexpected file field',
+          message: `Unexpected file field: ${err.field}`
+        });
+      }
+      
+      // Handle file type errors
+      if (err.message && err.message.includes('Unsupported file type')) {
+        return res.status(400).json({
+          success: false,
+          error: 'Unsupported file type',
+          message: err.message
+        });
+      }
+      
+      // Generic multer error
+      return res.status(400).json({
+        success: false,
+        error: 'File upload error',
+        message: err.message || 'Unknown upload error'
+      });
+    }
+    
+    next();
+  });
+}, validateUpload, async (req, res) => {
   try {
     const files = req.files as Express.Multer.File[];
+    
+    if (!files || files.length === 0) {
+      return res.status(400).json({
+        success: false,
+        error: 'No files received',
+        message: 'No files were received by the server'
+      });
+    }
+    
+    console.log(`Processing ${files.length} uploaded files`);
     const uploadedDocuments = [];
 
     for (const file of files) {
+      console.log(`Processing file: ${file.originalname}, size: ${file.size} bytes, type: ${file.mimetype}`);
+      
       // Extract title from filename or use provided title
       const title = req.body.title || path.basename(file.originalname, path.extname(file.originalname));
       const source = req.body.source || `Uploaded file: ${file.originalname}`;
@@ -42,27 +103,33 @@ router.post('/upload', upload.array('files', 10), validateUpload, async (req, re
       }
 
       // Create document record
-      const document = await storage.createDocument({
-        title,
-        content,
-        source,
-        sourceType,
-        url: null,
-        metadata: {
-          originalName: file.originalname,
-          uploadedAt: new Date().toISOString(),
+      try {
+        const document = await storage.createDocument({
+          title,
+          content,
+          source,
+          sourceType,
+          url: null,
+          metadata: {
+            originalName: file.originalname,
+            uploadedAt: new Date().toISOString(),
+            mimeType: file.mimetype,
+            size: file.size
+          },
+          embedding: null,
+          filePath: file.path,
+          fileName: file.originalname,
+          fileSize: file.size,
           mimeType: file.mimetype,
-          size: file.size
-        },
-        embedding: null,
-        filePath: file.path,
-        fileName: file.originalname,
-        fileSize: file.size,
-        mimeType: file.mimetype,
-        processingStatus: FileProcessor.requiresOCR(file.mimetype) ? 'pending' : 'completed'
-      } as any);
-
-      uploadedDocuments.push(document);
+          processingStatus: FileProcessor.requiresOCR(file.mimetype) ? 'pending' : 'completed'
+        } as any);
+
+        console.log(`Successfully created document record for ${file.originalname} with ID ${document.id}`);
+        uploadedDocuments.push(document);
+      } catch (dbError) {
+        console.error(`Failed to create document record for ${file.originalname}:`, dbError);
+        throw new Error(`Database error while saving ${file.originalname}: ${dbError instanceof Error ? dbError.message : 'Unknown database error'}`);
+      }
     }
 
     res.status(201).json({

server/file-upload.ts

@@ -14,10 +14,20 @@ const uploadsDir = process.env.NODE_ENV === 'production'
 try {
   if (!fs.existsSync(uploadsDir)) {
     fs.mkdirSync(uploadsDir, { recursive: true });
+    console.log(`✅ Created uploads directory: ${uploadsDir}`);
+  } else {
+    console.log(`✅ Uploads directory exists: ${uploadsDir}`);
   }
+  
+  // Test write permissions
+  const testFile = path.join(uploadsDir, 'test-permissions.txt');
+  fs.writeFileSync(testFile, 'test');
+  fs.unlinkSync(testFile);
+  console.log(`✅ Upload directory is writable: ${uploadsDir}`);
+  
 } catch (error) {
-  console.warn(`Failed to create uploads directory at ${uploadsDir}:`, error);
-  console.log('File uploads will be disabled');
+  console.warn(`❌ Failed to create or write to uploads directory at ${uploadsDir}:`, error);
+  console.log('File uploads may not work properly');
 }
 
 // Configure multer for file uploads
@@ -28,13 +38,17 @@ const storage = multer.diskStorage({
       const dateDir = new Date().toISOString().split('T')[0];
       const fullPath = path.join(uploadsDir, dateDir);
       
+      console.log(`Creating upload destination: ${fullPath} for file: ${file.originalname}`);
+      
       if (!fs.existsSync(fullPath)) {
         fs.mkdirSync(fullPath, { recursive: true });
+        console.log(`Created upload subdirectory: ${fullPath}`);
       }
       
       cb(null, fullPath);
     } catch (error) {
       console.error('Failed to create upload directory:', error);
+      console.log(`Falling back to base directory: ${uploadsDir}`);
       // Fallback to base uploads directory
       cb(null, uploadsDir);
     }

server/vite.ts

@@ -35,8 +35,8 @@ export async function setupVite(app: Express, server: Server) {
     customLogger: {
       ...viteLogger,
       error: (msg, options) => {
-        viteLogger.error(msg, options);
-        process.exit(1);
+        console.warn('Vite error (non-fatal):', msg);
+        // Don't exit on Vite errors - they're often non-critical
       },
     },
     server: serverOptions,