fix(player): 记忆桌面端播放器音量，避免每次打开视频都重置为最大

[liangyuR]

问题

Windows/桌面端每次打开视频音量都重置为最大（100）。原因：桌面端音量是 media_kit (mpv) 的应用内音量，从未持久化。PlayerController 按视频路由注册为 singleton，离开视频页后销毁，下次打开是全新实例，volume 回到哨兵值 -1，init 时被硬编码为 100。

修复

• 在 lib/services/storage/settings_keys.dart 新增持久化设置 defaultVolume（double，默认 100.0，分组 player），并加入 all 注册列表。
• PlayerController.init() 桌面分支用 GStorage.getSetting(SettingsKeys.defaultVolume) 替代硬编码 100，恢复上次音量。
• 桌面端音量变更（滚轮/快捷键经 setVolume、手势经 finishVolumeGesture）后写回存储，仅在取整值变化时写入以避免冗余写。
• 移动端音量走系统音量，逻辑完全不变。

测试

• ✅ flutter analyze --no-fatal-infos --fatal-warnings（无 warning/error，仅既有 info 级 lint）
• ✅ flutter test test/settings_default_volume_test.dart（验证新设置默认值、分组、注册、持久化键名）

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

This increment merges the "unified episode identity" work into the branch: history progress is now matched by episodePageUrl (with legacy episode-index fallback and synthetic-bucket allocation) so that progress survives source/episode reordering, plus the ResolvedEpisode → EpisodeRef rename and sortNumber/pageUrl plumbing.

The _HistoryEpisodeMatcher logic is internally consistent between history_sync.dart and history_repository.dart, the resume path in video_page.dart correctly prioritizes page-URL matching with a bounded fallback to the stored road/episode, and the behavior is extensively covered by new tests (page-url collisions, synthetic buckets, backfill, legacy fallback, codec round-trips). No high-confidence issues were found in the PR-changed lines.

Note: lib/pages/player/player_item.dart and player_models.dart appear in the raw multi-commit patch but are not part of the PR's net diff (already present in the base), so they were treated as out of scope.

Files Reviewed (changed code)

• lib/modules/history/history_sync.dart
• lib/repositories/history_repository.dart
• lib/pages/video/video_controller.dart
• lib/pages/video/video_page.dart
• lib/pages/history/history_controller.dart
• lib/services/sync/history_sync_service.dart
• lib/modules/history/history_module.dart
• test/history_repository_test.dart
• test/history_sync_test.dart
• test/episode_ref_test.dart

Previous Review Summaries (2 snapshots, latest commit c7ab691)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit c7ab691)

Status: No Issues Found | Recommendation: Merge

The latest commits only remove two explanatory comments in _persistDesktopVolume and delete the settings_default_volume_test.dart test file. The core fix (persisting desktop player volume via defaultVolume, clamping to 0.0..100.0, gating on isDesktop(), and avoiding redundant writes) is unchanged and remains correct. No functional code changes were introduced in this increment.

Note: deleting test/settings_default_volume_test.dart removes coverage for the defaultVolume setting; consider keeping it. This is advisory only and not a blocker.

Files Reviewed (2 files)

• lib/pages/player/player_controller.dart
• test/settings_default_volume_test.dart (deleted)

Previous review (commit 0733b0b)

Status: No Issues Found | Recommendation: Merge

The fix is well-scoped: desktop player volume is now persisted via the new defaultVolume setting and restored on init() instead of resetting to 100. Persistence is correctly gated behind isDesktop(), values are clamped to 0.0..100.0, and redundant writes are avoided by comparing rounded values. Mobile (system-volume) behavior is unchanged.

Files Reviewed (3 files)

• lib/pages/player/player_controller.dart
• lib/services/storage/settings_keys.dart
• test/settings_default_volume_test.dart

---

_{Reviewed by claude-4.8-opus-20260528 · Input: 4.5K · Output: 25.2K · Cached: 1.3M}

[Predidit]

移除注释，这里的注释过于显而易见

移除测试，此测试本质在测试 hive 存储可靠性，没有必要