open-code-review 注释重新定位Re-location提示词解析让 LLM 精确找回评论锚定代码【免费下载链接】open-code-reviewFast, efficient, battle-tested at Alibabas scale. Hybrid architecture code review tool: deterministic pipelines LLM Agent, precise line-level comments, built-in multi-language ruleset (NPE, thread-safety, XSS, SQL injection), OpenAI Anthropic compatible.项目地址: https://gitcode.com/GitHub_Trending/op/open-code-review导读在代码评审中评论必须锚定在 diff 的正确代码行上是行级评审体验的基石。当 LLM 生成的评论携带的existing_code原有代码片段无法通过纯文本匹配定位到 diff 时open-code-review 会触发一条**注释重新定位Re-location**链路构造一份专门的任务提示词请求模型从完整 diff 中抽取与评论语义最相关的连续代码范围再以这段新代码为锚点重新解析行号。本文以 re_location_task_user.md 为骨架结合其配套系统提示词、模板装配机制与底层 Go 实现完整讲解这条链路的工作原理、提示词设计意图、占位符替换规则与容错策略。读完本文你将理解 open-code-review 如何在评论行号解析失败时兜底修复并能够自行解读或定制这类结构化输出提示词。一、Re-location 任务在评审流水线中的位置1.1 为什么需要重新定位open-code-review 的评审 Agent 在产出每条评论时会附带一个ExistingCode字段见 internal/model/review.go它是评论所针对的原始代码片段。评审主循环拿到评论后需要把这段代码映射回 diff 中的具体行号StartLine/EndLine才能生成精确的行级评论。从 internal/diff/resolver.go 可以看到ResolveComment的解析策略若评论已带有行号StartLine 0 || EndLine 0直接视为已定位否则先尝试在 diff 的 hunk 范围内匹配ExistingCode再回退到完整文件内容匹配。问题在于LLM 生成的ExistingCode往往是近似的——可能混入了未改动前的旧代码、省略了缩进、或截取位置与 diff 不一致。纯文本匹配失败后评论就失去了锚点。1.2 三级定位链路在 internal/llmloop/loop.go 的resolveAndCollect中每条评论按以下顺序尝试定位同文件文本匹配diff.ResolveComment(cm, d)在自己的 diff 上做字符串匹配跨文件重定位diff.RelocateAcrossFiles(cm, allDiffs)不调用模型仅凭ExistingCode的逐字比对在内存中所有 diff 中寻找真实归属文件。这是处理Agent 读到了关联文件、却把评论挂到了被评审文件上典型如声明/实现分离的场景详见 internal/diff/resolver.goLLM 重新定位Re-location前两步都失败、且模板中配置了RE_LOCATION_TASK时才构造本文主角——re_location_task提示词调用模型从 diff 中重新抽取准确代码片段。也就是说Re-location 是整个定位链路中的最后一道兜底它在纯规则手段穷尽之后才动用 LLM。二、提示词文件原文解读2.1 用户提示词re_location_task_user.mdre_location_task_user.md 的完整内容如下这是 Re-location 的 user 消息模板Below is a unified diff and a review comment. Identify the minimal contiguous code range in the diff that the comment targets. Rules: 1. Copy the relevant lines VERBATIM from the diff — do not rewrite, reformat, or add anything. 2. Strip leading diff markers (, -, ) from each line before outputting. 3. Include only the lines directly related to the issue — no surrounding context. 4. If multiple disjoint locations apply, pick the single most relevant one. 5. Output ONLY a fenced code block. No explanation, no commentary. **Diff:** diff {diff} **Original code snippet (failed to match):**{existing_code}**Review comment:** {suggestion_content}这条提示词精确传达了三类关键信息任务定义给定一个 unified diff 与一条评审评论找出评论所指向的最小连续代码范围五条硬性规则逐字复制VERBATIM、剥离 diff 标记/-/空格、只保留直接相关行、多位置命中时选最相关的一处、输出必须是唯一的 fenced code block且禁止任何解释文字三个输入占位符{diff}完整 diff、{existing_code}匹配失败的原始代码片段、{suggestion_content}评论正文。2.2 系统提示词re_location_task_system.md配套的 re_location_task_system.md 仅一句话用于设定角色与约束You are a code location assistant. Given a unified diff and a review comment, your sole task is to extract the exact code snippet from the diff that the comment refers to. /no_think值得注意的细节是行尾的/no_think指令——它明确禁止模型输出思考过程配合 user 消息中Output ONLY a fenced code block的约束两者共同保证模型返回的是可被机器解析的纯代码块而非夹杂解释的文本。三、模板装配与占位符替换3.1 任务模板注册task_template.json 中注册了全部 LLM 任务模板其中 Re-location 的定义为RE_LOCATION_TASK: { messages: [ { role: system, prompt_file: re_location_task_system.md }, { role: user, prompt_file: re_location_task_user.md } ] }该 JSON 通过//go:embed打包进二进制见 internal/config/template/template.go由LoadDefault解析resolveOptionalConversation负责读取prompt_file对应的.md内容并组装成LlmConversation见 internal/config/template/template.go。ReLocationTask在结构体中声明为指针类型且带omitempty见 internal/config/template/template.go意味着它是一个可选任务——模板缺失时整条 LLM 重定位链路被静默跳过。3.2 占位符替换实现模板中的{diff}、{existing_code}、{suggestion_content}三个占位符在 internal/diff/relocation.go 的BuildReLocationMessages中被逐一替换content strings.ReplaceAll(content, {diff}, d.Diff) content strings.ReplaceAll(content, {existing_code}, cm.ExistingCode) content strings.ReplaceAll(content, {suggestion_content}, cm.Content)其中d.Diff是当前文件的完整 unified diffcm.ExistingCode是先前匹配失败被保留下来的原始代码片段failed to match的来源cm.Content是评论正文对应suggestion_content。替换后的消息通过llm.NewTextMessage组装为 system/user 两条消息。测试 relocation_test.go 的TestBuildReLocationMessages_Rendering精确验证了这一渲染过程期望的 user 消息文本为diff:\n d.Diff \ncode:\nx : 1\nsuggestion:\nunused variable与模板结构完全一致。同文件中的TestBuildReLocationMessages_NilOrEmptyTaskrelocation_test.go则验证了空任务时返回nil消息、整条链路直接退出的行为。四、Re-location 的调用与结果回收4.1 主循环中的触发逻辑在 internal/llmloop/loop.go 中触发条件严格限定为同文件与跨文件匹配均已失败!located、且模板存在ReLocationTask。触发后依次执行BuildReLocationMessages构造消息在文件会话file session上追加session.ReLocationTask任务记录用于报告与可观测性为本次请求注入 session keySessionTaskKey与requestCtx调用diff.ReLocateComment发起 LLM 请求并把响应的 token 用量prompt/completion/cache累加进总预算统计。4.2 模型调用与代码块抽取ReLocateCommentinternal/diff/relocation.go的工作流为用client.CompletionsWithCtx发起补全请求全程通过telemetry.StartLLMSpan/RecordLLMResult记录耗时与 token调用extractCodeBlockinternal/diff/relocation.go抽取第一个 fenced code block的内容——它跳过开 fence 后的语言标签行如go取到闭合 fence 前的文本并TrimSpace。这正是提示词要求Output ONLY a fenced code block的机器侧原因解析器只认代码块任何多余文字都会导致解析为空用抽取结果覆盖cm.ExistingCode再次调用ResolveComment尝试定位失败则回滚如果新代码片段仍无法在 diff 中解析出行号恢复原始ExistingCode避免用错误内容污染评论。4.3 边界情况均有测试覆盖relocation_test.go 中的测试矩阵完整覆盖了这条链路的成败分支TestReLocateComment_LLMReturnsValidCode模型返回合法代码块且能解析成功TestReLocateComment_LLMReturnsInvalidContent返回内容非代码块抽取为空TestReLocateComment_CodeBlockStillUnresolvable代码块格式正确但内容仍不匹配 diff验证原始ExistingCode的回滚relocation_test.goTestReLocateComment_LLMError与TestReLocateComment_NoMessages请求失败与空消息场景TestExtractCodeBlock单独验证 fence 解析逻辑。五、提示词设计要义与定制指引5.1 为什么要求逐字复制 剥离标记提示词规则 1、2 组合的目的是保证输出与 diff 内容字节级一致。行号解析依赖ExistingCode与 diff 的字符串匹配见ResolveComment→resolveFromHunk/resolveFromFileContent任何改写、重新格式化都会破坏匹配。剥离/-/空格前缀则是统一 diff 的固有要求——fence 内必须是实际代码而非补丁行。5.2 为什么强制单代码块、无解释extractCodeBlock只取第一个 fenced code block 并直接忽略块外文本internal/diff/relocation.go。若模型输出解释文字、或返回多个代码块解析结果都会偏离预期。因此提示词把输出唯一代码块、禁止评论提升为显式规则并通过 system 提示词中的/no_think从源头抑制思考过程混入。5.3 定制注意事项re_location_task_user.md与re_location_task_system.md位于 internal/config/template/prompts/ 目录通过go:embed随二进制编译。若需定制例如调整多位置命中的选择偏好、改变代码块语言标签要求需同时注意保持{diff}、{existing_code}、{suggestion_content}三个占位符不变否则BuildReLocationMessages的strings.ReplaceAll无法填充输出格式约束与 extractCodeBlock 的解析规则保持兼容必须且只能有一个 fenced code blockRE_LOCATION_TASK在task_template.json中缺失时链路整体降级为不尝试 LLM 重定位属安全行为而非错误见 relocation.go 的注释与TestBuildReLocationMessages_NilOrEmptyTask。六、小结Re-location 任务提示词是 open-code-review 精确定位能力的最后一道保障当纯文本匹配与跨文件搜索都无法确定评论锚点时它用一条高度约束的提示词system 角色声明 user 五条规则换取模型输出的单一、逐字、剥离 diff 标记的代码块再由 extractCodeBlock 解析并回填ExistingCode最终在 ResolveComment 中完成行号解析。失败时所有状态回滚评论保持原样不产生错误定位。从 task_template.json 的注册、relocation.go 的装配与调用到 relocation_test.go 的十数个测试用例这条链路体现了确定性规则优先、LLM 兜底、可观测、可回滚的工程设计哲学也是理解 open-code-review 混合架构确定性管线 LLM Agent的一个典型切面。【免费下载链接】open-code-reviewFast, efficient, battle-tested at Alibabas scale. Hybrid architecture code review tool: deterministic pipelines LLM Agent, precise line-level comments, built-in multi-language ruleset (NPE, thread-safety, XSS, SQL injection), OpenAI Anthropic compatible.项目地址: https://gitcode.com/GitHub_Trending/op/open-code-review创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考