← Browse

@typeddevs/review

A

Review a GitHub pull request; defaults to the current branch's PR when no number is given

skillclaude

Install

agr install @typeddevs/review --target claude

Writes 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., #42 or 42).

Instructions

  1. Resolve which PR to review:
  • If $ARGUMENTS contains 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 list and ask the user which one to review. Do NOT guess.
  1. Load the PR:
gh pr view <number> --json number,title,body,state,headRefName,baseRefName,url
gh pr diff <number>
  1. Review the diff against this project's standards (see .claude/CLAUDE.md and .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, not shfmt)
  • CHANGELOG.md updated for user-facing changes
  1. 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