refactor(hm-common): generalize CompactDuration over Into<Duration>

Blanket-impl CompactDuration for any `T: Into<Duration>` instead of only
`Duration`, so `DurationMs` (and any future duration newtype) gets `.compact()`
for free — no dependency edge into hm-pipeline-ir, since the impl never names
the concrete type. Render sites drop `Duration::from(*ms).compact()` back to
`ms.compact()`.

Author: markovejnovic

crates/hm-common/src/format.rs

@@ -5,11 +5,13 @@ use core::time::Duration;
 
 mod sealed {
     pub trait Sealed {}
-    impl Sealed for core::time::Duration {}
+    impl<T: Into<core::time::Duration>> Sealed for T {}
 }
 
-/// Extension trait adding a compact, stopwatch-style [`Display`] rendering to
-/// [`std::time::Duration`].
+/// Adds a compact, stopwatch-style [`Display`] rendering to any duration.
+///
+/// Implemented for any type convertible into a [`std::time::Duration`] —
+/// including [`Duration`] itself and newtypes like `hm_pipeline_ir::DurationMs`.
 ///
 /// ```
 /// # use hm_common::format::CompactDuration;
@@ -21,10 +23,11 @@ pub trait CompactDuration: sealed::Sealed {
     fn compact(self) -> StopwatchDurationDisplay;
 }
 
-impl CompactDuration for Duration {
+impl<T: Into<Duration>> CompactDuration for T {
     fn compact(self) -> StopwatchDurationDisplay {
+        let d: Duration = self.into();
         StopwatchDurationDisplay {
-            total_ms: u64::try_from(self.as_millis()).unwrap_or(u64::MAX),
+            total_ms: u64::try_from(d.as_millis()).unwrap_or(u64::MAX),
         }
     }
 }

crates/hm-render/src/progress.rs

@@ -10,7 +10,6 @@
 use std::collections::HashMap;
 use std::fmt;
 use std::io::Write;
-use std::time::Duration;
 
 use hm_plugin_protocol::ir::DurationMs;
 
@@ -156,7 +155,7 @@ impl<W: Write> ProgressRenderer<W> {
                 Some(StepOutcome::Succeeded { duration_ms }) => (
                     styled("✓", Style::new().green(), self.color),
                     styled(
-                        &Duration::from(*duration_ms).compact().to_string(),
+                        &duration_ms.compact().to_string(),
                         Style::new().dimmed(),
                         self.color,
                     ),
@@ -169,7 +168,7 @@ impl<W: Write> ProgressRenderer<W> {
                     styled(
                         &format!(
                             "{}  exit {exit_code}",
-                            Duration::from(*duration_ms).compact()
+                            duration_ms.compact()
                         ),
                         Style::new().red(),
                         self.color,
@@ -180,7 +179,7 @@ impl<W: Write> ProgressRenderer<W> {
                     styled(
                         &format!(
                             "{}  cancelled",
-                            Duration::from(*duration_ms).compact()
+                            duration_ms.compact()
                         ),
                         Style::new().dimmed(),
                         self.color,
@@ -306,7 +305,7 @@ where
                     }
                 } else if let Some(span) = self.step_spans.get(step_id) {
                     let name = self.step_names.get(step_id).map_or("?", String::as_str);
-                    let dur = Duration::from(*duration_ms).compact();
+                    let dur = duration_ms.compact();
                     span.pb_set_style(&completed_style(self.color));
                     span.pb_set_message(&format!("{name}  ({dur})"));
                 }
@@ -351,15 +350,15 @@ where
 
                 if *exit_code != 0 {
                     self.print_failure_report();
-                    let dur = Duration::from(*duration_ms).compact();
+                    let dur = duration_ms.compact();
                     let msg = format!("✗ Build failed in {dur}");
                     let _ = writeln!(
                         self.out,
                         "\n{}",
                         styled(&msg, Style::new().red().bold(), self.color)
                     );
                 } else {
-                    let dur = Duration::from(*duration_ms).compact();
+                    let dur = duration_ms.compact();
                     let msg = format!("✓ Build succeeded in {dur}");
                     let _ = writeln!(
                         self.out,