Fix boolean input parsing, hide built-in help sub-command, and improve release action (#133)

* Fix boolean input processing and handle invalid inputs gracefully

* Hide the built-in 'help' sub-command that shows up as a duplicate of the custom workflow help command in root command help

* Set the CLI version on build via ldflags

* Add Docker build and push job to release action

* Add version build arg for Docker image versioning in release job

Author: mhmdiaa

.github/workflows/release.yaml

@@ -31,3 +31,41 @@ jobs:
           goos: ${{ matrix.goos }}
           goarch: ${{ matrix.goarch }}
           asset_name: '${{ env.REPOSITORY_NAME }}-${{ env.RELEASE_TAG }}-${{ env.OS_NAME }}-${{ matrix.goarch }}'
+          ldflags: '-X github.com/trickest/trickest-cli/pkg/version.Version=v${{ env.RELEASE_TAG }}'
+  docker:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Docker meta
+        id: meta
+        uses: docker/metadata-action@v5
+        with:
+          images: |
+            quay.io/trickest/trickest-cli
+          tags: |
+            type=raw,value=latest
+            type=semver,pattern={{version}}
+
+      - name: Login to Quay.io
+        uses: docker/login-action@v3
+        with:
+          registry: quay.io
+          username: ${{ secrets.QUAY_USERNAME }}
+          password: ${{ secrets.QUAY_ROBOT_TOKEN }}
+  
+      - name: Set up QEMU
+        uses: docker/setup-qemu-action@v3
+
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v3
+
+      - name: Build and push
+        uses: docker/build-push-action@v6
+        with:
+          platforms: linux/amd64,linux/arm64
+          push: true
+          tags: ${{ steps.meta.outputs.tags }}
+          build-args: |
+            VERSION=v${{ env.RELEASE_TAG }}

Dockerfile

@@ -6,7 +6,8 @@ COPY . /app
 
 WORKDIR /app
 
-RUN env GOOS=linux GOARCH=amd64 go build .
+ARG VERSION=dev
+RUN env GOOS=linux GOARCH=amd64 go build -ldflags "-X github.com/trickest/trickest-cli/pkg/version.Version=${VERSION}" .
 
 FROM --platform=linux/amd64 alpine:3.21
 

cmd/root.go

@@ -65,4 +65,5 @@ func init() {
 	RootCmd.AddCommand(investigate.InvestigateCmd)
 
 	RootCmd.SetVersionTemplate(`{{printf "Trickest CLI %s\n" .Version}}`)
+	RootCmd.SetHelpCommand(&cobra.Command{Hidden: true})
 }

pkg/version/version.go

@@ -1,4 +1,5 @@
 package version
 
 // Version is the current version of the CLI
-var Version = "v2.0.0"
+// This is set during build via ldflags in the Dockerfile and GitHub Actions release workflow
+var Version = "dev"

pkg/workflowbuilder/primitive.go

@@ -113,6 +113,7 @@ func setPrimitiveNodeValue(pNode *trickest.PrimitiveNode, value any) error {
 func processPrimitiveNodeValue(nodeType string, value any) (any, string, error) {
 	var normalizedValue any
 	var label string
+	var err error
 
 	switch val := value.(type) {
 	case string:
@@ -130,7 +131,10 @@ func processPrimitiveNodeValue(nodeType string, value any) (any, string, error)
 			}
 			normalizedValue = val
 		case "BOOLEAN":
-			normalizedValue = val
+			normalizedValue, err = strconv.ParseBool(val)
+			if err != nil {
+				return nil, "", fmt.Errorf("invalid boolean value: %v", value)
+			}
 		}
 	case int:
 		normalizedValue = strconv.Itoa(val)