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

!git diff --stat

!git diff

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).

Más de este repositorio

mismo repositorio

Inspect the PostgreSQL schema, tables, and data for a running lifelog instance

Validate, stage, and commit current changes with a conventional commit message

Diagnose and fix build failures systematically

Explain how a specific part of the lifelog architecture works with diagrams and data flow

Trace a data modality end-to-end from proto definition through collector, server, and storage

Fix a GitHub issue end-to-end with tests and validation

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

!git diff --stat

!git diff

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).