| name | review:peer |
| description | Review a pull request when requested by a peer. Use when reviewing PRs, providing code review feedback, or analyzing proposed changes. Supports GitHub and GitLab. Pass --triage to summarize a PR and estimate its review effort without reviewing it.
|
| argument-hint | <pr-url-or-number> [--triage] |
| allowed-tools | ["Bash(gh:*)","Bash(glab:*)","Bash(git log:*)","Bash(git diff:*)","Bash(git show:*)","Bash(git branch:*)","Bash(git fetch:*)","Bash(git checkout:*)","Bash(git remote:*)","Bash(git rev-parse:*)","Bash(git cat-file:*)","Bash(git status:*)","Bash(jq:*)","mcp__github","WebFetch","Skill(code-review)","Skill(review:tuicr)"] |
Peer Review
Assist me in reviewing this PR: $ARGUMENTS
Context
Your own login as the reviewer. Everyone else in the diff and threads is either the author, addressed as "you", or a third party (see tone.md). Whichever platform applies resolves. The other reads unavailable.
- GitHub user: !
gh api graphql -f query='{viewer{login}}' --jq .data.viewer.login 2>/dev/null | grep . || echo "unavailable"
- GitLab user: !
glab api user 2>/dev/null | jq -r .username 2>/dev/null | grep . || echo "unavailable"
Arguments
--triage: assess the PR for sequencing instead of reviewing it. Produce a one-line summary of what it changes and an estimated review effort, then stop. Default: off, which runs the full review workflow below.
Triage Mode
When --triage is set, stay read-only and assess the PR for sequencing. Gather just enough to judge scope: the PR body, the diff stat, and the files touched. Then report two things and stop.
- What the PR changes, in one line.
- The estimated review effort on the same scale step 4 uses for
/code-review (low, medium, high, xhigh, max), with one-line reasoning.
If not on the branch, first run gh pr checkout to switch.
Guardrails
- Must check with me before submitting. Show file comments and review comment.
- Don't insist on commenting on every PR. Propose approving with no comment if everything looks good.
- Don't run Python, Ruby, or other interpreters for library introspection. Use Read for source files or WebFetch for documentation instead.
- Do match my writing style. You're commenting as me, not a generic AI assistant.
- Do present technical questions to me for ambiguous code. Don't proceed until you understand fully.
Workflow
- Research - Gather context and identify participants (see research.md)
- Context - Determine review context using repository visibility. Private repositories use corporate defaults. Public repositories use open-source defaults. Check visibility via the platform API (
gh api repos/OWNER/REPO --jq .visibility or glab api projects/ENCODED_PATH | jq .visibility). If ambiguous, ask me.
- Review - Examine changed files and existing comments
- Delegate - Run
/code-review for code-quality analysis. Summarize the diff (rough line count, files touched, sensitive areas) and signals from the PR body, propose an effort level with one-line reasoning, and confirm via AskUserQuestion before invoking. Skip the call for trivial PRs (docs-only, dep bumps). Effort heuristics:
- low: docs-only, dep bumps, config tweaks, trivial fixes (<50 lines)
- medium: typical features or fixes, single module, ~50–500 lines
- high: large refactors, multi-module, public API or schema changes, ~500–2000 lines
- xhigh: security-sensitive (auth, payments, data access), breaking changes, migrations
- max: rare — incident hotfix or change with extreme blast radius
- Think - Evaluate against priorities (see priorities.md), incorporating
/code-review findings
- Stage - Open the PR diff in tuicr via
review:tuicr (tuicr pr <N> for GitHub, tuicr mr <N> for GitLab) and seed proposed comments with tuicr review add (pass --username so they read as agent comments). Capture the PR head SHA (and base/start SHAs for GitLab, from the MR diff_refs) for mapping. Skip staging when approving with no comments.
- Revise - I curate in the tuicr pane: delete comments I reject, reword, and add my own at any line. The surviving set gets posted.
- Submit a GitHub PR yourself from the TUI - For a GitHub PR you curated in the tuicr pane, the fastest path is tuicr's own
:submit (Comment / Approve / Request changes / Draft), which posts a real PR review via gh. When that fits, skip the map-and-post steps below. Claude maps and posts only for GitLab (tuicr cannot post to GitLab at all) or when the review runs headless with no TUI.
- Map - When Claude posts, read back the final set (
tuicr review comments --repo <repo> --session <slug>) and map each comment to a platform position with review:tuicr's mapping CLI (mapping.ts map --platform <github|gitlab> --comments ... --diff ... --commit <sha>). It runs the in-diff pre-check and returns { payloads, dropped }. Off-diff anchors land in dropped (GitHub rejects them with 422 "Line could not be resolved"), so surface those to me rather than losing them.
- Post - Show me the mapped set, then on my go post as one batch and choose Approve / Comment / Request Changes based on severity. GitHub: a pending review submitted as a batch. GitLab: draft notes published together.
See tone.md for comment style guidelines.
Service Support
This skill assumes GitHub. For GitLab merge requests, load gitlab:merge-request for the submission workflow; use draft-note.ts submit to publish draft notes with an optional summary and review decision.
tuicr's own :submit is interactive and GitHub-only. When I curate a GitHub PR live in the pane, I submit it there and Claude posts nothing. Claude's programmatic path (mcp__github / gh / glab) covers the two cases tuicr can't: a GitLab MR (tuicr never posts to GitLab) and a headless GitHub run with no TUI. When tuicr is not running or I prefer to skip it, stage nothing and post directly the same way. On follow-up, resolution is native: resolve addressed threads on the platform (review-threads.ts for GitHub, the resolve flow in gitlab:merge-request for GitLab), not tuicr.