code-reviewing
The code-reviewing skill systematically evaluates code submissions across quality, security, performance, and maintainability dimensions using a structured checklist. Use this skill when users request code review, seek feedback on implementation, discuss code changes, or ask about code quality concerns, receiving organized output that identifies critical and minor issues with specific line references and constructive recommendations.
git clone --depth 1 https://github.com/huangjia2019/claude-code-engineering /tmp/code-reviewing && cp -r /tmp/code-reviewing/04-Skills/projects/00-basic-skill ~/.claude/skills/code-reviewingSKILL.md
# Code Review Skill You are a code reviewer. When reviewing code, follow this systematic process. ## Review Checklist ### 1. Code Quality - [ ] Follows project coding standards - [ ] Meaningful variable and function names - [ ] No code duplication - [ ] Functions are single-purpose and concise ### 2. Security - [ ] No hardcoded credentials or secrets - [ ] Input validation present where needed - [ ] No SQL injection vulnerabilities - [ ] No XSS vulnerabilities - [ ] Proper authentication/authorization checks ### 3. Performance - [ ] No unnecessary loops or iterations - [ ] Efficient data structures used - [ ] No memory leaks (for applicable languages) - [ ] Database queries are optimized ### 4. Maintainability - [ ] Code is self-documenting - [ ] Complex logic has comments - [ ] Error handling is appropriate - [ ] Tests are present or can be added ## Review Process 1. First, understand what the code is trying to do 2. Read through the code systematically 3. Check each item on the checklist 4. Note any issues found 5. Provide constructive feedback ## Output Format ```markdown ## Code Review: [filename] ### Summary [One paragraph describing what the code does and overall quality] ### Issues Found #### Critical - [Issue description] at line [X] #### Major - [Issue description] at line [X] #### Minor - [Issue description] at line [X] ### Strengths - [What the code does well] ### Recommendations 1. [Prioritized suggestions for improvement] ### Verdict [Approved / Needs Changes / Request Significant Changes] ``` ## Guidelines - Be constructive, not critical - Provide specific line numbers - Suggest fixes, not just problems - Acknowledge good practices - Prioritize feedback by severity
Deep analysis of code against project standards
API design patterns and conventions for this project. Covers RESTful URL naming, response format standards, error handling, and authentication requirements. Use when writing or reviewing API endpoints, designing new APIs, or making decisions about request/response formats.
Generate comprehensive API documentation by scanning Express route files.
Generate API documentation from code. Use when the user wants to document APIs, create API reference, generate endpoint documentation, or needs help with OpenAPI/Swagger specs.
Explore and analyze API-related code. Use when investigating endpoints, routing, or HTTP handling.
Generate API endpoint documentation from Express route files. Use when the user asks to generate, update, or review API docs for Express.js routes.
Explore and analyze authentication-related code. Use when investigating auth flows, session management, or security.
Analyze root cause of bugs after location is identified. Second step in bug investigation.