Repository navigation
fix(agent): 修复 Codex 会话启动与历史漏查，新增供应商筛选 - #1632
Merged
Merged
Conversation
ghfind-review
Bot
added
the
review: high
xiaoyumuxi
marked this pull request as ready for review
November 1, 2026 06:02
Lithe Review结论： 未发现需要阻塞合并的问题。发现两处供应商筛选下的归档折叠问题，均为非阻塞 P2。 Findings
Scope Check
Suggestions
Summary共享查询及元数据处理与既有架构一致，新增测试覆盖了主要筛选交集和传播路径，但遗漏了自动展开后的手动折叠行为。 Verification
|
Collaborator
Author
|
已在 350c2f5 修复本次 review 的两项 P2：
仅运行受影响检查：前端历史组件 4 项通过（含两个新增收起/展开场景），macOS 定向回归的 2 个参数场景通过，TypeScript typecheck 及分层、Windows 边界、i18n、Agent Notes、平台矩阵、bundle 只读门禁通过。没有放宽断言，也没有重跑无关全量测试。 未用模拟数据截图冒充真实产品验收；此次没有补充 Windows 整机或真实用户历史的前后截图，这部分视觉验证限制仍保留。 |
1lck
added a commit
that referenced
this pull request
Nov 1, 2026
* feat(workspace): enforce directory indexing exclusions (#958) * fix(workspace): honor indexing ownership across Windows roots Fix attached-root LSP gates and reevaluate documents when workspace roots change. Carry request-local ownership through inventory, search, replacement and incremental index caches; preserve scoped search when unrelated roots are unavailable. Disable unsupported remote indexing settings, localize settings failures, normalize both Java exclusion test expectations, and register regressions in Windows CI. * fix(workspace): 修复索引租约生命周期与跨窗口同步 * docs(workspace): 记录复审修复与 macOS 原生验收证据 * fix(lsp): 保留 Windows 冷启动索引策略的原生路径身份 * fix(macos): 保持打开身份并修复索引回归验证 将会话打开代次与资源投影代次分开，保留立即 Run 的等待意图及同根重开隔离。 避免无待确认 Java 决策时重复发布空值，并使设置与混合规则夹具等待真实生命周期边界。 为 Git 观察 CI 补齐逐项报告，按原始输出活性检测 Swift 停顿，保留既有断言与预算。 记录通过的 126 条受影响 Swift 及 116 条身份消费者验证。 * feat(plugins): 独立编辑器高亮与动作 API，复用普通函数调用 (#1617) * feat(plugins): 声明式语法高亮与运行标记（API-25/27/29） - languages 2：插件语言可带 Monarch 语法和编辑器配置，Core 校验，纯语法插件免重启 - runMarkers 1：backend 2 插件函数回答可运行行，编辑器画运行按钮，点击经 commands/didRun 带 runMarker - 契约、fixture、Agent Note、计划文档与功能矩阵同步更新；Windows 暂不支持 Refs #570 * feat(plugins): 独立编辑器高亮与动作扩展 - languages 2 提供静态语法和独立编辑配置，插件正则在可终止 worker 中运行\n- editor/registerActions 运行期注册文本动作，点击复用普通函数调用，不扩张 Run/LSP/Build/Debug 领域契约\n- 补齐文档版本、动作租约、取消与清理、独立工具后端启用及安全模式边界\n- 更新 Skill、审查规则与决策笔记，总结 #1581/#1558 的架构边界\n- 同步契约、fixtures、中英词条、功能矩阵与资源复用说明\n\nRefs #570\nReworks #1617 * docs(plugins): 明确 API-25 JSON 贡献的支持范围与局限 区分包内静态 JSON 贡献与尚未开放的运行期 Provider 注册、替换和释放；说明独立语义 Provider、新引擎适配及平台、SDK 和验收限制。同步计划、架构笔记、正式契约与功能矩阵。 * fix(plugins): 修复编辑器扩展审查中的生命周期与高亮问题 动作刷新立即确认调度并隔离非致命失败，整批收集共享八秒期限并在撤销时清理计时任务。 Core 保留校验后的 grammar 原始文本，worker 按源顺序解析 cases 并显式默认 root；配置-only 别名借用既有 tokenizer，重映射 token 语言身份以保留独立注释配置。 模块状态观察弱引用 runtime，补齐有界、确定性的 Swift/Rust/WebKit 回归与测试文档模型清理，同步契约、Agent Note 和功能矩阵。 验证：57 项 Swift、1459 项 Rust、39 项编辑器和 119 项真实 WebKit 测试通过；静态边界、i18n、只读 bundle、资源复用与矩阵门禁通过。未执行 Windows 原生构建或打包安装验收。 Refs #1617, #570 * fix(indexing): preserve scan ownership and snapshot publication Retry current-workspace indexing cancellation without retiring another owner, publish only the latest complete inventory, and restore saved sessions once per workspace. Apply scoped search before caps and cursors, preserve dotted provider configuration siblings, and normalize Windows mark identity and localized read failures. * docs(indexing): record final macOS native acceptance evidence * fix(search): localize indexing failures and synchronize deferred Run tests * test(macos): synchronize every initial Run inventory release * fix: preserve excluded history and validate indexing marks * fix(macos): 修复分支列表悬停长分支名时触控板无法滚动 (#1621) (#1625) * fix(macos): 修复分支列表悬停长分支名时触控板无法滚动 (#1621) 触控板手势绑定在开始时的窗口上。悬停扩展面板收到首个滚动事件就关闭， 导致同一手势的后续事件全部丢失。手势期间改为隐藏面板并继续转发给列表， 手势结束后再关闭。鼠标滚轮行为不变。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(matrix): 记录分支列表触控板滚动的验证要求 (#1621) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(macos): 手指停顿时保留滚动手势，点击立即回收隐藏面板 (#1621) 手势进行中的回收超时从 1 秒放宽到 10 秒，避免停顿后继续滚动时丢失手势。 隐藏面板收到点击时立即回收，并在功能矩阵补充对应验收步骤。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(macos): remove push dialog file request ordering race --------- Co-authored-by: ZJUSCODE <ZengJun326@outlook.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: Yao Jingxi <23722032@bjtu.edu.cn> * fix(macos): forward hidden branch expansion clicks and clean up on close (#1629) * fix(workspace): preserve directory roles and indexing watcher ownership * test(windows): use canonical roots in indexing lifecycle fixtures * test(windows): identify the owned Run fixture descendant explicitly * feat(macos): 状态栏内存只显示图标，帧率去掉图标并收紧间距 (#1630) - 内存指示器只保留内存芯片图标，不再常驻显示“总计 / Lithe”数值； 悬停提示仍给出总计、Lithe、LSP、服务内存，点击打开原有的托管内存详情， VoiceOver 读出“托管内存”和当前总计。 - 帧率只显示“60 FPS”文字，去掉速度表图标。 - 新增 18pt 的状态栏紧凑图标按钮样式，只读锁和内存图标改用它， 不再沿用 28pt 工具窗口按钮留出的大块空白。 - 删除不再使用的 "Total %@" 词条并重新生成本地化资源。 - 更新 memory-monitor 功能矩阵的 macOS 证据与验收说明。 * fix(macos): 虚拟化后台服务退出列表并补充滚动性能规范 (#1627) (#1633) * fix(macos): virtualize background service exit selection (#1627) * fix(macos): keep background service accessibility selection live * fix(agent): 修复 Codex 会话启动与历史漏查，新增供应商筛选 (#1632) * fix(agent): restore Codex history and add provider filtering * fix(agent): honor manual archive collapse under provider filters * feat(macos): 彩色 LSP 状态菜单与普通日志标签页 (#1631) * feat(macos): 增加彩色 LSP 状态菜单与普通日志标签页 * fix(macos): LSP 入口常驻并仅由启用插件注入服务器 * fix(lsp): 等待重启关闭完成并保留局部索引重置状态 * fix(markdown): preserve inline HTML image layout and alignment (#1623) (#1635) * feat(markdown): render PlantUML diagrams offline on macOS (#1637) * feat(markdown): add offline chemistry formulas and subscripts on macOS (#1636) * feat(macos): 终端标签任意拖拽分屏并保留 PTY (#1638) * feat(macos): 终端标签任意拖拽分屏并保留 PTY - 终端标签改用 AppKit 拖拽会话与私有 pasteboard，避免 Monaco 接收文本拖放 - 工具窗终端可拖入编辑器边缘或空编辑器，编辑器终端可拖回工具窗 - 抽出 EditorGroupLayout.splitting 复用分屏几何，补充模型、拖拽与 PTY 测试 Refs #1622 * fix(macos): preserve terminal drop order and deduplicate split previews - Limit the native tool-window drop overlay to the canvas so tab-strip before/after insertion remains available for terminals dragged from editors. - Publish editor-group preview targets only when they change, including repeated clearing and drag lifecycle cleanup. - Add focused native overlay geometry and preview publication regressions. - Update the docking Agent Note and terminal capability matrix evidence and verification steps without claiming unperformed runtime verification. Validation: six focused Swift tests passed on Swift 6.3.3 with macOS 26.5 SDK; runtime bundle, architecture, i18n, Agent Notes and matrix static gates passed. The exact PR-based push tree also passed the static gates. Refs #1638 --------- Co-authored-by: Yao Jingxi <23722032@bjtu.edu.cn> * fix: align project JDK, isolate application sessions, and improve search (#1642) [skip ci] Use the selected project JDK in JDT configuration on both platforms. Give Windows applications independent cancellable sessions across Run, IDE API and Debug, with isolated failure output. Avoid blocking cold file/text search on content index warmup and improve macOS search sizing and result layout. Include shared fixtures, regression coverage and architecture notes. * feat(plugins): 插件注入的工具链框架与 macOS 只读环境设置页 (#1581) * feat(plugins): 插件注入的工具链框架与 macOS 只读环境设置页 (#1581) 按 #1581 评审后的改写计划重做：宿主不认识任何工具，工具定义、版本管理器 （提供方）全部由插件声明；Core 只做校验和四层解析，不解析 mise 等管理器 的私有格式。 - 契约：新增 toolDefinitions 1、toolchainProviders 1，executableResolution 增加 v2（只用于工具定义候选）；同步 schema、README、toolchains.md、 fixture 和 rust-core-api.md；proposals/toolchains/ 放完整方案草案 - Rust Core：新领域模块 rust/lithe-core/src/toolchain/（定义注册表、 提供方答复校验、目录固定、固定/项目文件/全局/自动四层解析、版本匹配、 扫描计划）和六个 toolchains.* 命令，全部只处理宿主快照，不启动进程、 不扫描、不写文件 - macOS：设置 › 构建新增「运行环境」「环境来源」两页，内容来自已启用 插件的声明和磁盘上的真实文件；未实现的添加、切换、保存、查询入口不显示 - 登录 Shell：新增只读 inspect，来源页显示读取状态、变量数、耗时和 PATH 各目录来源；重新读取不改变本进程 PATH - Debug 预览入口 --toolchain-settings-demo / --toolchain-sources-demo - Agent Note、设置层级 Note、功能矩阵和中英词条同步 Refs #1514 * feat(plugins): 工具链安装同时支持声明式扫描与提供方查询 (#1581) 工具的已安装版本有两条途径，可以只用其一或同时使用： - 声明式扫描：工具定义的 discovery/homeLayouts/version 由宿主展开，不运行 插件代码；可用的根和通配写法固定，新的安装形态交给提供方回答。 - 提供方查询：toolchainProviders 通过 toolchains/query 返回安装和要求。 - watchFiles 改为可选：不声明的提供方只报告安装，答复里带版本要求会被拒绝； 定义工具的插件也可以作为自己工具的提供方。 - 自动层把两条途径合成一个候选集合：同一规范化目录只算一次，来源标注优先 提供方、其次 host:system，只用于显示；不同目录同版本仍报冲突。 - discoveryPlan 对每个工具同时列出 scans 和 providers。 - macOS 环境来源页对只报告安装的提供方显示「只报告已安装的版本」。 同时修复评审的两个 P2： - 符号链接安装在解码时已规范化，解析时却按原始路径查证据而误判 unsafePath； 现在文件证据同时证明其规范化目标，证据互相矛盾时拒绝快照。 - 环境变量候选声明的 path 布局被忽略、总取第一个布局；现在按声明的布局选择。 * feat(plugins): macOS 设置页接入声明式扫描与版本探测 (#1581) - Core 新增 toolchains.candidates：列出一个工具的登录 Shell home 变量、扫描到的安装、跳过原因和自动层的选择；和 resolve 共用事实校验，宿主分两轮调用完成版本探测，工作区内的目录在探测前拒绝 - toolchains.discoveryPlan 增加 pathPrepend 模板和各平台适用的 home 变量 - 新增 MacToolchainDiscovery：按插件声明展开扫描，只探测 Core 返回的可执行文件；扫描根、条目、文件事实、安装数、探测数、并发和总时长都有上限，超出时给出诊断 - 运行环境页：全局卡片显示 home 变量和探测到的版本，PATH 命令标为版本未验证；自动卡片列出真实安装、选中项、同版本冲突和跳过数，路径显示扫描到的条目名 - 加载器只把登记在官方目录、并经内置或插件市场安装的插件标为官方来源 - 预览改用完整的样例清单，扫描本机真实安装 - 更新契约 toolchains.md、rust-core-api.md、Agent Note、功能矩阵和中英词条 * test(macos): 设置页声明顺序与构建分组保持一致 (#1581) 运行环境、环境来源两页插在项目页和运行配置之间，使 SettingsCategory 声明顺序（搜索结果顺序）与侧栏构建分组顺序不一致， settingsCategoryPlacesProjectAndRunBetweenKeymapAndTerminal 在 CI 失败。 把两页的 case 移到项目页之前，测试改为断言 Keymap 与 Terminal 之间 恰好是构建分组，且分组以项目页、运行配置结尾。 * feat(plugins): macOS 设置页接入工具链提供方查询 (#1581) - Core: lsp.toolchainRequest 与 litheToolchains 协商，原样回传回复交 toolchains.decodeReply - Core: candidates 接受 providerInstallations 并保留提供方归属 - 修复别名链接 home 只有链接路径事实时被判 unsafePath（机器人 P2） - macOS: 启动或复用插件后端查询，全局+逐目录独立 operationID/代次，超时与取消 - macOS: watchFiles 与回复 watch 变化整份重查，短时间事件合并 - 设置页显示提供方要求、版本与查询结果；文案走 shared/i18n - 更新契约、功能矩阵与 Agent Note * feat(plugins): 运行环境页每行生效来源改由 Core toolchains.resolve 决定 (#1581) - 每个查询过的目录、每个工具调用一次 toolchains.resolve，摘要、徽标和 “第几层生效”都读它的结果，视图不再自己按固定 → 项目 → 全局 → 自动排序 - 解析与提供方查询共用同一份文件事实；同一轮查询的提供方共享代次， 每次查询仍有独立 operationID - 固定值交给 Core 校验：只有固定值的目录只解析它固定的工具（每次加载 最多 256 个），固定的 home 经 toolchains.candidates 检查后探测， 不会出现在自动检测列表；不可用时终止失败，不回落 - 未安装的项目要求记为 unmet 并退到下一层；Core 无法给出结果时给出诊断 - 已知限制写入提案 Note 并建 Issue：冲突选择与写入 #1639、 项目 wrapper #1640、PATH 命令 commandHomes #1641 - 新增文案走 shared/i18n 并重新生成 * fix(plugins): 加载途中 context 变化的回复不再显示项目要求 (#1581) - 同一目录一轮查询里，context 与本轮不一致的提供方回复不会进入 toolchains.resolve， 但它的项目要求仍被写进页面的 requirements，导致页面显示的要求和 Core 的生效结果来自 两个不同的 context；现在只记录进入本轮的回复的项目要求，全局要求和安装列表不变。 - 新增 aReplyUnderAChangedContextIsLeftOutOfItsDirectory 测试：两个提供方在加载途中拿到 不同代次的 context，断言只有进入本轮的提供方要求被显示、Core 用它解析；修复前该测试失败。 - 测试夹具 providerSource 支持声明多个提供方，原有用例行为不变。 - Agent Note 更正“每个路径只读一次”的说法：查询阶段和解析阶段分两次读取事实，单次解析 内部一致；并补充 context 变化时回复不进本轮的取舍。 * feat(plugins): 提供方超时或不可用交给 Core 记为 providerUnavailable，工具行区分未响应与不可用 (#1581) - Core 新增 toolchains.failureReply：查询超过 15 秒没有回复，或后端起不来、断开、没有协商这个提供方时，宿主用它生成标准错误回复（plugin.toolchain.timeout / plugin.toolchain.providerUnavailable），再交给 decodeReply，错误码映射只在 Core 维护一份 - toolchains.resolve 只为问过这个工具的查询记 providerUnavailable，并带上 cause 区分超时和不可用；修复一个提供方失败时把它没被问到的工具也记为缺失的问题 - macOS 接线层把 Core 的 requestTimeout 转成 MacToolchainDiscovery.QueryTimeout；取消不生成回复 - 这类回复照常进入目录轮次，下一层来源继续决定；提供方状态显示“15 秒内未响应”或后端原因 - 运行环境页工具行在结果旁写出“mise 未响应 / mise 不可用”，第 2 步在版本文件存在但提供方没回答时显示同样原因，不再显示“版本未查询”；固定值失效时不显示 - 新增中英词条三条；契约 toolchains.md、rust-core-api.md、Agent Note 与功能矩阵同步更新 - 测试：Rust 覆盖两种原因的回复往返、下一层决定、只记被问到的工具、非法输入拒绝；Swift 覆盖超时与后端停止两种提供方并存时的结果与状态 --------- Co-authored-by: Yao Jingxi <23722032@bjtu.edu.cn> Co-authored-by: FxRayHughes <25192142+FxRayHughes@users.noreply.github.com> --------- Co-authored-by: Xiaoyumuxi <23722032@bjtu.edu.cn> Co-authored-by: ZJUSCODE <152275506+ZJUSCODE@users.noreply.github.com> Co-authored-by: ZJUSCODE <ZengJun326@outlook.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: 十二翼堕落天使 <61737634+icefery@users.noreply.github.com> Co-authored-by: Lichenkang <2188718831@qq.com> Co-authored-by: 枫溪 <bushuaichen@yeah.net> Co-authored-by: FxRayHughes <25192142+FxRayHughes@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.
改动
修复 Codex 消息立即失败、旧会话被供应商过滤漏掉的问题，并增加用户主动选择的供应商历史筛选。
-c参数对齐显式认证路由，地址和凭据仍走标准输入；不修改用户配置、账号目录、已安装 CLI 或适配器，不绕过托管策略。modelProvider元数据，两端消费；缺失或空白来源保持未知，不从当前路由、模型或地址猜测，不扫描私有历史文件。本地验证
说明