From c36397031bd230f9c520264b08b87b6b4e5f43eb Mon Sep 17 00:00:00 2001
From: Andrew Gallant <andrew@astral.sh>
Date: Thu, 8 Jan 2026 09:18:42 -0500
Subject: [PATCH] [ty] Actually ignore hidden directories in completion
 evaluation

I apparently don't know how to use my own API. Previously,
we would skip, e.g., `.venv`, but still descend into it.

This was annoying in practice because I sometimes have an
environment in one of the truth task directories. The eval
command should ignore that entirely, but it ended up
choking on it without properly ignoring hidden files
and directories.
---
 crates/ty_completion_eval/src/main.rs | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/crates/ty_completion_eval/src/main.rs b/crates/ty_completion_eval/src/main.rs
index 8379c1ec11..e401cd3912 100644
--- a/crates/ty_completion_eval/src/main.rs
+++ b/crates/ty_completion_eval/src/main.rs
@@ -540,16 +540,17 @@ fn copy_project(src_dir: &SystemPath, dst_dir: &SystemPath) -> anyhow::Result<Ve
     std::fs::create_dir_all(dst_dir).with_context(|| dst_dir.to_string())?;
 
     let mut cursors = vec![];
-    for result in walkdir::WalkDir::new(src_dir.as_std_path()) {
+    let it = walkdir::WalkDir::new(src_dir.as_std_path())
+        .into_iter()
+        .filter_entry(|dent| {
+            !dent
+                .file_name()
+                .to_str()
+                .is_some_and(|name| name.starts_with('.'))
+        });
+    for result in it {
         let dent =
             result.with_context(|| format!("failed to get directory entry from {src_dir}"))?;
-        if dent
-            .file_name()
-            .to_str()
-            .is_some_and(|name| name.starts_with('.'))
-        {
-            continue;
-        }
 
         let src = SystemPath::from_std_path(dent.path()).ok_or_else(|| {
             anyhow::anyhow!("path `{}` is not valid UTF-8", dent.path().display())