traverse: fix git

example/cmd/modifier_test.go

@@ -121,15 +121,6 @@ func TestChdirF(t *testing.T) {
 				Tag("files").
 				Usage("ChdirF()"))
 
-		s.Env("GIT_DIR", "subdirA/") // TODO should also work for subdirA
-		s.Run("modifier", "--chdirf", "").Expect(
-			carapace.ActionValues(
-				"file2.txt",
-			).StyleF(style.ForPath).
-				NoSpace('/').
-				Tag("files").
-				Usage("ChdirF()"))
-
 		s.Env("GIT_WORK_TREE", "subdirB/") // TODO should also work for subdirB
 		s.Run("modifier", "--chdirf", "").Expect(
 			carapace.ActionValues(

pkg/traverse/git.go

@@ -9,14 +9,17 @@ func GitDir(tc Context) (string, error) {
 	if dir, ok := tc.LookupEnv("GIT_DIR"); ok {
 		return filepath.ToSlash(dir), nil
 	}
-	return Parent(".git")(tc)
+	dir, err := GitWorkTree(tc)
+	if err == nil {
+		dir += "/.git"
+	}
+	return dir, err
 }
 
 // GitWorkTree returns the location of the root of the working directory for a non-bare repository.
 func GitWorkTree(tc Context) (string, error) {
 	if dir, ok := tc.LookupEnv("GIT_WORK_TREE"); ok {
 		return filepath.ToSlash(dir), nil
 	}
-
-	return GitDir(tc)
+	return Parent(".git")(tc)
 }

pkg/traverse/traverse.go

@@ -40,7 +40,7 @@ func Parent(names ...string) func(tc Context) (string, error) {
 func traverse(path string, name string) (target string, err error) {
 	var absPath string
 	if absPath, err = filepath.Abs(path); err == nil {
-		target = filepath.ToSlash(absPath + "/" + name)
+		target = filepath.ToSlash(absPath + "/" + strings.TrimSuffix(name, "/"))
 		if _, err = os.Stat(target); err != nil {
 			parent := filepath.Dir(absPath)
 			if parent != path {