From 1d27f69f177828841944e2be2acd1d7427c6ec06 Mon Sep 17 00:00:00 2001
From: juliankang4 <200967947+juliankang4@users.noreply.github.com>
Date: Sat, 17 Oct 2026 06:31:13 +0900
Subject: [PATCH] refset: a target under a symlinked directory names its
 directory once

---
 crates/refset/src/arch/draft.rs | 33 +++++++++++++++++++++++++++++++--
 1 file changed, 31 insertions(+), 2 deletions(-)

diff --git a/crates/refset/src/arch/draft.rs b/crates/refset/src/arch/draft.rs
index 8c820a3a8..8c3efa7e0 100644
--- a/crates/refset/src/arch/draft.rs
+++ b/crates/refset/src/arch/draft.rs
@@ -239,9 +239,10 @@ fn beside_paths<'a>(path: &'static str, target: &'a Path) -> impl Iterator<Item
     let first = unresolved.and_then(|dir| name.map(|n| dir.join(n)));
     let resolved = std::iter::once_with(move || {
         let resolved = target.canonicalize().ok()?;
+        let same = unresolved.and_then(|dir| dir.join(".").canonicalize().ok());
         resolved
             .parent()
-            .filter(|dir| Some(*dir) != unresolved)
+            .filter(|dir| Some(*dir) != same.as_deref())
             .and_then(|dir| name.map(|n| dir.join(n)))
     })
     .flatten();
@@ -532,10 +533,38 @@ mod tests {
             ROW.pick(None, &target),
             (PathBuf::from(ROW_PATH), DraftFrom::Family)
         );
-        let beside = s.real_dir("only").join(ROW_NAME);
+        let beside = target.with_file_name(ROW_NAME);
         assert_eq!(ROW.places(&target), vec![beside, PathBuf::from(ROW_PATH)]);
     }
 
+    /// A plain shard opened through a symlinked directory sits in one
+    /// directory however it is spelled, so the off line names it once.
+    #[cfg(unix)]
+    #[test]
+    fn a_target_under_a_symlinked_directory_names_its_directory_once() {
+        let s = Scratch::new("dir-link");
+        s.file("real", "shard-00001.gguf");
+        let dir = s.0.join("dir");
+        std::os::unix::fs::symlink(s.real_dir("real"), &dir)
+            .unwrap_or_else(|e| panic!("{}: {e}", dir.display()));
+        let target = dir.join("shard-00001.gguf");
+        assert_eq!(
+            ROW.places(&target),
+            vec![dir.join(ROW_NAME), PathBuf::from(ROW_PATH)]
+        );
+    }
+
+    /// A bare file name sits in the working directory, which `cargo test`
+    /// sets to the package root, so the off line names that directory once.
+    #[test]
+    fn a_bare_file_name_names_the_working_directory_once() {
+        let target = Path::new("Cargo.toml");
+        assert_eq!(
+            ROW.places(target),
+            vec![PathBuf::from(ROW_NAME), PathBuf::from(ROW_PATH)]
+        );
+    }
+
     /// The beside path is the family's path when the target already sits in
     /// that directory, so the off line names it once.
     #[test]