Skip to content

feat: CH-2 视频 Inspector 角色区——抽共享 section + CH-1 注入的用户入口 - #230

Merged
KerroKapple merged 3 commits into
mainfrom
feat/ch-2-video-inspector-characters
Aug 20, 2026
Merged

feat: CH-2 视频 Inspector 角色区——抽共享 section + CH-1 注入的用户入口#230
KerroKapple merged 3 commits into
mainfrom
feat/ch-2-video-inspector-characters

Conversation

@KerroKapple

Copy link
Copy Markdown
Owner

概要

CH-2 收口(计划:docs/superpowers/plans/2026-08-20-ch-2-video-inspector-characters.md,随本 PR 入库)。CH-1(#209)打通了视频角色注入链路,但 video inspector 一直没有角色区——用户没有入口写 type_config.character_ids。本 PR 补上这个入口。

改动

  1. 抽共享 widget(行为零变化):_CharactersSection/_NameDialog/_CharacterChipimage_config_inspector.dart 整体搬到新 characters_section.dart 并转公开(CharactersSection/InspectorNameDialog/CharacterChip);_PresetsSection 改 import 复用,不留私有副本防漂移。image inspector 1033→672 行。零变化证据:既有角色区测试一行未改全绿。
  2. 门控参数化:新增 requireImageToImageMode——image 传 true 维持原门(maxRefImages>0 && imageToImage);video 传 false 对齐 CH-1 注入门(仅 maxRefImages>0不检查 modes——r2v/omni 的参考图语义不落在 modes 里,见 generation_controller._injectCharacterRefs 拍板注释)。
  3. video 侧挂载maxRefImages>0 才挂整段——与 image 侧「常挂+警示文案」有意不同:视频多数 provider 无 ref 能力,常挂 = 一屏死区。

测试

  • 新增 3 例(TDD 先红后绿):无 imageToImage mode 的视频 caps 也挂载(误用 image 门必红)/ maxRefImages=0 整段不挂 / chip 点选落 type_config.character_ids
  • 既有 image 角色区测试一行未改全绿;flutter analyze 无 issue;全量 flutter test --exclude-tags golden 1951 全绿。
  • 零新 ARB key(复用 inspectorCharacters* 既有键);未触碰 _PromptPreview(P1-17 债明令勿同窗)。

🤖 Generated with Claude Code

Kerro and others added 3 commits August 20, 2026 13:55
…t,image inspector 1033→672 行行为零变化

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… 对齐注入口径

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@KerroKapple
KerroKapple merged commit 388022a into main Aug 20, 2026
5 checks passed
@KerroKapple
KerroKapple deleted the feat/ch-2-video-inspector-characters branch August 20, 2026 06:11
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