chore(ci): give each test suite a job of its own

The lint job also ran the Vitest suite, and the job names mixed two
rules. Some named the subject, others named the tool.

Vitest moves into the js-tests job. It needs no Ruby and no Vite build,
so a checkout and the Bun packages are enough. The lint job now holds
static analysis only.

bats, rspec and rspec-integration become shell-tests, ruby-tests and
integration-tests. A name that holds no tool also survives a change of
the runner.

The status names change with this. Branch protection needs the new
names if it lists single jobs instead of ci-success.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: ledermann

.github/workflows/ci.yml

@@ -51,9 +51,6 @@ jobs:
       - name: Check for TypeScript errors
         run: bun run tsc
 
-      - name: Run JavaScript Tests
-        run: bun run test
-
       - name: Check JS size limit
         run: |
           bunx vite build
@@ -62,7 +59,25 @@ jobs:
       - name: Shellcheck shell scripts
         run: shellcheck $(git ls-files '*.sh')
 
-  bats:
+  js-tests:
+    runs-on: ubuntu-24.04
+    timeout-minutes: 5
+
+    steps:
+      - uses: actions/checkout@v7
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+        with:
+          bun-version-file: package.json
+
+      - name: Install Bun packages
+        run: bun install --frozen-lockfile
+
+      - name: Run JavaScript tests
+        run: bun run test
+
+  shell-tests:
     runs-on: ubuntu-24.04
     timeout-minutes: 3
 
@@ -80,7 +95,7 @@ jobs:
       - name: Run bats tests
         run: bats --recursive spec/bats/
 
-  rspec:
+  ruby-tests:
     runs-on: ubuntu-24.04
     timeout-minutes: 10
 
@@ -156,7 +171,7 @@ jobs:
           path: tmp/capybara/
           if-no-files-found: ignore
 
-  rspec-integration:
+  integration-tests:
     runs-on: ubuntu-24.04
     timeout-minutes: 15
 
@@ -185,7 +200,7 @@ jobs:
       # no lock, which the serial runner cannot do. It also creates the
       # database of every worker, so no separate setup step is needed.
       # NO_COVERAGE skips the instrumentation: this job never reports
-      # coverage, the `rspec` job above does.
+      # coverage, the `ruby-tests` job above does.
       - name: Run integration tests
         run: NO_COVERAGE=1 bin/turbo_tests -t integration spec/integration
 
@@ -271,7 +286,7 @@ jobs:
     runs-on: ubuntu-24.04
     timeout-minutes: 5
 
-    needs: [lint, bats, rspec, rspec-integration, build]
+    needs: [lint, js-tests, shell-tests, ruby-tests, integration-tests, build]
     if: always()
 
     steps: