test(ci): improve release workflow with better error handling and remove problematic conditional

Author: grdsdev

.github/workflows/release.yml

@@ -1,31 +1,22 @@
 name: Release
 
 on:
-  push:
+  workflow_run:
+    workflows: [CI]
+    types: [completed]
     branches:
       - main
       - rc
+
   workflow_dispatch:
 
 permissions:
   contents: read
 
 jobs:
-  check-ci-status:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Wait for CI workflow
-        uses: lewagon/[email protected]
-        with:
-          ref: ${{ github.ref }}
-          check-name: 'CI'
-          repo-token: ${{ secrets.GITHUB_TOKEN }}
-          wait-interval: 30
-
   release:
     runs-on: ubuntu-latest
-    if: "!contains(github.event.head_commit.message, 'skip ci')"
-    needs: [check-ci-status]
+    # Remove the problematic conditional since workflow_run events don't have direct access to commit messages
     permissions:
       contents: write
       issues: write
@@ -40,6 +31,7 @@ jobs:
         with:
           app-id: ${{ secrets.APP_ID }}
           private-key: ${{ secrets.PRIVATE_KEY }}
+      
       - name: Checkout
         uses: actions/checkout@v4
         with:
@@ -57,6 +49,12 @@ jobs:
         run: npm ci
 
       - name: Run semantic-release
+        id: semantic-release
         run: npx semantic-release
         env:
           GITHUB_TOKEN: ${{ steps.app-token.outputs.token }}
+        continue-on-error: false
+
+      - name: Check if release was created
+        if: steps.semantic-release.outcome == 'success'
+        run: echo "Release created successfully"