---
name: code-review
description: Perform code reviews following Sentry engineering practices. Use when reviewing pull requests, examining code changes, or providing feedback on code quality.…
---

# code-review

Perform code reviews following Sentry engineering practices. Use when reviewing pull requests, examining code changes, or providing feedback on code quality.…

# code-reviewby sentry
Perform code reviews following Sentry engineering practices. Use when reviewing pull requests, examining code changes, or providing feedback on code quality.…

`npx skills add https://github.com/getsentry/sentry-skills --skill code-review`Download ZIPGitHub

## Sentry Code Review

Follow these guidelines when reviewing code for Sentry projects.

## Review Checklist

### Identifying Problems

Look for these issues in code changes:

* Runtime errors: Potential exceptions, null pointer issues, out-of-bounds access

* Performance: Unbounded O(n²) operations, N+1 queries, unnecessary allocations

* Side effects: Unintended behavioral changes affecting other components

* Backwards compatibility: Breaking API changes without migration path

* ORM queries: Complex Django ORM with unexpected query performance

* Security vulnerabilities: Injection, XSS, access control gaps, secrets exposure

### Design Assessment

* Do component interactions make logical sense?

* Does the change align with existing project architecture?

* Are there conflicts with current requirements or goals?

### Test Coverage

Every PR should have appropriate test coverage:

* Functional tests for business logic

* Integration tests for component interactions

* End-to-end tests for critical user paths

Verify tests cover actual requirements and edge cases. Avoid excessive branching or looping in test code.

### Long-Term Impact

Flag for senior engineer review when changes involve:

* Database schema modifications

* API contract changes

* New framework or library adoption

* Performance-critical code paths

* Security-sensitive functionality

## Feedback Guidelines

### Tone

* Be polite and empathetic

* Provide actionable suggestions, not vague criticism

* Phrase as questions when uncertain: "Have you considered...?"

### Approval

* Approve when only minor issues remain

* Don't block PRs for stylistic preferences

* Remember: the goal is risk reduction, not perfect code

## Common Patterns to Flag

### Python/Django

```
`# Bad: N+1 query
for user in users:
print(user.profile.name) # Separate query per user

# Good: Prefetch related
users = User.objects.prefetch_related('profile')
`
```

### TypeScript/React

```
`// Bad: Missing dependency in useEffect
useEffect(() => {
fetchData(userId);
}, []); // userId not in deps

// Good: Include all dependencies
useEffect(() => {
fetchData(userId);
}, [userId]);
`
```

### Security

```
`# Bad: SQL injection risk
cursor.execute(f"SELECT * FROM users WHERE id = {user_id}")

# Good: Parameterized query
cursor.execute("SELECT * FROM users WHERE id = %s", [user_id])
`
```

## References

* Sentry Code Review Guidelines

## More skills from sentry
sentry-cocoa-sdkby sentryFull Sentry SDK setup for Apple platforms (iOS, macOS, tvOS, watchOS, visionOS). Use when asked to "add Sentry to iOS", "add Sentry to Swift", "install…sentry-create-alertby sentryCreate Sentry alerts using the workflow engine API. Use when asked to create alerts, set up notifications, configure issue priority alerts, or build workflow…sentry-dotnet-sdkby sentryFull Sentry SDK setup for .NET. Use when asked to "add Sentry to .NET", "install Sentry for C#", or configure error monitoring, tracing, profiling, logging, or…sentry-fix-issuesby sentryFind and fix issues from Sentry using MCP. Use when asked to fix Sentry errors, debug production issues, investigate exceptions, or resolve bugs reported in…sentry-go-sdkby sentryFull Sentry SDK setup for Go. Use when asked to "add Sentry to Go", "install sentry-go", "setup Sentry in Go", or configure error monitoring, tracing, logging,…sentry-ios-swift-setupby sentrySetup Sentry in iOS/Swift apps. Use when asked to add Sentry to iOS, install sentry-cocoa SDK, or configure error monitoring for iOS applications using Swift…sentry-nextjs-sdkby sentryFull Sentry SDK setup for Next.js. Use when asked to "add Sentry to Next.js", "install @sentry/nextjs", or configure error monitoring, tracing, session replay,…sentry-otel-exporter-setupby sentryConfigure the OpenTelemetry Collector with Sentry Exporter for multi-project routing and automatic project creation. Use when setting up OTel with Sentry,…

---

**Source**: https://github.com/getsentry/sentry-skills/tree/HEAD/skills/code-review
**Author**: sentry
**Discovered via**: mcpservers.org