GI Code Review

Review code for quality, security, and maintainability following team standards. Use when reviewing pull requests, examining code changes, or when the user asks for a code review.

Code Review 代码审查

按照团队规范对代码进行质量、安全和可维护性审查。适用于 Vue3、FastAPI、Python 技术栈。

何时使用

• 用户请求代码审查
• 审查 Pull Request 或代码变更
• 用户提到「帮我看看这段代码」「review 一下」「代码质量」

审查清单

1. 正确性与逻辑

• 逻辑正确，处理边界情况
• 无明显的空指针/未定义访问
• 异步逻辑正确（async/await 配对）
• 错误处理完整，有 try/except 或 .catch()

2. 安全

• 无 SQL 注入（使用参数化查询）
• 无 XSS（Vue 已转义，注意 v-html）
• 敏感信息不硬编码（用 get_env 或配置）
• 接口有权限校验（如需要）

3. 代码风格与可维护性

• 符合项目约定（router/service/dao 分层）
• 函数职责单一，长度适中
• 命名清晰（变量、函数、路由）
• 有必要的注释和日志

4. 性能

• 无 N+1 查询（批量查询或 join）
• 大列表考虑分页
• 前端避免不必要的重复渲染

5. 测试

• 关键逻辑有测试覆盖
• 边界情况有测试

反馈格式

• 🔴 必须修复：影响正确性、安全或严重性能问题
• 🟡 建议改进：可读性、可维护性、最佳实践
• 🟢 可选优化：锦上添花

项目规范参考

• 后端：app/dao、app/service、app/router、app/model（entity/dto/vo）
• 数据库：使用 AsyncSqlSessionTemplate（insert/update/query_one/query_list）
• 报错：from tkms.exception.api import ApiException
• 配置：from tkms import get_env
• 前端：Vue3 + Ant Design，components/views/services 分层