Skip to content

fix(interview): 移除数字人区域重复字幕 - #345

Merged
Scorbunny2 merged 1 commit into
1024XEngineer:devfrom
Scorbunny2:codex/issue-344-remove-interview-avatar-subtitle
Aug 5, 2026
Merged

fix(interview): 移除数字人区域重复字幕#345
Scorbunny2 merged 1 commit into
1024XEngineer:devfrom
Scorbunny2:codex/issue-344-remove-interview-avatar-subtitle

Conversation

@Scorbunny2

Copy link
Copy Markdown

功能描述

  • 移除沉浸式面试数字人区域底部的重复深色字幕框。
  • 下方对话区、消息操作、录音入口和其他面试流程保持不变。
  • 非面试沉浸式练习仍保留数字人区域字幕。

实现思路

  • 复用 AgentController.practiceSceneFamily 区分面试和非面试场景。
  • 面试场景不再向 _AvatarStage 传入最新 AI 消息;其他场景保持原有传值和渲染。
  • 不修改后端、API、会话模型或下方对话区布局。

测试方式

flutter test --no-pub test/practice/immersive_roleplay_test.dart

结果:沉浸式页面 14 个 Widget 测试全部通过,包含“面试不显示顶部字幕”和“非面试仍显示字幕”的隔离验证。

Closes #344

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found in this review.

Verification note: I attempted to run flutter test test/practice/immersive_roleplay_test.dart from mobile/, but flutter is not installed in this runner.

@Scorbunny2
Scorbunny2 force-pushed the codex/issue-344-remove-interview-avatar-subtitle branch from a99c581 to 56061aa Compare August 5, 2026 02:47
@Scorbunny2
Scorbunny2 force-pushed the codex/issue-344-remove-interview-avatar-subtitle branch from 56061aa to 7a17987 Compare August 5, 2026 03:23
@jyqin0203
jyqin0203 self-requested a review August 5, 2026 03:41
@Scorbunny2
Scorbunny2 merged commit 55ae096 into 1024XEngineer:dev Aug 5, 2026
6 checks passed
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.

3 participants