refactor: move path_components construction into maybe_color_slice

Defer the path_components Vec allocation to inside maybe_color_slice,
so it only happens when coloring is actually enabled. This avoids
unnecessary per-row allocation in the common no-color case.

Also removes the pipe_trait::Pipe dependency from methods.rs.

https://claude.ai/code/session_012139SmUSC1oYSKNkvsg78g

Author: claude

src/visualizer/coloring.rs

@@ -110,14 +110,21 @@ impl Width for ColoredTreeHorizontalSlice<'_> {
 }
 
 /// Wrap a [`TreeHorizontalSlice`] with color if coloring is available, otherwise return it as-is.
+///
+/// Path components are only constructed when coloring is enabled, avoiding
+/// unnecessary allocation in the common no-color case.
 pub(super) fn maybe_color_slice<'a, 'b>(
     coloring: Option<&'b Coloring<'a>>,
-    path_components: &[&'a OsStr],
+    ancestors: impl Iterator<Item = &'a OsStr>,
+    name: &'a OsStr,
     has_children: bool,
     slice: TreeHorizontalSlice<String>,
 ) -> MaybeColoredTreeHorizontalSlice<'b> {
     match coloring {
-        Some(coloring) => coloring.maybe_color_tree_slice(path_components, has_children, slice),
+        Some(coloring) => {
+            let path_components: Vec<&OsStr> = ancestors.chain(std::iter::once(name)).collect();
+            coloring.maybe_color_tree_slice(&path_components, has_children, slice)
+        }
         None => MaybeColoredTreeHorizontalSlice::Colorless(slice),
     }
 }

src/visualizer/methods.rs

@@ -14,8 +14,7 @@ use tree_table::*;
 
 use super::{coloring::maybe_color_slice, ColumnWidthDistribution, Visualizer};
 use crate::size;
-use pipe_trait::Pipe;
-use std::{cmp::min, ffi::OsStr, fmt::Display, iter::once};
+use std::{cmp::min, ffi::OsStr, fmt::Display};
 use zero_copy_pads::{align_left, align_right};
 
 impl<'a, Name, Size> Visualizer<'a, Name, Size>
@@ -92,15 +91,10 @@ where
                     tree_horizontal_slice,
                 } = tree_row;
 
-                let path_components: Vec<&OsStr> = initial_row
-                    .ancestors
-                    .iter()
-                    .map(|node| node.name.as_ref())
-                    .chain(initial_row.node_info.name.pipe_as_ref(once))
-                    .collect();
                 let tree = maybe_color_slice(
                     self.coloring,
-                    &path_components,
+                    initial_row.ancestors.iter().map(|node| node.name.as_ref()),
+                    initial_row.node_info.name.as_ref(),
                     initial_row.node_info.children_count > 0,
                     tree_horizontal_slice,
                 );