refactor(cli): remove scaletest from slim binary

mafredri · mafredri · commit a02bebd77b43 · 2023-09-01T13:41:25.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,13 @@ func (p *prettyErrorFormatter) printf(style lipgloss.Style, format string, a ...
 		),
 	)
 }
+
+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 (`
Original file line number	Diff line number	Diff line change
`@@ -1019,3 +1019,13 @@ func (p *prettyErrorFormatter) printf(style lipgloss.Style, format string, a ...`
`1019`	`1019`	`),`
`1020`	`1020`	`)`
`1021`	`1021`	`}`
	`1022`	`+`
	`1023`	`+func slimUnsupported(w io.Writer, cmd string) {`
	`1024`	`+ _, _ = fmt.Fprintf(w, "You are using a 'slim' build of Coder, which does not support the %s subcommand.\n", cliui.DefaultStyles.Code.Render(cmd))`
	`1025`	`+ _, _ = fmt.Fprintln(w, "")`
	`1026`	`+ _, _ = fmt.Fprintln(w, "Please use a build of Coder from GitHub releases:")`
	`1027`	`+ _, _ = fmt.Fprintln(w, " https://github.com/coder/coder/releases")`
	`1028`	`+`
	`1029`	`+ //nolint:revive`
	`1030`	`+ os.Exit(1)`
	`1031`	`+}`