refactor: standardize command layer to use RunE pattern with centralized error handling

- Convert deploy, docker, k8s, env commands to use RunE for proper exit codes
- Add centralized cmd/errors.go with LogError, LogSuccess, LogInfo helpers
- Standardize error formatting and messaging across CLI
- Remove unnecessary fmt imports from refactored commands
- Rename cmd/doctor.go to cmd/docker.go for clarity
- Improves CLI scriptability and error handling consistency

Author: AryanSharma9917

cmd/deploy.go

@@ -1,7 +1,6 @@
 package cmd
 
 import (
-	"fmt"
 
 	"github.com/aryansharma9917/codewise-cli/pkg/deploy"
 	"github.com/spf13/cobra"
@@ -20,32 +19,22 @@ var deployCmd = &cobra.Command{
 var deployRunCmd = &cobra.Command{
 	Use:   "run",
 	Short: "Execute deployment",
-	Run: func(cmd *cobra.Command, args []string) {
-
+	RunE: func(cmd *cobra.Command, args []string) error {
 		if deployEnv == "" {
-			fmt.Println("please provide --env")
-			return
-		}
-
-		if err := deploy.Run(deployEnv, dryRun); err != nil {
-			fmt.Println("deploy error:", err)
+			return ExitError("please provide --env")
 		}
+		return deploy.Run(deployEnv, dryRun)
 	},
 }
 
 var deployPlanCmd = &cobra.Command{
 	Use:   "plan",
 	Short: "Preview deployment execution plan",
-	Run: func(cmd *cobra.Command, args []string) {
-
+	RunE: func(cmd *cobra.Command, args []string) error {
 		if deployEnv == "" {
-			fmt.Println("please provide --env")
-			return
-		}
-
-		if err := deploy.Plan(deployEnv); err != nil {
-			fmt.Println("plan error:", err)
+			return ExitError("please provide --env")
 		}
+		return deploy.Plan(deployEnv)
 	},
 }
 

cmd/doctor.go cmd/docker.gocmd/doctor.go renamed to cmd/docker.go

@@ -1,7 +1,6 @@
 package cmd
 
 import (
-	"fmt"
 
 	"github.com/aryansharma9917/codewise-cli/pkg/docker"
 	"github.com/spf13/cobra"
@@ -17,32 +16,35 @@ var dockerCmd = &cobra.Command{
 var dockerInitCmd = &cobra.Command{
 	Use:   "init",
 	Short: "Generate a Dockerfile",
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := docker.InitDockerfile(); err != nil {
-			fmt.Println("ℹ️", err.Error())
-			return
+			return LogError(err.Error())
 		}
-		fmt.Println("✅ Dockerfile created")
+		LogSuccess("Dockerfile created")
+		return nil
 	},
 }
 
 var dockerValidateCmd = &cobra.Command{
 	Use:   "validate",
 	Short: "Validate Dockerfile best practices",
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := docker.ValidateDockerfile(); err != nil {
-			fmt.Println("❌", err.Error())
+			return LogError(err.Error())
 		}
+		return nil
 	},
 }
 
 var dockerBuildCmd = &cobra.Command{
 	Use:   "build",
 	Short: "Build Docker image",
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := docker.BuildDockerImage(imageTag); err != nil {
-			fmt.Println("❌ Docker build failed")
+			return LogError("Docker build failed: %v", err)
 		}
+		LogSuccess("Docker image built successfully")
+		return nil
 	},
 }
 

cmd/env_create.go

@@ -15,23 +15,22 @@ var envCreateCmd = &cobra.Command{
 	Use:   "create <name>",
 	Short: "Create a new environment",
 	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		name := args[0]
 
 		if interactive {
 			if err := createEnvInteractive(name); err != nil {
-				fmt.Println("error:", err)
-				return
+				return LogError("error: %v", err)
 			}
-			fmt.Println("environment", name, "created")
-			return
+			LogSuccess("environment %s created", name)
+			return nil
 		}
 
 		if err := env.CreateEnv(name, env.CreateOptions{}); err != nil {
-			fmt.Println("error:", err)
-			return
+			return LogError("error: %v", err)
 		}
-		fmt.Println("environment", name, "created")
+		LogSuccess("environment %s created", name)
+		return nil
 	},
 }
 

cmd/env_delete.go

@@ -14,7 +14,7 @@ var envDeleteCmd = &cobra.Command{
 	Use:   "delete <name>",
 	Short: "Delete an environment",
 	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		name := args[0]
 
 		if !yes {
@@ -23,21 +23,19 @@ var envDeleteCmd = &cobra.Command{
 				Message: fmt.Sprintf("Delete environment %q?", name),
 			}
 			if err := survey.AskOne(prompt, &confirm); err != nil {
-				fmt.Println("error:", err)
-				return
+				return LogError("error: %v", err)
 			}
 			if !confirm {
-				fmt.Println("aborted")
-				return
+				return LogError("aborted")
 			}
 		}
 
 		if err := env.DeleteEnv(name, env.DeleteOptions{Force: yes}); err != nil {
-			fmt.Println("error:", err)
-			return
+			return LogError("error: %v", err)
 		}
 
-		fmt.Println("environment", name, "deleted")
+		LogSuccess("environment %s deleted", name)
+		return nil
 	},
 }
 

cmd/env_list.go

@@ -1,34 +1,34 @@
 package cmd
 
 import (
-	"fmt"
+"fmt"
 
-	"github.com/aryansharma9917/codewise-cli/pkg/env"
-	"github.com/spf13/cobra"
+"github.com/aryansharma9917/codewise-cli/pkg/env"
+"github.com/spf13/cobra"
 )
 
 var envListCmd = &cobra.Command{
-	Use:   "list",
-	Short: "List environments",
-	Run: func(cmd *cobra.Command, args []string) {
-		envs, err := env.ListEnvs()
-		if err != nil {
-			fmt.Println("error:", err)
-			return
-		}
+Use:   "list",
+Short: "List environments",
+RunE: func(cmd *cobra.Command, args []string) error {
+envs, err := env.ListEnvs()
+if err != nil {
+return LogError(err.Error())
+}
 
-		if len(envs) == 0 {
-			fmt.Println("no environments found")
-			return
-		}
+if len(envs) == 0 {
+fmt.Println("no environments found")
+return nil
+}
 
-		for _, e := range envs {
-			fmt.Printf("%-10s namespace=%s context=%s\n",
-				e.Name, e.K8s.Namespace, e.K8s.Context)
-		}
-	},
+for _, e := range envs {
+fmt.Printf("%-10s namespace=%s context=%s\n",
+e.Name, e.K8s.Namespace, e.K8s.Context)
+}
+return nil
+},
 }
 
 func init() {
-	envCmd.AddCommand(envListCmd)
+envCmd.AddCommand(envListCmd)
 }

cmd/errors.go

@@ -0,0 +1,29 @@
+package cmd
+
+import (
+	"fmt"
+	"os"
+)
+
+// ExitError prints an error message and exits with status code 1.
+func ExitError(format string, args ...interface{}) error {
+	fmt.Fprintf(os.Stderr, "error: %s\n", fmt.Sprintf(format, args...))
+	return fmt.Errorf(format, args...)
+}
+
+// LogError prints an info/warning message but does not exit.
+func LogError(format string, args ...interface{}) error {
+	msg := fmt.Sprintf(format, args...)
+	fmt.Fprintf(os.Stderr, "info: %s\n", msg)
+	return fmt.Errorf(msg)
+}
+
+// LogInfo prints an info message to stderr.
+func LogInfo(format string, args ...interface{}) {
+	fmt.Fprintf(os.Stderr, "info: %s\n", fmt.Sprintf(format, args...))
+}
+
+// LogSuccess prints a success message to stdout.
+func LogSuccess(format string, args ...interface{}) {
+	fmt.Printf("✅ %s\n", fmt.Sprintf(format, args...))
+}

cmd/k8s_apply.go

@@ -1,7 +1,6 @@
 package cmd
 
 import (
-	"fmt"
 
 	"github.com/aryansharma9917/codewise-cli/pkg/k8s"
 	"github.com/spf13/cobra"
@@ -16,23 +15,16 @@ var (
 var k8sApplyCmd = &cobra.Command{
 	Use:   "apply",
 	Short: "Apply Kubernetes manifests to the current cluster",
-	Run: func(cmd *cobra.Command, args []string) {
-		// If not a dry run, check cluster connectivity
+	RunE: func(cmd *cobra.Command, args []string) error {
 		if !k8sDryRun {
 			if err := k8s.CheckKubectl(); err != nil {
-				fmt.Println("info:", err.Error())
-				return
+				return LogError(err.Error())
 			}
 			if err := k8s.CheckCluster(); err != nil {
-				fmt.Println("info:", err.Error())
-				return
+				return LogError(err.Error())
 			}
 		}
-
-		if err := k8s.ApplyManifests(k8sNamespace, k8sContext, k8sDryRun); err != nil {
-			fmt.Println("info:", err.Error())
-			return
-		}
+		return k8s.ApplyManifests(k8sNamespace, k8sContext, k8sDryRun)
 	},
 }
 

cmd/k8s_delete.go

@@ -1,7 +1,6 @@
 package cmd
 
 import (
-	"fmt"
 
 	"github.com/aryansharma9917/codewise-cli/pkg/k8s"
 	"github.com/spf13/cobra"
@@ -16,22 +15,16 @@ var (
 var k8sDeleteCmd = &cobra.Command{
 	Use:   "delete",
 	Short: "Delete Kubernetes resources from the current cluster",
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		if !k8sDeleteDryRun {
 			if err := k8s.CheckKubectl(); err != nil {
-				fmt.Println("info:", err.Error())
-				return
+				return LogError(err.Error())
 			}
 			if err := k8s.CheckCluster(); err != nil {
-				fmt.Println("info:", err.Error())
-				return
+				return LogError(err.Error())
 			}
 		}
-
-		if err := k8s.DeleteManifests(k8sDeleteNamespace, k8sDeleteContext, k8sDeleteDryRun); err != nil {
-			fmt.Println("info:", err.Error())
-			return
-		}
+		return k8s.DeleteManifests(k8sDeleteNamespace, k8sDeleteContext, k8sDeleteDryRun)
 	},
 }
 

cmd/k8s_init.go

@@ -1,7 +1,6 @@
 package cmd
 
 import (
-	"fmt"
 
 	"github.com/aryansharma9917/codewise-cli/pkg/k8s"
 	"github.com/spf13/cobra"
@@ -15,18 +14,18 @@ var (
 var k8sInitCmd = &cobra.Command{
 	Use:   "init",
 	Short: "Initialize Kubernetes deployment and service manifests",
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		opts := k8s.Options{
 			AppName: k8sInitAppName,
 			Image:   k8sInitImage,
 		}
 
 		if err := k8s.InitK8sManifests(opts); err != nil {
-			fmt.Println("info:", err.Error())
-			return
+			return LogError(err.Error())
 		}
 
-		fmt.Println("k8s manifests created at k8s/app")
+		LogSuccess("k8s manifests created at k8s/app")
+		return nil
 	},
 }