Skip to content

fix(review): keep Chinese as the primary review language - #7446

Merged
huangruiteng merged 1 commit into
mainfrom
codex/pr-review-chinese-primary-20261027
Oct 27, 2026
Merged

huangruiteng merged 1 commit into
mainfrom
codex/pr-review-chinese-primary-20261027

Conversation

@loopx-agent

Copy link
Copy Markdown
Collaborator

评审把“保留英文技术术语”误写成了“正文使用英文结构、配简短中文说明”,导致主要分析和证据成了长段英文。此修改让正文、问题分析、证据与结论以中文为主,保留准确的英文术语,以及一条简短的 English verdict。

四处共享指导同步修正:capability 的 review_template.output_hint、README、review skill 和默认 prompt。五段评审、原证据门槛与 finding 要求继续生效;没有增加语言比例或文本分类器,也没有改变 parser、policy revision、review/merge 权限或默认执行路径。边界内复用现有输出指导,不增加语言配置或独立 owner。

近期 7 个 PR 上的 9 份本执行评审已直接编辑原正文并读回;原 review ID、commit、状态和发表时间保持不变,两份 REQUEST_CHANGES 的阻塞发现保留。这里只修订自己的历史正文,没有给更新后的 head 新增批准。

验证针对 9ddf61a09003d8a2f554e35e0ba587d096a64a4c,基于抓取的目标 main bb150595664fe0e910db9587cff22ea3b471f5ee:

  • 五个现有 review 测试模块:333 通过、43 跳过。跳过的是 LOOPX_REVIEW_LIVE_TEST 未启用的真实模型资格测试,未执行或声称通过。
  • 风险预检:6 direct+17 selected 自动检查通过,包括 review workflow、语义词汇 advisory、RFC pairing 和四文件 public-boundary 扫描;无自动检查失败、无 tracked side effect。
  • 预检整体仍是 quality_stale_receipt:旧质量凭证不覆盖此 base/diff,合并前必须按当前 scope 重新资格化。没有绕过这一门禁,未查询或等待 GitHub CI。
  • 9 份远端正文逐字读回,并用原发表版本的契约检查结构、发现和证据一致性。这属于历史正文编辑核验,不代替新 head 的 runtime review。

此 PR 是完整的指导修复提案;尚未合并或安装。真实模型是否遵循新指导、跨执行采用和长期效果仍未资格化。

视觉证据

实际 GitHub 上的同一份 #7409 评审,仍绑定原 head 4d0524bc14708bbe0aa1b16e7f82a8870ff56071。以下是已发表正文的实际浏览器结果,不是新指导已安装的证明。没有 App/Lark/CLI 页面布局改动。

桌面 1200×888,修改前:

修改前:五段内出现长段英文分析

桌面 1200×888,修改后:

修改后:中文连续说明,保留英文技术术语

窄屏 390×844:

窄屏:正文中文换行,版本与声明仍可读

长内容下半页,含验证失败、限制与结论:

完整长评审:失败记录和结论仍保留

整页检查覆盖 35 段正文、一张对照表、长 commit 和混合的 PASS/FAIL/未验证状态。五段层级可区分,390px 页面没有全局横向溢出;风险与结论继续使用中文说明。首屏仍有必要的 reviewer 声明和原 head 提示,长评审保留其阅读成本。

Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
@huangruiteng
huangruiteng merged commit 47a525d into main Oct 27, 2026
10 of 33 checks passed
@huangruiteng
huangruiteng deleted the codex/pr-review-chinese-primary-20261027 branch October 27, 2026 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants