代码审查
检查具体差异,形成可定位、可复现的审查发现。
选择审查对象
代码审查适合在提交前检查未提交差异、比较某个基准分支,或复核单个提交。先说明希望审查的行为和风险,避免把整个仓库的问题混入本次改动。
| 目标 | CLI 入口 |
|---|---|
| 未提交改动 | lime review --uncommitted |
| 相对基准分支的差异 | lime review --base main |
| 一个提交 | lime review --commit <COMMIT_SHA> |
| 特定问题 | lime review "检查本次变更的授权和错误处理" |
从交互会话开始
在会话中说明审查目标,或直接使用 lime review。先用 git diff 与 git status 检查实际差异;未跟踪文件也应纳入注意范围。审查不会自动把发现的问题视为已修复。
非交互审查
lime review --uncommitted
lime review --base main
lime exec review --commit <COMMIT_SHA>--uncommitted、--base、--commit 与自定义提示词互斥。--title 用于提交审查标题,必须与 --commit 一起使用。引用真实分支或 SHA,并确认本地仓库有相应对象。
写清审查要求
审查当前差异。重点检查跨租户访问、失败后重试和旧数据兼容。只报告本次改动引入且有代码依据的问题,给出路径、触发条件、影响和修复建议。可将稳定要求写入 AGENTS.md。为审查提供行为约束、测试命令和已知边界,而不是只给一串风格偏好。
理解审查结果
| 信息 | 应回答的问题 |
|---|---|
| 位置 | 哪一个文件和相关代码触发问题? |
| 条件 | 什么输入、权限、状态或调用顺序会触发? |
| 影响 | 用户、数据或运行行为受到什么影响? |
| 证据 | 能否通过测试、调用路径或可复现步骤验证? |
优先处理确定的行为问题。可读性建议和猜测性的风险应与缺陷区分;没有发现问题也不等于已经证明全部行为正确。
修复与复核
- 确认发现仍适用于当前差异。
- 实施最小修复,避免混入无关重构。
- 执行相关测试并再次检查差异。
- 报告已解决、未解决及无法验证的部分。
仓库服务集成
Lime 的本地 review 命令可以作为组织 CI 的一部分。自动监听拉取请求、发布评论和托管审查需要组织另外配置执行器与仓库权限;本地审查不会自动创建远程检查。
查看文件与上下文
逐个检查改动文件,必要时阅读调用方、存储与测试。对差异提问时给出具体路径和行为,不让摘要替代源码核对。多个仓库分别确认基准、权限与验证入口,不能用一个仓库的结果代表全部。
暂存、还原与反馈
暂存状态决定下次提交包含哪些差异,与 Lime 是否看过文件无关。使用 Git 现有入口检查已暂存与未暂存内容;还原前确认范围,避免覆盖用户改动。未跟踪文件需要另外检查。
对一条发现继续追问时,引用其位置、触发条件和证据。发布行内评论或还原文件是独立写入动作,需要明确目标与授权。
拉取请求与远程审查
从仓库服务读取拉取请求时,核对作者、目标分支、头提交和检查状态。已更新的 PR 可能不再对应旧审查结论;先比较版本,再继续处理。
Lime 不自动提供聚合 PR 收件箱、托管云端审查面板或关注队列。可使用组织批准的 GitHub/GitLab 工具获取必要元数据,再把当前差异交给本地审查。