fix: 评审 / 复评 / diff 一批交互改良与修复#113
Merged
Merged
Conversation
定位路径过长换行时，路径(code)与行号(span)作为各自独立的 flex 项，行号被挤到第三行左侧。 改为定位文本以行内文本流排版（非 flex），路径与行号连续换行——行号紧跟路径末行（path:line）， 首行仍与图标同排。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
点击复评引用徽标定位到原 finding 卡片时背景渐隐高亮看不出来——finding 卡有实底 $bg-elev， 背景渐隐（chat-run-flash）会被洗掉。改用与卡片底色无关的「覆盖式高亮环 + inset 淡色填充」 （chat-finding-flash），任意底色都清晰可见；启用卡用主蓝、已关闭/拒绝卡用中性灰，色相区分两类 状态。run 级定位（查看复评）仍沿用背景渐隐。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
走到 manual_review 解析兜底时，理由文案对用户无价值。改为兜底 reason 置空——前端按空 reason 隐藏灰字（ConversationMessage 已有 `reason &&` 守卫），仅保留判定徽标。模型给出的合法 manual_review 理由仍照常展示。同步移除已无用的 parseFail label 与各 locale 文案。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
启用 finding 卡的定位高亮原先统一用主蓝，与卡片左条类别色不一致。改为高亮色 = 左条主题色 （--flash-color 按 sectionKey 取，逐一对应 border-left-color 分组）：代码反馈/建议=橙黄、 元信息=蓝、概览/总结=绿、过程/评分=灰；已关闭/拒绝卡仍统一中性灰。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
状态角标（绿勾/琥珀叹号）尺寸 16→14、内部符号同步 16→14；去掉角标彩底的灰色描边环 （box-shadow），纯彩底靠颜色与头像区分。头像本身不变。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
此前已有远端评论的行被算作「占用」、hover 不出 + 图标，无法在同一行继续追加评论。改为仅 「已有未发布草稿」的行才抑制 +（避免同行两个编辑器），已有远端评论的行照常 hover 出 +； 点击新建的草稿 zone 挂在评论 zone 之下（useCommentZones 先于 useDraftZones 挂载 → 草稿 ordinal 更大、渲染在已有评论下方），按时间序展示在已有评论下方。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- 收起态头部归零 margin-bottom：正文隐藏后下方多出的间距导致上下不对称，归零后内边距对称。 - 可折叠卡（分析过程 / 已拒绝 / 被复评关闭的代码反馈）整行标题区即展开/收起热区，扩大可点 面积（不止右侧 chevron）；忽略来自内部按钮（编辑/拒绝/引用/chevron）的点击避免误触。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
正文 + 代码对比包进 grid 容器始终挂载，按 .chat-finding-collapsed 用 grid-template-rows 0fr↔1fr 做高度过渡（inner overflow:hidden 裁切），收/展平滑而非瞬时跳变；chevron 旋转已有 过渡。尊重系统「减少动效」关掉过渡。仅可折叠卡触发，其余卡恒展开无动画。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
去掉评审总结卡横向额外内缩（margin $space-6→0），与 finding 卡同宽（同处 chat-pane-body 的 $space-6 内边距内）；加 3px 左侧色条，统一用蓝色，与卡片蓝色淡底成一套（不随判定变色）。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
字符数对用户无实际价值，标题简化为「原始输出」。各 locale 去掉 {{n}} 占位，调用处不再传 n。
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
raw HTML 的 <details><summary> 内文本不会被 markdown 二次解析（反引号原样漏出）。新增 withInlineSummary 给 <summary> 套 MdInline，让折叠标题里的 `代码` / **强调** 等内联格式生效； finding 卡 markdown（含 walkthrough）统一叠加。同步放开 shim 的 describe assessment 提示词 ——原强制标题纯文本「禁用反引号」，改为允许标题内用 `inline code` 标识符。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
评审总结正文此前 $fs-md + 默认行高，比 finding 卡（$fs-lg / $lh-normal）显得更窄。改为 同字号同行高；标题（h1-4）同步由 $fs-md 提到 $fs-lg（不小于正文、靠加粗区分层级）。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.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
1 participant
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.
概述
一批 PR / 评审 / 复评（/ask）界面的交互改良与若干修复，按任务拆为独立 commit（每个均过 lint/typecheck/test/build）。同步补齐 CHANGELOG。
变更
复评（/ask）交互
ChatPane / Diff 评审界面
验证
每个 commit 均本地跑通
lint→typecheck→test→build。🤖 Generated with Claude Code