vibe-gx 1.0.5 → 1.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vibe-gx",
-  "version": "1.0.5",
+  "version": "1.0.7",
   "description": "A lightweight, regex-based Node.js web framework built for speed and simplicity.",
   "type": "module",
   "main": "vibe.js",

package/utils/core/handler.js

@@ -140,21 +140,34 @@ export async function runIntercept(intercept, req, res, isRoute = true) {
  */
 export function handleError(error, req, res) {
   const isDev = process.env.NODE_ENV !== "production";
+  const message = error.message || "Unknown error";
 
   // Log error (full stack in dev, message only in production)
   if (isDev) {
     console.error("[VIBE ERROR]:", error);
   } else {
-    console.error("[VIBE ERROR]:", error.message || "Unknown error");
+    console.error("[VIBE ERROR]:", message);
   }
 
   if (!res.headersSent) {
-    res.writeHead(500, { "content-type": "application/json" });
+    // Determine status code based on error type
+    let statusCode = 500;
+    let errorType = "Internal Server Error";
+
+    if (message.includes("exceeds max size")) {
+      statusCode = 413;
+      errorType = "Payload Too Large";
+    } else if (message.includes("not allowed")) {
+      statusCode = 415;
+      errorType = "Unsupported Media Type";
+    }
+
+    res.writeHead(statusCode, { "content-type": "application/json" });
 
     // Only expose error details in development
     const responseBody = isDev
-      ? { error: "Internal Server Error", message: error.message }
-      : { error: "Internal Server Error" };
+      ? { error: errorType, message }
+      : { error: errorType };
 
     res.end(JSON.stringify(responseBody));
   }

package/utils/core/parser.js

@@ -56,9 +56,22 @@ function parseMultipart(req, res, media, options, resolve, reject) {
   const streaming = media.streaming === true;
   let pendingWrites = 0;
   let busboyFinished = false;
+  let alreadyRejected = false;
 
-  // Helper to check if we're done
+  // Helper to reject immediately (for errors that shouldn't wait)
+  const rejectNow = (err) => {
+    if (alreadyRejected) return;
+    alreadyRejected = true;
+    // Unpipe to stop processing more data
+    req.unpipe(bb);
+    // Drain the request to prevent hanging
+    req.resume();
+    reject(err);
+  };
+
+  // Helper to check if we're done (for normal completion)
   const checkComplete = () => {
+    if (alreadyRejected) return;
     if (busboyFinished && pendingWrites === 0) {
       if (fileError) {
         reject(fileError);
@@ -97,10 +110,12 @@ function parseMultipart(req, res, media, options, resolve, reject) {
         return allowed === mimeType;
       });
       if (!isAllowed) {
-        fileError = new Error(
-          `File type '${mimeType}' not allowed. Allowed: ${media.allowedTypes.join(", ")}`,
+        file.resume();
+        return rejectNow(
+          new Error(
+            `File type '${mimeType}' not allowed. Allowed: ${media.allowedTypes.join(", ")}`,
+          ),
         );
-        return file.resume();
       }
     }
 
@@ -156,16 +171,18 @@ function parseMultipart(req, res, media, options, resolve, reject) {
     // Handle file size limit exceeded
     file.on("limit", () => {
       truncated = true;
-      fileError = new Error(
+      const err = new Error(
         `File '${filename}' exceeds max size of ${media.maxSize || 10 * 1024 * 1024} bytes`,
       );
       file.unpipe(writeStream);
       writeStream.end();
-      // Clean up partial file
+      file.resume();
+      // Clean up partial file and reject immediately
       fs.unlink(filePath, () => {
         pendingWrites--;
-        checkComplete();
       });
+      // Reject NOW - don't wait for busboy to finish
+      rejectNow(err);
     });
 
     file.on("error", (err) => {