Required Review Action: omit author from reviewer assignment (#37653)

This PR fixes a bug where the required-review action would attempt to
add the PR author as a reviewer if a matching rule was hit, however this
is no longer allowed by GitHub and would result in the error `Review
cannot be requested from pull request author.`. To avoid this, if
computed reviewers list includes the PR author, a message is logged and
they are filtered out of the list.

Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/9318374096

Upstream-Ref: Automattic/jetpack@62f0e20

CHANGELOG.md

@@ -12,6 +12,9 @@ This is an alpha version! The changes listed here are not final.
 ### Changed
 - Updated package dependencies.
 
+### Fixed
+- Don't request review from the PR author.
+
 ## [4.0.0] - 2024-02-07
 ### Added
 - Added support for GitHub Enterprise by replacing github.com with a dynamic variable [#32974]

dist/index.js

@@ -36033,6 +36033,11 @@ async function requestReviewer( teams ) {
 	const owner = github.context.payload.repository.owner.login;
 	const repo = github.context.payload.repository.name;
 	const pr = github.context.payload.pull_request.number;
+	const author = `@${ github.context.payload.pull_request.user.login }`;
+	if ( teams.includes( author ) ) {
+		core.info( `Skipping review for author ${ author }` );
+		teams = teams.filter( team => team !== author );
+	}
 
 	const userReviews = [];
 	const teamReviews = [];

src/request-review.js

@@ -11,6 +11,11 @@ async function requestReviewer( teams ) {
 	const owner = github.context.payload.repository.owner.login;
 	const repo = github.context.payload.repository.name;
 	const pr = github.context.payload.pull_request.number;
+	const author = `@${ github.context.payload.pull_request.user.login }`;
+	if ( teams.includes( author ) ) {
+		core.info( `Skipping review for author ${ author }` );
+		teams = teams.filter( team => team !== author );
+	}
 
 	const userReviews = [];
 	const teamReviews = [];