Conversation
1lck
left a comment
There was a problem hiding this comment.
Lithe Review|代码初审
结论:未发现需要阻塞合并的问题。紧凑布局的方向合理,但合入前需要协调与 #1008 的权限名称策略。
范围:preview afb7a76 → bf56797;5 个文件,实际逻辑改动集中在 AgentSessionSelectors.swift。
Scope Check
这次缩小模型/权限菜单的行高和留白,并让 mode、model 名称保留上游文字,避免中文界面只把 Plan 翻成“计划”。已追踪配置解析、Composer、Selectors 和选项回调;没有改变权限 ID、选项顺序或 ACP 确认行为,展示层边界合理。
Suggestions
- 请与 #1008 协调同一处 choiceTitle 的策略:#1008 希望权限名称中文化,本 PR 希望 mode 保留原文,两者目标相反。#1008 另有 thought_level 保持英文的规则,当前 PR 不包含。建议将紧凑布局与名称策略分开处理,明确最终选择后同步 Note 和矩阵,避免解决冲突时覆盖另一边意图。这是跨 PR 的需求取舍,不作为本 PR 的 P1 缺陷。
- 建议补中文环境下 Plan 菜单/底栏一致、双行说明和长列表滚动的回归覆盖,尤其是整合 #1008 后。现有展示测试未专门覆盖这些边界。
Verification
macOS、Agent Notes、平台矩阵、macOS Plugin、Database 检查通过。Windows 前端通过,但 shared Rust 的首轮测试计时及清理后重试的整套预算耗尽,host 阶段跳过,整体 Windows CI 未通过。失败位置在共享 Rust 测试,并非本次 Swift 代码的编译错误;目前不能据此确定为偶发失败。
作者提供了相关回归和中文 UI 验收记录;本次仅静态审查及读取已有 CI,未重新运行测试或启动应用。
失败任务:https://github.com/1lck/Lithe-IDEA/actions/runs/36882847664/job/110439181492
|
已按 这条 review 调整并更新两份 PR:#1008 为
建议先审查并合并 #1008,再同步 #1023 的基线并重新生成矩阵。两份 PR 继续等待审查,本次没有合并。全仓服务边界检查中 preview 已有的 |
1lck
left a comment
There was a problem hiding this comment.
Lithe Review|拆分后复核
结论:本次限定复查未发现需要阻塞合并的问题。此前指出的源码策略覆盖、回归覆盖缺口及旧 HEAD 的 CI 失败,已在当前版本得到处理。
范围:#1008 f722599、#1023 af2ca0e,相对共同基线 afb7a76。
Scope Check
- #1023 已移除名称策略,由 #1008 统一负责名称、图标和说明。对固定版本的 AgentSessionSelectors.swift 做只读三方文本核对后,两边源码改动可同时保留,不会再覆盖 thought_level 规则;这不是一次完整组合构建验证。
- 已补中文/英文名称回归,以及双行说明、长列表滚动和末项原始 ID 的实际渲染回归。权限面板按内容高度调整并保留上限,测量仅更新局部状态,避免只按最小行高计算而截断双行说明。
Verification
当前两份 PR 各六项 CI 工作流均成功。新增三项名称测试、两项布局测试在 CI 日志中均有通过记录,Windows Core 和后续 host 阶段也已完成。
名称测试:https://github.com/1lck/Lithe-IDEA/actions/runs/36944552895/job/110643582842
布局测试:https://github.com/1lck/Lithe-IDEA/actions/runs/36945556778/job/110646823241
当前方案是保留上游选项名称、说明按界面语言显示。技术上拆分已落实;最终文案策略及合入顺序仍待确认,完整 Agent 运行时验收继续保留 pending。后续整合时请基于最终提交重新生成矩阵并复核检查。
本次仅静态源码、测试代码及既有 CI 复查,没有启动应用或重新执行测试;作者的本地组合回归和构建结果未由本轮重跑。
模型及权限菜单的行距过大;压缩权限行后,如果说明占两行,按最小行高计算的面板又会截掉短列表底部。本 PR 收紧菜单间距,并让权限面板容纳实际文字高度。
改动内容
与 #1008 的分工
按 本 PR 的 review 拆开呈现策略和布局。本 PR 已移除权限名称规则及对应文档改动;PR #1008 统一
model/mode/thought_level的上游名称、权限图标和本地化说明。最终中文界面采用英文选项名称、中文说明及通用标题;同一组名称和菜单/底栏保持一致,
Plan不会单独变成“计划”。建议先审查并合并 #1008,再同步本 PR 的基线并重新生成矩阵。本地已试合并:源码、Note 和 JSON 自动合并,只有生成的 Markdown/CSV 需要重新生成。两份 PR 均等待审查。验证
choice-29。测试窗口和进程已退出,没有启动额外 Lithe 应用。.artifacts/test-stability/pr1023-review-macos.html、pr1023-review-macos.junit.xml;组合报告为pr1008-1023-combined-macos.html/.junit.xml。AppModel+RunConfiguration.swift616 行超过 600 行处失败;本 PR 未修改该文件。Windows Agent UI 尚未实现,矩阵保留 pending,没有进行 Windows 实机 UI 验收。./.agents/skills/write-stable-tests/scripts/test-stability-macos.sh \ --report .artifacts/test-stability/pr1023-review-macos.json -- \ --filter 'AgentConversation|AgentSessionSelector|AgentHistory|AgentFileReference|AgentProviderConfiguration|AgentBrandIconResource|AgentManagement' ./scripts/build-macos.sh --configuration debug --triple arm64-apple-macosx ./scripts/verify-platform-feature-matrix-change.sh afb7a76649e4707db2455ff62b0f9b71c8247aa4 HEAD ./scripts/verify-platform-feature-matrix.sh ./scripts/verify-agent-notes.sh ./scripts/verify-runtime-bundle-immutability.sh ./scripts/verify-windows-boundaries.shWindows CI review 的处理
旧提交的 失败运行 首次在子模块发布检查处超时,清理缓存后的重试又耗尽共享套件预算,后续 host 测试跳过。不能把重试预算耗尽算成另一个用例自身失败,也不能仅凭一次失败认定为 flaky。
Rust Core 与 preview 的树哈希相同,Windows 工作流和计时工具也未变。对照 #1007 的成功运行,同一测试以 8555ms 通过;此次更新后的 #1008 Windows Rust CI 也完整通过,原失败测试为 7512ms,共享 Core 记录 779 项,Windows host 后续阶段全部运行完成。
本 PR 新提交
af2ca0ec的 Windows CI 已通过,共享 Rust Core 与后续 Windows host 阶段均完成。部分 macOS 检查仍在运行。没有通过加大超时、删除测试或修改无关 CI 代码规避失败。