refactor(cli): remove scaletest from slim binary

mafredri · mafredri · commit 7e277343ac5e · 2023-09-01T13:42:06.000Z
This change removes 3 MB when combined with #9483. Ref: #9380, #9483
diff --git a/cli/exp_scaletest.go b/cli/exp_scaletest.go
@@ -1,3 +1,5 @@
+//go:build !slim
+
 package cli
 
 import (
diff --git a/cli/exp_scaletest_slim.go b/cli/exp_scaletest_slim.go
@@ -0,0 +1,20 @@
+//go:build slim
+
+package cli
+
+import "github.com/coder/coder/v2/cli/clibase"
+
+func (r *RootCmd) scaletestCmd() *clibase.Cmd {
+	cmd := &clibase.Cmd{
+		Use:     "scaletest",
+		Short:   "Run a scale test against the Coder API",
+		RawArgs: true, // We accept RawArgs so all commands and flags are accepted.
+		Hidden:  true,
+		Handler: func(inv *clibase.Invocation) error {
+			slimUnsupported(inv.Stderr, "exp scaletest")
+			return nil
+		},
+	}
+
+	return cmd
+}
diff --git a/cli/root.go b/cli/root.go
@@ -1019,3 +1019,14 @@ func (p *prettyErrorFormatter) printf(style lipgloss.Style, format string, a ...
 		),
 	)
 }
+
+//nolint:unused
+func slimUnsupported(w io.Writer, cmd string) {
+	_, _ = fmt.Fprintf(w, "You are using a 'slim' build of Coder, which does not support the %s subcommand.\n", cliui.DefaultStyles.Code.Render(cmd))
+	_, _ = fmt.Fprintln(w, "")
+	_, _ = fmt.Fprintln(w, "Please use a build of Coder from GitHub releases:")
+	_, _ = fmt.Fprintln(w, "  https://github.com/coder/coder/releases")
+
+	//nolint:revive
+	os.Exit(1)
+}
diff --git a/cli/server_slim.go b/cli/server_slim.go
@@ -3,12 +3,7 @@
 package cli
 
 import (
-	"fmt"
-	"io"
-	"os"
-
 	"github.com/coder/coder/v2/cli/clibase"
-	"github.com/coder/coder/v2/cli/cliui"
 )
 
 func (r *RootCmd) Server(_ func()) *clibase.Cmd {
@@ -19,18 +14,10 @@ func (r *RootCmd) Server(_ func()) *clibase.Cmd {
 		RawArgs: true,
 		Hidden:  true,
 		Handler: func(inv *clibase.Invocation) error {
-			serverUnsupported(inv.Stderr)
+			slimUnsupported(inv.Stderr, "server")
 			return nil
 		},
 	}
 
 	return root
 }
-
-func serverUnsupported(w io.Writer) {
-	_, _ = fmt.Fprintf(w, "You are using a 'slim' build of Coder, which does not support the %s subcommand.\n", cliui.DefaultStyles.Code.Render("server"))
-	_, _ = fmt.Fprintln(w, "")
-	_, _ = fmt.Fprintln(w, "Please use a build of Coder from GitHub releases:")
-	_, _ = fmt.Fprintln(w, "  https://github.com/coder/coder/releases")
-	os.Exit(1)
-}

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+//go:build !slim`
	`2`	`+`
`1`	`3`	`package cli`
`2`	`4`
`3`	`5`	`import (`