Add packages info in telemetry to get better understanding (#333)

## Summary
Add packages info in telemetry to get better understanding.

Note that different command behaves differently:
- for devbox shell, run, and add command, path can be set via --config
- for devbox init. path is set via args

## How was it tested?
go build
devbox shell

Author: LucilleH

boxcli/args.go

@@ -4,10 +4,8 @@
 package boxcli
 
 import (
-	"fmt"
 	"path/filepath"
 
-	"github.com/fatih/color"
 	"github.com/pkg/errors"
 	"go.jetpack.io/devbox/boxcli/usererr"
 )
@@ -30,12 +28,13 @@ func configPathFromUser(args []string, flags *configFlags) (string, error) {
 	}
 
 	if len(args) > 0 {
-		fmt.Printf(
-			"%s devbox <command> <path> is deprecated, use devbox <command> --config <path> instead\n",
-			color.HiYellowString("Warning:"),
+		return "", usererr.New(
+			"devbox <command> <path> is deprecated, use devbox <command> --config <path> instead.",
 		)
 	}
-	return pathArg(args), nil
+
+	// current directory is ""
+	return "", nil
 }
 
 func pathArg(args []string) string {

boxcli/midcobra/telemetry.go

@@ -15,6 +15,7 @@ import (
 	"github.com/denisbrodbeck/machineid"
 	segment "github.com/segmentio/analytics-go"
 	"github.com/spf13/cobra"
+	"go.jetpack.io/devbox"
 )
 
 // We collect some light telemetry to be able to improve devbox over time.
@@ -85,6 +86,7 @@ func (m *telemetryMiddleware) postRun(cmd *cobra.Command, args []string, runErr
 		DeviceID:    deviceID(),
 		Duration:    time.Since(m.startTime),
 		Failed:      runErr != nil,
+		Packages:    getPackages(cmd),
 	})
 }
 
@@ -103,6 +105,26 @@ func getSubcommand(c *cobra.Command, args []string) (subcmd *cobra.Command, suba
 	return subcmd, subargs, err
 }
 
+func getPackages(c *cobra.Command) []string {
+	configFlag := c.Flag("config")
+	// for shell, run, and add command, path can be set via --config
+	// if --config is not set, default to current directory which is ""
+	// the only exception is the init command, for the path can be set with args
+	// since after running init there will be no packages set in devbox.json
+	// we can safely ignore this case.
+	var path string
+	if configFlag != nil {
+		path = configFlag.Value.String()
+	}
+
+	box, err := devbox.Open(path, os.Stdout)
+	if err != nil {
+		return []string{}
+	}
+
+	return box.Config().Packages
+}
+
 type event struct {
 	AppName     string
 	AppVersion  string
@@ -111,6 +133,7 @@ type event struct {
 	DeviceID    string
 	Duration    time.Duration
 	Failed      bool
+	Packages    []string
 }
 
 func trackEvent(client segment.Client, evt *event) {
@@ -133,6 +156,7 @@ func trackEvent(client segment.Client, evt *event) {
 			Set("command", evt.Command).
 			Set("command_args", evt.CommandArgs).
 			Set("failed", evt.Failed).
-			Set("duration", evt.Duration.Milliseconds()),
+			Set("duration", evt.Duration.Milliseconds()).
+			Set("packages", evt.Packages),
 	})
 }

devbox.go

@@ -88,6 +88,10 @@ func (d *Devbox) ConfigDir() string {
 	return d.configDir
 }
 
+func (d *Devbox) Config() *Config {
+	return d.cfg
+}
+
 // Add adds a Nix package to the config so that it's available in the devbox
 // environment. It validates that the Nix package exists, but doesn't install
 // it. Adding a duplicate package is a no-op.