From 75f8bcae550748eeeb5c5575d1712498a23db15e Mon Sep 17 00:00:00 2001
From: Ben Dichter <ben.dichter@gmail.com>
Date: Wed, 13 May 2026 14:10:58 -0400
Subject: [PATCH] CI: improve manual-dispatch UX for the all-notebooks sweep
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The workflow already had a workflow_dispatch trigger (added in PR #153),
but the only input was the regex filter — which is awkward for the
common ad-hoc use cases. Three improvements:

- Rename to 'Test all notebooks (weekly + on-demand)' so it's obvious
  from the Actions page that manual dispatch is supported.

- Add `single_notebook` input — paste a notebook path to test just
  that one, no regex needed. Overrides the filter if set.

- Add `file_issue` toggle (true/false dropdown, default true). On
  scheduled weekly runs the tracking issue gets filed/updated as
  before; on ad-hoc dispatch a user can set it to false to avoid
  noisy issue comments while iterating.

How to use:

  Actions tab → 'Test all notebooks (weekly + on-demand)' →
    'Run workflow' button (top right) → fill in inputs → Run.

Or from the CLI:

  gh workflow run 'Test all notebooks (weekly + on-demand)' \
    -f single_notebook='000582/Sargolini2006/000582_Sargolini2006_demo.ipynb' \
    -f file_issue=false

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../workflows/test-all-notebooks-weekly.yml   | 30 +++++++++++++++----
 1 file changed, 24 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/test-all-notebooks-weekly.yml b/.github/workflows/test-all-notebooks-weekly.yml
index 9f18fa7..3f5cfd9 100644
--- a/.github/workflows/test-all-notebooks-weekly.yml
+++ b/.github/workflows/test-all-notebooks-weekly.yml
@@ -1,4 +1,4 @@
-name: Test all notebooks (weekly)
+name: Test all notebooks (weekly + on-demand)
 
 on:
   schedule:
@@ -7,9 +7,21 @@ on:
   workflow_dispatch:
     inputs:
       filter:
-        description: 'Optional substring filter on notebook paths (regex)'
+        description: 'Regex on notebook paths (leave blank for all). Example: tutorials/.*'
         required: false
         default: ''
+      single_notebook:
+        description: 'Run a single notebook (full path). Overrides filter if set.'
+        required: false
+        default: ''
+      file_issue:
+        description: 'File/update the weekly-notebook-sweep tracking issue on failure'
+        required: false
+        default: 'true'
+        type: choice
+        options:
+          - 'true'
+          - 'false'
 
 permissions:
   contents: read
@@ -32,15 +44,21 @@ jobs:
         id: list
         run: |
           set -euo pipefail
-          all=$(python .github/scripts/list_notebooks.py)
+          single='${{ github.event.inputs.single_notebook }}'
           filter='${{ github.event.inputs.filter }}'
-          if [ -n "$filter" ]; then
-            all=$(echo "$all" | python -c "
+          if [ -n "$single" ]; then
+            all=$(python -c "import json; print(json.dumps(['$single']))")
+            echo "Manual single-notebook run: $single"
+          else
+            all=$(python .github/scripts/list_notebooks.py)
+            if [ -n "$filter" ]; then
+              all=$(echo "$all" | python -c "
           import json, re, sys
           pat = re.compile(r'''$filter''')
           nbs = json.load(sys.stdin)
           print(json.dumps([n for n in nbs if pat.search(n)]))
           ")
+            fi
           fi
           echo "notebooks=$all" >> "$GITHUB_OUTPUT"
           count=$(echo "$all" | python -c "import json,sys;print(len(json.load(sys.stdin)))")
@@ -161,7 +179,7 @@ jobs:
           EOF
 
       - name: File or update tracking issue if any failures
-        if: steps.report.outputs.failed_count != '0'
+        if: steps.report.outputs.failed_count != '0' && github.event.inputs.file_issue != 'false'
         uses: actions/github-script@v7
         with:
           script: |