Feat/overwrite category via cmdline (#18)

* feat(validate): Overwrite category

* fix(validate): Recursive mode with overwrite includes all dirs instead of filtering

Author: luckylittle

cmd/validate.go

@@ -9,10 +9,11 @@ import (
 )
 
 var (
-	validatePresetPath string
-	validateVerbose   bool
-	validateQuiet     bool
-	validateRecursive bool
+	validatePresetPath       string
+	validateVerbose          bool
+	validateQuiet            bool
+	validateRecursive        bool
+	validateOverwriteCategory string
 )
 
 var validateCmd = &cobra.Command{
@@ -26,6 +27,9 @@ the folder contents against the rules defined in the preset configuration file.
 When the recursive option (-r) is used, the command will search for valid
 release folders in all subdirectories of the specified folder(s).
 
+The --overwrite flag allows you to bypass automatic category detection and
+manually specify a category for validation.
+
 Examples:
   # Validate a single folder
   sfvbrr validate /path/to/release
@@ -34,14 +38,18 @@ Examples:
   sfvbrr validate /path/to/release1 /path/to/release2
 
   # Validate recursively
-  sfvbrr validate -r /path/to/releases`,
+  sfvbrr validate -r /path/to/releases
+
+  # Override category detection
+  sfvbrr validate --overwrite app /path/to/release`,
 	Args: cobra.MinimumNArgs(1),
 	Run: func(cmd *cobra.Command, args []string) {
 		opts := validate.Options{
-			PresetPath: validatePresetPath,
-			Verbose:    validateVerbose,
-			Quiet:      validateQuiet,
-			Recursive:  validateRecursive,
+			PresetPath:       validatePresetPath,
+			Verbose:          validateVerbose,
+			Quiet:            validateQuiet,
+			Recursive:        validateRecursive,
+			OverwriteCategory: validateOverwriteCategory,
 		}
 
 		if err := validate.ValidateFolders(args, opts); err != nil {
@@ -58,4 +66,5 @@ func init() {
 	validateCmd.Flags().BoolVarP(&validateVerbose, "verbose", "v", false, "Show detailed validation results for each rule")
 	validateCmd.Flags().BoolVarP(&validateQuiet, "quiet", "q", false, "Quiet mode - only show errors")
 	validateCmd.Flags().BoolVarP(&validateRecursive, "recursive", "r", false, "Recursively search for release folders in subdirectories")
+	validateCmd.Flags().StringVar(&validateOverwriteCategory, "overwrite", "", "Override category detection with specified category (bypasses automatic detection)")
 }

internal/validate/batch.go

@@ -10,7 +10,9 @@ import (
 
 // FindFoldersRecursive finds all folders recursively in the given directory
 // that can be validated (i.e., have a detectable category)
-func FindFoldersRecursive(dir string) ([]string, error) {
+// Note: overwriteCategory parameter is kept for API compatibility but not used here.
+// The overwrite category is applied during validation, not during folder discovery.
+func FindFoldersRecursive(dir string, overwriteCategory string) ([]string, error) {
 	var folders []string
 
 	err := filepath.Walk(dir, func(path string, info os.FileInfo, err error) error {
@@ -20,8 +22,9 @@ func FindFoldersRecursive(dir string) ([]string, error) {
 		}
 
 		if info.IsDir() {
-			// Try to detect category for this folder
-			category, err := DetectCategory(path)
+			// Always try to detect category for this folder to filter for valid releases
+			// The overwrite category will be applied later during validation
+			category, err := DetectCategory(path, "")
 			if err == nil && category != "" {
 				folders = append(folders, path)
 			}
@@ -39,8 +42,8 @@ func FindFoldersRecursive(dir string) ([]string, error) {
 
 // validateSingleFolder validates a single folder and displays results
 func validateSingleFolder(folderPath string, presetConfig *preset.PresetConfig, opts Options) (bool, error) {
-	// Detect category
-	category, err := DetectCategory(folderPath)
+	// Detect category (or use overwrite if provided)
+	category, err := DetectCategory(folderPath, opts.OverwriteCategory)
 	if err != nil {
 		return false, fmt.Errorf("failed to detect category for %s: %w", folderPath, err)
 	}
@@ -72,6 +75,13 @@ func ValidateFolders(folders []string, opts Options) error {
 		return fmt.Errorf("failed to load presets: %w", err)
 	}
 
+	// Validate overwrite category if provided
+	if opts.OverwriteCategory != "" {
+		if _, exists := presetConfig.Rules[opts.OverwriteCategory]; !exists {
+			return fmt.Errorf("invalid category '%s': category not found in preset configuration", opts.OverwriteCategory)
+		}
+	}
+
 	var hasErrors bool
 
 	for _, folder := range folders {
@@ -99,7 +109,7 @@ func ValidateFolders(folders []string, opts Options) error {
 
 		if opts.Recursive {
 			// Find all folders recursively
-			subFolders, err := FindFoldersRecursive(absPath)
+			subFolders, err := FindFoldersRecursive(absPath, opts.OverwriteCategory)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "Error: failed to find folders recursively in %s: %v\n", folder, err)
 				hasErrors = true

internal/validate/categories.go

@@ -8,7 +8,13 @@ import (
 )
 
 // DetectCategory detects the release category from a folder path
-func DetectCategory(folderPath string) (string, error) {
+// If overwriteCategory is provided and non-empty, it will be returned instead of detecting
+func DetectCategory(folderPath string, overwriteCategory string) (string, error) {
+	// If overwrite category is provided, use it directly
+	if overwriteCategory != "" {
+		return overwriteCategory, nil
+	}
+
 	// Extract folder name from path
 	folderName := filepath.Base(folderPath)
 

internal/validate/types.go

@@ -21,19 +21,21 @@ type ValidationResult struct {
 
 // Options contains configuration options for validation
 type Options struct {
-	PresetPath string // Path to preset YAML file (empty = auto-detect)
-	Verbose    bool   // Verbose output
-	Quiet      bool   // Quiet mode (minimal output)
-	Recursive  bool   // Recursive mode - search subdirectories
+	PresetPath       string // Path to preset YAML file (empty = auto-detect)
+	Verbose          bool   // Verbose output
+	Quiet            bool   // Quiet mode (minimal output)
+	Recursive        bool   // Recursive mode - search subdirectories
+	OverwriteCategory string // Override category detection (empty = use auto-detection)
 }
 
 // DefaultOptions returns default options for validation
 func DefaultOptions() Options {
 	return Options{
-		PresetPath: "",
-		Verbose:    false,
-		Quiet:      false,
-		Recursive:  false,
+		PresetPath:       "",
+		Verbose:          false,
+		Quiet:            false,
+		Recursive:        false,
+		OverwriteCategory: "",
 	}
 }