Skip to content

feat: 将播放偏好移至设置并按偏好语言选择字幕和音轨 - #100

Merged
mahoshojoHCG merged 2 commits into
mainfrom
codex/playback-preferences-settings
Oct 2, 2026
Merged

mahoshojoHCG merged 2 commits into
mainfrom
codex/playback-preferences-settings

Conversation

@mahoshojoHCG

Copy link
Copy Markdown
Contributor

将播放偏好从播放器移到「设置 → 播放偏好」,仅提供自动播放下一集、自动跳过片头片尾和统一偏好语言。移除持久化具体字幕/音轨的选择入口;字幕控制栏仍可临时切换或关闭,不改写档案偏好。

实现

  • 偏好按当前 profile 保存并同步,所有登录角色可查看,保存遵循现有 PlaybackWrite 权限;补齐中英日文案、加载/失败状态及播放器缓存更新。
  • 偏好语言支持跟随界面、中文、日语、英语,统一用于外置/内嵌字幕、MKV 音轨及 HLS 请求。沿用现有语言字段,保存时清除旧轨道标签,无数据库迁移。
  • MKV 探测和 Mediabunny proxy 使用同一语言选择函数,通过现有 Yarn patch 增加初始音轨选择回调;无匹配音轨时保留媒体默认。偏好变化和 HLS 回退保留播放进度及临时字幕选择。
  • 自动跳过继续使用单集标注或已确认的整季模板,开关文案明确此边界。补充 播放偏好与识别调研:Plex 按季比较重复音频,Intro Skipper 组合章节、音频指纹与黑帧/静音,mpv 脚本使用章节或 AniSkip 数据。建议后续先接入明确命名的 OP/ED 章节,再考虑后台音频指纹分析;本 PR 未新增自动识别引擎。

验证与限制

  • yarn install --immutable
  • yarn typecheck
  • yarn test:unit:现有 34 项通过
  • yarn build:budget:生产构建及全部包体积检查通过
  • yarn test:e2e:使用全新 mock 状态运行,现有 6 项通过
  • 变更文件 Prettier 检查、git diff --check;未新增测试或回归检查。
  • 浏览器手动验证设置保存与刷新保持、播放器偏好卡片/跳过开关移除、临时关闭字幕不改写偏好语言。

原生播放只在浏览器暴露 audioTracks 时切换音轨;MKV proxy 与服务端 HLS 不依赖该浏览器 API。未使用真实多音轨 MKV 做端到端媒体播放验证。后端未改动,未运行无关后端构建。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-27T16:46:25.978818Z d292861 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@mahoshojoHCG
mahoshojoHCG merged commit 41c8c22 into main Oct 2, 2026
10 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.

1 participant