Dependency Review

Scanned Files

No actionable comments were generated in the recent review. 🎉

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Greptile Summary

This PR is a targeted bug fix for the cleanup-preview job in .github/workflows/pages-preview.yml. It replaces the hardcoded per_page=100 query parameter (rejected by the Cloudflare Pages API with HTTP 400 when the page parameter is also present) with per_page=25, stored in a PER_PAGE shell variable. The pagination break condition is updated to reference the variable instead of the old literal, keeping the two values in sync.

Key changes:

• Introduces PER_PAGE=25 variable (Cloudflare's documented default page size) to fix the HTTP 400 error that broke the first run of the cleanup job after pagination was added in fix: correct deploy-pages SHA and improve preview cleanup reliability #304.
• Updates the break condition from [ "$PAGE_COUNT" -lt 100 ] to [ "$PAGE_COUNT" -lt "$PER_PAGE" ], eliminating the previously inconsistent hardcoded literal.
• Adds an inline comment explaining why 25 is used, which aids future maintainability.

The fix is minimal, correct, and low-risk. The trade-off is that per_page=25 requires up to 4× more API requests compared to per_page=100 for large deployment sets, but for a one-time PR-close cleanup this has no practical impact. It is worth noting that the Cloudflare Pages API may accept values between 25 and 100 (e.g., 50) — if a higher limit is ever confirmed, PER_PAGE can be updated in a single place.

Confidence Score: 4/5

• This PR is safe to merge — the fix directly resolves a confirmed HTTP 400 API rejection with a minimal, well-scoped change.
• The change is confined to two lines in a single shell script block, is internally consistent (variable defined once, used in both the URL and the break condition), and addresses a documented API constraint. No logic is restructured and no new failure modes are introduced.
• No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant GHA as GitHub Actions<br/>(cleanup-preview job)
    participant CF as Cloudflare Pages API

    GHA->>GHA: Set PER_PAGE=25, PAGE=1, DEPLOYMENTS=""
    loop Until page is incomplete
        GHA->>CF: GET /deployments?per_page=25&page=PAGE
        CF-->>GHA: HTTP 200 + JSON body
        GHA->>GHA: Extract matching deployment IDs for branch
        GHA->>GHA: PAGE_COUNT = length of .result
        alt PAGE_COUNT < PER_PAGE (25)
            GHA->>GHA: break (last page reached)
        else PAGE_COUNT == PER_PAGE
            GHA->>GHA: PAGE++, continue
        end
    end
    loop For each deployment ID
        GHA->>CF: DELETE /deployments/{id}?force=true
        CF-->>GHA: HTTP 200 (or 4xx for most-recent-deployment restriction)
    end
    GHA->>GHA: echo "Deleted N preview deployments"

_{Last reviewed commit: 290e983}