code-review

# Code Review Skill Submit structured code review requests to multiple AI providers and get unified diffs back. ## Supported Providers | Provider | CLI | Default Model | Notes | |----------|-----|---------------|-------| | `github` | `copilot` | `gpt-5` | GitHub Copilot (default) | | `anthropic` | `claude` | `sonnet` | Claude CLI | | `openai` | `codex` | `gpt-5.2-codex` | OpenAI Codex (high reasoning default) | | `google` | `gemini` | `gemini-2.5-flash` | Gemini CLI | ## Prerequisites ```bash # Check provider CLI availability python .agents/skills/code-review/code_review.py check python .agents/skills/code-review/code_review.py check --provider anthropic python .agents/skills/code-review/code_review.py check --provider openai python .agents/skills/code-review/code_review.py check --provider google ``` ## Quick Start ```bash # Single-step review (default: github/copilot) python .agents/skills/code-review/code_review.py review --file request.md # Use different provider python .agents/skills/code-review/code_review.py review --file request.md --provider anthropic # OpenAI with high reasoning python .agents/skills/code-review/code_review.py review --file request.md --provider openai --reasoning high # Include uncommitted local files via workspace python .agents/skills/code-review/code_review.py review --file request.md --workspace ./src --workspace ./tests # Full 3-step pipeline (generate -> judge -> finalize) python .agents/skills/code-review/code_review.py review-full --file request.md ``` ## Commands ### check Verify provider CLI and authentication. | Option | Short | Description | |--------|-------|-------------| | `--provider` | `-P` | Provider to check (default: github) | ### review Submit a single code review request. | Option | Short | Description | |--------|-------|-------------| | `--file` | `-f` | Markdown request file (required) | | `--provider` | `-P` | Provider: github, anthropic, openai, google | | `--model` | `-m` | Model (provider-specific, uses default if not set) | | `--add-dir` | `-d` | Add directory for file access | | `--workspace` | `-w` | Copy local paths to temp workspace (for uncommitted files) | | `--reasoning` | `-R` | Reasoning effort: low, medium, high (openai only) | | `--raw` | | Output raw response without JSON | | `--extract-diff` | | Extract only the diff block | ### review-full Run iterative code review pipeline. **Pipeline (per round):** 1. **Generate** - Initial review with diff and clarifying questions 2. **Judge** - Reviews output, answers questions, provides feedback 3. **Finalize** - Regenerates diff incorporating feedback | Option | Short | Description | |--------|-------|-------------| | `--file` | `-f` | Markdown request file (required) | | `--provider` | `-P` | Provider: github, anthropic, openai, google | | `--model` | `-m` | Model for all steps | | `--add-dir` | `-d` | Add directory for file access | | `--workspace` | `-w` | Copy local paths to temp workspace | | `--reasoning` | `-R` | Reasoning effort: low, medium, high (openai only) | | `--rounds` | `-r` | Iteration rounds (default: 2) | | `--save-intermediate` | `-s` | Save step outputs | | `--output-dir` | `-o` | Directory for output files | ```bash # Full pipeline with intermediate files saved python .agents/skills/code-review/code_review.py review-full \ --file request.md \ --save-intermediate \ --output-dir ./reviews # Creates: round1_step1.md, round1_step2.md, round1_final.md, round1.patch ``` ### build Build a request markdown file from options. | Option | Short | Description | |--------|-------|-------------| | `--title` | `-t` | Title describing the fix (required) | | `--repo` | `-r` | Repository owner/repo (required) | | `--branch` | `-b` | Branch name (required) | | `--path` | `-p` | Paths of interest (repeatable) | | `--summary` | `-s` | Problem summary | | `--objective` | `-o` | Objectives (repeatable) | | `--acceptance` | `-a` | Acceptance criteria (repeatable) | | `--touch` | | Known touch points (repeatable) | | `--output` | | Write to file instead of stdout | ### bundle Bundle request for copy/paste into GitHub Copilot web. **IMPORTANT:** Copilot web can only see committed & pushed changes! | Option | Short | Description | |--------|-------|-------------| | `--file` | `-f` | Markdown request file (required) | | `--repo-dir` | `-d` | Repository directory (for git status check) | | `--output` | `-o` | Output file (default: stdout) | | `--clipboard` | `-c` | Copy to clipboard (xclip/pbcopy) | | `--skip-git-check` | | Skip git status verification | ### models List available models for a provider. | Option | Short | Description | |--------|-------|-------------| | `--provider` | `-P` | Provider to list models for | ### template Print the example review request template. ### find Search for review request markdown files. ## Workspace Feature The `--workspace` flag copies local files to a temporary directory that providers can access. This is useful when you have uncommitted changes that aren't visible to remote-based providers. ```bash # Copy src/ and tests/ to temp workspace python .agents/skills/code-review/code_review.py review \ --file request.md \ --workspace ./src \ --workspace ./tests ``` The workspace is automatically cleaned up after the review completes. ## Provider-Specific Notes ### GitHub Copilot (`github`) - Requires `gh` CLI authenticated - Supports `--continue` for session continuity - Models: gpt-5, claude-sonnet-4, claude-sonnet-4.5, claude-haiku-4.5 ### Anthropic Claude (`anthropic`) - Requires `claude` CLI - Supports `--continue` for session continuity - Models: opus, sonnet, haiku, opus-4.5, sonnet-4.5, sonnet-4 ### OpenAI Codex (`openai`) - Requires `codex` CLI - Default reasoning: high (best results) - Models: gpt-5, gpt-5.2, gpt-5.2-codex, o3, o3-mini - Does NOT support `--continue` (session context lost between rounds) ### Google Gemini (`google`) - Requires `gemini` CLI - Models: gemini-3-pro, gemini-3-flash, gemini-2.5-pro, gemini-2.5-flash, auto - Does NOT support `--continue` (use /chat save/resume in interactive mode) ## Project Agent Workflow (Recommended) The intended workflow is for the **project agent (Claude) to interpret and apply** suggestions: ``` User asks for code review | Project agent creates review request (follows template) | review/review-full sends to provider CLI | Project agent parses output, decides what to apply | Project agent uses Edit tool to apply changes it concurs with | If unclear, agent asks provider (another round) or the user ``` **Why this approach:** - Patch format may be malformed (won't `git apply`) - Project agent can exercise judgment on suggestions - Agent can ask clarifying questions back to provider - Same workflow humans use, but automated ## Dependencies ```bash pip install typer rich ```