From 2c90c392114a8948190e4253f0cae9379f3a614d Mon Sep 17 00:00:00 2001
From: "Begonia, HE" <163421589+BegoniaHe@users.noreply.github.com>
Date: Sun, 25 Jan 2026 10:46:30 +0100
Subject: refactor(java): replace unwrap with expect for better error handling

Replace potentially panicking unwrap() call with expect() that includes a descriptive error message to aid debugging if the edge case occurs.

Reviewed-by: Claude Sonnet 4.5
---
 src-tauri/src/core/java/persistence.rs | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

(limited to 'src-tauri')

diff --git a/src-tauri/src/core/java/persistence.rs b/src-tauri/src/core/java/persistence.rs
index 5e263fb..d1e999e 100644
--- a/src-tauri/src/core/java/persistence.rs
+++ b/src-tauri/src/core/java/persistence.rs
@@ -42,7 +42,12 @@ pub fn load_java_config(app_handle: &AppHandle) -> JavaConfig {
 pub fn save_java_config(app_handle: &AppHandle, config: &JavaConfig) -> Result<(), String> {
     let config_path = get_java_config_path(app_handle);
     let content = serde_json::to_string_pretty(config).map_err(|e| e.to_string())?;
-    std::fs::create_dir_all(config_path.parent().unwrap()).map_err(|e| e.to_string())?;
+    std::fs::create_dir_all(
+        config_path
+            .parent()
+            .expect("Java config path should have a parent directory"),
+    )
+    .map_err(|e| e.to_string())?;
     std::fs::write(&config_path, content).map_err(|e| e.to_string())?;
     Ok(())
 }
-- 
cgit v1.2.3-70-g09d2