test: address latest review feedback

CHANGELOG.md

@@ -1,40 +1,14 @@
 # Changelog
 
-## v0.9.1 (2026-03-24)
+## v0.9.0 (2026-03-22)
 
 ### Changed
-- Release: Reduced packaged release size by excluding duplicate `extraResources` payload and pruning docs, tests, sourcemaps, and other source-only files from Electron bundles.
-
-### Fixed
-- Overlay: Restored controller navigation and lookup/mining controls while the subtitle sidebar is open, while keeping true modal dialogs blocking controller actions.
-- Tokenizer: Fixed subtitle annotation clearing so explanatory contrast endings like `んですけど` are excluded consistently across the shared tokenizer filter and annotation stage.
-
-## v0.9.0 (2026-03-23)
-
-### Added
-- Docs: Added a new WebSocket / Texthooker API and integration guide covering WebSocket payloads, custom client patterns, mpv plugin automation, and webhook-style relay examples. Linked from configuration and mining workflow docs for easier discovery.
-
-### Changed
-- Launcher: Added an app-owned YouTube subtitle flow that pauses mpv, uses absPlayer-style YouTube timedtext parsing/conversion to download subtitle tracks, and injects them as external files before playback resumes.
-- Launcher: Changed YouTube subtitle startup to auto-load the best-available primary and secondary subtitle tracks at launch instead of forcing the picker modal first. Secondary subtitle failures no longer block playback resume.
-- Launcher: Added `Ctrl+Alt+C` as the default keybinding to manually open the YouTube subtitle picker during active YouTube playback.
-- Launcher: Added yt-dlp metadata probing so YouTube playback and immersion tracking record canonical video title and channel metadata.
-- Launcher: Stopped forcing `--ytdl-raw-options=` before user-provided mpv options so existing YouTube cookie integrations in user `--args` are no longer clobbered.
-- Launcher: Disabled mpv native YouTube subtitle auto-loading for the app-owned flow so injected external subtitle files remain authoritative.
+- Subtitle Sidebar: Added subtitle sidebar state and behavior updates, including startup-auto-open controls and resume positioning improvements.
+- Subtitle Sidebar: Fixed subtitle prefetch and embedded overlay passthrough sync between sidebar and overlay subtitle rendering.
+- Launcher: Added an app-owned YouTube subtitle picker flow that boots mpv paused, opens an overlay track picker, and downloads selected subtitles into external files.
+- Launcher: Added explicit `download` and `generate` YouTube subtitle modes with `download` as the default path.
+- Launcher: Disabled mpv native YouTube subtitle auto-loading for the app-owned flow so external subtitle files stay authoritative.
 - Launcher: Added OSD status messages for YouTube playback startup, subtitle acquisition, and subtitle loading so the flow stays visible before and during the picker.
-- Subtitle Sidebar: Added startup-auto-open controls and resume positioning improvements so the sidebar jumps directly to the first resolved active cue.
-- Subtitle Sidebar: Improved subtitle prefetch and embedded overlay passthrough sync so sidebar and overlay subtitle states stay consistent across media transitions.
-- Subtitle Sidebar: Updated scroll handling, embedded layout styling, and active-cue visual behavior.
-- Stats: Stats Library tab now displays YouTube video title, channel name, and channel thumbnail for YouTube media entries, with retry logic to fill in metadata that arrives after initial load.
-
-### Fixed
-- Launcher: Fixed Anki media mining for mpv YouTube streams by unwrapping the stream URL so audio and screenshot capture work correctly for YouTube playback sessions.
-- Immersion: Fixed YouTube media path handling in the immersion runtime and tracking so YouTube sessions record correct media references, AniList guessing skips YouTube URLs, and post-watch state transitions do not fire for YouTube media.
-- Launcher: Fixed startup-launched YouTube playback so primary subtitle overlay updates continue after auto-load completes.
-- Launcher: Fixed auto-loaded YouTube primary subtitles so parsed cues appear in the subtitle sidebar without needing a manual picker retry.
-- Launcher: Fixed the YouTube picker to guard against duplicate subtitle submissions and tightened YouTube URL detection so follow-up runtime flows only treat real YouTube hosts as YouTube playback.
-- Launcher: Fixed primary subtitle failure notifications being shown while app-owned YouTube subtitle probing and downloads are still in flight.
-- Launcher: Preserved existing authoritative YouTube subtitle tracks when available; downloaded tracks are used only to fill missing sides, and native mpv secondary subtitle rendering is hidden so the overlay remains the sole secondary display.
 
 ## v0.8.0 (2026-03-22)
 

README.md

@@ -66,10 +66,6 @@ Local stats dashboard — watch time, anime library, vocabulary growth, mining t
 ### Integrations
 
 <table>
-  <tr>
-    <td><b>YouTube</b></td>
-    <td>Auto-loaded yt-dlp subtitle tracks at startup with a manual overlay picker on demand (<code>Ctrl+Alt+C</code>)</td>
-  </tr>
   <tr>
     <td><b>AniList</b></td>
     <td>Automatic episode tracking and progress sync</td>
@@ -222,7 +218,6 @@ subminer video.mkv          # play video with overlay
 subminer --start video.mkv  # explicit overlay start
 subminer stats              # open immersion dashboard
 subminer stats -b           # stats daemon in background
-subminer stats -s           # stop background stats daemon
 ```
 
 ---

backlog/tasks/task-143 - Keep-character-dictionary-auto-sync-non-blocking-during-startup.md

@@ -1,11 +1,11 @@
 ---
 id: TASK-143
 title: Keep character dictionary auto-sync non-blocking during startup
-status: Done
+status: In Progress
 assignee:
   - codex
 created_date: '2026-03-09 01:45'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 09:22'
 labels:
   - dictionary
   - startup
@@ -18,7 +18,7 @@ references:
   - >-
     /home/sudacode/projects/japanese/SubMiner/src/main/runtime/current-media-tokenization-gate.ts
 priority: high
-ordinal: 144500
+ordinal: 38500
 ---
 
 ## Description

backlog/tasks/task-177.1 - Fix-overview-lookup-rate-metric.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-19 17:46'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-19 17:54'
 labels:
   - stats
   - immersion-tracking
@@ -19,7 +19,6 @@ references:
   - src/core/services/stats-server.ts
 parent_task_id: TASK-177
 priority: medium
-ordinal: 132500
 ---
 
 ## Description

backlog/tasks/task-177.2 - Count-homepage-new-words-by-headword.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-19 19:38'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-19 19:40'
 labels:
   - stats
   - immersion-tracking
@@ -17,7 +17,6 @@ references:
   - stats/src/lib/dashboard-data.ts
 parent_task_id: TASK-177
 priority: medium
-ordinal: 130500
 ---
 
 ## Description

backlog/tasks/task-177.3 - Fix-attached-stats-command-flow-and-browser-config.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-19 20:15'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-19 20:17'
 labels:
   - launcher
   - stats
@@ -19,7 +19,6 @@ references:
   - src/main/runtime/stats-cli-command.test.ts
 parent_task_id: TASK-177
 priority: medium
-ordinal: 129500
 ---
 
 ## Description

backlog/tasks/task-182.2 - Improve-session-detail-known-word-chart-scaling.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-19 20:31'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-19 20:52'
 labels:
   - bug
   - stats
@@ -17,7 +17,6 @@ references:
   - >-
     /Users/sudacode/projects/japanese/SubMiner/stats/src/lib/session-detail.test.tsx
 parent_task_id: TASK-182
-ordinal: 128500
 ---
 
 ## Description

backlog/tasks/task-188 - Refactor-stats-chart-data-pipeline-to-use-backend-aggregated-series.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-18 00:29'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-18 00:55'
 labels:
   - stats
   - performance
@@ -22,7 +22,6 @@ references:
   - stats/src/types/stats.ts
   - stats/src/lib/dashboard-data.ts
 priority: medium
-ordinal: 138500
 ---
 
 ## Description

backlog/tasks/task-191 - Assess-PR-19-CodeRabbit-review-follow-ups.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-17 23:15'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-17 23:18'
 labels:
   - pr-review
   - stats
@@ -16,7 +16,6 @@ references:
   - src/core/services/immersion-tracker-service.ts
   - src/core/services/immersion-tracker-service.test.ts
 priority: medium
-ordinal: 139500
 ---
 
 ## Description

backlog/tasks/task-192 - Assess-remaining-PR-19-review-batch.md

@@ -0,0 +1,76 @@
+---
+id: TASK-192
+title: 'Assess remaining PR #19 review batch'
+status: Done
+assignee:
+  - codex
+created_date: '2026-03-17 23:24'
+updated_date: '2026-03-17 23:42'
+labels:
+  - pr-review
+  - stats
+  - docs
+milestone: m-1
+dependencies: []
+references:
+  - docs/superpowers/plans/2026-03-12-immersion-stats-page.md
+  - src/core/services/immersion-tracker/__tests__/query.test.ts
+  - src/core/services/ipc.ts
+  - src/core/services/stats-server.ts
+  - src/main.ts
+  - src/renderer/handlers/keyboard.ts
+  - stats/src
+priority: medium
+---
+
+## Description
+
+<!-- SECTION:DESCRIPTION:BEGIN -->
+Validate the remaining PR #19 automated review findings against the current branch, implement only the technically correct fixes, and document which comments are stale, already addressed, or not warranted.
+<!-- SECTION:DESCRIPTION:END -->
+
+## Acceptance Criteria
+<!-- AC:BEGIN -->
+- [x] #1 Each remaining review comment is classified as actionable, already fixed, stale, or not warranted
+- [x] #2 Confirmed bugs or correctness issues are fixed with focused regression coverage where it fits
+- [x] #3 Final notes record which comments were intentionally not applied and why
+<!-- AC:END -->
+
+## Implementation Plan
+
+<!-- SECTION:PLAN:BEGIN -->
+1. Inspect the referenced files in batches and compare each comment against current branch behavior.
+2. Separate correctness/security regressions from stylistic nitpicks and already-fixed items.
+3. Add tests first for confirmed behavior bugs where practical, apply the smallest safe fixes, and rerun targeted verification.
+<!-- SECTION:PLAN:END -->
+
+## Implementation Notes
+
+<!-- SECTION:NOTES:BEGIN -->
+Swept the pasted PR #19 review batch against the current branch.
+
+Classification:
+- Already fixed on current branch: `src/core/services/immersion-tracker/__tests__/query.test.ts` cleanup rethrow, `src/core/services/ipc.ts` limit validation, `src/core/services/stats-server.ts` max-limit parsing and CORS removal, `src/main.ts` quit-path TDZ issue, `src/renderer/handlers/keyboard.ts` stats-toggle shortcut ordering/config usage, `stats/src/components/vocabulary/WordList.tsx`, `stats/src/hooks/useSessions.ts`, `stats/src/hooks/useTrends.ts` stale-error reset, `src/core/services/__tests__/stats-server.test.ts` kanji endpoint/readability notes, `src/core/services/stats-window.ts`, `stats/src/App.tsx`, `stats/src/components/layout/TabBar.tsx`, `stats/src/components/overview/QuickStats.tsx`, `stats/src/components/overview/WatchTimeChart.tsx`, `stats/src/components/sessions/SessionDetail.tsx`, `stats/src/components/sessions/SessionRow.tsx`, `stats/src/components/trends/DateRangeSelector.tsx`, `stats/src/components/vocabulary/KanjiBreakdown.tsx`, `stats/src/components/vocabulary/VocabularyTab.tsx`, `stats/src/hooks/useVocabulary.ts`, `stats/src/lib/api-client.ts`, `stats/src/types/stats.ts`.
+- Stale / obsolete against current architecture: `docs/superpowers/plans/2026-03-12-immersion-stats-page.md` path does not exist on this branch; `stats/src/components/trends/TrendsTab.tsx` / monthly-range comments describe older client-side aggregation code that is no longer present because trends now come from `getTrendsDashboard`.
+- Not warranted as written: `stats/src/lib/formatters.ts` no longer emits negative `Xd ago`; current code short-circuits future timestamps to `just now`, so the reported bug condition is gone even though the suggested wording differs.
+- Actionable and fixed now: `src/core/services/ipc.ts` no-tracker `statsGetOverview` fallback omitted required hint fields (`totalLookupCount`, `totalLookupHits`, `newWordsToday`, `newWordsThisWeek`). Added the missing fields in the fallback object and updated IPC tests to assert the full shape.
+
+Verification:
+- `bun test src/core/services/ipc.test.ts`
+- `bun test src/core/services/ipc.test.ts --test-name-pattern "empty stats overview shape without a tracker|validates and clamps stats request limits"`
+- `bash .agents/skills/subminer-change-verification/scripts/classify_subminer_diff.sh src/core/services/ipc.ts src/core/services/ipc.test.ts`
+
+Repo verifier note:
+- `bash .agents/skills/subminer-change-verification/scripts/verify_subminer_change.sh --lane core src/core/services/ipc.ts src/core/services/ipc.test.ts`
+- That verifier run captured a temporary `bun run typecheck` failure in `src/anki-integration.test.ts` and `src/core/services/__tests__/stats-server.test.ts`, but a fresh rerun after the follow-up validation no longer reproduces those diagnostics.
+- Fresh verification: `bun run typecheck` passes locally.
+- artifact dir from the earlier failed verifier snapshot: `.tmp/skill-verification/subminer-verify-20260317-234027-i6QJ3n`
+<!-- SECTION:NOTES:END -->
+
+## Final Summary
+
+<!-- SECTION:FINAL_SUMMARY:BEGIN -->
+The larger pasted PR #19 review batch was not mostly new work on the current branch. After verifying each item against the live code, almost all were already fixed or stale. One additional item was still actionable: the no-tracker fallback returned by `statsGetOverview` in `src/core/services/ipc.ts` omitted required hint fields, which made the fallback shape inconsistent with the normal overview payload. That fallback is now fixed and covered by IPC tests.
+
+Count-wise: the earlier open CodeRabbit service comments contributed 2 actionable fixes, and this larger pasted batch contributed 1 additional actionable fix on top of those.
+<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-192 - Fix-stale-anime-cover-art-after-AniList-reassignment.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-20 00:12'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 00:14'
 labels:
   - stats
   - immersion-tracker
@@ -17,7 +17,6 @@ references:
   - src/core/services/immersion-tracker/query.ts
   - src/core/services/immersion-tracker-service.test.ts
 priority: medium
-ordinal: 127500
 ---
 
 ## Description

backlog/tasks/task-194 - Redesign-YouTube-subtitle-acquisition-around-download-first-track-selection.md

@@ -4,7 +4,6 @@ title: App-owned YouTube subtitle picker flow
 status: Done
 assignee: []
 created_date: '2026-03-18 07:52'
-updated_date: '2026-03-23 03:22'
 labels: []
 dependencies: []
 references:
@@ -14,7 +13,6 @@ references:
 documentation:
   - /home/sudacode/projects/japanese/SubMiner/youtube.md
 priority: medium
-ordinal: 137500
 ---
 
 ## Description

backlog/tasks/task-196 - Fix-subtitle-prefetch-cache-key-mismatch-and-active-cue-window.md

@@ -4,16 +4,13 @@ title: Fix subtitle prefetch cache-key mismatch and active-cue window
 status: Done
 assignee: []
 created_date: '2026-03-18 16:05'
-updated_date: '2026-03-23 03:22'
 labels: []
 dependencies: []
 references:
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/core/services/subtitle-processing-controller.ts
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/core/services/subtitle-prefetch.ts
+  - /home/sudacode/projects/japanese/SubMiner/src/core/services/subtitle-processing-controller.ts
+  - /home/sudacode/projects/japanese/SubMiner/src/core/services/subtitle-prefetch.ts
+documentation: []
 priority: high
-ordinal: 136500
 ---
 
 ## Description

backlog/tasks/task-197 - Eliminate-per-line-plain-subtitle-flash-on-prefetch-cache-hit.md

@@ -4,19 +4,15 @@ title: Eliminate per-line plain subtitle flash on prefetch cache hit
 status: Done
 assignee: []
 created_date: '2026-03-18 16:28'
-updated_date: '2026-03-23 03:22'
 labels: []
 dependencies:
   - TASK-196
 references:
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/core/services/subtitle-processing-controller.ts
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/main/runtime/mpv-main-event-actions.ts
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/main/runtime/mpv-main-event-main-deps.ts
+  - /home/sudacode/projects/japanese/SubMiner/src/core/services/subtitle-processing-controller.ts
+  - /home/sudacode/projects/japanese/SubMiner/src/main/runtime/mpv-main-event-actions.ts
+  - /home/sudacode/projects/japanese/SubMiner/src/main/runtime/mpv-main-event-main-deps.ts
+documentation: []
 priority: high
-ordinal: 135500
 ---
 
 ## Description

backlog/tasks/task-199 - Forward-launcher-log-level-into-mpv-plugin-script-opts.md

@@ -4,7 +4,6 @@ title: Forward launcher log level into mpv plugin script opts
 status: Done
 assignee: []
 created_date: '2026-03-18 21:16'
-updated_date: '2026-03-23 03:22'
 labels: []
 dependencies:
   - TASK-198
@@ -13,8 +12,8 @@ references:
   - /home/sudacode/projects/japanese/SubMiner/launcher/mpv.ts
   - /home/sudacode/projects/japanese/SubMiner/launcher/main.test.ts
   - /home/sudacode/projects/japanese/SubMiner/launcher/aniskip-metadata.test.ts
+documentation: []
 priority: medium
-ordinal: 134500
 ---
 
 ## Description

backlog/tasks/task-200 - Address-latest-PR-19-CodeRabbit-follow-ups.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-19 07:18'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-19 07:28'
 labels:
   - pr-review
   - anki-integration
@@ -19,7 +19,6 @@ references:
   - src/anki-integration/runtime.ts
   - src/anki-integration/known-word-cache.ts
 priority: medium
-ordinal: 133500
 ---
 
 ## Description

backlog/tasks/task-201 - Suppress-repeated-macOS-overlay-loading-OSD-during-fullscreen-tracker-flaps.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-19 18:47'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-19 19:01'
 labels:
   - bug
   - macos
@@ -20,7 +20,6 @@ references:
   - >-
     /Users/sudacode/projects/japanese/SubMiner/src/core/services/overlay-visibility.test.ts
 priority: high
-ordinal: 131500
 ---
 
 ## Description

backlog/tasks/task-204.1 - Restore-stale-only-startup-known-word-cache-refresh.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@Codex'
 created_date: '2026-03-20 02:52'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 03:02'
 labels:
   - anki
   - cache
@@ -17,7 +17,6 @@ references:
   - docs/plans/2026-03-19-known-word-cache-incremental-sync-design.md
 parent_task_id: TASK-204
 priority: high
-ordinal: 124500
 ---
 
 ## Description

backlog/tasks/task-205 - Address-PR-19-Claude-frontend-review-follow-ups.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-20 02:41'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 02:46'
 labels: []
 milestone: m-1
 dependencies: []
@@ -14,7 +14,6 @@ references:
   - stats/src/hooks/useSessions.ts
   - stats/src/hooks/useTrends.ts
 priority: medium
-ordinal: 126500
 ---
 
 ## Description

backlog/tasks/task-206 - Assess-latest-PR-19-CodeRabbit-review-comments.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-20 02:51'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 02:59'
 labels:
   - pr-review
   - launcher
@@ -22,7 +22,6 @@ references:
   - src/anki-integration.ts
   - src/anki-integration/known-word-cache.ts
 priority: medium
-ordinal: 125500
 ---
 
 ## Description

backlog/tasks/task-207 - Verify-PR-19-follow-up-typecheck-blocker-is-cleared.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-20 03:03'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 03:04'
 labels:
   - pr-review
   - anki-integration
@@ -15,7 +15,6 @@ dependencies: []
 references:
   - src/anki-integration/anki-connect-proxy.test.ts
 priority: medium
-ordinal: 123500
 ---
 
 ## Description

backlog/tasks/task-208 - Assess-newest-PR-19-CodeRabbit-round-after-1227706.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-20 03:37'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 03:47'
 labels:
   - pr-review
   - launcher
@@ -17,7 +17,6 @@ references:
   - launcher/mpv.ts
   - src/anki-integration.ts
 priority: medium
-ordinal: 122500
 ---
 
 ## Description

backlog/tasks/task-209 - Exclude-grammar-tail-そうだ-from-subtitle-annotations.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-20 04:06'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 04:33'
 labels:
   - bug
   - tokenizer
@@ -18,7 +18,6 @@ references:
   - >-
     /Users/sudacode/projects/japanese/SubMiner/src/core/services/tokenizer.test.ts
 priority: high
-ordinal: 120500
 ---
 
 ## Description

backlog/tasks/task-210 - Show-latest-session-position-in-anime-episode-progress.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@Codex'
 created_date: '2026-03-20 04:09'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 04:25'
 labels:
   - stats
   - bug
@@ -17,7 +17,6 @@ references:
   - src/core/services/immersion-tracker/query.ts
   - src/core/services/immersion-tracker/session.ts
   - src/core/services/immersion-tracker-service.ts
-ordinal: 121500
 ---
 
 ## Description

backlog/tasks/task-211 - Recover-anime-episode-progress-from-subtitle-timing-when-checkpoints-are-missing.md

@@ -1,13 +1,11 @@
 ---
 id: TASK-211
-title: >-
-  Recover anime episode progress from subtitle timing when checkpoints are
-  missing
+title: Recover anime episode progress from subtitle timing when checkpoints are missing
 status: Done
 assignee:
   - '@Codex'
 created_date: '2026-03-20 10:15'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 10:22'
 labels:
   - stats
   - bug
@@ -16,26 +14,20 @@ dependencies: []
 references:
   - src/core/services/immersion-tracker/query.ts
   - src/core/services/immersion-tracker/__tests__/query.test.ts
-ordinal: 119500
 ---
 
 ## Description
 
-<!-- SECTION:DESCRIPTION:BEGIN -->
 Anime episode progress can still show `0%` for older sessions that have watch-time and subtitle timing but no persisted `ended_media_ms` checkpoint. Recover progress from the latest retained subtitle/event segment end so already-recorded sessions render a useful progress percentage.
-<!-- SECTION:DESCRIPTION:END -->
 
 ## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 `getAnimeEpisodes` returns the latest known session position even when `ended_media_ms` is null but subtitle/event timing exists.
-- [x] #2 Existing ended-session metrics and aggregation totals do not regress.
-- [x] #3 Regression coverage locks the fallback behavior.
-<!-- AC:END -->
+
+- [x] `getAnimeEpisodes` returns the latest known session position even when `ended_media_ms` is null but subtitle/event timing exists.
+- [x] Existing ended-session metrics and aggregation totals do not regress.
+- [x] Regression coverage locks the fallback behavior.
 
 ## Implementation Notes
 
-<!-- SECTION:NOTES:BEGIN -->
 Added a query-side fallback for anime episode progress: when the newest session for a video has no persisted `ended_media_ms`, `getAnimeEpisodes` now uses the latest retained subtitle-line or session-event `segment_end_ms` from that same session. This recovers useful progress for already-recorded sessions that have timing data but predate or missed checkpoint persistence.
 
 Verification: `bun test src/core/services/immersion-tracker/__tests__/query.test.ts` passed. `bun run typecheck` passed.
-<!-- SECTION:NOTES:END -->

backlog/tasks/task-212 - Fix-mac-texthooker-helper-startup-blocking-mpv-launch.md

@@ -1,10 +1,10 @@
 ---
 id: TASK-212
 title: Fix mac texthooker helper startup blocking mpv launch
-status: Done
+status: In Progress
 assignee: []
 created_date: '2026-03-20 08:27'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 08:45'
 labels:
   - bug
   - macos
@@ -15,7 +15,6 @@ references:
   - /Users/sudacode/projects/japanese/SubMiner/src/main.ts
   - /Users/sudacode/projects/japanese/SubMiner/plugin/subminer/process.lua
 priority: high
-ordinal: 140500
 ---
 
 ## Description

backlog/tasks/task-213 - Show-character-dictionary-progress-during-paused-startup-waits.md

@@ -1,10 +1,10 @@
 ---
 id: TASK-213
 title: Show character dictionary progress during paused startup waits
-status: Done
+status: In Progress
 assignee: []
 created_date: '2026-03-20 08:59'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-20 09:22'
 labels:
   - bug
   - ux
@@ -18,7 +18,6 @@ references:
     /Users/sudacode/projects/japanese/SubMiner/src/main/runtime/character-dictionary-auto-sync-notifications.ts
   - /Users/sudacode/projects/japanese/SubMiner/src/main.ts
 priority: medium
-ordinal: 141500
 ---
 
 ## Description

backlog/tasks/task-214 - Jump-subtitle-sidebar-directly-to-resume-position-on-first-resolved-cue.md

@@ -1,10 +1,10 @@
 ---
 id: TASK-214
 title: Jump subtitle sidebar directly to resume position on first resolved cue
-status: Done
+status: In Progress
 assignee: []
 created_date: '2026-03-21 11:15'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-21 11:15'
 labels:
   - bug
   - ux
@@ -12,12 +12,9 @@ labels:
   - subtitles
 dependencies: []
 references:
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/subtitle-sidebar.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/subtitle-sidebar.test.ts
+  - /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/subtitle-sidebar.ts
+  - /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/subtitle-sidebar.test.ts
 priority: medium
-ordinal: 142500
 ---
 
 ## Description

backlog/tasks/task-215 - Add-startup-auto-open-option-for-subtitle-sidebar.md

@@ -1,10 +1,10 @@
 ---
 id: TASK-215
 title: Add startup auto-open option for subtitle sidebar
-status: Done
+status: In Progress
 assignee: []
 created_date: '2026-03-21 11:35'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-21 11:35'
 labels:
   - feature
   - ux
@@ -13,15 +13,11 @@ labels:
 dependencies: []
 references:
   - /Users/sudacode/projects/japanese/SubMiner/src/types.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/config/definitions/defaults-subtitle.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/config/resolve/subtitle-domains.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/subtitle-sidebar.ts
+  - /Users/sudacode/projects/japanese/SubMiner/src/config/definitions/defaults-subtitle.ts
+  - /Users/sudacode/projects/japanese/SubMiner/src/config/resolve/subtitle-domains.ts
+  - /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/subtitle-sidebar.ts
   - /Users/sudacode/projects/japanese/SubMiner/src/renderer/renderer.ts
 priority: medium
-ordinal: 143500
 ---
 
 ## Description

backlog/tasks/task-217 - Fix-embedded-overlay-passthrough-sync-between-subtitle-and-sidebar.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-21 23:16'
-updated_date: '2026-03-23 03:22'
+updated_date: '2026-03-21 23:28'
 labels:
   - bug
   - overlay
@@ -18,7 +18,6 @@ references:
 documentation:
   - docs/workflow/verification.md
 priority: high
-ordinal: 118500
 ---
 
 ## Description

backlog/tasks/task-218 - Delete-zero-session-media-from-stats-library-and-trends.md

@@ -1,69 +0,0 @@
----
-id: TASK-218
-title: Delete zero-session media from stats library and trends
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-22 16:20'
-updated_date: '2026-03-24 06:41'
-labels:
-  - stats
-  - immersion-tracker
-dependencies: []
-references:
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/query.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/lifetime.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/maintenance.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/__tests__/query.test.ts
-priority: medium
-ordinal: 153500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Deleting the last retained session for a video still left stale lifetime media rows and trend rollups behind, so the stats dashboard could continue showing ghost entries in Library and Trends after all sessions were gone.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Deleting the final session for a video removes that media from Library queries and detail reads
-- [x] #2 Deleting the final session for a video removes stale daily/monthly trend rollups for that media
-- [x] #3 Regression coverage proves zero-session media disappears from affected stats surfaces after deletion
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Add a failing regression around deleting the only retained session for a video while preexisting lifetime and rollup rows exist.
-2. Patch the deletion path to rebuild lifetime and rollup state from retained sessions inside the same transaction.
-3. Run focused immersion-tracker tests plus the repo-native verifier core lane and record results.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Added a query regression that seeds a finished session plus stale lifetime media/anime rows and daily/monthly rollups, deletes that only session, and asserts Library, Anime detail, and Trends all drop the media immediately.
-
-Refactored lifetime rebuild logic so it can run inside an existing delete transaction, then reused that helper from `deleteSession`, `deleteSessions`, and `deleteVideo`.
-
-Added a rollup rebuild helper that clears existing daily/monthly rollups and reconstructs them from retained telemetry inside the current transaction so deleted sessions cannot leave ghost trend points behind.
-
-Verification passed:
-- `bun test src/core/services/immersion-tracker/__tests__/query.test.ts`
-- `bun test src/core/services/immersion-tracker-service.test.ts`
-- `bash .agents/skills/subminer-change-verification/scripts/classify_subminer_diff.sh src/core/services/immersion-tracker/query.ts src/core/services/immersion-tracker/lifetime.ts src/core/services/immersion-tracker/maintenance.ts src/core/services/immersion-tracker/__tests__/query.test.ts`
-- `bash .agents/skills/subminer-change-verification/scripts/verify_subminer_change.sh --lane core src/core/services/immersion-tracker/query.ts src/core/services/immersion-tracker/lifetime.ts src/core/services/immersion-tracker/maintenance.ts src/core/services/immersion-tracker/__tests__/query.test.ts`
-
-Verifier artifact dir: `.tmp/skill-verification/subminer-verify-20260322-210718-n6sGL8`
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Delete paths now rebuild lifetime summaries and trend rollups after removing sessions, so when the last session for a video disappears the stats database also drops that media from Library, related detail reads, and chart data. Added a regression proving a video with only stale lifetime/rollup rows vanishes after its final session is deleted, and verified the change with focused immersion-tracker tests plus the SubMiner core verification lane.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-219 - Restore-streamed-video-progress-in-anime-episodes.md

@@ -1,43 +0,0 @@
----
-id: TASK-219
-title: Restore streamed video progress in anime episodes
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-22 21:25'
-updated_date: '2026-03-24 06:44'
-labels:
-  - stats
-  - immersion-tracker
-  - youtube
-dependencies: []
-references:
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/query.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker-service.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/__tests__/query.test.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker-service.test.ts
-priority: medium
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Episode progress for streamed media can stay at `0%` because some remote sessions persist `ended_media_ms = 0` even when subtitle timing and watch activity clearly advanced, and the anime episode query currently treats `0` as a valid progress checkpoint.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Anime episode progress ignores zero-valued session checkpoints and falls back to subtitle/event timing
-- [x] #2 New streamed sessions persist meaningful progress even when playback-position updates are missing or sparse
-- [x] #3 Regression tests cover the zero-checkpoint remote-session case
-<!-- AC:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Restored anime episode progress handling for streamed sessions by ignoring zero-valued `ended_media_ms` checkpoints and falling back to subtitle/event timing, with regression coverage for the remote-session zero-checkpoint case.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-220 - Restore-YouTube-overlay-mpv-keybindings-after-picker-routing.md

@@ -1,66 +0,0 @@
----
-id: TASK-220
-title: Restore YouTube overlay mpv keybindings after picker routing
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-22 00:00'
-updated_date: '2026-03-22 23:49'
-labels:
-  - bug
-  - overlay
-  - youtube
-  - keyboard
-dependencies: []
-references:
-  - src/renderer/handlers/keyboard.ts
-  - src/renderer/modals/youtube-track-picker.ts
-  - src/renderer/handlers/keyboard.test.ts
-  - src/renderer/modals/youtube-track-picker.test.ts
-documentation:
-  - docs/workflow/verification.md
-priority: high
-ordinal: 118800
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Regression: after adding the YouTube subtitle picker modal path, visible-overlay keydown handling can stop before reaching the shared mpv keybinding dispatch path. Result: default overlay mpv bindings like `Space` pause/play and `q` quit stop working while the overlay owns focus during YouTube playback.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Unhandled keys while the YouTube track picker state is active still fall through to the shared overlay mpv keybinding dispatcher.
-- [x] #2 The YouTube picker continues to consume `Enter` and `Escape` for its own actions.
-- [x] #3 Renderer regression tests cover both the picker modal key contract and the shared keyboard dispatch fallback.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Add a failing renderer keyboard regression test covering YouTube picker state plus shared mpv keybinding fallback.
-2. Update the global keyboard handler to return early only when the YouTube picker actually handles the key event.
-3. Update the picker modal handler to return false for unhandled keys while preserving `Enter`/`Escape`.
-4. Run the cheap renderer verification lane and record results.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Fixed the regression by making the global renderer keyboard handler stop early for the YouTube picker only when the picker actually consumes the key. The picker modal now returns `false` for unrelated keys, so shared overlay mpv bindings like `Space` and `KeyQ` still dispatch while the visible overlay has focus.
-
-Added regression coverage in the keyboard handler suite for mpv keybinding fallback during YouTube picker state, plus a picker-modal contract test that keeps `Escape` handled but leaves unrelated keys unclaimed.
-
-Verification:
-- `bun test src/renderer/handlers/keyboard.test.ts src/renderer/modals/youtube-track-picker.test.ts`
-- `bash .agents/skills/subminer-change-verification/scripts/classify_subminer_diff.sh src/renderer/handlers/keyboard.ts src/renderer/handlers/keyboard.test.ts src/renderer/modals/youtube-track-picker.ts src/renderer/modals/youtube-track-picker.test.ts`
-- `bash .agents/skills/subminer-change-verification/scripts/verify_subminer_change.sh --lane core src/renderer/handlers/keyboard.ts src/renderer/handlers/keyboard.test.ts src/renderer/modals/youtube-track-picker.ts src/renderer/modals/youtube-track-picker.test.ts`
-- verifier artifact: `.tmp/skill-verification/subminer-verify-20260322-234831-b2m6nJ`
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Restored YouTube-session overlay mpv keybindings by removing an unconditional early return added to the renderer keyboard path for the YouTube subtitle picker modal. Unhandled keys now fall through to the shared mpv keybinding dispatcher, while handled picker keys (`Enter`, `Escape`) still stay local to the picker. Added renderer regression tests for both the keyboard fallback path and the picker modal key-consumption contract.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-221 - Assess-and-address-PR-31-latest-CodeRabbit-review.md

@@ -1,58 +0,0 @@
----
-id: TASK-221
-title: 'Assess and address PR #31 latest CodeRabbit review'
-status: Done
-assignee: []
-created_date: '2026-03-23 07:53'
-updated_date: '2026-03-24 06:41'
-labels:
-  - pr-review
-  - coderabbit
-dependencies: []
-references:
-  - >-
-    PR #31 feat: add app-owned YouTube subtitle flow with absPlayer-style
-    parsing
-priority: medium
-ordinal: 152500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Inspect the latest CodeRabbit review on PR #31, evaluate each actionable comment against the current branch, implement valid fixes, verify the changes, and prepare PR thread updates.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Inspect latest CodeRabbit review on PR #31 and separate valid action items from non-blocking suggestions.
-2. Add regression coverage for any real bugs before changing production code.
-3. Implement the minimal fixes for confirmed issues in runtime, renderer modal flow, and test fixtures.
-4. Run targeted tests plus repo-native verification lanes.
-5. Update PR threads with fix status and rationale for any comments not actioned yet.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Implemented and pushed commit 207151db to PR #31.
-
-Replied in-thread to the CodeRabbit comments for YouTube host matching, duplicate picker submissions, and missing MediaDetailView test fixture videoId fields.
-
-Follow-up scope added: update release-facing docs/changelog for the YouTube subtitle picker work and run a release-readiness gate before handoff.
-
-Added release-facing docs/changelog updates in commit b7e0026d and pushed them to PR #31.
-
-Ran the release-readiness gate: changelog:lint, changelog:pr-check, verify:config-example, typecheck, test:fast, test:env, build, test:smoke:dist, docs:test, docs:build.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Assessed the latest CodeRabbit review on PR #31 and applied the confirmed fixes. Tightened `isYoutubeMediaPath()` to match only exact YouTube hosts or subdomains with a regression test for `notyoutube.com`, added an in-flight guard plus temporary control disabling to the YouTube track picker with a duplicate-submit regression test, replaced the picker empty-state `innerHTML` fallback with explicit DOM construction, and added the missing `videoId` fields to the `MediaDetailView` test fixtures. Verified with targeted Bun tests and the `runtime-compat` verification lane (`build`, `test:runtime:compat`, `test:smoke:dist`).
-
-Updated `README.md`, `docs-site/usage.md`, and `changes/2026-03-23-immersion-youtube.md` so the PR is release-facing and user-visible surfaces describe the YouTube subtitle picker flow plus its latest hardening.
-
-Release-readiness checks passed locally: `bun run changelog:lint`, `bun run changelog:pr-check`, `bun run verify:config-example`, `bun run typecheck`, `bun run test:fast`, `bun run test:env`, `bun run build`, `bun run test:smoke:dist`, `bun run docs:test`, and `bun run docs:build`.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-222 - Fix-YouTube-overlay-keybindings-in-subtitle-path.md

@@ -1,56 +0,0 @@
----
-id: TASK-222
-title: Fix YouTube overlay keybindings in subtitle path
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-23 08:32'
-updated_date: '2026-03-24 06:41'
-labels:
-  - bug
-dependencies: []
-references:
-  - /Users/sudacode/projects/japanese/SubMiner/src/main/runtime
-  - /Users/sudacode/projects/japanese/SubMiner/src/core/services
-priority: high
-ordinal: 151500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Users watching video through the YouTube subtitle path cannot use some overlay keyboard controls such as quit and pause/play. Restore expected overlay keybinding behavior for that playback path without regressing other overlay input handling.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Overlay quit and pause/play keybindings work while using the YouTube subtitle path.
-- [x] #2 Existing overlay keybinding behavior for non-YouTube playback remains unchanged.
-- [x] #3 Regression coverage exercises the YouTube subtitle path keyboard handling.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Add a regression test around YouTube track-picker close to verify it requests main-process main-window focus restoration before returning overlay focus locally.
-2. Update the YouTube track-picker close flow to call `window.electronAPI.focusMainWindow()` alongside the existing `window.focus()` and `overlay.focus()` restoration.
-3. Run targeted tests for the picker/keyboard paths to verify YouTube playback regains overlay keybindings without regressing existing overlay behavior.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Investigated overlay input path. Renderer already maps Space/KeyQ to mpv commands, but YouTube track-picker close only restores DOM focus (`window.focus` + `overlay.focus`) and does not invoke main-process window focus recovery, unlike the keyboard-mode focus reclaim path. Suspected root cause: overlay BrowserWindow focus is not restored after the YouTube picker closes, so playback keybindings stop reaching renderer keydown handlers.
-
-User approved implementation plan on 2026-03-23. Proceeding with TDD: add failing regression first, then minimal fix, then targeted verification.
-
-Implemented fix in the YouTube track-picker close path: request main-process `focusMainWindow()` before restoring renderer window/overlay focus so overlay keydown handlers regain input after YouTube subtitle selection.
-
-Verification: `bun test src/renderer/modals/youtube-track-picker.test.ts` and `bun test src/renderer/handlers/keyboard.test.ts` both pass.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Restored overlay keyboard focus after closing the YouTube subtitle picker by invoking the main-process `focusMainWindow()` recovery path before local window/overlay focus restoration. Added regression coverage to the YouTube picker modal test and verified existing keyboard handler coverage for YouTube picker passthrough keys (`Space`, `KeyQ`) remains green.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-223 - Fix-YouTube-overlay-Anki-initialization-regression.md

@@ -1,109 +0,0 @@
----
-id: TASK-223
-title: Fix YouTube overlay Anki initialization regression
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-23 08:41'
-updated_date: '2026-03-24 06:41'
-labels:
-  - bug
-  - youtube
-  - anki
-dependencies: []
-references:
-  - /Users/sudacode/projects/japanese/SubMiner/src/main.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/overlay-runtime-init.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/main/runtime/cli-command-runtime-handler.ts
-documentation:
-  - /Users/sudacode/projects/japanese/SubMiner/docs/workflow/verification.md
-priority: high
-ordinal: 154500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Restore Anki-backed lookup and known-word behavior during YouTube playback. Recent startup changes appear to let the YouTube flow initialize the overlay before runtime prerequisites exist, leaving the Anki integration unavailable for popup Mine actions and known-word highlighting.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 YouTube playback initializes Anki integration once overlay startup prerequisites are available so lookup can offer card-add actions again
-- [x] #2 Known-word / N+1 state is available during YouTube playback when the user has Anki-backed known-word highlighting enabled
-- [x] #3 Regression coverage fails before the fix and passes after it for the YouTube startup path
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Add a regression test covering the YouTube playback command path and assert overlay startup prerequisites are established before the flow runs.
-2. Reuse the overlay startup prerequisite bootstrap for the YouTube playback path so Anki integration sees subtitle tracker, mpv client, and runtime options manager before initialization.
-3. Verify with focused runtime/CLI tests, then run the cheapest sufficient verification lane for the touched files.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Identified regression path in CLI command runtime: YouTube playback commands could reach overlay initialization without first materializing overlay startup prerequisites, leaving Anki integration unavailable during the initial startup attempt.
-
-Added a regression test at src/main/runtime/cli-command-runtime-handler.test.ts covering youtubePlay command dispatch outside texthooker-only mode.
-
-Verified with bun test src/main/runtime/cli-command-runtime-handler.test.ts src/main/runtime/cli-command-prechecks.test.ts src/main/runtime/cli-command-prechecks-main-deps.test.ts and bun test src/core/services/cli-command.test.ts src/main/runtime/cli-command-context-main-deps.test.ts src/main/runtime/cli-command-prechecks-main-deps.test.ts src/main/runtime/cli-command-runtime-handler.test.ts.
-
-Removed the YouTube-only ensureOverlayRuntimeReady path from src/main.ts after confirming regular app startup already loads Yomitan and the shared CLI overlay pre-dispatch bootstrap now covers overlay prerequisites.
-
-Moved overlay bootstrap into the generic initial-args startup path for any initial command that needs overlay runtime, so overlay prerequisites and overlay initialization happen before CLI dispatch instead of inside a YouTube-only or last-moment command path.
-
-Additional verification passed: bun test src/main/runtime/initial-args-runtime-handler.test.ts src/main/runtime/initial-args-handler.test.ts src/main/runtime/initial-args-main-deps.test.ts, bun run typecheck, bun run test:runtime:compat
-
-Follow-up regression: subtitle picker was still auto-submitting the default selection during YouTube startup. Investigating renderer-side immediate Enter key bleed-through on picker open.
-
-Root cause for remaining picker regression: the YouTube track picker accepted Enter immediately on open, so the launch keypress could auto-submit the default track selection before the modal was visible to the user.
-
-Added renderer regression coverage in src/renderer/modals/youtube-track-picker.test.ts proving immediate Enter after open is ignored and a later Enter still submits normally.
-
-Implemented a 200ms open-key guard in src/renderer/modals/youtube-track-picker.ts for Enter-based submission only; Escape/click behavior unchanged.
-
-New follow-up regression report: YouTube subtitle picker can open before the mpv playback window is ready, leaving the picker behind the overlay after geometry snaps into place. Investigating picker-open gating and modal-targeting timing.
-
-Identified likely cause of picker-behind-overlay regression: YouTube picker open logic mixed overlay targets. First attempt preferred the visible main overlay, timeout retry switched to the dedicated modal window, allowing a late first open to cover the modal.
-
-Extracted picker-open policy into src/main/runtime/youtube-picker-open.ts and changed YouTube picker startup to always target the dedicated modal window, including retries. This keeps the picker on a single window path and lets overlay-runtime hide/click-through the main overlay while the modal is active.
-
-Added regression tests in src/main/runtime/youtube-picker-open.test.ts covering dedicated modal first-open, dedicated-modal retry, and failure when no modal target is available.
-
-User reports overlay flow still feels wrong: YouTube path appears to preload subtitles before mandatory selection and may open the picker before mpv window readiness. Re-evaluating flow design against regular video startup before further implementation.
-
-New follow-up regression report: duplicate overlay windows appear during YouTube playback and only one window shows subtitles. Investigating main-overlay versus dedicated modal-window handoff/cleanup.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Fixed the YouTube Anki initialization regression by making CLI commands that require overlay runtime bootstrap overlay startup prerequisites before command dispatch when not in texthooker-only mode. This ensures the YouTube playback flow has the mpv client, runtime options manager, and subtitle timing tracker ready before overlay/Anki initialization runs, restoring Mine actions and known-word-backed behavior.
-
-Added a regression test covering youtubePlay command dispatch in src/main/runtime/cli-command-runtime-handler.test.ts.
-
-Verification:
-- bun test src/main/runtime/cli-command-runtime-handler.test.ts src/main/runtime/cli-command-prechecks.test.ts src/main/runtime/cli-command-prechecks-main-deps.test.ts
-- bun test src/core/services/cli-command.test.ts src/main/runtime/cli-command-context-main-deps.test.ts src/main/runtime/cli-command-prechecks-main-deps.test.ts src/main/runtime/cli-command-runtime-handler.test.ts
-
-Updated the fix to avoid a YouTube-specific startup path: removed the dedicated ensureOverlayRuntimeReady helper from src/main.ts and relied on the shared CLI overlay prerequisite bootstrap instead.
-
-Additional verification: bun run typecheck
-
-Follow-up adjustment: initial overlay-runtime commands now bootstrap overlay prerequisites and initialize overlay during the shared initial-args startup path, rather than waiting for command dispatch. This keeps YouTube on the regular startup path while preserving earlier overlay availability.
-
-Additional verification: bun test src/main/runtime/initial-args-runtime-handler.test.ts src/main/runtime/initial-args-handler.test.ts src/main/runtime/initial-args-main-deps.test.ts; bun run test:runtime:compat
-
-Follow-up fix: the YouTube subtitle picker now ignores immediate Enter key bleed-through right after opening, preventing the startup keypress from auto-submitting the default track selection before the modal is visible.
-
-Added renderer regression coverage for immediate Enter suppression and verified with bun test src/renderer/modals/youtube-track-picker.test.ts plus the runtime-compat verification lane for the touched files.
-
-Follow-up fix: YouTube subtitle picker startup now uses a dedicated modal-window path consistently instead of mixing main-overlay first-open with modal-window retry. That prevents late overlay opens from covering the interactive picker while mpv/window tracking settles.
-
-Verified with bun test src/main/runtime/youtube-picker-open.test.ts, bun test src/renderer/modals/youtube-track-picker.test.ts, and the runtime-compat verification lane for src/main.ts plus the touched picker files.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-224 - Auto-load-default-YouTube-subtitles-at-playback-start-and-make-picker-manual-only.md

@@ -1,61 +0,0 @@
----
-id: TASK-224
-title: >-
-  Auto-load default YouTube subtitles at playback start and make picker
-  manual-only
-status: Done
-assignee:
-  - Codex
-created_date: '2026-03-23 18:51'
-updated_date: '2026-03-24 06:41'
-labels:
-  - youtube
-  - mpv
-  - overlay
-  - keybindings
-dependencies: []
-references:
-  - /Users/sudacode/projects/japanese/SubMiner/src/main/runtime/youtube-flow.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/youtube-track-picker.ts
-  - /Users/sudacode/projects/japanese/SubMiner/src/config/definitions/shared.ts
-priority: high
-ordinal: 150500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Replace the mandatory YouTube subtitle picker startup flow with automatic default-track loading. On YouTube playback start, attempt to load the default primary subtitle and best-effort secondary subtitle without prompting. Gate playback only on primary subtitle load/tokenization readiness. If primary subtitle probing/download/loading fails, resume playback and report the failure through the configured notification/output path. Keep the YouTube subtitle picker as a regular overlay modal opened by a new default keybinding during active YouTube playback.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Opening a YouTube URL auto-selects and attempts to load the default primary subtitle without opening the picker modal.
-- [x] #2 Opening a YouTube URL also attempts to load the default secondary subtitle when available, but playback never waits on secondary success.
-- [x] #3 Playback remains gated only until the primary subtitle is loaded and tokenization is ready; primary failure resumes playback immediately.
-- [x] #4 Primary auto-load failures report through the existing configured notification/output path and keep playback running.
-- [x] #5 The YouTube subtitle picker can be opened manually during active YouTube playback via a new default keybinding.
-- [x] #6 Regression tests cover startup auto-load success, primary failure fallback, and the manual picker keybinding flow.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Add failing tests for YouTube startup auto-load success, primary failure fallback, and manual picker keybinding flow.
-2. Refactor the YouTube runtime to auto-select default tracks on startup, gate playback only on primary subtitle/tokenization readiness, and route failures through the configured notification/output path.
-3. Add a new default keybinding and command path to open the YouTube picker manually during active YouTube playback.
-4. Run targeted tests, then SubMiner verification lanes for launcher/runtime changes; update docs/changelog if required by the final behavior change.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Verification blocker outside this change: `bun run test:fast` still fails at `scripts/update-aur-package.test.ts` on macOS because `scripts/update-aur-package.sh` uses `mapfile`, which is unavailable in the system Bash 3.x environment used here.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Reworked app-owned YouTube playback to auto-load the default primary subtitle plus a best-effort secondary subtitle at startup instead of forcing the picker modal first. Playback now waits only on primary subtitle load/tokenization readiness, routes startup primary-failure messaging through the configured notification output path, and keeps the YouTube subtitle picker available on demand via a new default `Ctrl+Shift+J` keybinding during active YouTube playback. Updated the runtime/IPC/config plumbing, user-facing help/docs, and added regression coverage for startup auto-load, primary-failure fallback, and manual picker invocation.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-225 - Fix-frozen-primary-YouTube-subtitle-display-after-auto-load-startup.md

@@ -1,41 +0,0 @@
----
-id: TASK-225
-title: Fix frozen primary YouTube subtitle display after auto-load startup
-status: Done
-assignee: []
-created_date: '2026-03-23 20:07'
-updated_date: '2026-03-24 06:41'
-labels:
-  - bug
-  - youtube
-  - subtitles
-dependencies:
-  - TASK-224
-priority: high
-ordinal: 149500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-After the new YouTube auto-load startup flow, the primary subtitle overlay can stay stuck on an older line while the subtitle sidebar continues advancing. Investigate startup suppression / subtitle refresh timing and restore live primary overlay updates after auto-loaded subtitles are injected.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 When YouTube auto-load succeeds, the visible primary subtitle continues advancing after playback resumes.
-- [x] #2 Startup suppression does not leave the primary subtitle display stuck on a stale line.
-- [x] #3 A regression test covers the startup path that previously froze the visible primary subtitle while sidebar timing continued advancing.
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Root cause: applyStartupState seeded youtubePlaybackFlowPending from initialArgs.youtubePlay, and runYoutubePlaybackFlowMain restored that preexisting true value after startup auto-load. Result: primary subtitle events stayed suppressed for startup-launched YouTube playback while sidebar timing still advanced.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Stopped pre-seeding youtubePlaybackFlowPending from startup CLI args so only the actual YouTube playback bootstrap window suppresses subtitle events. Added a regression test covering startup YouTube args and re-ran targeted YouTube/runtime subtitle tests plus typecheck.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-226 - Restore-subtitle-sidebar-cues-for-auto-loaded-YouTube-subtitles.md

@@ -1,42 +0,0 @@
----
-id: TASK-226
-title: Restore subtitle sidebar cues for auto-loaded YouTube subtitles
-status: Done
-assignee: []
-created_date: '2026-03-23 20:21'
-updated_date: '2026-03-24 06:41'
-labels:
-  - bug
-  - youtube
-  - subtitle-sidebar
-dependencies:
-  - TASK-224
-  - TASK-225
-priority: high
-ordinal: 148500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-After fixing startup subtitle event suppression, the primary subtitle overlay updates for auto-loaded YouTube playback but the subtitle sidebar reports no parsed subtitle cues available. Investigate parsed subtitle source registration / refresh for auto-loaded YouTube subtitle files and restore sidebar cue population.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 When YouTube auto-load succeeds, the subtitle sidebar receives parsed cues for the active primary subtitle source.
-- [x] #2 Auto-loaded YouTube subtitle source changes refresh the sidebar snapshot without requiring manual picker interaction.
-- [x] #3 A regression test covers the startup auto-load path where live primary subtitles render but sidebar cues remain empty.
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Root cause: successful YouTube auto-load refreshed visible primary subtitle state, but did not explicitly initialize parsed subtitle cues from the resolved downloaded primary subtitle file. Sidebar cue population depended on later mpv source rediscovery, which could leave snapshots empty.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Added a regression test for YouTube auto-load sidebar cue refresh and wired the YouTube subtitle flow to explicitly refresh parsed subtitle cues from the resolved primary subtitle path after a successful load. Verified with targeted YouTube/sidebar/runtime tests plus typecheck.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-227 - Assess-and-address-PR-31-latest-CodeRabbit-review-round.md

@@ -1,65 +0,0 @@
----
-id: TASK-227
-title: 'Assess and address PR #31 latest CodeRabbit review round'
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-24 03:53'
-updated_date: '2026-03-24 06:41'
-labels:
-  - pr-review
-  - coderabbit
-dependencies: []
-references:
-  - >-
-    PR #31 feat: add app-owned YouTube subtitle flow with absPlayer-style
-    parsing
-priority: medium
-ordinal: 147500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Inspect the latest CodeRabbit review round on PR #31, verify each actionable comment against the current branch, implement only the valid fixes, add regression coverage where appropriate, and prepare thread replies for resolved or declined items.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Latest CodeRabbit comments on PR #31 are triaged into valid fixes vs non-actioned suggestions with rationale.
-- [x] #2 Confirmed issues are fixed with regression coverage where appropriate.
-- [x] #3 Relevant verification passes for the touched areas.
-- [x] #4 PR reply notes are ready for each addressed or declined latest-review comment.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Verify the five latest CodeRabbit inline comments against the current branch and separate valid bugs from non-actioned suggestions.
-2. Add failing regression coverage for confirmed issues in launcher playback tests, CLI YouTube flow error handling, and renderer YouTube picker disabled-state behavior.
-3. Implement the minimal production fixes for the confirmed issues, plus remove the duplicate overlay Anki initialization if still redundant.
-4. Inspect the YouTube primary-subtitle failure timer wiring to decide whether a code change is warranted in this round or whether a technical reply declining the comment is more correct.
-5. Run targeted Bun tests for the touched files and prepare concise PR thread replies for each latest-review comment.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Triaged the latest PR #31 CodeRabbit round: five inline comments were current action items; implemented all five. Strengthened the launcher playback test fixture so YouTube pause coverage no longer piggybacks on generic overlay auto-pause settings.
-
-Added regression tests for CLI YouTube flow rejection handling, no-track picker disabled-state restoration, and app-owned YouTube notification suppression while subtitle acquisition is still in flight.
-
-Implemented `runAsyncWithOsd(...)` handling for `args.youtubePlay`, kept no-track picker controls disabled after failed continue attempts, added `setAppOwnedFlowInFlight(...)` to the YouTube primary-subtitle notification runtime with main-process wiring around `runYoutubePlaybackFlowMain(...)`, and removed the duplicate `initializeOverlayAnkiIntegrationCore(...)` call from `initializeOverlayRuntime()`.
-
-Verification passed: `bun test launcher/commands/playback-command.test.ts src/core/services/cli-command.test.ts src/renderer/modals/youtube-track-picker.test.ts src/main/runtime/youtube-primary-subtitle-notification.test.ts` and `bun run typecheck`.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Assessed the latest CodeRabbit review round on PR #31 and implemented all five current inline action items. Strengthened the launcher playback regression test so app-owned YouTube pause behavior is asserted independently from generic overlay auto-pause settings, wrapped the CLI `youtubePlay` branch in the existing `runAsyncWithOsd(...)` path so probe/download/startup failures surface in logs and OSD, kept the no-track YouTube picker controls disabled after rejected continue attempts, suppressed the generic primary-subtitle failure timer while the app-owned YouTube flow is still probing/downloading and restarted it only after the flow settles, and removed the duplicate overlay Anki initialization from `initializeOverlayRuntime()`.
-
-Verification passed with `bun test launcher/commands/playback-command.test.ts src/core/services/cli-command.test.ts src/renderer/modals/youtube-track-picker.test.ts src/main/runtime/youtube-primary-subtitle-notification.test.ts` and `bun run typecheck`.
-
-Prepared thread-reply notes for the five latest inline comments; did not post them because GitHub replies are an external side effect.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-228 - Assess-and-address-PR-31-subsequent-CodeRabbit-review-round.md

@@ -1,64 +0,0 @@
----
-id: TASK-228
-title: 'Assess and address PR #31 subsequent CodeRabbit review round'
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-24 04:10'
-updated_date: '2026-03-24 06:41'
-labels:
-  - pr-review
-  - coderabbit
-dependencies: []
-references:
-  - >-
-    PR #31 feat: add app-owned YouTube subtitle flow with absPlayer-style
-    parsing
-  - 'commit cdb12827 fix: address PR #31 latest review follow-ups'
-priority: medium
-ordinal: 146500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Inspect the subsequent CodeRabbit review round on PR #31 after commit cdb12827, verify each newly reported issue against the current branch, implement the valid fixes with regression coverage where appropriate, and prepare/update PR thread replies.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 New CodeRabbit comments after cdb12827 are triaged into valid fixes vs declined suggestions with rationale.
-- [x] #2 Confirmed issues are fixed with regression coverage where appropriate.
-- [x] #3 Relevant verification passes for the touched areas.
-- [x] #4 PR threads are updated for the addressed comments.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Verify the new CodeRabbit comments after cdb12827 and separate valid bugs from refactor-only suggestions.
-2. Add failing regression coverage for the valid runtime issues: `track.selected` fallback in the YouTube primary-subtitle notifier and consistent no-track handling in the picker.
-3. Inspect existing test seams for the `main.ts` flow-entry guards; if lightweight coverage exists, add it before patching. Otherwise apply the minimal `main.ts` fixes and rely on typecheck plus targeted regression tests around the affected runtime helpers.
-4. Implement the confirmed fixes: picker re-entry guard, broader `inFlight` cleanup, `track.selected` fallback, and a single canonical `hasTracks` check.
-5. Run targeted tests/typecheck and update the new PR threads with landed fix refs.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Triaged the post-cdb12827 CodeRabbit round. Implemented the 4 concrete follow-ups: manual picker re-entry guard, broader `setAppOwnedFlowInFlight(...)` cleanup, `track.selected` fallback in the YouTube primary-subtitle notifier, and a single canonical `payloadHasTracks(...)` helper in the picker. Also took the adjacent `replaceChildren()` cleanup while touching the same picker paths.
-
-Verification passed: `bun test src/main/runtime/youtube-primary-subtitle-notification.test.ts src/renderer/modals/youtube-track-picker.test.ts launcher/commands/playback-command.test.ts src/core/services/cli-command.test.ts` and `bun run typecheck`.
-
-Updated the new CodeRabbit inline threads with landed fix refs and left a top-level PR comment noting the large refactor suggestions are intentionally out of scope for this bugfix round.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Assessed the subsequent CodeRabbit review round on PR #31 after cdb12827 and applied the valid follow-ups in commit 5f6f93cd. Added a guard in `openYoutubeTrackPickerFromPlayback()` so the manual picker cannot re-enter while another YouTube flow session is active, widened the app-owned in-flight suppression to cover synchronous Windows mpv bootstrap and connect failures, taught the primary-subtitle notifier to honor `track.selected` before `sid` arrives, and unified the picker’s subtitle-availability logic behind `payloadHasTracks(...)` while swapping node clearing to `replaceChildren()`.
-
-Verification passed with `bun test src/main/runtime/youtube-primary-subtitle-notification.test.ts src/renderer/modals/youtube-track-picker.test.ts launcher/commands/playback-command.test.ts src/core/services/cli-command.test.ts` and `bun run typecheck`.
-
-Updated the latest inline CodeRabbit threads plus a top-level PR comment summarizing the round and explicitly deferred the large refactor suggestions as non-blocking maintainability nits.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-229 - Address-PR-31-final-CodeRabbit-picker-test-follow-up.md

@@ -1,55 +0,0 @@
----
-id: TASK-229
-title: 'Address PR #31 final CodeRabbit picker test follow-up'
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-24 04:27'
-updated_date: '2026-03-24 06:41'
-labels:
-  - pr-review
-  - coderabbit
-dependencies: []
-references:
-  - >-
-    PR #31 feat: add app-owned YouTube subtitle flow with absPlayer-style
-    parsing
-  - >-
-    CodeRabbit comment on src/renderer/modals/youtube-track-picker.test.ts
-    global restoration / harness duplication
-priority: medium
-ordinal: 145500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Fix the remaining CodeRabbit comment on the YouTube picker test file by restoring absent globals correctly and reducing repeated test harness setup so global stubbing is consistent and isolated.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Picker tests restore `window`, `document`, and `CustomEvent` without leaving undefined-valued globals behind.
-- [x] #2 Repeated picker test setup is consolidated enough to remove the current review complaint.
-- [x] #3 Relevant picker tests pass and PR thread is updated.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Add a failing regression around global restoration semantics in the YouTube picker test harness.
-2. Extract shared DOM/environment helpers and restore logic using delete when globals were originally absent.
-3. Re-run focused tests and typecheck, then commit/push and reply on the PR thread.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Latest CodeRabbit comment targets youtube-track-picker.test.ts harness cleanup and correct restoration of global properties.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Addressed the last PR #31 CodeRabbit comment by refactoring the YouTube picker test harness to use shared DOM/env helpers, restoring absent globals via delete semantics, adding a regression for cleanup behavior, and pushing commit 039e2f56 with focused picker tests plus typecheck passing.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-231 - Restore-controller-input-while-subtitle-sidebar-is-open.md

@@ -1,57 +0,0 @@
----
-id: TASK-231
-title: Restore controller input while subtitle sidebar is open
-status: Done
-assignee:
-  - '@codex'
-created_date: '2026-03-24 00:15'
-updated_date: '2026-03-24 00:15'
-labels:
-  - bug
-  - controller
-  - subtitle-sidebar
-  - overlay
-dependencies: []
-references:
-  - /home/sudacode/projects/japanese/SubMiner/src/renderer/renderer.ts
-  - /home/sudacode/projects/japanese/SubMiner/src/renderer/controller-interaction-blocking.ts
-  - /home/sudacode/projects/japanese/SubMiner/src/renderer/controller-interaction-blocking.test.ts
-priority: high
-ordinal: 54900
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-
-When keyboard-only mode is active, opening the subtitle sidebar should not disable controller navigation and lookup/mining controls. Restore controller input while the sidebar is open, while keeping true modal dialogs blocking controller actions.
-
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-
-<!-- AC:BEGIN -->
-
-- [x] #1 Opening the subtitle sidebar does not block controller input for keyboard-only mode actions.
-- [x] #2 Controller-select/debug and other true modal dialogs still block controller actions while open.
-- [x] #3 Focused regression coverage exists for the sidebar-open controller gating rule.
-
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-
-Root cause: renderer gamepad polling used the broad `isAnyModalOpen()` check as its interaction gate, and that list includes `subtitleSidebarModalOpen`. The subtitle sidebar is non-modal for controller usage, so gamepad input was being suppressed whenever the sidebar was visible.
-
-Fixed by extracting a dedicated controller-interaction blocking helper that excludes the subtitle sidebar but keeps the existing blocking behavior for true modal dialogs.
-
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-
-Restored controller input while the subtitle sidebar is open by switching gamepad polling to a dedicated modal-blocking rule that leaves the sidebar controller-passive. Added a regression test covering the sidebar-open exception and preserving hard blocks for actual modal dialogs.
-
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-232 - Trim-release-package-size-by-pruning-duplicate-and-source-only-assets.md

@@ -1,66 +0,0 @@
----
-id: TASK-232
-title: Trim release package size by pruning duplicate and source-only assets
-status: Done
-assignee:
-  - '@codex'
-created_date: '2026-03-24 12:05'
-updated_date: '2026-03-24 12:30'
-labels:
-  - release
-  - packaging
-priority: medium
-ordinal: 54700
-dependencies: []
-references:
-  - /home/sudacode/projects/japanese/SubMiner/package.json
-  - /home/sudacode/projects/japanese/SubMiner/src/release-workflow.test.ts
-  - /home/sudacode/projects/japanese/SubMiner/src/core/services/texthooker.ts
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-
-Reduce packaged release artifact size without changing user-visible functionality by pruning files that are duplicated between `app.asar` and `extraResources`, excluding source/test/doc-only trees from Electron packaging, and trimming obviously non-runtime vendored payload.
-
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-
-<!-- AC:BEGIN -->
-
-- [x] #1 Electron packaging excludes repo content that is source-only, test-only, docs-only, or duplicated by `extraResources`.
-- [x] #2 Release packaging tests cover the new exclusion rules.
-- [x] #3 Verification includes at least targeted release-packaging tests and one packaging-oriented validation step.
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-
-Completed scope:
-- Exclude `assets`, `plugin`, and `vendor/yomitan-jlpt-vocab` from `files` because they are already staged via `extraResources`.
-- Exclude `dist` sourcemaps/tests, repo docs/tests/packaging metadata, and stats source leftovers from `files`.
-- Exclude non-runtime `vendor/texthooker-ui` payload such as `public/`, `.vscode/`, and package metadata.
-- Exclude Linux musl libsql binary from packaged app payload for AppImage-focused savings.
-
-Verification:
-- `bun test src/release-workflow.test.ts`
-- `bun run build`
-- `node_modules/.bin/electron-builder --linux dir --publish never`
-- `node_modules/.bin/electron-builder --linux AppImage --publish never`
-
-Observed result:
-- `release/linux-unpacked/resources/app.asar` dropped from about `100 MB` to `29 MB`.
-- `release/SubMiner-0.9.0.AppImage` dropped from about `256 MB` to `194 MB`.
-
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-
-Trimmed Electron packaging so release artifacts no longer bundle duplicated `extraResources`, source/test/doc-only repo content, non-runtime `texthooker-ui` files, or the Linux musl libsql binary. Added release-packaging regression coverage and verified the Linux package shrink with fresh local builds.
-
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-233 - Cut-patch-release-v0.9.1-for-package-size-pruning.md

@@ -1,69 +0,0 @@
----
-id: TASK-233
-title: Cut patch release v0.9.1 for package size pruning
-status: Done
-assignee:
-  - '@codex'
-created_date: '2026-03-24 12:40'
-updated_date: '2026-03-24 12:55'
-labels:
-  - release
-  - patch
-dependencies:
-  - TASK-232
-references:
-  - /home/sudacode/projects/japanese/SubMiner/package.json
-  - /home/sudacode/projects/japanese/SubMiner/CHANGELOG.md
-  - /home/sudacode/projects/japanese/SubMiner/release/release-notes.md
-  - /home/sudacode/projects/japanese/SubMiner/docs-site/changelog.md
-priority: high
-ordinal: 54800
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-
-Publish a patch release for the packaging-size cleanup by bumping the app version to `0.9.1`, generating committed release metadata, and keeping release-facing docs/changelog surfaces aligned.
-
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-
-<!-- AC:BEGIN -->
-
-- [x] #1 Repository version metadata is updated to `0.9.1`.
-- [x] #2 `CHANGELOG.md`, `release/release-notes.md`, and `docs-site/changelog.md` contain the committed `v0.9.1` release line and the consumed fragment is removed.
-- [x] #3 Release-readiness verification passes for changelog, docs, tests, and build lanes.
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-
-Completed:
-- Bumped `package.json` to `0.9.1`.
-- Ran `bun run changelog:build --version 0.9.1 --date 2026-03-24`, which generated `CHANGELOG.md` + `release/release-notes.md` and consumed both pending release fragments.
-- Synced `docs-site/changelog.md` with the generated `v0.9.1` release line.
-- Confirmed no additional README/docs wording changes were needed beyond changelog surfaces.
-
-Verification:
-- `bun run changelog:lint`
-- `bun run changelog:check --version 0.9.1`
-- `bun run verify:config-example`
-- `bun run typecheck`
-- `bun run test:fast`
-- `bun run test:env`
-- `bun run build`
-- `bun run docs:test`
-- `bun run docs:build`
-
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-
-Prepared patch release `v0.9.1` locally. Version metadata, committed changelog artifacts, release notes, and docs-site changelog are aligned, and the release gate is green. Pending manual release actions are the release-prep commit, `git tag v0.9.1`, and push/tag publication.
-
-<!-- SECTION:FINAL_SUMMARY:END -->

changes/2026-03-22-websocket-texthooker-docs.md

@@ -0,0 +1,5 @@
+type: docs
+area: docs
+
+- Added a new WebSocket / Texthooker API and integration guide covering websocket payloads, custom client patterns, mpv plugin automation, and webhook-style relay examples.
+- Linked the new integration guide from configuration and mining workflow docs for easier discovery.

changes/2026-03-23-immersion-youtube.md

@@ -0,0 +1,5 @@
+type: fixed
+area: immersion
+
+- Hardened immersion tracker storage/session/query paths with the updated YouTube metadata flow.
+- Added metadata probe support for YouTube subtitle retrieval edge cases.

config.example.jsonc

@@ -187,7 +187,7 @@
   // ==========================================
   // Secondary Subtitles
   // Dual subtitle track options.
-  // Used by the YouTube subtitle loading flow as secondary language preferences.
+  // Used by subminer YouTube subtitle generation as secondary language preferences.
   // Hot-reload: defaultMode updates live while SubMiner is running.
   // ==========================================
   "secondarySub": {
@@ -414,24 +414,24 @@
   }, // Jimaku API configuration and defaults.
 
   // ==========================================
-  // YouTube Playback Settings
-  // Defaults for SubMiner YouTube subtitle loading and languages.
+  // YouTube Subtitle Generation
+  // Defaults for SubMiner YouTube subtitle generation.
   // ==========================================
   "youtubeSubgen": {
-    "whisperBin": "", // Legacy compatibility path kept for external subtitle fallback tools; not used by default.
-    "whisperModel": "", // Legacy compatibility model path kept for external subtitle fallback tooling; not used by default.
-    "whisperVadModel": "", // Legacy compatibility VAD path kept for external subtitle fallback tooling; not used by default.
-    "whisperThreads": 4, // Legacy thread tuning for subtitle fallback tooling; not used by default.
-    "fixWithAi": false, // Legacy subtitle fallback post-processing switch kept for compatibility; use is currently disabled by default. Values: true | false
+    "whisperBin": "", // Path to whisper.cpp CLI used as fallback transcription engine.
+    "whisperModel": "", // Path to whisper model used for fallback transcription.
+    "whisperVadModel": "", // Path to optional whisper VAD model used for subtitle generation.
+    "whisperThreads": 4, // Thread count passed to whisper.cpp subtitle generation runs.
+    "fixWithAi": false, // Use shared AI provider to post-process whisper-generated YouTube subtitles. Values: true | false
     "ai": {
-      "model": "", // Optional model override for legacy subtitle fallback post-processing; not used by default.
-      "systemPrompt": "" // Optional system prompt override for legacy subtitle fallback post-processing; not used by default.
+      "model": "", // Optional model override for YouTube subtitle AI post-processing.
+      "systemPrompt": "" // Optional system prompt override for YouTube subtitle AI post-processing.
     }, // Ai setting.
     "primarySubLanguages": [
       "ja",
       "jpn"
     ] // Comma-separated primary subtitle language priority used by the launcher.
-  }, // Defaults for SubMiner YouTube subtitle loading and languages.
+  }, // Defaults for SubMiner YouTube subtitle generation.
 
   // ==========================================
   // Anilist

docs-site/changelog.md

@@ -1,23 +1,11 @@
 # Changelog
 
-## v0.9.1 (2026-03-24)
-- Reduced packaged release size by excluding duplicate `extraResources` payload and pruning docs, tests, sourcemaps, and other source-only files from Electron bundles.
-- Restored controller navigation and lookup/mining controls while the subtitle sidebar is open, while keeping true modal dialogs blocking controller actions.
-- Fixed subtitle annotation clearing so explanatory contrast endings like `んですけど` are excluded consistently across the shared tokenizer filter and annotation stage.
-
-## v0.9.0 (2026-03-23)
-- Added an app-owned YouTube subtitle flow with absPlayer-style timedtext parsing that auto-loads the default primary subtitle plus a best-effort secondary at startup and resumes once the primary is ready.
-- Added a manual YouTube subtitle picker on `Ctrl+Alt+C` so subtitle selection can be retried on demand during active YouTube playback.
-- Added yt-dlp metadata probing so YouTube playback and immersion tracking record canonical video title and channel metadata.
-- Disabled conflicting mpv native subtitle auto-selection for the app-owned flow so injected explicit tracks stay authoritative.
+## v0.9.0 (2026-03-22)
+- Added an app-owned YouTube subtitle picker flow that boots mpv paused, opens an overlay picker, and downloads selected subtitles into external files before playback resumes.
+- Added explicit launcher/app YouTube subtitle modes `download` and `generate`, with `download` as the default path.
+- Disabled mpv native YouTube subtitle auto-loading for the app-owned flow so injected external subtitle files stay authoritative.
 - Added OSD status updates covering YouTube playback startup, subtitle acquisition, and subtitle loading.
-- Stopped forcing `--ytdl-raw-options=` before user-provided mpv options so existing YouTube cookie integrations are preserved.
-- Improved sidebar startup/resume behavior, scroll handling, and overlay/sidebar subtitle synchronization.
-- Stats Library tab now shows YouTube video title, channel name, and thumbnail for YouTube media entries.
-- Added a new WebSocket / Texthooker API integration guide covering payload formats, custom client patterns, and mpv plugin automation.
-- Fixed Anki media mining for mpv YouTube streams so audio and screenshot capture work correctly during YouTube playback sessions.
-- Fixed YouTube media path handling in immersion tracking so YouTube sessions record correct media references and AniList state transitions do not fire for YouTube media.
-- Reused existing authoritative YouTube subtitle tracks when present, fell back only for missing sides, and kept native mpv secondary subtitle rendering hidden so the overlay remains the visible secondary subtitle surface.
+- Improved sidebar startup/resume behavior and overlay/sidebar subtitle synchronization.
 
 ## v0.8.0 (2026-03-22)
 - Added a configurable subtitle sidebar feature (`subtitleSidebar`) with overlay/embedded rendering, click-to-seek cue list, and hot-reloadable visibility and behavior controls.

docs-site/configuration.md

@@ -127,7 +127,7 @@ The configuration file includes several main sections:
 - [**Discord Rich Presence**](#discord-rich-presence) - Optional Discord activity card updates
 - [**Immersion Tracking**](#immersion-tracking) - Track subtitle sessions and mining activity in SQLite
 - [**Stats Dashboard**](#stats-dashboard) - Local dashboard and overlay for immersion progress
-- [**YouTube Playback Settings**](#youtube-playback-settings) - Defaults for YouTube subtitle loading
+- [**YouTube Subtitle Generation**](#youtube-subtitle-generation) - Launcher defaults for yt-dlp + local whisper fallback
 
 ## Core Settings
 
@@ -469,7 +469,6 @@ See `config.example.jsonc` for detailed configuration options and more examples.
 | `Space`              | `["cycle", "pause"]`         | Toggle pause                          |
 | `KeyJ`               | `["cycle", "sid"]`           | Cycle primary subtitle track          |
 | `Shift+KeyJ`         | `["cycle", "secondary-sid"]` | Cycle secondary subtitle track        |
-| `Ctrl+Alt+KeyC`    | `["__youtube-picker-open"]`     | Open the manual YouTube subtitle picker |
 | `ArrowRight`         | `["seek", 5]`                | Seek forward 5 seconds                |
 | `ArrowLeft`          | `["seek", -5]`               | Seek backward 5 seconds               |
 | `ArrowUp`            | `["seek", 60]`               | Seek forward 60 seconds               |
@@ -741,7 +740,7 @@ Palette controls:
 ### Shared AI Provider
 
 Shared OpenAI-compatible transport settings live at the top level under `ai`.
-Anki reads this provider directly. Legacy subtitle fallback keeps the same provider shape for compatibility, then applies feature-local overrides where supported.
+Anki and YouTube subtitle cleanup both read this provider, then apply feature-local overrides where supported.
 
 ```json
 {
@@ -765,10 +764,10 @@ Anki reads this provider directly. Legacy subtitle fallback keeps the same provi
 | `systemPrompt`     | string                | Optional system prompt override for shared provider workflows |
 | `requestTimeoutMs` | integer milliseconds  | Shared request timeout (default: `15000`)            |
 
-SubMiner uses the shared provider for:
+SubMiner uses the shared provider in two places:
 
 - Anki translation/enrichment when `ankiConnect.ai.enabled` is `true`
-- Legacy subtitle fallback compatibility when `youtubeSubgen.fixWithAi` is `true`
+- YouTube whisper subtitle post-processing when `youtubeSubgen.fixWithAi` is `true`
 
 ### AnkiConnect
 
@@ -1277,14 +1276,6 @@ Enable or disable local immersion analytics stored in SQLite for mined subtitles
 | `retention.monthlyRollupsDays` | integer (`0`-`36500`)         | Monthly rollup retention window. Default `0` (keep all).                                                    |
 | `retention.vacuumIntervalDays` | integer (`0`-`3650`)          | Minimum spacing between `VACUUM` passes. `0` disables vacuum. Default `0` (disabled).                         |
 
-You can also disable immersion tracking for a single session using:
-
-```bash
-SUBMINER_DISABLE_IMMERSION_TRACKING=1 subminer
-```
-
-When this is set, SubMiner skips immersion-tracker startup and does not initialize or read the immersion SQLite database for that session.
-
 Default behavior keeps raw events, telemetry, sessions, and rollups forever while still maintaining lifetime summary tables and daily/monthly rollups for faster reads. If you later want bounded retention, switch `retentionMode` or set explicit `retention.*` values.
 
 When `dbPath` is blank or omitted, SubMiner writes telemetry and session summaries to the default app-data location:
@@ -1326,13 +1317,22 @@ Usage notes:
 - The dashboard reads from the same immersion-tracking database, so keep `immersionTracking.enabled` on if you want data to appear.
 - The UI includes Overview, Library, Trends, Vocabulary, and Sessions tabs.
 
-### YouTube Playback Settings
+### YouTube Subtitle Generation
 
-Set defaults used by the `subminer` launcher for YouTube subtitle loading:
+Set defaults used by the `subminer` launcher for YouTube subtitle generation:
 
 ```json
 {
   "youtubeSubgen": {
+    "whisperBin": "/path/to/whisper-cli",
+    "whisperModel": "/path/to/ggml-model.bin",
+    "whisperVadModel": "/path/to/ggml-vad.bin",
+    "whisperThreads": 4,
+    "fixWithAi": false,
+    "ai": {
+      "model": "openai/gpt-4o-mini",
+      "systemPrompt": "Fix subtitle mistakes only."
+    },
     "primarySubLanguages": ["ja", "jpn"]
   }
 }
@@ -1340,22 +1340,27 @@ Set defaults used by the `subminer` launcher for YouTube subtitle loading:
 
 | Option                | Values               | Description                                                                                    |
 | --------------------- | -------------------- | ---------------------------------------------------------------------------------------------- |
-| `primarySubLanguages` | string[]             | Primary subtitle language priority for YouTube auto-loading (default `["ja", "jpn"]`) |
+| `whisperBin`          | string path          | Path to `whisper.cpp` CLI binary used as fallback transcription engine                        |
+| `whisperModel`        | string path          | Path to whisper model used by fallback transcription                                          |
+| `whisperVadModel`     | string path          | Optional whisper VAD model path                                                               |
+| `whisperThreads`      | integer              | Thread count passed to whisper runs                                                           |
+| `fixWithAi`           | `true`, `false`      | Run shared AI post-processing on whisper-generated subtitles                                  |
+| `ai.model`            | string               | Optional model override for YouTube AI subtitle cleanup                                       |
+| `ai.systemPrompt`     | string               | Optional system prompt override for YouTube AI subtitle cleanup                               |
+| `primarySubLanguages` | string[]             | Primary subtitle language priority for YouTube subtitle generation (default `["ja", "jpn"]`) |
 
-Current launcher behavior:
+Launcher behavior:
 
-- For YouTube URLs, SubMiner probes subtitle tracks with yt-dlp after mpv bootstrap and binds auto-selected tracks before normal playback resumes.
-- If YouTube/mpv already exposes an authoritative matching subtitle track, SubMiner reuses it; otherwise it downloads and injects only the missing side.
-- SubMiner loads the primary subtitle plus a best-effort secondary subtitle.
-- Playback waits only for primary subtitle readiness; secondary failures do not block playback.
-- English secondary subtitles are selected from `secondarySub.secondarySubLanguages` when primary language matches are unavailable.
-- Native mpv secondary subtitle rendering stays hidden during this flow so the SubMiner overlay remains the visible secondary subtitle surface.
-- If primary subtitle loading fails, use `Ctrl+Alt+C` to open the subtitle modal and pick a track.
+- For YouTube URLs, subtitle generation now runs before mpv launch.
+- SubMiner probes manual/native YouTube subtitle tracks first.
+- Missing tracks fall back to local `whisper.cpp`.
+- English secondary subtitles can use whisper translate fallback when no manual track exists.
+- If `fixWithAi` is enabled, only whisper-generated `.srt` output is post-processed with the shared top-level `ai` provider.
 
 Language targets are derived from subtitle config:
 
 - primary track: `youtubeSubgen.primarySubLanguages` (falls back to `["ja","jpn"]`)
 - secondary track: `secondarySub.secondarySubLanguages` (falls back to English when empty)
-- Tracks are resolved and loaded before mpv starts; the older launcher mode switch has been removed.
+- Subtitle files are generated or downloaded before mpv starts; the older launcher mode switch has been removed.
 
 Precedence for launcher defaults is: CLI flag > environment variable > `config.jsonc` > built-in default.

docs-site/development.md

@@ -231,13 +231,13 @@ Run `make help` for a full list of targets. Key ones:
 | `SUBMINER_ROFI_THEME`              | Override rofi theme path for launcher picker                                   |
 | `SUBMINER_LOG_LEVEL`               | Override app logger level (`debug`, `info`, `warn`, `error`)                   |
 | `SUBMINER_MPV_LOG`                 | Override mpv/app shared log file path                                          |
-| `SUBMINER_WHISPER_BIN`             | Override legacy `youtubeSubgen.whisperBin` fallback compatibility path          |
-| `SUBMINER_WHISPER_MODEL`           | Override legacy `youtubeSubgen.whisperModel` fallback compatibility path        |
-| `SUBMINER_WHISPER_VAD_MODEL`       | Override legacy `youtubeSubgen.whisperVadModel` fallback compatibility path      |
-| `SUBMINER_WHISPER_THREADS`         | Override legacy `youtubeSubgen.whisperThreads` fallback compatibility value      |
-| `SUBMINER_YT_SUBGEN_OUT_DIR`       | Override legacy fallback subtitle output directory                               |
-| `SUBMINER_YT_SUBGEN_AUDIO_FORMAT`  | Override extraction format used by legacy fallback subtitle path                  |
-| `SUBMINER_YT_SUBGEN_KEEP_TEMP`     | Set to `1` to keep legacy fallback subtitle workspace                          |
+| `SUBMINER_WHISPER_BIN`             | Override `youtubeSubgen.whisperBin` for launcher                               |
+| `SUBMINER_WHISPER_MODEL`           | Override `youtubeSubgen.whisperModel` for launcher                             |
+| `SUBMINER_WHISPER_VAD_MODEL`       | Override `youtubeSubgen.whisperVadModel` for launcher                          |
+| `SUBMINER_WHISPER_THREADS`         | Override `youtubeSubgen.whisperThreads` for launcher                           |
+| `SUBMINER_YT_SUBGEN_OUT_DIR`       | Override generated subtitle output directory                                   |
+| `SUBMINER_YT_SUBGEN_AUDIO_FORMAT`  | Override extraction format used for whisper fallback                           |
+| `SUBMINER_YT_SUBGEN_KEEP_TEMP`     | Set to `1` to keep temporary subtitle-generation workspace                     |
 | `SUBMINER_JIMAKU_API_KEY`          | Override Jimaku API key for launcher subtitle downloads                        |
 | `SUBMINER_JIMAKU_API_KEY_COMMAND`  | Command used to resolve Jimaku API key at runtime                              |
 | `SUBMINER_JIMAKU_API_BASE_URL`     | Override Jimaku API base URL                                                   |

docs-site/docs-sync.test.ts

@@ -20,7 +20,7 @@ function extractReleaseHeadings(content: string, count: number): string[] {
 test('docs reflect current launcher and release surfaces', () => {
   expect(usageContents).not.toContain('--mode preprocess');
   expect(usageContents).not.toContain('"automatic" (default)');
-  expect(usageContents).toContain('during startup while mpv is paused');
+  expect(usageContents).toContain('before mpv starts');
 
   expect(installationContents).toContain('bun run build:appimage');
   expect(installationContents).toContain('bun run build:win');

docs-site/index.md

@@ -51,8 +51,8 @@ features:
   - icon:
       src: /assets/video.svg
       alt: Video playback icon
-    title: YouTube Playback
-    details: Play YouTube URLs or ytsearch targets directly — SubMiner automatically selects and loads subtitles for the video.
+    title: YouTube & Whisper
+    details: Play YouTube URLs or searches with native subtitles, or generate them with whisper.cpp and optional AI cleanup.
     link: /usage#youtube-playback
     linkText: YouTube playback
   - icon:
@@ -72,10 +72,10 @@ features:
   - icon:
       src: /assets/tokenization.svg
       alt: Tracking chart icon
-    title: Stats Dashboard
-    details: Browse session history, streak calendars, vocabulary frequency, and per-series progress in a local dashboard — then mine cards straight from your viewing history.
+    title: Immersion Tracking
+    details: Logs watch time, words encountered, and cards mined to SQLite, then surfaces the same data in a local stats dashboard with rollups and session drill-down.
     link: /immersion-tracking
-    linkText: Dashboard & tracking
+    linkText: Stats details
   - icon:
       src: /assets/cross-platform.svg
       alt: Cross-platform icon
@@ -120,7 +120,7 @@ const demoAssetVersion = '20260223-2';
       <div class="workflow-step" style="animation-delay: 240ms">
         <div class="step-number">05</div>
         <div class="step-title">Track</div>
-        <div class="step-desc">Open the stats dashboard to review sessions, vocabulary trends, and mine cards from past viewing history.</div>
+        <div class="step-desc">Review immersion history and repeat high-value patterns over time.</div>
       </div>
     </div>
   </section>

docs-site/launcher-script.md

@@ -58,30 +58,24 @@ subminer --start video.mkv      # optional explicit overlay start when plugin au
 subminer -S video.mkv           # same as above via --start-overlay
 subminer https://youtu.be/...   # YouTube playback (requires yt-dlp)
 subminer ytsearch:"jp news"     # YouTube search
-subminer stats                  # open immersion dashboard
-subminer stats -b               # start background stats daemon
-subminer stats -s               # stop background stats daemon
 subminer --setup                # Open first-run setup popup
 ```
 
 ## Subcommands
 
-| Subcommand                   | Purpose                                                    |
-| ---------------------------- | ---------------------------------------------------------- |
-| `subminer jellyfin` / `jf`   | Jellyfin workflows (`-d` discovery, `-p` play, `-l` login) |
-| `subminer stats`             | Start stats server and open immersion dashboard in browser |
-| `subminer stats -b`          | Start or reuse background stats daemon (non-blocking)      |
-| `subminer stats -s`          | Stop the background stats daemon                           |
-| `subminer stats cleanup`     | Backfill vocabulary metadata and prune stale rows          |
-| `subminer doctor`            | Dependency + config + socket diagnostics                   |
-| `subminer config path`       | Print active config file path                              |
-| `subminer config show`       | Print active config contents                               |
-| `subminer mpv status`        | Check mpv socket readiness                                 |
-| `subminer mpv socket`        | Print active socket path                                   |
-| `subminer mpv idle`          | Launch detached idle mpv instance                          |
-| `subminer dictionary <path>` | Generate character dictionary ZIP from file/dir target     |
-| `subminer texthooker`        | Launch texthooker-only mode                                |
-| `subminer app`               | Pass arguments directly to SubMiner binary                 |
+| Subcommand                 | Purpose                                                    |
+| -------------------------- | ---------------------------------------------------------- |
+| `subminer jellyfin` / `jf` | Jellyfin workflows (`-d` discovery, `-p` play, `-l` login) |
+| `subminer yt` / `youtube`  | YouTube shorthand (`-o`, `-m`)                             |
+| `subminer doctor`          | Dependency + config + socket diagnostics                   |
+| `subminer config path`     | Print active config file path                              |
+| `subminer config show`     | Print active config contents                               |
+| `subminer mpv status`      | Check mpv socket readiness                                 |
+| `subminer mpv socket`      | Print active socket path                                   |
+| `subminer mpv idle`        | Launch detached idle mpv instance                          |
+| `subminer dictionary <path>` | Generate character dictionary ZIP from file/dir target   |
+| `subminer texthooker`      | Launch texthooker-only mode                                |
+| `subminer app`             | Pass arguments directly to SubMiner binary                 |
 
 Use `subminer <subcommand> -h` for command-specific help.
 

docs-site/public/config.example.jsonc

@@ -187,7 +187,7 @@
   // ==========================================
   // Secondary Subtitles
   // Dual subtitle track options.
-  // Used by the YouTube subtitle loading flow as secondary language preferences.
+  // Used by subminer YouTube subtitle generation as secondary language preferences.
   // Hot-reload: defaultMode updates live while SubMiner is running.
   // ==========================================
   "secondarySub": {
@@ -414,24 +414,24 @@
   }, // Jimaku API configuration and defaults.
 
   // ==========================================
-  // YouTube Playback Settings
-  // Defaults for SubMiner YouTube subtitle loading and languages.
+  // YouTube Subtitle Generation
+  // Defaults for SubMiner YouTube subtitle generation.
   // ==========================================
   "youtubeSubgen": {
-    "whisperBin": "", // Legacy compatibility path kept for external subtitle fallback tools; not used by default.
-    "whisperModel": "", // Legacy compatibility model path kept for external subtitle fallback tooling; not used by default.
-    "whisperVadModel": "", // Legacy compatibility VAD path kept for external subtitle fallback tooling; not used by default.
-    "whisperThreads": 4, // Legacy thread tuning for subtitle fallback tooling; not used by default.
-    "fixWithAi": false, // Legacy subtitle fallback post-processing switch kept for compatibility; use is currently disabled by default. Values: true | false
+    "whisperBin": "", // Path to whisper.cpp CLI used as fallback transcription engine.
+    "whisperModel": "", // Path to whisper model used for fallback transcription.
+    "whisperVadModel": "", // Path to optional whisper VAD model used for subtitle generation.
+    "whisperThreads": 4, // Thread count passed to whisper.cpp subtitle generation runs.
+    "fixWithAi": false, // Use shared AI provider to post-process whisper-generated YouTube subtitles. Values: true | false
     "ai": {
-      "model": "", // Optional model override for legacy subtitle fallback post-processing; not used by default.
-      "systemPrompt": "" // Optional system prompt override for legacy subtitle fallback post-processing; not used by default.
+      "model": "", // Optional model override for YouTube subtitle AI post-processing.
+      "systemPrompt": "" // Optional system prompt override for YouTube subtitle AI post-processing.
     }, // Ai setting.
     "primarySubLanguages": [
       "ja",
       "jpn"
     ] // Comma-separated primary subtitle language priority used by the launcher.
-  }, // Defaults for SubMiner YouTube subtitle loading and languages.
+  }, // Defaults for SubMiner YouTube subtitle generation.
 
   // ==========================================
   // Anilist

docs-site/shortcuts.md

@@ -67,7 +67,6 @@ Mouse-hover playback behavior is configured separately from shortcuts: `subtitle
 | `Ctrl/Cmd+Shift+V` | Cycle secondary subtitle mode (hidden → visible → hover) | `shortcuts.toggleSecondarySub` |
 | `Ctrl/Cmd+Shift+O` | Open runtime options palette                             | `shortcuts.openRuntimeOptions` |
 | `Ctrl+Shift+J`     | Open Jimaku subtitle search modal                        | `shortcuts.openJimaku`         |
-| `Ctrl+Alt+C`       | Open the manual YouTube subtitle picker                  | `keybindings`                  |
 | `Ctrl+Alt+S`       | Open subtitle sync (subsync) modal                       | `shortcuts.triggerSubsync`     |
 | `\`                | Toggle subtitle sidebar                                  | `subtitleSidebar.toggleKey`    |
 | `` ` ``            | Toggle stats overlay                                     | `stats.toggleKey`              |

docs-site/troubleshooting.md

@@ -29,7 +29,7 @@ SubMiner retries the connection automatically with increasing delays (200 ms, 50
 - Common spikes come from:
   - first subtitle parse/tokenization bursts
   - media generation (`ffmpeg` audio/image and AVIF paths)
-  - media sync and subtitle tooling (`alass`, `ffsubsync`)
+  - media sync and subtitle tooling (`alass`, `ffsubsync`, `whisper` fallback path)
   - `ankiConnect` enrichment (plus polling overhead when proxy mode is disabled)
 
 ### If playback feels sluggish
@@ -57,7 +57,7 @@ SubMiner retries the connection automatically with increasing delays (200 ms, 50
 
 - disable AI translation when not needed (`ankiConnect.ai.enabled: false`)
 - if needed, run immersion telemetry with lower duration expectations (`immersionTracking.enabled: false` for constrained sessions)
-- favor the default lightweight YouTube subtitle startup settings on low-resource systems
+- prefer YouTube `--mode automatic` over `preprocess` on low-resource systems
 
 ### Practical low-impact profile
 

docs-site/usage.md

@@ -78,6 +78,8 @@ subminer mpv idle                 # Launch detached idle mpv with SubMiner defau
 subminer dictionary /path/to/file-or-directory  # Generate character dictionary ZIP from target (manual Yomitan import)
 subminer texthooker               # Launch texthooker-only mode
 subminer app --anilist            # Pass args directly to SubMiner binary (example: AniList login flow)
+subminer yt -o ~/subs https://youtu.be/...  # YouTube subcommand: output directory shortcut
+subminer yt --keep-temp --whisper-bin /path/to/whisper-cli --whisper-model /path/to/model.bin --whisper-vad-model /path/to/ggml-vad.bin https://youtu.be/...  # Keep generated subtitle workspace for debugging
 
 # Direct packaged app control
 SubMiner.AppImage --background             # Start in background (tray + IPC wait, minimal logs)
@@ -135,13 +137,14 @@ This flow requires `mpv.exe` to be on `PATH`. If it is installed elsewhere, set
 ### Launcher Subcommands
 
 - `subminer jellyfin` / `subminer jf`: Jellyfin-focused workflow aliases.
+- `subminer yt` / `subminer youtube`: YouTube-focused shorthand flags (`-o`, `--keep-temp`, `--whisper-*`).
 - `subminer doctor`: health checks for core dependencies and runtime paths.
 - `subminer config`: config helpers (`path`, `show`).
 - `subminer mpv`: mpv helpers (`status`, `socket`, `idle`).
 - `subminer dictionary <path>`: generates a Yomitan-importable character dictionary ZIP from a file/directory target.
 - `subminer texthooker`: texthooker-only shortcut (same behavior as `--texthooker`).
 - `subminer app` / `subminer bin`: direct passthrough to the SubMiner binary/AppImage.
-- Subcommand help pages are available (for example `subminer jellyfin -h`).
+- Subcommand help pages are available (for example `subminer jellyfin -h`, `subminer yt -h`).
 
 ### First-Run Setup
 
@@ -171,8 +174,8 @@ AniList character dictionary auto-sync (optional):
 - SubMiner syncs the currently watched AniList media into a per-media snapshot, then rebuilds one merged `SubMiner Character Dictionary` from the most recently used snapshots.
 - Rotation limit defaults to 3 recent media snapshots in that merged dictionary (`maxLoaded`).
 
-Use subcommands for Jellyfin workflows (`subminer jellyfin ...`).
-Top-level launcher flags like `--jellyfin-*` are intentionally rejected.
+Use subcommands for Jellyfin/YouTube command families (`subminer jellyfin ...`, `subminer yt ...`).
+Top-level launcher flags like `--jellyfin-*` and `--yt-subgen-*` are intentionally rejected.
 
 ### MPV Profile Example (mpv.conf)
 
@@ -225,18 +228,26 @@ If you also use Yomitan in a browser, configure that browser profile separately;
 ### YouTube Playback
 
 `subminer` accepts direct URLs (for example, YouTube links) and `ytsearch:` targets.
-For YouTube playback, SubMiner resolves subtitle selection during startup while mpv is paused: it auto-selects the default primary subtitle track plus a best-effort secondary track, then resumes when primary subtitles are ready.
+For YouTube playback, SubMiner now generates or downloads subtitle tracks before mpv starts, then launches mpv with the resolved subtitle files attached.
 
 Notes:
 
 - Install `yt-dlp` so mpv can resolve YouTube streams and subtitle tracks reliably.
-- For YouTube URLs, startup no longer requires opening the picker first; SubMiner loads subtitles and keeps the overlay available for retries.
-- Press `Ctrl+Alt+C` during active YouTube playback to open the manual YouTube subtitle picker and retry track selection.
-- For YouTube URLs, `subminer` probes available YouTube subtitle tracks, reuses existing authoritative tracks when available, and downloads only missing sides.
-- Native mpv secondary subtitle rendering stays hidden so the overlay remains the visible secondary subtitle surface.
+- For YouTube URLs, `subminer` now generates any missing subtitles before mpv launch.
+- It probes manual/native YouTube subtitle tracks first, then falls back to local `whisper.cpp` only for missing tracks.
 - Primary subtitle target languages come from `youtubeSubgen.primarySubLanguages` (defaults to `["ja","jpn"]`).
 - Secondary target languages come from `secondarySub.secondarySubLanguages` (defaults to English if unset).
-- Configure defaults in `$XDG_CONFIG_HOME/SubMiner/config.jsonc` (or `~/.config/SubMiner/config.jsonc`) under `youtubeSubgen` and `secondarySub`.
+- Whisper translation fallback currently only supports English secondary targets; non-English secondary targets rely on native/manual subtitle availability.
+- Optional AI cleanup for whisper-generated subtitles is controlled by `youtubeSubgen.fixWithAi` plus the shared top-level `ai` config (with optional `youtubeSubgen.ai` overrides).
+- Configure defaults in `$XDG_CONFIG_HOME/SubMiner/config.jsonc` (or `~/.config/SubMiner/config.jsonc`) under `youtubeSubgen`, `secondarySub`, and `ai`.
+- CLI overrides are available through `subminer yt` / `subminer youtube`:
+  - `-o, --out-dir`
+  - `--keep-temp`
+  - `--whisper-bin`
+  - `--whisper-model`
+  - `--whisper-vad-model`
+  - `--whisper-threads`
+  - `--yt-subgen-audio-format`
 
 ## Controller Support
 

launcher/commands/doctor-command.ts

@@ -51,7 +51,7 @@ export function runDoctorCommand(
       ok: deps.commandExists('ffmpeg'),
       detail: deps.commandExists('ffmpeg')
         ? 'found'
-        : 'missing (optional unless legacy subtitle fallback is enabled)',
+        : 'missing (optional unless subtitle generation)',
     },
     {
       label: 'fzf',

launcher/commands/playback-command.test.ts

@@ -82,32 +82,16 @@ function createContext(): LauncherCommandContext {
   };
 }
 
-test('youtube playback launches overlay with app-owned youtube flow args', async () => {
+test('youtube playback launches overlay with youtube-play args in the primary app start', async () => {
   const calls: string[] = [];
   const context = createContext();
-  context.pluginRuntimeConfig = {
-    ...context.pluginRuntimeConfig,
-    autoStart: false,
-    autoStartVisibleOverlay: false,
-    autoStartPauseUntilReady: false,
-  };
-  let receivedStartMpvOptions: Record<string, unknown> | null = null;
 
   await runPlaybackCommandWithDeps(context, {
     ensurePlaybackSetupReady: async () => {},
     chooseTarget: async (_args, _scriptPath) => ({ target: context.args.target, kind: 'url' }),
     checkDependencies: () => {},
     registerCleanup: () => {},
-    startMpv: async (
-      _target,
-      _targetKind,
-      _args,
-      _socketPath,
-      _appPath,
-      _preloadedSubtitles,
-      options,
-    ) => {
-      receivedStartMpvOptions = options ?? null;
+    startMpv: async () => {
       calls.push('startMpv');
     },
     waitForUnixSocketReady: async () => true,
@@ -126,8 +110,4 @@ test('youtube playback launches overlay with app-owned youtube flow args', async
     'startMpv',
     'startOverlay:--youtube-play https://www.youtube.com/watch?v=65Ovd7t8sNw --youtube-mode download',
   ]);
-  assert.deepEqual(receivedStartMpvOptions, {
-    startPaused: true,
-    disableYoutubeSubtitleAutoLoad: true,
-  });
 });

launcher/commands/playback-command.ts

@@ -31,12 +31,11 @@ function checkDependencies(args: Args): void {
 
   if (!commandExists('mpv')) missing.push('mpv');
 
-  const isYoutubeUrl = args.targetKind === 'url' && isYoutubeTarget(args.target);
-  if (args.targetKind === 'url' && !isYoutubeUrl && !commandExists('yt-dlp')) {
+  if (args.targetKind === 'url' && isYoutubeTarget(args.target) && !commandExists('yt-dlp')) {
     missing.push('yt-dlp');
   }
 
-  if (args.targetKind === 'url' && !isYoutubeUrl && !commandExists('ffmpeg')) {
+  if (args.targetKind === 'url' && isYoutubeTarget(args.target) && !commandExists('ffmpeg')) {
     missing.push('ffmpeg');
   }
 

launcher/config.test.ts

@@ -10,6 +10,7 @@ test('launcher root help lists subcommands', () => {
 
   assert.match(output, /Commands:/);
   assert.match(output, /jellyfin\|jf/);
+  assert.match(output, /yt\|youtube/);
   assert.match(output, /doctor/);
   assert.match(output, /config/);
   assert.match(output, /mpv/);

launcher/config/args-normalizer.ts

@@ -111,6 +111,7 @@ export function createDefaultArgs(launcherConfig: LauncherYoutubeSubgenConfig):
     youtubeSubgenAudioFormat: process.env.SUBMINER_YT_SUBGEN_AUDIO_FORMAT || 'm4a',
     youtubeSubgenKeepTemp: process.env.SUBMINER_YT_SUBGEN_KEEP_TEMP === '1',
     youtubeFixWithAi: launcherConfig.fixWithAi === true,
+    youtubeMode: undefined,
     jimakuApiKey: process.env.SUBMINER_JIMAKU_API_KEY || '',
     jimakuApiKeyCommand: process.env.SUBMINER_JIMAKU_API_KEY_COMMAND || '',
     jimakuApiBaseUrl: process.env.SUBMINER_JIMAKU_API_BASE_URL || DEFAULT_JIMAKU_API_BASE_URL,
@@ -249,6 +250,29 @@ export function applyInvocationsToArgs(parsed: Args, invocations: CliInvocations
     parsed.jellyfinLogout = Boolean(modeFlags.logout);
   }
 
+  if (invocations.ytInvocation) {
+    if (invocations.ytInvocation.mode) {
+      parsed.youtubeMode = invocations.ytInvocation.mode;
+    }
+    if (invocations.ytInvocation.logLevel)
+      parsed.logLevel = parseLogLevel(invocations.ytInvocation.logLevel);
+    if (invocations.ytInvocation.outDir)
+      parsed.youtubeSubgenOutDir = invocations.ytInvocation.outDir;
+    if (invocations.ytInvocation.keepTemp) parsed.youtubeSubgenKeepTemp = true;
+    if (invocations.ytInvocation.whisperBin)
+      parsed.whisperBin = invocations.ytInvocation.whisperBin;
+    if (invocations.ytInvocation.whisperModel)
+      parsed.whisperModel = invocations.ytInvocation.whisperModel;
+    if (invocations.ytInvocation.whisperVadModel)
+      parsed.whisperVadModel = invocations.ytInvocation.whisperVadModel;
+    if (invocations.ytInvocation.whisperThreads)
+      parsed.whisperThreads = invocations.ytInvocation.whisperThreads;
+    if (invocations.ytInvocation.ytSubgenAudioFormat) {
+      parsed.youtubeSubgenAudioFormat = invocations.ytInvocation.ytSubgenAudioFormat;
+    }
+    if (invocations.ytInvocation.target) ensureTarget(invocations.ytInvocation.target, parsed);
+  }
+
   if (invocations.dictionaryLogLevel) {
     parsed.logLevel = parseLogLevel(invocations.dictionaryLogLevel);
   }

launcher/config/cli-parser-builder.ts

@@ -14,6 +14,19 @@ export interface JellyfinInvocation {
   logLevel?: string;
 }
 
+export interface YtInvocation {
+  target?: string;
+  mode?: 'download' | 'generate';
+  outDir?: string;
+  keepTemp?: boolean;
+  whisperBin?: string;
+  whisperModel?: string;
+  whisperVadModel?: string;
+  whisperThreads?: number;
+  ytSubgenAudioFormat?: string;
+  logLevel?: string;
+}
+
 export interface CommandActionInvocation {
   action: string;
   logLevel?: string;
@@ -21,6 +34,7 @@ export interface CommandActionInvocation {
 
 export interface CliInvocations {
   jellyfinInvocation: JellyfinInvocation | null;
+  ytInvocation: YtInvocation | null;
   configInvocation: CommandActionInvocation | null;
   mpvInvocation: CommandActionInvocation | null;
   appInvocation: { appArgs: string[] } | null;
@@ -76,6 +90,8 @@ function getTopLevelCommand(argv: string[]): { name: string; index: number } | n
   const commandNames = new Set([
     'jellyfin',
     'jf',
+    'yt',
+    'youtube',
     'doctor',
     'config',
     'mpv',
@@ -127,6 +143,7 @@ export function parseCliPrograms(
   invocations: CliInvocations;
 } {
   let jellyfinInvocation: JellyfinInvocation | null = null;
+  let ytInvocation: YtInvocation | null = null;
   let configInvocation: CommandActionInvocation | null = null;
   let mpvInvocation: CommandActionInvocation | null = null;
   let appInvocation: { appArgs: string[] } | null = null;
@@ -201,6 +218,43 @@ export function parseCliPrograms(
       };
     });
 
+  commandProgram
+    .command('yt')
+    .alias('youtube')
+    .description('YouTube workflows')
+    .argument('[target]', 'YouTube URL or ytsearch: query')
+    .option('--mode <mode>', 'YouTube subtitle acquisition mode')
+    .option('-o, --out-dir <dir>', 'Subtitle output dir')
+    .option('--keep-temp', 'Keep temp files')
+    .option('--whisper-bin <path>', 'whisper.cpp CLI path')
+    .option('--whisper-model <path>', 'whisper model path')
+    .option('--whisper-vad-model <path>', 'whisper.cpp VAD model path')
+    .option('--whisper-threads <n>', 'whisper.cpp thread count')
+    .option('--yt-subgen-audio-format <format>', 'Audio extraction format')
+    .option('--log-level <level>', 'Log level')
+    .action((target: string | undefined, options: Record<string, unknown>) => {
+      ytInvocation = {
+        target,
+        mode:
+          typeof options.mode === 'string' && (options.mode === 'download' || options.mode === 'generate')
+            ? options.mode
+            : undefined,
+        outDir: typeof options.outDir === 'string' ? options.outDir : undefined,
+        keepTemp: options.keepTemp === true,
+        whisperBin: typeof options.whisperBin === 'string' ? options.whisperBin : undefined,
+        whisperModel: typeof options.whisperModel === 'string' ? options.whisperModel : undefined,
+        whisperVadModel:
+          typeof options.whisperVadModel === 'string' ? options.whisperVadModel : undefined,
+        whisperThreads:
+          typeof options.whisperThreads === 'number' && Number.isFinite(options.whisperThreads)
+            ? Math.floor(options.whisperThreads)
+            : undefined,
+        ytSubgenAudioFormat:
+          typeof options.ytSubgenAudioFormat === 'string' ? options.ytSubgenAudioFormat : undefined,
+        logLevel: typeof options.logLevel === 'string' ? options.logLevel : undefined,
+      };
+    });
+
   commandProgram
     .command('dictionary')
     .alias('dict')
@@ -334,6 +388,7 @@ export function parseCliPrograms(
     rootTarget: rootProgram.processedArgs[0],
     invocations: {
       jellyfinInvocation,
+      ytInvocation,
       configInvocation,
       mpvInvocation,
       appInvocation,

launcher/main.test.ts

@@ -362,7 +362,7 @@ ${bunBinary} -e "const net=require('node:net'); const fs=require('node:fs'); con
   });
 });
 
-test('launcher routes youtube urls through regular playback startup', { timeout: 15000 }, () => {
+test('launcher disables plugin startup pause gate for app-owned youtube flow', { timeout: 15000 }, () => {
   withTempDir((root) => {
     const homeDir = path.join(root, 'home');
     const xdgConfigHome = path.join(root, 'xdg');
@@ -430,16 +430,13 @@ ${bunBinary} -e "const net=require('node:net'); const fs=require('node:fs'); con
       SUBMINER_TEST_MPV_ARGS: mpvArgsPath,
       SUBMINER_TEST_CAPTURE: path.join(root, 'captured-args.txt'),
     };
-    const result = runLauncher(['https://www.youtube.com/watch?v=abc123'], env);
+    const result = runLauncher(['yt', 'https://www.youtube.com/watch?v=abc123'], env);
 
     assert.equal(result.status, 0, `stdout:\n${result.stdout}\nstderr:\n${result.stderr}`);
-    const forwardedArgs = fs
-      .readFileSync(mpvArgsPath, 'utf8')
-      .trim()
-      .split('\n')
-      .map((item) => item.trim())
-      .filter(Boolean);
-    assert.equal(forwardedArgs.includes('https://www.youtube.com/watch?v=abc123'), true);
+    assert.match(
+      fs.readFileSync(mpvArgsPath, 'utf8'),
+      /--script-opts=.*subminer-auto_start_pause_until_ready=no/,
+    );
   });
 });
 

launcher/mpv.ts

@@ -557,34 +557,38 @@ export async function startMpv(
   const mpvArgs: string[] = [];
   if (args.profile) mpvArgs.push(`--profile=${args.profile}`);
   mpvArgs.push(...DEFAULT_MPV_SUBMINER_ARGS);
-  if (targetKind === 'url' && isYoutubeTarget(target)) {
-    log('info', args.logLevel, 'Applying URL playback options');
-    mpvArgs.push('--ytdl=yes');
-    const subtitleLangs = uniqueNormalizedLangCodes([
-      ...args.youtubePrimarySubLangs,
-      ...args.youtubeSecondarySubLangs,
-    ]).join(',');
-    const audioLangs = uniqueNormalizedLangCodes(args.youtubeAudioLangs).join(',');
-    log('info', args.logLevel, 'Applying YouTube playback options');
-    log('debug', args.logLevel, `YouTube subtitle langs: ${subtitleLangs}`);
-    log('debug', args.logLevel, `YouTube audio langs: ${audioLangs}`);
-    mpvArgs.push(`--ytdl-format=${DEFAULT_YOUTUBE_YTDL_FORMAT}`, `--alang=${audioLangs}`);
-    if (options?.disableYoutubeSubtitleAutoLoad !== true) {
-      mpvArgs.push(
-        '--sub-auto=fuzzy',
-        `--slang=${subtitleLangs}`,
-        '--ytdl-raw-options-append=write-subs=',
-        '--ytdl-raw-options-append=sub-format=vtt/best',
-        `--ytdl-raw-options-append=sub-langs=${subtitleLangs}`,
-      );
-    } else {
-      mpvArgs.push('--sub-auto=no');
-    }
-  }
   if (args.mpvArgs) {
     mpvArgs.push(...parseMpvArgString(args.mpvArgs));
   }
 
+  if (targetKind === 'url' && isYoutubeTarget(target)) {
+    log('info', args.logLevel, 'Applying URL playback options');
+    mpvArgs.push('--ytdl=yes', '--ytdl-raw-options=');
+
+    if (isYoutubeTarget(target)) {
+      const subtitleLangs = uniqueNormalizedLangCodes([
+        ...args.youtubePrimarySubLangs,
+        ...args.youtubeSecondarySubLangs,
+      ]).join(',');
+      const audioLangs = uniqueNormalizedLangCodes(args.youtubeAudioLangs).join(',');
+      log('info', args.logLevel, 'Applying YouTube playback options');
+      log('debug', args.logLevel, `YouTube subtitle langs: ${subtitleLangs}`);
+      log('debug', args.logLevel, `YouTube audio langs: ${audioLangs}`);
+      mpvArgs.push(`--ytdl-format=${DEFAULT_YOUTUBE_YTDL_FORMAT}`, `--alang=${audioLangs}`);
+      if (options?.disableYoutubeSubtitleAutoLoad !== true) {
+        mpvArgs.push(
+          '--sub-auto=fuzzy',
+          `--slang=${subtitleLangs}`,
+          '--ytdl-raw-options-append=write-subs=',
+          '--ytdl-raw-options-append=sub-format=vtt/best',
+          `--ytdl-raw-options-append=sub-langs=${subtitleLangs}`,
+        );
+      } else {
+        mpvArgs.push('--sub-auto=no');
+      }
+    }
+  }
+
   if (preloadedSubtitles?.primaryPath) {
     mpvArgs.push(`--sub-file=${preloadedSubtitles.primaryPath}`);
   }

launcher/parse-args.test.ts

@@ -85,6 +85,13 @@ test('parseArgs maps mpv idle action', () => {
   assert.equal(parsed.mpvStatus, false);
 });
 
+test('parseArgs captures youtube mode forwarding', () => {
+  const parsed = parseArgs(['youtube', 'https://example.com', '--mode', 'generate'], 'subminer', {});
+
+  assert.equal(parsed.target, 'https://example.com');
+  assert.equal(parsed.youtubeMode, 'generate');
+});
+
 test('parseArgs maps dictionary command and log-level override', () => {
   const parsed = parseArgs(['dictionary', '.', '--log-level', 'debug'], 'subminer', {});
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "subminer",
-  "version": "0.9.1",
+  "version": "0.9.0",
   "description": "All-in-one sentence mining overlay with AnkiConnect and dictionary integration",
   "packageManager": "bun@1.3.5",
   "main": "dist/main-entry.js",
@@ -166,44 +166,20 @@
     },
     "files": [
       "**/*",
-      "!assets{,/**/*}",
       "!src{,/**/*}",
       "!launcher{,/**/*}",
-      "!docs{,/**/*}",
-      "!tests{,/**/*}",
-      "!packaging{,/**/*}",
-      "!README.md",
-      "!CHANGELOG.md",
-      "!AGENTS.md",
-      "!CLAUDE.md",
       "!stats/src{,/**/*}",
       "!stats/index.html",
-      "!stats/public{,/**/*}",
-      "!stats/package.json",
-      "!stats/tsconfig.json",
-      "!stats/vite.config.ts",
       "!docs-site{,/**/*}",
       "!changes{,/**/*}",
       "!backlog{,/**/*}",
       "!.tmp{,/**/*}",
       "!release-*{,/**/*}",
-      "!dist/**/*.map",
-      "!dist/**/*.test.*",
-      "!dist/**/__tests__{,/**/*}",
-      "!scripts/**/*.test.*",
-      "!plugin{,/**/*}",
       "!vendor/subminer-yomitan{,/**/*}",
-      "!vendor/yomitan-jlpt-vocab{,/**/*}",
       "!vendor/texthooker-ui/src{,/**/*}",
       "!vendor/texthooker-ui/node_modules{,/**/*}",
       "!vendor/texthooker-ui/.svelte-kit{,/**/*}",
-      "!vendor/texthooker-ui/.vscode{,/**/*}",
-      "!vendor/texthooker-ui/public{,/**/*}",
-      "!vendor/texthooker-ui/README.md",
-      "!vendor/texthooker-ui/package.json",
-      "!vendor/texthooker-ui/package-lock.json",
-      "!vendor/texthooker-ui/tsconfig*.json",
-      "!node_modules/@libsql/linux-x64-musl{,/**/*}"
+      "!vendor/texthooker-ui/package-lock.json"
     ],
     "extraResources": [
       {

release/release-notes.md

@@ -1,19 +0,0 @@
-## Highlights
-### Changed
-- Release: Reduced packaged release size by excluding duplicate `extraResources` payload and pruning docs, tests, sourcemaps, and other source-only files from Electron bundles.
-
-### Fixed
-- Overlay: Restored controller navigation and lookup/mining controls while the subtitle sidebar is open, while keeping true modal dialogs blocking controller actions.
-- Tokenizer: Fixed subtitle annotation clearing so explanatory contrast endings like `んですけど` are excluded consistently across the shared tokenizer filter and annotation stage.
-
-## Installation
-
-See the README and docs/installation guide for full setup steps.
-
-## Assets
-
-- Linux: `SubMiner.AppImage`
-- macOS: `SubMiner-*.dmg` and `SubMiner-*.zip`
-- Optional extras: `subminer-assets.tar.gz` and the `subminer` launcher
-
-Note: the `subminer` wrapper script uses Bun (`#!/usr/bin/env bun`), so `bun` must be installed and on `PATH`.

src/cli/args.test.ts

@@ -56,13 +56,8 @@ test('parseArgs captures launch-mpv targets and keeps it out of app startup', ()
   assert.equal(shouldStartApp(args), false);
 });
 
-test('parseArgs captures youtube startup forwarding flags', () => {
-  const args = parseArgs([
-    '--youtube-play',
-    'https://youtube.com/watch?v=abc',
-    '--youtube-mode',
-    'generate',
-  ]);
+test('parseArgs captures youtube playback commands and mode', () => {
+  const args = parseArgs(['--youtube-play', 'https://youtube.com/watch?v=abc', '--youtube-mode', 'generate']);
 
   assert.equal(args.youtubePlay, 'https://youtube.com/watch?v=abc');
   assert.equal(args.youtubeMode, 'generate');

src/cli/args.ts

@@ -499,7 +499,7 @@ export function commandNeedsOverlayRuntime(args: CliArgs): boolean {
     args.triggerFieldGrouping ||
     args.triggerSubsync ||
     args.markAudioCard ||
-    args.openRuntimeOptions ||
-    Boolean(args.youtubePlay)
+    args.openRuntimeOptions
+    || Boolean(args.youtubePlay)
   );
 }

src/cli/help.ts

@@ -13,6 +13,8 @@ ${B}Session${R}
   --background                   Start in tray/background mode
   --start                        Connect to mpv and launch overlay
   --launch-mpv ${D}[targets...]${R}       Launch mpv with the SubMiner mpv profile and exit
+  --youtube-play ${D}URL${R}              Open YouTube subtitle picker flow for a URL
+  --youtube-mode ${D}download|generate${R}  Subtitle acquisition mode for YouTube flow
   --stop                         Stop the running instance
   --stats                        Open the stats dashboard in your browser
   --texthooker                   Start texthooker server only ${D}(no overlay)${R}

src/config/config.test.ts

@@ -1735,7 +1735,7 @@ test('accepts top-level ai config', () => {
   assert.equal(config.ai.requestTimeoutMs, 20000);
 });
 
-test('accepts per-feature ai overrides for anki and YouTube subtitles', () => {
+test('accepts per-feature ai overrides for anki and youtube subtitle generation', () => {
   const dir = makeTempDir();
   fs.writeFileSync(
     path.join(dir, 'config.jsonc'),
@@ -2074,16 +2074,16 @@ test('template generator includes known keys', () => {
   );
   assert.match(
     output,
-    /"fixWithAi": false,? \/\/ Legacy subtitle fallback post-processing switch kept for compatibility; use is currently disabled by default\. Values: true \| false/,
+    /"fixWithAi": false,? \/\/ Use shared AI provider to post-process whisper-generated YouTube subtitles\. Values: true \| false/,
   );
   assert.match(
     output,
-    /"systemPrompt": "",? \/\/ Optional system prompt override for legacy subtitle fallback post-processing; not used by default\./,
+    /"systemPrompt": "",? \/\/ Optional system prompt override for YouTube subtitle AI post-processing\./,
   );
   assert.doesNotMatch(output, /"mode": "automatic"/);
   assert.match(
     output,
-    /"whisperThreads": 4,? \/\/ Legacy thread tuning for subtitle fallback tooling; not used by default\./,
+    /"whisperThreads": 4,? \/\/ Thread count passed to whisper\.cpp subtitle generation runs\./,
   );
   assert.match(
     output,

src/config/definitions/domain-registry.test.ts

@@ -77,7 +77,6 @@ test('default keybindings include primary and secondary subtitle track cycling o
   );
   assert.deepEqual(keybindingMap.get('KeyJ'), ['cycle', 'sid']);
   assert.deepEqual(keybindingMap.get('Shift+KeyJ'), ['cycle', 'secondary-sid']);
-  assert.deepEqual(keybindingMap.get('Ctrl+Alt+KeyC'), ['__youtube-picker-open']);
 });
 
 test('default keybindings include fullscreen on F', () => {

src/config/definitions/options-integrations.ts

@@ -369,47 +369,43 @@ export function buildIntegrationConfigOptionRegistry(
       path: 'youtubeSubgen.whisperBin',
       kind: 'string',
       defaultValue: defaultConfig.youtubeSubgen.whisperBin,
-      description: 'Legacy compatibility path kept for external subtitle fallback tools; not used by default.',
+      description: 'Path to whisper.cpp CLI used as fallback transcription engine.',
     },
     {
       path: 'youtubeSubgen.whisperModel',
       kind: 'string',
       defaultValue: defaultConfig.youtubeSubgen.whisperModel,
-      description: 'Legacy compatibility model path kept for external subtitle fallback tooling; not used by default.',
+      description: 'Path to whisper model used for fallback transcription.',
     },
     {
       path: 'youtubeSubgen.whisperVadModel',
       kind: 'string',
       defaultValue: defaultConfig.youtubeSubgen.whisperVadModel,
-      description:
-        'Legacy compatibility VAD path kept for external subtitle fallback tooling; not used by default.',
+      description: 'Path to optional whisper VAD model used for subtitle generation.',
     },
     {
       path: 'youtubeSubgen.whisperThreads',
       kind: 'number',
       defaultValue: defaultConfig.youtubeSubgen.whisperThreads,
-      description: 'Legacy thread tuning for subtitle fallback tooling; not used by default.',
+      description: 'Thread count passed to whisper.cpp subtitle generation runs.',
     },
     {
       path: 'youtubeSubgen.fixWithAi',
       kind: 'boolean',
       defaultValue: defaultConfig.youtubeSubgen.fixWithAi,
-      description:
-        'Legacy subtitle fallback post-processing switch kept for compatibility; use is currently disabled by default.',
+      description: 'Use shared AI provider to post-process whisper-generated YouTube subtitles.',
     },
     {
       path: 'youtubeSubgen.ai.model',
       kind: 'string',
       defaultValue: defaultConfig.youtubeSubgen.ai.model,
-      description:
-        'Optional model override for legacy subtitle fallback post-processing; not used by default.',
+      description: 'Optional model override for YouTube subtitle AI post-processing.',
     },
     {
       path: 'youtubeSubgen.ai.systemPrompt',
       kind: 'string',
       defaultValue: defaultConfig.youtubeSubgen.ai.systemPrompt,
-      description:
-        'Optional system prompt override for legacy subtitle fallback post-processing; not used by default.',
+      description: 'Optional system prompt override for YouTube subtitle AI post-processing.',
     },
     {
       path: 'youtubeSubgen.primarySubLanguages',

src/config/definitions/shared.ts

@@ -46,7 +46,6 @@ export const SPECIAL_COMMANDS = {
   PLAY_NEXT_SUBTITLE: '__play-next-subtitle',
   SHIFT_SUB_DELAY_TO_NEXT_SUBTITLE_START: '__sub-delay-next-line',
   SHIFT_SUB_DELAY_TO_PREVIOUS_SUBTITLE_START: '__sub-delay-prev-line',
-  YOUTUBE_PICKER_OPEN: '__youtube-picker-open',
 } as const;
 
 export const DEFAULT_KEYBINDINGS: NonNullable<ResolvedConfig['keybindings']> = [
@@ -65,7 +64,6 @@ export const DEFAULT_KEYBINDINGS: NonNullable<ResolvedConfig['keybindings']> = [
     key: 'Shift+BracketLeft',
     command: [SPECIAL_COMMANDS.SHIFT_SUB_DELAY_TO_PREVIOUS_SUBTITLE_START],
   },
-  { key: 'Ctrl+Alt+KeyC', command: [SPECIAL_COMMANDS.YOUTUBE_PICKER_OPEN] },
   { key: 'Ctrl+Shift+KeyH', command: [SPECIAL_COMMANDS.REPLAY_SUBTITLE] },
   { key: 'Ctrl+Shift+KeyL', command: [SPECIAL_COMMANDS.PLAY_NEXT_SUBTITLE] },
   { key: 'KeyQ', command: ['quit'] },

src/config/definitions/template-sections.ts

@@ -74,7 +74,7 @@ const CORE_TEMPLATE_SECTIONS: ConfigTemplateSection[] = [
     title: 'Secondary Subtitles',
     description: [
       'Dual subtitle track options.',
-      'Used by the YouTube subtitle loading flow as secondary language preferences.',
+      'Used by subminer YouTube subtitle generation as secondary language preferences.',
     ],
     notes: ['Hot-reload: defaultMode updates live while SubMiner is running.'],
     key: 'secondarySub',
@@ -130,8 +130,8 @@ const INTEGRATION_TEMPLATE_SECTIONS: ConfigTemplateSection[] = [
     key: 'jimaku',
   },
   {
-    title: 'YouTube Playback Settings',
-    description: ['Defaults for SubMiner YouTube subtitle loading and languages.'],
+    title: 'YouTube Subtitle Generation',
+    description: ['Defaults for SubMiner YouTube subtitle generation.'],
     key: 'youtubeSubgen',
   },
   {

src/core/services/cli-command.test.ts

@@ -186,8 +186,8 @@ function createDeps(overrides: Partial<CliCommandServiceDeps> = {}) {
     runJellyfinCommand: async () => {
       calls.push('runJellyfinCommand');
     },
-    runYoutubePlaybackFlow: async (request) => {
-      calls.push(`runYoutubePlaybackFlow:${request.url}:${request.mode}:${request.source}`);
+    runYoutubePlaybackFlow: async ({ url, mode }) => {
+      calls.push(`runYoutubePlaybackFlow:${url}:${mode}`);
     },
     printHelp: () => {
       calls.push('printHelp');
@@ -212,6 +212,25 @@ function createDeps(overrides: Partial<CliCommandServiceDeps> = {}) {
   return { deps, calls, osd };
 }
 
+test('handleCliCommand reconnects MPV for second-instance --start when overlay runtime is already initialized', () => {
+  const { deps, calls } = createDeps({
+    isOverlayRuntimeInitialized: () => true,
+  });
+  const args = makeArgs({ start: true });
+
+  handleCliCommand(args, 'second-instance', deps);
+
+  assert.ok(calls.includes('setMpvClientSocketPath:/tmp/subminer.sock'));
+  assert.equal(
+    calls.some((value) => value.includes('connectMpvClient')),
+    true,
+  );
+  assert.equal(
+    calls.some((value) => value.includes('initializeOverlayRuntime')),
+    false,
+  );
+});
+
 test('handleCliCommand starts youtube playback flow on initial launch', () => {
   const { deps, calls } = createDeps({
     runYoutubePlaybackFlow: async (request) => {
@@ -246,43 +265,6 @@ test('handleCliCommand defaults youtube mode to download when omitted', () => {
   ]);
 });
 
-test('handleCliCommand reports youtube playback flow failures to logs and OSD', async () => {
-  const { deps, calls, osd } = createDeps({
-    runYoutubePlaybackFlow: async () => {
-      throw new Error('yt failed');
-    },
-  });
-
-  handleCliCommand(
-    makeArgs({ youtubePlay: 'https://youtube.com/watch?v=abc', youtubeMode: 'download' }),
-    'initial',
-    deps,
-  );
-  await new Promise((resolve) => setImmediate(resolve));
-
-  assert.ok(calls.some((value) => value.startsWith('error:runYoutubePlaybackFlow failed:')));
-  assert.ok(osd.includes('YouTube playback failed: yt failed'));
-});
-
-test('handleCliCommand reconnects MPV for second-instance --start when overlay runtime is already initialized', () => {
-  const { deps, calls } = createDeps({
-    isOverlayRuntimeInitialized: () => true,
-  });
-  const args = makeArgs({ start: true });
-
-  handleCliCommand(args, 'second-instance', deps);
-
-  assert.ok(calls.includes('setMpvClientSocketPath:/tmp/subminer.sock'));
-  assert.equal(
-    calls.some((value) => value.includes('connectMpvClient')),
-    true,
-  );
-  assert.equal(
-    calls.some((value) => value.includes('initializeOverlayRuntime')),
-    false,
-  );
-});
-
 test('handleCliCommand processes --start for second-instance when overlay runtime is not initialized', () => {
   const { deps, calls } = createDeps();
   const args = makeArgs({ start: true });

src/core/services/immersion-tracker-service.test.ts

@@ -1284,40 +1284,6 @@ test('flushTelemetry checkpoints latest playback position on the active session
   }
 });
 
-test('recordSubtitleLine advances session checkpoint progress when playback position is unavailable', async () => {
-  const dbPath = makeDbPath();
-  let tracker: ImmersionTrackerService | null = null;
-
-  try {
-    const Ctor = await loadTrackerCtor();
-    tracker = new Ctor({ dbPath });
-
-    tracker.handleMediaChange('https://stream.example.com/subtitle-progress.m3u8', 'Subtitle Progress');
-    tracker.recordSubtitleLine('line one', 170, 185, [], null);
-
-    const privateApi = tracker as unknown as {
-      db: DatabaseSync;
-      sessionState: { sessionId: number } | null;
-      flushTelemetry: (force?: boolean) => void;
-      flushNow: () => void;
-    };
-    const sessionId = privateApi.sessionState?.sessionId;
-    assert.ok(sessionId);
-
-    privateApi.flushTelemetry(true);
-    privateApi.flushNow();
-
-    const row = privateApi.db
-      .prepare('SELECT ended_media_ms FROM imm_sessions WHERE session_id = ?')
-      .get(sessionId) as { ended_media_ms: number | null } | null;
-
-    assert.equal(row?.ended_media_ms, 185_000);
-  } finally {
-    tracker?.destroy();
-    cleanupDbPath(dbPath);
-  }
-});
-
 test('deleteSession ignores the currently active session and keeps new writes flushable', async () => {
   const dbPath = makeDbPath();
   let tracker: ImmersionTrackerService | null = null;
@@ -2446,23 +2412,6 @@ printf '%s\n' '${ytDlpOutput}'
         `,
       )
       .get() as { canonicalTitle: string } | null;
-    const animeRow = privateApi.db
-      .prepare(
-        `
-          SELECT
-            a.canonical_title AS canonicalTitle,
-            v.parsed_title AS parsedTitle,
-            v.parser_source AS parserSource
-          FROM imm_videos v
-          JOIN imm_anime a ON a.anime_id = v.anime_id
-          WHERE v.video_id = 1
-        `,
-      )
-      .get() as {
-      canonicalTitle: string;
-      parsedTitle: string | null;
-      parserSource: string | null;
-    } | null;
 
     assert.ok(row);
     assert.ok(videoRow);
@@ -2478,9 +2427,6 @@ printf '%s\n' '${ytDlpOutput}'
     assert.equal(row.uploaderUrl, 'https://www.youtube.com/@creator');
     assert.equal(row.description, 'Video description');
     assert.equal(videoRow.canonicalTitle, 'Video Name');
-    assert.equal(animeRow?.canonicalTitle, 'Creator Name');
-    assert.equal(animeRow?.parsedTitle, 'Creator Name');
-    assert.equal(animeRow?.parserSource, 'youtube');
   } finally {
     process.env.PATH = originalPath;
     globalThis.fetch = originalFetch;
@@ -2492,419 +2438,6 @@ printf '%s\n' '${ytDlpOutput}'
   }
 });
 
-test('getMediaLibrary lazily backfills missing youtube metadata for existing rows', async () => {
-  const dbPath = makeDbPath();
-  let tracker: ImmersionTrackerService | null = null;
-  const originalPath = process.env.PATH;
-  let fakeBinDir: string | null = null;
-
-  try {
-    fakeBinDir = fs.mkdtempSync(path.join(os.tmpdir(), 'subminer-yt-dlp-bin-'));
-    const ytDlpOutput =
-      '{"id":"backfill123","title":"Backfilled Video Title","webpage_url":"https://www.youtube.com/watch?v=backfill123","thumbnail":"https://i.ytimg.com/vi/backfill123/hqdefault.jpg","channel_id":"UCbackfill123","channel":"Backfill Creator","channel_url":"https://www.youtube.com/channel/UCbackfill123","uploader_id":"@backfill","uploader_url":"https://www.youtube.com/@backfill","description":"Backfilled description","thumbnails":[{"url":"https://i.ytimg.com/vi/backfill123/hqdefault.jpg"},{"url":"https://yt3.googleusercontent.com/backfill-avatar=s88"}]}';
-    if (process.platform === 'win32') {
-      const outputPath = path.join(fakeBinDir, 'output.json');
-      fs.writeFileSync(outputPath, ytDlpOutput, 'utf8');
-      fs.writeFileSync(
-        path.join(fakeBinDir, 'yt-dlp.cmd'),
-        '@echo off\r\ntype "%~dp0output.json"\r\n',
-        'utf8',
-      );
-    } else {
-      const scriptPath = path.join(fakeBinDir, 'yt-dlp');
-      fs.writeFileSync(
-        scriptPath,
-        `#!/bin/sh
-printf '%s\n' '${ytDlpOutput}'
-`,
-        { mode: 0o755 },
-      );
-    }
-    process.env.PATH = `${fakeBinDir}${path.delimiter}${originalPath ?? ''}`;
-
-    const Ctor = await loadTrackerCtor();
-    tracker = new Ctor({ dbPath });
-    const privateApi = tracker as unknown as { db: DatabaseSync };
-    const nowMs = Date.now();
-
-    privateApi.db
-      .prepare(
-        `
-          INSERT INTO imm_videos (
-            video_key,
-            canonical_title,
-            source_type,
-            source_path,
-            source_url,
-            duration_ms,
-            file_size_bytes,
-            codec_id,
-            container_id,
-            width_px,
-            height_px,
-            fps_x100,
-            bitrate_kbps,
-            audio_codec_id,
-            hash_sha256,
-            screenshot_path,
-            metadata_json,
-            CREATED_DATE,
-            LAST_UPDATE_DATE
-          ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-        `,
-      )
-      .run(
-        'remote:https://www.youtube.com/watch?v=backfill123',
-        'watch?v=backfill123',
-        2,
-        null,
-        'https://www.youtube.com/watch?v=backfill123',
-        0,
-        null,
-        null,
-        null,
-        null,
-        null,
-        null,
-        null,
-        null,
-        null,
-        null,
-        null,
-        nowMs,
-        nowMs,
-      );
-    privateApi.db
-      .prepare(
-        `
-          INSERT INTO imm_lifetime_media (
-            video_id,
-            total_sessions,
-            total_active_ms,
-            total_cards,
-            total_lines_seen,
-            total_tokens_seen,
-            completed,
-            first_watched_ms,
-            last_watched_ms,
-            CREATED_DATE,
-            LAST_UPDATE_DATE
-          ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-        `,
-      )
-      .run(1, 1, 5_000, 0, 0, 50, 0, nowMs, nowMs, nowMs, nowMs);
-
-    const before = await tracker.getMediaLibrary();
-    assert.equal(before[0]?.channelName ?? null, null);
-
-    await waitForCondition(() => {
-      const row = privateApi.db
-        .prepare(
-          `
-            SELECT
-              video_title AS videoTitle,
-              channel_name AS channelName,
-              channel_thumbnail_url AS channelThumbnailUrl
-            FROM imm_youtube_videos
-            WHERE video_id = 1
-          `,
-        )
-        .get() as {
-        videoTitle: string | null;
-        channelName: string | null;
-        channelThumbnailUrl: string | null;
-      } | null;
-      return (
-        row?.videoTitle === 'Backfilled Video Title' &&
-        row.channelName === 'Backfill Creator' &&
-        row.channelThumbnailUrl === 'https://yt3.googleusercontent.com/backfill-avatar=s88'
-      );
-    }, 5_000);
-
-    const after = await tracker.getMediaLibrary();
-    assert.equal(after[0]?.videoTitle, 'Backfilled Video Title');
-    assert.equal(after[0]?.channelName, 'Backfill Creator');
-    assert.equal(
-      after[0]?.channelThumbnailUrl,
-      'https://yt3.googleusercontent.com/backfill-avatar=s88',
-    );
-  } finally {
-    process.env.PATH = originalPath;
-    tracker?.destroy();
-    cleanupDbPath(dbPath);
-    if (fakeBinDir) {
-      fs.rmSync(fakeBinDir, { recursive: true, force: true });
-    }
-  }
-});
-
-test('getAnimeLibrary lazily relinks youtube rows to channel groupings', async () => {
-  const dbPath = makeDbPath();
-  let tracker: ImmersionTrackerService | null = null;
-
-  try {
-    const Ctor = await loadTrackerCtor();
-    tracker = new Ctor({ dbPath });
-    const privateApi = tracker as unknown as { db: DatabaseSync };
-    const nowMs = Date.now();
-
-    privateApi.db.exec(`
-      INSERT INTO imm_anime (
-        anime_id,
-        normalized_title_key,
-        canonical_title,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES
-        (1, 'watch v first', 'watch?v first', ${nowMs}, ${nowMs}),
-        (2, 'watch v second', 'watch?v second', ${nowMs}, ${nowMs});
-
-      INSERT INTO imm_videos (
-        video_id,
-        anime_id,
-        video_key,
-        canonical_title,
-        parsed_title,
-        parser_source,
-        source_type,
-        source_path,
-        source_url,
-        duration_ms,
-        file_size_bytes,
-        codec_id,
-        container_id,
-        width_px,
-        height_px,
-        fps_x100,
-        bitrate_kbps,
-        audio_codec_id,
-        hash_sha256,
-        screenshot_path,
-        metadata_json,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES
-        (
-          1,
-          1,
-          'remote:https://www.youtube.com/watch?v=first',
-          'watch?v first',
-          'watch?v first',
-          'fallback',
-          2,
-          NULL,
-          'https://www.youtube.com/watch?v=first',
-          0,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          ${nowMs},
-          ${nowMs}
-        ),
-        (
-          2,
-          2,
-          'remote:https://www.youtube.com/watch?v=second',
-          'watch?v second',
-          'watch?v second',
-          'fallback',
-          2,
-          NULL,
-          'https://www.youtube.com/watch?v=second',
-          0,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          NULL,
-          ${nowMs},
-          ${nowMs}
-        );
-
-      INSERT INTO imm_youtube_videos (
-        video_id,
-        youtube_video_id,
-        video_url,
-        video_title,
-        video_thumbnail_url,
-        channel_id,
-        channel_name,
-        channel_url,
-        channel_thumbnail_url,
-        uploader_id,
-        uploader_url,
-        description,
-        metadata_json,
-        fetched_at_ms,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES
-        (
-          1,
-          'first',
-          'https://www.youtube.com/watch?v=first',
-          'First Video',
-          'https://i.ytimg.com/vi/first/hqdefault.jpg',
-          'UCchannel1',
-          'Shared Channel',
-          'https://www.youtube.com/channel/UCchannel1',
-          'https://yt3.googleusercontent.com/shared=s88',
-          '@shared',
-          'https://www.youtube.com/@shared',
-          NULL,
-          '{}',
-          ${nowMs},
-          ${nowMs},
-          ${nowMs}
-        ),
-        (
-          2,
-          'second',
-          'https://www.youtube.com/watch?v=second',
-          'Second Video',
-          'https://i.ytimg.com/vi/second/hqdefault.jpg',
-          'UCchannel1',
-          'Shared Channel',
-          'https://www.youtube.com/channel/UCchannel1',
-          'https://yt3.googleusercontent.com/shared=s88',
-          '@shared',
-          'https://www.youtube.com/@shared',
-          NULL,
-          '{}',
-          ${nowMs},
-          ${nowMs},
-          ${nowMs}
-        );
-
-      INSERT INTO imm_sessions (
-        session_id,
-        session_uuid,
-        video_id,
-        started_at_ms,
-        ended_at_ms,
-        status,
-        total_watched_ms,
-        active_watched_ms,
-        lines_seen,
-        tokens_seen,
-        cards_mined,
-        lookup_count,
-        lookup_hits,
-        yomitan_lookup_count,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES
-        (
-          1,
-          'session-youtube-1',
-          1,
-          ${nowMs - 70000},
-          ${nowMs - 10000},
-          2,
-          65000,
-          60000,
-          0,
-          100,
-          0,
-          0,
-          0,
-          0,
-          ${nowMs},
-          ${nowMs}
-        ),
-        (
-          2,
-          'session-youtube-2',
-          2,
-          ${nowMs - 50000},
-          ${nowMs - 5000},
-          2,
-          35000,
-          30000,
-          0,
-          50,
-          0,
-          0,
-          0,
-          0,
-          ${nowMs},
-          ${nowMs}
-        );
-
-      INSERT INTO imm_lifetime_anime (
-        anime_id,
-        total_sessions,
-        total_active_ms,
-        total_cards,
-        total_lines_seen,
-        total_tokens_seen,
-        episodes_started,
-        episodes_completed,
-        first_watched_ms,
-        last_watched_ms,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES
-        (1, 1, 60000, 0, 0, 100, 1, 0, ${nowMs}, ${nowMs}, ${nowMs}, ${nowMs}),
-        (2, 1, 30000, 0, 0, 50, 1, 0, ${nowMs}, ${nowMs}, ${nowMs}, ${nowMs});
-
-      INSERT INTO imm_lifetime_media (
-        video_id,
-        total_sessions,
-        total_active_ms,
-        total_cards,
-        total_lines_seen,
-        total_tokens_seen,
-        completed,
-        first_watched_ms,
-        last_watched_ms,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES
-        (1, 1, 60000, 0, 0, 100, 0, ${nowMs}, ${nowMs}, ${nowMs}, ${nowMs}),
-        (2, 1, 30000, 0, 0, 50, 0, ${nowMs}, ${nowMs}, ${nowMs}, ${nowMs});
-    `);
-
-    const rows = await tracker.getAnimeLibrary();
-    const sharedRows = rows.filter((row) => row.canonicalTitle === 'Shared Channel');
-
-    assert.equal(sharedRows.length, 1);
-    assert.equal(sharedRows[0]?.episodeCount, 2);
-
-    const relinked = privateApi.db
-      .prepare(
-        `
-          SELECT a.canonical_title AS canonicalTitle, COUNT(*) AS total
-          FROM imm_videos v
-          JOIN imm_anime a ON a.anime_id = v.anime_id
-          GROUP BY a.anime_id, a.canonical_title
-          ORDER BY total DESC, a.anime_id ASC
-        `,
-      )
-      .all() as Array<{ canonicalTitle: string; total: number }>;
-
-    assert.equal(relinked[0]?.canonicalTitle, 'Shared Channel');
-    assert.equal(relinked[0]?.total, 2);
-  } finally {
-    tracker?.destroy();
-    cleanupDbPath(dbPath);
-  }
-});
-
 test('reassignAnimeAnilist clears description when description is explicitly null', async () => {
   const dbPath = makeDbPath();
   let tracker: ImmersionTrackerService | null = null;

src/core/services/immersion-tracker-service.ts

@@ -20,7 +20,6 @@ import {
   getOrCreateAnimeRecord,
   getOrCreateVideoRecord,
   linkVideoToAnimeRecord,
-  linkYoutubeVideoToAnimeRecord,
   type TrackerPreparedStatements,
   updateVideoMetadataRecord,
   updateVideoTitleRecord,
@@ -162,7 +161,6 @@ const YOUTUBE_COVER_RETRY_MS = 5 * 60 * 1000;
 const YOUTUBE_SCREENSHOT_MAX_SECONDS = 120;
 const YOUTUBE_OEMBED_ENDPOINT = 'https://www.youtube.com/oembed';
 const YOUTUBE_ID_PATTERN = /^[A-Za-z0-9_-]{6,}$/;
-const YOUTUBE_METADATA_REFRESH_MS = 24 * 60 * 60 * 1000;
 
 function isValidYouTubeVideoId(value: string | null): boolean {
   return Boolean(value && YOUTUBE_ID_PATTERN.test(value));
@@ -537,15 +535,11 @@ export class ImmersionTrackerService {
   }
 
   async getMediaLibrary(): Promise<MediaLibraryRow[]> {
-    const rows = getMediaLibrary(this.db);
-    this.backfillYoutubeMetadataForLibrary();
-    return rows;
+    return getMediaLibrary(this.db);
   }
 
   async getMediaDetail(videoId: number): Promise<MediaDetailRow | null> {
-    const detail = getMediaDetail(this.db, videoId);
-    this.backfillYoutubeMetadataForVideo(videoId);
-    return detail;
+    return getMediaDetail(this.db, videoId);
   }
 
   async getMediaSessions(videoId: number, limit = 100): Promise<SessionSummaryQueryRow[]> {
@@ -561,12 +555,10 @@ export class ImmersionTrackerService {
   }
 
   async getAnimeLibrary(): Promise<AnimeLibraryRow[]> {
-    this.relinkYoutubeAnimeLibrary();
     return getAnimeLibrary(this.db);
   }
 
   async getAnimeDetail(animeId: number): Promise<AnimeDetailRow | null> {
-    this.relinkYoutubeAnimeLibrary();
     return getAnimeDetail(this.db, animeId);
   }
 
@@ -917,7 +909,6 @@ export class ImmersionTrackerService {
           return;
         }
         upsertYoutubeVideoMetadata(this.db, videoId, metadata);
-        linkYoutubeVideoToAnimeRecord(this.db, videoId, metadata);
         if (metadata.videoTitle?.trim()) {
           updateVideoTitleRecord(this.db, videoId, metadata.videoTitle.trim());
         }
@@ -936,174 +927,6 @@ export class ImmersionTrackerService {
     });
   }
 
-  private backfillYoutubeMetadataForLibrary(): void {
-    const candidate = this.db
-      .prepare(
-        `
-          SELECT
-            v.video_id AS videoId,
-            v.source_url AS sourceUrl
-          FROM imm_videos v
-          JOIN imm_lifetime_media lm ON lm.video_id = v.video_id
-          LEFT JOIN imm_youtube_videos yv ON yv.video_id = v.video_id
-          WHERE
-            v.source_type = ?
-            AND v.source_url IS NOT NULL
-            AND (
-              LOWER(v.source_url) LIKE 'https://www.youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://m.youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://youtu.be/%'
-            )
-            AND (
-              yv.video_id IS NULL
-              OR yv.video_title IS NULL
-              OR yv.channel_name IS NULL
-              OR yv.channel_thumbnail_url IS NULL
-            )
-            AND (
-              yv.fetched_at_ms IS NULL
-              OR yv.fetched_at_ms <= ?
-            )
-          ORDER BY lm.last_watched_ms DESC, v.video_id DESC
-          LIMIT 1
-        `,
-      )
-      .get(
-        SOURCE_TYPE_REMOTE,
-        Date.now() - YOUTUBE_METADATA_REFRESH_MS,
-      ) as { videoId: number; sourceUrl: string | null } | null;
-    if (!candidate?.sourceUrl) {
-      return;
-    }
-    this.captureYoutubeMetadataAsync(candidate.videoId, candidate.sourceUrl);
-  }
-
-  private backfillYoutubeMetadataForVideo(videoId: number): void {
-    const candidate = this.db
-      .prepare(
-        `
-          SELECT
-            v.source_url AS sourceUrl
-          FROM imm_videos v
-          LEFT JOIN imm_youtube_videos yv ON yv.video_id = v.video_id
-          WHERE
-            v.video_id = ?
-            AND v.source_type = ?
-            AND v.source_url IS NOT NULL
-            AND (
-              LOWER(v.source_url) LIKE 'https://www.youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://m.youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://youtu.be/%'
-            )
-            AND (
-              yv.video_id IS NULL
-              OR yv.video_title IS NULL
-              OR yv.channel_name IS NULL
-              OR yv.channel_thumbnail_url IS NULL
-            )
-            AND (
-              yv.fetched_at_ms IS NULL
-              OR yv.fetched_at_ms <= ?
-            )
-        `,
-      )
-      .get(
-        videoId,
-        SOURCE_TYPE_REMOTE,
-        Date.now() - YOUTUBE_METADATA_REFRESH_MS,
-      ) as { sourceUrl: string | null } | null;
-    if (!candidate?.sourceUrl) {
-      return;
-    }
-    this.captureYoutubeMetadataAsync(videoId, candidate.sourceUrl);
-  }
-
-  private relinkYoutubeAnimeLibrary(): void {
-    const candidates = this.db
-      .prepare(
-        `
-          SELECT
-            v.video_id AS videoId,
-            yv.youtube_video_id AS youtubeVideoId,
-            yv.video_url AS videoUrl,
-            yv.video_title AS videoTitle,
-            yv.video_thumbnail_url AS videoThumbnailUrl,
-            yv.channel_id AS channelId,
-            yv.channel_name AS channelName,
-            yv.channel_url AS channelUrl,
-            yv.channel_thumbnail_url AS channelThumbnailUrl,
-            yv.uploader_id AS uploaderId,
-            yv.uploader_url AS uploaderUrl,
-            yv.description AS description,
-            yv.metadata_json AS metadataJson
-          FROM imm_videos v
-          JOIN imm_youtube_videos yv ON yv.video_id = v.video_id
-          LEFT JOIN imm_anime a ON a.anime_id = v.anime_id
-          LEFT JOIN imm_lifetime_media lm ON lm.video_id = v.video_id
-          WHERE
-            v.source_type = ?
-            AND v.source_url IS NOT NULL
-            AND (
-              LOWER(v.source_url) LIKE 'https://www.youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://m.youtube.com/%'
-              OR LOWER(v.source_url) LIKE 'https://youtu.be/%'
-            )
-            AND yv.channel_name IS NOT NULL
-            AND (
-              v.anime_id IS NULL
-              OR a.metadata_json IS NULL
-              OR a.metadata_json NOT LIKE '%"source":"youtube-channel"%'
-              OR a.canonical_title IS NULL
-              OR TRIM(a.canonical_title) != TRIM(yv.channel_name)
-            )
-          ORDER BY lm.last_watched_ms DESC, v.video_id DESC
-        `,
-      )
-      .all(SOURCE_TYPE_REMOTE) as Array<{
-          videoId: number;
-          youtubeVideoId: string | null;
-          videoUrl: string | null;
-          videoTitle: string | null;
-          videoThumbnailUrl: string | null;
-          channelId: string | null;
-          channelName: string | null;
-          channelUrl: string | null;
-          channelThumbnailUrl: string | null;
-          uploaderId: string | null;
-          uploaderUrl: string | null;
-          description: string | null;
-          metadataJson: string | null;
-        }>;
-
-    if (candidates.length === 0) {
-      return;
-    }
-
-    for (const candidate of candidates) {
-      if (!candidate.youtubeVideoId || !candidate.videoUrl) {
-        continue;
-      }
-      linkYoutubeVideoToAnimeRecord(this.db, candidate.videoId, {
-        youtubeVideoId: candidate.youtubeVideoId,
-        videoUrl: candidate.videoUrl,
-        videoTitle: candidate.videoTitle,
-        videoThumbnailUrl: candidate.videoThumbnailUrl,
-        channelId: candidate.channelId,
-        channelName: candidate.channelName,
-        channelUrl: candidate.channelUrl,
-        channelThumbnailUrl: candidate.channelThumbnailUrl,
-        uploaderId: candidate.uploaderId,
-        uploaderUrl: candidate.uploaderUrl,
-        description: candidate.description,
-        metadataJson: candidate.metadataJson,
-      });
-    }
-    rebuildLifetimeSummaryTables(this.db);
-  }
-
   handleMediaChange(mediaPath: string | null, mediaTitle: string | null): void {
     const normalizedPath = normalizeMediaPath(mediaPath);
     const normalizedTitle = normalizeText(mediaTitle);
@@ -1148,14 +971,14 @@ export class ImmersionTrackerService {
       `Starting immersion session for path=${normalizedPath} videoId=${sessionInfo.videoId}`,
     );
     this.startSession(sessionInfo.videoId, sessionInfo.startedAtMs);
-    const youtubeVideoId =
-      sourceType === SOURCE_TYPE_REMOTE ? extractYouTubeVideoId(normalizedPath) : null;
-    if (youtubeVideoId) {
-      void this.ensureYouTubeCoverArt(sessionInfo.videoId, normalizedPath, youtubeVideoId);
-      this.captureYoutubeMetadataAsync(sessionInfo.videoId, normalizedPath);
-    } else {
-      this.captureAnimeMetadataAsync(sessionInfo.videoId, normalizedPath, normalizedTitle || null);
+    if (sourceType === SOURCE_TYPE_REMOTE) {
+      const youtubeVideoId = extractYouTubeVideoId(normalizedPath);
+      if (youtubeVideoId) {
+        void this.ensureYouTubeCoverArt(sessionInfo.videoId, normalizedPath, youtubeVideoId);
+        this.captureYoutubeMetadataAsync(sessionInfo.videoId, normalizedPath);
+      }
     }
+    this.captureAnimeMetadataAsync(sessionInfo.videoId, normalizedPath, normalizedTitle || null);
     this.captureVideoMetadataAsync(sessionInfo.videoId, sourceType, normalizedPath);
   }
 
@@ -1183,7 +1006,6 @@ export class ImmersionTrackerService {
     }
 
     const startMs = secToMs(startSec);
-    const endMs = secToMs(endSec);
     const subtitleKey = `${startMs}:${cleaned}`;
     if (this.recordedSubtitleKeys.has(subtitleKey)) {
       return;
@@ -1197,9 +1019,6 @@ export class ImmersionTrackerService {
     this.sessionState.currentLineIndex += 1;
     this.sessionState.linesSeen += 1;
     this.sessionState.tokensSeen += tokenCount;
-    if (this.sessionState.lastMediaMs === null || endMs > this.sessionState.lastMediaMs) {
-      this.sessionState.lastMediaMs = endMs;
-    }
     this.sessionState.pendingTelemetry = true;
 
     const wordOccurrences = new Map<string, CountedWordOccurrence>();
@@ -1249,8 +1068,8 @@ export class ImmersionTrackerService {
       sessionId: this.sessionState.sessionId,
       videoId: this.sessionState.videoId,
       lineIndex: this.sessionState.currentLineIndex,
-      segmentStartMs: startMs,
-      segmentEndMs: endMs,
+      segmentStartMs: secToMs(startSec),
+      segmentEndMs: secToMs(endSec),
       text: cleaned,
       secondaryText: secondaryText ?? null,
       wordOccurrences: Array.from(wordOccurrences.values()),

src/core/services/immersion-tracker/__tests__/query.test.ts

@@ -280,78 +280,6 @@ test('getAnimeEpisodes falls back to the latest subtitle segment end when sessio
   }
 });
 
-test('getAnimeEpisodes ignores zero-valued session checkpoints and falls back to subtitle progress', () => {
-  const dbPath = makeDbPath();
-  const db = new Database(dbPath);
-
-  try {
-    ensureSchema(db);
-    const stmts = createTrackerPreparedStatements(db);
-    const videoId = getOrCreateVideoRecord(db, 'remote:https://www.youtube.com/watch?v=zero123', {
-      canonicalTitle: 'Zero Checkpoint Stream',
-      sourcePath: null,
-      sourceUrl: 'https://www.youtube.com/watch?v=zero123',
-      sourceType: SOURCE_TYPE_REMOTE,
-    });
-    const animeId = getOrCreateAnimeRecord(db, {
-      parsedTitle: 'Zero Checkpoint Anime',
-      canonicalTitle: 'Zero Checkpoint Anime',
-      anilistId: null,
-      titleRomaji: null,
-      titleEnglish: null,
-      titleNative: null,
-      metadataJson: null,
-    });
-    linkVideoToAnimeRecord(db, videoId, {
-      animeId,
-      parsedBasename: 'watch?v=zero123',
-      parsedTitle: 'Zero Checkpoint Anime',
-      parsedSeason: 1,
-      parsedEpisode: 1,
-      parserSource: 'fallback',
-      parserConfidence: 1,
-      parseMetadataJson: '{"episode":1}',
-    });
-    db.prepare('UPDATE imm_videos SET duration_ms = ? WHERE video_id = ?').run(600_000, videoId);
-
-    const startedAtMs = 1_200_000;
-    const sessionId = startSessionRecord(db, videoId, startedAtMs).sessionId;
-    db.prepare(
-      `
-      UPDATE imm_sessions
-      SET
-        ended_at_ms = ?,
-        status = 2,
-        ended_media_ms = 0,
-        active_watched_ms = ?,
-        LAST_UPDATE_DATE = ?
-      WHERE session_id = ?
-      `,
-    ).run(startedAtMs + 30_000, 180_000, startedAtMs + 30_000, sessionId);
-    stmts.eventInsertStmt.run(
-      sessionId,
-      startedAtMs + 29_000,
-      EVENT_SUBTITLE_LINE,
-      1,
-      170_000,
-      185_000,
-      4,
-      0,
-      '{"line":"stream progress"}',
-      startedAtMs + 29_000,
-      startedAtMs + 29_000,
-    );
-
-    const [episode] = getAnimeEpisodes(db, animeId);
-    assert.ok(episode);
-    assert.equal(episode?.endedMediaMs, 185_000);
-    assert.equal(episode?.durationMs, 600_000);
-  } finally {
-    db.close();
-    cleanupDbPath(dbPath);
-  }
-});
-
 test('getSessionTimeline returns the full session when no limit is provided', () => {
   const dbPath = makeDbPath();
   const db = new Database(dbPath);
@@ -2846,200 +2774,3 @@ test('deleteSession rebuilds word and kanji aggregates from retained subtitle li
     cleanupDbPath(dbPath);
   }
 });
-
-test('deleteSession removes zero-session media from library and trends', () => {
-  const dbPath = makeDbPath();
-  const db = new Database(dbPath);
-
-  try {
-    ensureSchema(db);
-
-    const animeId = getOrCreateAnimeRecord(db, {
-      parsedTitle: 'Delete Me Anime',
-      canonicalTitle: 'Delete Me Anime',
-      anilistId: 404_404,
-      titleRomaji: 'Delete Me Anime',
-      titleEnglish: 'Delete Me Anime',
-      titleNative: 'Delete Me Anime',
-      metadataJson: null,
-    });
-    const videoId = getOrCreateVideoRecord(db, 'local:/tmp/delete-last-session.mkv', {
-      canonicalTitle: 'Delete Last Session',
-      sourcePath: '/tmp/delete-last-session.mkv',
-      sourceUrl: null,
-      sourceType: SOURCE_TYPE_LOCAL,
-    });
-    linkVideoToAnimeRecord(db, videoId, {
-      animeId,
-      parsedBasename: 'Delete Last Session',
-      parsedTitle: 'Delete Me Anime',
-      parsedSeason: 1,
-      parsedEpisode: 1,
-      parserSource: 'fallback',
-      parserConfidence: 1,
-      parseMetadataJson: '{"episode":1}',
-    });
-
-    const startedAtMs = 9_000_000;
-    const endedAtMs = startedAtMs + 120_000;
-    const rollupDay = Math.floor(startedAtMs / 86_400_000);
-    const rollupMonth = 197001;
-    const { sessionId } = startSessionRecord(db, videoId, startedAtMs);
-
-    db.prepare(
-      `
-      UPDATE imm_sessions
-      SET
-        ended_at_ms = ?,
-        ended_media_ms = ?,
-        total_watched_ms = ?,
-        active_watched_ms = ?,
-        lines_seen = ?,
-        tokens_seen = ?,
-        cards_mined = ?,
-        LAST_UPDATE_DATE = ?
-      WHERE session_id = ?
-      `,
-    ).run(endedAtMs, 120000, 120000, 120000, 12, 120, 3, endedAtMs, sessionId);
-
-    db.prepare(
-      `
-      INSERT INTO imm_lifetime_applied_sessions (
-        session_id,
-        applied_at_ms,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES (?, ?, ?, ?)
-      `,
-    ).run(sessionId, endedAtMs, endedAtMs, endedAtMs);
-    db.prepare(
-      `
-      INSERT INTO imm_lifetime_media (
-        video_id,
-        total_sessions,
-        total_active_ms,
-        total_cards,
-        total_lines_seen,
-        total_tokens_seen,
-        completed,
-        first_watched_ms,
-        last_watched_ms,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-      `,
-    ).run(videoId, 1, 120_000, 3, 12, 120, 0, startedAtMs, endedAtMs, endedAtMs, endedAtMs);
-    db.prepare(
-      `
-      INSERT INTO imm_lifetime_anime (
-        anime_id,
-        total_sessions,
-        total_active_ms,
-        total_cards,
-        total_lines_seen,
-        total_tokens_seen,
-        episodes_started,
-        episodes_completed,
-        first_watched_ms,
-        last_watched_ms,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-      `,
-    ).run(animeId, 1, 120000, 3, 12, 120, 1, 0, startedAtMs, endedAtMs, endedAtMs, endedAtMs);
-    db.prepare(
-      `
-      UPDATE imm_lifetime_global
-      SET
-        total_sessions = 1,
-        total_active_ms = 120000,
-        total_cards = 3,
-        active_days = 1,
-        episodes_started = 1,
-        episodes_completed = 0,
-        anime_completed = 0,
-        last_rebuilt_ms = ?,
-        LAST_UPDATE_DATE = ?
-      WHERE global_id = 1
-      `,
-    ).run(endedAtMs, endedAtMs);
-    db.prepare(
-      `
-      INSERT INTO imm_daily_rollups (
-        rollup_day,
-        video_id,
-        total_sessions,
-        total_active_min,
-        total_lines_seen,
-        total_tokens_seen,
-        total_cards,
-        cards_per_hour,
-        tokens_per_min,
-        lookup_hit_rate,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-      `,
-    ).run(rollupDay, videoId, 1, 2, 12, 120, 3, 90, 60, null, endedAtMs, endedAtMs);
-    db.prepare(
-      `
-      INSERT INTO imm_monthly_rollups (
-        rollup_month,
-        video_id,
-        total_sessions,
-        total_active_min,
-        total_lines_seen,
-        total_tokens_seen,
-        total_cards,
-        CREATED_DATE,
-        LAST_UPDATE_DATE
-      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
-      `,
-    ).run(rollupMonth, videoId, 1, 2, 12, 120, 3, endedAtMs, endedAtMs);
-
-    deleteSession(db, sessionId);
-
-    assert.deepEqual(getMediaLibrary(db), []);
-    assert.equal(getMediaDetail(db, videoId) ?? null, null);
-    assert.deepEqual(getAnimeLibrary(db), []);
-    assert.equal(getAnimeDetail(db, animeId) ?? null, null);
-
-    const trends = getTrendsDashboard(db, 'all', 'day');
-    assert.deepEqual(trends.activity.watchTime, []);
-    assert.deepEqual(trends.activity.sessions, []);
-
-    const dailyRollups = getDailyRollups(db, 30);
-    const monthlyRollups = getMonthlyRollups(db, 30);
-    assert.deepEqual(dailyRollups, []);
-    assert.deepEqual(monthlyRollups, []);
-
-    const lifetimeMediaCount = Number(
-      (
-        db.prepare('SELECT COUNT(*) AS total FROM imm_lifetime_media WHERE video_id = ?').get(
-          videoId,
-        ) as { total: number }
-      ).total,
-    );
-    const lifetimeAnimeCount = Number(
-      (
-        db.prepare('SELECT COUNT(*) AS total FROM imm_lifetime_anime WHERE anime_id = ?').get(
-          animeId,
-        ) as { total: number }
-      ).total,
-    );
-    const appliedSessionCount = Number(
-      (
-        db
-          .prepare('SELECT COUNT(*) AS total FROM imm_lifetime_applied_sessions WHERE session_id = ?')
-          .get(sessionId) as { total: number }
-      ).total,
-    );
-
-    assert.equal(lifetimeMediaCount, 0);
-    assert.equal(lifetimeAnimeCount, 0);
-    assert.equal(appliedSessionCount, 0);
-  } finally {
-    db.close();
-    cleanupDbPath(dbPath);
-  }
-});

src/core/services/immersion-tracker/lifetime.ts

@@ -134,49 +134,6 @@ function resetLifetimeSummaries(db: DatabaseSync, nowMs: number): void {
   ).run(nowMs, nowMs);
 }
 
-function rebuildLifetimeSummariesInternal(
-  db: DatabaseSync,
-  rebuiltAtMs: number,
-): LifetimeRebuildSummary {
-  const sessions = db
-    .prepare(
-      `
-      SELECT
-        session_id AS sessionId,
-        video_id AS videoId,
-        started_at_ms AS startedAtMs,
-        ended_at_ms AS endedAtMs,
-        total_watched_ms AS totalWatchedMs,
-        active_watched_ms AS activeWatchedMs,
-        lines_seen AS linesSeen,
-        tokens_seen AS tokensSeen,
-        cards_mined AS cardsMined,
-        lookup_count AS lookupCount,
-        lookup_hits AS lookupHits,
-        yomitan_lookup_count AS yomitanLookupCount,
-        pause_count AS pauseCount,
-        pause_ms AS pauseMs,
-        seek_forward_count AS seekForwardCount,
-        seek_backward_count AS seekBackwardCount,
-        media_buffer_events AS mediaBufferEvents
-      FROM imm_sessions
-      WHERE ended_at_ms IS NOT NULL
-      ORDER BY started_at_ms ASC, session_id ASC
-      `,
-    )
-    .all() as RetainedSessionRow[];
-
-  resetLifetimeSummaries(db, rebuiltAtMs);
-  for (const session of sessions) {
-    applySessionLifetimeSummary(db, toRebuildSessionState(session), session.endedAtMs);
-  }
-
-  return {
-    appliedSessions: sessions.length,
-    rebuiltAtMs,
-  };
-}
-
 function toRebuildSessionState(row: RetainedSessionRow): SessionState {
   return {
     sessionId: row.sessionId,
@@ -525,22 +482,50 @@ export function applySessionLifetimeSummary(
 
 export function rebuildLifetimeSummaries(db: DatabaseSync): LifetimeRebuildSummary {
   const rebuiltAtMs = Date.now();
+  const sessions = db
+    .prepare(
+      `
+      SELECT
+        session_id AS sessionId,
+        video_id AS videoId,
+        started_at_ms AS startedAtMs,
+        ended_at_ms AS endedAtMs,
+        total_watched_ms AS totalWatchedMs,
+        active_watched_ms AS activeWatchedMs,
+        lines_seen AS linesSeen,
+        tokens_seen AS tokensSeen,
+        cards_mined AS cardsMined,
+        lookup_count AS lookupCount,
+        lookup_hits AS lookupHits,
+        yomitan_lookup_count AS yomitanLookupCount,
+        pause_count AS pauseCount,
+        pause_ms AS pauseMs,
+        seek_forward_count AS seekForwardCount,
+        seek_backward_count AS seekBackwardCount,
+        media_buffer_events AS mediaBufferEvents
+      FROM imm_sessions
+      WHERE ended_at_ms IS NOT NULL
+      ORDER BY started_at_ms ASC, session_id ASC
+      `,
+    )
+    .all() as RetainedSessionRow[];
+
   db.exec('BEGIN');
   try {
-    const summary = rebuildLifetimeSummariesInTransaction(db, rebuiltAtMs);
+    resetLifetimeSummaries(db, rebuiltAtMs);
+    for (const session of sessions) {
+      applySessionLifetimeSummary(db, toRebuildSessionState(session), session.endedAtMs);
+    }
     db.exec('COMMIT');
-    return summary;
   } catch (error) {
     db.exec('ROLLBACK');
     throw error;
   }
-}
 
-export function rebuildLifetimeSummariesInTransaction(
-  db: DatabaseSync,
-  rebuiltAtMs = Date.now(),
-): LifetimeRebuildSummary {
-  return rebuildLifetimeSummariesInternal(db, rebuiltAtMs);
+  return {
+    appliedSessions: sessions.length,
+    rebuiltAtMs,
+  };
 }
 
 export function reconcileStaleActiveSessions(db: DatabaseSync): number {

src/core/services/immersion-tracker/maintenance.ts

@@ -113,14 +113,6 @@ function setLastRollupSampleMs(db: DatabaseSync, sampleMs: number): void {
   ).run(ROLLUP_STATE_KEY, sampleMs);
 }
 
-function resetRollups(db: DatabaseSync): void {
-  db.exec(`
-    DELETE FROM imm_daily_rollups;
-    DELETE FROM imm_monthly_rollups;
-  `);
-  setLastRollupSampleMs(db, ZERO_ID);
-}
-
 function upsertDailyRollupsForGroups(
   db: DatabaseSync,
   groups: Array<{ rollupDay: number; videoId: number }>,
@@ -289,20 +281,8 @@ function dedupeGroups<T extends { rollupDay?: number; rollupMonth?: number; vide
 }
 
 export function runRollupMaintenance(db: DatabaseSync, forceRebuild = false): void {
-  if (forceRebuild) {
-    db.exec('BEGIN IMMEDIATE');
-    try {
-      rebuildRollupsInTransaction(db);
-      db.exec('COMMIT');
-    } catch (error) {
-      db.exec('ROLLBACK');
-      throw error;
-    }
-    return;
-  }
-
   const rollupNowMs = Date.now();
-  const lastRollupSampleMs = getLastRollupSampleMs(db);
+  const lastRollupSampleMs = forceRebuild ? ZERO_ID : getLastRollupSampleMs(db);
 
   const maxSampleRow = db
     .prepare('SELECT MAX(sample_ms) AS maxSampleMs FROM imm_session_telemetry')
@@ -344,41 +324,6 @@ export function runRollupMaintenance(db: DatabaseSync, forceRebuild = false): vo
   }
 }
 
-export function rebuildRollupsInTransaction(db: DatabaseSync): void {
-  const rollupNowMs = Date.now();
-  const maxSampleRow = db
-    .prepare('SELECT MAX(sample_ms) AS maxSampleMs FROM imm_session_telemetry')
-    .get() as unknown as RollupTelemetryResult | null;
-
-  resetRollups(db);
-  if (!maxSampleRow?.maxSampleMs) {
-    return;
-  }
-
-  const affectedGroups = getAffectedRollupGroups(db, ZERO_ID);
-  if (affectedGroups.length === 0) {
-    setLastRollupSampleMs(db, Number(maxSampleRow.maxSampleMs));
-    return;
-  }
-
-  const dailyGroups = dedupeGroups(
-    affectedGroups.map((group) => ({
-      rollupDay: group.rollupDay,
-      videoId: group.videoId,
-    })),
-  );
-  const monthlyGroups = dedupeGroups(
-    affectedGroups.map((group) => ({
-      rollupMonth: group.rollupMonth,
-      videoId: group.videoId,
-    })),
-  );
-
-  upsertDailyRollupsForGroups(db, dailyGroups, rollupNowMs);
-  upsertMonthlyRollupsForGroups(db, monthlyGroups, rollupNowMs);
-  setLastRollupSampleMs(db, Number(maxSampleRow.maxSampleMs));
-}
-
 export function runOptimizeMaintenance(db: DatabaseSync): void {
   db.exec('PRAGMA optimize');
 }

src/core/services/immersion-tracker/query.ts

@@ -31,8 +31,6 @@ import type {
   VocabularyStatsRow,
 } from './types';
 import { buildCoverBlobReference, normalizeCoverBlobBytes } from './storage';
-import { rebuildLifetimeSummariesInTransaction } from './lifetime';
-import { rebuildRollupsInTransaction } from './maintenance';
 import { PartOfSpeech, type MergedToken } from '../../../types';
 import { shouldExcludeTokenFromVocabularyPersistence } from '../tokenizer/annotation-stage';
 import { deriveStoredPartOfSpeech } from '../tokenizer/part-of-speech';
@@ -1748,7 +1746,7 @@ export function getAnimeEpisodes(db: DatabaseSync, animeId: number): AnimeEpisod
       v.duration_ms AS durationMs,
       (
         SELECT COALESCE(
-          NULLIF(s_recent.ended_media_ms, 0),
+          s_recent.ended_media_ms,
           (
             SELECT MAX(line.segment_end_ms)
             FROM imm_subtitle_lines line
@@ -2469,8 +2467,6 @@ export function deleteSession(db: DatabaseSync, sessionId: number): void {
   try {
     deleteSessionsByIds(db, sessionIds);
     refreshLexicalAggregates(db, affectedWordIds, affectedKanjiIds);
-    rebuildLifetimeSummariesInTransaction(db);
-    rebuildRollupsInTransaction(db);
     db.exec('COMMIT');
   } catch (error) {
     db.exec('ROLLBACK');
@@ -2487,8 +2483,6 @@ export function deleteSessions(db: DatabaseSync, sessionIds: number[]): void {
   try {
     deleteSessionsByIds(db, sessionIds);
     refreshLexicalAggregates(db, affectedWordIds, affectedKanjiIds);
-    rebuildLifetimeSummariesInTransaction(db);
-    rebuildRollupsInTransaction(db);
     db.exec('COMMIT');
   } catch (error) {
     db.exec('ROLLBACK');
@@ -2525,8 +2519,6 @@ export function deleteVideo(db: DatabaseSync, videoId: number): void {
     cleanupUnusedCoverArtBlobHash(db, artRow?.coverBlobHash ?? null);
     db.prepare('DELETE FROM imm_videos WHERE video_id = ?').run(videoId);
     refreshLexicalAggregates(db, affectedWordIds, affectedKanjiIds);
-    rebuildLifetimeSummariesInTransaction(db);
-    rebuildRollupsInTransaction(db);
     db.exec('COMMIT');
   } catch (error) {
     db.exec('ROLLBACK');

src/core/services/immersion-tracker/storage-session.test.ts

@@ -15,14 +15,8 @@ import {
   getOrCreateAnimeRecord,
   getOrCreateVideoRecord,
   linkVideoToAnimeRecord,
-  linkYoutubeVideoToAnimeRecord,
 } from './storage';
-import {
-  EVENT_SUBTITLE_LINE,
-  SESSION_STATUS_ENDED,
-  SOURCE_TYPE_LOCAL,
-  SOURCE_TYPE_REMOTE,
-} from './types';
+import { EVENT_SUBTITLE_LINE, SESSION_STATUS_ENDED, SOURCE_TYPE_LOCAL } from './types';
 
 function makeDbPath(): string {
   const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'subminer-imm-storage-session-'));
@@ -823,123 +817,6 @@ test('anime rows are reused by normalized parsed title and upgraded with AniList
   }
 });
 
-test('youtube videos can be regrouped under a shared channel anime identity', () => {
-  const dbPath = makeDbPath();
-  const db = new Database(dbPath);
-
-  try {
-    ensureSchema(db);
-
-    const firstVideoId = getOrCreateVideoRecord(
-      db,
-      'remote:https://www.youtube.com/watch?v=video-1',
-      {
-        canonicalTitle: 'watch?v video-1',
-        sourcePath: null,
-        sourceUrl: 'https://www.youtube.com/watch?v=video-1',
-        sourceType: SOURCE_TYPE_REMOTE,
-      },
-    );
-    const secondVideoId = getOrCreateVideoRecord(
-      db,
-      'remote:https://www.youtube.com/watch?v=video-2',
-      {
-        canonicalTitle: 'watch?v video-2',
-        sourcePath: null,
-        sourceUrl: 'https://www.youtube.com/watch?v=video-2',
-        sourceType: SOURCE_TYPE_REMOTE,
-      },
-    );
-
-    const firstAnimeId = getOrCreateAnimeRecord(db, {
-      parsedTitle: 'watch?v video-1',
-      canonicalTitle: 'watch?v video-1',
-      anilistId: null,
-      titleRomaji: null,
-      titleEnglish: null,
-      titleNative: null,
-      metadataJson: null,
-    });
-    linkVideoToAnimeRecord(db, firstVideoId, {
-      animeId: firstAnimeId,
-      parsedBasename: null,
-      parsedTitle: 'watch?v video-1',
-      parsedSeason: null,
-      parsedEpisode: null,
-      parserSource: 'fallback',
-      parserConfidence: 0.2,
-      parseMetadataJson: '{"source":"fallback"}',
-    });
-
-    const secondAnimeId = getOrCreateAnimeRecord(db, {
-      parsedTitle: 'watch?v video-2',
-      canonicalTitle: 'watch?v video-2',
-      anilistId: null,
-      titleRomaji: null,
-      titleEnglish: null,
-      titleNative: null,
-      metadataJson: null,
-    });
-    linkVideoToAnimeRecord(db, secondVideoId, {
-      animeId: secondAnimeId,
-      parsedBasename: null,
-      parsedTitle: 'watch?v video-2',
-      parsedSeason: null,
-      parsedEpisode: null,
-      parserSource: 'fallback',
-      parserConfidence: 0.2,
-      parseMetadataJson: '{"source":"fallback"}',
-    });
-
-    linkYoutubeVideoToAnimeRecord(db, firstVideoId, {
-      youtubeVideoId: 'video-1',
-      videoUrl: 'https://www.youtube.com/watch?v=video-1',
-      videoTitle: 'Video One',
-      videoThumbnailUrl: 'https://i.ytimg.com/vi/video-1/hqdefault.jpg',
-      channelId: 'UC123',
-      channelName: 'Channel Name',
-      channelUrl: 'https://www.youtube.com/channel/UC123',
-      channelThumbnailUrl: 'https://yt3.googleusercontent.com/channel-123=s176-c-k-c0x00ffffff-no-rj',
-      uploaderId: '@channelname',
-      uploaderUrl: 'https://www.youtube.com/@channelname',
-      description: null,
-      metadataJson: '{"id":"video-1"}',
-    });
-    linkYoutubeVideoToAnimeRecord(db, secondVideoId, {
-      youtubeVideoId: 'video-2',
-      videoUrl: 'https://www.youtube.com/watch?v=video-2',
-      videoTitle: 'Video Two',
-      videoThumbnailUrl: 'https://i.ytimg.com/vi/video-2/hqdefault.jpg',
-      channelId: 'UC123',
-      channelName: 'Channel Name',
-      channelUrl: 'https://www.youtube.com/channel/UC123',
-      channelThumbnailUrl: 'https://yt3.googleusercontent.com/channel-123=s176-c-k-c0x00ffffff-no-rj',
-      uploaderId: '@channelname',
-      uploaderUrl: 'https://www.youtube.com/@channelname',
-      description: null,
-      metadataJson: '{"id":"video-2"}',
-    });
-
-    const animeRows = db.prepare('SELECT anime_id, canonical_title FROM imm_anime').all() as Array<{
-      anime_id: number;
-      canonical_title: string;
-    }>;
-    const videoRows = db
-      .prepare('SELECT video_id, anime_id, parsed_title FROM imm_videos ORDER BY video_id ASC')
-      .all() as Array<{ video_id: number; anime_id: number | null; parsed_title: string | null }>;
-
-    const channelAnimeRows = animeRows.filter((row) => row.canonical_title === 'Channel Name');
-    assert.equal(channelAnimeRows.length, 1);
-    assert.equal(videoRows[0]?.anime_id, channelAnimeRows[0]?.anime_id);
-    assert.equal(videoRows[1]?.anime_id, channelAnimeRows[0]?.anime_id);
-    assert.equal(videoRows[0]?.parsed_title, 'Channel Name');
-    assert.equal(videoRows[1]?.parsed_title, 'Channel Name');
-  } finally {
-    db.close();
-    cleanupDbPath(dbPath);
-  }
-});
-
 test('start/finalize session updates ended_at and status', () => {
   const dbPath = makeDbPath();
   const db = new Database(dbPath);

src/core/services/immersion-tracker/storage.ts

@@ -39,41 +39,6 @@ export interface VideoAnimeLinkInput {
   parseMetadataJson: string | null;
 }
 
-function buildYoutubeChannelAnimeIdentity(metadata: YoutubeVideoMetadata): {
-  parsedTitle: string;
-  canonicalTitle: string;
-  metadataJson: string;
-} | null {
-  const channelId = metadata.channelId?.trim() || null;
-  const channelUrl = metadata.channelUrl?.trim() || null;
-  const channelName = metadata.channelName?.trim() || null;
-  const uploaderId = metadata.uploaderId?.trim() || null;
-  const videoTitle = metadata.videoTitle?.trim() || null;
-
-  const parsedTitle = channelId
-    ? `youtube-channel:${channelId}`
-    : channelUrl
-      ? `youtube-channel-url:${channelUrl}`
-      : channelName
-        ? `youtube-channel-name:${channelName}`
-        : null;
-  if (!parsedTitle) {
-    return null;
-  }
-
-  return {
-    parsedTitle,
-    canonicalTitle: channelName || uploaderId || videoTitle || parsedTitle,
-    metadataJson: JSON.stringify({
-      source: 'youtube-channel',
-      channelId,
-      channelUrl,
-      channelName,
-      uploaderId,
-    }),
-  };
-}
-
 const COVER_BLOB_REFERENCE_PREFIX = '__subminer_cover_blob_ref__:';
 const WAL_JOURNAL_SIZE_LIMIT_BYTES = 64 * 1024 * 1024;
 
@@ -474,38 +439,6 @@ export function linkVideoToAnimeRecord(
   );
 }
 
-export function linkYoutubeVideoToAnimeRecord(
-  db: DatabaseSync,
-  videoId: number,
-  metadata: YoutubeVideoMetadata,
-): number | null {
-  const identity = buildYoutubeChannelAnimeIdentity(metadata);
-  if (!identity) {
-    return null;
-  }
-
-  const animeId = getOrCreateAnimeRecord(db, {
-    parsedTitle: identity.parsedTitle,
-    canonicalTitle: identity.canonicalTitle,
-    anilistId: null,
-    titleRomaji: null,
-    titleEnglish: null,
-    titleNative: null,
-    metadataJson: identity.metadataJson,
-  });
-  linkVideoToAnimeRecord(db, videoId, {
-    animeId,
-    parsedBasename: null,
-    parsedTitle: identity.canonicalTitle,
-    parsedSeason: null,
-    parsedEpisode: null,
-    parserSource: 'youtube',
-    parserConfidence: 1,
-    parseMetadataJson: identity.metadataJson,
-  });
-  return animeId;
-}
-
 function migrateLegacyAnimeMetadata(db: DatabaseSync): void {
   addColumnIfMissing(db, 'imm_videos', 'anime_id', 'INTEGER REFERENCES imm_anime(anime_id)');
   addColumnIfMissing(db, 'imm_videos', 'parsed_basename', 'TEXT');

src/core/services/ipc-command.test.ts

@@ -15,7 +15,6 @@ function createOptions(overrides: Partial<Parameters<typeof handleMpvCommandFrom
       PLAY_NEXT_SUBTITLE: '__play-next-subtitle',
       SHIFT_SUB_DELAY_TO_NEXT_SUBTITLE_START: '__sub-delay-next-line',
       SHIFT_SUB_DELAY_TO_PREVIOUS_SUBTITLE_START: '__sub-delay-prev-line',
-      YOUTUBE_PICKER_OPEN: '__youtube-picker-open',
     },
     triggerSubsyncFromConfig: () => {
       calls.push('subsync');
@@ -23,9 +22,6 @@ function createOptions(overrides: Partial<Parameters<typeof handleMpvCommandFrom
     openRuntimeOptionsPalette: () => {
       calls.push('runtime-options');
     },
-    openYoutubeTrackPicker: () => {
-      calls.push('youtube-picker');
-    },
     runtimeOptionsCycle: () => ({ ok: true }),
     showMpvOsd: (text) => {
       osd.push(text);
@@ -102,14 +98,6 @@ test('handleMpvCommandFromIpc dispatches special subtitle-delay shift command',
   assert.deepEqual(osd, []);
 });
 
-test('handleMpvCommandFromIpc dispatches special youtube picker open command', () => {
-  const { options, calls, sentCommands, osd } = createOptions();
-  handleMpvCommandFromIpc(['__youtube-picker-open'], options);
-  assert.deepEqual(calls, ['youtube-picker']);
-  assert.deepEqual(sentCommands, []);
-  assert.deepEqual(osd, []);
-});
-
 test('handleMpvCommandFromIpc does not forward commands while disconnected', () => {
   const { options, sentCommands, osd } = createOptions({
     isMpvConnected: () => false,

src/core/services/ipc-command.ts

@@ -14,11 +14,9 @@ export interface HandleMpvCommandFromIpcOptions {
     PLAY_NEXT_SUBTITLE: string;
     SHIFT_SUB_DELAY_TO_NEXT_SUBTITLE_START: string;
     SHIFT_SUB_DELAY_TO_PREVIOUS_SUBTITLE_START: string;
-    YOUTUBE_PICKER_OPEN: string;
   };
   triggerSubsyncFromConfig: () => void;
   openRuntimeOptionsPalette: () => void;
-  openYoutubeTrackPicker: () => void | Promise<void>;
   runtimeOptionsCycle: (id: RuntimeOptionId, direction: 1 | -1) => RuntimeOptionApplyResult;
   showMpvOsd: (text: string) => void;
   mpvReplaySubtitle: () => void;
@@ -92,11 +90,6 @@ export function handleMpvCommandFromIpc(
     return;
   }
 
-  if (first === options.specialCommands.YOUTUBE_PICKER_OPEN) {
-    void options.openYoutubeTrackPicker();
-    return;
-  }
-
   if (
     first === options.specialCommands.SHIFT_SUB_DELAY_TO_NEXT_SUBTITLE_START ||
     first === options.specialCommands.SHIFT_SUB_DELAY_TO_PREVIOUS_SUBTITLE_START

src/core/services/overlay-visibility.test.ts

@@ -200,44 +200,6 @@ test('Windows visible overlay stays click-through and does not steal focus while
   assert.ok(!calls.includes('focus'));
 });
 
-test('visible overlay stays hidden while a modal window is active', () => {
-  const { window, calls } = createMainWindowRecorder();
-  const tracker: WindowTrackerStub = {
-    isTracking: () => true,
-    getGeometry: () => ({ x: 0, y: 0, width: 1280, height: 720 }),
-  };
-
-  updateVisibleOverlayVisibility({
-    visibleOverlayVisible: true,
-    modalActive: true,
-    mainWindow: window as never,
-    windowTracker: tracker as never,
-    trackerNotReadyWarningShown: false,
-    setTrackerNotReadyWarningShown: () => {},
-    updateVisibleOverlayBounds: () => {
-      calls.push('update-bounds');
-    },
-    ensureOverlayWindowLevel: () => {
-      calls.push('ensure-level');
-    },
-    syncPrimaryOverlayWindowLayer: () => {
-      calls.push('sync-layer');
-    },
-    enforceOverlayLayerOrder: () => {
-      calls.push('enforce-order');
-    },
-    syncOverlayShortcuts: () => {
-      calls.push('sync-shortcuts');
-    },
-    isMacOSPlatform: true,
-    isWindowsPlatform: false,
-  } as never);
-
-  assert.ok(calls.includes('hide'));
-  assert.ok(!calls.includes('show'));
-  assert.ok(!calls.includes('update-bounds'));
-});
-
 test('macOS tracked visible overlay stays visible without passively stealing focus', () => {
   const { window, calls } = createMainWindowRecorder();
   const tracker: WindowTrackerStub = {

src/core/services/overlay-visibility.ts

@@ -4,7 +4,6 @@ import { WindowGeometry } from '../../types';
 
 export function updateVisibleOverlayVisibility(args: {
   visibleOverlayVisible: boolean;
-  modalActive?: boolean;
   forceMousePassthrough?: boolean;
   mainWindow: BrowserWindow | null;
   windowTracker: BaseWindowTracker | null;
@@ -29,12 +28,6 @@ export function updateVisibleOverlayVisibility(args: {
 
   const mainWindow = args.mainWindow;
 
-  if (args.modalActive) {
-    mainWindow.hide();
-    args.syncOverlayShortcuts();
-    return;
-  }
-
   const showPassiveVisibleOverlay = (): void => {
     const forceMousePassthrough = args.forceMousePassthrough === true;
     if (args.isWindowsPlatform || forceMousePassthrough) {

src/core/services/tokenizer.test.ts

@@ -3893,172 +3893,6 @@ test('tokenizeSubtitle keeps frequency for content-led merged token with trailin
   assert.equal(result.tokens?.[0]?.frequencyRank, 5468);
 });
 
-test('tokenizeSubtitle clears all annotations for explanatory contrast endings', async () => {
-  const result = await tokenizeSubtitle(
-    '最近辛いものが続いとるんですけど',
-    makeDepsFromYomitanTokens(
-      [
-        { surface: '最近', reading: 'さいきん', headword: '最近' },
-        { surface: '辛い', reading: 'つらい', headword: '辛い' },
-        { surface: 'もの', reading: 'もの', headword: 'もの' },
-        { surface: 'が', reading: 'が', headword: 'が' },
-        { surface: '続いとる', reading: 'つづいとる', headword: '続く' },
-        { surface: 'んですけど', reading: 'んですけど', headword: 'ん' },
-      ],
-      {
-        getFrequencyDictionaryEnabled: () => true,
-        getFrequencyRank: (text) =>
-          text === '最近' ? 120 : text === '辛い' ? 800 : text === '続く' ? 240 : 77,
-        getJlptLevel: (text) =>
-          text === '最近' ? 'N4' : text === '辛い' ? 'N2' : text === '続く' ? 'N4' : null,
-        isKnownWord: (text) => text === '最近',
-        getMinSentenceWordsForNPlusOne: () => 1,
-        tokenizeWithMecab: async () => [
-          {
-            headword: '最近',
-            surface: '最近',
-            reading: 'サイキン',
-            startPos: 0,
-            endPos: 2,
-            partOfSpeech: PartOfSpeech.noun,
-            pos1: '名詞',
-            pos2: '副詞可能',
-            isMerged: false,
-            isKnown: false,
-            isNPlusOneTarget: false,
-          },
-          {
-            headword: '辛い',
-            surface: '辛い',
-            reading: 'ツライ',
-            startPos: 2,
-            endPos: 4,
-            partOfSpeech: PartOfSpeech.i_adjective,
-            pos1: '形容詞',
-            pos2: '自立',
-            isMerged: false,
-            isKnown: false,
-            isNPlusOneTarget: false,
-          },
-          {
-            headword: 'もの',
-            surface: 'もの',
-            reading: 'モノ',
-            startPos: 4,
-            endPos: 6,
-            partOfSpeech: PartOfSpeech.noun,
-            pos1: '名詞',
-            pos2: '一般',
-            isMerged: false,
-            isKnown: false,
-            isNPlusOneTarget: false,
-          },
-          {
-            headword: 'が',
-            surface: 'が',
-            reading: 'ガ',
-            startPos: 6,
-            endPos: 7,
-            partOfSpeech: PartOfSpeech.particle,
-            pos1: '助詞',
-            pos2: '格助詞',
-            isMerged: false,
-            isKnown: false,
-            isNPlusOneTarget: false,
-          },
-          {
-            headword: '続く',
-            surface: '続いとる',
-            reading: 'ツヅイトル',
-            startPos: 7,
-            endPos: 11,
-            partOfSpeech: PartOfSpeech.verb,
-            pos1: '動詞',
-            pos2: '自立',
-            isMerged: false,
-            isKnown: false,
-            isNPlusOneTarget: false,
-          },
-          {
-            headword: 'ん',
-            surface: 'んですけど',
-            reading: 'ンデスケド',
-            startPos: 11,
-            endPos: 16,
-            partOfSpeech: PartOfSpeech.other,
-            pos1: '名詞|助動詞|助詞',
-            pos2: '非自立',
-            isMerged: false,
-            isKnown: false,
-            isNPlusOneTarget: false,
-          },
-        ],
-      },
-    ),
-  );
-
-  assert.deepEqual(
-    result.tokens?.map((token) => ({
-      surface: token.surface,
-      headword: token.headword,
-      isKnown: token.isKnown,
-      isNPlusOneTarget: token.isNPlusOneTarget,
-      frequencyRank: token.frequencyRank,
-      jlptLevel: token.jlptLevel,
-    })),
-    [
-      {
-        surface: '最近',
-        headword: '最近',
-        isKnown: true,
-        isNPlusOneTarget: false,
-        frequencyRank: 120,
-        jlptLevel: 'N4',
-      },
-      {
-        surface: '辛い',
-        headword: '辛い',
-        isKnown: false,
-        isNPlusOneTarget: false,
-        frequencyRank: 800,
-        jlptLevel: 'N2',
-      },
-      {
-        surface: 'もの',
-        headword: 'もの',
-        isKnown: false,
-        isNPlusOneTarget: false,
-        frequencyRank: 77,
-        jlptLevel: undefined,
-      },
-      {
-        surface: 'が',
-        headword: 'が',
-        isKnown: false,
-        isNPlusOneTarget: false,
-        frequencyRank: undefined,
-        jlptLevel: undefined,
-      },
-      {
-        surface: '続いとる',
-        headword: '続く',
-        isKnown: false,
-        isNPlusOneTarget: false,
-        frequencyRank: 240,
-        jlptLevel: 'N4',
-      },
-      {
-        surface: 'んですけど',
-        headword: 'ん',
-        isKnown: false,
-        isNPlusOneTarget: false,
-        frequencyRank: undefined,
-        jlptLevel: undefined,
-      },
-    ],
-  );
-});
-
 test('tokenizeSubtitle excludes default non-independent pos2 from N+1 when JLPT/frequency are disabled', async () => {
   let mecabCalls = 0;
   const result = await tokenizeSubtitle(

src/core/services/tokenizer/annotation-stage.test.ts

@@ -246,18 +246,6 @@ test('shouldExcludeTokenFromSubtitleAnnotations excludes explanatory pondering e
   assert.equal(shouldExcludeTokenFromSubtitleAnnotations(token), true);
 });
 
-test('shouldExcludeTokenFromSubtitleAnnotations excludes explanatory contrast endings', () => {
-  const token = makeToken({
-    surface: 'んですけど',
-    headword: 'ん',
-    reading: 'ンデスケド',
-    pos1: '名詞|助動詞|助詞',
-    pos2: '非自立',
-  });
-
-  assert.equal(shouldExcludeTokenFromSubtitleAnnotations(token), true);
-});
-
 test('shouldExcludeTokenFromSubtitleAnnotations excludes auxiliary-stem そうだ grammar tails', () => {
   const token = makeToken({
     surface: 'そうだ',

src/core/services/tokenizer/annotation-stage.ts

@@ -46,7 +46,6 @@ const SUBTITLE_ANNOTATION_EXCLUDED_EXPLANATORY_ENDING_TRAILING_PARTICLES = [
   'ね',
   'よ',
   'な',
-  'けど',
   'よね',
   'かな',
   'かね',

src/core/services/tokenizer/subtitle-annotation-filter.ts

@@ -41,7 +41,6 @@ const SUBTITLE_ANNOTATION_EXCLUDED_EXPLANATORY_ENDING_TRAILING_PARTICLES = [
   'ね',
   'よ',
   'な',
-  'けど',
   'よね',
   'かな',
   'かね',

src/core/services/youtube/generate.ts

@@ -1,10 +1,16 @@
+import type { YoutubeFlowMode } from '../../../types';
 import type { YoutubeTrackOption } from './track-probe';
 import { downloadYoutubeSubtitleTrack, downloadYoutubeSubtitleTracks } from './track-download';
 
+export function isYoutubeGenerationMode(mode: YoutubeFlowMode): boolean {
+  return mode === 'generate';
+}
+
 export async function acquireYoutubeSubtitleTrack(input: {
   targetUrl: string;
   outputDir: string;
   track: YoutubeTrackOption;
+  mode: YoutubeFlowMode;
 }): Promise<{ path: string }> {
   return await downloadYoutubeSubtitleTrack(input);
 }
@@ -13,6 +19,7 @@ export async function acquireYoutubeSubtitleTracks(input: {
   targetUrl: string;
   outputDir: string;
   tracks: YoutubeTrackOption[];
+  mode: YoutubeFlowMode;
 }): Promise<Map<string, string>> {
   return await downloadYoutubeSubtitleTracks(input);
 }

src/core/services/youtube/metadata-probe.test.ts

@@ -26,18 +26,6 @@ process.stdout.write(${JSON.stringify(payload)});
   fs.writeFileSync(scriptPath + '.cmd', `@echo off\r\nnode "${scriptPath}"\r\n`, 'utf8');
 }
 
-function makeHangingFakeYtDlpScript(dir: string): void {
-  const scriptPath = path.join(dir, 'yt-dlp');
-  const script = `#!/usr/bin/env node
-setInterval(() => {}, 1000);
-`;
-  fs.writeFileSync(scriptPath, script, 'utf8');
-  if (process.platform !== 'win32') {
-    fs.chmodSync(scriptPath, 0o755);
-  }
-  fs.writeFileSync(scriptPath + '.cmd', `@echo off\r\nnode "${scriptPath}"\r\n`, 'utf8');
-}
-
 async function withFakeYtDlp<T>(payload: string, fn: () => Promise<T>): Promise<T> {
   return await withTempDir(async (root) => {
     const binDir = path.join(root, 'bin');
@@ -53,37 +41,9 @@ async function withFakeYtDlp<T>(payload: string, fn: () => Promise<T>): Promise<
   });
 }
 
-async function withHangingFakeYtDlp<T>(fn: () => Promise<T>): Promise<T> {
-  return await withTempDir(async (root) => {
-    const binDir = path.join(root, 'bin');
-    fs.mkdirSync(binDir, { recursive: true });
-    makeHangingFakeYtDlpScript(binDir);
-    const originalPath = process.env.PATH ?? '';
-    process.env.PATH = `${binDir}${path.delimiter}${originalPath}`;
-    try {
-      return await fn();
-    } finally {
-      process.env.PATH = originalPath;
-    }
-  });
-}
-
 test('probeYoutubeVideoMetadata returns null on malformed yt-dlp JSON', async () => {
   await withFakeYtDlp('not-json', async () => {
     const result = await probeYoutubeVideoMetadata('https://www.youtube.com/watch?v=abc123');
     assert.equal(result, null);
   });
 });
-
-test(
-  'probeYoutubeVideoMetadata times out when yt-dlp hangs',
-  { timeout: 20_000 },
-  async () => {
-    await withHangingFakeYtDlp(async () => {
-      await assert.rejects(
-        probeYoutubeVideoMetadata('https://www.youtube.com/watch?v=abc123'),
-        /timed out after 15000ms/,
-      );
-    });
-  },
-);

src/core/services/youtube/metadata-probe.ts

@@ -1,8 +1,6 @@
 import { spawn } from 'node:child_process';
 import type { YoutubeVideoMetadata } from '../immersion-tracker/types';
 
-const YOUTUBE_METADATA_PROBE_TIMEOUT_MS = 15_000;
-
 type YtDlpThumbnail = {
   url?: string;
   width?: number;
@@ -23,19 +21,11 @@ type YtDlpYoutubeMetadata = {
   description?: string;
 };
 
-function runCapture(
-  command: string,
-  args: string[],
-  timeoutMs = YOUTUBE_METADATA_PROBE_TIMEOUT_MS,
-): Promise<{ stdout: string; stderr: string }> {
+function runCapture(command: string, args: string[]): Promise<{ stdout: string; stderr: string }> {
   return new Promise((resolve, reject) => {
     const proc = spawn(command, args, { stdio: ['ignore', 'pipe', 'pipe'] });
     let stdout = '';
     let stderr = '';
-    const timer = setTimeout(() => {
-      proc.kill();
-      reject(new Error(`yt-dlp timed out after ${timeoutMs}ms`));
-    }, timeoutMs);
     proc.stdout.setEncoding('utf8');
     proc.stderr.setEncoding('utf8');
     proc.stdout.on('data', (chunk) => {
@@ -44,12 +34,8 @@ function runCapture(
     proc.stderr.on('data', (chunk) => {
       stderr += String(chunk);
     });
-    proc.once('error', (error) => {
-      clearTimeout(timer);
-      reject(error);
-    });
+    proc.once('error', reject);
     proc.once('close', (code) => {
-      clearTimeout(timer);
       if (code === 0) {
         resolve({ stdout, stderr });
         return;

src/core/services/youtube/timedtext.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert/strict';
 import test from 'node:test';
-import { convertYoutubeTimedTextToVtt, normalizeYoutubeAutoVtt } from './timedtext';
+import { convertYoutubeTimedTextToVtt } from './timedtext';
 
 test('convertYoutubeTimedTextToVtt leaves malformed numeric entities literal', () => {
   const result = convertYoutubeTimedTextToVtt(
@@ -38,38 +38,3 @@ test('convertYoutubeTimedTextToVtt does not swallow text after zero-length overl
     ].join('\n'),
   );
 });
-
-test('normalizeYoutubeAutoVtt strips cumulative rolling-caption prefixes', () => {
-  const result = normalizeYoutubeAutoVtt(
-    [
-      'WEBVTT',
-      '',
-      '00:00:01.000 --> 00:00:02.000',
-      '今日は',
-      '',
-      '00:00:02.000 --> 00:00:03.000',
-      '今日はいい天気ですね',
-      '',
-      '00:00:03.000 --> 00:00:04.000',
-      '今日はいい天気ですね本当に',
-      '',
-    ].join('\n'),
-  );
-
-  assert.equal(
-    result,
-    [
-      'WEBVTT',
-      '',
-      '00:00:01.000 --> 00:00:02.000',
-      '今日は',
-      '',
-      '00:00:02.000 --> 00:00:03.000',
-      'いい天気ですね',
-      '',
-      '00:00:03.000 --> 00:00:04.000',
-      '本当に',
-      '',
-    ].join('\n'),
-  );
-});

src/core/services/youtube/timedtext.ts

@@ -115,52 +115,3 @@ export function convertYoutubeTimedTextToVtt(xml: string): string {
 
   return `WEBVTT\n\n${blocks.join('\n\n')}\n`;
 }
-
-function normalizeRollingCaptionText(text: string, previousText: string): string {
-  if (!previousText || !text.startsWith(previousText)) {
-    return text;
-  }
-  return text.slice(previousText.length).trimStart();
-}
-
-export function normalizeYoutubeAutoVtt(content: string): string {
-  const normalizedContent = content.replace(/\r\n?/g, '\n');
-  const blocks = normalizedContent.split(/\n{2,}/);
-  if (blocks.length === 0) {
-    return content;
-  }
-
-  let previousText = '';
-  let changed = false;
-  const normalizedBlocks = blocks.map((block) => {
-    if (!block.includes('-->')) {
-      return block;
-    }
-
-    const lines = block.split('\n');
-    const timingLineIndex = lines.findIndex((line) => line.includes('-->'));
-    if (timingLineIndex < 0 || timingLineIndex === lines.length - 1) {
-      return block;
-    }
-
-    const textLines = lines.slice(timingLineIndex + 1);
-    const originalText = textLines.join('\n').trim();
-    if (!originalText) {
-      return block;
-    }
-
-    const normalizedText = normalizeRollingCaptionText(originalText, previousText);
-    previousText = originalText;
-    if (!normalizedText || normalizedText === originalText) {
-      return block;
-    }
-
-    changed = true;
-    return [...lines.slice(0, timingLineIndex + 1), normalizedText].join('\n');
-  });
-
-  if (!changed) {
-    return content;
-  }
-  return `${normalizedBlocks.join('\n\n')}\n`;
-}

src/core/services/youtube/track-download.test.ts

@@ -174,6 +174,7 @@ test('downloadYoutubeSubtitleTrack prefers subtitle files over later webp artifa
         kind: 'auto',
         label: 'Japanese (auto)',
       },
+      mode: 'download',
     });
 
     assert.equal(path.extname(result.path), '.vtt');
@@ -203,6 +204,7 @@ test('downloadYoutubeSubtitleTrack ignores stale subtitle files from prior runs'
             kind: 'auto',
             label: 'Japanese (auto)',
           },
+          mode: 'download',
         }),
       /No subtitle file was downloaded/,
     );
@@ -231,6 +233,7 @@ test('downloadYoutubeSubtitleTrack uses auto subtitle flags and raw source langu
             kind: 'auto',
             label: 'Japanese (auto)',
           },
+          mode: 'download',
         });
 
         assert.equal(path.extname(result.path), '.vtt');
@@ -261,6 +264,7 @@ test('downloadYoutubeSubtitleTrack keeps manual subtitle flag for manual tracks'
             kind: 'manual',
             label: 'Japanese (manual)',
           },
+          mode: 'download',
         });
 
         assert.equal(path.extname(result.path), '.vtt');
@@ -269,43 +273,6 @@ test('downloadYoutubeSubtitleTrack keeps manual subtitle flag for manual tracks'
   });
 });
 
-test('downloadYoutubeSubtitleTrack normalizes rolling auto-caption vtt output from yt-dlp', async () => {
-  if (process.platform === 'win32') {
-    return;
-  }
-
-  await withFakeYtDlp('rolling-auto', async (root) => {
-    const result = await downloadYoutubeSubtitleTrack({
-      targetUrl: 'https://www.youtube.com/watch?v=abc123',
-      outputDir: path.join(root, 'out'),
-      track: {
-        id: 'auto:ja-orig',
-        language: 'ja',
-        sourceLanguage: 'ja-orig',
-        kind: 'auto',
-        label: 'Japanese (auto)',
-      },
-    });
-
-    assert.equal(
-      fs.readFileSync(result.path, 'utf8'),
-      [
-        'WEBVTT',
-        '',
-        '00:00:01.000 --> 00:00:02.000',
-        '今日は',
-        '',
-        '00:00:02.000 --> 00:00:03.000',
-        'いい天気ですね',
-        '',
-        '00:00:03.000 --> 00:00:04.000',
-        '本当に',
-        '',
-      ].join('\n'),
-    );
-  });
-});
-
 test('downloadYoutubeSubtitleTrack prefers direct download URL when available', async () => {
   await withTempDir(async (root) => {
     await withStubFetch(
@@ -326,6 +293,7 @@ test('downloadYoutubeSubtitleTrack prefers direct download URL when available',
             downloadUrl: 'https://example.com/subs/ja.vtt',
             fileExtension: 'vtt',
           },
+          mode: 'download',
         });
 
         assert.equal(path.basename(result.path), 'auto-ja-orig.ja-orig.vtt');
@@ -352,6 +320,7 @@ test('downloadYoutubeSubtitleTrack sanitizes metadata source language in filenam
             downloadUrl: 'https://example.com/subs/ja.vtt',
             fileExtension: 'vtt',
           },
+          mode: 'download',
         });
 
         assert.equal(path.dirname(result.path), path.join(root, 'out'));
@@ -390,6 +359,7 @@ test('downloadYoutubeSubtitleTrack converts srv3 auto subtitles into regular vtt
             downloadUrl: 'https://example.com/subs/ja.srv3',
             fileExtension: 'srv3',
           },
+          mode: 'download',
         });
 
         assert.equal(path.basename(result.path), 'auto-ja-orig.ja-orig.vtt');
@@ -440,6 +410,7 @@ test('downloadYoutubeSubtitleTracks downloads primary and secondary in one invoc
           label: 'English (auto)',
         },
       ],
+      mode: 'download',
     });
 
     assert.match(path.basename(result.get('auto:ja-orig') ?? ''), /\.ja-orig\.vtt$/);
@@ -473,6 +444,7 @@ test('downloadYoutubeSubtitleTracks preserves successfully downloaded primary fi
           label: 'English (auto)',
         },
       ],
+      mode: 'download',
     });
 
     assert.match(path.basename(result.get('auto:ja-orig') ?? ''), /\.ja-orig\.vtt$/);
@@ -512,6 +484,7 @@ test('downloadYoutubeSubtitleTracks prefers direct download URLs when available'
               fileExtension: 'vtt',
             },
           ],
+          mode: 'download',
         });
 
         assert.deepEqual(seen, [
@@ -557,6 +530,7 @@ test('downloadYoutubeSubtitleTracks keeps duplicate source-language direct downl
               fileExtension: 'vtt',
             },
           ],
+          mode: 'download',
         });
 
         assert.deepEqual(seen, [