feat: refactor cli parsing

Signed-off-by: Shubharanshu Mahapatra <[email protected]>
runfinch · Dec 5, 2024 · d8cdca0 · d8cdca0
1 parent 0c143a9
commit d8cdca0
Show file tree

Hide file tree

Showing 9 changed files with 587 additions and 1,297 deletions.
diff --git a/cmd/finch/devcontainer_patch.go b/cmd/finch/devcontainer_patch.go
@@ -15,7 +15,6 @@ import (
 	"github.com/containerd/nerdctl/v2/pkg/inspecttypes/dockercompat"
 	"github.com/docker/go-connections/nat"
 	"github.com/sirupsen/logrus"
-	"golang.org/x/exp/slices"
 
 	"github.com/runfinch/finch/pkg/command"
 )
@@ -111,15 +110,12 @@ func inspectContainerOutputHandler(cmd command.Command) error {
 	return err
 }
 
-func handleDockerCompatComposeVersion(cmdName string, nc nerdctlCommand, runArgs []string) error {
-	if cmdName == "compose" && nc.fc.DockerCompat && slices.Contains(runArgs, "version") {
-		ver := nc.systemDeps.Env("DOCKER_COMPOSE_VERSION")
-		if ver != "" {
-			logrus.Warn("Displaying docker compose version set as environment variable DOCKER_COMPOSE_VERSION...")
-			fmt.Println(ver)
-			return nil
-		}
-		return errors.New("DOCKER_COMPOSE_VERSION environment variable is not set")
+func handleDockerCompatComposeVersion(nc nerdctlCommand) error {
+	ver := nc.systemDeps.Env("DOCKER_COMPOSE_VERSION")
+	if ver != "" {
+		logrus.Warn("Displaying docker compose version set as environment variable DOCKER_COMPOSE_VERSION...")
+		fmt.Println(ver)
+		return nil
 	}
-	return errors.New("")
+	return errors.New("DOCKER_COMPOSE_VERSION environment variable is not set")
 }
diff --git a/cmd/finch/main.go b/cmd/finch/main.go
@@ -42,13 +42,26 @@ func initializeNerdctlCommands(
 	nerdctlCommandCreator := newNerdctlCommandCreator(ncc, ecc, system.NewStdLib(), logger, fs, fc)
 	var allNerdctlCommands []*cobra.Command
 	for cmdName, cmdDescription := range nerdctlCmds {
-		allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.create(cmdName, cmdDescription))
-	}
-
-	if fc != nil && fc.DockerCompat {
-		for cmdName, cmdDescription := range dockerCompatCmds {
-			allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.create(cmdName, cmdDescription))
+		if fc != nil && fc.DockerCompat {
+			switch cmdName {
+			case "build":
+				allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.createDockerCompatBuildCmd())
+				continue
+			case "image":
+				allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.createDockerCompatImageCmd())
+				continue
+			case "inspect":
+				allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.createDockerCompatInspectCmd())
+				continue
+			case "run":
+				allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.createDockerCompatRunCmd())
+				continue
+			case "compose":
+				allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.createDockerCompatComposeCmd())
+				continue
+			}
 		}
+		allNerdctlCommands = append(allNerdctlCommands, nerdctlCommandCreator.create(cmdName, cmdDescription))
 	}
 
 	return allNerdctlCommands