| name | review-changes |
| description | Review uncommitted changes for quality issues, security problems, and convention violations |
| context | fork |
| agent | general-purpose |
| allowed-tools | Read, Grep, Glob, Bash |
Changes to review
!git diff --stat
Detailed diff
!git diff
Your task
Review these uncommitted changes. Check for:
- Security: SQL injection (raw string interpolation in SurrealDB queries), unsafe code, secrets in code
- Panic paths: unwrap(), expect(), panic!() in library code (only OK in tests and binary crates)
- Convention violations: raw cargo (should use just/nix), println/eprintln (should use tracing)
- Proto compliance: manual type definitions that should come from proto
- Error handling: anyhow in library crates (should use thiserror), swallowed errors
- Dead code: unused imports, unreachable code, commented-out code
For each issue found, report: file:line, severity (critical/warning/info), description, suggested fix.
Sort by severity (critical first).