Use hash for proxy's hex_encoding (#319)

- Reuse existing hash-encoding pattern to avoid creating filenames
breaching MacOS 255-bytes limit
- Update debugger to use same hash-encoding when retrieving the HTTP
port
- Address #318

Author: tartarughina

proxy/src/main.rs

@@ -15,6 +15,7 @@ use proxy_common::{
     spawn_parent_monitor, LspReader,
 };
 use serde_json::{json, Value};
+use sha1::{Digest, Sha1};
 use std::{
     collections::{HashMap, HashSet},
     env, fs,
@@ -588,7 +589,7 @@ fn queue_rewrite_response(
 // --- Utilities ---
 
 fn hex_encode(s: &str) -> String {
-    s.as_bytes().iter().map(|b| format!("{b:02x}")).collect()
+    hex::encode(Sha1::digest(s.as_bytes()))
 }
 
 fn clear_latest_workspace(latest: &Mutex<Option<u64>>, token: u64) {
@@ -1246,4 +1247,34 @@ mod tests {
 
         assert_eq!(completion_resolve_fallback(&error, &tracked), None);
     }
+
+    #[test]
+    fn proxy_id_stays_under_macos_filename_limit() {
+        let long_workspace = format!("/Users/test/{}", "a".repeat(200));
+
+        let id = hex_encode(&long_workspace);
+
+        assert!(
+            id.len() <= 255,
+            "proxy_id exceeded macOS filename limit: {} bytes",
+            id.len()
+        );
+        assert_eq!(id.len(), 40, "SHA-1 hex should always be 40 chars");
+    }
+
+    #[test]
+    fn proxy_port_file_works_for_long_workdir() {
+        let temp = std::env::temp_dir().join("java-proxy-long-path-test");
+        let _ = fs::remove_dir_all(&temp);
+        let long_workspace = temp.join("a".repeat(200));
+        fs::create_dir_all(&long_workspace).unwrap();
+
+        let proxy_id = hex_encode(long_workspace.to_string_lossy().trim_end_matches('/'));
+        let port_file = long_workspace.join("proxy").join(&proxy_id);
+
+        fs::create_dir_all(port_file.parent().unwrap()).unwrap();
+        fs::write(&port_file, "3000").unwrap();
+
+        assert!(port_file.file_name().unwrap().len() <= 255);
+    }
 }

src/lsp.rs

@@ -5,6 +5,7 @@ use std::{
 
 use serde::{Deserialize, Serialize, de::DeserializeOwned};
 use serde_json::json;
+use sha1::{Digest, Sha1};
 use zed_extension_api::{
     self as zed,
     http_client::{HttpMethod, HttpRequest, fetch},
@@ -89,11 +90,7 @@ where
 }
 
 fn string_to_hex(s: &str) -> String {
-    let mut hex_string = String::new();
-    for byte in s.as_bytes() {
-        hex_string.push_str(&format!("{byte:02x}"));
-    }
-    hex_string
+    hex::encode(Sha1::digest(s.as_bytes()))
 }
 
 #[derive(Serialize, Deserialize)]