fix(windows): quote paths in JDTLS and Lombok arguments

When the user path contains spaces (e.g. `C:\Users\User Name`), the Java process fails to start with a `ClassNotFoundException` because the paths in arguments are split by the shell/process.

This change:
- Quotes the path for `-Dosgi.sharedConfiguration.area` in JDTLS launch args on Windows.
- Quotes the path for `-javaagent` (Lombok) on Windows.
- Adds unit tests to verify path escaping behavior across platforms.

Closes #150

Author: Uriel Curiel

src/java.rs

@@ -143,6 +143,37 @@ impl Java {
     }
 }
 
+fn format_lombok_agent_arg(path: &str, os: zed::Os) -> String {
+    if os == zed::Os::Windows {
+        format!("-javaagent:\"{path}\"")
+    } else {
+        format!("-javaagent:{path}")
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use zed_extension_api::Os;
+
+    #[test]
+    fn test_format_lombok_agent_arg_windows() {
+        let path = "C:\\Users\\User Name\\lombok.jar";
+        let arg = format_lombok_agent_arg(path, Os::Windows);
+        assert_eq!(arg, format!("-javaagent:\"{path}\""));
+    }
+
+    #[test]
+    fn test_format_lombok_agent_arg_unix() {
+        let path = "/Users/username/lombok.jar";
+        let arg = format_lombok_agent_arg(path, Os::Mac);
+        assert_eq!(arg, format!("-javaagent:{path}"));
+
+        let arg_linux = format_lombok_agent_arg(path, Os::Linux);
+        assert_eq!(arg_linux, format!("-javaagent:{path}"));
+    }
+}
+
 impl Extension for Java {
     fn new() -> Self
     where
@@ -288,7 +319,10 @@ impl Extension for Java {
                 self.lombok_jar_path(language_server_id, &configuration, worktree)?;
             let canonical_lombok_jar_path = path_to_string(current_dir.join(lombok_jar_path))?;
 
-            Some(format!("-javaagent:{canonical_lombok_jar_path}"))
+            Some(format_lombok_agent_arg(
+                &canonical_lombok_jar_path,
+                zed::current_platform().0,
+            ))
         } else {
             None
         };

src/jdtls.rs

@@ -72,7 +72,7 @@ pub fn build_jdtls_launch_args(
         "-Dosgi.checkConfiguration=true".to_string(),
         format!(
             "-Dosgi.sharedConfiguration.area={}",
-            path_to_string(shared_config_path)?
+            escape_path_if_needed(path_to_string(shared_config_path)?, current_platform().0)
         ),
         "-Dosgi.sharedConfiguration.area.readOnly=true".to_string(),
         "-Dosgi.configuration.cascaded=true".to_string(),
@@ -353,3 +353,34 @@ fn get_shared_config_path(jdtls_base_directory: &Path) -> PathBuf {
     };
     jdtls_base_directory.join(config_to_use)
 }
+
+fn escape_path_if_needed(path: String, os: Os) -> String {
+    if os == Os::Windows {
+        format!("\"{}\"", path)
+    } else {
+        path
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use zed_extension_api::Os;
+
+    #[test]
+    fn test_escape_path_if_needed_windows() {
+        let path = "C:\\Users\\User Name\\Projects\\zed-extension-java".to_string();
+        let escaped = escape_path_if_needed(path.clone(), Os::Windows);
+        assert_eq!(escaped, format!("\"{}\"", path));
+    }
+
+    #[test]
+    fn test_escape_path_if_needed_unix() {
+        let path = "/Users/username/Projects/zed-extension-java".to_string();
+        let escaped = escape_path_if_needed(path.clone(), Os::Mac);
+        assert_eq!(escaped, path);
+
+        let escaped_linux = escape_path_if_needed(path.clone(), Os::Linux);
+        assert_eq!(escaped_linux, path);
+    }
+}