Skip to content

feat(quick-start): add guided creation and animation review - #95

Open
xyh202131 wants to merge 11 commits into
1024XEngineer:mainfrom
xyh202131:feat/quick-start-animation-review
Open

feat(quick-start): add guided creation and animation review#95
xyh202131 wants to merge 11 commits into
1024XEngineer:mainfrom
xyh202131:feat/quick-start-animation-review

Conversation

@xyh202131

Copy link
Copy Markdown

功能项

完成 Quick Start 一句话创作流程,并让用户在正式保存动作前播放和逐帧检查完整动画。

Quick Start 动作审核页面

用户流程

  1. 输入一句角色描述并创建项目。
  2. 生成 4 张角色候选,选择 1 张保存为正式角色。
  3. 在同一页面创建独立动作任务,生成 4 张动作首帧候选。
  4. 选择首帧后生成完整动画。
  5. 按动作 FPS 播放动画,也可暂停并逐帧检查。
  6. 确认后写入 Character,并使用稳定 ID 打开 Playtest。

主要改动

  • Quick Start 页面与页面用例层
  • 完整动画播放、暂停和逐帧检查
  • 刷新后通过 Controller 恢复同一审核结果
  • 模块 README、实际界面截图和页面/服务测试
  • 未装配生产依赖时明确禁用,不回退到 Mock

携带依赖

本 PR 以 main 为目标。由于 main 尚未包含 Quick Start 所需接口,当前同时携带:

  • WorkflowRun 核心、Store 与 Service
  • WorkflowController 页面协调层
  • 动作审核结果 ActionReviewResult

WorkflowRun 的独立审核入口为 #86。前置功能进入 main 后,本 PR 的重复差异会随基线更新而缩小;本 PR 的功能项仍只有 Quick Start。

边界

  • 仅包含前端 WorkflowRun、WorkflowController 和 Quick Start
  • 不包含后端、数据库、日志、依赖目录或构建产物
  • 不包含 Playtest 页面、历史记录、资产库和导出模块
  • 不执行合并

验证

  • Quick Start 定向测试:8 项通过
  • 当前分支全量测试:39 项通过
  • TypeScript:通过
  • Lint:通过
  • 改动文件格式检查:通过
  • 生产构建:通过
  • 1440×1000 桌面端与 390×844 移动端实际截图检查:通过

@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
windup Ready Ready Preview Aug 4, 2026 8:23am

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found one issue in this review.

Verification note: local npm test -- ... and npm run typecheck could not run in this workspace because the invoked vitest and tsc commands failed with Permission denied.

Comment thread frontend/src/entities/workflow-run/service/workflow-run-service.ts Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant