fix: properly handle symlinks - show but don't follow (fixes #1)

Author: caffienerd

src/display.rs

@@ -61,7 +61,13 @@ pub fn display_tree(
     // Sort: directories first, then alphabetically
     entries.sort_by_key(|e| {
         let path = e.path();
-        let is_dir = path.is_dir();
+        // Check if it's a symlink pointing to a directory
+        let is_dir = if path.is_symlink() {
+            // Don't follow symlinks to avoid infinite loops
+            false
+        } else {
+            path.is_dir()
+        };
         let name = e.file_name().to_string_lossy().to_lowercase();
         (!is_dir, name)
     });
@@ -72,7 +78,14 @@ pub fn display_tree(
         let is_last_entry = idx == total - 1;
         let path = entry.path();
         let name = entry.file_name().to_string_lossy().to_string();
-        let is_dir = path.is_dir();
+        
+        // Check if it's a symlink first - NEVER recurse into symlinks
+        let is_symlink = path.is_symlink();
+        let is_dir = if is_symlink {
+            false  // Treat symlinks as files to prevent recursion
+        } else {
+            path.is_dir()
+        };
 
         // Check if we should skip this entry
         if is_dir {
@@ -90,6 +103,7 @@ pub fn display_tree(
             if should_skip {
                 // Count files in ignored directory
                 let ignored_count = WalkDir::new(&path)
+                    .follow_links(false)
                     .into_iter()
                     .filter_map(|e| e.ok())
                     .filter(|e| e.file_type().is_file())
@@ -145,7 +159,14 @@ pub fn display_tree(
 
         // Display the entry
         let connector = if is_last_entry { "└── " } else { "├── " };
-        let display_name = if is_dir {
+        let display_name = if is_symlink {
+            // Show symlink with arrow
+            if let Ok(target) = fs::read_link(&path) {
+                format!("{} -> {}", name, target.display()).cyan()
+            } else {
+                name.cyan()
+            }
+        } else if is_dir {
             format!("{}/", name).blue().bold()
         } else if is_executable(&path) {
             name.green().bold()

src/search.rs

@@ -26,6 +26,7 @@ pub fn search_files(pattern: &str, start_path: &Path, max_depth: usize, flat: bo
 
     // Search through all files
     for entry in WalkDir::new(start_path)
+        .follow_links(false)
         .max_depth(max_depth)
         .into_iter()
         .filter_entry(|e| {

src/summary.rs

@@ -65,6 +65,7 @@ pub fn display_summary(path: &Path) {
             // Track ignored items
             if is_dir {
                 let file_count = WalkDir::new(&entry_path)
+                    .follow_links(false)
                     .into_iter()
                     .filter_map(|e| e.ok())
                     .filter(|e| e.file_type().is_file())
@@ -121,6 +122,7 @@ fn display_directory_summary(entry_path: &Path, name: &str, custom_ignores: &[Re
             if is_subdir && (should_ignore_dir(&subname) || matches_custom_pattern(&subname, custom_ignores)) {
                 // Count files in ignored subdir
                 let ignored_count = WalkDir::new(&subpath)
+                    .follow_links(false)
                     .into_iter()
                     .filter_map(|e| e.ok())
                     .filter(|e| e.file_type().is_file())
@@ -132,6 +134,7 @@ fn display_directory_summary(entry_path: &Path, name: &str, custom_ignores: &[Re
 
     // Walk recursively to count visible items (skip ignored directories)
     for sub_entry in WalkDir::new(entry_path)
+        .follow_links(false)
         .into_iter()
         .filter_entry(|e| {
             // Skip ignored directories during traversal
@@ -165,7 +168,10 @@ fn display_directory_summary(entry_path: &Path, name: &str, custom_ignores: &[Re
     }
 
     // Get ALL stats recursively (including everything)
-    for sub_entry in WalkDir::new(entry_path).into_iter().filter_map(|e| e.ok()) {
+    for sub_entry in WalkDir::new(entry_path)
+        .follow_links(false)
+        .into_iter()
+        .filter_map(|e| e.ok()) {
         if sub_entry.file_type().is_file() {
             total_file_count += 1;
             if let Ok(metadata) = sub_entry.metadata() {

src/utils.rs

@@ -45,6 +45,7 @@ pub fn is_executable(path: &Path) -> bool {
 /// Get total size of a directory recursively
 pub fn get_dir_size(path: &Path) -> u64 {
     WalkDir::new(path)
+        .follow_links(false)
         .into_iter()
         .filter_map(|e| e.ok())
         .filter_map(|e| e.metadata().ok())