feat(project): 支持为现有角色增加动作 - #375
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Review conclusion
本次改动在三条关键链路上保持一致:角色详情仅在当前 Outfit 有母版时开放入口;Quick Start 与 Workflow Editor 都会创建独立 WorkflowRun,并预置已完成的角色设定/母版节点;编辑器再从节点中的 characterId 恢复原角色,同时校验项目归属。保存动作仍沿用现有发布与冲突回滚流程,未发现会导致错误角色绑定、复用原 Run 或破坏现有动作资产的可报告问题。
验证方面,git diff --check 通过。尝试运行四个定向测试和 typecheck,但当前工作区未安装可用的前端依赖(vitest 启动器不可执行且模块不存在),因此无法在本地复跑;GitHub 上 Frontend checks 与 lint-and-test 在审查时仍在运行。
5497d43 to
ec0cd2f
Compare
|
@xyh202131 projects 界面已更新,请适配最新版本 |
johnnyzhang-eng
left a comment
There was a problem hiding this comment.
抽出 createExistingCharacterActionRun 之后,原先那段拉 100 条 run 再按 characterId 猜的查法也一并去掉了,改成新建独立 Run,这比按列表猜稳。母版取 outfit.previewUrl 而不是角色级参考图,方向和 #253 要做的一致。CI 全绿。
Closes #374
改动
验证
npm run test -- --run --testTimeout=20000:621 passed,4 skippednpm run typechecknpm run lintnpm run buildoxfmt --checkgit diff --check