53 lines
3.1 KiB
Markdown
53 lines
3.1 KiB
Markdown
---
|
||
title: "Code Reviewer Agent Personality"
|
||
type: source
|
||
tags: []
|
||
date: 2026-05-02
|
||
---
|
||
|
||
## Source File
|
||
- [[Agent/agency-agents/engineering/engineering-code-reviewer.md]]
|
||
|
||
## Summary（用中文描述）
|
||
- 核心主题：Code Reviewer AI Agent 人格定义——专注于代码审查与质量保证的专业 AI Agent
|
||
- 问题域：代码质量保障（正确性、安全性、可维护性、性能）
|
||
- 方法/机制：分层审查清单（Blocker/Suggestion/Nit 三级优先级）、标准化审查评论格式、教导式反馈风格
|
||
- 结论/价值：最佳审查教会而非批评，提升代码质量 AND 开发者技能
|
||
|
||
## Key Claims（用中文描述）
|
||
- Code Reviewer 提供构造性、可操作的反馈，聚焦正确性、安全性、可维护性、性能，而非代码风格偏好
|
||
- 审查重点五大维度：正确性（功能是否符合预期）、安全性（注入/XSS/认证漏洞）、可维护性（六个月后是否可理解）、性能（瓶颈/N+1查询）、测试（重要路径是否覆盖）
|
||
- 审查评论必须具体化：指出具体行号和机制，而非泛泛说"有安全问题"
|
||
- 必须解释原因：不仅说改什么，还要说为什么
|
||
- 建议而非要求：用"考虑使用X因为Y"而非"改成X"
|
||
- 标记优先级：🔴 blocker（必须修复）、🟡 suggestion（应该修复）、💭 nit（锦上添花）
|
||
- 赞美好代码：指出聪明的解决方案和干净的模式
|
||
- 一次审查完整反馈：不要多轮滴水式评论
|
||
|
||
## Key Quotes
|
||
> "Reviews code like a mentor, not a gatekeeper. Every comment teaches something." — Code Reviewer 核心理念
|
||
|
||
> "🔴 Security: SQL Injection Risk — Line 42: User input is interpolated directly into the query. Why: An attacker could inject `'; DROP TABLE users; --`. Suggestion: Use parameterized queries." — 标准化审查评论格式示例
|
||
|
||
## Key Concepts
|
||
- [[CodeReview]]: 代码审查的核心概念——通过系统性检查提升代码质量的过程
|
||
- [[SecurityAudit]]: 安全审查维度——注入、XSS、认证绕过等漏洞检测
|
||
- [[PerformanceProfiling]]: 性能分析——N+1查询、不必要内存分配等瓶颈识别
|
||
- [[Mentorship]]: 教导式反馈——审查即教学，每次评论都有教育意义
|
||
- [[ReviewChecklist]]: 审查清单——结构化的三层优先级体系（Blocker/Suggestion/Nit）
|
||
|
||
## Key Entities
|
||
- [[CodeReviewerAgent]]: Code Reviewer AI Agent 本身——代码审查与质量保证专家
|
||
|
||
## Connections
|
||
- [[SoftwareArchitect]] ← informs → [[CodeReviewerAgent]]：架构决策影响审查重点
|
||
- [[BackendArchitect]] ← depends_on → [[CodeReviewerAgent]]：后端实现依赖代码审查保障质量
|
||
- [[SeniorDeveloper]] ← extends → [[CodeReviewerAgent]]：资深开发者兼具审查职责
|
||
- [[QualityGate]] ← implements → [[CodeReviewerAgent]]：质量门控的自动化实现依赖审查标准
|
||
|
||
## Contradictions
|
||
- 与纯风格审查工具（如 linter）的冲突：
|
||
- 冲突点：风格工具负责格式化，Code Reviewer 负责人工判断
|
||
- 当前观点：Code Reviewer 明确拒绝审查风格偏好（"not tabs vs spaces"），将风格交给 linter
|
||
- 对方观点：某些团队要求审查者确保代码风格一致
|