# Requesting Code Review

> Structures code review requests with context, commit hashes, and requirements. Facilitates early and frequent review cycles.

- **Type:** Skill
- **Install:** `agentstack add skill-lucface-claude-skills-requesting-code-review`
- **Verified:** Yes — security-reviewed for prompt injection and unsafe behavior
- **Seller:** [Lucface](https://agentstack.voostack.com/s/lucface)
- **Installs:** 0
- **Category:** [Agent Skills](https://agentstack.voostack.com/c/agent-skills)
- **Latest version:** 0.1.0
- **License:** MIT
- **Upstream author:** [Lucface](https://github.com/Lucface)
- **Source:** https://github.com/Lucface/claude-skills/tree/main/skills/requesting-code-review

## Install

```sh
agentstack add skill-lucface-claude-skills-requesting-code-review
```

Requires the [AgentStack CLI](https://agentstack.voostack.com/docs/cli). Works with Claude Code, Cursor, and any MCP-compatible agent.

## About

# Requesting Code Review

Structure code review requests for effective feedback.

## Core Principle

**Review early, review often.** Prevents issues from compounding.

## When Reviews Are Required

| Scenario | Priority |
|----------|----------|
| After each task in subagent workflow | Required |
| Upon finishing major features | Required |
| Before merging to main | Required |
| When stuck | Optional |
| Before refactoring | Optional |
| After fixing complex bugs | Optional |

## Request Process

### 1. Gather Context

```bash
# Get commit range
BASELINE=$(git merge-base HEAD main)
CURRENT=$(git rev-parse HEAD)

echo "Baseline: $BASELINE"
echo "Current: $CURRENT"

# Show what changed
git log $BASELINE..$CURRENT --oneline
git diff $BASELINE..$CURRENT --stat
```

### 2. Dispatch Review Request

```
Task({
  prompt: `
    Code review request.

    ## What Was Built
    [Brief description of feature/fix]

    ## Requirements
    [Original requirements or task description]

    ## Commits
    - Baseline: ${BASELINE}
    - Current: ${CURRENT}

    ## Description
    [What changed and why]

    ## Areas of Concern
    [Specific things to look at closely]

    Please review for:
    - Spec compliance
    - Code quality
    - Test coverage
    - Potential issues
  `,
  subagent_type: "general-purpose"
})
```

### 3. Act on Findings

| Severity | Action |
|----------|--------|
| Critical | Fix immediately |
| Important | Fix before proceeding |
| Minor | Document for later |

## Request Template

```markdown
## Code Review Request

### Summary
[One sentence describing what this code does]

### Requirements Fulfilled
- [x] Requirement 1
- [x] Requirement 2

### Changes
**Files modified:**
- `src/feature.ts` - Added validation logic
- `src/feature.test.ts` - Added tests

**Commits:**
```
abc123 - Add user validation
def456 - Add error handling
```

### How to Test
```bash
bun test src/feature.test.ts
```

### Concerns
- Performance of X might be an issue with large data
- Not sure if Y pattern is idiomatic

### Questions
- Should we also handle edge case Z?
```

## Reviewer Categories

Reviewers will categorize feedback:

| Category | Meaning | Your Action |
|----------|---------|-------------|
| Critical | Blocks merge | Fix now |
| Important | Should fix | Fix before continuing |
| Minor | Nice to have | Schedule for later |
| Question | Clarification | Answer promptly |

## Handling Disagreement

If feedback seems incorrect:

1. Don't dismiss it immediately
2. Provide technical evidence
3. Explain your reasoning
4. Be open to being wrong

```
I hear your concern about X. Here's why I chose this approach:

1. [Technical reason]
2. [Evidence from testing]
3. [Alternative considered and why rejected]

If you still think Y is better, I'm happy to change it.
```

## Post-Review

After receiving feedback:

```bash
# Fix issues
# ...

# Verify fixes
bun test

# Commit fixes
git commit -m "Address review feedback: [summary]"

# Re-request if changes were significant
```

## Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

- **Author:** [Lucface](https://github.com/Lucface)
- **Source:** [Lucface/claude-skills](https://github.com/Lucface/claude-skills)
- **License:** MIT

Install and usage instructions live in the source repository linked above.

## Pricing

- **Free** — Free

## Security capabilities

Automated source analysis of v0.1.0 — what this tool can access:

- **Network access:** no
- **Filesystem access:** no
- **Shell / process execution:** no
- **Environment & secrets:** no
- **Dynamic code execution:** no

*"Yes" means the capability is present in the source — more access means more to trust, not that it is unsafe.*


## Versions

- **0.1.0** — security scan: passed — Imported from the upstream source.

## Links

- Listing page: https://agentstack.voostack.com/l/skill-lucface-claude-skills-requesting-code-review
- Seller: https://agentstack.voostack.com/s/lucface
- Browse the marketplace: https://agentstack.voostack.com/browse

---
Listed on AgentStack — the marketplace for AI agent skills and MCP servers. Every listing is security-reviewed. Creators keep 70%.