test(sessions): retry unknown membership on second lookup

ScriptedAlchemy · ScriptedAlchemy · commit b9c066a9b920 · 2026-08-03T07:33:52.000Z
diff --git a/src/sessions/claude.rs b/src/sessions/claude.rs
@@ -1379,7 +1379,7 @@ mod tests {
             .ancestors()
             .find(|ancestor| ancestor.file_name().is_some_and(|name| name == "repo"))
             .unwrap_or(path);
-        if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 0 {
+        if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 1 {
             return crate::worktree::GitRepoIdentityOutcome::Unknown;
         }
         crate::worktree::GitRepoIdentityOutcome::Resolved(crate::worktree::GitRepoIdentity {
diff --git a/src/sessions/codex.rs b/src/sessions/codex.rs
@@ -1623,7 +1623,7 @@ mod source_matcher_cache_tests {
             .ancestors()
             .find(|ancestor| ancestor.file_name().is_some_and(|name| name == "repo"))
             .unwrap_or(path);
-        if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 0 {
+        if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 1 {
             return crate::worktree::GitRepoIdentityOutcome::Unknown;
         }
         crate::worktree::GitRepoIdentityOutcome::Resolved(crate::worktree::GitRepoIdentity {

Original file line number	Diff line number	Diff line change
`@@ -1379,7 +1379,7 @@ mod tests {`
`1379`	`1379`	`.ancestors()`
`1380`	`1380`	`.find(\|ancestor\| ancestor.file_name().is_some_and(\|name\| name == "repo"))`
`1381`	`1381`	`.unwrap_or(path);`
`1382`		`- if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 0 {`
	`1382`	`+ if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 1 {`
`1383`	`1383`	`return crate::worktree::GitRepoIdentityOutcome::Unknown;`
`1384`	`1384`	`}`
`1385`	`1385`	`crate::worktree::GitRepoIdentityOutcome::Resolved(crate::worktree::GitRepoIdentity {`
Original file line number	Diff line number	Diff line change
`@@ -1623,7 +1623,7 @@ mod source_matcher_cache_tests {`
`1623`	`1623`	`.ancestors()`
`1624`	`1624`	`.find(\|ancestor\| ancestor.file_name().is_some_and(\|name\| name == "repo"))`
`1625`	`1625`	`.unwrap_or(path);`
`1626`		`- if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 0 {`
	`1626`	`+ if UNKNOWN_PATH_ATTEMPTS.fetch_add(1, Ordering::SeqCst) == 1 {`
`1627`	`1627`	`return crate::worktree::GitRepoIdentityOutcome::Unknown;`
`1628`	`1628`	`}`
`1629`	`1629`	`crate::worktree::GitRepoIdentityOutcome::Resolved(crate::worktree::GitRepoIdentity {`