refactor(hm-plugin-protocol): Mark the wire-protocol BuildEvent enum #[non_exhaustive] (#123)

Author: markovejnovic

crates/hm-plugin-protocol/src/events.rs

@@ -30,6 +30,7 @@ pub enum StdStream {
     Debug, Clone, PartialEq, Eq, Serialize, Deserialize, DeriveJsonSchema, derive_more::IsVariant,
 )]
 #[serde(tag = "kind", rename_all = "snake_case")]
+#[non_exhaustive]
 pub enum BuildEvent {
     BuildStart {
         run_id: Uuid,

crates/hm-render/src/human.rs

@@ -126,13 +126,12 @@ where
                 duration_ms,
             } => format!("build: end exit={exit_code} duration={duration_ms}ms\n").into_bytes(),
 
-            BuildEvent::BuildAccepted { build, watch_url } => {
-                if let Some(url) = watch_url {
-                    let n = build.number.map(|n| format!("#{n} ")).unwrap_or_default();
-                    format!("build {n}\u{2192} {url}\n").into_bytes()
-                } else {
-                    return;
-                }
+            BuildEvent::BuildAccepted {
+                build,
+                watch_url: Some(url),
+            } => {
+                let n = build.number.map(|n| format!("#{n} ")).unwrap_or_default();
+                format!("build {n}\u{2192} {url}\n").into_bytes()
             }
 
             BuildEvent::ChainFailed {
@@ -152,6 +151,8 @@ where
                 )
                 .into_bytes()
             }
+
+            _ => return, // unknown future event: no visible output
         };
 
         let _ = self.out.write_all(&bytes);

crates/hm-render/src/progress.rs

@@ -336,15 +336,14 @@ where
                 }
             }
 
-            BuildEvent::BuildAccepted { build, watch_url } => {
-                if let Some(url) = watch_url {
-                    let n = build.number.map(|n| format!("#{n} ")).unwrap_or_default();
-                    let _ = writeln!(self.out, "build {n}\u{2192} {url}");
-                }
+            BuildEvent::BuildAccepted {
+                build,
+                watch_url: Some(url),
+            } => {
+                let n = build.number.map(|n| format!("#{n} ")).unwrap_or_default();
+                let _ = writeln!(self.out, "build {n}\u{2192} {url}");
             }
 
-            BuildEvent::ChainFailed { .. } => {}
-
             BuildEvent::BuildEnd {
                 exit_code,
                 duration_ms,
@@ -373,6 +372,8 @@ where
                     );
                 }
             }
+
+            _ => {} // unknown future event: no progress update
         }
     }
 }