多轴代码审查与质量门禁

解决什么问题

很多团队的合并前审查止步于「能跑就行」。审查者看一眼 diff、随手点几个文件，就给出 approve，于是缺陷在运行环境里才暴露：逻辑在边界条件上翻车、安全漏洞混在重构里、慢查询被当成「性能之后再说」。这类浅层 spot check 的问题在于，它把审查当成一次个人判断，而不是一套可重复的工程流程。

技能如何工作

这个技能把审查拆成固定轴线，逐一推进，避免遗漏：

• Correctness：核对改动在边界输入、错误路径、并发场景下的实际行为，而不是只看函数签名。
• Readability：检查命名、嵌套深度、函数长度是否已经妨碍后续维护。
• Architecture：判断本次改动是顺着现有分层走，还是在悄悄破坏依赖方向。
• Security：审注入、鉴权、密钥与敏感数据处理路径。
• Performance：识别 N+1 查询、重复计算、无界缓存这类随流量放大的问题。

工作方式是结构化预合并审查：拿到仓库或 PR diff 之后，沿各轴线产出具体结论，并把测试与验证信号作为证据，而不是靠直觉。

适用边界

它不替代持续集成，也不自动改动代码。项目约定、测试覆盖和可用的验证信号决定了审查能走多深；缺少这些输入时，结论需要人工确认。