style: remove openapi command, replace with internal command

Generating the openapi spec is a developer operation, not an end-user
operation. This PR removes the sub-command from the CLI and adds a new
internal/openapigen command. This matches the pattern we use for
generating the user documentation (internal/docsgen).

Makefile

@@ -95,4 +95,4 @@ openapi-lint: docs/api/openapi3.json
 	openapi lint $<
 
 docs/api/openapi3.json:
-	go run . openapi
+	go run ./internal/openapigen $@

internal/cmd/cmd.go

@@ -350,23 +350,6 @@ func canonicalPath(in string) (string, error) {
 	return abs, nil
 }
 
-// TODO: remove
-func newOpenAPICmd() *cobra.Command {
-	cmd := &cobra.Command{
-		Use:    "openapi",
-		Short:  "generate the openapi spec",
-		Hidden: true,
-
-		RunE: func(cmd *cobra.Command, args []string) error {
-			s := &server.Server{}
-			s.GenerateRoutes()
-			return nil
-		},
-	}
-
-	return cmd
-}
-
 func newServerCmd() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "server",
@@ -595,7 +578,6 @@ func NewRootCmd() (*cobra.Command, error) {
 	rootCmd.AddCommand(newTokensCmd())
 	rootCmd.AddCommand(newInfoCmd())
 	rootCmd.AddCommand(newServerCmd())
-	rootCmd.AddCommand(newOpenAPICmd())
 	rootCmd.AddCommand(newConnectorCmd())
 	rootCmd.AddCommand(newVersionCmd())
 

internal/openapigen/main.go

@@ -0,0 +1,27 @@
+package main
+
+import (
+	"fmt"
+	"os"
+
+	"github.com/infrahq/infra/internal/server"
+)
+
+func main() {
+	if err := run(os.Args[1:]); err != nil {
+		fmt.Fprintln(os.Stderr, err.Error())
+		os.Exit(1)
+	}
+}
+
+func run(args []string) error {
+	if len(args) < 1 {
+		return fmt.Errorf("missing command line argument: path to openapi spec file")
+	}
+	filename := args[0]
+
+	s := server.Server{}
+	s.GenerateRoutes()
+
+	return server.WriteOpenAPISpecToFile(filename)
+}

internal/server/openapi.go

@@ -236,7 +236,7 @@ func writeOpenAPISpec(version string, out io.Writer) error {
 	return nil
 }
 
-func writeOpenAPISpecToFile(filename string) error {
+func WriteOpenAPISpecToFile(filename string) error {
 	old, err := readOpenAPISpec(filename)
 	if err != nil {
 		return err

internal/server/openapi_test.go

@@ -15,6 +15,6 @@ func TestWriteOpenAPISpec(t *testing.T) {
 	s.GenerateRoutes()
 
 	filename := "../../docs/api/openapi3.json"
-	err := writeOpenAPISpecToFile(filename)
+	err := WriteOpenAPISpecToFile(filename)
 	require.NoError(t, err)
 }