From 220de447e614b9ee82a93ce4f0cafddccb0a34ef Mon Sep 17 00:00:00 2001
From: tdgao <mr.trumgao@gmail.com>
Date: Wed, 12 Aug 2026 16:43:25 -0600
Subject: [PATCH 1/6] fix: stale function call set_webview_visible

---
 apps/app/src/api/ads.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/app/src/api/ads.rs b/apps/app/src/api/ads.rs
index 268811a929..47ab2b0cb8 100644
--- a/apps/app/src/api/ads.rs
+++ b/apps/app/src/api/ads.rs
@@ -190,7 +190,7 @@ async fn sync_ads_occlusion<R: Runtime>(app: &tauri::AppHandle<R>) {
     drop(state);
 
     if let Some(webview) = app.webviews().get("ads-window") {
-        set_webview_visible_for_window(app, webview, visible);
+        set_webview_visible(webview, visible);
     }
 }
 

From 4c86edb4e5eaa0f3692652aa1a891da149bd661f Mon Sep 17 00:00:00 2001
From: tdgao <mr.trumgao@gmail.com>
Date: Wed, 12 Aug 2026 16:51:36 -0600
Subject: [PATCH 2/6] fix: modpack export progress bar and add notification
 when done

---
 .../src/components/ui/ExportModal.vue         | 50 +++++++++++---
 .../app-frontend/src/locales/en-US/index.json |  6 ++
 .../app-lib/src/api/instance/export_mrpack.rs | 69 +++++++++++++------
 packages/app-lib/src/event/mod.rs             |  4 ++
 4 files changed, 98 insertions(+), 31 deletions(-)

diff --git a/apps/app-frontend/src/components/ui/ExportModal.vue b/apps/app-frontend/src/components/ui/ExportModal.vue
index 718a602acf..d0cdd1410b 100644
--- a/apps/app-frontend/src/components/ui/ExportModal.vue
+++ b/apps/app-frontend/src/components/ui/ExportModal.vue
@@ -1,11 +1,12 @@
 <script setup>
-import { XIcon } from '@modrinth/assets'
+import { FolderOpenIcon, XIcon } from '@modrinth/assets'
 import {
 	Button,
 	commonMessages,
 	defineMessages,
 	FileTreeSelect,
 	injectNotificationManager,
+	injectPopupNotificationManager,
 	NewModal,
 	StyledInput,
 	useVIntl,
@@ -15,8 +16,10 @@ import { ref, shallowRef } from 'vue'
 
 import { PackageIcon } from '@/assets/icons'
 import { export_instance_mrpack, get_pack_export_candidates } from '@/helpers/instance'
+import { highlightInFolder } from '@/helpers/utils'
 
 const { handleError } = injectNotificationManager()
+const popupNotificationManager = injectPopupNotificationManager()
 const { formatMessage } = useVIntl()
 
 const messages = defineMessages({
@@ -39,6 +42,14 @@ const messages = defineMessages({
 		defaultMessage: 'Enter modpack description...',
 	},
 	exportButton: { id: 'app.export-modal.export-button', defaultMessage: 'Export' },
+	exportComplete: {
+		id: 'app.export-modal.export-complete',
+		defaultMessage: 'Export complete',
+	},
+	exportCompleteDescription: {
+		id: 'app.export-modal.export-complete-description',
+		defaultMessage: '{name} was exported successfully.',
+	},
 })
 
 const props = defineProps({
@@ -93,16 +104,35 @@ const exportPack = async () => {
 	})
 
 	if (outputPath) {
-		export_instance_mrpack(
-			props.instance.id,
-			outputPath,
-			includedFilePaths.value,
-			excludedFilePaths.value,
-			versionInput.value,
-			exportDescription.value,
-			nameInput.value,
-		).catch((err) => handleError(err))
 		exportModal.value.hide()
+
+		try {
+			await export_instance_mrpack(
+				props.instance.id,
+				outputPath,
+				includedFilePaths.value,
+				excludedFilePaths.value,
+				versionInput.value,
+				exportDescription.value,
+				nameInput.value,
+			)
+
+			const fileName = outputPath.split(/[\\/]/).pop() ?? outputPath
+			popupNotificationManager.addPopupNotification({
+				title: formatMessage(messages.exportComplete),
+				text: formatMessage(messages.exportCompleteDescription, { name: fileName }),
+				type: 'success',
+				buttons: [
+					{
+						label: formatMessage(commonMessages.openInFolderButton),
+						icon: FolderOpenIcon,
+						action: () => highlightInFolder(outputPath).catch(handleError),
+					},
+				],
+			})
+		} catch (error) {
+			handleError(error)
+		}
 	}
 }
 
diff --git a/apps/app-frontend/src/locales/en-US/index.json b/apps/app-frontend/src/locales/en-US/index.json
index 2d69b77eed..564749452e 100644
--- a/apps/app-frontend/src/locales/en-US/index.json
+++ b/apps/app-frontend/src/locales/en-US/index.json
@@ -287,6 +287,12 @@
   "app.export-modal.export-button": {
     "message": "Export"
   },
+  "app.export-modal.export-complete": {
+    "message": "Export complete"
+  },
+  "app.export-modal.export-complete-description": {
+    "message": "{name} was exported successfully."
+  },
   "app.export-modal.header": {
     "message": "Export modpack"
   },
diff --git a/packages/app-lib/src/api/instance/export_mrpack.rs b/packages/app-lib/src/api/instance/export_mrpack.rs
index b0ba914642..fef1155f5f 100644
--- a/packages/app-lib/src/api/instance/export_mrpack.rs
+++ b/packages/app-lib/src/api/instance/export_mrpack.rs
@@ -19,6 +19,7 @@ use std::collections::{HashMap, HashSet};
 use std::path::PathBuf;
 use std::time::UNIX_EPOCH;
 use tokio::fs::File;
+use tokio::io::{AsyncReadExt, AsyncWriteExt};
 use tokio_util::compat::FuturesAsyncWriteCompatExt;
 
 const DEFAULT_SELECTED_EXPORT_PATH_PREFIXES: &[&str] = &[
@@ -29,6 +30,7 @@ const DEFAULT_SELECTED_EXPORT_PATH_PREFIXES: &[&str] = &[
     "config",
 ];
 const EXPORT_CANDIDATE_METADATA_CONCURRENCY: usize = 32;
+const EXPORT_COPY_BUFFER_SIZE: usize = 256 * 1024;
 
 const NEVER_EXPORTABLE_PATH_PREFIXES: &[&str] = &[
     "profile.json",
@@ -196,16 +198,8 @@ pub async fn export_mrpack(
         .iter()
         .map(|file| file.path.as_str().to_string())
         .collect::<HashSet<_>>();
-    let loading_bar = init_loading(
-        LoadingBarType::ZipExtract {
-            instance_id: metadata.instance.id.clone(),
-            instance_name: metadata.instance.name.clone(),
-        },
-        1.0,
-        "Exporting instance to .mrpack",
-    )
-    .await?;
 
+    let mut override_files = Vec::new();
     let mut directories = vec![instance_base_path.clone()];
     while let Some(directory) = directories.pop() {
         let mut read_dir = io::read_dir(&directory).await?;
@@ -238,24 +232,57 @@ pub async fn export_mrpack(
                 continue;
             }
 
-            let mut stream = writer
-                .write_entry_stream(
-                    ZipEntryBuilder::new(
-                        format!("overrides/{relative_path}").into(),
-                        Compression::Deflate,
-                    )
-                    .build(),
+            let size = entry
+                .metadata()
+                .await
+                .map_err(|e| IOError::with_path(e, &path))?
+                .len();
+            override_files.push((path, relative_path, size));
+        }
+    }
+
+    let total_bytes = override_files
+        .iter()
+        .fold(1_u64, |total, (_, _, size)| total.saturating_add(*size));
+    let loading_bar = init_loading(
+        LoadingBarType::PackExport {
+            instance_id: metadata.instance.id.clone(),
+            instance_name: metadata.instance.name.clone(),
+        },
+        total_bytes as f64,
+        "Exporting instance to .mrpack",
+    )
+    .await?;
+    let mut buffer = vec![0_u8; EXPORT_COPY_BUFFER_SIZE];
+    for (path, relative_path, _) in override_files {
+        let mut stream = writer
+            .write_entry_stream(
+                ZipEntryBuilder::new(
+                    format!("overrides/{relative_path}").into(),
+                    Compression::Deflate,
                 )
-                .await?
-                .compat_write();
-            let mut source = File::open(&path)
+                .build(),
+            )
+            .await?
+            .compat_write();
+        let mut source = File::open(&path)
+            .await
+            .map_err(|e| IOError::with_path(e, &path))?;
+        loop {
+            let bytes_read = source
+                .read(&mut buffer)
                 .await
                 .map_err(|e| IOError::with_path(e, &path))?;
-            tokio::io::copy(&mut source, &mut stream)
+            if bytes_read == 0 {
+                break;
+            }
+            stream
+                .write_all(&buffer[..bytes_read])
                 .await
                 .map_err(IOError::from)?;
-            stream.into_inner().close().await?;
+            emit_loading(&loading_bar, bytes_read as f64, None)?;
         }
+        stream.into_inner().close().await?;
     }
 
     let data = serde_json::to_vec_pretty(&packfile)?;
diff --git a/packages/app-lib/src/event/mod.rs b/packages/app-lib/src/event/mod.rs
index af1d5fe69f..63a317d247 100644
--- a/packages/app-lib/src/event/mod.rs
+++ b/packages/app-lib/src/event/mod.rs
@@ -172,6 +172,10 @@ pub enum LoadingBarType {
         instance_id: String,
         instance_name: String,
     },
+    PackExport {
+        instance_id: String,
+        instance_name: String,
+    },
     ConfigChange {
         new_path: PathBuf,
     },

From 04d7beb6af6ac94f9c0998ffc9ea91af95f4dab5 Mon Sep 17 00:00:00 2001
From: tdgao <mr.trumgao@gmail.com>
Date: Wed, 12 Aug 2026 17:06:49 -0600
Subject: [PATCH 3/6] remove: force_no_zip64

---
 packages/app-lib/src/api/instance/export_mrpack.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/app-lib/src/api/instance/export_mrpack.rs b/packages/app-lib/src/api/instance/export_mrpack.rs
index fef1155f5f..873e60ba91 100644
--- a/packages/app-lib/src/api/instance/export_mrpack.rs
+++ b/packages/app-lib/src/api/instance/export_mrpack.rs
@@ -186,7 +186,7 @@ pub async fn export_mrpack(
     let mut file = File::create(&export_path)
         .await
         .map_err(|e| IOError::with_path(e, &export_path))?;
-    let mut writer = ZipFileWriter::with_tokio(&mut file).force_no_zip64();
+    let mut writer = ZipFileWriter::with_tokio(&mut file);
     let version_id = version_id.unwrap_or("1.0.0".to_string());
     let mut packfile =
         create_mrpack_json(&metadata, version_id, description).await?;

From 5435c4e020a0e7244071892ad3acfc0c81da63ef Mon Sep 17 00:00:00 2001
From: tdgao <mr.trumgao@gmail.com>
Date: Thu, 13 Aug 2026 09:25:11 -0600
Subject: [PATCH 4/6] feat: switch to zip crate

---
 .../app-lib/src/api/instance/export_mrpack.rs | 148 +++++++++++++-----
 1 file changed, 110 insertions(+), 38 deletions(-)

diff --git a/packages/app-lib/src/api/instance/export_mrpack.rs b/packages/app-lib/src/api/instance/export_mrpack.rs
index 873e60ba91..2851875cf5 100644
--- a/packages/app-lib/src/api/instance/export_mrpack.rs
+++ b/packages/app-lib/src/api/instance/export_mrpack.rs
@@ -10,17 +10,15 @@ use crate::state::{
     CacheBehaviour, CachedEntry, InstanceMetadata, ModLoader, SideType, State,
 };
 use crate::util::io::{self, IOError};
-use async_zip::tokio::write::ZipFileWriter;
-use async_zip::{Compression, ZipEntryBuilder};
 use futures::{StreamExt, stream};
 use path_util::SafeRelativeUtf8UnixPathBuf;
 use serde::Serialize;
 use std::collections::{HashMap, HashSet};
+use std::io::{Read, Seek, Write};
 use std::path::PathBuf;
 use std::time::UNIX_EPOCH;
-use tokio::fs::File;
-use tokio::io::{AsyncReadExt, AsyncWriteExt};
-use tokio_util::compat::FuturesAsyncWriteCompatExt;
+use zip::write::SimpleFileOptions;
+use zip::{CompressionMethod, ZipWriter};
 
 const DEFAULT_SELECTED_EXPORT_PATH_PREFIXES: &[&str] = &[
     "mods",
@@ -31,6 +29,7 @@ const DEFAULT_SELECTED_EXPORT_PATH_PREFIXES: &[&str] = &[
 ];
 const EXPORT_CANDIDATE_METADATA_CONCURRENCY: usize = 32;
 const EXPORT_COPY_BUFFER_SIZE: usize = 256 * 1024;
+const STANDARD_ZIP_FILE_SIZE_ERROR: &str = "Your modpack cannot be exported as it contains a file over the size limit of 4 GB";
 
 const NEVER_EXPORTABLE_PATH_PREFIXES: &[&str] = &[
     "profile.json",
@@ -183,10 +182,6 @@ pub async fn export_mrpack(
     );
 
     let instance_base_path = get_full_path(instance_id).await?;
-    let mut file = File::create(&export_path)
-        .await
-        .map_err(|e| IOError::with_path(e, &export_path))?;
-    let mut writer = ZipFileWriter::with_tokio(&mut file);
     let version_id = version_id.unwrap_or("1.0.0".to_string());
     let mut packfile =
         create_mrpack_json(&metadata, version_id, description).await?;
@@ -237,6 +232,7 @@ pub async fn export_mrpack(
                 .await
                 .map_err(|e| IOError::with_path(e, &path))?
                 .len();
+            ensure_standard_zip_file_size(size)?;
             override_files.push((path, relative_path, size));
         }
     }
@@ -253,46 +249,69 @@ pub async fn export_mrpack(
         "Exporting instance to .mrpack",
     )
     .await?;
+    let data = serde_json::to_vec_pretty(&packfile)?;
+    tokio::task::spawn_blocking(move || {
+        let file = std::fs::File::create(&export_path)
+            .map_err(|error| IOError::with_path(error, &export_path))?;
+        write_mrpack_archive(file, override_files, &data, |bytes_written| {
+            emit_loading(&loading_bar, bytes_written as f64, None)
+        })
+    })
+    .await??;
+
+    Ok(())
+}
+
+fn ensure_standard_zip_file_size(size: u64) -> crate::Result<()> {
+    if size > zip::ZIP64_BYTES_THR {
+        return Err(crate::ErrorKind::OtherError(
+            STANDARD_ZIP_FILE_SIZE_ERROR.to_string(),
+        )
+        .into());
+    }
+
+    Ok(())
+}
+
+fn write_mrpack_archive<W, F>(
+    writer: W,
+    override_files: Vec<(PathBuf, SafeRelativeUtf8UnixPathBuf, u64)>,
+    packfile_data: &[u8],
+    mut emit_progress: F,
+) -> crate::Result<()>
+where
+    W: Write + Seek,
+    F: FnMut(u64) -> crate::Result<()>,
+{
+    let options = SimpleFileOptions::default()
+        .compression_method(CompressionMethod::Deflated);
+    let mut writer = ZipWriter::new(writer);
     let mut buffer = vec![0_u8; EXPORT_COPY_BUFFER_SIZE];
+
     for (path, relative_path, _) in override_files {
-        let mut stream = writer
-            .write_entry_stream(
-                ZipEntryBuilder::new(
-                    format!("overrides/{relative_path}").into(),
-                    Compression::Deflate,
-                )
-                .build(),
-            )
-            .await?
-            .compat_write();
-        let mut source = File::open(&path)
-            .await
-            .map_err(|e| IOError::with_path(e, &path))?;
+        writer
+            .start_file(format!("overrides/{relative_path}"), options)
+            .map_err(std::io::Error::from)?;
+        let mut source = std::fs::File::open(&path)
+            .map_err(|error| IOError::with_path(error, &path))?;
         loop {
             let bytes_read = source
                 .read(&mut buffer)
-                .await
-                .map_err(|e| IOError::with_path(e, &path))?;
+                .map_err(|error| IOError::with_path(error, &path))?;
             if bytes_read == 0 {
                 break;
             }
-            stream
-                .write_all(&buffer[..bytes_read])
-                .await
-                .map_err(IOError::from)?;
-            emit_loading(&loading_bar, bytes_read as f64, None)?;
+            writer.write_all(&buffer[..bytes_read])?;
+            emit_progress(bytes_read as u64)?;
         }
-        stream.into_inner().close().await?;
     }
 
-    let data = serde_json::to_vec_pretty(&packfile)?;
-    let builder = ZipEntryBuilder::new(
-        "modrinth.index.json".to_string().into(),
-        Compression::Deflate,
-    );
-    writer.write_entry_whole(builder, &data).await?;
-    writer.close().await?;
-    emit_loading(&loading_bar, 1.0, None)?;
+    writer
+        .start_file("modrinth.index.json", options)
+        .map_err(std::io::Error::from)?;
+    writer.write_all(packfile_data)?;
+    writer.finish().map_err(std::io::Error::from)?;
+    emit_progress(1)?;
 
     Ok(())
 }
@@ -549,3 +568,56 @@ pub async fn create_mrpack_json(
         dependencies,
     })
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use std::io::Cursor;
+
+    #[test]
+    fn writes_mrpack_archive_with_deflated_entries_and_progress()
+    -> Result<(), Box<dyn std::error::Error>> {
+        let temp_dir = tempfile::tempdir()?;
+        let override_path = temp_dir.path().join("options.txt");
+        let override_data = b"renderDistance:12";
+        std::fs::write(&override_path, override_data)?;
+        let relative_path = SafeRelativeUtf8UnixPathBuf::try_from(
+            "config/options.txt".to_string(),
+        )?;
+        let manifest_data = br#"{"formatVersion":1}"#;
+        let mut archive_data = Cursor::new(Vec::new());
+        let mut progress = 0;
+
+        write_mrpack_archive(
+            &mut archive_data,
+            vec![(override_path, relative_path, override_data.len() as u64)],
+            manifest_data,
+            |increment| {
+                progress += increment;
+                Ok(())
+            },
+        )?;
+
+        assert_eq!(progress, override_data.len() as u64 + 1);
+        archive_data.set_position(0);
+        let mut archive = zip::ZipArchive::new(archive_data)?;
+        assert_eq!(archive.len(), 2);
+
+        {
+            let mut entry = archive.by_name("overrides/config/options.txt")?;
+            assert_eq!(entry.compression(), CompressionMethod::Deflated);
+            let mut data = Vec::new();
+            entry.read_to_end(&mut data)?;
+            assert_eq!(data, override_data);
+        }
+        {
+            let mut entry = archive.by_name("modrinth.index.json")?;
+            assert_eq!(entry.compression(), CompressionMethod::Deflated);
+            let mut data = Vec::new();
+            entry.read_to_end(&mut data)?;
+            assert_eq!(data, manifest_data);
+        }
+
+        Ok(())
+    }
+}

From 9b0e8a9278b0caa088bfa089c2d269dba43a4ca2 Mon Sep 17 00:00:00 2001
From: tdgao <mr.trumgao@gmail.com>
Date: Thu, 13 Aug 2026 09:32:42 -0600
Subject: [PATCH 5/6] add typo exception

---
 _typos.toml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/_typos.toml b/_typos.toml
index 74efbceb9b..7775afa964 100644
--- a/_typos.toml
+++ b/_typos.toml
@@ -24,3 +24,7 @@ gam = "gam"
 consts = "consts"
 # short for "Copy"
 Cpy = "Cpy"
+
+[default.extend-identifiers]
+# Constant from the `zip` crate
+ZIP64_BYTES_THR = "ZIP64_BYTES_THR"

From 001f858d440814f2f1695c2e08a589fda2d52475 Mon Sep 17 00:00:00 2001
From: "Calum H. (IMB11)" <contact@cal.engineer>
Date: Thu, 13 Aug 2026 17:18:48 +0100
Subject: [PATCH 6/6] fix: remove unit test (dont do)

---
 .../app-lib/src/api/instance/export_mrpack.rs | 53 -------------------
 1 file changed, 53 deletions(-)

diff --git a/packages/app-lib/src/api/instance/export_mrpack.rs b/packages/app-lib/src/api/instance/export_mrpack.rs
index f72c21a0be..d3ad218c04 100644
--- a/packages/app-lib/src/api/instance/export_mrpack.rs
+++ b/packages/app-lib/src/api/instance/export_mrpack.rs
@@ -601,56 +601,3 @@ pub async fn create_mrpack_json(
         dependencies,
     })
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::io::Cursor;
-
-    #[test]
-    fn writes_mrpack_archive_with_deflated_entries_and_progress()
-    -> Result<(), Box<dyn std::error::Error>> {
-        let temp_dir = tempfile::tempdir()?;
-        let override_path = temp_dir.path().join("options.txt");
-        let override_data = b"renderDistance:12";
-        std::fs::write(&override_path, override_data)?;
-        let relative_path = SafeRelativeUtf8UnixPathBuf::try_from(
-            "config/options.txt".to_string(),
-        )?;
-        let manifest_data = br#"{"formatVersion":1}"#;
-        let mut archive_data = Cursor::new(Vec::new());
-        let mut progress = 0;
-
-        write_mrpack_archive(
-            &mut archive_data,
-            vec![(override_path, relative_path, override_data.len() as u64)],
-            manifest_data,
-            |increment| {
-                progress += increment;
-                Ok(())
-            },
-        )?;
-
-        assert_eq!(progress, override_data.len() as u64 + 1);
-        archive_data.set_position(0);
-        let mut archive = zip::ZipArchive::new(archive_data)?;
-        assert_eq!(archive.len(), 2);
-
-        {
-            let mut entry = archive.by_name("overrides/config/options.txt")?;
-            assert_eq!(entry.compression(), CompressionMethod::Deflated);
-            let mut data = Vec::new();
-            entry.read_to_end(&mut data)?;
-            assert_eq!(data, override_data);
-        }
-        {
-            let mut entry = archive.by_name("modrinth.index.json")?;
-            assert_eq!(entry.compression(), CompressionMethod::Deflated);
-            let mut data = Vec::new();
-            entry.read_to_end(&mut data)?;
-            assert_eq!(data, manifest_data);
-        }
-
-        Ok(())
-    }
-}