cleaner actions

Author: radenkovic

.github/workflows/pr-tests.yml

@@ -4,38 +4,37 @@ on:
   pull_request:
     branches: [main]
     types: [opened, synchronize, reopened]
-  # Also run on pushes to main for consistency
-  push:
-    branches: [main]
 
 # Cancel in-progress runs for the same PR
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
   cancel-in-progress: true
 
+env:
+  NODE_VERSION: ${{ vars.NODE_VERSION }}
+  PNPM_VERSION: ${{ vars.PNPM_VERSION }}
+
 jobs:
-  test:
-    name: Test Suite
+  # Code quality checks (fast, no build required)
+  quality:
+    name: Code Quality
     runs-on: ubuntu-latest
-    timeout-minutes: 15
-
+    timeout-minutes: 10
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
         with:
-          # Fetch full history for Nx affected commands (optional optimization)
           fetch-depth: 0
 
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: ${{ vars.NODE_VERSION }}
-          registry-url: "https://registry.npmjs.org"
+          node-version: ${{ env.NODE_VERSION }}
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
         with:
-          version: "10.6.3"
+          version: ${{ env.PNPM_VERSION }}
           run_install: false
 
       - name: Get pnpm store directory
@@ -63,6 +62,44 @@ jobs:
       - name: Check formatting
         run: pnpm format:check
 
+  # Build and test (slower, runs in parallel with quality)
+  test:
+    name: Build & Test
+    runs-on: ubuntu-latest
+    timeout-minutes: 15
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ env.NODE_VERSION }}
+
+      - name: Setup pnpm
+        uses: pnpm/action-setup@v4
+        with:
+          version: ${{ env.PNPM_VERSION }}
+          run_install: false
+
+      - name: Get pnpm store directory
+        shell: bash
+        run: |
+          echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV
+
+      - name: Setup pnpm cache
+        uses: actions/cache@v4
+        with:
+          path: ${{ env.STORE_PATH }}
+          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
+          restore-keys: |
+            ${{ runner.os }}-pnpm-store-
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile
+
       - name: Build packages
         run: pnpm build
 

.github/workflows/release.yml

@@ -9,6 +9,10 @@ on:
 # Prevent multiple releases from running at the same time
 concurrency: ${{ github.workflow }}-${{ github.ref }}
 
+env:
+  NODE_VERSION: ${{ vars.NODE_VERSION }}
+  PNPM_VERSION: ${{ vars.PNPM_VERSION }}
+
 jobs:
   release:
     name: Release
@@ -31,13 +35,13 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: ${{ vars.NODE_VERSION }}
+          node-version: ${{ env.NODE_VERSION }}
           registry-url: "https://registry.npmjs.org"
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
         with:
-          version: "10.6.3"
+          version: ${{ env.PNPM_VERSION }}
           run_install: false
 
       - name: Get pnpm store directory

.github/workflows/snapshot.yml

@@ -32,8 +32,8 @@ on:
 run-name: "Publishing ${{ inputs.snapshot_tag }} for ${{ inputs.packages }} to ${{ inputs.environment }} by @${{ github.actor }}"
 
 env:
-    NODE_VERSION: 22.12.0
-    PNPM_VERSION: 10.7.1
+    NODE_VERSION: ${{ vars.NODE_VERSION }}
+    PNPM_VERSION: ${{ vars.PNPM_VERSION }}
     IS_CI: true
 
 permissions:
@@ -52,7 +52,7 @@ jobs:
                   fetch-depth: 0
 
             # Setup Node.js and pnpm
-            - uses: pnpm/action-setup@v3
+            - uses: pnpm/action-setup@v4
               with:
                   version: ${{ env.PNPM_VERSION }}
                   run_install: false