fix: disable pager mouse interactions and add global --pager flag

atmos.yaml

@@ -352,7 +352,7 @@ settings:
   # Terminal settings for displaying content
   terminal:
     max_width: 120          # Maximum width for terminal output
-    pager: true             # Pager setting for all terminal output
+    pager: false            # Pager setting for all terminal output
     unicode: true           # Use unicode characters
 
     syntax_highlighting:

cmd/describe.go

@@ -15,7 +15,6 @@ var describeCmd = &cobra.Command{
 
 func init() {
 	describeCmd.PersistentFlags().StringP("query", "q", "", "Query the results of an `atmos describe` command using `yq` expressions")
-	describeCmd.PersistentFlags().String("pager", "true", "Disable / Enable the paging user experience")
 
 	RootCmd.AddCommand(describeCmd)
 }

cmd/describe_affected.go

@@ -70,13 +70,7 @@ func getRunnableDescribeAffectedCmd(
 			}
 		}
 
-		if cmd.Flags().Changed("pager") {
-			// TODO: update this post pr:https://github.com/cloudposse/atmos/pull/1174 is merged
-			props.CLIConfig.Settings.Terminal.Pager, err = cmd.Flags().GetString("pager")
-			if err != nil {
-				return err
-			}
-		}
+		// Global --pager flag is now handled in cfg.InitCliConfig
 
 		err = newDescribeAffectedExec(props.CLIConfig).Execute(&props)
 		return err

cmd/describe_config.go

@@ -33,13 +33,7 @@ var describeConfigCmd = &cobra.Command{
 			return err
 		}
 
-		if cmd.Flags().Changed("pager") {
-			// TODO: update this post pr:https://github.com/cloudposse/atmos/pull/1174 is merged
-			atmosConfig.Settings.Terminal.Pager, err = cmd.Flags().GetString("pager")
-			if err != nil {
-				return err
-			}
-		}
+		// Global --pager flag is now handled in cfg.InitCliConfig
 
 		err = e.NewDescribeConfig(&atmosConfig).ExecuteDescribeConfigCmd(query, format, "")
 		return err

cmd/describe_dependents.go

@@ -54,12 +54,7 @@ func getRunnableDescribeDependentsCmd(
 			return err
 		}
 
-		if cmd.Flags().Changed("pager") {
-			atmosConfig.Settings.Terminal.Pager, err = cmd.Flags().GetString("pager")
-			if err != nil {
-				return err
-			}
-		}
+		// Global --pager flag is now handled in cfg.InitCliConfig
 
 		describe.Component = args[0]
 		err = newDescribeDependentsExec(&atmosConfig).Execute(describe)

cmd/describe_stacks.go

@@ -70,13 +70,7 @@ func getRunnableDescribeStacksCmd(
 			return err
 		}
 
-		if cmd.Flags().Changed("pager") {
-			// TODO: update this post pr:https://github.com/cloudposse/atmos/pull/1174 is merged
-			atmosConfig.Settings.Terminal.Pager, err = cmd.Flags().GetString("pager")
-			if err != nil {
-				return err
-			}
-		}
+		// Global --pager flag is now handled in cfg.InitCliConfig
 
 		err = g.newDescribeStacksExec.Execute(&atmosConfig, describe)
 		return err

cmd/describe_workflows.go

@@ -52,12 +52,7 @@ func getRunnableDescribeWorkflowsCmd(
 			return err
 		}
 
-		pager, err := cmd.Flags().GetString("pager")
-		if err != nil {
-			return err
-		}
-
-		atmosConfig.Settings.Terminal.Pager = pager
+		// Global --pager flag is now handled in cfg.InitCliConfig
 		err = describeWorkflowsExec.Execute(&atmosConfig, describeWorkflowArgs)
 		return err
 	}

cmd/root.go

@@ -209,6 +209,9 @@
 	RootCmd.PersistentFlags().StringSlice("config", []string{}, "Paths to configuration files (comma-separated or repeated flag)")
 	RootCmd.PersistentFlags().StringSlice("config-path", []string{}, "Paths to configuration directories (comma-separated or repeated flag)")
 	RootCmd.PersistentFlags().Bool("no-color", false, "Disable color output")
+	RootCmd.PersistentFlags().String("pager", "", "Enable pager for output (--pager or --pager=true to enable, --pager=false to disable, --pager=less to use specific pager)")
+	// Set NoOptDefVal so --pager without value means "true"
+	RootCmd.PersistentFlags().Lookup("pager").NoOptDefVal = "true"
 	// Set custom usage template
 	err := templates.SetCustomUsageFunc(RootCmd)
 	if err != nil {
@@ -266,7 +269,24 @@
 			if err := oldUsageFunc(command); err != nil {
 				errUtils.CheckErrorPrintAndExit(err, "", "")
 			}
-			pager := pager.NewWithAtmosConfig(atmosConfig.Settings.Terminal.IsPagerEnabled())
+
+			// Check if pager should be enabled based on flag, env var, or config
+			pagerEnabled := atmosConfig.Settings.Terminal.IsPagerEnabled()
+
+			// Check if --pager flag was explicitly set
+			if pagerFlag, err := command.Flags().GetString("pager"); err == nil && pagerFlag != "" {
+				// Handle --pager flag values
+				if pagerFlag == "true" || pagerFlag == "on" || pagerFlag == "yes" || pagerFlag == "1" {
+					pagerEnabled = true
+				} else if pagerFlag == "false" || pagerFlag == "off" || pagerFlag == "no" || pagerFlag == "0" {
+					pagerEnabled = false
+				} else {
+					// Assume it's a pager command like "less" or "more"
+					pagerEnabled = true
+				}
+			}
+
+			pager := pager.NewWithAtmosConfig(pagerEnabled)
 			if err := pager.Run("Atmos CLI Help", buf.String()); err != nil {
 				log.Error("Failed to run pager", "error", err)
 				utils.OsExit(1)

pkg/config/config.go

@@ -79,9 +79,15 @@ func setLogConfig(atmosConfig *schema.AtmosConfiguration) {
 	}
 	if flagKeyValue, ok := flagKeyValue["no-color"]; ok || flagKeyValue == "true" {
 		atmosConfig.Settings.Terminal.NoColor = true
+		atmosConfig.Settings.Terminal.Color = false
 	} else if flagKeyValue == "false" {
 		atmosConfig.Settings.Terminal.NoColor = false
 	}
+
+	// Handle --pager global flag
+	if v, ok := flagKeyValue["pager"]; ok {
+		atmosConfig.Settings.Terminal.Pager = v
+	}
 }
 
 // TODO: This function works well, but we should generally avoid implementing manual flag parsing,