画布验收遗留问题与后续需求 #419

Merged

suzmii merged 16 commits from fix/agc-acceptance-followup into master 2026-09-21 02:02:20 +08:00

Conversation 0 Commits 16 Files Changed 48

+5095 -373

suzmii commented 2026-09-18 15:58:40 +08:00

Member

Copy Link

Copy Source

关联 #411，承接已合并 PR #412。分支 fix/agc-acceptance-followup（已合并 master eed72ba39 与本轮再次合并的 master；HEAD 5574f4112）。

本批交付

1. 派生/修改类任务进「生成任务」侧栏（AGC-039）：快速编辑、生成动画、视频 / 音效 / 背景音乐这类走资源编辑链路的任务与图片类生成任务同一条列表。数据源 = 原生 list_pending_local_project_resource_edits + 本会话本地提交记录，按 operationId 合并去重。未改原生账本 schema，未改请求载荷。

2. 卡片浮层只负责提交任务（AGC-040 + 口径澄清）：点提交当帧即关面板，进度与结果全在侧栏；受理之后的失败不重开面板（侧栏收口为失败 + 提示条，草稿回落「未完成编辑」）；派生请求身份按「入口 + 资源路径」记忆，重开再提交、提示词没变就沿用同一 operationId / 幂等键；同一张素材还有在途提交时，快速编辑与生成动画都会拦下（此前动画这条会卡在原生锁上或重复付费）。

3. 共享画布面板样式按美术画布照抄 + 守卫：补 .image-canvas-editor__generation-close 的第二条同名规则（浮层内绝对定位）、提示词聚焦 / aria-invalid 两档、composer 参数簇底色、@media (max-width: 760px) 的动画面板那一支；删除重复回显提示词的「动画描述」行。守卫 resourceCanvasCharacterAnimationPanelStyle.test.tsx 覆盖动画面板 / 快速编辑面板两份 DOM，逐条比对「网页端命中这份 DOM 的规则（选择器分支 + 声明）AGC 是否都有一条」，宿主差异逐条显式登记。

4. 生图浮层美化：字段行两列、比例与尺寸并排、提示词 3 行、润色 / 参考计数 / 两个动作收进同一行——面板高度回落到几何上界以内，滚动条物理上不再出现；相关卡片外形收敛到 .game-project-workbench 上的 --game-canvas-card-* 基类 token（取值照抄网页端画布面板），占位卡与生成浮层 / 弹窗共用一套圆角、描边、底色与投影。

5. 其它现场问题：生图弹窗点输入框就弹素材选择框（<label> 把点击转发给内部第一个可标注控件 = @ 按钮）已修并加复现用例。

验证口径（重要更正）

• 规范跑法是仓库根 npm test（CI 的 frontend-tests 也是这条）。此前描述里"appSurface 16 条既有失败"是从 apps/ai-game-creator-shell 目录跑造成的 cwd 假红：resolve(process.cwd(), 'apps/…') 路径翻倍 → ENOENT。
• 本批新增 apps/ai-game-creator-shell/tests/repoPath.ts（按 import.meta.url 反推仓库根），19 个用例文件改用它，另修掉 HOST_STYLE_SHEETS.filter(existsSync) 的同类假红。
• 合并 master 后从仓库根跑全量 npx vitest run：373 个测试文件全过，4512 通过 / 34 跳过 / 0 失败；tests/appSurface.test.ts 单独跑 555 项（17 跳过）0 失败。
• 其它门禁：npm --prefix apps/ai-game-creator-shell run typecheck（含 check-config）、npm run check:encoding、git diff --check、prettier、eslint 全过；关键判据（草稿让位、在途重复提交拦截、样式缺口守卫、label 转发点击）都做过变异验证或先红后绿。
• 范围核对：48 个改动文件全部落在 AGC 前端、3 个共享画布组件与 3 份文档；server-rs / spacetime / docs/openapi / src-tauri 零改动。

结论与未做

合并前复核结论：可合并（无影响合并的问题），PR No Conflicts。

未复验且不阻塞合并的两项：① 真实客户端观感（动画面板样式、生图浮层紧凑排布、卡片基类 token 都只有声明级 / 结构级守卫，真机需客户端验收）；② 远程 CI（runner / 镜像问题见 Issue #431，按用户要求本批不追，本地全量结果作为替代证据）。另有一项已保留行为：同一次修改成功之后再提交仍会再派生一次（按用户显式重复计费，未做同提示词去重）。

关联 #411，承接已合并 PR #412。分支 `fix/agc-acceptance-followup`（已合并 master `eed72ba39` 与本轮再次合并的 master；HEAD `5574f4112`）。 ## 本批交付 **1. 派生/修改类任务进「生成任务」侧栏**（AGC-039）：快速编辑、生成动画、视频 / 音效 / 背景音乐这类走**资源编辑链路**的任务与图片类生成任务同一条列表。数据源 = 原生 `list_pending_local_project_resource_edits` + 本会话本地提交记录，按 `operationId` 合并去重。未改原生账本 schema，未改请求载荷。 **2. 卡片浮层只负责提交任务**（AGC-040 + 口径澄清）：点提交当帧即关面板，进度与结果全在侧栏；受理之后的失败不重开面板（侧栏收口为失败 + 提示条，草稿回落「未完成编辑」）；派生请求身份按「入口 + 资源路径」记忆，重开再提交、提示词没变就沿用同一 `operationId` / 幂等键；同一张素材还有在途提交时，快速编辑与生成动画都会拦下（此前动画这条会卡在原生锁上或重复付费）。 **3. 共享画布面板样式按美术画布照抄 + 守卫**：补 `.image-canvas-editor__generation-close` 的第二条同名规则（浮层内绝对定位）、提示词聚焦 / `aria-invalid` 两档、composer 参数簇底色、`@media (max-width: 760px)` 的动画面板那一支；删除重复回显提示词的「动画描述」行。守卫 `resourceCanvasCharacterAnimationPanelStyle.test.tsx` 覆盖**动画面板 / 快速编辑面板**两份 DOM，逐条比对「网页端命中这份 DOM 的规则（选择器分支 + 声明）AGC 是否都有一条」，宿主差异逐条显式登记。 **4. 生图浮层美化**：字段行两列、比例与尺寸并排、提示词 3 行、润色 / 参考计数 / 两个动作收进同一行——面板高度回落到几何上界以内，**滚动条物理上不再出现**；相关卡片外形收敛到 `.game-project-workbench` 上的 `--game-canvas-card-*` **基类 token**（取值照抄网页端画布面板），占位卡与生成浮层 / 弹窗共用一套圆角、描边、底色与投影。 **5. 其它现场问题**：生图弹窗点输入框就弹素材选择框（`<label>` 把点击转发给内部第一个可标注控件 = `@` 按钮）已修并加复现用例。 ## 验证口径（重要更正） - **规范跑法是仓库根 `npm test`**（CI 的 frontend-tests 也是这条）。此前描述里"appSurface 16 条既有失败"是**从 `apps/ai-game-creator-shell` 目录跑**造成的 cwd 假红：`resolve(process.cwd(), 'apps/…')` 路径翻倍 → ENOENT。 - 本批新增 `apps/ai-game-creator-shell/tests/repoPath.ts`（按 `import.meta.url` 反推仓库根），19 个用例文件改用它，另修掉 `HOST_STYLE_SHEETS.filter(existsSync)` 的同类假红。 - **合并 master 后从仓库根跑全量 `npx vitest run`：373 个测试文件全过，4512 通过 / 34 跳过 / 0 失败**；`tests/appSurface.test.ts` 单独跑 555 项（17 跳过）0 失败。 - 其它门禁：`npm --prefix apps/ai-game-creator-shell run typecheck`（含 `check-config`）、`npm run check:encoding`、`git diff --check`、prettier、eslint 全过；关键判据（草稿让位、在途重复提交拦截、样式缺口守卫、label 转发点击）都做过变异验证或先红后绿。 - 范围核对：48 个改动文件全部落在 AGC 前端、3 个共享画布组件与 3 份文档；`server-rs` / `spacetime` / `docs/openapi` / `src-tauri` **零改动**。 ## 结论与未做 **合并前复核结论：可合并（无影响合并的问题）**，PR `No Conflicts`。 未复验且不阻塞合并的两项：① 真实客户端观感（动画面板样式、生图浮层紧凑排布、卡片基类 token 都只有声明级 / 结构级守卫，真机需客户端验收）；② 远程 CI（runner / 镜像问题见 Issue #431，按用户要求本批不追，本地全量结果作为替代证据）。另有一项已保留行为：同一次修改**成功之后**再提交仍会再派生一次（按用户显式重复计费，未做同提示词去重）。

suzmii added 1 commit 2026-09-18 15:58:40 +08:00

登记首批合并后的画布验收遗留问题 ... ca80cb28d2

从master已合并的PR412建立后续修复范围。
区分动画与编辑等缺陷、卡片与任务浮层需求及未验证风险，不宣称已修复。

suzmii added 1 commit 2026-09-20 17:44:18 +08:00

补齐画布验收第一批：润色回包提示、未提交草稿恢复与派生名预检 ... 1e17d2c852

AI 润色回包与原文相同时给出明确提示，不再落下点了等于没点的「恢复原文」假入口
润色「与原文相同」的判据改为规范化之后要写回宿主的文本，回包被长度上限截回原文同样算没变化
共享聊天输入区显示润色提示，并让提示随用户手改提示词失效
快速编辑未提交草稿改按资源路径归属，收起面板或被旁路重投影后都不再静默丢掉
恢复入口按路径与本会话草稿对号，并区分「本会话未提交」与原生账本条目
账本读取失败时入口仍报出本会话草稿条数，「继续编辑」重开面板回填提示词与 @ 引用
派生资源名预检接入快速编辑与角色动画两条提交路径，早于正规化写盘
名称预检自身抛错时回写面板失败态，不再留下「点了没反应」的提交按钮
浮层关闭判定改在捕获阶段冻结，避免浮层内按钮卸载自己时整块面板被自己人收掉
恢复入口动作按钮收进统一容器，补移动端纵向排布与禁用态样式
空草稿表改工厂函数，跨语言对表用例按模块路径解析 Rust 源码
补回归用例：旁路重投影后的草稿、原样回包提示、提交前名称拦截、草稿表读写与跨卡隔离
同步 decision-log 与画布验收待办的进展说明

suzmii added 1 commit 2026-09-20 19:26:00 +08:00

Merge remote-tracking branch 'origin/master' into fix/agc-acceptance-followup 6fb17de9db

suzmii added 1 commit 2026-09-20 21:37:32 +08:00

Merge remote-tracking branch 'origin/master' into fix/agc-acceptance-followup a0d432b63d

suzmii added 6 commits 2026-09-21 00:37:27 +08:00

补齐画布验收第二批：派生任务进生成任务侧栏、提交中让出草稿与重复提交防护 ... 5b73082ad0

- 新增 resourceCanvasResourceEditTaskModel：原生资源编辑阶段到面板状态/文案的映射，本地提交记录与原生待办按 operationId 合并，派生任务的终态与耗时判据
- 「生成任务」侧栏同时收派生/修改类任务：与图片类生成任务同一条列表、同一套分栏与状态徽标，多一行提示词，样式沿用既有卡片类
- 快速编辑、生成动画、视频/音效/背景音乐三条派生提交：按下提交当帧写入本地记录并在侧栏可见，成功后收口为已完成、失败带原因并重读原生待办
- 「管理未完成编辑」不再列出正在提交的那一笔草稿，失败后自动回落，提示词与 @ 引用不丢
- 同一张素材存在在途提交时，重新打开面板再提交会被拦下并给出原因，避免重复派生与重复计费
- 侧栏与工具条入口的在途计数把派生任务一并计入
- 新增 tests/resourceCanvasResourceEditTasks.test.ts（9 条）；resourceCanvasQuickEditDraft.test.tsx 补两组宿主用例（提交当帧进侧栏并收口、在途重复提交被拦），两处新判据均做过变异验证
- 同步 PRD 派生任务侧栏口径、decision-log 与画布验收待办文档

补齐 AGC「生成动画」面板的宿主样式并加样式缺口守卫 ... 573f9f447a

- resourceCanvasChrome.css 逐条照抄 src/index.css 的角色动画面板规则：面板 / 提示词 textarea / 动作预设 / 底部动作行 / 提交按钮，以及关闭键、素材缩略图与 reference-chip 全族
- 窄屏（≤720px）与网页端同口径：面板落到底部整宽、动作竖排、提交按钮整宽
- 新增 tests/resourceCanvasCharacterAnimationPanelStyle.test.tsx：渲染共享动画面板，按 src/main.tsx 的真实 import 关系解析 AGC 实际加载的样式表，断言「网页端给出规则的类 AGC 必须也有一条」并抽查撑开布局的关键声明
- 变异验证：临时移除本次补的样式块，守卫用例列出缺规则的 15 个类并红灯

卡片浮层改为提交即关：快速编辑 / 生成动画只负责交任务 ... 973e965f4f

- 快速编辑与生成动画点提交当帧即关面板（不等 IPC、不留等待态），进度、阶段与结果只由「生成任务」侧栏承载，与图片类生成面板同一口径
- 受理之后的失败不再重开面板：侧栏那条收口为失败 + 提示条，草稿回落成「未完成编辑」等用户重开；只有点击瞬间就失败（名称预检、源素材不在画布、无客户端）留在面板里
- 派生请求身份改为按「入口 + 资源路径」记忆：重开面板再提交时提示词没变就沿用同一 operationId / 幂等键，命中同一 operation 账本；派生成成功后清除
- 删除 canDismissResourceCanvasQuickEdit 与宿主里只为它服务的两块面板状态 ref，clearResourceCanvasFocus 回到「清选中 + 关两块浮层」的直线逻辑
- 三条既有用例按新口径改写为「重开面板再重试，身份不变」；PRD §3.10 与 decision-log 同步

按美术画布原样补齐动画面板样式：关闭键落位、提示词态与 760px 整档 ... 5b4f9e961d

- 补 `.image-canvas-editor__generation-close` 的第二条同名规则（浮层内绝对定位）：上一版只搬了基础外观，关闭键因此掉进网格流、独自占一行并居中
- 补提示词输入区的聚焦态与 aria-invalid 两档，以及 composer 内参数簇的透明底色与 hover / 展开态
- 新增 `@media (max-width: 760px)` 整档照抄（浮层落底部整宽、composer 定宽、动作竖排、参数与提交整宽），替换上一版按 720px 手写的那一小段，选择器与断点都与网页端逐字一致
- 守卫升级为「选择器 + 声明」逐条比对：同名规则只搬第一条不再判过；范围限定在 `image-canvas-editor__` 命名空间（共享 chrome 类与全局 reset 不归宿主 chrome 表照抄）

删除角色动画面板里重复回显提示词的「动画描述」行 ... 55259683b8

- 共享组件 ImageCanvasCharacterAnimationPanelView 不再渲染只读的生成文本回显（与上方输入区内容重复，且两个宿主都没有为它定义样式）
- 用例同步：原断言「生成文本：…」改为钉住该节点不再渲染，并保留「输入区确实收下这段文本」的检查

修掉生图弹窗点输入框就弹素材选择框：label 转发点击到 @ 按钮 ... aa3e29f04e

- 现象：点生成提示词输入框任意位置都会弹出「选择素材」。根因是这一格用 `<label>` 包住，而 `<label>` 会把点击转发给内部第一个可标注控件——正好是引用输入区里的「插入素材引用」按钮
- 处理：该格改为 `<div>` 包裹；可访问名仍由控件自身的 aria-label 提供（引用输入区 `ariaLabel` 与 `PlatformTextField` 的 `aria-label`），不依赖 label 关联
- 用例：`resourceCanvasAssetGenerationReferences` 新增「点提示词输入区不该自己弹出素材选择框，只有 @ 按钮才开」——先用旧代码复现出红灯再修
- 顺带扫过资源工作台其余 `<label>`：都只包纯 input / textarea（音频与视频生成面板用纯 textarea，没有 @ 按钮），无同类隐患
- 待办文档补「2026-09-21 第三批」：卡片浮层提交即关、动画面板样式整档照抄与守卫升级、本条 label 修复

suzmii added 1 commit 2026-09-21 00:38:10 +08:00

Merge branch 'master' into fix/agc-acceptance-followup eed72ba39a

suzmii marked the pull request as ready for review 2026-09-21 00:38:14 +08:00

suzmii added 2 commits 2026-09-21 01:24:39 +08:00

按 review 修：动画在途拦截、样式守卫扩到三面板、窄屏差异显式放行、用例不再依赖 cwd ... 2a2e4c1cea

- 在途拦截补齐生成动画：提交记录的「属于哪张资源」从 draftKey 拆成 resourcePath，
  `resourceCanvasResourceEditSubmissionBlocksResource` 被快速编辑与生成动画共用；
  此前动画这条键对不上、拦不住，重开面板再提交会复用同一 operationId 卡在原生锁上，
  或（改了提示词）直接多出一笔付费生成
- 样式缺口守卫扩到三块面板（动画面板 / 快速编辑面板 / 生图面板），判据改为按
  「选择器分支 + 声明」逐条比对；两个面板的实测缺口一并照抄补上（快速编辑面板的
  `overflow: auto → visible` 与窄屏底栏两条按宿主差异显式登记，不静默漂移）
- 窄屏只搬动画面板那一支：网页端 ≤760px 会把快速编辑浮层也变成整宽底栏，AGC 的
  快速编辑是「居中贴资源卡」的绝对定位，跟随它会压住栏目工具栏与任务侧栏
- 用例路径不再依赖 cwd：新增 tests/repoPath.ts（按 import.meta.url 反推仓库根），
  19 个用例文件的 `resolve(process.cwd(), …)` 统一改用它；顺手修掉
  `HOST_STYLE_SHEETS.filter(existsSync)` 把仓库相对路径交给 cwd 解析的假红
- 清掉两处死代码：不可达的 `quickEditPanel.status === 'generating'` 判据与
  未被使用的 prune 导出

美化生图浮层：紧凑排布不再出现滚动条，画布卡片外形收敛到基类 token ... 329dd749eb

- 字段行改两列（标签在左、控件在右），比例与尺寸并排一行，提示词 6 行降到 3 行
- 润色入口、参考图计数与取消 / 生成两个按钮收进同一行动作：原先它们各占一行把内容顶到
  几何上界之外，面板因此出现滚动条；重排后面板高度回落到上界以内（overflow 仅作兜底）
- 新增画布卡片基类 token（.game-project-workbench 上的 --game-canvas-card-*），取值照抄
  网页端美术画布的面板；占位卡与生成浮层/弹窗从此共用一套圆角、描边、底色与投影，各卡只表达差异
- 用例补一条结构回归：动作行同时装下润色 / 参考计数 / 两个动作，比例与尺寸同处一格

suzmii added 1 commit 2026-09-21 01:25:10 +08:00

补齐画布验收待办：review 修复与生图浮层美化的进展记录 ... 4bc0f04d74

- 记下在途拦截补齐生成动画、样式守卫扩到三面板、窄屏差异显式放行、生图浮层紧凑化与基类 token
- 更正「appSurface 16 条既有失败」的口径：那是从 apps 目录跑造成的 cwd 假红，仓库根跑为 0 失败

suzmii added 1 commit 2026-09-21 01:33:53 +08:00

合并 master：音频后台化与 Mac 构建管线进分支，冲突按新口径解 ... cd98e0603e

- index.tsx 的 audio/video 提交：取 master 侧（#437 已把它改成后台任务队列 + 项目内账本），
  卡片浮层那套本地提交记录在这条链路上不再需要——队列本身就是它的可见性与幂等来源
- PRD 更新时间两边的两条事实合成一行；decision-log 保留 master 的音频章节与我们的
  派生任务/提交即关章节，并把「appSurface 16 条既有失败」更正为 cwd 假红的口径
- 样式守卫里撤掉「生图面板」这条探针：它走 AGC 自有类名，本就不吃
  `image-canvas-editor__*` 那一族（守卫自身加的空集断言把这条假探针暴露了出来）
- 合并后从仓库根验证：appSurface 538 通过 / 17 跳过 / 0 失败；AGC 定向 84 条、typecheck、
  check:encoding、git diff --check 全通过

suzmii referenced this issue from a commit 2026-09-21 01:45:30 +08:00

文档同步：PR #419 合并前复核结论与全量测试口径

suzmii added 1 commit 2026-09-21 01:45:30 +08:00

文档同步：PR #419 合并前复核结论与全量测试口径 ... 5574f41129

- 待办文档补「合并前复核（2026-09-21，PR #419）」：仓库根全量 373 文件 / 4512 通过 / 34 跳过 / 0 失败、No Conflicts、范围核对（后端与契约零改动）
- decision-log 的验证段补同一口径；同时记下未复验的两类（真实客户端观感、远程 CI）都不阻塞合并

suzmii merged commit 1655a2eab4 into master 2026-09-21 02:02:20 +08:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

Compat/Breaking

Breaking change that won't be backward compatible

Kind/Bug

Something is not working

Kind/Documentation

Documentation changes

Kind/Enhancement

Improve existing functionality

Kind/Feature

New functionality

Kind/Security

This is security issue

Kind/Testing

Issue or pull request related to testing

Priority

Critical

1

The priority is critical

Priority

High

2

The priority is high

Priority

Low

4

The priority is low

Priority

Medium

3

The priority is medium

Reviewed

Confirmed

1

Issue has been confirmed

Reviewed

Duplicate

2

This issue or pull request already exists

Reviewed

Invalid

3

Invalid issue

Reviewed

Won't Fix

3

This issue won't be fixed

Status

Abandoned

3

Somebody has started to work on this but abandoned work

Status

Blocked

1

Something is blocking this issue or pull request

Status

Need More Info

2

Feedback is required to reproduce issue or to continue work

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

HuangJiaJia JenkenB k88936 (王德宇) kdletters (段舒康) laura2008 (王乐) lhk229 (孔令弘) Loer (农洛儿) loong suzmii (董羽秦) taobaisu (叶东旭) wuxiangwanzi (王子民) YixuanLiu ziyu

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: GenarrativeAI/Genarrative#419