Skip to content

docs(agents): 同步 Core、插件、UI 截图与 Windows 虚拟机规范 - #1941

Merged
1lck merged 1 commit into
previewfrom
codex/update-development-skills
Nov 14, 2026
Merged

1lck merged 1 commit into
previewfrom
codex/update-development-skills

Conversation

@1lck

@1lck 1lck commented Nov 14, 2026 •

Copy link
Copy Markdown
Owner

关联 Issue / Linked issue

PR 类型 / Type

  • feat:新功能或用户可见的改进
  • fix:缺陷修复
  • docs:文档、Agent Note、Skill 等规则文件
  • release:稳定版发布准备

改动说明 / What changes

背景 / Why

Viva 重构后,开发与审查规范需要明确通用宿主逻辑和工具专属知识的归属;UI 验证还缺少将真实无头截图附到 PR 的要求。Windows 本地验证改用本机虚拟机,原独立云服务器已退役,但旧 Skill 仍指向 Tauri、MSVC/ARM64 构建和旧日志路径。

改动 / Changes

  • develop-lithe:区分通用宿主规则、工具专属知识与平台 API;通用规则进入 Rust Core,新增工具集成优先插件化,禁止把工具专属逻辑换成 Rust 后塞进 Core,补齐 Go 平台层的下沉约束。
  • UI 规范:受影响场景必须执行 Viva 无头测试、生成并检查真实截图,附到 PR 或可访问的上传 artifact;说明场景、主题、语言、尺寸、命令与提交。视觉变化需前后对照,跳过的测试、本机路径或日志不算截图证据;目标平台原生验收继续保留。
  • debug-windows-on-parallels:重写为当前 Viva Go/cgo + Rust Core 工作流;使用本机 Parallels 与交互用户执行,识别 Git Bash、x64 MinGW/GNU 目标和 ARM 来宾中的 x64 仿真限制。更新桌面构建、计时、日志来源及进程/虚拟机清理要求;插件仍使用自己的工具链和架构。
  • write-stable-tests 及参考页:当前 Windows 桌面计时使用 DesktopGo;Core 测试在实际链接的外部 Core 仓库运行;旧 Swift/Tauri、旧仓内 Core scope 和历史云测记录不再作为当前执行指南。保留已有 GitHub Actions Windows CI。
  • create-lithe-plugin:插件 UI 改动引用统一无头截图规则;Windows 验收引用本机虚拟机的传输与清理规则,不套用宿主构建目标。
  • lithe-code-review:增加截图与 Windows 虚拟机证据核对;Core 逻辑归属改为所属 crate,而非全部挤入 C ABI crate;审查输出同步记录截图和仿真范围。
  • AGENTS.md 与 PR 模板:入口引用当前真源,去除 Windows/Tauri 运行时表述,增加 UI 证据检查项及 Windows 本地验收记录要求。

不在本 PR 范围内 / Out of scope

计时执行器仍暴露指向旧测试来源的参数,已建立 refactor Issue #1940;本 PR 更新调用规范,执行器的兼容策略和实现由该 Issue 跟进。

类型专属:docs

  • 只改 Skill、参考页、AGENTS 入口和 PR 模板,没有产品代码行为变化。
  • [N/A] 本次没有修改或迁移 Agent Note;维护的是 Skill 规范真源及其审查/提交入口。
  • 具体规则由 develop-lithe 与对应执行 Skill 维护,插件、测试和审查入口引用该规则。
  • 文中的路径、命令与本地 Markdown 链接已对照当前实现检查。
  • [N/A] 插件契约和宿主方法没有变化。

范围检查 / Scope checks

功能矩阵 / Feature matrix

N/A:仅规范文档,没有宿主功能或平台实现变更;矩阵校验通过,未提交生成视图,无需 matrix-exempt。

插件归属 / Plugin ownership

  • 已运行 develop-lithe 第三节的宿主零依赖自查,结果为空。
  • 未添加具体插件的测试、能力矩阵、翻译、资源路径或构建说明。
  • [N/A] 没有修改通用扩展点或新增测试,无需运行具体插件端到端验证。

UI 验证 / UI evidence

N/A:新增的是未来 UI 改动的截图要求,本 PR 没有修改产品 UI,无需生成界面截图。

统一日志系统 / Structured logging

N/A:未修改产品日志事件或错误处理;仅校正 Windows 日志来源的文档。

界面文案 / i18n

N/A:没有修改产品文案或翻译词条。

逻辑归属 / Logic placement

N/A:本次明确未来实现的所有权要求,没有修改业务逻辑、Core 命令或 fixture。

性能与内存 / Performance and memory

N/A:无运行时改动。规范保留本机虚拟机按初始状态清理和资源有界退出要求。

设置 / Settings

N/A:未修改产品设置。

插件契约 / Plugin contract

N/A:未修改插件协议、SDK、能力版本或宿主方法。

发行包与资源 / Bundle and resources

N/A:没有新增运行时资源或下载入口;发行目录只读静态检查通过。

无用代码与重构 / Dead code and refactoring

已为检查过程中发现的既有计时执行器旧 scope 建立 #1940,没有把执行器改动夹带进本 PR。

验证 / Verification

命令或操作 平台 结果
quick_validate.py 检查 develop-lithe、create-lithe-plugin、debug-windows-on-parallels、write-stable-tests、lithe-code-review macOS,uv 临时提供 PyYAML 5 个 Skill 均通过
检查修改文件中的本地 Markdown 链接 macOS 25 个链接均有效
git diff --check macOS 通过
scripts/platform-feature-matrix/verify-platform-feature-matrix.sh macOS 191 项能力校验通过
scripts/desktop/verify/verify-runtime-bundle-immutability.sh macOS 桌面 macOS/Windows 源码静态检查通过
develop-lithe 第三节宿主零依赖自查 macOS 无命中

未能执行的检查和原因: 纯文档改动,不运行无关的产品全量构建、UI 截图或 Windows 虚拟机工作流;没有把静态检查描述成原生运行验收。

提交前确认 / Final checks

  • 分支从最新 preview 创建,提交前已再次获取并确认目标分支未前进;目标为 preview。
  • 先创建 Draft PR,本地验证通过后再标记 ready。
  • 仅提交本次 10 个规范文件;保留用户已有的未跟踪目录,不提交产物、凭据或个人路径。
  • 本次没有启动 Lithe 应用或虚拟机,没有新增运行进程需要清理。

CI 基线失败核对

  • 本 PR 的 Desktop CI 在国际化检查中报告 desktop/internal/workbench/shell_view.go 两处违规。
  • PR 基线 9dbcb321ecbd5f1d1fd0a9ac32d20a7f8fedcdd8 的 preview CI 已报告同一文件、同样两处违规;本地运行相同检查也得到相同结果。
  • git diff origin/preview...HEAD -- desktop/internal/workbench/shell_view.go scripts/i18n/verify-app-language-localization.mjs shared/i18n 为空。未修改界面实现、翻译或检查器,也没有放宽门禁。
  • 上述 preview 基线的 Windows 检查另有 HTML 路径与进程内存测试失败;PR 检查已完成,同样三个 Windows 用例失败,见下方最终结果。基线的 Linux、macOS 和 pins 检查通过。

最终 CI 结果

本轮 Desktop CI 已结束:

  • Verify pins、Linux、macOS:通过。
  • Format and script tests:国际化检查的两处 shell_view.go 违规,与上面的 preview 基线一致。
  • Windows:结果 中 TestHTMLPreviewSharedFixture、TestProcessesListsThisProcessAndItsChild、TestMemoryDetailsShowTheSummaryOfTheWholeInstance 失败;它们在 原基线 Windows 运行 中已以同样的路径/内存断言失败。

本 PR 相对基线仅修改列出的 10 个规范文件,产品源码、测试实现、国际化检查器及翻译数据均未改变。没有重试失败断言、放宽检查、修改保护规则或声明全部 CI 通过。维护者已明确要求 squash 合入 preview;使用仓库已有维护者 PR 合并权限完成本次纯文档合并。

@ghfind-review ghfind-review Bot added the review: high ghfind author score; see https://ghfind.com label Nov 14, 2026
@1lck
1lck marked this pull request as ready for review November 14, 2026 17:45
@lithe-code-review

lithe-code-review Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

Lithe Review

结论: ❓ 信息不足
范围: preview ← codex/update-development-skills · head: f7505b7

静态审查未发现需要阻塞合并的问题。但已观察到的 Format and script tests 失败缺少日志,验证结果尚不能确认。

Scope Check

  • 关联需求与完成范围:10 个文件均为规范文档;Core、插件归属、截图证据和 Windows 虚拟机规则已同步。[Feature] 清理 Viva 重构后 Windows 计时执行器的失效 scope #1940 明确保留为独立跟进项。
  • 插件归属:按指定目录、文件类型和大小限制执行自查,无命中;未新增测试、插件矩阵、翻译或资源登记。
  • 架构归属对账:见下表;本次没有产品实现逻辑。
  • 未沉入清单:无。其余运行时、性能及契约检查不适用。

架构归属对账

逻辑块 归属 依据
无 不适用 仅修改 Skill、参考页、AGENTS 入口和 PR 模板,没有新增命令、状态规则或平台实现。

Checked and cleared

  • Windows 架构说明与 scripts/desktop/desktop-go.sh:57 一致:amd64 映射 GNU Core 目标,arm64 明确拒绝。
  • DesktopGo 参数与计时执行器一致;包装器从实际链接的 Core 检出导出契约及 fixture 路径。
  • 截图入口的 build tags、环境变量和跳过条件确实存在;新增规范明确要求实际生成截图,不能以跳过结果代替。
  • Windows 默认日志位置与 desktop/internal/logstore/dir_windows.go:19 一致;清理要求保留用户原有虚拟机和进程。

Observations

Summary

规范修改与所核对的构建、计时和日志实现一致,没有确认的本次引入问题。当前信息缺口是失败 CI 的原因及其最终结果。

Verification

  • 实际检查:固定 9dbcb32...f7505b7 的全部 10 个文件,以及直接引用的构建、计时、截图和日志入口;git diff --check 通过。
  • 作者提供结果:5 个 Skill 校验、25 个本地链接、191 项矩阵及发行目录只读检查通过;本次未复跑这些检查。
  • 已提供 CI:Verify pins 成功;Format and script tests 失败;其余包含进行中、跳过和取消状态。需要补充失败日志及最终结果,不能据此确认失败由本 PR 引入。
  • 本次未运行仓库代码、测试、应用或虚拟机,也未访问网络。新增虚拟机指令仅完成静态核对。
  • UI 截图不适用:本 PR 没有产品界面变化。

@lithe-code-review

Copy link
Copy Markdown

Lithe 运行验收

版本:f7505b727f804568a6a8f39d106e9b2591403d01;状态:blocked。
本报告独立于代码审查;只报告实际执行的场景,不代表所有问题均已修复。

执行未完成:Invalid scenario identity

本次未形成完整验收结果,详见执行原因。
详细步骤和原始证据保存在服务器受限目录(14 天、总计不超过 1 GiB);当前没有公开下载链接。

@1lck
1lck merged commit b60252f into preview Nov 14, 2026
5 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: high ghfind author score; see https://ghfind.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant