文档

审核并合并最终结果

pull request 中收到的内容、核查要点及退回工作的方法。

Coroid 完成后会在独立分支上创建 pull request,该分支基于你的基准分支生成。它不会直接合并代码,最终决策权仍归你所有。

随附的内容包括:

  • 代码变更内容
  • 随代码一同编写或更新的测试用例
  • 运行现有测试套件后的结果
  • 组织内配置的所有质量门控的判定结果
  • 对代码差异的评审记录,其中已修复的问题会标注清楚,未修复的问题也会列明

此举并非要取代你的评审流程,而是让你的评审成为后续环节,而非首个环节。

实际需核查的内容

机械性校验工作已由系统完成,你只需重点关注只有你才能判断的事项:

  1. **它是否解决了正确的问题?**请将其与需求文档中的验收标准对比,而非凭记忆判断你当初的要求。
  2. **它是否与代码库风格匹配?**编码规范、命名规则、抽象结构等。Coroid 会通过读取代码推断这些规则,大部分情况下都能准确匹配——但偶尔也会有偏差。
  3. **它是否触碰了你意料之外的内容?**查看代码差异前的文件列表即可知晓。
  4. **新增的测试用例是否有实际意义?**无法验证任何逻辑的通过型测试用例,还不如没有测试用例。

退回工作

你有三种可选方案,成本依次递增:

  • 添加评论并请求返工——该任务会连同你的反馈一起返回给开发智能体,上下文信息得以保留。适用于“内容正确但尚未完善”的情况。
  • 驳回方案并重新执行——适用于方案有误而非执行过程出错的情况。
  • 取消任务——适用于该工作根本无需开展的情况。取消操作会终止运行并释放智能体资源。

返工评论需像给同事提意见一样具体明确。“此代码未处理空值情况”这类表述能直接给出修复方向;而“需进一步修改”这类表述则只能靠猜测。parseRange“ produces a fix”能引导出修复方案;“needs work”则只能让人盲目猜测。

合并代码

通过你常用的工具合并代码,遵循常规的分支保护规则、必填校验项及审批要求。Coroid 会创建 pull request,后续的流程仍由你现有的机制管控。

合并流程并无特殊之处——这也是刻意设计的结果。pull request 创建的合并请求与普通请求无异,因此会走你信任的现有评审及CI流程。

下一步

后续指引——根据你想要实现的目标,跳转至文档的其他部分。