fix: bail out on incomplete pagination instead of acting on partial data

When a GraphQL or jq error breaks the pagination loop mid-flight,
skip resolution entirely rather than proceeding with threads from
only the earlier pages. This prevents incorrectly leaving threads
open (or resolving them) based on an incomplete view.

Author: derekmisler

review-pr/action.yml

@@ -237,6 +237,7 @@ runs:
         ALL_THREADS="[]"
         CURSOR=""
         PAGE=0
+        FETCH_OK=true
 
         while true; do
           PAGE=$((PAGE + 1))
@@ -248,21 +249,25 @@ runs:
 
           RESULT=$(gh api graphql "${GQL_ARGS[@]}" 2>&1) || {
             echo "::warning::GraphQL query failed (page $PAGE): $RESULT"
-            exit 0
+            FETCH_OK=false
+            break
           }
 
           # Check for GraphQL errors in response (HTTP 200 can still contain errors)
           if echo "$RESULT" | jq -e '.errors' > /dev/null 2>&1; then
             echo "::warning::GraphQL returned errors: $(echo "$RESULT" | jq -c '.errors')"
-            exit 0
+            FETCH_OK=false
+            break
           fi
 
           THREADS=$(echo "$RESULT" | jq '.data.repository.pullRequest.reviewThreads.nodes // []') || {
             echo "::warning::Failed to parse threads on page $PAGE"
+            FETCH_OK=false
             break
           }
           ALL_THREADS=$(echo "$ALL_THREADS" "$THREADS" | jq -s '.[0] + .[1]') || {
             echo "::warning::Failed to merge threads on page $PAGE"
+            FETCH_OK=false
             break
           }
 
@@ -273,6 +278,11 @@ runs:
           CURSOR=$(echo "$RESULT" | jq -r '.data.repository.pullRequest.reviewThreads.pageInfo.endCursor')
         done
 
+        if [ "$FETCH_OK" != "true" ]; then
+          echo "::warning::Thread fetch incomplete — skipping resolution to avoid acting on partial data"
+          exit 0
+        fi
+
         TOTAL_THREADS=$(echo "$ALL_THREADS" | jq 'length')
         echo "📋 Fetched $TOTAL_THREADS review threads (across $PAGE page(s))"