@shipstatic/drop 2.10.4-beta.6 → 2.10.4-beta.8

package/dist/index.cjs

@@ -1110,7 +1110,7 @@ var initialState = {
 };
 function useDrop({ ship }) {
   const [state, setState] = react.useState(initialState);
-  const isProcessingRef = react.useRef(false);
+  const runRef = react.useRef(null);
   const folderInputRef = react.useRef(null);
   const filesInputRef = react.useRef(null);
   const isProcessing = state.phase === "processing";
@@ -1121,24 +1121,36 @@ function useDrop({ ship }) {
     [state.files]
   );
   const getFilesForUpload = react.useCallback(() => validFiles.map((f) => f.file), [validFiles]);
-  const processFiles2 = react.useCallback(
-    async (newFiles) => {
-      if (isProcessingRef.current) {
+  const prepare = react.useCallback(
+    async (collect) => {
+      if (runRef.current) {
         console.warn("File processing already in progress. Ignoring duplicate call.");
         return;
       }
-      if (!newFiles || newFiles.length === 0) return;
-      isProcessingRef.current = true;
+      const run = new AbortController();
+      runRef.current = run;
       setState({
         ...initialState,
         phase: "processing",
         status: { title: "Processing...", details: "Validating and preparing files." }
       });
       try {
-        const outcome = await processFiles(newFiles, {
-          limits: await ship.getLimits(),
-          onStatus: (status) => 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);
+          return;
+        }
+        const outcome = await processFiles(files, {
+          limits,
+          onStatus: (status) => {
+            if (!run.signal.aborted) setState((prev) => ({ ...prev, status }));
+          }
         });
+        if (run.signal.aborted) return;
         setState({
           phase: outcome.phase,
           isDragging: false,
@@ -1147,15 +1159,33 @@ function useDrop({ ship }) {
           status: outcome.status,
           needsBuild: outcome.needsBuild
         });
+      } catch (error) {
+        if (run.signal.aborted) return;
+        setState({
+          ...initialState,
+          phase: "error",
+          status: {
+            title: "Processing Failed",
+            details: error instanceof Error ? error.message : String(error)
+          }
+        });
       } finally {
-        isProcessingRef.current = false;
+        if (runRef.current === run) runRef.current = null;
       }
     },
     [ship]
   );
+  const processFiles2 = react.useCallback(
+    async (newFiles) => {
+      if (!newFiles || newFiles.length === 0) return;
+      await prepare(async () => newFiles);
+    },
+    [prepare]
+  );
   const reset = react.useCallback(() => {
+    runRef.current?.abort();
+    runRef.current = null;
     setState(initialState);
-    isProcessingRef.current = false;
   }, []);
   const handleDragOver = react.useCallback((e) => {
     e.preventDefault();
@@ -1190,13 +1220,14 @@ 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);
+      const plain = Array.from(e.dataTransfer.files);
+      if (directories.length === 0 && files.length === 0 && plain.length === 0) return;
+      await prepare(async () => {
+        await Promise.all(directories.map((d) => traverseFileTree(d.entry, files, d.path)));
+        return files.length > 0 ? files : plain;
+      });
     },
-    [processFiles2]
+    [prepare]
   );
   const handleInputChange = react.useCallback(
     (e) => {