fix(redirect): never pin a manifest under target/

Member discovery skipped cargo's build directory only while expanding
a glob: a literal `members = ["target/gen"]` entry, or a path
dependency pointing into target/, was returned and pinned — which the
key predicate's own doc comment said could not happen. The next
`cargo clean` deletes that manifest, and a recorded hosted edit whose
file no longer exists refuses the rollback of every other cargo edit.

A `target` segment is now excluded wherever a member path is accepted,
in discovery and in the key predicate both.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: mikolalysenko

crates/socket-patch-core/src/patch/redirect/mod.rs

@@ -1094,9 +1094,9 @@ fn is_cargo_member_manifest_key(key: &str) -> bool {
         && !key.starts_with('/')
         && !key.contains('\\')
         && !key.contains(':')
-        && dir
-            .split('/')
-            .all(|seg| !seg.is_empty() && seg != "." && seg != ".." && seg != ".socket")
+        && dir.split('/').all(|seg| {
+            !seg.is_empty() && seg != "." && seg != ".." && seg != ".socket" && seg != "target"
+        })
 }
 
 /// The not-declared warning for a crate no manifest names at the patched
@@ -9502,6 +9502,8 @@ mod tests {
             "a/../b/Cargo.toml",
             "./a/Cargo.toml",
             ".socket/vendor/cargo/x/Cargo.toml",
+            "target/generated/Cargo.toml",
+            "crates/a/target/gen/Cargo.toml",
             "a//Cargo.toml",
             "a/Cargo.toml.orig",
         ] {

crates/socket-patch-core/src/utils/cargo_workspace.rs

@@ -98,9 +98,15 @@ fn enqueue(
     dirs: &mut BTreeSet<String>,
     queue: &mut Vec<(String, DocumentMut)>,
 ) {
+    // `target` is cargo's build directory at every level: the glob walk
+    // already skips it, and a literal `members = ["target/gen"]` or a path
+    // dependency into it names a manifest the next `cargo clean` deletes —
+    // pinning it would record a hosted edit whose file can vanish, blocking
+    // the rollback of every other cargo edit.
     if dir.is_empty()
         || dirs.len() >= MAX_MANIFESTS
         || dirs.contains(&dir)
+        || dir.split('/').any(|seg| seg == "target")
         || !is_real_dir_path(root, &dir)
     {
         return;
@@ -160,7 +166,7 @@ fn path_dependencies(doc: &DocumentMut) -> Vec<String> {
 
 /// `base/rel` lexically normalized to a repo-relative slash path; `None`
 /// when it is absolute or climbs out of the root.
-fn normalize_rel(base: &str, rel: &str) -> Option<String> {
+pub(crate) fn normalize_rel(base: &str, rel: &str) -> Option<String> {
     let rel = rel.replace('\\', "/");
     if rel.starts_with('/') || Path::new(&rel).is_absolute() {
         return None;
@@ -389,6 +395,32 @@ mod tests {
         assert!(member_manifests(&root).is_empty());
     }
 
+    /// Cargo's build directory is never a member, whichever way it is
+    /// named: the glob walk skips it, and so do a literal `members` entry
+    /// and a path dependency pointing into it. `cargo clean` deletes those
+    /// manifests, and a recorded hosted edit for a file that no longer
+    /// exists blocks the rollback of every other cargo edit.
+    #[test]
+    fn build_directory_manifests_are_never_members() {
+        let tmp = tempfile::tempdir().unwrap();
+        let root = tmp.path();
+        write(
+            root,
+            "Cargo.toml",
+            &format!(
+                "[workspace]\nmembers = [\"crates/*\", \"target/generated\"]\n\n\
+                 {}[dependencies]\ngen2 = {{ path = \"target/gen2\" }}\n\
+                 nested = {{ path = \"crates/a/target/gen3\" }}\n",
+                pkg("root")
+            ),
+        );
+        write(root, "crates/a/Cargo.toml", &pkg("a"));
+        write(root, "crates/a/target/gen3/Cargo.toml", &pkg("gen3"));
+        write(root, "target/generated/Cargo.toml", &pkg("generated"));
+        write(root, "target/gen2/Cargo.toml", &pkg("gen2"));
+        assert_eq!(member_manifests(root), vec!["crates/a/Cargo.toml"]);
+    }
+
     #[test]
     fn wildcard_matching() {
         assert!(wildcard_match(b"a*c", b"abbc"));