# Pr Review

> Use when reviewing a pull request, diff or merge readiness.

- Skill: `vinilana/pr-review-3` (Agent Skill)
- Install (CLI): `npx skillmds@latest add vinilana/pr-review-3`
- Raw SKILL.md: https://api.skillmd.com/api/skills/vinilana/pr-review-3/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: vinilana (https://skillmd.com/u/vinilana)
- Updated: 2026-09-22
- Page: https://skillmd.com/skills/vinilana/pr-review-3

---


# PR Review

## Procedure

1. Confirm the task scope and active PREVC phase.
2. Load the relevant project docs from `.context/docs/`.
3. Apply the checklist below to the specific files in scope.
4. Capture validation evidence and unresolved risks.

## Checklist

- Use repository-specific conventions, not generic defaults.
- Reference concrete files, commands and outputs.
- Keep recommendations actionable and prioritized.
- If information is missing, inspect the codebase first, then ask a focused question only when needed.

## Project Notes

- Use this skill against a pi-extension codebase with TypeScript as the primary language.
- Does the change preserve existing user files and manual edits?
- Are public commands/tools/events documented in .context/docs/api.md and README when behavior changes?
- Were relevant validation commands run and captured with exit status?
- Were security-sensitive paths and fail-safe behavior reviewed?

## Useful Context

- `.context/docs/project-overview.md`
- `.context/docs/development-workflow.md`
- `.context/docs/architecture.md`
- `.context/docs/security.md`
- `.context/docs/data-flow.md`
- `.context/docs/contributing.md`
- `.context/docs/testing-strategy.md`
- `.context/docs/api.md`