fix(test): Fix hunk filtering on MacOS and update covbadge

README.md

@@ -4,7 +4,7 @@ Code Ownership & Review Assignment Tool - GitHub CODEOWNERS but better
 
 [![Go Report Card](https://goreportcard.com/badge/github.com/multimediallc/codeowners-plus)](https://goreportcard.com/report/github.com/multimediallc/codeowners-plus?kill_cache=1)
 [![Tests](https://github.com/multimediallc/codeowners-plus/actions/workflows/go.yml/badge.svg)](https://github.com/multimediallc/codeowners-plus/actions/workflows/go.yml)
-![Coverage](https://img.shields.io/badge/Coverage-83.6%25-brightgreen)
+![Coverage](https://img.shields.io/badge/Coverage-83.7%25-brightgreen)
 [![License](https://img.shields.io/badge/License-BSD%203--Clause-blue.svg)](https://opensource.org/licenses/BSD-3-Clause)
 [![Contributor Covenant](https://img.shields.io/badge/Contributor%20Covenant-2.1-4baaaa.svg)](CODE_OF_CONDUCT.md)
 

internal/app/app.go

@@ -158,11 +158,22 @@ func resolveHookPath(path, workspace string) (string, error) {
 	return resolved, nil
 }
 
+// resolveSymlinks resolves symlinks in the longest existing prefix of path,
+// so a path that does not exist yet still gets the same canonical prefix as
+// its existing parents (e.g. /var -> /private/var on macOS).
 func resolveSymlinks(path string) string {
-	if real, err := filepath.EvalSymlinks(path); err == nil {
-		return real
+	path = filepath.Clean(path)
+	rest := ""
+	for dir := path; ; dir = filepath.Dir(dir) {
+		if real, err := filepath.EvalSymlinks(dir); err == nil {
+			return filepath.Join(real, rest)
+		}
+		parent := filepath.Dir(dir)
+		if parent == dir {
+			return path
+		}
+		rest = filepath.Join(filepath.Base(dir), rest)
 	}
-	return filepath.Clean(path)
 }
 
 func toHookHunks(bodies []string) []hook.Hunk {

internal/app/hunk_filter_test.go

@@ -169,12 +169,18 @@ func TestResolveHookPathFollowsASymlinkIntoTheCheckout(t *testing.T) {
 
 // Local runs have no GITHUB_WORKSPACE, so an absolute path stands on its own.
 func TestResolveHookPathWithoutAWorkspace(t *testing.T) {
-	path := filepath.Join(t.TempDir(), "filter")
+	dir := t.TempDir()
+	path := filepath.Join(dir, "filter")
 	got, err := resolveHookPath(path, "")
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
-	if got != filepath.Clean(path) {
-		t.Errorf("expected %q, got %q", path, got)
+	// The temp dir may sit behind a symlink (/var -> /private/var on macOS).
+	realDir, err := filepath.EvalSymlinks(dir)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if want := filepath.Join(realDir, "filter"); got != want {
+		t.Errorf("expected %q, got %q", want, got)
 	}
 }