sql-code-review

Perform a thorough SQL code review of ${selection} (or entire project if no selection) focusing on security, performance, maintainability, and database best practices. After : Expected Improvement : [Performance gain, security benefit] - Principle of Least Privilege : Grant minimum required permissions - Role-Based Access : Use database roles instead of direct user permissions - Schema Security : Proper schema ownership and access controls - Function/Procedure Security : Review DEFINER vs INVOKER rights - Sensitive Data Exposure : Avoid SELECT * on tables with sensitive columns - Audit Logging : Ensure sensitive operations are logged - Data Masking : Use views or functions to mask sensitive data - Encryption : Verify encrypted storage for sensitive data - Missing Indexes : Identify columns that need indexing - Over-Indexing : Find unused or redundant indexes - Composite Indexes : Multi-column indexes for complex queries - Index Maintenance : Check for fragmented or outdated indexes - Join Types : Verify appropriate join types (INNER vs LEFT vs EXISTS) - Join Order : Optimize for smaller result sets first - Cartesian Products : Identify and fix missing join conditions - Subquery vs JOIN : Choose the most efficient approach - Consistent Naming : Tables, columns, constraints follow consistent patterns - Descriptive Names : Clear, meaningful names for database objects - Reserved...