Skip to content

fix(ios): reconcile final lyrics state and playback intent - #11

Merged
a3384379 merged 1 commit into
masterfrom
codex/ios-lyrics-audit-round2
Oct 5, 2026
Merged

a3384379 merged 1 commit into
masterfrom
codex/ios-lyrics-audit-round2

Conversation

@a3384379

@a3384379 a3384379 commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

快速回切歌词开关时,Live Activity 可能停在中间状态;PiP 的播放目标也可能在旧回报尚未确认时丢失。此改动落实本轮 R1–R4 审计,让最后展示选择在在途提交完成后生效,并由新的已接受播放回报自动兑现最后播放目标。

  • R1:保留最新待提交内容,提交完成后重新协调;结束活动失效旧提交、防抖与过渡回调。
  • R2:使用同一展示快照及其语义标识判断和发布,覆盖前奏进入第一句、间奏及同文不同行。
  • R3:仅在应用有效锚点后续期;陈旧采样不改写播放字段,拖动跳过锚点不延长有效期,保留合法旧协议回退。
  • R4:分别维护最后期望、已确认状态及在途命令;确认后自动继续最后目标,超时不盲目重复 toggle。尊重既有传输门控,切歌、断线和 PiP 停止清理未发送意图。

R5 的显示 readiness、帧重试、缓冲恢复和系统停止清理保持,PiP 停止不发送 Sony 暂停。协议、Android、偏好格式、Widget 布局和工程配置未修改;原工作目录六项本地修改未纳入。

验证:最终模拟器 XCTest 104/104 通过(新增 13 项,原播放目标测试改为确认回报自动协调),generic iOS App/Widget BUILD SUCCEEDED,无 Swift 编译警告,git diff --check 通过。测试观测真实管理器的发布和命令接口,不代替系统 ActivityKit 或 Sony 设备验收。

边界:quick smoke overall FAIL、Required 2/6,其中两项为跳过 build/install;iPhone unavailable,启动、日志、偏好与容器未完成。代码图谱工作树重建再次 worker crash,影响范围由源码、差异、构建与测试复核。跨应用 PiP、灵动岛、BLE/后台及音频真机验收仍待完成。用户已明确要求在这些边界下修改、提交并合并。

详见 docs/LYRICS_DISPLAY_IMPLEMENTATION.md 的 R1–R4 整改与验证记录。

GitHub 对当前提交 77905901bf42936de1110d8847d1d973b03083d4 的验证:PR 与 push 的 iOS XCTest、Android 单测/APK 四项检查最终均为成功。PR 的 iOS 首轮 104/104 通过;push 的 iOS 首轮新增歌词测试 37/37 通过,但原有 PerformanceStabilityTests.testLastNowPlayingSnapshotStoreSaveLoadAndClear 出现既有的 load=nil 偶发失败,工作流自动重启模拟器后 104/104 通过。该存储测试文件本轮未修改,不能把重试成功表述为已修复该偶发问题。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 5, 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-10-05T13:36:28.714503Z 7790590 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.

@a3384379
a3384379 merged commit c67a712 into master Oct 5, 2026
4 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