@umituz/react-native-filesystem 2.1.19 → 2.1.21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-filesystem",
-  "version": "2.1.19",
+  "version": "2.1.21",
   "description": "File operations and import/export functionality for React Native apps",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/infrastructure/services/download.constants.ts

@@ -0,0 +1,6 @@
+/**
+ * Download Service Constants
+ */
+
+export const SUPPORTED_DOWNLOAD_EXTENSIONS = ["mp4", "mov", "m4v", "webm", "jpg", "png", "pdf"] as const;
+export const DEFAULT_DOWNLOAD_EXTENSION = "mp4" as const;

package/src/infrastructure/services/download.service.ts

@@ -1,72 +1,48 @@
 /**
  * Download Service
- * Single Responsibility: Handle file download operations
  */
 
 import { File, Paths, Directory } from "expo-file-system";
 import type { FileOperationResult } from "../../domain/entities/File";
 import { FileUtils } from "../../domain/entities/File";
+import { SUPPORTED_DOWNLOAD_EXTENSIONS, DEFAULT_DOWNLOAD_EXTENSION } from "./download.constants";
 import type { DownloadProgressCallback, DownloadWithProgressResult } from "./download.types";
 
-const hashUrl = (url: string): string => {
+const hashUrl = (url: string) => {
   let hash = 0;
   for (let i = 0; i < url.length; i++) hash = ((hash << 5) - hash + url.charCodeAt(i)) | 0;
   return Math.abs(hash).toString(36);
 };
 
-const getExt = (url: string): string => {
-  const ext = url.split("?")[0].split(".").pop()?.toLowerCase() || "mp4";
-  return ["mp4", "mov", "m4v", "webm", "jpg", "png", "pdf"].includes(ext) ? ext : "mp4";
+const getExt = (url: string) => {
+  const parts = url.split("?")[0]?.split(".") || [];
+  const ext = parts.length > 1 ? parts.pop()?.toLowerCase() || DEFAULT_DOWNLOAD_EXTENSION : DEFAULT_DOWNLOAD_EXTENSION;
+  return (SUPPORTED_DOWNLOAD_EXTENSIONS as readonly string[]).includes(ext) ? ext : DEFAULT_DOWNLOAD_EXTENSION;
 };
 
-const getCacheUri = (url: string, dir: string): string => {
-  return FileUtils.joinPaths(dir, `cached_${hashUrl(url)}.${getExt(url)}`);
-};
-
-interface DownloadError extends Error {
-  message: string;
-}
+const getCacheUri = (url: string, dir: string) => FileUtils.joinPaths(dir, `cached_${hashUrl(url)}.${getExt(url)}`);
 
 export async function downloadFile(url: string, dest?: string): Promise<FileOperationResult> {
   try {
-    const destination = dest
-      ? new File(dest)
-      : new File(Paths.document, FileUtils.generateUniqueFilename("download"));
+    const destination = dest ? new File(dest) : new File(Paths.document, FileUtils.generateUniqueFilename("download"));
     const res = await File.downloadFileAsync(url, destination, { idempotent: true });
     return { success: true, uri: res.uri };
-  } catch (error) {
-    const downloadError = error as DownloadError;
-    return { success: false, error: downloadError.message || "Unknown error" };
-  }
+  } catch (e: any) { return { success: false, error: e.message }; }
 }
 
-export async function downloadFileWithProgress(
-  url: string,
-  cacheDir: string,
-  onProgress?: DownloadProgressCallback,
-): Promise<DownloadWithProgressResult> {
+export async function downloadFileWithProgress(url: string, cacheDir: string, onProgress?: DownloadProgressCallback): Promise<DownloadWithProgressResult> {
   try {
     const dir = new Directory(cacheDir);
-    if (!dir.exists) {
-      dir.create({ intermediates: true, idempotent: true });
-    }
+    if (!dir.exists) dir.create({ intermediates: true, idempotent: true });
 
     const destUri = getCacheUri(url, cacheDir);
-    const cachedFile = new File(destUri);
-    if (cachedFile.exists) {
-      return { success: true, uri: destUri, fromCache: true };
-    }
+    if (new File(destUri).exists) return { success: true, uri: destUri, fromCache: true };
 
     const response = await fetch(url);
-    if (!response.ok) {
-      throw new Error(`HTTP ${response.status}`);
-    }
-
+    if (!response.ok) throw new Error(`HTTP ${response.status}`);
+    
     const totalBytes = parseInt(response.headers.get("content-length") || "0", 10);
-    if (!response.body) {
-      const downloadResult = await downloadFile(url, destUri);
-      return { ...downloadResult, fromCache: false };
-    }
+    if (!response.body) return { ...(await downloadFile(url, destUri)), fromCache: false };
 
     const reader = response.body.getReader();
     const chunks: Uint8Array[] = [];
@@ -77,39 +53,22 @@ export async function downloadFileWithProgress(
       if (done) break;
       chunks.push(value);
       received += value.length;
-      onProgress?.({
-        totalBytesWritten: received,
-        totalBytesExpectedToWrite: totalBytes || received,
-      });
+      onProgress?.({ totalBytesWritten: received, totalBytesExpectedToWrite: totalBytes || received });
     }
 
     const all = new Uint8Array(received);
     let pos = 0;
-    for (const chunk of chunks) {
-      all.set(chunk, pos);
-      pos += chunk.length;
-    }
+    for (const c of chunks) { all.set(c, pos); pos += c.length; }
     new File(destUri).write(all);
 
     return { success: true, uri: destUri, fromCache: false };
-  } catch (error) {
-    const downloadError = error as DownloadError;
-    return { success: false, error: downloadError.message || "Unknown error" };
-  }
+  } catch (e: any) { return { success: false, error: e.message }; }
 }
 
-export const isUrlCached = (url: string, dir: string): boolean => {
-  return new File(getCacheUri(url, dir)).exists;
-};
-
-export const getCachedFileUri = (url: string, dir: string): string | null => {
-  return isUrlCached(url, dir) ? getCacheUri(url, dir) : null;
-};
-
-export const deleteCachedFile = (url: string, dir: string): boolean => {
-  const file = new File(getCacheUri(url, dir));
-  if (file.exists) {
-    file.delete();
-  }
+export const isUrlCached = (url: string, dir: string) => new File(getCacheUri(url, dir)).exists;
+export const getCachedFileUri = (url: string, dir: string) => isUrlCached(url, dir) ? getCacheUri(url, dir) : null;
+export const deleteCachedFile = (url: string, dir: string) => {
+  const f = new File(getCacheUri(url, dir));
+  if (f.exists) f.delete();
   return true;
 };

package/src/infrastructure/utils/blob.utils.ts

@@ -11,8 +11,7 @@ export function blobToBase64(blob: Blob): Promise<string> {
     const reader = new FileReader();
     reader.onloadend = () => {
       const result = reader.result as string;
-      // Remove data URL prefix if present
-      const base64 = result.includes(",") ? result.split(",")[1] : result;
+      const base64 = result.includes(",") ? result.split(",")[1] ?? result : result;
       resolve(base64);
     };
     reader.onerror = reject;