bug:画布节点中，调用生成角色动作节点，出现非完成任务不应该携带result

[xiaocheny214]

问题

画布节点中，调用生成角色动作节点，出现非完成任务不应该携带result

验收

• 调用生成角色动作节点，不在出现此弹窗。

[huyanxius]

@xiaocheny214 我查了一下，怎么感觉问题在后端？

[huyanxius]

1. executor.py 捕获 PromptRejected 后先调用 update_result()，写入拒绝信息。
2. task_repo.py 的 update_result() 会把任务标记为 completed。
3. 随后执行器又将任务改成 failed，但没有清空刚写入的 result。
4. SSE 最终发出 status=failed + result≠null，撞上 api.ts 的合同校验，于是弹出“非完成任务不应携带 result”。
   根因在后端 PromptRejected 异常处理

[huyanxius]

用户点击“生成完整动画”后，后端创建任务并开始执行；执行前会检查动作描述，如果描述不符合生成规则，任务本应直接变成“失败”，且 result 为空。但现在后端先写入了拒绝原因作为 result，随后又把任务标成 failed，于是前端收到一份“任务失败但又携带生成结果”的矛盾数据，触发校验并弹出“非完成任务不应携带 result”

[huyanxius]

所以感觉应该是，后端失败流程写错了任务状态与结果

[huyanxius]

建议提示词被拒绝时不要调用会把任务标成 completed 的 update_result()，直接将任务更新为 failed，并保证 result 为空

[huyanxius]

@fennoai 请定位一下这个问题是否是我说的这个情况，以及我的这个解决方法是否可靠？

[fennoai]

定位结论

已确认你判断的情况成立，截图中的弹窗就是前端收到“status=failed 但 result 非空”后触发的合同校验：

• backend/packages/app/src/windup_app/server/orchestrator/executor.py:272 的 PromptRejected 分支先调用 task_repo.update_result()。
• backend/packages/app/src/windup_app/server/orchestrator/task_repo.py:172 的 update_result() 会同时写入 result、设置 result_type，并强制把状态改为 completed。
• 随后 executor.py:281 再调用 update_status(..., FAILED)；update_status() 只改状态和错误信息，不会清空刚写入的 result。
• SSE 序列化会把任务中的结果原样发出（task_repo.py:65-80），而前端 frontend/src/entities/generation/api.ts:314-324 明确拒绝非 completed 任务携带 result，所以会出现图中的弹窗。

对解决方案的判断

你的方案方向正确，而且能直接修复这条触发链：PromptRejected 时不要调用 update_result()，直接将任务置为 failed，并保证 result 为空。这样前端会收到失败状态和 error_message，不会再触发“非完成任务不应携带 result”。

建议再补一层后端不变量，避免未来其他失败路径复现：失败状态更新应同时清空 result 和 result_type（最好封装成统一的 fail_task()，而不是让每个调用方自行保证）。仅删除这一次 update_result() 对新建任务是可靠的，但不能从持久化层保证“所有 failed 任务都没有 result”。

建议验证

• 增加 PromptRejected 集成测试：断言最终 status=failed、result is None、result_type is None、error_message 非空。
• 同时断言失败 SSE payload 的 result 为 null，覆盖前端实际弹窗使用的链路。
• 验收时使用会触发提示词拒绝的生成角色动作请求，确认只显示失败提示，不出现该合同错误弹窗。

本次为定位与方案评估，未修改代码、未提交变更。