@shipstatic/drop 2.10.4-beta.7 → 2.10.4-beta.9

package/dist/index.js

@@ -1119,13 +1119,12 @@ function useDrop({ ship }) {
     [state.files]
   );
   const getFilesForUpload = useCallback(() => validFiles.map((f) => f.file), [validFiles]);
-  const processFiles2 = useCallback(
-    async (newFiles) => {
+  const prepare = useCallback(
+    async (collect) => {
       if (runRef.current) {
         console.warn("File processing already in progress. Ignoring duplicate call.");
         return;
       }
-      if (!newFiles || newFiles.length === 0) return;
       const run = new AbortController();
       runRef.current = run;
       setState({
@@ -1134,11 +1133,24 @@ function useDrop({ ship }) {
         status: { title: "Processing...", details: "Validating and preparing files." }
       });
       try {
-        const outcome = await processFiles(newFiles, {
-          limits: await ship.getLimits({ signal: run.signal }),
-          onStatus: (status) => {
-            if (!run.signal.aborted) setState((prev) => ({ ...prev, status }));
-          }
+        const [files, limits] = await Promise.all([
+          collect(),
+          ship.getLimits({ signal: run.signal })
+        ]);
+        if (run.signal.aborted) return;
+        if (files.length === 0) {
+          setState({
+            ...initialState,
+            phase: "error",
+            status: { title: "Empty Folder", details: "It has no files to deploy." }
+          });
+          return;
+        }
+        const outcome = await processFiles(files, {
+          limits,
+          // The pipeline says its status before its first wait, so a
+          // cancelled run has none left to say.
+          onStatus: (status) => setState((prev) => ({ ...prev, status }))
         });
         if (run.signal.aborted) return;
         setState({
@@ -1165,6 +1177,13 @@ function useDrop({ ship }) {
     },
     [ship]
   );
+  const processFiles2 = useCallback(
+    async (newFiles) => {
+      if (!newFiles || newFiles.length === 0) return;
+      await prepare(async () => newFiles);
+    },
+    [prepare]
+  );
   const reset = useCallback(() => {
     runRef.current?.abort();
     runRef.current = null;
@@ -1203,13 +1222,13 @@ function useDrop({ ship }) {
           if (file) files.push(file);
         }
       }
-      await Promise.all(directories.map((d) => traverseFileTree(d.entry, files, d.path)));
-      if (files.length === 0 && e.dataTransfer.files.length > 0) {
-        files.push(...Array.from(e.dataTransfer.files));
-      }
-      if (files.length > 0) await processFiles2(files);
+      if (directories.length === 0 && files.length === 0) return;
+      await prepare(async () => {
+        await Promise.all(directories.map((d) => traverseFileTree(d.entry, files, d.path)));
+        return files;
+      });
     },
-    [processFiles2]
+    [prepare]
   );
   const handleInputChange = useCallback(
     (e) => {