Update CI triggering mechanism (#222)

.github/workflows/examples-apo.yml

@@ -8,27 +8,15 @@ on:
 
   workflow_dispatch:
 
-  pull_request_target:
-    types: [reopened, ready_for_review]
+  repository_dispatch:
+    types: [ci-apo, ci-all]
 
 jobs:
-  label-check:
-    runs-on: ubuntu-latest
-    outputs:
-      should-run: ${{ steps.evaluate.outputs.should-run }}
-    steps:
-      - uses: actions/checkout@v4
-      - name: Decide whether to run
-        id: evaluate
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const evaluateTrigger = require('./scripts/trigger_on_label.js');
-            evaluateTrigger({ core, context, labelName: 'ci-apo' });
-
   apo:
-    needs: label-check
-    if: needs.label-check.outputs.should-run == 'true'
+    if: >
+      github.event_name != 'repository_dispatch' ||
+      github.event.action == 'ci-apo' ||
+      github.event.action == 'ci-all'
     name: APO (Python ${{ matrix.python-version }}, ${{ matrix.setup-script }})
     # This job is run on GitHub hosted runners rather than self-hosted runners because it needs no GPU.
     runs-on: ubuntu-latest
@@ -46,7 +34,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          ref: ${{ github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number) || github.ref }}
+          ref: ${{ github.event_name == 'repository_dispatch' && github.event.client_payload.pr_ref || (github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number)) || github.ref }}
       - uses: astral-sh/setup-uv@v7
         with:
           enable-cache: true

.github/workflows/examples-calc-x.yml

@@ -8,27 +8,15 @@ on:
 
   workflow_dispatch:
 
-  pull_request_target:
-    types: [reopened, ready_for_review]
+  repository_dispatch:
+    types: [ci-calc-x, ci-all]
 
 jobs:
-  label-check:
-    runs-on: ubuntu-latest
-    outputs:
-      should-run: ${{ steps.evaluate.outputs.should-run }}
-    steps:
-      - uses: actions/checkout@v4
-      - name: Decide whether to run
-        id: evaluate
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const evaluateTrigger = require('./scripts/trigger_on_label.js');
-            evaluateTrigger({ core, context, labelName: 'ci-calc-x' });
-
   calc-x:
-    needs: label-check
-    if: needs.label-check.outputs.should-run == 'true'
+    if: >
+      github.event_name != 'repository_dispatch' ||
+      github.event.action == 'ci-calc-x' ||
+      github.event.action == 'ci-all'
     name: Calc-X (Python ${{ matrix.python-version }}, ${{ matrix.setup-script }})
     runs-on: [self-hosted, 1ES.Pool=agl-runner-gpu]
     timeout-minutes: 90
@@ -49,7 +37,7 @@ jobs:
         run: df -h
       - uses: actions/checkout@v4
         with:
-          ref: ${{ github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number) || github.ref }}
+          ref: ${{ github.event_name == 'repository_dispatch' && github.event.client_payload.pr_ref || (github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number)) || github.ref }}
       - uses: astral-sh/setup-uv@v7
         with:
           enable-cache: true

.github/workflows/examples-compat.yml

@@ -8,27 +8,15 @@ on:
 
   workflow_dispatch:
 
-  pull_request_target:
-    types: [reopened, ready_for_review]
+  repository_dispatch:
+    types: [ci-compat, ci-all]
 
 jobs:
-  label-check:
-    runs-on: ubuntu-latest
-    outputs:
-      should-run: ${{ steps.evaluate.outputs.should-run }}
-    steps:
-      - uses: actions/checkout@v4
-      - name: Decide whether to run
-        id: evaluate
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const evaluateTrigger = require('./scripts/trigger_on_label.js');
-            evaluateTrigger({ core, context, labelName: 'ci-compat' });
-
   backward-compatibility:
-    needs: label-check
-    if: needs.label-check.outputs.should-run == 'true'
+    if: >
+      github.event_name != 'repository_dispatch' ||
+      github.event.action == 'ci-compat' ||
+      github.event.action == 'ci-all'
     name: Backward Compatibility (Python ${{ matrix.python-version }}, ${{ matrix.setup-script }})
     runs-on: [self-hosted, 1ES.Pool=agl-runner-gpu]
     timeout-minutes: 30
@@ -47,7 +35,7 @@ jobs:
         run: df -h
       - uses: actions/checkout@v4
         with:
-          ref: ${{ github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number) || github.ref }}
+          ref: ${{ github.event_name == 'repository_dispatch' && github.event.client_payload.pr_ref || (github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number)) || github.ref }}
       - uses: astral-sh/setup-uv@v7
         with:
           enable-cache: true

.github/workflows/examples-spider.yml

@@ -8,27 +8,15 @@ on:
 
   workflow_dispatch:
 
-  pull_request_target:
-    types: [reopened, ready_for_review]
+  repository_dispatch:
+    types: [ci-spider, ci-all]
 
 jobs:
-  label-check:
-    runs-on: ubuntu-latest
-    outputs:
-      should-run: ${{ steps.evaluate.outputs.should-run }}
-    steps:
-      - uses: actions/checkout@v4
-      - name: Decide whether to run
-        id: evaluate
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const evaluateTrigger = require('./scripts/trigger_on_label.js');
-            evaluateTrigger({ core, context, labelName: 'ci-spider' });
-
   spider:
-    needs: label-check
-    if: needs.label-check.outputs.should-run == 'true'
+    if: >
+      github.event_name != 'repository_dispatch' ||
+      github.event.action == 'ci-spider' ||
+      github.event.action == 'ci-all'
     name: Spider (Python ${{ matrix.python-version }}, ${{ matrix.setup-script }})
     runs-on: [self-hosted, 1ES.Pool=agl-runner-gpu]
     timeout-minutes: 60
@@ -49,7 +37,7 @@ jobs:
         run: df -h
       - uses: actions/checkout@v4
         with:
-          ref: ${{ github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number) || github.ref }}
+          ref: ${{ github.event_name == 'repository_dispatch' && github.event.client_payload.pr_ref || (github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number)) || github.ref }}
       - uses: astral-sh/setup-uv@v7
         with:
           enable-cache: true

.github/workflows/examples-unsloth.yml

@@ -8,27 +8,15 @@ on:
 
   workflow_dispatch:
 
-  pull_request_target:
-    types: [reopened, ready_for_review]
+  repository_dispatch:
+    types: [ci-unsloth, ci-all]
 
 jobs:
-  label-check:
-    runs-on: ubuntu-latest
-    outputs:
-      should-run: ${{ steps.evaluate.outputs.should-run }}
-    steps:
-      - uses: actions/checkout@v4
-      - name: Decide whether to run
-        id: evaluate
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const evaluateTrigger = require('./scripts/trigger_on_label.js');
-            evaluateTrigger({ core, context, labelName: 'ci-unsloth' });
-
   unsloth:
-    needs: label-check
-    if: needs.label-check.outputs.should-run == 'true'
+    if: >
+      github.event_name != 'repository_dispatch' ||
+      github.event.action == 'ci-unsloth' ||
+      github.event.action == 'ci-all'
     name: Unsloth (Python ${{ matrix.python-version }}, ${{ matrix.setup-script }})
     runs-on: [self-hosted, 1ES.Pool=agl-runner-gpu]
     timeout-minutes: 60
@@ -48,7 +36,7 @@ jobs:
         run: df -h
       - uses: actions/checkout@v4
         with:
-          ref: ${{ github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number) || github.ref }}
+          ref: ${{ github.event_name == 'repository_dispatch' && github.event.client_payload.pr_ref || (github.event.pull_request.number && format('refs/pull/{0}/merge', github.event.pull_request.number)) || github.ref }}
       - uses: astral-sh/setup-uv@v7
         with:
           enable-cache: true

.github/workflows/issue-comment.yml

@@ -7,9 +7,10 @@ on:
 permissions:
   pull-requests: write
   issues: write
+  contents: write
 
 jobs:
-  reopen:
+  dispatch:
     # Only run for comments on pull requests AND when the comment starts with "/ci"
     if: >
       github.event.issue.pull_request != null &&
@@ -28,7 +29,8 @@ jobs:
               core.setOutput('skip', 'true');
             }
 
-      - name: Close and Reopen PR
+      - name: Trigger repository dispatch
+        id: dispatch
         if: steps.guard.outputs.skip != 'true'
         uses: actions/github-script@v8
         with:
@@ -36,6 +38,7 @@ jobs:
             const owner = context.repo.owner;
             const repo = context.repo.repo;
             const pull_number = context.payload.issue.number;
+            const comment = context.payload.comment;
 
             // Fetch current PR state
             const { data: pr } = await github.rest.pulls.get({ owner, repo, pull_number });
@@ -43,33 +46,87 @@ jobs:
             // Add reaction so folks know we saw it
             try {
               await github.rest.reactions.createForIssueComment({
-                owner, repo,
-                comment_id: context.payload.comment.id,
+                owner,
+                repo,
+                comment_id: comment.id,
                 content: 'rocket'
               });
             } catch (e) {
               core.info('Could not add reaction (likely due to permissions). Continuing.');
             }
 
-            if (pr.state === 'open') {
-              core.info(`Closing PR #${pull_number}…`);
-              await github.rest.pulls.update({ owner, repo, pull_number, state: 'closed' });
-              await new Promise(r => setTimeout(r, 1500));
-            } else {
-              core.info(`PR #${pull_number} is already closed; will reopen it.`);
+            const labels = (pr.labels ?? []).map(label => label.name);
+            const directCiLabels = labels.filter(label => label.startsWith('ci-'));
+            const hasCiAll = directCiLabels.includes('ci-all');
+            const dedupe = new Set(
+              directCiLabels.filter(label => label !== 'ci-all')
+            );
+
+            if (!hasCiAll && dedupe.size === 0) {
+              core.notice('No ci-* labels found on the pull request; nothing to dispatch.');
+              core.setOutput('dispatched', 'false');
+              core.setOutput('event_types', '');
+              return;
             }
 
-            core.info(`Reopening PR #${pull_number} to trigger CI (pull_request.reopened)…`);
-            await github.rest.pulls.update({ owner, repo, pull_number, state: 'open' });
+            const clientPayload = {
+              pull_number,
+              pr_ref: `refs/pull/${pull_number}/merge`,
+              pr_head_ref: pr.head.ref,
+              pr_head_sha: pr.head.sha,
+              pr_base_ref: pr.base.ref,
+              pr_base_sha: pr.base.sha,
+              trigger_comment_id: comment.id,
+              trigger_comment_user: comment.user.login,
+              trigger_comment_body: comment.body,
+              trigger_comment_association: comment.author_association
+            };
+
+            const eventTypes = hasCiAll
+              ? ['ci-all']
+              : Array.from(dedupe);
+            for (const eventType of eventTypes) {
+              await github.rest.repos.createDispatchEvent({
+                owner,
+                repo,
+                event_type: eventType,
+                client_payload: { ...clientPayload, ci_label: eventType }
+              });
+              core.notice(`Dispatched '${eventType}' event for PR #${pull_number}.`);
+            }
+
+            core.setOutput('dispatched', 'true');
+            core.setOutput('event_types', eventTypes.join(','));
 
       - name: Acknowledge in thread (optional)
-        if: steps.guard.outputs.skip != 'true'
+        if: steps.guard.outputs.skip != 'true' && steps.dispatch.outputs.dispatched == 'true'
+        uses: actions/github-script@v8
+        env:
+          EVENT_TYPES: ${{ steps.dispatch.outputs.event_types }}
+        with:
+          script: |
+            const eventTypes = (process.env.EVENT_TYPES || '')
+              .split(',')
+              .map(label => label.trim())
+              .filter(Boolean);
+            const formatted = eventTypes.map(label => `\`repository_dispatch:${label}\``).join(', ');
+            const { owner, repo } = context.repo;
+            const issue_number = context.payload.issue.number;
+            await github.rest.issues.createComment({
+              owner, repo, issue_number,
+              body: `✅ CI retrigger requested by @${context.payload.comment.user.login}. Fired ${formatted}.`
+            });
+
+      - name: Notify missing ci label
+        if: steps.guard.outputs.skip != 'true' && steps.dispatch.outputs.dispatched != 'true'
         uses: actions/github-script@v8
         with:
           script: |
             const { owner, repo } = context.repo;
             const issue_number = context.payload.issue.number;
             await github.rest.issues.createComment({
-              owner, repo, issue_number,
-              body: `✅ CI retrigger requested by @${context.payload.comment.user.login}. Closed & reopened the PR to fire \`pull_request: reopened\`.`
+              owner,
+              repo,
+              issue_number,
+              body: `⚠️ CI retrigger ignored because the pull request has no \`ci-*\` labels (e.g. \`ci-apo\`, \`ci-calc-x\`). Add the desired labels and try \`/ci\` again.`
             });

.github/workflows/tests-full.yml

@@ -8,27 +8,15 @@ on:
 
   workflow_dispatch:
 
-  pull_request_target:
-    types: [reopened, ready_for_review]
+  repository_dispatch:
+    types: [ci-unsloth, ci-all]
 
 jobs:
-  label-check:
-    runs-on: ubuntu-latest
-    outputs:
-      should-run: ${{ steps.evaluate.outputs.should-run }}
-    steps:
-      - uses: actions/checkout@v4
-      - name: Decide whether to run
-        id: evaluate
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const evaluateTrigger = require('./scripts/trigger_on_label.js');
-            evaluateTrigger({ core, context, labelName: 'ci-gpu' });
-
   tests-full:
-    needs: label-check
-    if: needs.label-check.outputs.should-run == 'true'
+    if: >
+      github.event_name != 'repository_dispatch' ||
+      github.event.action == 'ci-gpu' ||
+      github.event.action == 'ci-all'
     name: GPU Test with Python ${{ matrix.python-version }} (${{ matrix.setup-script }})
 
     runs-on: [self-hosted, 1ES.Pool=agl-runner-gpu]