fix(ci): fail the IndexNow step when the oldest run's parent cannot be resolved

Author: waleedlatif1

.github/workflows/ci.yml

@@ -672,7 +672,8 @@ jobs:
       # changed URLs for good. The check-runs lookup asks for this one check per commit, so it
       # stays small and counts every attempt of a re-run; a lookup that errors counts as no
       # success. With no success in view, the base is the first parent of the oldest run in view,
-      # so URLs that no submission ever covered, that run's own included, are sent now. The diff
+      # so URLs that no submission ever covered, that run's own included, are sent now; a failed
+      # parent lookup fails the step, so the next run retries from the same base. The diff
       # needs only the two trees, so that one commit is fetched alone; an all-zero `before` (the
       # branch's first push) has no base, which the script skips.
       - name: Submit changed URLs to IndexNow
@@ -692,9 +693,7 @@ jobs:
           done < <(gh api "repos/$GITHUB_REPOSITORY/actions/workflows/ci.yml/runs?branch=main&event=push&per_page=30" \
             --jq ".workflow_runs[] | select(.id < $GITHUB_RUN_ID) | .head_sha")
           if [[ -z "$BASE_SHA" && -n "$OLDEST_SHA" ]]; then
-            if ! BASE_SHA="$(gh api "repos/$GITHUB_REPOSITORY/commits/$OLDEST_SHA" --jq '.parents[0].sha // empty')"; then
-              BASE_SHA="$OLDEST_SHA"
-            fi
+            BASE_SHA="$(gh api "repos/$GITHUB_REPOSITORY/commits/$OLDEST_SHA" --jq '.parents[0].sha // empty')"
           fi
           BASE_SHA="${BASE_SHA:-$PUSH_BEFORE}"
           echo "Submitting URLs changed in $BASE_SHA..$GITHUB_SHA"