STAC-23287: Address comments

Author: viliakov

cmd/stackpack/stackpack_install.go

@@ -17,8 +17,8 @@ type InstallArgs struct {
 	Name             string
 	UnlockedStrategy string
 	Params           map[string]string
-	Wait             bool          // New: whether to wait for installation to complete
-	Timeout          time.Duration // New: timeout for wait operation
+	Wait             bool
+	Timeout          time.Duration
 }
 
 func StackpackInstallCommand(cli *di.Deps) *cobra.Command {
@@ -61,7 +61,7 @@ func RunStackpackInstallCommand(args *InstallArgs) di.CmdWithApiFn {
 			return common.NewResponseError(err, resp)
 		}
 
-		// New wait functionality: monitor installation until completion
+		// Wait functionality: monitor installation until completion
 		if args.Wait {
 			if !cli.IsJson() {
 				cli.Printer.PrintLn("Waiting for installation to complete...")
@@ -121,7 +121,6 @@ func RunStackpackInstallCommand(args *InstallArgs) di.CmdWithApiFn {
 				)
 			}
 		} else {
-			// Original behavior - just show the immediate response
 			if cli.IsJson() {
 				cli.Printer.PrintJson(map[string]interface{}{
 					"instance": instance,

cmd/stackpack/stackpack_upgrade.go

@@ -21,8 +21,8 @@ var (
 type UpgradeArgs struct {
 	TypeName         string
 	UnlockedStrategy string
-	Wait             bool          // New: whether to wait for upgrade to complete
-	Timeout          time.Duration // New: timeout for wait operation
+	Wait             bool
+	Timeout          time.Duration
 }
 
 func StackpackUpgradeCommand(cli *di.Deps) *cobra.Command {
@@ -68,7 +68,7 @@ func RunStackpackUpgradeCommand(args *UpgradeArgs) di.CmdWithApiFn {
 			return common.NewResponseError(err, resp)
 		}
 
-		// New wait functionality: monitor upgrade until completion
+		// Wait functionality: monitor upgrade until completion
 		if args.Wait {
 			if !cli.IsJson() {
 				cli.Printer.PrintLn("Waiting for upgrade to complete...")
@@ -129,7 +129,6 @@ func RunStackpackUpgradeCommand(args *UpgradeArgs) di.CmdWithApiFn {
 				)
 			}
 		} else {
-			// Original behavior - just show the immediate response
 			if cli.IsJson() {
 				cli.Printer.PrintJson(map[string]interface{}{
 					"success":         true,