ci: auto-remove needs review label when PR is reviewed

[serhalp]

🔗 Linked issue

N/A

🧭 Context

We've been using the needs review label on PRs that are not stale, desirable, meet our contribution guidelines, passing CI, haven't been reviewed yet or are ready for a re-review, etc.

But the label gets less valuable if you can't rely on it at a glance or to filter PRs. This can happen if we don't manually remove the label when submitting a PR review.

📚 Description

This PR adds a CI workflow that removes the needs review label from a PR, if present, when a PR review is submitted by a maintainer.

✅ Ignores "reviews" (some comments count as reviews) from the PR author:

Skipping self-review from @serhalp on PR #2402.

✅ Ignores reviews from bots like CodeRabbit:

Skipping bot review from @CodeRabbit on PR #2402.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Apr 7, 2026 9:51pm

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Apr 7, 2026 9:51pm
npmx-lunaria	Ignored		Apr 7, 2026 9:51pm

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[43081j]

looks somewhat tolerable to me

[coderabbitai]

📝 Walkthrough

Walkthrough

A new GitHub Actions workflow, remove-needs-review-on-review, was added. It triggers on pull_request_review events of type submitted and runs only for the npmx-dev/npmx.dev repository. The workflow inspects the submitted review and PR payload, verifies the PR has the needs review label, checks the reviewer is not a bot or the PR author, queries the reviewer’s permission level, and removes the needs review label when the permission is admin, maintain or write.

🚥 Pre-merge checks | ✅ 1

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description check	✅ Passed	The pull request description is directly related to the changeset, explaining the motivation for the new workflow and detailing its key behaviours.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ci/remove-needs-review-label-on-pr-review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d038b959-6826-42d5-9a95-b55ec5a1bb9e

📥 Commits

Reviewing files that changed from the base of the PR and between 973b9c7 and 962e9b3.

📒 Files selected for processing (1)

• .github/workflows/remove-needs-review-on-review.yml

[serhalp]

this is a test

[serhalp]

@CodeRabbit review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[ghostdevv]

I got a failure here, not sure if it's a fluke

https://github.com/npmx-dev/npmx.dev/actions/runs/24199409062/job/70638557339