feat(logging): add log-level flag and debug logging with test and linter fixes

- Add `--log-level` flag to root.go with debug, info, warn, error levels
- Implement log level handling in root.go and cli.go with type assertion checks
- Enhance logger.go with custom `zap.Config` and ISO8601 timestamps
- Add Debugf logging in fs.go `RemoveBinary` for constructed binary path
- Update mock files to include changes and accommodate test updates
- Add .gitattributes file to manage line endings
- Upgrade `wsl` linter to `wsl_v5`
- Update mockery configuration for mockery version 3.3.2

Author: nicholas-fedor

.gitattributes

@@ -0,0 +1,43 @@
+# Normalize all text files to LF line endings
+* text=auto eol=lf
+
+# Explicitly set Go-related text files to LF
+*.go text eol=lf
+*.mod text eol=lf
+*.sum text eol=lf
+*.work text eol=lf
+*.md text eol=lf
+*.txt text eol=lf
+*.yaml text eol=lf
+*.yml text eol=lf
+*.json text eol=lf
+*.proto text eol=lf
+
+# Treat configuration and script files as text with LF
+*.toml text eol=lf
+*.ini text eol=lf
+*.sh text eol=lf
+Dockerfile text eol=lf
+*.dockerignore text eol=lf
+*.gitignore text eol=lf
+*.gitattributes text eol=lf
+
+# Treat binary files as binary (no line-ending conversion or diff)
+*.png binary
+*.jpg binary
+*.jpeg binary
+*.gif binary
+*.ico binary
+*.pdf binary
+*.zip binary
+*.tar binary
+*.gz binary
+*.exe binary
+*.dll binary
+*.so binary
+*.db binary
+*.sqlite binary
+
+# Ensure generated Go binaries or test caches are treated as binary
+*.out binary
+*.test binary

.golangci.yaml

@@ -50,7 +50,7 @@ linters:
     - usestdlibvars # A linter that detect the possibility to use variables/constants from the Go standard library.
     - usetesting # Reports uses of functions with replacement inside the testing package.
     - whitespace # Whitespace is a linter that checks for unnecessary newlines at the start and end of functions, if, for, etc.
-    - wsl # Add or remove empty lines.
+    - wsl_v5 # Add or remove empty lines.
     ##################################################################################################
     # Enabled linters that require manual issue resolution
     - asasalint # Check for pass []any as any in variadic func(...any).

.mockery.yaml

@@ -5,15 +5,15 @@
 # go-remove: https://github.com/nicholas-fedor/go-remove/                                            #
 # Mockery: https://vektra.github.io/mockery/latest/                                                  #
 # Configuration: https://vektra.github.io/mockery/latest/configuration                               #
-# Mockery Version: 2.53                                                                              #
+# Mockery Version: 3.3.2                                                                              #
 #                                                                                                    #
 ######################################################################################################
 
 ######################################################################################################
 # Mockery Usage                                                                                      #
 #                                                                                                    #
 # Install Mockery:                                                                                   #
-# go install github.com/vektra/mockery/v2@v2.53.3                                                     #
+# go install github.com/vektra/mockery/v3@v3.3.2                                                     #
 #                                                                                                    #
 # Run CLI command from root:                                                                         #
 # mockery                                                                                            #
@@ -65,7 +65,7 @@
 # Disable all warnings for deprecated behavior.
 # When true, suppresses deprecation messages (e.g., for quiet, with-expecter in v3).
 # Default: false
-disable-deprecation-warnings: true
+# disable-deprecation-warnings: true
 
 # Selectively disable specific deprecation warnings by name.
 # Use names from deprecation-name in log messages (e.g., "quiet", "with-expecter").
@@ -142,7 +142,7 @@ recursive: true
 # Generate testify expecter methods (e.g., EXPECT()) for mocks when true.
 # Enhances compatibility with github.com/stretchr/testify; v3 deprecation: permanently enabled.
 # Default: false
-with-expecter: true
+# with-expecter: true
 
 # Specify build tags for the generated mock files.
 # Matches Go’s build constraint syntax (e.g., "// +build tag1,tag2").
@@ -157,7 +157,7 @@ with-expecter: true
 
 # Package name for generated mock files.
 # Default: "{{.PackageName}}"
-outpkg: mocks
+pkgname: "mocks"
 
 # Treat variadic parameters as a single slice when false, rather than unrolling them.
 # Affects how variadic methods are mocked (e.g., func(...int) vs. func([]int)).

cmd/root.go

@@ -19,16 +19,22 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
 package cmd
 
 import (
+	"errors"
 	"fmt"
 	"os"
 
 	"github.com/spf13/cobra"
+	"go.uber.org/zap"
+	"go.uber.org/zap/zapcore"
 
 	"github.com/nicholas-fedor/go-remove/internal/cli"
 	"github.com/nicholas-fedor/go-remove/internal/fs"
 	"github.com/nicholas-fedor/go-remove/internal/logger"
 )
 
+// ErrInvalidLoggerType indicates that the logger is not of the expected *ZapLogger type.
+var ErrInvalidLoggerType = errors.New("logger is not a *ZapLogger")
+
 // rootCmd defines the root command for go-remove.
 var rootCmd = &cobra.Command{
 	Use:   "go-remove [binary]",
@@ -49,11 +55,45 @@ var rootCmd = &cobra.Command{
 		// Extract flag values to configure CLI behavior; defaults to TUI mode if no binary is given.
 		verbose, _ := cmd.Flags().GetBool("verbose")
 		goroot, _ := cmd.Flags().GetBool("goroot")
+		logLevel, _ := cmd.Flags().GetString("log-level")
 		config := cli.Config{
-			Binary:  "",
-			Verbose: verbose,
-			Goroot:  goroot,
-			Help:    false, // Cobra manages help output automatically
+			Binary:   "",
+			Verbose:  verbose,
+			Goroot:   goroot,
+			Help:     false, // Cobra manages help output automatically
+			LogLevel: logLevel,
+		}
+
+		// Set log level based on config if verbose mode is enabled.
+		if verbose {
+			zapLogger, ok := log.(*logger.ZapLogger)
+			if !ok {
+				return fmt.Errorf(
+					"failed to set log level: %w with type %T",
+					ErrInvalidLoggerType,
+					log,
+				)
+			}
+			switch logLevel {
+			case "debug":
+				zapLogger.Logger = zapLogger.WithOptions(
+					zap.IncreaseLevel(zapcore.DebugLevel),
+				)
+			case "warn":
+				zapLogger.Logger = zapLogger.WithOptions(
+					zap.IncreaseLevel(zapcore.WarnLevel),
+				)
+			case "error":
+				zapLogger.Logger = zapLogger.WithOptions(
+					zap.IncreaseLevel(zapcore.ErrorLevel),
+				)
+			default:
+				zapLogger.Logger = zapLogger.WithOptions(
+					zap.IncreaseLevel(zapcore.InfoLevel),
+				)
+			}
+			log = zapLogger // Update the logger in deps
+			deps.Logger = log
 		}
 
 		// If a binary name is provided as an argument, run in direct removal mode.
@@ -77,6 +117,7 @@ var rootCmd = &cobra.Command{
 func init() {
 	rootCmd.Flags().BoolP("verbose", "v", false, "Enable verbose output")
 	rootCmd.Flags().BoolP("goroot", "", false, "Target GOROOT/bin instead of GOBIN or GOPATH/bin")
+	rootCmd.Flags().StringP("log-level", "l", "info", "Set log level (debug, info, warn, error)")
 }
 
 // Execute runs the root command and handles any execution errors.

cmd/root_test.go

@@ -36,7 +36,7 @@ func TestRootCommand(t *testing.T) {
 		{
 			name:       "help flag",
 			args:       []string{"-h"},
-			wantStderr: "A tool to remove Go binaries\n\nUsage:\n  go-remove [binary] [flags]\n\nFlags:\n      --goroot    Target GOROOT/bin instead of GOBIN or GOPATH/bin\n  -h, --help      help for go-remove\n  -v, --verbose   Enable verbose output\n",
+			wantStderr: "A tool to remove Go binaries\n\nUsage:\n  go-remove [binary] [flags]\n\nFlags:\n      --goroot             Target GOROOT/bin instead of GOBIN or GOPATH/bin\n  -h, --help               help for go-remove\n  -l, --log-level string   Set log level (debug, info, warn, error) (default \"info\")\n  -v, --verbose            Enable verbose output\n",
 			wantErr:    false,
 		},
 	}
@@ -65,7 +65,6 @@ func TestRootCommand(t *testing.T) {
 			w.Close()
 
 			var buf bytes.Buffer
-
 			buf.ReadFrom(r)
 			gotStderr := buf.String()
 

internal/cli/cli.go

@@ -19,18 +19,27 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
 package cli
 
 import (
+	"errors"
 	"fmt"
+	"os"
+
+	"go.uber.org/zap"
+	"go.uber.org/zap/zapcore"
 
 	"github.com/nicholas-fedor/go-remove/internal/fs"
 	"github.com/nicholas-fedor/go-remove/internal/logger"
 )
 
+// ErrInvalidLoggerType indicates that the logger is not of the expected *ZapLogger type.
+var ErrInvalidLoggerType = errors.New("logger is not a *ZapLogger")
+
 // Config holds command-line configuration options.
 type Config struct {
-	Binary  string // Binary name to remove; empty for TUI mode
-	Verbose bool   // Enable verbose logging
-	Goroot  bool   // Use GOROOT/bin instead of GOBIN or GOPATH/bin
-	Help    bool   // Show help; managed by Cobra
+	Binary   string // Binary name to remove; empty for TUI mode
+	Verbose  bool   // Enable verbose logging
+	Goroot   bool   // Use GOROOT/bin instead of GOBIN or GOPATH/bin
+	Help     bool   // Show help; managed by Cobra
+	LogLevel string // Log level (debug, info, warn, error)
 }
 
 // Dependencies holds runtime dependencies for CLI execution.
@@ -43,10 +52,31 @@ type Dependencies struct {
 func Run(deps Dependencies, config Config) error {
 	log := deps.Logger
 
+	// Set log level based on config if verbose mode is enabled.
+	if config.Verbose {
+		zapLogger, ok := log.(*logger.ZapLogger)
+		if !ok {
+			return fmt.Errorf("failed to set log level: %w with type %T", ErrInvalidLoggerType, log)
+		}
+
+		switch config.LogLevel {
+		case "debug":
+			zapLogger.Logger = zapLogger.WithOptions(zap.IncreaseLevel(zapcore.DebugLevel))
+		case "warn":
+			zapLogger.Logger = zapLogger.WithOptions(zap.IncreaseLevel(zapcore.WarnLevel))
+		case "error":
+			zapLogger.Logger = zapLogger.WithOptions(zap.IncreaseLevel(zapcore.ErrorLevel))
+		default:
+			zapLogger.Logger = zapLogger.WithOptions(zap.IncreaseLevel(zapcore.InfoLevel))
+		}
+
+		log = zapLogger // Update the logger for subsequent use
+	}
+
 	// Determine the binary directory based on GOROOT or GOPATH/GOBIN settings.
 	binDir, err := deps.FS.DetermineBinDir(config.Goroot)
 	if err != nil {
-		_ = log.Sync() // Ensure logs are flushed despite the error
+		_ = log.Sync() // Flush logs; errors are ignored
 
 		return fmt.Errorf("failed to determine binary directory: %w", err)
 	}
@@ -56,11 +86,16 @@ func Run(deps Dependencies, config Config) error {
 		err = RunTUI(binDir, config, log, deps.FS, DefaultRunner{})
 	} else {
 		binaryPath := deps.FS.AdjustBinaryPath(binDir, config.Binary)
+
 		err = deps.FS.RemoveBinary(binaryPath, config.Binary, config.Verbose, log)
+		if err == nil && !config.Verbose {
+			// Print success message to stdout for non-verbose mode.
+			fmt.Fprintf(os.Stdout, "Successfully removed %s\n", config.Binary)
+		}
 	}
 
 	if err != nil {
-		_ = log.Sync() // Flush logs before returning the error
+		_ = log.Sync() // Flush logs; errors are ignored
 
 		if config.Binary == "" {
 			return fmt.Errorf("failed to run TUI: %w", err)
@@ -70,9 +105,7 @@ func Run(deps Dependencies, config Config) error {
 	}
 
 	// Sync the logger to ensure all logs are written before exit.
-	if err := log.Sync(); err != nil {
-		return fmt.Errorf("failed to sync logger: %w", err)
-	}
+	_ = log.Sync() // Errors are ignored
 
 	return nil
 }