fix lint

Author: xuanyu66

go.mod

@@ -4,6 +4,7 @@ go 1.24.0
 
 require (
 	github.com/AlecAivazis/survey/v2 v2.3.6
+	github.com/AlekSi/pointer v1.2.0
 	github.com/aws/aws-sdk-go-v2 v1.27.1
 	github.com/aws/aws-sdk-go-v2/feature/s3/manager v1.16.23
 	github.com/charmbracelet/bubbles v0.17.1
@@ -44,7 +45,6 @@ require (
 
 require (
 	filippo.io/edwards25519 v1.1.0 // indirect
-	github.com/AlekSi/pointer v1.2.0 // indirect
 	github.com/alecthomas/chroma v0.10.0 // indirect
 	github.com/alecthomas/chroma/v2 v2.14.0 // indirect
 	github.com/alessio/shellescape v1.4.1 // indirect

internal/cli/serverless/migration/create.go

@@ -23,6 +23,7 @@ import (
 	"strings"
 	"time"
 
+	"github.com/AlekSi/pointer"
 	aws "github.com/aws/aws-sdk-go-v2/aws"
 	"github.com/fatih/color"
 	"github.com/juju/errors"
@@ -56,7 +57,7 @@ func CreateCmd(h *internal.Helper) *cobra.Command {
 			}
 			ctx := cmd.Context()
 
-			dryRun, err := cmd.Flags().GetBool(flag.MigrationDryRun)
+			dryRun, err := cmd.Flags().GetBool(flag.DryRun)
 			if err != nil {
 				return errors.Trace(err)
 			}
@@ -121,7 +122,7 @@ func CreateCmd(h *internal.Helper) *cobra.Command {
 	cmd.Flags().StringP(flag.ClusterID, flag.ClusterIDShort, "", "The ID of the target cluster.")
 	cmd.Flags().StringP(flag.DisplayName, flag.DisplayNameShort, "", "Display name for the migration.")
 	cmd.Flags().String(flag.MigrationConfigFile, "", "Path to a migration config JSON file. Use \"ticloud serverless migration template --mode <mode>\" to print templates.")
-	cmd.Flags().Bool(flag.MigrationDryRun, false, "Run a migration precheck (dry run) with the provided inputs without creating a migration.")
+	cmd.Flags().Bool(flag.DryRun, false, "Run a migration precheck (dry run) with the provided inputs without creating a migration.")
 
 	return cmd
 }
@@ -216,16 +217,12 @@ func printPrecheckSummary(result *pkgmigration.MigrationPrecheck, h *internal.He
 		if !shouldPrintPrecheckItem(item.Status) {
 			continue
 		}
-		var status string
-		if item.Status != nil {
-			status = string(*item.Status)
-		}
 		rows = append(rows, output.Row{
-			precheckItemType(item.Type),
-			status,
-			aws.ToString(item.Description),
-			aws.ToString(item.Reason),
-			aws.ToString(item.Solution),
+			string(pointer.Get(item.Type)),
+			string(pointer.Get(item.Status)),
+			pointer.Get(item.Description),
+			pointer.Get(item.Reason),
+			pointer.Get(item.Solution),
 		})
 	}
 	if len(rows) == 0 {
@@ -234,13 +231,6 @@ func printPrecheckSummary(result *pkgmigration.MigrationPrecheck, h *internal.He
 	return true, output.PrintHumanTable(h.IOStreams.Out, columns, rows)
 }
 
-func precheckItemType(value *pkgmigration.PrecheckItemType) string {
-	if value == nil {
-		return ""
-	}
-	return string(*value)
-}
-
 // shouldPrintPrecheckItem reports whether a precheck item should be shown to users.
 // Currently only WARNING and FAILED statuses surface because SUCCESS does not
 // provide actionable information.

internal/cli/serverless/migration/delete.go

@@ -119,16 +119,16 @@ func DeleteCmd(h *internal.Helper) *cobra.Command {
 					return errors.New("The terminal doesn't support prompt, please run with --force to delete the migration")
 				}
 				prompt := &survey.Input{
-					Message: fmt.Sprintf("%s %s %s", color.BlueString("Please type"), color.HiBlueString("yes"), color.BlueString("to confirm:")),
+					Message: fmt.Sprintf("%s %s %s", color.BlueString("Please type"), color.HiBlueString(config.Confirmed), color.BlueString("to confirm:")),
 				}
 				var confirmation string
 				if err := survey.AskOne(prompt, &confirmation); err != nil {
-					if err == terminal.InterruptErr {
+					if errors.Is(err, terminal.InterruptErr) {
 						return util.InterruptError
 					}
 					return err
 				}
-				if confirmation != "yes" {
+				if confirmation != config.Confirmed {
 					return errors.New("Incorrect confirm string entered, skipping migration deletion")
 				}
 			}

internal/flag/flag.go

@@ -129,7 +129,7 @@ const (
 	MigrationIDShort    string = "m"
 	MigrationConfigFile string = "config-file"
 	MigrationMode       string = "mode"
-	MigrationDryRun     string = "dry-run"
+	DryRun              string = "dry-run"
 )
 
 const OutputHelp = "Output format, one of [\"human\" \"json\"]. For the complete result, please use json format."