fix: Always use semicolon

**Problem**
With sbt 2.x, the runner defaults to using sbtn,
which interprets sbt foo bar as a single command.

**Solution**
This injects semicolon as sbt "foo; bar"

Author: eed3si9n

.github/workflows/ci.yml

@@ -66,15 +66,15 @@ jobs:
 
       - name: Check that workflows are up to date
         shell: bash
-        run: sbt '++ ${{ matrix.scala }}' githubWorkflowCheck
+        run: sbt '++ ${{ matrix.scala }}; githubWorkflowCheck'
 
       - if: matrix.java != 'zulu@8'
         shell: bash
-        run: sbt '++ ${{ matrix.scala }}' '+ test' '+ scripted'
+        run: sbt '++ ${{ matrix.scala }}; + test; + scripted'
 
       - if: matrix.java == 'zulu@8'
         shell: bash
-        run: sbt '++ ${{ matrix.scala }}' test scripted
+        run: sbt '++ ${{ matrix.scala }}; test; scripted'
 
       - name: Clean up Ivy Local repo
         shell: bash

src/main/scala/sbtghactions/GenerativePlugin.scala

@@ -281,18 +281,12 @@ ${indent(rendered.mkString("\n"), 1)}"""
       case sbtStep: Sbt =>
         import sbtStep.commands
 
-        val sbtClientMode = sbt.matches("""sbt.* --client($| .*)""")
-        val safeCommands = if (sbtClientMode)
-          s"'${(sbtStepPreamble ::: commands).mkString("; ")}'"
-        else (sbtStepPreamble ::: commands).map { c =>
-          if (c.indexOf(' ') >= 0)
-            s"'$c'"
-          else
-            c
-        }.mkString(" ")
-
+        val multi = (sbtStepPreamble ::: commands) match {
+          case x :: Nil => x
+          case xs       => s"'${xs.mkString("; ")}'"
+        }
         renderRunBody(
-          commands = List(s"$sbt $safeCommands"),
+          commands = List(s"$sbt $multi"),
           params = sbtStep.params,
           renderedShell = renderedShell
         )

src/test/scala/sbtghactions/GenerativePluginSpec.scala

@@ -431,7 +431,7 @@ class GenerativePluginSpec extends Specification {
     "compile sbt using the command provided" in {
       compileStep(
         Sbt(List("show scalaVersion", "compile", "test")),
-        "$SBT") mustEqual s"- run: $$SBT '++ $${{ matrix.scala }}' 'show scalaVersion' compile test"
+        "$SBT") mustEqual s"- run: $$SBT '++ $${{ matrix.scala }}; show scalaVersion; compile; test'"
     }
 
     "compile sbt without switch command" in {
@@ -444,7 +444,7 @@ class GenerativePluginSpec extends Specification {
     "compile sbt with parameters" in {
       compileStep(
         Sbt(List("compile", "test"), params = Map("abc" -> "def", "cafe" -> "@42")),
-        "$SBT") mustEqual s"""- run: $$SBT '++ $${{ matrix.scala }}' compile test
+        "$SBT") mustEqual s"""- run: $$SBT '++ $${{ matrix.scala }}; compile; test'
                          |  with:
                          |    abc: def
                          |    cafe: '@42'""".stripMargin
@@ -699,7 +699,7 @@ class GenerativePluginSpec extends Specification {
   env:
     not: now
   steps:
-    - run: csbt '++ $${{ matrix.scala }}' +compile"""
+    - run: csbt '++ $${{ matrix.scala }}; +compile'"""
     }
 
     "compile a job with an environment" in {