Skip PersistentPreRun during shell completion commands (#579)

* Fix completion bug: skip logging during shell completion commands

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: frjcomp <107982661+frjcomp@users.noreply.github.com>

Author: Copilot

internal/cmd/root.go

@@ -41,6 +41,9 @@ var (
 		Example: "pipeleek gl scan --token glpat-xxxxxxxxxxx --gitlab https://gitlab.com",
 		Version: getVersion(),
 		PersistentPreRun: func(cmd *cobra.Command, args []string) {
+			if isCompletionCommand(cmd) {
+				return
+			}
 			initLogger(cmd)
 			setGlobalLogLevel(cmd)
 			loadConfigFile(cmd)
@@ -241,6 +244,18 @@ func formatLevelWithHitColor(colorEnabled bool) zerolog.Formatter {
 	}
 }
 
+// isCompletionCommand returns true if the command being executed is a shell
+// completion command. Logging must be suppressed for these commands because
+// any output to stdout will break the generated completion script.
+func isCompletionCommand(cmd *cobra.Command) bool {
+	for c := cmd; c != nil; c = c.Parent() {
+		if c.Name() == "completion" || c.Name() == cobra.ShellCompRequestCmd {
+			return true
+		}
+	}
+	return false
+}
+
 func setGlobalLogLevel(cmd *cobra.Command) {
 	if LogLevel != "" {
 		switch LogLevel {

internal/cmd/root_test.go

@@ -8,6 +8,7 @@ import (
 
 	"github.com/rs/zerolog"
 	"github.com/rs/zerolog/log"
+	"github.com/spf13/cobra"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
@@ -417,3 +418,80 @@ func TestLoadConfigFile_NoConfigFile(t *testing.T) {
 		loadConfigFile(rootCmd)
 	})
 }
+
+func TestIsCompletionCommand(t *testing.T) {
+	tests := []struct {
+		name     string
+		buildCmd func() *cobra.Command
+		expected bool
+	}{
+		{
+			name: "completion bash subcommand",
+			buildCmd: func() *cobra.Command {
+				root := &cobra.Command{Use: "pipeleek"}
+				completion := &cobra.Command{Use: "completion"}
+				bashCmd := &cobra.Command{Use: "bash"}
+				completion.AddCommand(bashCmd)
+				root.AddCommand(completion)
+				return bashCmd
+			},
+			expected: true,
+		},
+		{
+			name: "completion command itself",
+			buildCmd: func() *cobra.Command {
+				root := &cobra.Command{Use: "pipeleek"}
+				completion := &cobra.Command{Use: "completion"}
+				root.AddCommand(completion)
+				return completion
+			},
+			expected: true,
+		},
+		{
+			name: "__complete hidden command",
+			buildCmd: func() *cobra.Command {
+				root := &cobra.Command{Use: "pipeleek"}
+				complete := &cobra.Command{Use: cobra.ShellCompRequestCmd}
+				root.AddCommand(complete)
+				return complete
+			},
+			expected: true,
+		},
+		{
+			name: "regular command",
+			buildCmd: func() *cobra.Command {
+				root := &cobra.Command{Use: "pipeleek"}
+				scan := &cobra.Command{Use: "scan"}
+				root.AddCommand(scan)
+				return scan
+			},
+			expected: false,
+		},
+		{
+			name: "root command",
+			buildCmd: func() *cobra.Command {
+				return &cobra.Command{Use: "pipeleek"}
+			},
+			expected: false,
+		},
+		{
+			name: "nested subcommand not under completion",
+			buildCmd: func() *cobra.Command {
+				root := &cobra.Command{Use: "pipeleek"}
+				gl := &cobra.Command{Use: "gl"}
+				scan := &cobra.Command{Use: "scan"}
+				gl.AddCommand(scan)
+				root.AddCommand(gl)
+				return scan
+			},
+			expected: false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			cmd := tt.buildCmd()
+			assert.Equal(t, tt.expected, isCompletionCommand(cmd))
+		})
+	}
+}