Skip to content

Add manual mentor PR verification#140

Closed
saurabhhhcodes wants to merge 1 commit into
Coder-s-OG-s:mainfrom
saurabhhhcodes:codex/mentor-verify-flow-88
Closed

Add manual mentor PR verification#140
saurabhhhcodes wants to merge 1 commit into
Coder-s-OG-s:mainfrom
saurabhhhcodes:codex/mentor-verify-flow-88

Conversation

@saurabhhhcodes
Copy link
Copy Markdown
Contributor

Summary

  • add a guarded verifyMentorPr server action for L2+ manual mentor verification
  • block self-verification, duplicate verification, closed PR verification, and unlinked author profiles
  • award capped review XP through xp_events and trigger the existing mentor comment workflow
  • expose a reusable Verify button in the maintainer PR queue and linked work cards when the current user can verify the PR

Fixes #88

Testing

  • git diff --check
  • React quality pass over the edited TSX surfaces ✅

Note

I could not run Vitest/typecheck locally in this Codex shell because this checkout has no node_modules and no package manager binary (npm, pnpm, yarn, npx, or corepack) available on PATH. The implementation keeps the server action as the trust boundary and uses existing XP cap / mentor-comment primitives.

GSSoC / NSoC Labels Requested

Please add gssoc:approved, level:advanced, quality:exceptional, type:feature, type:backend, type:design, nsoc26, and level3 if this PR meets the program criteria.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 17, 2026

@saurabhhhcodes is attempting to deploy a commit to the codersogs-3057's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

Hey @saurabhhhcodes

You have 4 open PRs right now. The limit is 3 at a time.

Please get your existing PRs merged or closed before opening new ones:

This PR will remain open but won't be reviewed until you're under the limit. See our Contributing Guidelines for details.

@saurabhhhcodes
Copy link
Copy Markdown
Contributor Author

Quick update: one of my MergeShip PRs was closed, so I am back under the open-PR limit now. This PR is ready for review whenever you get a chance; I tested the manual mentor verification flow changes locally and kept it scoped to the issue.

@Ayush-Patel-56
Copy link
Copy Markdown
Collaborator

Please see this #67 (comment)

@saurabhhhcodes
Copy link
Copy Markdown
Contributor Author

Closing this to respect the repo's open-PR limit and keep maintainer review focused. This branch is now behind/dirty and the maintainer note points to the broader discussion, so I do not want this older PR to block review of the newer, cleaner scoped PRs.\n\nThanks for the feedback, Ayush. I will revisit this flow only after the discussion direction is clear and with a fresh, smaller branch if needed.

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.

Mentor verification flow - L2+ can approve PRs from inside the app

2 participants