fix(vmm): hide app url until instance id is ready

Author: kvinwang

vmm/src/app/qemu.rs

@@ -69,6 +69,26 @@ fn networking_to_proto(n: &Networking) -> pb::NetworkingConfig {
     pb::NetworkingConfig { mode: mode.into() }
 }
 
+fn sanitize_optional_string(value: Option<String>) -> Option<String> {
+    value.and_then(|value| {
+        if value.trim().is_empty() {
+            None
+        } else {
+            Some(value)
+        }
+    })
+}
+
+fn sanitize_optional_str<'a>(value: Option<&'a str>) -> Option<&'a str> {
+    value.and_then(|value| {
+        if value.trim().is_empty() {
+            None
+        } else {
+            Some(value)
+        }
+    })
+}
+
 #[derive(Debug, Deserialize)]
 pub struct InstanceInfo {
     #[serde(default)]
@@ -271,8 +291,9 @@ impl VmInfo {
             },
             app_url: self
                 .gateway_enabled
-                .then_some(self.instance_id.as_ref())
+                .then_some(self.instance_id.as_deref())
                 .flatten()
+                .and_then(|id| sanitize_optional_str(Some(id)))
                 .map(|id| {
                     // Use custom gateway URL if available, otherwise fall back to global config
                     if let Some(custom_gw_url) = custom_gateway_urls.first() {
@@ -297,7 +318,7 @@ impl VmInfo {
                     }
                 }),
             app_id: self.manifest.app_id.clone(),
-            instance_id: self.instance_id.as_deref().map(Into::into),
+            instance_id: sanitize_optional_string(self.instance_id.clone()),
             exited_at: self.exited_at.clone(),
             events: self.events.clone(),
         }
@@ -336,7 +357,8 @@ impl VmState {
         }
         let uptime = display_ts(proc_state.and_then(|info| info.state.started_at.as_ref()));
         let exited_at = display_ts(proc_state.and_then(|info| info.state.stopped_at.as_ref()));
-        let instance_id = workdir.instance_info().ok().map(|info| info.instance_id);
+        let instance_id =
+            sanitize_optional_string(workdir.instance_info().ok().map(|info| info.instance_id));
         VmInfo {
             manifest: self.config.manifest.clone(),
             workdir: workdir.path().to_path_buf(),
@@ -354,6 +376,31 @@ impl VmState {
     }
 }
 
+#[cfg(test)]
+mod tests {
+    use super::{sanitize_optional_str, sanitize_optional_string};
+
+    #[test]
+    fn sanitize_optional_string_filters_empty_values() {
+        assert_eq!(sanitize_optional_string(Some(String::new())), None);
+        assert_eq!(sanitize_optional_string(Some("   ".to_string())), None);
+        assert_eq!(
+            sanitize_optional_string(Some("instance-123".to_string())),
+            Some("instance-123".to_string())
+        );
+    }
+
+    #[test]
+    fn sanitize_optional_str_filters_empty_values() {
+        assert_eq!(sanitize_optional_str(Some("")), None);
+        assert_eq!(sanitize_optional_str(Some("   ")), None);
+        assert_eq!(
+            sanitize_optional_str(Some("instance-123")),
+            Some("instance-123")
+        );
+    }
+}
+
 impl VmConfig {
     pub fn config_qemu(
         &self,