code-reviewer
Review code for best practices, security issues, and potential bugs. Use when reviewing code changes, checking PRs, analyzing code quality, or performing security audits.
Works with
--- name: code-reviewer description: Review code for best practices, security issues, and potential bugs. Use when reviewing code changes, checking PRs, analyzing code quality, or performing security audits. license: MIT --- # Code Reviewer Perform comprehensive code reviews focusing on quality, security, and maintainability. ## Instructions 1. Read the target files using the Read tool 2. Search for patterns and related code using Grep 3. Find related files using Glob 4. Analyze code against the review checklist 5. Provide structured feedback with severity levels ## Review Checklist ### Code Quality - [ ] Code is simple and readable - [ ] Functions and variables are well-named - [ ] No duplicated code (DRY principle) - [ ] Appropriate comments for complex logic - [ ] Consistent code style ### Security - [ ] No exposed secrets or API keys - [ ] Input validation implemented - [ ] SQL injection prevention - [ ] XSS prevention for web code - [ ] Proper authentication/authorization checks ### Error Handling - [ ] Errors are caught and handled appropriately - [ ] Meaningful error messages - [ ] No silent failures - [ ] Proper logging for debugging ### Performance - [ ] No obvious performance bottlenecks - [ ] Efficient algorithms and data structures - [ ] Appropriate caching where needed - [ ] Database queries are optimized ### Testing - [ ] Adequate test coverage - [ ] Edge cases are tested - [ ] Tests are readable and maintainable ## Output Format Organize feedback by severity: ### Critical (Must Fix) Issues that could cause security vulnerabilities, data loss, or crashes. ### Warning (Should Fix) Issues that could cause bugs, poor performance, or maintenance problems. ### Suggestion (Consider) Improvements for readability, consistency, or best practices. ## Example Feedback ```markdown ### Critical - **SQL Injection vulnerability** in `user_service.py:45` - User input passed directly to query without sanitization - Fix: Use parameterized queries ### Warning - **Missing error handling** in `api_client.py:23` - Network errors will crash the application - Fix: Add try/catch with appropriate error response ### Suggestion - Consider extracting the validation logic in `validators.py:78-95` into a separate function for reusability ```
More Security skills
azure-cost
microsoft/azure-skills
Azure cost management: query costs, forecast spending, optimize to reduce waste. WHEN: \"Azure costs\", \"Azure bill\", \"cost breakdown\", \"how much am I spending\", \"forecast spending\", \"optimize costs\", \"reduce spending\", \"orphaned resources\", \"rightsize VMs\", \"cost spike\", \"reduce storage costs\", \"AKS cost\". DO NOT USE FOR: deploying resources, provisioning, diagnostics, or security audits.
entra-app-registration
microsoft/azure-skills
Guides Microsoft Entra ID app registration, OAuth 2.0 authentication, and MSAL integration. USE FOR: create app registration, register Azure AD app, configure OAuth, set up authentication, add API permissions, generate service principal, MSAL example, console app auth, Entra ID setup, Azure AD authentication. DO NOT USE FOR: Key Vault secrets (use azure-keyvault-expiration-audit), general Azure resource security guidance.
azure-messaging
microsoft/azure-skills
Troubleshoot and resolve issues with Azure Messaging SDKs for Event Hubs and Service Bus. Covers connection failures, authentication errors, message processing issues, and SDK configuration problems. WHEN: event hub SDK error, service bus SDK issue, messaging connection failure, AMQP error, event processor host issue, message lock lost, message lock expired, lock renewal, lock renewal batch, send timeout, receiver disconnected, SDK troubleshooting, azure messaging SDK, event hub consumer, service bus queue issue, topic subscription error, enable logging event hub, service bus logging, eventhub python, servicebus java, eventhub javascript, servicebus dotnet, event hub checkpoint, event hub not receiving messages, service bus dead letter, batch processing lock, session lock expired, idle timeout, connection inactive, link detach, slow reconnect, session error, duplicate events, offset reset, receive batch.
