该技能用于评审 mcp-windows 的 Windows MCP 变更,目标是发现可观察的正确性与用户影响问题,而非风格偏好。评审期间不编辑产品代码、不合并 PR、不更改仓库设置。
用途与流程:先阅读当前 PR head、diff、关联 issue 与验收标准,区分有意的破坏性变更与意外回归,不要求变更已明确排除的兼容性;遵循仓库说明与测试指南,用现有实现和测试确认公共接口的实际承诺,并追踪受影响的调用方与结果消费方,包括工具注册与 schema、原始请求校验、CLI 解析与分发、共享自动化服务、批次、宏保存/回放、结果模型、恢复提示、帮助与示例。复审时检查已有讨论串和完整评审摘要(含被抑制的发现),逐条对照最新代码,不重复已修复项,也不假定未解决线程仍是缺陷。
重点关注:输入契约(省略、null、空、空白、未知及模式不适用参数在契约要求下仍可区分,schema 本身不能证明运行时拒绝);MCP/CLI 行为一致(别名、引号、子参数、相对路径、工作目录、输出与退出码保留意图,未查解析器不推断别名无效);目标身份(ID 必须解析到原观察控件或失败,替换、句柄/进程复用、提供方失败、属主重启、驱逐、自动化线程释放不得改走名称或位置搜索);读写与动作范围(显式目标失败不得扩大为整窗口文本、OCR、其他表格或无关对话框,父级/邻近引用与坐标回退须属于预期窗口与控件);批次与宏(副作用前校验整个请求,保存与回放校验一致,处理 null 步骤与非法属性存在性,只有无歧义发现结果才能作为后续动作引用);有界工作(限制在批量获取或分配结果前执行,不完整搜索不能证明不存在或唯一,检查截止时间、最终探测、取消与清理);服务生命周期(调用方隔离、管道权限、启动协调、构建兼容、请求/输出上限、资源归属,批次不得交错,只读等待不得阻塞触发动作,响应不确定时不得重放);Windows 输入与结果(动作前验证前台归属、焦点、当前坐标/DPI 与输入返回值,分发、对话框消失与实际应用结果是不同证据,清理不得关闭用户应用或释放其他操作资源);快照(差异须基于调用方已知基线并重建当前树,含可操作 ID,缺失或过期基线需完整响应,不得将旧 ID 赋给替换控件);度量与指引(用等价捕获与元数据比较实际序列化输出,不重贴历史基准,按当前参数名、校验与 MCP/CLI 归属核对示例与恢复提示)。MCP 实例与 CLI 守护进程有意保持独立运行时状态,共享实现不意味着共享会话,提出新服务或会话抽象前须查阅仓库架构说明。
证据与安全:可用只读 GitHub 上下文查看相关代码、issue 与 CI 结果,确认结果覆盖当前提交,旧的成功运行不算新变更证据;上下文或工具不可用时须说明。变更涉及 Windows 自动化、模态对话框与等待行为时参考 FlaUI 与 pywinauto;跨切面服务模式使用仓库说明中的姊妹项目参考,不假设他项目注释证明其实现,也不假设 Office 特定行为适用。有执行条件时从最小相关现有测试开始,边界缺陷优先真实 SDK 调用与独立 CLI 进程回归,纯单元测试适合解析、预算、比较与受控时钟。禁止把评审工具连接到活动用户桌面或在共享机器上启用桌面输入测试,鼠标键盘行为须使用专用 Windows CI 证据;未经明确许可不运行付费 LLM 测试或 Office/浏览器基准。不得弱化断言、向任务型 LLM 提示注入工具提示、盲目重跑有副作用操作,或因重试通过而称失败无害。
报告:每个发现须给出触发输入或状态、实际与要求行为、用户影响及相关代码路径或回归证据,挂到合适的变更行并描述聚焦修正;只报告由变更引入或实质恶化的问题,先阅读足够上下文排除已有防护,避免推测性重构、纯风格问题、同一根因的重复症状及无关既有问题,不为凑数编造发现。使用简明英文,区分已确认缺陷、未验证风险与无法验证项;未发现可操作问题时如实说明,且不得暗示未运行的测试已通过。
分类与限制:该技能属于代码评审工作流,分类为编程与开发。其具体安装或命令行调用方式、支持的平台与版本、执行环境要求等,资料未提供。