Agent skill · software engineering · aiskillstore
code-review
Perform code reviews following Sentry engineering practices. Use when reviewing pull requests, examining code changes, or providing feedback on code quality. Covers security, performance, testing, and design review.
Why this skill is useful
Provides specific guidelines and patterns for code review in Sentry projects that the AI wouldn't reliably generate on its own.
What it needs
About 1k tokens when loaded. Last updated 2026-08-07. 405 stars on the source repository.
What this skill does
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 TypeScript/React Security References Sentry Code Review Guidelines
How to use it
Reference it in AdaL, Claude Code, Cursor or any coding agent — nothing to install:
@skills aiskillstore/code-review--de1bea