ci(release): fix completion syntax in smokes; run tests in parallel (remove build dependency) and gate publish on tests

.github/workflows/release.yml

@@ -115,7 +115,7 @@ jobs:
 
   build-binaries:
     name: Build ${{ matrix.target }}
-    needs: [determine-version, preflight-tests]
+    needs: [determine-version]
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
@@ -273,7 +273,7 @@ jobs:
           set -euo pipefail
           exe="codex-rs/target/${{ matrix.target }}/release/code"
           "$exe" --version
-          "$exe" completion --shell bash > /dev/null
+          "$exe" completion bash > /dev/null
 
       - name: Post-build smoke (run binary) [Windows]
         if: matrix.os == 'windows-latest' && matrix.target == 'x86_64-pc-windows-msvc'
@@ -281,7 +281,7 @@ jobs:
         run: |
           $exe = "codex-rs/target/${{ matrix.target }}/release/code.exe"
           & $exe --version | Out-Null
-          & $exe completion --shell bash | Out-Null
+          & $exe completion bash | Out-Null
 
       - name: sccache stats
         shell: bash
@@ -357,7 +357,7 @@ jobs:
 
   release:
     name: Publish to npm
-    needs: [determine-version, build-binaries]
+    needs: [determine-version, build-binaries, preflight-tests]
     runs-on: ubuntu-latest
     if: "!contains(github.event.head_commit.message, '[skip ci]')"
     timeout-minutes: 30