用途:为代码审查提供统一框架与标准,适用于审查 Pull Request、执行代码评审、定义评审标准以及建立评审流程,目标是尽早发现缺陷、避免问题进入生产环境。该技能聚合多个专项技能的标准,覆盖正确性、安全、TypeScript 质量、测试、性能、代码质量与架构合规等维度。
主要能力:采用 4 级严重度分类——CRITICAL(合并前必须修复,如安全漏洞、数据丢失风险、认证绕过、SQL 注入)、HIGH(合并前应修复,如 TypeScript 严格模式违规、缺少错误处理、性能问题如 N+1 查询、缺少输入校验)、MEDIUM(可带计划合并,如代码质量、缺少测试、命名与文档问题)、LOW(风格建议、优化与重构想法)。提供审查清单,覆盖正确性(边界情况、错误条件、异步等待、竞态、off-by-one)、安全(硬编码密钥、Zod 输入校验、认证与授权、SQL 注入防护、XSS、CSRF、日志敏感数据、密码哈希 bcrypt 12+ 轮、JWT 校验)、TypeScript 质量(不用 any、不使用 @ts-ignore 与 ! 除非有充分注释、参数与返回值显式类型、类型守卫、泛型使用)、测试(AAA 模式、覆盖率阈值 75%/90%、UI 测试验证 DOM、E2E 测试、无理由跳过测试、测试独立与清理)、性能(N+1、查询优化、异步并行、分页、图片优化、不必要重渲染、昂贵计算记忆化)、代码质量(生产代码无 console.log、无注释掉的代码、TODO 需关联 GitHub issue、单一职责、命名描述性、无死代码与重复逻辑)、架构合规(模式选择与实现是否正确、Next.js 最佳实践、服务端与客户端组件、状态管理)。
使用方式:按流程推进——先做预审(阅读 PR 描述、理解改动与原因、检查 CI/CD 状态、识别高风险区域),再做安全审查(所有 PR 检查硬编码密钥、输入校验、认证授权;Auth/API/Data 类 PR 运行 security-sentinel 并参考 OWASP Top 10 与注入风险),随后进行代码审查(正确性、TypeScript、测试、性能、质量、架构),再按严重级别与模板撰写反馈,最后给出结论:APPROVE、REQUEST CHANGES 或 COMMENT。反馈模板包含位置、问题、风险、修复建议与文件路径。审查框架还提供代码坏味道示例,如超长函数、深层嵌套(用早返回改善)、魔法数字(改为命名常量)。渐进披露文件包括 security-checklist.md、performance-criteria.md、maintainability-rules.md、review-templates.md。
适用场景:团队建立或统一代码评审标准,评审认证、支付、数据处理等高风险改动,以及作为 PR 审查流程的检查清单。
限制:资料未提供安装命令、运行环境、支持平台、收费与价格等信息;主要面向 TypeScript/Next.js 技术栈,部分标准(如 Drizzle、Zod、bcrypt、JWT)与特定技术选型相关。