360 Secure SkillHub
360 安全技能中心

GI Code Review

核心功能：围绕正确性、安全、代码风格、性能、测试5类维度，按预设清单及团队规范对代码多维度检查，输出「必须修复/建议改进/可选优化」分级反馈。典型场景：响应代码审查请求、Pull Request审查、代码变更校验。适配Vue3、FastAPI、Python栈，输入待审查代码，输出结构化审查意见。

laimiaohua
laimiaohua
热度 66.05
最近更新 2026-05-11 17:32:28
分类：开发依赖：纯提示词应用场景：代码审查

安装技能

复制下面的话术发给你的 Agent，它会通过 secure-skills 检索并安装该技能。

执行命令安装此 skill： npx secure-skills-cli add gi-code-review
Metadata
name
gi-code-review
description
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.
tags
["code-review","quality","security","best-practices","vue","fastapi","python"]

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 分层