ci: add ci for the cli for rollkit start cmd (#1603)

<!--
Please read and fill out this form before submitting your PR.

Please make sure you have reviewed our contributors guide before
submitting your
first PR.
-->

## Overview
We will want to add this new CLI CI to the required checks when this
gets merged in.
As such, I won't use auto merge on this so that I can update the branch
protection rules right after I merge this.
<!-- 
Please provide an explanation of the PR, including the appropriate
context,
background, goal, and rationale. If there is an issue with this
information,
please provide a tl;dr and link the issue. 
-->

## Checklist

<!-- 
Please complete the checklist to ensure that the PR is ready to be
reviewed.

IMPORTANT:
PRs should be left in Draft until the below checklist is completed.
-->

- [ ] New and updated code has appropriate documentation
- [ ] New and updated code has new and/or updated testing
- [ ] Required CI checks are passing
- [ ] Visual proof for any user facing features like CLI or
documentation updates
- [ ] Linked issues closed with keywords


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit


- **New Features**
- Introduced a new job in the CI workflow specifically for testing the
CLI with enhanced parameters.
- **Refactor**
- Updated the CLI's run node command to support CI mode, preventing
indefinite blocking during automated tests by adding a new `ci` flag.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: MSevey

.github/workflows/ci_release.yml

@@ -50,6 +50,21 @@ jobs:
   proto:
     uses: ./.github/workflows/proto.yml
 
+  cli:
+    name: Test CLI
+    needs: [setup]
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: set up go
+        uses: actions/setup-go@v5
+        with:
+          go-version: ${{ needs.setup.outputs.go-version }}
+      - name: Build cli
+        run: make install
+      - name: Run cli
+        run: rollkit start --ci
+
   # get_merged_pr_labels uses the listPullRequestsAssociatedWithCommit API
   # endpoint to get the PR information for the commit during a push event. Once
   # the PR information is received, we check to see if the create-release label

cmd/rollkit/commands/run_node.go

@@ -161,8 +161,17 @@ func NewRunNodeCmd() *cobra.Command {
 				}
 			})
 
-			// Run forever.
-			select {}
+			// Check if we are running in CI mode
+			inCI, err := cmd.Flags().GetBool("ci")
+			if err != nil {
+				return err
+			}
+			if !inCI {
+				// Block forever
+				select {}
+			}
+
+			return nil
 		},
 	}
 
@@ -187,6 +196,7 @@ func addNodeFlags(cmd *cobra.Command) {
 	cmtcmd.AddNodeFlags(cmd)
 
 	cmd.Flags().String("transport", config.ABCI, "specify abci transport (socket | grpc)")
+	cmd.Flags().Bool("ci", false, "run node for ci testing")
 
 	// Add Rollkit flags
 	rollconf.AddFlags(cmd)

cmd/rollkit/docs/rollkit_start.md

@@ -10,6 +10,7 @@ rollkit start [flags]
 
 ```
       --abci string                                     specify abci transport (socket | grpc) (default "socket")
+      --ci                                              run node for ci testing
       --consensus.create_empty_blocks                   set this to false to only produce blocks when there are txs or when the AppHash changes (default true)
       --consensus.create_empty_blocks_interval string   the possible interval between empty blocks (default "0s")
       --consensus.double_sign_check_height int          how many blocks to look back to check existence of the node's consensus votes before joining consensus