[BUG] 模板动作没有生效 _template_

[cuiliang]

Quicker 版本号

2.0.7

问题描述

新面板窗口里正常
场景与动作窗口里创建动作不正常

[cuiliang]

Codex 分析：

我对比了 1.x 与当前 V2 的新建组合动作 / _template_ 模板处理流程，判断这里存在一个 V2 迁移后的行为遗漏，能够解释“模板没有生效时，新建动作连默认 text 变量也没有”的现象。

代码事实：

• 1.x 新建组合动作时先创建空动作，再按标题精确查找 _template_；如果找到模板，会复制模板动作的数据、关联、右键菜单数据和变量求值开关。
• 1.x 如果没有找到模板，或模板数据为空，动作设计器会按“空动作”处理，并自动补一个默认变量：text，说明为“默认的文本变量”，类型为文本。
• V2 新建组合动作时会先创建 ActionItem2，并直接放入一个空的 XAction payload；之后再尝试查找 _template_ 并复制模板 payload。
• V2 动作设计器判断“是否为空动作”时，会把这个空 payload 序列化成非空 JSON，因此不会进入 1.x 那个“补默认 text 变量”的兜底分支。
• V2 的模板查找还增加了 1 秒等待限制；如果模板查询超时，会静默按“未找到模板”继续创建空 payload。
• V2 从动作索引中枚举全部动作查 _template_，包括不在页面上的动作；如果存在多个同名 _template_，命中顺序也可能和 1.x 的页面遍历顺序不同。

影响范围：

• 如果 _template_ 正常命中，用户可能感觉不到问题。
• 如果 _template_ 未命中、查询超时、命中了错误的同名模板，V2 会创建一个没有模板内容、也没有默认 text 变量的空组合动作。
• 这会让 2.x 相比 1.x 少了一个用户习惯上的兜底行为。

可选方案：

• 方案 A：在 V2 新建组合动作时，如果没有成功应用有效 _template_，显式创建带默认 text 变量的 XAction payload，恢复 1.x 行为。
• 方案 B：让 V2 动作设计器识别“空 XAction payload”并按空动作处理，继续走默认 text 变量兜底。
• 方案 C：同时改进模板查找：超时要可诊断，不要静默失败；多个 _template_ 时应有更稳定的选择规则。

推荐方案：

• 优先采用方案 A，并同步处理模板查询超时的可诊断性。这样改动点更靠近新建动作入口，也更符合“没有模板时仍给用户一个默认可用组合动作”的 1.x 语义。
• 多模板命中规则可以作为后续增强，避免这次修复范围扩大。

需要确认：

• 是否确认 V2 在 _template_ 不可用时也应继续保留 1.x 的默认 text 变量兜底。
• 因为修复会改变新建动作保存出来的 payload 内容，属于同步数据内容层面的行为变化，按当前安全流程不建议自动实现和提交，等待人工确认后再处理。

本次仅记录分析，未修改代码，未添加“已修复待验证”标签。

关联：Refs #117

[cuiliang]

Codex 补充分析：

根据进一步验证，需要修正前一条分析的范围：不是所有 V2 新建组合动作入口都表现一致。

新增代码事实：

• 新面板窗口的空位菜单对“组合动作”有专门分支：当创建项的 OperationType 是 XAction 时，会直接走新面板的组合动作创建路径，因此可以保留 _template_ 模板逻辑。
• 场景与动作管理窗口的动作页空位菜单走旧的动作按钮菜单构造逻辑；该路径在创建菜单项时使用通用 Operation 创建器生成默认 Operation。
• 通用 Operation 创建器对 XAction 的默认行为是创建一个空的 XAction payload，不会查找 _template_，也不会补 1.x 的默认 text 变量。
• 因此用户观察到的现象“新面板窗口创建正常，场景和动作管理窗口创建不正常”与当前代码路径相符。

修正后的根因判断：

• 根因更准确地说，是“部分旧/通用创建入口没有复用新建组合动作的专用创建逻辑”，而不是 _template_ 全局查找逻辑一定失败。
• 新面板入口已经做了 XAction 特判；场景与动作管理窗口等复用通用 Operation 创建器的入口缺少同等特判。

推荐修复方向：

• 不建议在通用 Operation 创建器里直接套 _template_，因为它也服务触发器、轮盘、Operation 参数编辑等场景，可能扩大行为影响。
• 更合适的是在“创建独立组合动作”的入口统一调用现有的新建组合动作专用逻辑；至少应让场景与动作管理窗口的空位菜单在选择“组合动作”时走与新面板相同的创建路径。
• 对于通用 Operation 编辑器是否也应补默认 text 变量，需要单独判断，因为它不一定代表“创建独立动作本体”。

本次仍只是分析补充，未修改代码，未添加“已修复待验证”标签。

关联：Refs #117