review-code

分类: 工具与效率 | 上传者: haacked | 下载: 0 | 版本: v1.0（最新）

Run specialized code review agents on code changes or pull requests

更新日志: Source: GitHub https://github.com/haacked/review-code

目录结构

当前层级: skills/review-code/

📁 context/
- 📁 frameworks/
  - 📄 android.md 12.8 KB
  - 📄 aspnetcore.md 11.7 KB
  - 📄 django.md 6.7 KB
  - 📄 flutter.md 15.6 KB
  - 📄 ios.md 12.1 KB
  - 📄 kea.md 3.3 KB
  - 📄 nodejs.md 11.4 KB
  - 📄 react-kea.md 2.8 KB
  - 📄 react-native.md 11.5 KB
  - 📄 react.md 6.5 KB
- 📁 languages/
  - 📄 bash.md 10.7 KB
  - 📄 csharp.md 4.7 KB
  - 📄 dart.md 8.8 KB
  - 📄 elixir.md 5.8 KB
  - 📄 go.md 4.8 KB
  - 📄 java.md 6.2 KB
  - 📄 javascript.md 8.9 KB
  - 📄 kotlin.md 7.8 KB
  - 📄 php.md 8.1 KB
  - 📄 python.md 4.5 KB
  - 📄 ruby.md 5.1 KB
  - 📄 rust.md 6.5 KB
  - 📄 sql.md 3.3 KB
  - 📄 swift.md 7.5 KB
  - 📄 typescript.md 4.3 KB
- 📁 orgs/
  - 📁 posthog/
    
    📁 repos/
    
    📄 posthog.md 656 B
    
    📄 org.md 5.7 KB
📁 handlers/
- 📄 find.md 2.7 KB
- 📄 learn.md 6.5 KB
- 📄 review.md 31.1 KB
📁 learnings/
- 📄 .gitkeep 0 B
- 📄 README.md 1.5 KB
📁 scripts/
- 📁 helpers/
  - 📄 config-helpers.sh 1.4 KB
  - 📄 date-helpers.sh 1.8 KB
  - 📄 debug-helpers.sh 9.9 KB
  - 📄 error-helpers.sh 1.2 KB
  - 📄 exclusion-patterns.sh 2.1 KB
  - 📄 gh-wrapper.sh 332 B
  - 📄 git-helpers.sh 4.0 KB
  - 📄 json-helpers.sh 837 B
  - 📄 validation-helpers.sh 455 B
- 📄 chunk-diff.sh 14.3 KB
- 📄 code-language-detect.sh 6.4 KB
- 📄 create-draft-review.sh 10.6 KB
- 📄 debug-artifact-writer.sh 4.4 KB
- 📄 detect-comment-drift.sh 13.6 KB
- 📄 diff-position-mapper.sh 6.2 KB
- 📄 get-review-diff.sh 6.1 KB
- 📄 git-context.sh 1.6 KB
- 📄 git-diff-context.sh 2.2 KB
- 📄 git-diff-filter.sh 5.2 KB
- 📄 inject-handler.sh 1.4 KB
- 📄 learn-apply.sh 7.4 KB
- 📄 learn-batch.sh 7.9 KB
- 📄 learn-from-pr.sh 10.0 KB
- 📄 learn-orchestrator.sh 5.7 KB
- 📄 load-review-context.sh 7.0 KB
- 📄 parse-review-arg.sh 17.1 KB
- 📄 parse-review-findings.sh 8.4 KB
- 📄 pr-context.sh 10.8 KB
- 📄 pre-review-context.sh 5.0 KB
- 📄 review-file-path.sh 14.5 KB
- 📄 review-orchestrator.sh 43.7 KB
- 📄 review-safety-hook.sh 1.7 KB
- 📄 review-status-handler.sh 6.7 KB
- 📄 session-manager.sh 8.1 KB
- 📄 submit-review.sh 4.9 KB
📄 SKILL.md 10.9 KB

SKILL.md

---
description: Run specialized code review agents on code changes or pull requests
argument-hint: [find|learn|pr|commit|branch|range|area]
allowed-tools: Bash(~/.claude/skills/review-code/scripts/*:*), Read(~/.claude/**), Write(~/.claude/skills/review-code/learnings/*), Edit(~/.claude/skills/review-code/learnings/*)
hooks:
  PreToolUse:
    - matcher: Bash
      hooks:
        - type: command
          command: ~/.claude/skills/review-code/scripts/review-safety-hook.sh
---

Run specialized code review agent(s) with comprehensive context on local changes or pull requests.

**Arguments:**

- `find` - Find existing review notes without starting a new review
  - `find` (no argument) - Find review for current branch/PR
  - `find <pr-number>` - Find review for specific PR (e.g., `find 123`)
  - `find <branch>` - Find review for specific branch (e.g., `find feature-branch`)
- `learn` - Learn from PR review outcomes to improve future reviews
  - `learn <pr-number>` - Analyze outcomes of a specific PR (e.g., `learn 123`)
  - `learn` - Batch analyze all unanalyzed PRs with existing reviews
  - `learn --apply` - Apply accumulated learnings to context files
- `<pr-url>` - Review Pull Request by URL - works from anywhere (e.g., `https://github.com/org/repo/pull/123`)
  - Uses `gh` CLI to fetch PR context from GitHub
  - No git repository required - review any PR without cloning
  - Automatically uses local git for speed when on PR's branch
- `<commit>` - Review that specific commit's changes (e.g., `356ded2`)
- `<branch>` - Review all changes in branch vs base (e.g., `feature-branch`)
- `<range>` - Review specific git range (e.g., `abc123..HEAD`, `v1.0.0..v2.0.0`)
- `security` - Deep security vulnerability analysis only (local changes)
- `performance` - Performance bottlenecks and optimization only (local changes)
- `correctness` - Functional correctness, intent verification, and integration boundaries only (local changes)
- `maintainability` - Code clarity, simplicity, and maintainability only (local changes)
- `testing` - Test coverage, quality, and patterns only (local changes)
- `compatibility` - Backwards compatibility with shipped code only (local changes)
- `architecture` - High-level design, patterns, and necessity only (local changes)
- (no argument) - Run ALL 7 specialized agents in parallel on local changes (default)

**Optional Flags:**

- `--force` or `-f` - Skip the pre-flight context clear prompt
- `--draft` or `-d` - Create a pending GitHub review with inline comments (PR mode only). Automatically detects and adjusts for comment drift when the PR receives new commits between review generation and draft posting.
- `--self` - Allow creating draft review on your own PR (for testing)

**Optional File Pattern:**

Add a file pattern as a second argument to filter changes by file:
- `<arg> <pattern>` - Review only files matching the pattern (e.g., `356ded2..HEAD "*.sh"`)

Examples:
- `/review-code 356ded2..HEAD "*.sh"` - Review only shell script changes in range
- `/review-code "lib/*.sh"` - Review only shell scripts in lib/ (local changes)
- `/review-code feature-branch "*.py"` - Review only Python files in branch
- `/review-code 123 "src/**/*.ts"` - Review only TypeScript files in PR #123

**Usage examples:**

- `/review-code` - Comprehensive review of local uncommitted changes (default)
- `/review-code https://github.com/org/repo/pull/123` - Review any PR by URL (works from anywhere!)
- `/review-code 356ded2` - Review that specific commit
- `/review-code feature-branch` - Review all changes in branch vs main
- `/review-code abc123..HEAD` - Review changes from abc123 to HEAD
- `/review-code security` - Run only security review on local changes
- `/review-code find 123` - Find review for PR #123
- `/review-code learn 123` - Analyze what happened after reviewing PR #123
- `/review-code 123 --draft -f` - Review PR, create draft review, skip confirmation

---

## Implementation

Uses session-based caching to run the orchestrator once and reuse data across bash invocations. This reduces token usage by ~60%.

**CRITICAL SAFEGUARDS** - These rules are NON-NEGOTIABLE:

1. **NEVER improvise when errors occur** - If any step fails (API errors, script failures, etc.), STOP and inform the user. Do not try to work around failures.

2. **NEVER submit reviews without explicit approval** - If the user asks to add comments to a pending review and you cannot amend it, ASK before submitting. Never submit on their behalf unless explicitly told to.

3. **NEVER fall back to regular comments** - If `create-draft-review.sh` fails, do NOT post a regular comment as a workaround. This submits the review instead of keeping it pending.

4. **Follow the structured session flow** - If session initialization fails, STOP and inform the user. NEVER run review agents manually or post to GitHub directly.

### Step 1: Parse Arguments

Run the parse script to determine the review mode and parameters:

```bash
~/.claude/skills/review-code/scripts/parse-review-arg.sh $ARGUMENTS 2>&1
```

Save the JSON output as `PARSE_RESULT`. Reference this throughout — do not run the parse script again.

**Handler File Selection:**

When directed to load a handler, select the file based on `PARSE_RESULT`:

- If `mode` is `"error"`: No handler needed (error flow handles it)
- If `mode` is `"learn"`: Read `~/.claude/skills/review-code/handlers/learn.md`
- If `find_mode` is `"true"`: Read `~/.claude/skills/review-code/handlers/find.md`
- Otherwise: Read `~/.claude/skills/review-code/handlers/review.md`

Use the Read tool to load the selected handler file, then follow its instructions.

**If `mode` is `"learn"`:** Load and follow the learn handler now (see Handler File Selection above). Do not proceed to Step 2 or Step 3.

**If `find_mode` is `"true"`:** Skip the pre-flight prompt and go directly to Step 3 (Initialize Session). The session handler will route to the find handler.

### Step 2: Pre-flight Context Check

Code reviews are context-heavy and work best with a fresh context.

**If `--force` was specified:** Skip this step entirely and proceed to Step 3.

**Otherwise**, use AskUserQuestion:
- Question: "Code reviews work best with a fresh context. Clear conversation history before starting?"
- Options:
  1. "Yes, clear and review (Recommended)" - Clear context, then start the review
     Description: "Ensures clean review without prior conversation influencing results"
  2. "No, continue anyway" - Keep current context and proceed
     Description: "Use only if you need to reference earlier conversation"

If user selects "Yes, clear and review":
- Tell the user: "Please run `/clear` and then run the review command again."
- Stop here.

If user selects "No, continue anyway", proceed to Step 3.

### Step 3: Initialize Session

Initialize the review session by running the orchestrator and caching the result:

```bash
~/.claude/skills/review-code/scripts/review-status-handler.sh init $ARGUMENTS
```

Save the output as `SESSION_ID` — you'll need it for all subsequent operations. Then get the status:

```bash
~/.claude/skills/review-code/scripts/review-status-handler.sh get-status "<SESSION_ID>"
```

Save the output as `STATUS`. The `--force`/`-f` flags are handled automatically; when present, the session data will include `"force": true`.

**CRITICAL: If session initialization fails, STOP IMMEDIATELY.**

Failure conditions:
- `SESSION_ID` is empty
- Command outputs errors (jq parse errors, "ERROR:", etc.)
- `STATUS` is empty or not one of: `error`, `ambiguous`, `prompt`, `prompt_pull`, `find`, `ready`

If any failure condition is met: tell the user "Session initialization failed. Please check the error output above" and stop. Do not improvise, call GitHub APIs directly, or run review agents without a valid session. Falling back to manual posting can accidentally submit reviews instead of keeping them as drafts.

In all handlers below, substitute the actual `SESSION_ID` value when calling scripts.

### Handler: "error"

If STATUS is "error", get the error message:

```bash
~/.claude/skills/review-code/scripts/review-status-handler.sh get-error-data "<SESSION_ID>"
```

Display the error to the user. Then clean up the session:

```bash
~/.claude/skills/review-code/scripts/review-status-handler.sh cleanup "<SESSION_ID>"
```

Stop — do not proceed with review.

### Handler: "ambiguous"

If STATUS is "ambiguous", get the disambiguation data:

```bash
~/.claude/skills/review-code/scripts/review-status-handler.sh get-ambiguous-data "<SESSION_ID>"
```

Save the JSON output. Extract the fields: `arg`, `ref_type`, `is_branch`, `is_current`, `base_branch`.

Use AskUserQuestion to disambiguate based on the scenario.

**For commits:**
- Question: "The reference '$arg' is a commit. What would you like to review?"
- Options:
  1. "This commit only" - Review just the changes in commit $arg
  2. "Changes since this commit" - Review all changes from $arg to HEAD

**For current branch:**
- Question: "You're on branch '$arg'. What would you like to review?"
- Options:
  1. "Uncommitted changes" - Review only staged and unstaged files
  2. "Branch changes" - Review all changes in $arg vs $base_branch
  3. "Both" - Review branch changes plus uncommitted

**For other branches:**
- Question: "What would you like to review?"
- Options:
  1. "Branch $arg" - Review all changes in $arg vs $base_branch
  2. "Changes up to $arg" - Review all changes from $arg to HEAD

After user selects, re-run orchestrator with appropriate argument.

### Handler: "prompt"

If STATUS is "prompt", get the prompt data:

```bash
~/.claude/skills/review-code/scripts/review-status-handler.sh get-prompt-data "<SESSION_ID>"
```

Save the JSON output. Extract the fields: `current_branch`, `base_branch`, `has_uncommitted`.

Use AskUserQuestion:
- Question: "You're on branch '$current_branch' with uncommitted changes. What would you like to review?"
- Options:
  1. "Uncommitted only" - Review staged and unstaged files only
  2. "Branch changes" - Review all changes vs $base_branch
  3. "Comprehensive" - Review branch + uncommitted changes

After user selects, cleanup the old session and re-initialize with the chosen mode.

### Handler: "prompt_pull"

If STATUS is "prompt_pull", get the pull prompt data:

```bash
~/.claude/skills/review-code/scripts/review-status-handler.sh get-prompt-pull-data "<SESSION_ID>"
```

Save the JSON output. Extract the fields: `branch`, `associated_pr` (defaults to "none").

Use AskUserQuestion:
- Question: "Remote branch '$branch' is ahead of local. Would you like to pull changes first?"
- Options:
  1. "Pull and review" - Run `git pull` then proceed with review
  2. "Review local anyway" - Review the local branch as-is

After user selects:
- If "Pull and review": Run `git pull`, cleanup old session, then reinitialize with empty argument
- If "Review local anyway": Cleanup old session, then reinitialize with the branch name explicitly

### Handler: "find"

If STATUS is "find", load and follow the handler (see Handler File Selection above).

### Handler: "ready"

If STATUS is "ready", load and follow the handler (see Handler File Selection above).

登录后下载/点赞/收藏 ❤ 10 | ★ 0

review-code

目录结构

SKILL.md

举报内容

提示