@zip.js/zip.js 2.8.60 → 2.8.61

package/dist/zip.js

@@ -6660,6 +6660,13 @@
 		const { comment } = metadataInfo;
 		const extraField = options[PROPERTY_NAME_EXTRA_FIELD];
 		zipWriter.fileEntries.set(name, UNDEFINED_VALUE);
+		const previousFileEntry = zipWriter.lastFileEntry;
+		const pendingFileEntry = {};
+		let releaseLockFileEntry;
+		if (metadataInfo.resolvedOptions.keepOrder) {
+			pendingFileEntry.lockFileEntry = new Promise(resolve => releaseLockFileEntry = resolve);
+		}
+		zipWriter.lastFileEntry = pendingFileEntry;
 		let fileEntry;
 		try {
 			const { resolvedOptions } = metadataInfo;
@@ -6681,10 +6688,21 @@
 			const headerInfo = getHeaderInfo(options);
 			const dataDescriptorInfo = getDataDescriptorInfo(options);
 			const metadataSize = getLength(headerInfo.localHeaderArray, dataDescriptorInfo.dataDescriptorArray);
-			fileEntry = await getFileEntry(zipWriter, name, reader, { headerInfo, dataDescriptorInfo, metadataSize }, options);
+			fileEntry = await getFileEntry(zipWriter, name, reader, {
+				headerInfo,
+				dataDescriptorInfo,
+				metadataSize,
+				fileEntry: pendingFileEntry,
+				previousFileEntry,
+				releaseLockFileEntry
+			}, options);
 		} catch (error) {
 			zipWriter.fileEntries.delete(name);
 			throw error;
+		} finally {
+			if (releaseLockFileEntry) {
+				releaseLockFileEntry(previousFileEntry && previousFileEntry.lockFileEntry);
+			}
 		}
 		Object.assign(fileEntry, {
 			name,
@@ -7070,27 +7088,23 @@
 			signal
 		} = options;
 		const {
-			headerInfo
+			headerInfo,
+			fileEntry: pendingFileEntry,
+			previousFileEntry,
+			releaseLockFileEntry
 		} = entryInfo;
 		const usdz = zipWriter.options[OPTION_USDZ];
-		const previousFileEntry = zipWriter.lastFileEntry;
-		let fileEntry = {};
+		let fileEntry = pendingFileEntry;
 		let bufferedWrite;
 		let releaseLockWriter;
-		let releaseLockCurrentFileEntry;
 		let writingBufferedEntryData;
 		let writingEntryData;
 		let writerSizeBeforeEntry;
 		let flushedBufferedSize = 0;
 		let fileWriter;
+		const lockPreviousFileEntry = keepOrder && previousFileEntry ? previousFileEntry.lockFileEntry : UNDEFINED_VALUE;
 		fileEntries.set(name, fileEntry);
-		zipWriter.lastFileEntry = fileEntry;
 		try {
-			let lockPreviousFileEntry;
-			if (keepOrder) {
-				lockPreviousFileEntry = previousFileEntry && previousFileEntry.lockFileEntry;
-				requestLockCurrentFileEntry();
-			}
 			if (options.bufferedWrite || !keepOrder || zipWriter.writerLocked || zipWriter.bufferedWrites || (!dataDescriptor && !emptyEntry)) {
 				bufferedWrite = true;
 				zipWriter.bufferedWrites++;
@@ -7103,6 +7117,7 @@
 				await initStream(writer);
 			} else {
 				fileWriter = writer;
+				await lockPreviousFileEntry;
 				await requestLockWriter();
 			}
 			await initStream(fileWriter);
@@ -7115,7 +7130,6 @@
 			}
 			const { localHeaderArray } = headerInfo;
 			if (!bufferedWrite) {
-				await lockPreviousFileEntry;
 				await skipDiskIfNeeded();
 			}
 			const diskNumberStart = getDiskNumber(writer);
@@ -7180,8 +7194,8 @@
 			if (bufferedWrite) {
 				zipWriter.bufferedWrites--;
 			}
-			if (releaseLockCurrentFileEntry) {
-				releaseLockCurrentFileEntry();
+			if (releaseLockFileEntry) {
+				releaseLockFileEntry(lockPreviousFileEntry);
 			}
 			if (releaseLockWriter) {
 				releaseLockWriter();
@@ -7195,10 +7209,6 @@
 			}
 		}
 
-		function requestLockCurrentFileEntry() {
-			fileEntry.lockFileEntry = new Promise(resolve => releaseLockCurrentFileEntry = resolve);
-		}
-
 		async function requestLockWriter() {
 			zipWriter.writerLocked = true;
 			const { lockWriter } = zipWriter;
@@ -8518,7 +8528,7 @@
 	 EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 	 */
 
-	const VERSION = "2.8.60";
+	const VERSION = "2.8.61";
 
 	/*
 	 Copyright (c) 2025 Gildas Lormeau. All rights reserved.