@typeddevs/review
AReview a GitHub pull request; defaults to the current branch's PR when no number is given
Install
agr install @typeddevs/review --target claudeWrites 1 file into .claude/skills/, pinned to git-acab61f9.
- .claude/skills/review/SKILL.md
Document
name: review description: Review a GitHub pull request; defaults to the current branch's PR when no number is given user-invocable: true argument-hint: "[#N]" allowed-tools: Bash, Read, Grep, Glob
Review a Pull Request
Review a GitHub PR. When no number is given, review the PR for the current branch.
For reviewing your uncommitted working diff instead, use
/code-review.
Current Branch Context
- Branch: !
git branch --show-current - PR for this branch: !
gh pr view --json number,title,state,url --jq '"#\(.number) \(.title) [\(.state)] \(.url)"' 2>/dev/null || echo "none"
Arguments
$ARGUMENTS- PR number to review (optional, e.g.,#42or42).
Instructions
- Resolve which PR to review:
- If
$ARGUMENTScontains a number, review that PR. - Otherwise, use the current branch's PR shown in the context above.
- If neither yields a PR (no argument and the branch has no open PR), run
gh pr listand ask the user which one to review. Do NOT guess.
- Load the PR:
gh pr view <number> --json number,title,body,state,headRefName,baseRefName,url
gh pr diff <number>
- Review the diff against this project's standards (see
.claude/CLAUDE.mdand.claude/rules/):
- Bash 3.0+ compatibility (no
declare -A,[[ ]],${var,,}, negative indexing,&>>) - Tests exist and follow TDD; both success and failure paths covered
- Naming, namespacing (
bashunit::*/_private), and dynamic-scope safety - ShellCheck cleanliness and 2-space indentation (
make lint, notshfmt) - CHANGELOG.md updated for user-facing changes
- Report findings grouped by severity (blocker / suggestion / nit), each as
path:line — problem. fix.. No praise, no scope creep. State plainly if the PR looks good.
Trustgrade A
- passBody integrity
Whether the stored document is plausibly the kind of file the artifact declares, rather than something fetched by mistake.
- passType matchnot applicable to this artifact type
Whether the artifact is really the kind of thing its metadata claims it is.
- passFreshness
How long since the source repository was last pushed to.
- passPrompt injection
Scans the artifact's own text for instructions aimed at your agent rather than at you.
- passLicense
Whether the source repository declares an SPDX license permissive enough to redistribute.
How the grade is calculated
Each check contributes 0 points when it passes, 1 when it warns, and 2 when it fails. The total maps to a letter:
- Aevery check passed
- Bone warning
- Ctwo warnings
- Dprompt injection or body integrity failed, or three warnings
- Fone of those failed, and something else is wrong
These are automated hygiene checks, not a security audit, and not a dependency or vulnerability scan. A grade of A means nothing was flagged — not that the artifact is safe.
Versions
git-acab61f9d7542026-07-31