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
@@ -153,4 +153,4 @@ jobs:
           PGP_SECRET: ${{ secrets.PGP_SECRET }}
           SONATYPE_PASSWORD: ${{ secrets.SONATYPE_PASSWORD }}
           SONATYPE_USERNAME: ${{ secrets.SONATYPE_USERNAME }}
-        run: sbt ci-release
+        run: sbt 'ci-release'

src/main/scala/sbtghactions/GenerativePlugin.scala

@@ -281,18 +281,9 @@ ${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 = s"'${(sbtStepPreamble ::: commands).mkString("; ")}'"
         renderRunBody(
-          commands = List(s"$sbt $safeCommands"),
+          commands = List(s"$sbt $multi"),
           params = sbtStep.params,
           renderedShell = renderedShell
         )