images: Use indicatif::BinaryBytes for size formatting

Replace custom format_size() function with indicatif::BinaryBytes
for consistent binary size formatting across the codebase.

Also make DiskImageMetadata::compute_cache_hash() public for use
by base_disks module.

Signed-off-by: Claude <[email protected]>
Signed-off-by: Colin Walters <[email protected]>

Author: cgwalters

crates/kit/src/cache_metadata.rs

@@ -70,7 +70,7 @@ pub struct DiskImageMetadata {
 
 impl DiskImageMetadata {
     /// Generate SHA256 hash of all build inputs
-    fn compute_cache_hash(&self) -> String {
+    pub fn compute_cache_hash(&self) -> String {
         let inputs = CacheInputs {
             image_digest: self.digest.clone(),
             filesystem: self.filesystem.clone(),

crates/kit/src/images.rs

@@ -69,7 +69,7 @@ impl ImagesOpts {
                             .map(|dt| format_relative_time(dt))
                             .unwrap_or_else(|| "N/A".to_string());
 
-                        let size = format_size(image.size);
+                        let size = indicatif::BinaryBytes(image.size).to_string();
 
                         table.add_row(vec![repository, tag, id.to_string(), created, size]);
                     }
@@ -116,24 +116,6 @@ pub struct ImageInspect {
     pub created: Option<chrono::DateTime<chrono::Utc>>,
 }
 
-/// Format bytes into human-readable size string.
-fn format_size(bytes: u64) -> String {
-    const UNITS: &[&str] = &["B", "KB", "MB", "GB", "TB"];
-    let mut size = bytes as f64;
-    let mut unit_idx = 0;
-
-    while size >= 1024.0 && unit_idx < UNITS.len() - 1 {
-        size /= 1024.0;
-        unit_idx += 1;
-    }
-
-    if unit_idx == 0 {
-        format!("{} {}", size as u64, UNITS[unit_idx])
-    } else {
-        format!("{:.1} {}", size, UNITS[unit_idx])
-    }
-}
-
 /// Format a datetime as relative time (e.g., "2 hours ago", "3 days ago").
 fn format_relative_time(dt: chrono::DateTime<chrono::Utc>) -> String {
     let now = chrono::Utc::now();