ci: skip dev-build for documentation-only and release commits

Author: d0ugal

.github/workflows/ci.yml

@@ -116,11 +116,96 @@ jobs:
         exit-code: '1'
         severity: 'CRITICAL,HIGH'
 
+  check-build-needed:
+    name: Check if build is needed
+    runs-on: ubuntu-latest
+    needs: [test]
+    if: ${{ github.event_name != 'workflow_run' || github.event.workflow_run.conclusion == 'success' }}
+    outputs:
+      should-build: ${{ steps.check.outputs.should-build }}
+    steps:
+    - name: Checkout code
+      uses: actions/checkout@v5
+      with:
+        fetch-depth: 0
+        ref: ${{ github.event.workflow_run.head_branch || github.ref }}
+        sha: ${{ github.event.workflow_run.head_sha || github.sha }}
+
+    - name: Check if build should be skipped
+      id: check
+      run: |
+        # Only check for push and workflow_run events (dev-build only runs on these)
+        if [ "${{ github.event_name }}" != "push" ] && [ "${{ github.event_name }}" != "workflow_run" ]; then
+          echo "should-build=false" >> $GITHUB_OUTPUT
+          exit 0
+        fi
+        
+        # Get the current commit SHA
+        CURRENT_SHA="${{ github.event.workflow_run.head_sha || github.sha }}"
+        
+        # Get commit message from git log (works for both push and workflow_run)
+        COMMIT_MSG=$(git log -1 --format="%s" "$CURRENT_SHA" 2>/dev/null || echo "")
+        
+        echo "Checking commit: $CURRENT_SHA"
+        echo "Commit message: $COMMIT_MSG"
+        
+        # Check if it's a release commit (pattern: "chore(main): release" or similar)
+        if echo "$COMMIT_MSG" | grep -qiE "^(chore|release|version).*(release|version|changelog)"; then
+          echo "Skip reason: Release commit detected"
+          echo "should-build=false" >> $GITHUB_OUTPUT
+          exit 0
+        fi
+        
+        # For push events, check changed files against base branch
+        if [ "${{ github.event_name }}" = "push" ]; then
+          # Get the base commit (previous commit for single commits, or base of range)
+          BASE_SHA="${{ github.event.before }}"
+          HEAD_SHA="${{ github.event.after }}"
+          
+          if [ -z "$BASE_SHA" ] || [ "$BASE_SHA" = "0000000000000000000000000000000000000000" ]; then
+            # Initial commit or no base, check all files
+            CHANGED_FILES=$(git ls-tree -r --name-only HEAD)
+          else
+            # Get list of changed files
+            CHANGED_FILES=$(git diff --name-only "$BASE_SHA" "$HEAD_SHA")
+          fi
+        else
+          # For workflow_run events, we need to compare against main branch
+          BASE_SHA="origin/main"
+          HEAD_SHA="$CURRENT_SHA"
+          git fetch origin main || true
+          CHANGED_FILES=$(git diff --name-only "$BASE_SHA" "$HEAD_SHA" 2>/dev/null || git diff --name-only HEAD~1 HEAD)
+        fi
+        
+        if [ -z "$CHANGED_FILES" ]; then
+          echo "No changed files detected, skipping build"
+          echo "should-build=false" >> $GITHUB_OUTPUT
+          exit 0
+        fi
+        
+        # Documentation file patterns to ignore
+        DOC_PATTERNS="^README\.md$|^CHANGELOG\.md$|^docs/|\.md$|^LICENSE"
+        
+        # Check if all changed files are documentation
+        NON_DOC_FILES=$(echo "$CHANGED_FILES" | grep -vE "$DOC_PATTERNS" || true)
+        
+        if [ -z "$NON_DOC_FILES" ]; then
+          echo "Skip reason: Only documentation files changed"
+          echo "Changed files:"
+          echo "$CHANGED_FILES" | sed 's/^/  - /'
+          echo "should-build=false" >> $GITHUB_OUTPUT
+        else
+          echo "Build needed: Non-documentation files changed"
+          echo "Non-doc files:"
+          echo "$NON_DOC_FILES" | sed 's/^/  - /'
+          echo "should-build=true" >> $GITHUB_OUTPUT
+        fi
+
   dev-build:
     name: Development Build
     runs-on: ubuntu-latest
-    needs: [test]
-    if: github.event_name == 'push' || (github.event_name == 'workflow_run' && github.event.workflow_run.conclusion == 'success')
+    needs: [test, check-build-needed]
+    if: needs.check-build-needed.outputs.should-build == 'true'
     permissions:
       contents: read
       packages: write