ci: make release workflow depend on test workflow success

- Use workflow_run trigger to only run release after tests pass
- Update Node.js version to 22 in release workflow
- Fix job name to correctly show Node 22 in test workflow
- Remove manual wait-for-tests logic since workflow_run handles it

Author: amilajack

.github/workflows/release.yml

@@ -1,7 +1,10 @@
 name: Release
 
 on:
-  push:
+  workflow_run:
+    workflows: ["Test"]
+    types:
+      - completed
     branches:
       - main
   workflow_dispatch:
@@ -29,60 +32,12 @@ jobs:
     name: Release
     runs-on: ubuntu-latest
     needs: [authorize]
-    if: always() && (needs.authorize.result == 'success' || needs.authorize.result == 'skipped')
+    if: |
+      always() && 
+      (needs.authorize.result == 'success' || needs.authorize.result == 'skipped') &&
+      (github.event_name == 'workflow_dispatch' || github.event.workflow_run.conclusion == 'success')
 
     steps:
-      # Wait for tests to complete successfully before releasing
-      - name: Wait for Test workflow to succeed
-        if: github.event_name == 'push'
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const owner = context.repo.owner;
-            const repo = context.repo.repo;
-            const sha = context.sha;
-            
-            // Wait for test workflow checks to complete
-            let attempts = 0;
-            const maxAttempts = 60; // 10 minutes max wait
-            
-            while (attempts < maxAttempts) {
-              const { data: checks } = await github.rest.checks.listForRef({
-                owner,
-                repo,
-                ref: sha,
-              });
-              
-              const testChecks = checks.check_runs.filter(check => 
-                check.name.startsWith('Test (Node 20, ESLint')
-              );
-              
-              if (testChecks.length === 0) {
-                console.log('No test checks found yet, waiting...');
-              } else {
-                const allCompleted = testChecks.every(check => check.status === 'completed');
-                const allSucceeded = testChecks.every(check => check.conclusion === 'success');
-                
-                if (allCompleted) {
-                  if (allSucceeded) {
-                    console.log('All tests passed successfully!');
-                    break;
-                  } else {
-                    core.setFailed('Tests failed - cannot proceed with release');
-                    return;
-                  }
-                }
-                console.log(`Waiting for tests to complete... (${testChecks.filter(c => c.status === 'completed').length}/${testChecks.length})`);
-              }
-              
-              await new Promise(resolve => setTimeout(resolve, 10000)); // Wait 10 seconds
-              attempts++;
-            }
-            
-            if (attempts >= maxAttempts) {
-              core.setFailed('Timeout waiting for tests to complete');
-            }
-
       - name: Checkout
         uses: actions/checkout@v4
         with:
@@ -92,7 +47,7 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: 22
           cache: npm
 
       - name: Install dependencies

.github/workflows/test.yml

@@ -7,14 +7,14 @@ jobs:
       matrix:
         os: [macos-latest, ubuntu-latest, windows-latest]
         eslint-version: [8, 9]
-    name: Test (Node 20, ESLint ${{ matrix.eslint-version }}, ${{ matrix.os }})
+    name: Test (Node 22, ESLint ${{ matrix.eslint-version }}, ${{ matrix.os }})
     steps:
       - name: Check out Git repository
         uses: actions/checkout@v4
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: 22
           cache: npm
       - name: Install dependencies
         run: npm ci