fix: 使用 errors.New 替代 fmt.Errorf 以改进错误处理

Author: Sisyphus-Junior

internal/app/app.go

@@ -63,7 +63,7 @@ func (a *App) Run(args []string) {
 
 func (a *App) Execute(args []string) error {
 	if err := a.validateConfig(); err != nil {
-		return fmt.Errorf(a.tf("err.invalid_config", err))
+		return errors.New(a.tf("err.invalid_config", err))
 	}
 	cmd := a.newRootCommand()
 	cmd.SetArgs(args)
@@ -97,7 +97,7 @@ func (a *App) newRootCommand() *cobra.Command {
 		if st, err := os.Stat(abs); err != nil {
 			return err
 		} else if !st.IsDir() {
-			return fmt.Errorf(a.tf("err.chdir_not_directory", abs))
+			return errors.New(a.tf("err.chdir_not_directory", abs))
 		}
 		return os.Chdir(abs)
 	}
@@ -182,7 +182,7 @@ func (a *App) newRootCommand() *cobra.Command {
 	cleanup := &cobra.Command{Use: "cleanup", Args: cobra.NoArgs, RunE: func(cmd *cobra.Command, args []string) error {
 		testArtifacts, _ := cmd.Flags().GetBool("test-artifacts")
 		if !testArtifacts {
-			return fmt.Errorf(a.t("err.cleanup_usage"))
+			return errors.New(a.t("err.cleanup_usage"))
 		}
 		if err := a.cleanup(); err != nil {
 			return err

internal/app/service.go

@@ -1,6 +1,7 @@
 package app
 
 import (
+	"errors"
 	"fmt"
 	"os"
 	"os/exec"
@@ -98,9 +99,9 @@ func (a *App) serviceAction(action, _ string) error {
 		if err != nil {
 			return err
 		}
-		fmt.Printf(a.tf("service.status_line", serviceName, status))
+		fmt.Print(a.tf("service.status_line", serviceName, status))
 	default:
-		return fmt.Errorf(a.tf("err.service_unsupported_action", action))
+		return errors.New(a.tf("err.service_unsupported_action", action))
 	}
 	a.instanceLog.Infof(a.tf("log.service_action_completed", action))
 	return nil

internal/app/workspace.go

@@ -49,7 +49,7 @@ func (a *App) workspaceDir(name string) (string, error) {
 		return "", err
 	}
 	if !found {
-		return "", fmt.Errorf(a.t("err.workspace_not_initialized_run_init"))
+		return "", errors.New(a.t("err.workspace_not_initialized_run_init"))
 	}
 	return dir, nil
 }
@@ -189,7 +189,7 @@ func (a *App) startInstance(name string) error {
 	cfg, err := a.readWorkspaceConfig(selected)
 	if err != nil {
 		if errors.Is(err, os.ErrNotExist) {
-			return fmt.Errorf(a.t("err.workspace_not_initialized_run_init"))
+			return errors.New(a.t("err.workspace_not_initialized_run_init"))
 		}
 		return err
 	}
@@ -256,7 +256,7 @@ func (a *App) stopInstance(name string) error {
 	cfg, err := a.readWorkspaceConfig(selected)
 	if err != nil {
 		if errors.Is(err, os.ErrNotExist) {
-			return fmt.Errorf(a.t("err.workspace_not_initialized_run_init"))
+			return errors.New(a.t("err.workspace_not_initialized_run_init"))
 		}
 		return err
 	}
@@ -288,7 +288,7 @@ func (a *App) statusInstance(name string) error {
 	cfg, err := a.readWorkspaceConfig(name)
 	if err != nil {
 		if errors.Is(err, os.ErrNotExist) {
-			return fmt.Errorf(a.t("err.workspace_not_initialized"))
+			return errors.New(a.t("err.workspace_not_initialized"))
 		}
 		return err
 	}
@@ -316,7 +316,7 @@ func (a *App) logsInstance(name string, tail int) error {
 	data, err := os.ReadFile(logPath)
 	if err != nil {
 		if errors.Is(err, os.ErrNotExist) {
-			return fmt.Errorf(a.t("err.workspace_log_not_found"))
+			return errors.New(a.t("err.workspace_log_not_found"))
 		}
 		return err
 	}
@@ -356,7 +356,7 @@ func (a *App) updateInstance(name string) error {
 	cfg, err := a.readWorkspaceConfig(name)
 	if err != nil {
 		if errors.Is(err, os.ErrNotExist) {
-			return fmt.Errorf(a.t("err.workspace_not_initialized_run_init"))
+			return errors.New(a.t("err.workspace_not_initialized_run_init"))
 		}
 		return err
 	}