Move interfactive check

Author: MDrakos

cmd/state-installer/cmd.go

@@ -45,7 +45,6 @@ type Params struct {
 	command         string
 	force           bool
 	isUpdate        bool
-	isInteractive   bool
 	activate        *project.Namespaced
 	activateDefault *project.Namespaced
 }
@@ -128,7 +127,6 @@ func main() {
 	an.Event(AnalyticsFunnelCat, "start")
 
 	params := newParams()
-	params.isInteractive = terminal.IsTerminal(int(os.Stdin.Fd()))
 	cmd := captain.NewCommand(
 		"state-installer",
 		"",
@@ -389,7 +387,7 @@ func postInstallEvents(out output.Outputer, cfg *config.Instance, an analytics.D
 			an.EventWithLabel(AnalyticsFunnelCat, "forward-activate-default-err", err.Error())
 			return errs.Silence(errs.Wrap(err, "Could not activate %s, error returned: %s", params.activateDefault.String(), errs.JoinMessage(err)))
 		}
-	case !params.isUpdate && params.isInteractive && os.Getenv(constants.InstallerNoSubshell) != "true":
+	case !params.isUpdate && terminal.IsTerminal(int(os.Stdin.Fd())) && os.Getenv(constants.InstallerNoSubshell) != "true":
 		ss := subshell.New(cfg)
 		ss.SetEnv(osutils.InheritEnv(envMap(binPath)))
 		if err := ss.Activate(nil, cfg, out); err != nil {