Repository navigation
fix(review): keep Chinese as the primary review language - #7446
Merged
Merged
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
评审把“保留英文技术术语”误写成了“正文使用英文结构、配简短中文说明”,导致主要分析和证据成了长段英文。此修改让正文、问题分析、证据与结论以中文为主,保留准确的英文术语,以及一条简短的
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,基于抓取的目标 mainbb150595664fe0e910db9587cff22ea3b471f5ee:LOOPX_REVIEW_LIVE_TEST未启用的真实模型资格测试,未执行或声称通过。quality_stale_receipt:旧质量凭证不覆盖此 base/diff,合并前必须按当前 scope 重新资格化。没有绕过这一门禁,未查询或等待 GitHub CI。此 PR 是完整的指导修复提案;尚未合并或安装。真实模型是否遵循新指导、跨执行采用和长期效果仍未资格化。
视觉证据
实际 GitHub 上的同一份 #7409 评审,仍绑定原 head
4d0524bc14708bbe0aa1b16e7f82a8870ff56071。以下是已发表正文的实际浏览器结果,不是新指导已安装的证明。没有 App/Lark/CLI 页面布局改动。桌面 1200×888,修改前:
桌面 1200×888,修改后:
窄屏 390×844:
长内容下半页,含验证失败、限制与结论:
整页检查覆盖 35 段正文、一张对照表、长 commit 和混合的 PASS/FAIL/未验证状态。五段层级可区分,390px 页面没有全局横向溢出;风险与结论继续使用中文说明。首屏仍有必要的 reviewer 声明和原 head 提示,长评审保留其阅读成本。