refactor: paths filter

Author: roeil

src/GitVersion.Core.Tests/VersionCalculation/PathFilterTests.cs

@@ -36,19 +36,6 @@ public void WhenPathMismatchShouldNotExclude()
         reason.ShouldBeNull();
     }
 
-    [Test]
-    public void WhenCommitSourceStartsWithCommitTagShouldNotBeExcluded()
-    {
-        var commit = GitRepositoryTestingExtensions.CreateMockCommit(["/path"]);
-        BaseVersion version = new("Git tag: v1.0.0", new SemanticVersion(1), commit);
-        var sut = new PathFilter(commit.DiffPaths);
-
-        var result = sut.Exclude(version, out var reason);
-
-        result.ShouldBeFalse();
-        reason.ShouldBeNull();
-    }
-
     [Test]
     public void ExcludeShouldAcceptVersionWithNullCommit()
     {

src/GitVersion.Core/VersionCalculation/PathFilter.cs

@@ -5,7 +5,13 @@
 
 namespace GitVersion.VersionCalculation;
 
-internal class PathFilter(IReadOnlyList<string> paths) : IVersionFilter
+internal enum PathFilterMode
+{
+    Inclusive,  // All commit paths must match for commit to be excluded
+    //Exclusive // Any commit path must match for commit to be excluded
+}
+
+internal class PathFilter(IReadOnlyList<string> paths, PathFilterMode mode = PathFilterMode.Inclusive) : IVersionFilter
 {
     private readonly IReadOnlyList<Regex> pathsRegexes = [.. paths.Select(path => new Regex(path, RegexOptions.Compiled))];
     private readonly ConcurrentDictionary<string, bool> pathMatchCache = [];
@@ -16,25 +22,33 @@ public bool Exclude(IBaseVersion baseVersion, [NotNullWhen(true)] out string? re
         return Exclude(baseVersion.BaseVersionSource, out reason);
     }
 
+    private bool IsMatch(string path)
+    {
+        if (!pathMatchCache.TryGetValue(path, out var isMatch))
+        {
+            isMatch = this.pathsRegexes.Any(regex => regex.IsMatch(path));
+            pathMatchCache[path] = isMatch;
+        }
+        return isMatch;
+    }
+
     public bool Exclude(ICommit? commit, [NotNullWhen(true)] out string? reason)
     {
         reason = null;
 
         if (commit != null)
         {
-            foreach (var path in commit.DiffPaths)
+            switch (mode)
             {
-                if (!pathMatchCache.TryGetValue(path, out var isMatch))
-                {
-                    isMatch = this.pathsRegexes.Any(regex => regex.IsMatch(path));
-                    pathMatchCache[path] = isMatch;
-                }
-
-                if (isMatch)
-                {
-                    reason = "Source was ignored due to commit path matching ignore regex";
-                    return true;
-                }
+                case PathFilterMode.Inclusive:
+                    {
+                        if (commit.DiffPaths.All(this.IsMatch))
+                        {
+                            reason = "Source was ignored due to all commit paths matching ignore regex";
+                            return true;
+                        }
+                        break;
+                    }
             }
         }