Web 实践
代码审查
Agent 自检和评审流程
质量审查原则
- 生成不等于完成:代码/文档生成后必须经过审查才算完成
- 关键节点 Review:每完成一个 todo item 后,主动询问用户是否需要 review
- 发现问题立即修复:不要累积问题到后面再处理
Review 内容
代码审查
| 检查项 | 说明 |
|---|---|
| 逻辑正确性 | 代码是否实现了预期功能 |
| 类型安全 | TypeScript 类型是否完整 |
| 边界处理 | 空值、异常、边界条件 |
| 安全性 | 无 XSS、SQL 注入等漏洞 |
文档审查
| 检查项 | 说明 |
|---|---|
| 内容准确性 | 描述是否与代码一致 |
| 表达清晰度 | 是否易于理解 |
| 示例可用性 | 代码示例是否可运行 |
对话评审(自动执行)
每次输出末尾
附加简化进度:
进度:📐设计 → ⚡执行 | 轮次 5 | +32 -10 行阶段切换时
显示完整进度条:
进入 ⚡执行 阶段
[🔍 澄清] → [📐 设计] → [⚡ 执行] → [✅ 校验] → [📝 总结]
▲ 当前任务完成时
显示效率指标:
📊 本次对话统计
轮次:12 | Tokens:~8.2k | 变更:+156 -23 ~45
🤖 15min | 🧑💻 3h | ⬇️ 2.75h工作流阶段
[🔍 澄清] → [📐 设计] → [⚡ 执行] → [✅ 校验] → [📝 总结]| 阶段 | 说明 |
|---|---|
| 🔍 澄清 | 理解需求,确认范围 |
| 📐 设计 | 方案设计,技术选型 |
| ⚡ 执行 | 编写代码,实现功能 |
| ✅ 校验 | 测试验证,代码审查 |
| 📝 总结 | 更新文档,提交变更 |
PR 检查清单
提交 PR 前确认:
-
bun run lint通过 -
bun run build通过 -
bun run registry:sync已执行 - PROJECT_STATE.md 已更新
- Commit 信息规范