Fix dry-run pagination

Author: Mirko-von-Leipzig

.github/workflows/cleanup-workflows.yml

@@ -109,6 +109,8 @@ jobs:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           MODE: ${{ inputs.mode }}
           DELETED_WORKFLOWS: ${{ steps.deleted.outputs.workflows }}
+          OWNER: ${{ github.repository_owner }}
+          REPO: ${{ github.repository }}
         run: |
           set -euo pipefail
 
@@ -135,18 +137,18 @@ jobs:
           # Total table width
           TOTAL_WIDTH=$(( INDEX_WIDTH + 3 + MAX_WF_LENGTH + 3 + WORKFLOW_COUNT_WIDTH + 3 + GLOBAL_TOTAL_WIDTH ))
 
-          # Function to print dynamic table headers padded with '=' and spaces
+          # Function to print dynamic table headers padded with '=' and spaces around the name
           print_header() {
             local name="$1"
             local name_len=${#name}
-            # Add 2 for the spaces around the name
             local pad=$(( (TOTAL_WIDTH - name_len - 2) / 2 ))
             local left_pad=$pad
             local right_pad=$(( TOTAL_WIDTH - name_len - 2 - left_pad ))
 
             left_str=$(printf '=%0.s' $(seq 1 $left_pad))
             right_str=$(printf '=%0.s' $(seq 1 $right_pad))
 
+            printf " \n"
             printf "%s %s %s\n" "$left_str" "$name" "$right_str"
           }
 
@@ -171,18 +173,28 @@ jobs:
             CURRENT_INDEX=$((i + 1))
             WORKFLOW_COUNT=0
 
+            # Resolve workflow ID using gh workflow list (legible)
+            WORKFLOW_ID=$(gh workflow list --all --json id,path \
+                          --jq ".[] | select(.path==\"$workflow\") | .id" 2>/dev/null || true)
+
+            if [ -z "$WORKFLOW_ID" ]; then
+              echo " ⚠️ Workflow not found: $workflow, skipping..."
+              SUMMARY+=("$workflow|0")
+              continue
+            fi
+
+            # GraphQL pagination variables
+            AFTER_CURSOR=null
+            PER_PAGE=100
+
             while true; do
-              # List workflow runs.
-              #
-              # Note that github will return 404 once all runs are deleted.
-              # We use `|| true` to ignore this error since it is expected when all runs are deleted.
-              RUN_IDS=$(gh run list \
-                --workflow "$workflow" \
-                --limit 100 \
-                --json databaseId \
-                --jq '.[].databaseId' 2>/dev/null || true)
-
-              [ -z "$RUN_IDS" ] && break  # stop when no runs remain
+              # GraphQL query to fetch workflow runs
+              QUERY=$(printf 'query { repository(owner: "%s", name: "%s") { workflow(id: "%s") { runs(first: 100, after: %s) { pageInfo { hasNextPage endCursor } nodes { databaseId } } } } }' \
+                "$OWNER" "$REPO" "$WORKFLOW_ID" "$AFTER_CURSOR")
+              RESPONSE=$(gh api graphql -f query="$QUERY")
+
+              RUN_IDS=$(echo "$RESPONSE" | jq -r '.data.repository.workflow.runs.nodes[].databaseId')
+              [ -z "$RUN_IDS" ] && break
 
               BATCH_COUNT=$(echo "$RUN_IDS" | wc -l | tr -d ' ')
               WORKFLOW_COUNT=$((WORKFLOW_COUNT + BATCH_COUNT))
@@ -201,22 +213,24 @@ jobs:
                 done
               fi
 
-              # TEMPORARY BREAK for testing
-              if [ "$WORKFLOW_COUNT" -gt 20 ]; then
-                echo " ⚠️ Temporary break: workflow count exceeded 20, stopping early for safety."
+              # Prepare next page
+              AFTER_CURSOR=$(echo "$RESPONSE" | jq -r '.data.repository.workflow.runs.pageInfo.endCursor')
+              HAS_NEXT=$(echo "$RESPONSE" | jq -r '.data.repository.workflow.runs.pageInfo.hasNextPage')
+              [ "$HAS_NEXT" != "true" ] && break
+
+              # TEMPORARY break for testing
+              if [ "$WORKFLOW_COUNT" -gt 200 ]; then
+                echo " ⚠️ Temporary break: workflow count exceeded 200, stopping early for safety."
                 break
               fi
             done
 
-            # Save summary
             SUMMARY+=("$workflow|$WORKFLOW_COUNT")
           done
 
           # === Summary Header ===
-          printf " \n"
           print_header "Workflow Cleanup Summary"
 
-          # Summary table uses the same total width as progress table
           printf "%*s | %-*s | %-*s | %-*s\n" \
             "$INDEX_WIDTH" "" \
             "$MAX_WF_LENGTH" "Workflow" \
@@ -229,7 +243,7 @@ jobs:
             "$WORKFLOW_COUNT_WIDTH" "$(printf '%.0s-' $(seq 1 $WORKFLOW_COUNT_WIDTH))" \
             "$GLOBAL_TOTAL_WIDTH" "$(printf '%.0s-' $(seq 1 $GLOBAL_TOTAL_WIDTH))"
 
-          # Print final summary table
+          # Print summary rows
           for entry in "${SUMMARY[@]}"; do
             wf="${entry%%|*}"
             count="${entry##*|}"
@@ -254,8 +268,6 @@ jobs:
             "$WORKFLOW_COUNT_WIDTH" "$TOTAL_AFFECTED" \
             "$GLOBAL_TOTAL_WIDTH" ""
 
-          # Final mode message
-          printf " \n"
           if [ "$MODE" = "dry run" ]; then
             echo "Dry run complete. No runs were deleted."
           else