Skip to content

feat(docs): automate PR review triage - #19078

Open
sfanahata wants to merge 4 commits into
masterfrom
docs/pr-review-triage-automation
Open

feat(docs): automate PR review triage#19078
sfanahata wants to merge 4 commits into
masterfrom
docs/pr-review-triage-automation

Conversation

@sfanahata

@sfanahata sfanahata commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

DESCRIBE YOUR PR

Automates priority triage and non-blocking reviewer assignment for docs PRs.

What changes:

  • Parses the PR urgency section into mutually exclusive Urgent, Deadline, Needs Triage, or Normal labels.
  • Adds one non-blocking reminder comment when a ready PR has missing or malformed priority metadata, and removes it after correction.
  • Requests getsentry/docs only for ready, human-authored PRs that are urgent/deadline-based, external, significant changes under docs/**, or significant changes overall.
  • Preserves the existing commented CODEOWNERS specialist assignments without making Docs or specialists required reviewers.
  • Moves external-PR evaluation to trusted pull_request_target workflows with least-privilege tokens and base-SHA validation.
  • Removes nine unused, overlapping labels.
  • Updates the PR template to require exactly one urgency selection and ISO YYYY-MM-DD deadline dates.

Slack alerts and a daily digest will be added in as a part of this workflow in a follow-up PR.

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.): YYYY-MM-DD
  • Other deadline: YYYY-MM-DD
  • No deadline: Not urgent, can wait up to 1 week+

TEST PLAN

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

EXTRA RESOURCES

@sfanahata
sfanahata requested a review from a team August 19, 2026 22:45
@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview Aug 20, 2026 6:52pm
sentry-docs Ready Ready Preview Aug 20, 2026 6:52pm

Request Review

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

🏷 The following changes will be made to the repository labels

Progress: resolved 1, reused 0, downloaded 0, added 0
Progress: resolved 80, reused 0, downloaded 74, added 0
 WARN  3 deprecated subdependencies found: har-validator@5.1.5, request@2.88.2, uuid@3.4.0
Packages: +104
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Progress: resolved 104, reused 0, downloaded 104, added 104, done
Syncing labels for "getsentry/sentry-docs"
Validating provided labels
Fetching labels from GitHub
 > Changed: the "Docs" label in the repo is out of date. It will be updated to "Docs" with color "#ededed" and description "Documentation content issues".
 > Changed: the "Docs Platform" label in the repo is out of date. It will be updated to "Docs Platform" with color "#ededed" and description "Documentation site and tooling issues".
 > Missing: the "Priority: Urgent" label is missing from the repo. It will be created.
 > Missing: the "Priority: Deadline" label is missing from the repo. It will be created.
 > Missing: the "Priority: Needs Triage" label is missing from the repo. It will be created.
 > Missing: the "Priority: Normal" label is missing from the repo. It will be created.
 > Changed: the "Team: SDKs" label in the repo is out of date. It will be updated to "Team: SDKs" with color "#ededed" and description "Issues that affect all or multiple SDK teams".
 > Added: the "All SDKs" label in the repo is not expected. It will be deleted.
 > Added: the "Effort: Large" label in the repo is not expected. It will be deleted.
 > Added: the "Effort: Medium" label in the repo is not expected. It will be deleted.
 > Added: the "Effort: Small" label in the repo is not expected. It will be deleted.
 > Added: the "Impact: Large" label in the repo is not expected. It will be deleted.
 > Added: the "Impact: Medium" label in the repo is not expected. It will be deleted.
 > Added: the "Impact: Small" label in the repo is not expected. It will be deleted.
 > Added: the "Launch 🚀" label in the repo is not expected. It will be deleted.
 > Added: the "Type: Platform" label in the repo is not expected. It will be deleted.
This is a dry run. No changes have been made on GitHub

Comment thread scripts/priority-metadata-workflow.test.mjs
Comment thread scripts/codeowner-workflow.test.mjs
Comment thread scripts/docs-pr-triage.mjs Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7f69e99. Configure here.

Comment thread .github/workflows/pr-priority-metadata.yml
Comment thread scripts/sync-pr-priority-metadata.mjs Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants