Use fflate instead of JSZip.

Author: ignaciogros

README.md

@@ -169,7 +169,7 @@ Firefox doesn't support PWA installation natively. Use the [PWAs for Firefox](ht
    - Caches the application shell for offline use.
    - Intercepts requests to `/viewer/*` and serves extracted ZIP content from memory.
 
-2. **ZIP processing** (`js/app.js` + `js/zip.worker.js`): Uses JSZip in a Web Worker to extract content in a background thread without blocking the UI. Files are sent to the Service Worker as ArrayBuffers using Transferable objects (zero-copy transfer).
+2. **ZIP processing** (`js/app.js` + `js/zip.worker.js`): Uses fflate in a Web Worker to extract content in a background thread without blocking the UI. Files are sent to the Service Worker as ArrayBuffers using Transferable objects (zero-copy transfer).
 
 3. **Content display**: An iframe loads `/viewer/index.html`. The Service Worker intercepts this request and serves the corresponding file from the extracted content.
 
@@ -244,7 +244,7 @@ exeviewer/
 ├── vendor/             # Third-party libraries
 │   ├── bootstrap/      # Bootstrap 5.3.2
 │   ├── bootstrap-icons/# Bootstrap Icons 1.11.1
-│   └── jszip/          # JSZip 3.10.1
+│   └── fflate/         # fflate 0.8.2
 └── scripts/
     └── generate-icons.js   # Icon generation script (requires Node.js + sharp)
 ```
@@ -265,7 +265,7 @@ You should have received a copy of the GNU Affero General Public License along w
 
 - [Bootstrap](https://getbootstrap.com/) v5.3.2 - Copyright 2011-2023 The Bootstrap Authors - MIT License
 - [Bootstrap Icons](https://icons.getbootstrap.com/) v1.11.1 - Copyright 2019-2023 The Bootstrap Authors - MIT License
-- [JSZip](https://stuk.github.io/jszip/) v3.10.1 - Copyright 2009-2016 Stuart Knightley - MIT License or GPLv3
+- [fflate](https://github.com/101arrowz/fflate) v0.8.2 - Copyright 2020 Arjun Barrett - MIT License
 
 ### Related projects
 

README_es.md

@@ -169,7 +169,7 @@ Firefox no soporta la instalación de PWA de forma nativa. Usa la extensión [PW
    - Almacena en caché la estructura de la aplicación para uso sin conexión.
    - Intercepta las peticiones a `/viewer/*` y sirve el contenido ZIP extraído desde memoria.
 
-2. **Procesamiento ZIP** (`js/app.js` + `js/zip.worker.js`): Usa JSZip en un Web Worker para extraer el contenido en un hilo secundario sin bloquear la interfaz. Los ficheros se envían al Service Worker como ArrayBuffers usando objetos Transferable (transferencia sin copia).
+2. **Procesamiento ZIP** (`js/app.js` + `js/zip.worker.js`): Usa fflate en un Web Worker para extraer el contenido en un hilo secundario sin bloquear la interfaz. Los ficheros se envían al Service Worker como ArrayBuffers usando objetos Transferable (transferencia sin copia).
 
 3. **Visualización del contenido**: Un iframe carga `/viewer/index.html`. El Service Worker intercepta esta petición y sirve el fichero correspondiente del contenido extraído.
 
@@ -244,7 +244,7 @@ exeviewer/
 ├── vendor/             # Bibliotecas de terceros
 │   ├── bootstrap/      # Bootstrap 5.3.2
 │   ├── bootstrap-icons/# Bootstrap Icons 1.11.1
-│   └── jszip/          # JSZip 3.10.1
+│   └── fflate/         # fflate 0.8.2
 └── scripts/
     └── generate-icons.js   # Script de generación de iconos (requiere Node.js + sharp)
 ```
@@ -265,7 +265,7 @@ Deberías haber recibido una copia de la Licencia Pública General Affero de GNU
 
 - [Bootstrap](https://getbootstrap.com/) v5.3.2 - Copyright 2011-2023 The Bootstrap Authors - Licencia MIT
 - [Bootstrap Icons](https://icons.getbootstrap.com/) v1.11.1 - Copyright 2019-2023 The Bootstrap Authors - Licencia MIT
-- [JSZip](https://stuk.github.io/jszip/) v3.10.1 - Copyright 2009-2016 Stuart Knightley - Licencia MIT o GPLv3
+- [fflate](https://github.com/101arrowz/fflate) v0.8.2 - Copyright 2020 Arjun Barrett - Licencia MIT
 
 ### Proyectos relacionados
 

index.html

@@ -231,9 +231,6 @@ <h5 class="modal-title" id="shareModalLabel" data-i18n="share.modalTitle">Share
         </div>
     </div>
 
-    <!-- JSZip Library -->
-    <script src="vendor/jszip/jszip.min.js"></script>
-
     <!-- Bootstrap JS -->
     <script src="vendor/bootstrap/js/bootstrap.bundle.min.js"></script>
 

js/zip.worker.js

@@ -1,10 +1,29 @@
 /**
  * eXeViewer - ZIP Extraction Web Worker
- * Handles ZIP file extraction in a background thread to avoid blocking the UI
+ * Handles ZIP file extraction in a background thread using fflate
  */
 
-// Import JSZip library
-importScripts('../vendor/jszip/jszip.min.js');
+// Import fflate library
+importScripts('../vendor/fflate/fflate.min.js');
+
+/**
+ * Find common prefix in file paths (for ZIPs with a root folder)
+ * @param {string[]} fileList - List of file paths
+ * @returns {string} Common prefix or empty string
+ */
+function findCommonPrefix(fileList) {
+    const htmlFiles = fileList.filter(f => f.endsWith('.html') || f.endsWith('.htm'));
+    if (htmlFiles.length === 0) return '';
+
+    const indexFile = htmlFiles.find(f => f === 'index.html' || f.endsWith('/index.html'));
+    if (indexFile && indexFile.includes('/')) {
+        const parts = indexFile.split('/');
+        if (parts.length === 2) {
+            return parts[0] + '/';
+        }
+    }
+    return '';
+}
 
 /**
  * Handle messages from the main thread
@@ -16,87 +35,65 @@ self.onmessage = async function(e) {
         // Notify start of reading
         self.postMessage({ type: 'progress', message: 'reading' });
 
-        const zip = await JSZip.loadAsync(file);
-        const files = {};
-        const fileList = Object.keys(zip.files);
+        // Read file as ArrayBuffer
+        const arrayBuffer = await file.arrayBuffer();
+        const uint8Array = new Uint8Array(arrayBuffer);
 
-        console.log(`[Worker] ZIP contains ${fileList.length} entries`);
-
-        // Find common prefix (if files are in a subdirectory)
-        let commonPrefix = '';
-        const htmlFiles = fileList.filter(f => f.endsWith('.html') || f.endsWith('.htm'));
-
-        if (htmlFiles.length > 0) {
-            // Check if there's an index.html at root or in a subfolder
-            const indexFile = htmlFiles.find(f =>
-                f === 'index.html' ||
-                f.endsWith('/index.html')
-            );
-
-            if (indexFile && indexFile.includes('/')) {
-                // Get the prefix (folder containing index.html)
-                const parts = indexFile.split('/');
-                if (parts.length === 2) {
-                    commonPrefix = parts[0] + '/';
-                }
+        // Use fflate to unzip
+        fflate.unzip(uint8Array, (err, unzipped) => {
+            if (err) {
+                console.error('[Worker] fflate error:', err);
+                self.postMessage({ type: 'error', message: err.message });
+                return;
             }
-        }
 
-        console.log(`[Worker] Common prefix: "${commonPrefix}"`);
+            const fileList = Object.keys(unzipped);
+            console.log(`[Worker] ZIP contains ${fileList.length} entries`);
 
-        let processed = 0;
-        const total = fileList.length;
-        const transferables = [];
+            // Find common prefix (if files are in a subdirectory)
+            const commonPrefix = findCommonPrefix(fileList);
+            console.log(`[Worker] Common prefix: "${commonPrefix}"`);
 
-        for (const relativePath of fileList) {
-            const zipEntry = zip.files[relativePath];
+            const files = {};
+            const transferables = [];
 
-            // Skip directories
-            if (zipEntry.dir) {
-                processed++;
-                continue;
-            }
+            for (const path of fileList) {
+                const content = unzipped[path];
 
-            // Get the normalized path (remove common prefix if exists)
-            let normalizedPath = relativePath;
-            if (commonPrefix && relativePath.startsWith(commonPrefix)) {
-                normalizedPath = relativePath.substring(commonPrefix.length);
-            }
+                // Skip empty entries (directories in fflate have zero-length arrays)
+                if (content.length === 0) {
+                    continue;
+                }
 
-            // Skip empty paths
-            if (!normalizedPath) {
-                processed++;
-                continue;
-            }
+                // Get the normalized path (remove common prefix if exists)
+                let normalizedPath = path;
+                if (commonPrefix && path.startsWith(commonPrefix)) {
+                    normalizedPath = path.substring(commonPrefix.length);
+                }
 
-            // Read file as ArrayBuffer (more efficient than base64)
-            try {
-                const content = await zipEntry.async('arraybuffer');
-                files[normalizedPath] = content;
-                transferables.push(content);
-            } catch (error) {
-                console.warn(`[Worker] Error reading ${relativePath}:`, error);
-            }
+                // Skip empty paths
+                if (!normalizedPath) {
+                    continue;
+                }
 
-            processed++;
-            if (processed % 50 === 0) {
-                self.postMessage({
-                    type: 'progress',
-                    message: 'extracting',
-                    processed,
-                    total
-                });
+                // Convert Uint8Array to ArrayBuffer for transfer
+                const buffer = content.buffer.slice(
+                    content.byteOffset,
+                    content.byteOffset + content.byteLength
+                );
+                files[normalizedPath] = buffer;
+                transferables.push(buffer);
             }
-        }
 
-        const fileCount = Object.keys(files).length;
-        console.log(`[Worker] Extracted ${fileCount} files`);
+            const fileCount = Object.keys(files).length;
+            console.log(`[Worker] Extracted ${fileCount} files`);
 
-        // Transfer ArrayBuffers to main thread (zero-copy transfer)
-        self.postMessage(
-            { type: 'complete', files, fileCount },
-            transferables
-        );
+            // Transfer ArrayBuffers to main thread (zero-copy transfer)
+            self.postMessage(
+                { type: 'complete', files, fileCount },
+                transferables
+            );
+        });
 
     } catch (error) {
         console.error('[Worker] Error:', error);

sw.js

@@ -3,7 +3,7 @@
  * Handles PWA caching and serves extracted ZIP content from memory/IndexedDB
  */
 
-const SW_VERSION = '1.6.0';
+const SW_VERSION = '1.7.0';
 const CACHE_NAME = `exeviewer-v${SW_VERSION}`;
 
 // IndexedDB configuration
@@ -30,7 +30,7 @@ const APP_SHELL_FILES = [
     './vendor/bootstrap-icons/bootstrap-icons.min.css',
     './vendor/bootstrap-icons/fonts/bootstrap-icons.woff2',
     './vendor/bootstrap-icons/fonts/bootstrap-icons.woff',
-    './vendor/jszip/jszip.min.js'
+    './vendor/fflate/fflate.min.js'
 ];
 
 // In-memory storage for the extracted ZIP contents