[global] Add update/services/run commands to global (#1046)

## Summary

This adds:

```bash
devbox global update
devbox global services [subcommand]
devbox global run
```

commands to global. 

I fixed a few issues:

* Some commands were missing `--config` flag
* We were using `IsDevboxShellEnabled` instead of `IsEnvEnabled`. The
first (obsolete?) command only looks if we're in a devbox shell while
the second returns true if we are in an environment created for the
specific config.

I found bug that I didn't fix in this PR which is that if a package with
plugins/services is installed the environment must be refreshed. If done
with while `IsEnvEnabled` is false this is not an issue. This can be
fixed by sourcing the environment prior to running the service with
special care not to override user set variables.

cc: @Lagoja 

## How was it tested?

```bash
devbox global add apache
eval "$(devbox global shellenv)" # due to env bug
devbox global services up
```

Author: mikeland73

internal/boxcli/config.go

@@ -17,3 +17,9 @@ func (flags *configFlags) register(cmd *cobra.Command) {
 		&flags.path, "config", "c", "", "path to directory containing a devbox.json config file",
 	)
 }
+
+func (flags *configFlags) registerPersistent(cmd *cobra.Command) {
+	cmd.PersistentFlags().StringVarP(
+		&flags.path, "config", "c", "", "path to directory containing a devbox.json config file",
+	)
+}

internal/boxcli/global.go

@@ -30,9 +30,12 @@ func globalCmd() *cobra.Command {
 	}
 
 	addCommandAndHideConfigFlag(globalCmd, addCmd())
-	addCommandAndHideConfigFlag(globalCmd, removeCmd())
 	addCommandAndHideConfigFlag(globalCmd, installCmd())
+	addCommandAndHideConfigFlag(globalCmd, removeCmd())
+	addCommandAndHideConfigFlag(globalCmd, runCmd())
+	addCommandAndHideConfigFlag(globalCmd, servicesCmd())
 	addCommandAndHideConfigFlag(globalCmd, shellEnvCmd())
+	addCommandAndHideConfigFlag(globalCmd, updateCmd())
 
 	// Create list for non-global? Mike: I want it :)
 	globalCmd.AddCommand(globalListCmd())

internal/boxcli/services.go

@@ -14,18 +14,15 @@ type servicesCmdFlags struct {
 }
 
 type serviceUpFlags struct {
-	configFlags
 	background         bool
 	processComposeFile string
 }
 
 type serviceStopFlags struct {
-	configFlags
 	allProjects bool
 }
 
 func (flags *serviceUpFlags) register(cmd *cobra.Command) {
-	flags.configFlags.register(cmd)
 	cmd.Flags().StringVar(
 		&flags.processComposeFile,
 		"process-compose-file",
@@ -38,7 +35,6 @@ func (flags *serviceUpFlags) register(cmd *cobra.Command) {
 }
 
 func (flags *serviceStopFlags) register(cmd *cobra.Command) {
-	flags.configFlags.register(cmd)
 	cmd.Flags().BoolVar(
 		&flags.allProjects, "all-projects", false, "Stop all running services across all your projects.\nThis flag cannot be used simultaneously with the [services] argument")
 }
@@ -74,7 +70,7 @@ func servicesCmd() *cobra.Command {
 		Short: "Stop one or more services in the current project. If no service is specified, stops all services in the current project.",
 		Long:  `Stop one or more services in the current project. If no service is specified, stops all services in the current project. \nIf the --all-projects flag is specified, stops all running services across all your projects. This flag cannot be used with [service] arguments.`,
 		RunE: func(cmd *cobra.Command, args []string) error {
-			return stopServices(cmd, args, serviceStopFlags)
+			return stopServices(cmd, args, flags, serviceStopFlags)
 		},
 	}
 
@@ -90,11 +86,11 @@ func servicesCmd() *cobra.Command {
 		Use:   "up [service]...",
 		Short: "Starts process manager with specified services. If no services are listed, starts the process manager with all the services in your project",
 		RunE: func(cmd *cobra.Command, args []string) error {
-			return startProcessManager(cmd, args, serviceUpFlags)
+			return startProcessManager(cmd, args, flags, serviceUpFlags)
 		},
 	}
 
-	flags.config.register(servicesCommand)
+	flags.config.registerPersistent(servicesCommand)
 	serviceUpFlags.register(upCommand)
 	serviceStopFlags.register(stopCommand)
 	servicesCommand.AddCommand(lsCommand)
@@ -112,7 +108,6 @@ func listServices(cmd *cobra.Command, flags servicesCmdFlags) error {
 	}
 
 	return box.ListServices(cmd.Context())
-
 }
 
 func startServices(cmd *cobra.Command, services []string, flags servicesCmdFlags) error {
@@ -124,8 +119,13 @@ func startServices(cmd *cobra.Command, services []string, flags servicesCmdFlags
 	return box.StartServices(cmd.Context(), services...)
 }
 
-func stopServices(cmd *cobra.Command, services []string, flags serviceStopFlags) error {
-	box, err := devbox.Open(flags.configFlags.path, cmd.ErrOrStderr())
+func stopServices(
+	cmd *cobra.Command,
+	services []string,
+	servicesFlags servicesCmdFlags,
+	flags serviceStopFlags,
+) error {
+	box, err := devbox.Open(servicesFlags.config.path, cmd.ErrOrStderr())
 	if err != nil {
 		return errors.WithStack(err)
 	}
@@ -148,8 +148,13 @@ func restartServices(
 	return box.RestartServices(cmd.Context(), services...)
 }
 
-func startProcessManager(cmd *cobra.Command, args []string, flags serviceUpFlags) error {
-	box, err := devbox.Open(flags.path, cmd.ErrOrStderr())
+func startProcessManager(
+	cmd *cobra.Command,
+	args []string,
+	servicesFlags servicesCmdFlags,
+	flags serviceUpFlags,
+) error {
+	box, err := devbox.Open(servicesFlags.config.path, cmd.ErrOrStderr())
 	if err != nil {
 		return errors.WithStack(err)
 	}

internal/devconfig/config.go

@@ -69,7 +69,7 @@ func DefaultConfig() *Config {
 			},
 			InitHook: &shellcmd.Commands{
 				Cmds: []string{
-					"echo 'Welcome to devbox!'",
+					"echo 'Welcome to devbox!' > /dev/null",
 				},
 			},
 		},

internal/impl/devbox.go

@@ -457,7 +457,7 @@ func (d *Devbox) Services() (services.Services, error) {
 }
 
 func (d *Devbox) StartServices(ctx context.Context, serviceNames ...string) error {
-	if !envir.IsDevboxShellEnabled() {
+	if !d.IsEnvEnabled() {
 		return d.RunScript("devbox", append([]string{"services", "start"}, serviceNames...))
 	}
 
@@ -494,7 +494,7 @@ func (d *Devbox) StartServices(ctx context.Context, serviceNames ...string) erro
 }
 
 func (d *Devbox) StopServices(ctx context.Context, allProjects bool, serviceNames ...string) error {
-	if !envir.IsDevboxShellEnabled() {
+	if !d.IsEnvEnabled() {
 		args := []string{"services", "stop"}
 		args = append(args, serviceNames...)
 		if allProjects {
@@ -533,7 +533,7 @@ func (d *Devbox) StopServices(ctx context.Context, allProjects bool, serviceName
 }
 
 func (d *Devbox) ListServices(ctx context.Context) error {
-	if !envir.IsDevboxShellEnabled() {
+	if !d.IsEnvEnabled() {
 		return d.RunScript("devbox", []string{"services", "ls"})
 	}
 
@@ -571,7 +571,7 @@ func (d *Devbox) ListServices(ctx context.Context) error {
 }
 
 func (d *Devbox) RestartServices(ctx context.Context, serviceNames ...string) error {
-	if !envir.IsDevboxShellEnabled() {
+	if !d.IsEnvEnabled() {
 		return d.RunScript("devbox", append([]string{"services", "restart"}, serviceNames...))
 	}
 
@@ -637,7 +637,7 @@ func (d *Devbox) StartProcessManager(
 			return err
 		}
 	}
-	if !envir.IsDevboxShellEnabled() {
+	if !d.IsEnvEnabled() {
 		args := []string{"services", "up"}
 		args = append(args, requestedServices...)
 		if processComposeFileOrDir != "" {