@shipstatic/drop 1.0.1 → 1.0.3

package/dist/index.cjs

@@ -10070,6 +10070,26 @@ function useDrop(options) {
       needsBuild: false
     });
     let detectedSourceName = "";
+    const setErrorState = (params) => {
+      const { title, details, errors, files = [], needsBuild = false } = params;
+      setState({
+        value: "error",
+        files,
+        sourceName: detectedSourceName,
+        needsBuild,
+        status: {
+          title,
+          details,
+          ...errors?.length && { errors }
+        }
+      });
+      onValidationError?.({
+        error: title,
+        details,
+        errors: errors ?? [],
+        isClientError: true
+      });
+    };
     try {
       if (newFiles.length === 1 && isZipFile(newFiles[0])) {
         detectedSourceName = newFiles[0].name.replace(/\.zip$/i, "");
@@ -10126,22 +10146,11 @@ function useDrop(options) {
             status: FILE_STATUSES.VALIDATION_FAILED,
             statusMessage: message
           }));
-          setState({
-            value: "error",
-            files: filesWithStatus2,
-            sourceName: detectedSourceName,
-            needsBuild,
-            status: {
-              title: "Validation Failed",
-              details: message,
-              errors: [message]
-            }
-          });
-          onValidationError?.({
-            error: "Validation Failed",
+          setErrorState({
+            title: "Validation Failed",
             details: message,
-            errors: [message],
-            isClientError: true
+            files: filesWithStatus2,
+            needsBuild
           });
           return;
         }
@@ -10171,25 +10180,11 @@ function useDrop(options) {
         statusMessage: validation.files[idx]?.statusMessage || processedFile.statusMessage
       }));
       if (!validation.canDeploy) {
-        const errorMessages = validation.errors.map(
-          (err2) => `${err2.file}: ${err2.message}`
-        );
-        setState({
-          value: "error",
-          files: filesWithStatus,
-          sourceName: detectedSourceName,
-          needsBuild: false,
-          status: {
-            title: "Validation Failed",
-            details: `${ship.pluralize(validation.errors.length, "file", "files", true)} failed validation`,
-            errors: errorMessages
-          }
-        });
-        onValidationError?.({
-          error: "Validation Failed",
-          details: ship.pluralize(validation.errors.length, "error", "errors", true),
-          errors: errorMessages,
-          isClientError: true
+        setErrorState({
+          title: "Validation Failed",
+          details: `${ship.pluralize(validation.errors.length, "file", "files", true)} failed validation`,
+          errors: validation.errors.map((err2) => `${err2.file}: ${err2.message}`),
+          files: filesWithStatus
         });
       } else if (validation.validFiles.length > 0) {
         let details = `${ship.pluralize(validation.validFiles.length, "file", "files", true)} ready`;
@@ -10225,42 +10220,20 @@ function useDrop(options) {
             }
           });
         } else {
-          const noValidError = {
-            error: "No Valid Files",
+          setErrorState({
+            title: "No Valid Files",
             details: "None of the provided files could be processed.",
-            errors: [],
-            isClientError: true
-          };
-          setState({
-            value: "error",
-            files: filesWithStatus,
-            sourceName: detectedSourceName,
-            needsBuild: false,
-            status: { title: noValidError.error, details: noValidError.details }
+            files: filesWithStatus
           });
-          onValidationError?.(noValidError);
         }
       }
     } catch (error) {
       const message = error instanceof Error ? error.message : String(error);
       const isValidation = isShipError(error);
-      const clientError = {
-        error: isValidation ? "Validation Failed" : "Processing Failed",
-        details: isValidation ? message : `Failed to process files: ${message}`,
-        errors: [],
-        isClientError: true
-      };
-      setState({
-        value: "error",
-        files: [],
-        sourceName: detectedSourceName,
-        needsBuild: false,
-        status: {
-          title: clientError.error,
-          details: clientError.details
-        }
+      setErrorState({
+        title: isValidation ? "Validation Failed" : "Processing Failed",
+        details: isValidation ? message : `Failed to process files: ${message}`
       });
-      onValidationError?.(clientError);
     } finally {
       isProcessingRef.current = false;
     }