dong/webhtv-mys
0
Fork 0
mirror of https://github.com/Silent1566/webhtv.git synced 2026-10-13 18:27:10 +00:00
Code Issues Releases Packages Activity

merge: 合并 origin/beta 最新代码（PR#435 dev2「退出播放页必定停播」）并复评 dev1 已提交未推送的 C59/C60

Browse source 
合并：git merge --no-commit --no-ff origin/beta（00ad80114fa971c32de3a74f072e33714b9f488e，Merge pull request #435 from Silent1566/dev2），0 冲突；纳入 beta 侧 5 个提交（含 dev2 的 fix(playback): 退出播放页必定停播与两份 C58 文档）。零复活/零丢失：beta tip 5025 个文件逐个 blob 比对，缺失 0、逐字节一致 5021，差异仅 dev1 自己的 C59/C60 路径；行级严格判据（407 个单亲回退提交的 53889 条删除行 ∩ dev1 新增 432 行）复活行数 0。

复评（2 轮）发现并修复 2 项真实缺陷：

- F1（判定方向错误，可导致自动降级）：AppVersion.isNewer 原文把「长度不同的两个可用构建时间戳」当成不可比较并直接判定目标为更新，负对照实测 isNewer("v5.6.0-beta-20261009175522", "v5.6.0-202609221328") = true，即 14 位秒级时间戳的装机版本会把更早的 12 位渠道包当作更新推荐，与 BUILD_STAMP 的 \d{12,14} 及 buildStamp() javadoc 声明的 yyyyMMddHHmm[ss] 相矛盾。修复：新增 STAMP_LENGTH/padStamp()，把分钟级时间戳补齐为该分钟第 0 秒后比较，只有真正缺失时间戳才回退历史语义；新增用例 AppVersionBuildStampTest#stampsOfDifferentLengthsAreStillOrdered 锁定，修复后同一探针返回 false 且 10 条探针断言全部通过。
- F2（注释与实现矛盾）：AppDatabase 的 IntegrityRecoveringOpenHelperFactory 与 quarantine() 注释称「不可读的数据库文件被保全而非删除、原文件留在磁盘上」，但实现会清理 -wal/-shm/-journal，且 renameTo 失败时删除主库文件。注释改为如实描述（主库改名保全 + 边车文件清理 + 无法改名时删除），行为零改动。

验证：bash ./gradlew :app:compile{Leanback,Mobile}Arm64_v8aDebugJavaWithJavac :app:compile{Leanback,Mobile}Arm64_v8aDebugAndroidTestJavaWithJavac :app:test{Leanback,Mobile}Arm64_v8aDebugUnitTest => BUILD SUCCESSFUL in 7m 8s；leanback 4191 tests / 0 failures / 0 errors / 2 skipped，mobile 5006 / 0 / 0 / 2（相对 C58 基线 4172/4987 各 +19 = C59 +12、C60 +4、合并带入的 PlaybackOwnershipSourceTest +3）；bash scripts/check_ui_tokens.sh => UI_TOKEN_STATUS PASS，violations=1 与基线一致（零新增）。

未使用真机：本轮对生产代码的改动为 1 处判定分支 + 2 处注释，不改变正常升级方向与数据库打开/迁移路径，被复评的 C59/C60 已由用户在同一机位（192.168.50.3:5555，Android 9）按四种场景实机验收。

Verification: 合并 origin/beta 00ad80114（0 冲突，MERGE_HEAD=beta tip，双父合并提交）；零复活行级严格判据=0（407 个单亲回退提交 / 53889 删除行 ∩ dev1 新增行 432 行 = 56 行交集，全部存在于 beta tip）；零丢失=beta tip 5025 文件逐个 blob 比对，缺失 0、新增仅 dev1 自己的 5 个新文件；复评 2 轮，第 1 轮发现 F1（AppVersion.isNewer 跨长度时间戳误判为更新，负对照实测 isNewer(14位,更早的12位)=true）与 F2（AppDatabase 注释与实现矛盾）并修复，第 2 轮复审通过；双 flavor Java 编译 + 双 flavor AndroidTest 编译 + 双 flavor 全量 JVM 套件 BUILD SUCCESSFUL（leanback 4191/0/0、mobile 5006/0/0，相对 C58 基线 4172/4987 各 +19，可逐项解释：C59 +12、C60 +4、合并带入 beta 的 PlaybackOwnershipSourceTest +3）；UI token 门禁 PASS（violations=1 与基线一致，零新增）

Task-Guard: C61-beta-merge-review-dev1-20261011
This commit is contained in:
Silent1566 2026-10-11 14:00:09 +08:00
commit 8b54e0e97c
9 changed files with 576 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files