Fix workflow race conditions and test triggering

Issue 1: Multiple label additions triggered duplicate version-bump runs
Solution: Add concurrency control to cancel in-progress runs per PR

Issue 2: Test workflow never ran after version bump (GITHUB_TOKEN commits
don't trigger workflows), blocking auto-merge
Solution: Add workflow_run trigger to test.yml to run after version-bump
completes successfully

Changes:
- version-bump.yml: Add concurrency group by PR number with cancel-in-progress
- version-bump.yml: Remove [skip ci] (not needed, doesn't work anyway)
- test.yml: Add workflow_run trigger from Version Bump workflow
- test.yml: Remove skip ci checking logic (not needed)
- test.yml: Checkout correct SHA for both pull_request and workflow_run events

Now: Version bump → test runs automatically → auto-merge proceeds

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: kompre

.github/workflows/test.yml

@@ -3,54 +3,60 @@ name: Run Tests
 on:
   pull_request:
     branches: [ main ]
+  workflow_run:
+    workflows: ["Version Bump"]
+    types: [completed]
 
 permissions:
   contents: read
 
 jobs:
   test:
+    # Only run on PRs or successful version bump completions
+    if: |
+      github.event_name == 'pull_request' ||
+      (github.event_name == 'workflow_run' && github.event.workflow_run.conclusion == 'success')
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 2
-
-      - name: Check for skip ci
-        id: check-skip
+      - name: Get PR details
+        if: github.event_name == 'workflow_run'
+        id: pr
+        env:
+          GH_TOKEN: ${{ github.token }}
         run: |
-          COMMIT_MSG=$(git log -1 --pretty=%B)
-          if echo "$COMMIT_MSG" | grep -q "\[skip ci\]"; then
-            echo "skip=true" >> $GITHUB_OUTPUT
-            echo "Skipping tests due to [skip ci] in commit message"
-          else
-            echo "skip=false" >> $GITHUB_OUTPUT
-          fi
+          # Get PR number from the workflow run
+          PR_NUMBER=$(gh api repos/${{ github.repository }}/pulls \
+            --jq ".[] | select(.head.sha == \"${{ github.event.workflow_run.head_sha }}\") | .number" \
+            | head -1)
+
+          echo "number=$PR_NUMBER" >> $GITHUB_OUTPUT
+          echo "PR number: $PR_NUMBER"
+
+      - name: Checkout PR branch
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event_name == 'workflow_run' && github.event.workflow_run.head_sha || github.event.pull_request.head.sha }}
 
       - name: Set up Python
-        if: steps.check-skip.outputs.skip != 'true'
         uses: actions/setup-python@v5
         with:
           python-version-file: ".python-version"
 
       - name: Install uv
-        if: steps.check-skip.outputs.skip != 'true'
         uses: astral-sh/setup-uv@v6
         with:
           version: "latest"
           enable-cache: true
           cache-dependency-glob: "uv.lock"
 
       - name: Install Quarto
-        if: steps.check-skip.outputs.skip != 'true'
         uses: quarto-dev/quarto-actions/setup@v2
         with:
           version: 1.8.25
 
       - name: Install dependencies
-        if: steps.check-skip.outputs.skip != 'true'
         run: uv sync --locked --all-extras --dev
 
       - name: Run tests
-        if: steps.check-skip.outputs.skip != 'true'
         run: uv run pytest tests -v

.github/workflows/version-bump.yml

@@ -5,6 +5,10 @@ on:
     types: [labeled]
     branches: [main]
 
+concurrency:
+  group: version-bump-${{ github.event.pull_request.number }}
+  cancel-in-progress: true
+
 jobs:
   bump-version:
     # Only run if PR has at least one bump: label
@@ -98,7 +102,7 @@ jobs:
           if git diff --staged --quiet; then
             echo "No changes to commit"
           else
-            git commit -m "Bump version to ${{ steps.bump.outputs.new_version }} [skip ci]"
+            git commit -m "Bump version to ${{ steps.bump.outputs.new_version }}"
 
             # Pull with rebase in case another workflow pushed
             git pull --rebase origin ${{ github.head_ref }}