科技行业研发部工程师代码审查规范手册（执行版）.docxVIP

科技行业研发部工程师代码审查规范手册（执行版）

第1章代码审查基础与职责界定

1.1审查流程概览与准入标准

审查流程始于代码提交系统（CI/CDPipeline）触发，当PullRequest（PR）合并请求状态变更为“待审查”时，系统自动向指定Reviewer发送审查通知，此时审查者需确认自己拥有该分支的访问权限及代码审查权限，否则流程将自动阻断。准入标准规定，代码必须经过至少一次本地静态分析（如SonarQube扫描）且无严重（Critical）及以上级别的阻塞性错误才能进入审查阶段，严重错误通常指代码中使用了未授权的API或存在逻辑死循环，此类代码严禁进入审查环节。

审查者需确保代码提交者完成了所有必要的单元测试覆盖，单元测试覆盖率低于70%的代码块将被视为未准备好审查，审查者有权要求提交者补充测试用例直至达标。审查流程中必须包含代码风格检查（CodeStyleCheck），若代码未通过ESLint或Prettier的代码规范检查，审查者应直接拒绝合并请求并通知提交者修正，以避免后续维护成本增加。审查者需确认代码中是否存在未处理的空指针异常（NPE）或空字符串处理逻辑，若发现此类隐患，审查者应要求提交者增加防御性编程代码，如添加非空判断或默认值填充。

审查流程的闭环要求审查结论必须明确记录为“合并”、“拒绝合并”或“建议修