feat: trigger review only on ready PR and once

[ymelois]

---

Note

Adds draft-PR filtering (skip triage on opened if draft, trigger on ready_for_review), an idempotency guard (has_pratrol_review) scanning up to 3 pages of reviews to detect an existing Pratrol review, bot-login resolution at startup via /app, and a CONTRIBUTIONS_CAP bump from 200 to 300.

^{Written by Mendral for commit a801e4d.}

[pratrol]

Pratrol Triage Brief: 🟢 High Confidence

Pratrol assessed this pull request across the primary risk vectors below.

Risk Vector	Score	Tier
Contributor Context	72/100	🟢 High
Logic Risk (Mistral)	80/100	🟢 High
Overall Confidence	77/100	🟢 High

Summary: Adds draft PR support and skips triage for already reviewed PRs.

Key Signal: The change is well-structured and adds meaningful functionality.

Recommendation: Review the new has_pratrol_review method for correctness.

[cloudflare-workers-and-pages]

Deploying pratrol with    Cloudflare Pages

Latest commit:	a801e4d
Status:	✅  Deploy successful!
Preview URL:	https://c30c1002.pratrol.pages.dev
Branch Preview URL:	https://feat-trigger-once-ready.pratrol.pages.dev

View logs

[mendral-app]

🤖 Code Review

Assessment ✅

All previous comments are addressed. The new changes are correct:

• Draft filtering: #[serde(default)] on draft is correct; the match on action string is the right approach since ready_for_review events have draft: false but the action is the reliable signal.
• Idempotency guard: Pagination logic (page.items.len() < 100 sentinel) is correct. MAX_REVIEW_PAGES = 3 cap is intentional and documented.
• Bot login: format!("{}[bot]", app_info.slug) matches GitHub Apps bot login format.
• fetch_events_count: (n - 1) * 100 + items_len is correct under GitHub's guarantee that all non-last pages are full.

No new issues found.

Note

Tag @mendral-app with feedback or questions. View session