refactor: allow depth and base commit simultaneously

Author: cx-rui-oliveira

plugins/git.go

@@ -303,10 +303,10 @@ func (p *GitPlugin) buildScanOptions() string {
 		options = append(options, "--all")
 	}
 
-	// If base commit is specified, use commit range instead of depth
 	if p.baseCommit != "" {
 		options = append(options, fmt.Sprintf("%s..HEAD", p.baseCommit))
-	} else if p.depth > 0 {
+	}
+	if p.depth > 0 {
 		options = append(options, fmt.Sprintf("-n %d", p.depth))
 	}
 

plugins/git_test.go

@@ -71,18 +71,18 @@ func TestBuildScanOptions(t *testing.T) {
 			expectedOptions: "--full-history --all def456..HEAD",
 		},
 		{
-			name:            "Base commit takes precedence over depth",
+			name:            "Base commit with depth: both flags are used",
 			scanAllBranches: false,
 			depth:           10,
 			baseCommit:      "ghi789",
-			expectedOptions: "--full-history ghi789..HEAD",
+			expectedOptions: "--full-history ghi789..HEAD -n 10",
 		},
 		{
-			name:            "Base commit with all branches takes precedence over depth",
+			name:            "Base commit with all branches and depth: all flags are used",
 			scanAllBranches: true,
 			depth:           15,
 			baseCommit:      "jkl012",
-			expectedOptions: "--full-history --all jkl012..HEAD",
+			expectedOptions: "--full-history --all jkl012..HEAD -n 15",
 		},
 	}