Repository navigation
docs: document saved Photo selections and optional approval - #2160
Conversation
VisActor 工作流结论：流程已完成。
请核对 BugServer 默认全选清单，改选后保存，再手动 Merge；Approve 可选，仅记录操作。 Photo 基准选择尚未保存；不影响三项检查建议或人工合入。 Photo 执行明细Photo 原始执行：completed；结果：passed；缺少基准 0；等待 0；运行中 0；不完整 0；创建未知 0。 Photo 差异解释（非阻断建议）差异共 0 项；已逐项分析 0 项；未分析 0 项。 图片差异解释（0 项）暂无已解释差异。 合入后 Photo 基准更新清单没有已授权的基准更新。 版本与合入后状态当前提交： 合入后状态：skipped。 CN 结果仅证明上述实际范围；完整报告和原始差异继续保留。 |
![visactor-review-bot[bot]](https://avatars.githubusercontent.com/u/126661221?s=60&v=4){kind=small}
| ### 批准、合入和异常恢复 | ||
|
|
||
| 工作流识别具有仓库 Maintain/Admin 权限的维护者对**当前提交**作出的有效 GitHub Approve，该批准同时表示已人工核查 Photo。Bot 不自动 Approve/Merge；GitHub 分支保护和其他检查仍适用，工作流不新增强制 Checks 或修改仓库规则。 | ||
| 工作流中的 GitHub Approve 仅作审计；维护者核对当前提交的 CR、单测、Playwright 和 Photo 后，可以直接手动 Merge。GitHub 分支保护或仓库评审要求仍然适用，Bot 不会修改这些规则，也不会代为 Approve 或 Merge。 |
There was a problem hiding this comment.
[P2] 同步更新仍将有效 Approve 写为必需的合入指引
当仓库规则本身不要求 Approve 且维护者已核查当前提交时，这里允许直接 Merge；但同文件第 108、113 行仍把有效批准写成进入“等待人工合入”及新提交的必要条件，英文版也保留相同说法。两种指引对同一场景相互冲突，维护者可能继续等待一个按新流程并非必要的批准。请将状态表和版本说明改为与可选 Approve 一致，并明确仓库规则要求批准时的例外。
已确认修复 (def407a): 原触发条件是仓库不强制 Approve 且维护者已核查当前提交。当前中文状态表已把等待合入定义为三项建议完成后按仓库规则手动 Merge，版本说明改为重新核对当前结果和保存修订；批准段明确 Approve 仅作审计且仓库规则仍适用。英文对应状态表与版本说明已同步，原先将有效 Approve 作为工作流必要条件的矛盾消失。
| Photo 自动差异评估及标准图自动更新暂停。Approve/Merge 不授权自动采用测试图；提前合入不会使未完成或失败的检查变为通过。新增/修改共享基准需管理员另行获得明确授权、人工核查并安排复验，原报告保留。 | ||
| 对已明确启用图文选择的 PR，BugServer 展示当前版本的图片和短说明，新候选默认全选。维护者应逐项核查；需要保留旧基准的项目取消选择，并在合入前保存成功。以服务端显示的保存修订为准，未保存草稿不生效；已保存的人工选择不会被默认全选覆盖。 | ||
|
|
||
| 合入后只更新该修订已选的基准，未选项不变。写入前仍核对冻结提交、三项检查、实际合入内容、保存时间及原基准；旧版本、合入后补选或冲突不能自动补认。请求超时或部分成功时联系维护者按原任务恢复，不重新派发或更换请求 ID。 |
There was a problem hiding this comment.
[P2] 区分启用图文选择后的基准更新与原有跳过状态
已启用选择且合入前成功保存选中项时，本段要求合入后更新这些基准；但同文件第 109 行的“已合入／核对基准阶段”仍说 report-only 下跳过自动基准更新是预期行为，英文版也一样。Photo 在新流程中仍为 report-only，因此维护者遇到选中项未更新时可能照状态表认定无需处理，错过本段要求的恢复。请把该行明确限定为未启用选择或清单不可用的 PR，并说明已保存选中项却未更新时需要核查或恢复。
已确认修复 (def407a): 原触发条件是启用选择、合入前已保存选中项，合入后更新却被跳过。当前中英文状态表均明确启用选择时须核对所选基准处理结果，跳过自动更新只适用于未启用选择；正文要求只更新已选项，无可用清单的人工处理边界也已明确，原来将遗漏误判为 report-only 正常跳过的矛盾消失。
同步 Photo 图文选择的操作指引：新候选默认全选、改选须保存、Approve 仅作审计，人工 Merge 后仅处理已保存的所选基准。旧人工基准流程明确限定为未启用或无可用清单的情况，避免重复写入；GitHub 分支保护、原报告和未知请求恢复要求保持。
本 PR 作为 VRender 四仓扩展的精确验收目标，已转 Ready，按现有单重型调度执行。文档合入不代表全量开放，实际能力以维护者明确配置为准；Bot 不代为 Approve 或 Merge。
验证：
VisActor 工作流（CN）：completed
当前提交：
def407a086077568184ad9e6263df73719a3bc1d三项检查建议与 BugServer 人工确认