feat: add playlist browser overlay modal

- Add overlay modal for browsing sibling video files and live mpv queue
- Add IPC commands for playlist operations (add, remove, move, play)
- Add playlist-browser-runtime and playlist-browser-sort modules
- Add keyboard handler and preload bindings for playlist browser
- Add default Ctrl+Alt+P keybinding to open the modal
- Add HTML structure, renderer wiring, and state for the modal
- Add changelog fragment and docs updates

CHANGELOG.md

@@ -1,10 +1,5 @@
 # Changelog
 
-## Unreleased
-
-### Fixed
-- AniList: Stopped post-watch tracking from sending a second progress update when the current episode was already satisfied by a ready retry item in the same watch-completion pass.
-
 ## v0.10.0 (2026-03-29)
 
 ### Changed

README.md

@@ -108,12 +108,12 @@ Browse sibling episode files and the active mpv queue in one overlay modal. Open
 
 ## Requirements
 
-|                | Required                                | Optional                                                   |
-| -------------- | --------------------------------------- | ---------------------------------------------------------- |
-| **Player**     | [`mpv`](https://mpv.io) with IPC socket | —                                                          |
+|                | Required                                | Optional                               |
+| -------------- | --------------------------------------- | -------------------------------------- |
+| **Player**     | [`mpv`](https://mpv.io) with IPC socket | —                                      |
 | **Processing** | `ffmpeg`, `mecab` + `mecab-ipadic`      | `guessit` (AniSkip), `alass` / `ffsubsync` (subtitle sync) |
-| **Media**      | —                                       | `yt-dlp`, `chafa`, `ffmpegthumbnailer`                     |
-| **Selection**  | —                                       | `fzf` / `rofi`                                             |
+| **Media**      | —                                       | `yt-dlp`, `chafa`, `ffmpegthumbnailer` |
+| **Selection**  | —                                       | `fzf` / `rofi`                         |
 
 > [!NOTE]
 > [`bun`](https://bun.sh) is required if building from source or using the CLI wrapper: `subminer`. Pre-built releases (AppImage, DMG, installer) do not require it.
@@ -236,6 +236,8 @@ subminer stats -b           # stats daemon in background
 subminer stats -s           # stop background stats daemon
 ```
 
+---
+
 ## Documentation
 
 Full guides on configuration, Anki setup, Jellyfin, immersion tracking, and more: **[docs.subminer.moe](https://docs.subminer.moe)**

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

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-22 21:25'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-24 06:44'
 labels:
   - stats
   - immersion-tracker
@@ -21,7 +21,6 @@ references:
   - >-
     /Users/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker-service.test.ts
 priority: medium
-ordinal: 178500
 ---
 
 ## Description

backlog/tasks/task-234 - Address-PR-35-latest-CodeRabbit-review-round.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-26 03:59'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-26 04:01'
 labels:
   - review-comments
   - coderabbit
@@ -18,7 +18,6 @@ references:
   - >-
     /Users/sudacode/projects/japanese/SubMiner/src/main/runtime/youtube-playback-launch.ts
 priority: medium
-ordinal: 177500
 ---
 
 ## Description

backlog/tasks/task-235 - Address-PR-35-autoplay-retry-CodeRabbit-follow-up.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-26 04:30'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-26 04:31'
 labels:
   - review-comments
   - coderabbit
@@ -13,7 +13,6 @@ dependencies: []
 references:
   - /Users/sudacode/projects/japanese/SubMiner/src/main.ts
 priority: medium
-ordinal: 176500
 ---
 
 ## Description

backlog/tasks/task-238.3 - Introduce-domain-type-entrypoints-and-shrink-src-types.ts-import-surface.md

@@ -4,7 +4,7 @@ title: Introduce domain type entrypoints and shrink src/types.ts import surface
 status: Done
 assignee: []
 created_date: '2026-03-26 20:49'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-27 00:14'
 labels:
   - tech-debt
   - types
@@ -18,7 +18,6 @@ references:
   - docs/architecture/README.md
 parent_task_id: TASK-238
 priority: medium
-ordinal: 174500
 ---
 
 ## Description
@@ -28,6 +27,7 @@ ordinal: 174500
 <!-- SECTION:DESCRIPTION:END -->
 
 ## Acceptance Criteria
+
 <!-- AC:BEGIN -->
 - [x] #1 Domain-focused type modules exist for the main clusters currently mixed together in `src/types.ts` (for example Anki, config/runtime, subtitle/media, and integration/runtime-option types).
 - [x] #2 `src/types.ts` becomes a thinner compatibility layer or barrel instead of the sole source of truth for every shared type.

backlog/tasks/task-238.4 - Decompose-character-dictionary-runtime-into-fetch-build-and-cache-modules.md

@@ -1,10 +1,10 @@
 ---
 id: TASK-238.4
-title: 'Decompose character dictionary runtime into fetch, build, and cache modules'
+title: Decompose character dictionary runtime into fetch, build, and cache modules
 status: Done
+updated_date: '2026-03-27 00:20'
 assignee: []
 created_date: '2026-03-26 20:49'
-updated_date: '2026-03-31 19:37'
 labels:
   - tech-debt
   - runtime
@@ -19,7 +19,6 @@ references:
   - docs/architecture/README.md
 parent_task_id: TASK-238
 priority: medium
-ordinal: 173500
 ---
 
 ## Description
@@ -29,6 +28,7 @@ ordinal: 173500
 <!-- SECTION:DESCRIPTION:END -->
 
 ## Acceptance Criteria
+
 <!-- AC:BEGIN -->
 - [x] #1 AniList fetch/parsing logic, dictionary-entry building, and snapshot/cache/zip persistence no longer live in one giant file.
 - [x] #2 The public runtime API stays behavior-compatible for current callers.

backlog/tasks/task-238.5 - Split-immersion-tracker-query-layer-into-focused-read-model-modules.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-26 20:49'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-27 00:00'
 labels:
   - tech-debt
   - stats
@@ -20,7 +20,6 @@ references:
   - src/core/services/immersion-tracker-service.ts
 parent_task_id: TASK-238
 priority: medium
-ordinal: 175500
 ---
 
 ## Description
@@ -30,6 +29,7 @@ ordinal: 175500
 <!-- SECTION:DESCRIPTION:END -->
 
 ## Acceptance Criteria
+
 <!-- AC:BEGIN -->
 - [x] #1 Query responsibilities are grouped into focused modules such as library/session detail, vocabulary/kanji detail, and maintenance/cleanup helpers.
 - [x] #2 The stats server and immersion tracker service depend on stable exported query surfaces instead of one monolithic file.

backlog/tasks/task-238.6 - Extract-remaining-inline-runtime-logic-and-composer-gaps-from-src-main.ts.md

@@ -4,7 +4,7 @@ title: Extract remaining inline runtime logic and composer gaps from src/main.ts
 status: Done
 assignee: []
 created_date: '2026-03-27 00:00'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-27 22:13'
 labels:
   - tech-debt
   - runtime
@@ -24,7 +24,6 @@ references:
   - src/main/runtime/composers
 parent_task_id: TASK-238
 priority: high
-ordinal: 172500
 ---
 
 ## Description
@@ -34,6 +33,7 @@ ordinal: 172500
 <!-- SECTION:DESCRIPTION:END -->
 
 ## Acceptance Criteria
+
 <!-- AC:BEGIN -->
 - [x] #1 `runYoutubePlaybackFlow`, `maybeSignalPluginAutoplayReady`, `refreshSubtitlePrefetchFromActiveTrack`, `publishDiscordPresence`, and `handleModalInputStateChange` no longer live as substantial inline logic in `src/main.ts`.
 - [x] #2 The large subtitle/prefetch, stats startup, and overlay visibility dependency groupings are wrapped behind named composer helpers instead of remaining inline in `src/main.ts`.

backlog/tasks/task-238.7 - Split-src-main.ts-into-boot-phase-services-runtimes-and-handlers.md

@@ -1,10 +1,10 @@
 ---
 id: TASK-238.7
-title: 'Split src/main.ts into boot-phase services, runtimes, and handlers'
+title: Split src/main.ts into boot-phase services, runtimes, and handlers
 status: Done
 assignee: []
 created_date: '2026-03-27 00:00'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-27 22:45'
 labels:
   - tech-debt
   - runtime
@@ -21,7 +21,6 @@ references:
   - src/main/runtime/composers
 parent_task_id: TASK-238
 priority: high
-ordinal: 171500
 ---
 
 ## Description
@@ -31,6 +30,7 @@ After the remaining inline runtime logic and composer gaps are extracted, `src/m
 <!-- SECTION:DESCRIPTION:END -->
 
 ## Acceptance Criteria
+
 <!-- AC:BEGIN -->
 - [x] #1 Service instantiation lives in a dedicated boot module instead of a large inline setup block in `src/main.ts`.
 - [x] #2 Domain runtime composition lives in a dedicated boot module, separate from lifecycle and handler dispatch.

backlog/tasks/task-238.8 - Refactor-src-main.ts-composition-root-into-domain-runtimes.md

@@ -1,43 +0,0 @@
----
-id: TASK-238.8
-title: Refactor src/main.ts composition root into domain runtimes
-status: In Progress
-assignee: []
-created_date: '2026-03-31 06:28'
-updated_date: '2026-04-01 07:07'
-labels:
-  - tech-debt
-  - runtime
-  - maintainability
-  - composition-root
-dependencies: []
-references:
-  - src/main.ts
-  - src/main/boot/services
-  - src/main/runtime/composers
-  - docs/architecture/README.md
-parent_task_id: TASK-238
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Refactor `src/main.ts` so it becomes a thin composition root and the domain-specific runtime wiring moves into short wrapper modules under `src/main/`. Preserve all current behavior, IPC contracts, and config/schema semantics while reducing the entrypoint to boot services, grouped runtime instantiation, startup execution, and process-level quit handling.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [ ] #1 `src/main.ts` is bootstrap/composition only: platform preflight, boot services, runtime creation, startup execution, and top-level quit/signal handling.
-- [ ] #2 `src/main.ts` no longer imports `src/main/runtime/*-main-deps.ts` directly.
-- [ ] #3 `src/main.ts` has no local names like `build*MainDepsHandler`, `*MainDeps`, or trivial `*Handler` pass-through wrappers.
-- [ ] #4 New wrapper files stay under ~500 LOC each; if one exceeds that, split before merge.
-- [ ] #5 Cross-domain coordination stays in `main.ts`; wrapper modules stay acyclic and communicate via injected callbacks.
-- [ ] #6 No user-facing behavior, config fields, or IPC channel names change.
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-CI follow-up: typecheck failed after the runtime split because playlist-browser IPC deps were not threaded through the new bootstrap/composer surfaces. Wiring the missing open action and registration deps now.
-<!-- SECTION:NOTES:END -->

backlog/tasks/task-242 - Fix-stats-server-Bun-fallback-in-coverage-lane.md

@@ -4,7 +4,7 @@ title: Fix stats server Bun fallback in coverage lane
 status: Done
 assignee: []
 created_date: '2026-03-29 07:31'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 07:37'
 labels:
   - ci
   - bug
@@ -13,7 +13,6 @@ dependencies: []
 references:
   - 'PR #36'
 priority: high
-ordinal: 170500
 ---
 
 ## Description

backlog/tasks/task-246 - Migrate-Discord-Rich-Presence-to-maintained-RPC-wrapper.md

@@ -4,14 +4,13 @@ title: Migrate Discord Rich Presence to maintained RPC wrapper
 status: Done
 assignee: []
 created_date: '2026-03-29 08:17'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 08:22'
 labels:
   - dependency
   - discord
   - presence
 dependencies: []
 priority: medium
-ordinal: 169500
 ---
 
 ## Description

backlog/tasks/task-247 - Strip-inline-subtitle-markup-from-subtitle-sidebar-cues.md

@@ -5,14 +5,13 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-29 10:01'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 10:10'
 labels: []
 dependencies: []
 references:
   - src/core/services/subtitle-cue-parser.ts
   - src/renderer/modals/subtitle-sidebar.ts
   - src/core/services/subtitle-cue-parser.test.ts
-ordinal: 168500
 ---
 
 ## Description

backlog/tasks/task-248 - Fix-macOS-visible-overlay-toggle-getting-immediately-restored.md

@@ -4,7 +4,7 @@ title: Fix macOS visible overlay toggle getting immediately restored
 status: Done
 assignee: []
 created_date: '2026-03-29 10:03'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 22:14'
 labels: []
 dependencies: []
 references:
@@ -13,7 +13,6 @@ references:
   - /Users/sudacode/projects/japanese/SubMiner/src/core/services/cli-command.ts
   - >-
     /Users/sudacode/projects/japanese/SubMiner/src/main/overlay-visibility-runtime.ts
-ordinal: 165500
 ---
 
 ## Description

backlog/tasks/task-249 - Fix-AniList-token-persistence-on-setup-login.md

@@ -4,7 +4,7 @@ title: Fix AniList token persistence on setup login
 status: Done
 assignee: []
 created_date: '2026-03-29 10:08'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 19:42'
 labels:
   - anilist
   - bug
@@ -15,7 +15,6 @@ documentation:
   - src/main/runtime/anilist-token-refresh.ts
   - docs-site/anilist-integration.md
 priority: high
-ordinal: 166500
 ---
 
 ## Description

backlog/tasks/task-250 - Restore-macOS-mpv-passthrough-while-overlay-subtitle-sidebar-is-open.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - '@codex'
 created_date: '2026-03-29 10:10'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 10:23'
 labels:
   - bug
   - macos
@@ -24,7 +24,6 @@ references:
   - >-
     /Users/sudacode/projects/japanese/SubMiner/src/renderer/overlay-mouse-ignore.test.ts
 priority: high
-ordinal: 167500
 ---
 
 ## Description

backlog/tasks/task-251 - Docs-add-subtitle-sidebar-and-Jimaku-integration-pages.md

@@ -4,12 +4,11 @@ title: 'Docs: add subtitle sidebar and Jimaku integration pages'
 status: Done
 assignee: []
 created_date: '2026-03-29 22:36'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 22:38'
 labels:
   - docs
 dependencies: []
 priority: medium
-ordinal: 164500
 ---
 
 ## Description

backlog/tasks/task-252 - Harden-AUR-publish-release-step-against-transient-SSH-failures.md

@@ -4,14 +4,13 @@ title: Harden AUR publish release step against transient SSH failures
 status: Done
 assignee: []
 created_date: '2026-03-29 23:46'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-29 23:49'
 labels:
   - release
   - ci
   - aur
 dependencies: []
 priority: high
-ordinal: 163500
 ---
 
 ## Description

backlog/tasks/task-253 - Fix-animated-AVIF-lead-in-alignment-with-sentence-audio.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-30 01:59'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-30 02:03'
 labels: []
 dependencies: []
 references:
@@ -14,7 +14,6 @@ references:
   - /Users/sudacode/projects/japanese/SubMiner/src/anki-integration.ts
   - /Users/sudacode/projects/japanese/SubMiner/src/core/services/stats-server.ts
   - /Users/sudacode/projects/japanese/SubMiner/src/media-generator.ts
-ordinal: 162500
 ---
 
 ## Description

backlog/tasks/task-254 - Fix-AniList-token-persistence-when-safe-storage-is-unavailable.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-30 02:10'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-30 02:20'
 labels:
   - bug
   - anilist
@@ -17,7 +17,6 @@ references:
   - >-
     /Users/sudacode/projects/japanese/SubMiner/src/main/runtime/anilist-token-refresh.ts
 priority: high
-ordinal: 161500
 ---
 
 ## Description

backlog/tasks/task-255 - Add-overlay-playlist-browser-modal-for-sibling-video-files-and-mpv-queue.md

@@ -1,18 +1,17 @@
 ---
 id: TASK-255
 title: Add overlay playlist browser modal for sibling video files and mpv queue
-status: Done
+status: In Progress
 assignee:
-  - '@codex'
+  - codex
 created_date: '2026-03-30 05:46'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-30 08:34'
 labels:
   - feature
   - overlay
   - mpv
   - launcher
 dependencies: []
-ordinal: 180500
 ---
 
 ## Description
@@ -23,12 +22,12 @@ Add an in-session overlay modal that opens from a keybinding during active playb
 
 ## Acceptance Criteria
 <!-- AC:BEGIN -->
-- [x] #1 An overlay modal can be opened during active playback from a dedicated keybinding and closed without disrupting existing modal behavior.
-- [x] #2 The modal shows video files from the current media file's parent directory in best-effort episode order and highlights the current file when present.
-- [x] #3 The modal shows the active mpv playlist/queue with enough metadata to identify the current item and queued order.
-- [x] #4 The user can add a directory file to the mpv playlist, remove playlist items, and reorder playlist items from the modal using both mouse and keyboard interactions.
-- [x] #5 Modal state stays in sync after playlist mutations so the rendered queue reflects mpv's current playlist order.
-- [x] #6 Feature coverage includes automated tests for ordering/playlist behavior and docs or shortcut/help updates for the new modal.
+- [ ] #1 An overlay modal can be opened during active playback from a dedicated keybinding and closed without disrupting existing modal behavior.
+- [ ] #2 The modal shows video files from the current media file's parent directory in best-effort episode order and highlights the current file when present.
+- [ ] #3 The modal shows the active mpv playlist/queue with enough metadata to identify the current item and queued order.
+- [ ] #4 The user can add a directory file to the mpv playlist, remove playlist items, and reorder playlist items from the modal using both mouse and keyboard interactions.
+- [ ] #5 Modal state stays in sync after playlist mutations so the rendered queue reflects mpv's current playlist order.
+- [ ] #6 Feature coverage includes automated tests for ordering/playlist behavior and docs or shortcut/help updates for the new modal.
 <!-- AC:END -->
 
 ## Implementation Plan
@@ -41,14 +40,6 @@ Add an in-session overlay modal that opens from a keybinding during active playb
 5. Write failing renderer and keyboard tests for modal open/close, split-pane interaction, keyboard controls, and degraded states.
 6. Implement playlist-browser modal markup, DOM/state, renderer composition, keyboard routing, and session-help labeling.
 7. Run targeted test lanes first, then the maintained verification gate relevant to the touched surfaces; update task notes/criteria as checks pass.
-
-2026-03-30 CodeRabbit follow-up: 1) add failing runtime coverage for unreadable playlist-browser file stat failures, 2) add failing renderer coverage for stale snapshot UI reset on refresh failure/close, 3) add failing renderer coverage to block playlist-browser open when another modal already owns the overlay, 4) implement minimal fixes, 5) rerun targeted tests plus typecheck for touched surfaces.
-
-2026-03-30 current CodeRabbit round: verify 4 unresolved threads, ignore already-fixed outdated dblclick thread if current code matches, add failing-first coverage for selection preservation / timestamp fixture consistency / string test-clock alignment, implement minimal fixes, rerun targeted tests plus typecheck.
-
-2026-03-30 latest CodeRabbit round on PR #37: 1) add failing coverage for negative fractional numeric __subminerTestNowMs input so nowMs() matches the string-backed path, 2) add failing coverage that playlist-browser modal tests restore absent window/document globals without leaving undefined-valued properties behind, 3) refactor repeated playlist-browser modal test harness into a shared setup/teardown fixture while preserving assertions, 4) implement minimal fixes, 5) rerun touched tests plus typecheck.
-
-2026-03-30 latest CodeRabbit follow-up after ff760ea: tighten the new cleanup regression so env.restore() always runs under assertion failure, and make the keydown test's append mock return a post-append mutated snapshot before exercising Ctrl+ArrowDown. Re-run targeted playlist-browser tests plus typecheck.
 <!-- SECTION:PLAN:END -->
 
 ## Implementation Notes
@@ -67,26 +58,4 @@ Repo gate blockers outside this feature: `bun run test:fast` hits existing Bun `
 2026-03-30: Follow-up subtitle regression fix. Pre-jump `sid=auto` was ineffective because mpv resolved it against the current episode before `playlist-play-index`. Local playlist jumps now set `sub-auto=fuzzy`, switch episodes, then schedule a delayed rearm of `sid=auto` and `secondary-sid=auto` so selection happens against the new file's tracks. Added failing-first runtime coverage for delayed local rearm and remote no-op behavior.
 
 2026-03-30: Cleaned up playlist-browser runtime local-play subtitle-rearm flow by extracting focused helpers without changing behavior. Added public docs/readme coverage for the default `Ctrl+Alt+P` playlist browser keybinding and modal, plus changelog fragment `changes/260-playlist-browser.md`. Verification: `bun test src/main/runtime/playlist-browser-runtime.test.ts`, `bun run typecheck`, `bun run docs:test`, `bun run docs:build`, `bun run changelog:lint`, `bun run build`.
-
-2026-03-30: Pulled unresolved CodeRabbit review threads for PR #37. Actionable set is three items: unreadable-file stat error handling in playlist-browser runtime, stale playlist-browser DOM after failed refresh/close, and missing modal-ownership guard before opening the playlist-browser overlay. Proceeding test-first for each.
-
-2026-03-30: Addressed current CodeRabbit follow-up findings for PR #37. Fixed playlist-browser unreadable-file stat handling, stale playlist-browser DOM reset on refresh failure/close, modal-ownership guard before opening the playlist-browser overlay, async rejection surfacing for PLAYLIST_BROWSER_OPEN IPC commands, overlay bootstrap before playlist-browser open dispatch, texthooker option normalization in the mpv plugin, and superseded local subtitle-rearm suppression. Added targeted regressions plus new playlist-browser-open helper coverage. Verification: `bun test src/main/runtime/playlist-browser-runtime.test.ts src/main/runtime/playlist-browser-open.test.ts src/core/services/ipc-command.test.ts src/renderer/modals/playlist-browser.test.ts`, `lua scripts/test-plugin-start-gate.lua`, `bun run typecheck`, `bun run build`.
-
-Addressed CodeRabbit follow-ups on the playlist browser PR: clamped stale playingIndex values, failed mutation paths when MPV rejects send(), added temp-dir cleanup in runtime tests, and blocked action-button dblclick bubbling in the renderer. Verification: `bun run typecheck`, `bun run build`, `bun test src/main/runtime/playlist-browser-runtime.test.ts src/renderer/modals/playlist-browser.test.ts`.
-
-Additional follow-up: moved playlist-browser keydown handling ahead of keyboard-driven lookup controls so KeyH/ArrowLeft/ArrowRight and related chords are routed to the modal first. Verification refreshed with `bun test src/main/runtime/playlist-browser-runtime.test.ts src/renderer/modals/playlist-browser.test.ts src/renderer/handlers/keyboard.test.ts`, `bun run typecheck`, and `bun run build`.
-
-Split playlist-browser UI row rendering into `src/renderer/modals/playlist-browser-renderer.ts` and left `src/renderer/modals/playlist-browser.ts` as the controller/wiring layer. Moved playlist-browser IPC/runtime wiring into `src/main/runtime/playlist-browser-ipc.ts` and collapsed the `src/main.ts` registration block to use that helper. Verification after refactor: `bun run typecheck`, `bun run build`, `bun test src/main/runtime/playlist-browser-runtime.test.ts src/renderer/modals/playlist-browser.test.ts src/renderer/handlers/keyboard.test.ts`.
-
-2026-03-30 PR #37 unresolved CodeRabbit threads currently reduce to three likely-actionable items plus one outdated renderer dblclick thread to verify against HEAD before touching code.
-
-2026-03-30 Addressed latest unresolved CodeRabbit items on PR #37: preserved playlist-browser selection across mutation snapshots, taught nowMs() to honor string-backed test clocks so it stays aligned with currentDbTimestamp(), and normalized maintenance test timestamp fixtures to toDbTimestamp(). The older playlist-browser dblclick thread remains unresolved in GitHub state but current HEAD already contains that fix in playlist-browser-renderer.ts.
-
-2026-03-30 latest CodeRabbit remediation on PR #37: switched nowMs() numeric test-clock branch from Math.floor() to Math.trunc() so numeric and string-backed mock clocks agree for negative fractional values. Refactored playlist-browser modal tests onto a shared setup/teardown fixture that restores global window/document descriptors correctly, and added regression coverage that injected globals are deleted when originally absent. Verification: `bun test src/core/services/immersion-tracker/time.test.ts src/renderer/modals/playlist-browser.test.ts`, `bun run typecheck`.
-
-2026-03-30 CodeRabbit follow-up: wrapped the injected-globals cleanup regression in try/finally so restore always runs, and changed the keydown test append mock to return createMutationSnapshot() before exercising Ctrl+ArrowDown. Verified with `bun test src/renderer/modals/playlist-browser.test.ts` and `bun run typecheck`.
-
-2026-03-31 assessment: the playlist-browser feature is landed on `main` via `d51e7fe4 Add playlist browser overlay modal (#37)` with runtime, IPC, renderer, keybinding, and changelog/docs coverage present. Verified passes: `bun test src/main/runtime/playlist-browser-runtime.test.ts src/main/runtime/playlist-browser-open.test.ts src/main/runtime/playlist-browser-sort.test.ts src/renderer/handlers/keyboard.test.ts src/core/services/ipc.test.ts src/core/services/ipc-command.test.ts src/config/definitions/domain-registry.test.ts`.
-
-Remaining action item before close: fix `src/renderer/modals/playlist-browser.test.ts` so the cleanup regression does not assume `globalThis.window` / `globalThis.document` start absent under Bun, rerun the playlist-browser modal lane (and then typecheck/build if you want the full closeout proof), then finalize the task.
 <!-- SECTION:NOTES:END -->

backlog/tasks/task-256 - Fix-texthooker-page-live-websocket-connect-send-regression.md

@@ -5,14 +5,13 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-30 06:04'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-30 06:12'
 labels:
   - bug
   - texthooker
   - websocket
 dependencies: []
 priority: medium
-ordinal: 160500
 ---
 
 ## Description

backlog/tasks/task-257 - Fix-texthooker-only-mode-startup-to-initialize-websocket-pipeline.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-30 06:15'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-30 06:17'
 labels:
   - bug
   - texthooker
@@ -13,7 +13,6 @@ labels:
   - startup
 dependencies: []
 priority: high
-ordinal: 159500
 ---
 
 ## Description

backlog/tasks/task-258 - Stop-plugin-auto-start-from-spawning-separate-texthooker-helper.md

@@ -5,7 +5,7 @@ status: Done
 assignee:
   - codex
 created_date: '2026-03-30 06:25'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-30 06:26'
 labels:
   - bug
   - texthooker
@@ -14,7 +14,6 @@ labels:
   - startup
 dependencies: []
 priority: high
-ordinal: 158500
 ---
 
 ## Description

backlog/tasks/task-259 - Fix-integrated-start-texthooker-startup-skipping-texthooker-server.md

@@ -4,14 +4,13 @@ title: Fix integrated --start --texthooker startup skipping texthooker server
 status: Done
 assignee: []
 created_date: '2026-03-30 06:48'
-updated_date: '2026-03-31 19:37'
+updated_date: '2026-03-30 06:56'
 labels:
   - bug
   - texthooker
   - startup
 dependencies: []
 priority: high
-ordinal: 157500
 ---
 
 ## Description

backlog/tasks/task-260 - Fix-macOS-overlay-subtitle-sidebar-passthrough-without-requiring-a-subtitle-hover-cycle.md

@@ -1,68 +0,0 @@
----
-id: TASK-260
-title: >-
-  Fix macOS overlay subtitle sidebar passthrough without requiring a subtitle
-  hover cycle
-status: Done
-assignee:
-  - '@codex'
-created_date: '2026-03-31 00:58'
-updated_date: '2026-03-31 19:37'
-labels:
-  - bug
-  - macos
-  - overlay
-  - subtitle-sidebar
-  - passthrough
-dependencies: []
-references:
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/renderer/modals/subtitle-sidebar.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/renderer/overlay-mouse-ignore.ts
-  - /Users/sudacode/projects/japanese/SubMiner/src/renderer/handlers/mouse.ts
-  - /Users/sudacode/projects/japanese/SubMiner/src/main/overlay-runtime.ts
-  - >-
-    /Users/sudacode/projects/japanese/SubMiner/src/core/services/overlay-visibility.ts
-documentation:
-  - docs/workflow/verification.md
-priority: high
-ordinal: 156500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-On macOS, opening the overlay-layout subtitle sidebar should allow click-through outside the sidebar immediately. Users should not need to first hover subtitle content before passthrough/click-through starts working, including when no subtitle line is currently visible.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 With the overlay-layout subtitle sidebar open on macOS, areas outside the sidebar pass clicks through immediately after open without requiring a prior subtitle hover.
-- [x] #2 When no subtitle line is currently visible, opening the subtitle sidebar still leaves non-sidebar overlay regions click-through on macOS.
-- [x] #3 Regression coverage exercises the first-open/idle passthrough path so overlay interactivity does not depend on a later hover cycle.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Add/adjust focused overlay visibility regressions for the tracked macOS visible overlay so the default idle state stays click-through instead of forcing mouse interaction.
-2. Update main-process visible overlay visibility sync to keep the tracked macOS overlay passive by default and let renderer hover/sidebar state opt into interaction.
-3. Run focused verification for overlay visibility and any dependent runtime tests, then update task notes/criteria/final summary with the confirmed outcome.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Investigation points to a main-process override on macOS: renderer sidebar open path already requests mouse passthrough outside the panel, but visible-overlay visibility sync still hard-sets the tracked overlay window interactive on macOS (`mouse-ignore:false`). Window-tracker focus/visibility resync can therefore undo renderer passthrough until a later hover cycle re-applies it.
-
-Added a failing regression in `src/core/services/overlay-visibility.test.ts` showing the tracked macOS visible overlay was still forced interactive by main-process visibility sync (`mouse-ignore:false`) instead of staying forwarded click-through.
-
-Updated `src/core/services/overlay-visibility.ts` so tracked macOS visible overlays now default to `setIgnoreMouseEvents(true, { forward: true })`, matching the renderer-side passthrough model and preventing window-tracker/focus resync from undoing idle sidebar clickthrough.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Fixed the macOS subtitle-sidebar passthrough regression by changing tracked visible-overlay startup/visibility sync to stay click-through by default in the main process. Previously `updateVisibleOverlayVisibility` forced the macOS overlay window interactive, which could override renderer sidebar passthrough until a later hover cycle repaired it. Added a regression in `src/core/services/overlay-visibility.test.ts` and verified with `bun test src/core/services/overlay-visibility.test.ts`, `bun test src/renderer/modals/subtitle-sidebar.test.ts src/renderer/handlers/mouse.test.ts`, and `bun run typecheck`.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-261 - Fix-immersion-tracker-SQLite-timestamp-truncation.md

@@ -1,40 +0,0 @@
----
-id: TASK-261
-title: Fix immersion tracker SQLite timestamp truncation
-status: Done
-assignee: []
-created_date: '2026-03-31 01:45'
-updated_date: '2026-03-31 19:37'
-labels:
-  - immersion-tracker
-  - sqlite
-  - bug
-dependencies: []
-references:
-  - src/core/services/immersion-tracker
-priority: medium
-ordinal: 179500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Current-epoch millisecond values are being truncated by the libsql driver when bound as numeric parameters, which corrupts session, telemetry, lifetime, and rollup timestamps.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Current-epoch millisecond timestamps persist correctly in session, telemetry, lifetime, and rollup tables
-- [x] #2 Startup backfill and destroy/finalize flows keep retained sessions and lifetime summaries consistent
-- [x] #3 Regression tests cover the destroyed-session, startup backfill, and distinct-day/distinct-video lifetime semantics
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-2026-03-31 assessment: epoch-ms timestamp writes now route through `toDbMs()` / `toDbTimestamp()` in `src/core/services/immersion-tracker/query-shared.ts`, which avoids libsql numeric-parameter truncation by binding BigInt/string values before they hit SQLite. The fix is wired through the session, storage/telemetry, lifetime, and rollup-maintenance paths in `src/core/services/immersion-tracker/session.ts`, `src/core/services/immersion-tracker/storage.ts`, `src/core/services/immersion-tracker/lifetime.ts`, and `src/core/services/immersion-tracker/maintenance.ts`.
-
-Acceptance coverage is present: `bun test src/core/services/immersion-tracker-service.test.ts` passed with explicit regressions for destroy/finalize persistence, startup backfill when retained sessions exist but lifetime tables are empty, startup reconciliation of stale active sessions, `rebuildLifetimeSummaries`, and distinct-day / distinct-video lifetime semantics. `bun test src/core/services/immersion-tracker/time.test.ts src/core/services/immersion-tracker/maintenance.test.ts` also passed.
-
-Remaining action item before close: fix the two `src/main/runtime/stats-cli-command.test.ts` cleanup-lifetime assertions that currently use Bun-misparsed underscored millisecond literals (`1_710_000_000_000` evaluates to `-2147483648` under Bun 1.3.11), rerun that verification lane, then write the final summary and mark the task Done.
-<!-- SECTION:NOTES:END -->

backlog/tasks/task-262 - Create-overlay-UI-bootstrap-input-helper.md

@@ -1,27 +0,0 @@
----
-id: TASK-262
-title: Create overlay UI bootstrap input helper
-status: To Do
-assignee: []
-created_date: '2026-03-31 17:04'
-labels:
-  - refactor
-  - main
-  - overlay-ui
-dependencies: []
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Add a coarse input-builder/helper module to reduce the large createOverlayUiRuntime(...) callsite in src/main.ts without changing runtime behavior. Do not edit src/main.ts in this task.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [ ] #1 New helper module(s) live under src/main/
-- [ ] #2 Helper accepts grouped overlay UI/domain inputs instead of giant inline literals
-- [ ] #3 Helper keeps files under 500 LOC
-- [ ] #4 Optional focused tests added if useful
-- [ ] #5 No runtime behavior changes
-<!-- AC:END -->

backlog/tasks/task-262 - Fix-duplicate-AniList-post-watch-updates-for-watched-episodes.md

@@ -1,51 +0,0 @@
----
-id: TASK-262
-title: Fix duplicate AniList post-watch updates for watched episodes
-status: Done
-assignee:
-  - codex
-created_date: '2026-03-31 19:03'
-updated_date: '2026-03-31 19:37'
-labels:
-  - bug
-  - anilist
-dependencies: []
-ordinal: 155500
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Watching an episode can currently produce two AniList activity updates for the same episode. The duplicate happens when the post-watch flow drains a queued retry for the current episode and then proceeds to run the live post-watch update for that same media/episode in the same pass. User report says this reproduces both when crossing the watched threshold naturally and when using the mark-watched keybinding. Fix the duplicate so one successful watch produces at most one AniList progress update for a given mediaKey/episode pair.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 A watched episode triggers at most one AniList post-watch progress update for a given media key and episode during a single post-watch pass, even if that episode already exists in the retry queue.
-- [x] #2 Both watched-threshold and manual mark-watched flows are protected by regression coverage for the duplicate-update case.
-- [x] #3 Relevant user-visible change note is added if required by repo policy.
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Reproduce the duplicate in a unit test around `createMaybeRunAnilistPostWatchUpdateHandler` by simulating a ready retry for the same `mediaKey::episode` the live path would also submit.
-2. Fix the handler so that after processing a queued retry, it does not perform a second live update when the retry already satisfied the current attempt key.
-3. Run focused AniList runtime tests and adjacent immersion tests to confirm both threshold-driven and manual mark-watched entry points stay covered through the shared post-watch path.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Added a regression in `src/main/runtime/anilist-post-watch.test.ts` for the case where `processNextAnilistRetryUpdate()` already satisfies the current `mediaKey::episode` before the live path runs.
-
-Updated `createMaybeRunAnilistPostWatchUpdateHandler` to re-check `hasAttemptedUpdateKey(attemptKey)` immediately after draining the retry queue and short-circuit before a second live AniList submission.
-
-Verification: `bun test src/main/runtime/anilist-post-watch.test.ts src/main/runtime/anilist-post-watch-main-deps.test.ts`; `bun test src/core/services/immersion-tracker-service.test.ts --test-name-pattern 'recordPlaybackPosition marks watched at 85% completion|markActiveVideoWatched'`; `bun run typecheck`; `bun run changelog:lint`.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Fixed duplicate AniList post-watch submissions by short-circuiting the live update path when a ready retry item already handled the current `mediaKey::episode` in the same pass. Added a focused regression test for the retry-plus-live duplicate scenario and a changelog fragment documenting the fix.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-263 - Create-coarse-startup-bootstrap-wrapper.md

@@ -1,28 +0,0 @@
----
-id: TASK-263
-title: Create coarse startup bootstrap wrapper
-status: To Do
-assignee: []
-created_date: '2026-03-31 17:21'
-labels:
-  - refactor
-  - main
-  - startup
-dependencies: []
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Move the large createMainStartupRuntime construction and its self-reference handling out of src/main.ts into a coarse startup bootstrap wrapper. Keep behavior identical and shrink the startup section materially.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [ ] #1 New wrapper module(s) live under src/main/
-- [ ] #2 Wrapper owns createMainStartupRuntime construction and self-reference handling
-- [ ] #3 src/main.ts startup section becomes materially smaller
-- [ ] #4 Files stay under 500 LOC
-- [ ] #5 Focused tests cover the wrapper if useful
-- [ ] #6 No runtime behavior changes
-<!-- AC:END -->

backlog/tasks/task-263 - Reuse-pre-add-duplicate-IDs-for-generic-Kiku-field-grouping.md

@@ -1,39 +0,0 @@
----
-id: TASK-263
-title: Reuse pre-add duplicate IDs for generic Kiku field grouping
-status: Done
-assignee: []
-created_date: '2026-03-31 20:44'
-updated_date: '2026-03-31 20:48'
-labels:
-  - anki
-  - kiku
-dependencies: []
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Avoid the extra post-add duplicate lookup on the generic sentence-card creation path by capturing duplicate note IDs before add and reusing that result for Kiku field grouping. Keep Yomitan semantics aligned where practical so duplicate selection is consistent across mining paths.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Generic sentence-card creation captures duplicate note IDs before add and reuses them for Kiku field grouping instead of running the existing post-add duplicate finder
-- [x] #2 Duplicate selection remains deterministic when multiple matching notes exist
-- [x] #3 Regression tests cover the generic path duplicate reuse behavior and preserve existing non-Kiku behavior
-- [x] #4 Internal docs/config comments are updated if the behavior or operator-facing semantics changed
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-No docs update was required because this is internal duplicate-selection plumbing and does not change user-facing config surface.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Generic sentence-card creation now captures exact duplicate note IDs before add when Kiku field grouping is enabled and stores that context by created note ID. Manual field grouping reuses the tracked duplicate IDs first and deterministically picks the most recent matching note, falling back to the legacy duplicate finder only when no tracked context exists. Verified with bun test src/anki-integration/duplicate.test.ts src/anki-integration/card-creation.test.ts src/anki-integration/field-grouping.test.ts and bun run typecheck.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-263.1 - Reuse-Yomitan-popup-duplicate-IDs-in-SubMiner-bridge.md

@@ -1,41 +0,0 @@
----
-id: TASK-263.1
-title: Reuse Yomitan popup duplicate IDs in SubMiner bridge
-status: Done
-assignee: []
-created_date: '2026-03-31 22:15'
-updated_date: '2026-03-31 22:21'
-labels:
-  - anki
-  - kiku
-  - yomitan
-dependencies: []
-parent_task_id: TASK-263
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Thread Yomitan popup/search duplicate note IDs through the existing SubMiner bridge so Kiku/manual grouping can reuse the same duplicate context that already drives the Add duplicate button. Implement and test against the vendored Yomitan copy first; do not rely on upstreamed fork changes yet.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Vendored Yomitan bridge returns duplicate note IDs for popup/search mining when available
-- [x] #2 SubMiner consumes the bridged duplicate IDs and prefers them for Kiku/manual grouping on the Yomitan mining path
-- [x] #3 Regression tests cover the popup/search bridge payload and duplicate-id reuse behavior
-- [x] #4 No commit is made for vendored Yomitan-only changes in this repo state
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Vendored files changed locally for validation only: vendor/subminer-yomitan/ext/js/display/display-anki.js, vendor/subminer-yomitan/ext/js/comm/api.js, vendor/subminer-yomitan/ext/js/comm/anki-connect.js, vendor/subminer-yomitan/ext/js/background/backend.js. Do not commit those vendor changes in this repo; port them to the fork instead.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Vendored Yomitan popup/search mining now precomputes duplicate note IDs, sends them to the SubMiner Anki proxy as private addNote metadata, and still returns note/duplicate data through the parser bridge. The proxy strips the private metadata before forwarding to upstream AnkiConnect, associates the duplicate IDs with the created note before auto-enrichment begins, and SubMiner also records the bridge result as a secondary cache path. Verified with bun test src/anki-integration/duplicate.test.ts src/anki-integration/card-creation.test.ts src/anki-integration/field-grouping.test.ts src/anki-integration/anki-connect-proxy.test.ts src/core/services/tokenizer/yomitan-parser-runtime.test.ts and bun run typecheck.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-263.2 - Keep-Yomitan-popup-responsive-during-background-add-and-pause-close-before-Kiku-modal.md

@@ -1,43 +0,0 @@
----
-id: TASK-263.2
-title: >-
-  Keep Yomitan popup responsive during background add and pause/close before
-  Kiku modal
-status: Done
-assignee: []
-created_date: '2026-04-01 00:42'
-updated_date: '2026-04-01 02:35'
-labels:
-  - anki
-  - yomitan
-  - kiku
-  - ux
-dependencies: []
-parent_task_id: TASK-263
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Make Yomitan popup add run in background without blocking popup responsiveness. Before opening Kiku field-grouping modal, pause MPV and close the Yomitan popup/parser window if open.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Definition of Done
-<!-- DOD:BEGIN -->
-- [x] #1 Popup save path returns immediately and prevents duplicate submits
-- [x] #2 Field-grouping modal request pauses MPV and closes Yomitan popup window first
-- [x] #3 Regression tests cover async save dispatch and main-side pause/close hook
-<!-- DOD:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-2026-03-31: Removed the custom pending label/gray save-button presentation from vendored Yomitan. Background add still runs asynchronously with the internal pending-save guard, so duplicate clicks are ignored while the button keeps its stock appearance.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Yomitan popup save dispatches note creation/add in the background with an internal pending-save guard so repeated clicks are ignored without blocking the popup. Before opening the Kiku field-grouping modal, the renderer now closes the visible lookup popup and pauses MPV. Follow-up UX polish removed the custom pending label/gray styling so the save button keeps Yomitan’s stock presentation while the background action runs.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-264 - Replace-axios-with-native-fetch-across-the-project.md

@@ -1,26 +0,0 @@
----
-id: TASK-264
-title: Replace axios with native fetch across the project
-status: To Do
-assignee: []
-created_date: '2026-04-01 00:44'
-labels: []
-dependencies: []
-priority: medium
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Remove axios from the codebase and migrate all project HTTP requests to the platform fetch API, preserving existing request behavior and error handling where applicable.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [ ] #1 No production code paths import or depend on axios.
-- [ ] #2 All existing HTTP requests use fetch or a project-local abstraction built on fetch.
-- [ ] #3 Request behavior remains functionally equivalent for headers, query params, bodies, status handling, and abort/error cases that are currently supported.
-- [ ] #4 Tests are updated or added to cover the migrated request flows.
-- [ ] #5 Documentation is updated if any request semantics or setup steps change.
-- [ ] #6 axios is removed from project dependencies if it is no longer needed.
-<!-- AC:END -->

backlog/tasks/task-265 - Add-remote-backend-for-immersion-tracking-and-stats-prefer-Postgres.md

@@ -1,53 +0,0 @@
----
-id: TASK-265
-title: Add remote backend for immersion tracking and stats (prefer Postgres)
-status: To Do
-assignee: []
-created_date: '2026-04-01 00:47'
-labels: []
-dependencies: []
-references:
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker-service.ts
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/storage.ts
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/src/core/services/immersion-tracker/sqlite.ts
-  - /home/sudacode/projects/japanese/SubMiner/src/stats-daemon-runner.ts
-  - /home/sudacode/projects/japanese/SubMiner/src/core/services/stats-server.ts
-  - /home/sudacode/projects/japanese/SubMiner/src/main/boot/services.ts
-  - /home/sudacode/projects/japanese/SubMiner/package.json
-documentation:
-  - /home/sudacode/projects/japanese/SubMiner/docs/architecture/README.md
-  - >-
-    /home/sudacode/projects/japanese/SubMiner/docs/architecture/stats-trends-data-flow.md
-  - /home/sudacode/projects/japanese/SubMiner/README.md
-  - /home/sudacode/projects/japanese/SubMiner/config.example.jsonc
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Enable immersion tracking/stats to use a remote authoritative backend so multiple devices can share the same history.
-
-Current state: `ImmersionTrackerService` opens a local `immersion.sqlite` file from the app data/config path, `stats-daemon-runner` points at that same local file, and `config.example.jsonc` only exposes `immersionTracking.dbPath` for a local path override. The stats API/dashboard reads from the same tracker service and assumes the local database is the source of truth.
-
-Goal: add a remote backend option that avoids shared filesystem/database-file syncing between devices. Do not use SSH/rsync/shared network filesystem as the primary sync strategy for live multi-device use.
-
-Backend choice: prefer Postgres if it can be integrated without a broad new dependency surface or destabilizing the current runtime; otherwise use the least invasive remote backend that can be shipped with the current stack and document the tradeoff clearly. Preserve the current local SQLite mode as the default/offline fallback if possible.
-
-This ticket should cover the full product/architecture change: configuration, storage access, stats reads, startup/error handling, migration/bootstrap from existing local data, tests, and docs.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [ ] #1 The app can be configured to use a remote authoritative backend for immersion tracking instead of only a local `immersion.sqlite` file.
-- [ ] #2 The chosen backend persists tracker writes and serves the existing stats read models across app restarts.
-- [ ] #3 Two devices can point at the same remote backend without relying on a shared filesystem or raw SQLite file sync.
-- [ ] #4 Local SQLite remains supported as the default or fallback mode for offline use.
-- [ ] #5 If the remote backend is unavailable or misconfigured, startup/write paths fail with actionable errors instead of silent data loss.
-- [ ] #6 A migration or bootstrap path exists to move existing local immersion data into the remote backend or seed a new device from it.
-- [ ] #7 Config/examples/docs explain the backend choice, required connection/setup details, and any security/network assumptions.
-- [ ] #8 Tests cover backend selection plus at least one representative write/read path against the remote backend.
-<!-- AC:END -->

backlog/tasks/task-266 - Preserve-paused-state-for-configured-subtitle-jump-keybindings.md

@@ -1,34 +0,0 @@
----
-id: TASK-266
-title: Preserve paused state for configured subtitle-jump keybindings
-status: Done
-assignee: []
-created_date: '2026-04-01 03:19'
-updated_date: '2026-04-01 03:19'
-labels:
-  - renderer
-  - mpv
-  - keybindings
-  - regression
-dependencies: []
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Regression: configured overlay keybindings that forward raw mpv subtitle-jump commands (for example previous-subtitle on H) can resume playback when invoked while paused. Keyboard-driven edge jumps already preserve paused state; configured keybindings should match that behavior.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Configured subtitle-jump keybindings preserve paused playback state after backward seek
-- [x] #2 Existing keyboard-driven subtitle navigation behavior remains unchanged
-- [x] #3 Regression test covers paused configured subtitle-jump keybinding handling
-<!-- AC:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Configured overlay keybindings that forward `sub-seek` commands now re-check paused state and reapply pause after the seek when playback was already paused. This aligns raw configured subtitle-jump keybindings with the existing keyboard-driven edge-jump behavior and adds regression coverage for the paused backward-seek case.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-267 - Port-validated-Yomitan-popup-changes-to-fork-and-resync-submodule.md

@@ -1,34 +0,0 @@
----
-id: TASK-267
-title: Port validated Yomitan popup changes to fork and resync submodule
-status: Done
-assignee: []
-created_date: '2026-04-01 03:30'
-updated_date: '2026-04-01 03:33'
-labels:
-  - yomitan
-  - submodule
-  - git
-  - integration
-dependencies: []
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Take the locally validated Yomitan popup/bridge changes from the vendored copy, apply them to the standalone `../subminer-yomitan` fork, verify the fork, push the fork commit, then reset the vendored working tree in SubMiner and update the submodule pointer to the pushed fork commit.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 Standalone `../subminer-yomitan` contains the validated popup/bridge changes and passes the relevant regression test
-- [x] #2 The fork commit is pushed to its configured remote branch
-- [x] #3 SubMiner vendored Yomitan working tree is reset and the submodule pointer is updated to the pushed fork commit
-<!-- AC:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Applied the validated popup/bridge changes from the vendored Yomitan copy into `../subminer-yomitan`, added the focused async-save regression test there, installed fork deps, and verified with `npx vitest run test/display-anki-save.test.js`. Committed the fork changes as `feat: preserve async popup save state and duplicate metadata`, rebased onto the updated remote `main`, and pushed commit `69620abc` to `origin/main`. Then reset the vendored submodule working tree in SubMiner, checked it out at `69620abc`, and left the superproject with the submodule pointer updated from `3c9ee577` to `69620abc`.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-268 - Address-CodeRabbit-review-action-items-for-PR-38.md

@@ -1,44 +0,0 @@
----
-id: TASK-268
-title: 'Address CodeRabbit review action items for PR #38'
-status: Done
-assignee: []
-created_date: '2026-04-01 05:35'
-updated_date: '2026-04-01 06:07'
-labels:
-  - pr-review
-  - coderabbit
-dependencies: []
-references:
-  - 'https://github.com/ksyasuda/SubMiner/pull/38'
-priority: medium
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Review unresolved CodeRabbit feedback on PR #38 and implement the actionable fixes without regressing duplicate grouping or popup behavior.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 All unresolved actionable CodeRabbit review comments on PR #38 are triaged and either fixed in code or explicitly identified as non-actionable or ambiguous.
-- [x] #2 Code changes preserve duplicate grouping and popup flow behavior covered by existing or added regression tests.
-- [x] #3 Relevant local verification for the affected areas passes.
-<!-- AC:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-2026-04-01: Reopened for follow-up CodeRabbit round after commit 233bde58. Remaining actionable items: guard maxMatches <= 0 in duplicate exact-match helper and strengthen the duplicate tracking test fixture to prove deduplication as well as sorting.
-
-2026-04-01: Follow-up round addressed locally. Added guard for maxMatches <= 0 in duplicate exact-match scanning and strengthened the pre-add duplicate tracking test fixture to prove deduplication as well as sorting.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Addressed all unresolved actionable CodeRabbit comments on PR #38. Fixed duplicate tracking so empty duplicate lists are not persisted after sentence-card creation, sanitized Yomitan add-note noteId values to accept only positive integers, preserved paused playback for configured subtitle-seek keybindings when pause state is unknown, and short-circuited duplicate exact-match scanning for single-result lookups. Added regression tests for each case and verified with `bun test` on the affected suites plus `bun run typecheck`, `bun run test:fast`, `bun run test:env`, `bun run build`, and `bun run test:smoke:dist`.
-
-Follow-up CodeRabbit round addressed locally: `findExactDuplicateNoteIds()` now returns early when `maxMatches <= 0`, and the sentence-card duplicate tracking regression test now uses a repeated duplicate ID to assert deduplication plus sorting. Re-verified with targeted duplicate/card tests, `bun run typecheck`, and `bun run test:fast`.
-<!-- SECTION:FINAL_SUMMARY:END -->

backlog/tasks/task-269 - Assess-and-address-PR-39-latest-CodeRabbit-review-round.md

@@ -1,59 +0,0 @@
----
-id: TASK-269
-title: 'Assess and address PR #39 latest CodeRabbit review round'
-status: Done
-assignee:
-  - codex
-created_date: '2026-04-01 07:22'
-updated_date: '2026-04-01 07:55'
-labels: []
-dependencies: []
-references:
-  - src/main
-  - docs/architecture/README.md
-priority: high
----
-
-## Description
-
-<!-- SECTION:DESCRIPTION:BEGIN -->
-Assess unresolved CodeRabbit review threads on PR #39, fix valid findings in the current branch, and document any non-actionable findings so the review state is clear for follow-up.
-<!-- SECTION:DESCRIPTION:END -->
-
-## Acceptance Criteria
-<!-- AC:BEGIN -->
-- [x] #1 All unresolved CodeRabbit review threads on PR #39 are assessed and classified as fix or non-actionable with rationale
-- [x] #2 Valid findings are addressed in code without regressing current runtime behavior
-- [x] #3 Regression tests or targeted coverage are added when the reviewed behavior can be exercised locally
-- [x] #4 Relevant verification commands are run and results recorded in the task summary
-<!-- AC:END -->
-
-## Implementation Plan
-
-<!-- SECTION:PLAN:BEGIN -->
-1. Confirm each unresolved CodeRabbit thread against current branch state and mark already-addressed items as non-actionable without code churn.
-2. Fix validated runtime issues in focused patches: AniList setup window stale close handler, headless known-word refresh effective config usage, main startup websocket probe wiring, startup warmup delegation, mpv media-path duplicate side effects, overlay visibility readiness guard, Discord presence service cleanup, and stats daemon self-stop handling.
-3. Tighten headless startup runtime typing so custom bootstrap deps require an explicit factory instead of unsafe casting.
-4. Add or extend targeted tests for behaviors that can be exercised locally, especially overlay visibility, Discord lifecycle cleanup, stats self-stop, and headless startup typing/runtime coverage.
-5. Run targeted tests first, then the relevant broader verification lane, and capture which CodeRabbit items were fixed versus assessed as already addressed.
-<!-- SECTION:PLAN:END -->
-
-## Implementation Notes
-
-<!-- SECTION:NOTES:BEGIN -->
-Reassessed current branch after reset concern: most runtime/test fixes were still present; only live worktree delta was IPC playlist-browser cleanup plus the untracked backlog task record.
-
-Validated and fixed CodeRabbit findings for stale AniList setup window clearing, effective headless Anki config usage, CLI websocket probe wiring, duplicate mpv media-path side effects, overlay visibility readiness, Discord presence service cleanup, stats self-stop, unsafe headless startup custom deps typing, and playlist-browser IPC wiring.
-
-Assessed main-startup-runtime-bootstrap warmup comment as non-actionable at this layer: the guarded startBackgroundWarmupsIfAllowed wrapper exists in main-startup-bootstrap, while this lower bootstrap still needs to expose the raw mpv warmup command.
-<!-- SECTION:NOTES:END -->
-
-## Final Summary
-
-<!-- SECTION:FINAL_SUMMARY:BEGIN -->
-Assessed the unresolved CodeRabbit round on PR #39 and applied the confirmed fixes across the main-process runtime split. Updated AniList setup window teardown to avoid clearing newer windows, made headless known-word refresh use one effective Anki config end-to-end, wired the real mpv websocket probe into CLI startup, removed duplicate mpv media-path side effects, guarded overlay visibility changes behind overlay readiness, stopped Discord presence services before disable/replace, and prevented stats daemon stop from SIGTERMing the current Electron process. Also tightened headless startup typing so custom bootstrap deps require an explicit factory, normalized stats coordinator note-id typing, and completed playlist-browser IPC wiring/types so typecheck stays green.
-
-Added targeted regression coverage for overlay visibility initialization, Discord lifecycle cleanup, stats self-stop, and headless startup custom-deps typing expectations. Verification run from the current tree: `bun run typecheck`, targeted `bun test` for overlay/discord/stats/headless startup files, and full `bun run test:fast` all passed.
-
-Assessment outcome for remaining review noise: the playlist-browser open action was already wired in IPC bootstrap, and the warmup-delegation comment on `main-startup-runtime-bootstrap.ts` was not applied because the guarded wrapper lives one layer higher in `main-startup-bootstrap.ts`; this lower bootstrap still needs to surface the raw mpv warmup command consumed by that wrapper.
-<!-- SECTION:FINAL_SUMMARY:END -->

changes/260-main-runtime-refactor.md

@@ -1,6 +0,0 @@
-type: internal
-area: main
-
-- Split `src/main.ts` into domain runtime wrappers and startup sequencing helpers.
-- Removed the last direct `src/main/runtime/*-main-deps.ts` imports from `src/main.ts`.
-- Kept startup behavior and IPC contracts stable while reducing composition-root size.

changes/261-macos-overlay-passthrough.md

@@ -1,5 +0,0 @@
-type: fixed
-area: overlay
-
-- Keep tracked macOS visible overlays click-through by default so subtitle sidebar passthrough works immediately without requiring a subtitle hover cycle first.
-- Add regression coverage for the macOS visible-overlay passthrough default.

changes/262-anilist-post-watch-dedupe.md

@@ -1,5 +0,0 @@
-type: fixed
-area: anilist
-
-- Stop AniList post-watch from sending a second progress update when the current episode was already satisfied by a ready retry item in the same watch-completion pass.
-- Add regression coverage for the retry-queue plus live-update duplicate path.

changes/267-yomitan-kiku-popup.md

@@ -1,6 +0,0 @@
-type: fixed
-area: overlay
-
-- Fixed Kiku duplicate grouping to reuse duplicate note IDs from both generic sentence-card creation and Yomitan popup mining instead of running extra duplicate scans after add.
-- Fixed the Yomitan popup mining flow to add cards in the background while keeping the stock popup progress feedback, then pause playback and close the lookup popup before the Kiku merge modal opens.
-- Fixed configured subtitle-jump keybindings so backward and forward subtitle seeks keep playback paused when invoked from a paused state.

docs-site/configuration.md

@@ -969,7 +969,6 @@ To refresh roughly once per day, set:
 | `disabled` | No field grouping; duplicate cards are left as-is                                                                          |
 
 `deleteDuplicateInAuto` controls whether `auto` mode deletes the duplicate after merge (default: `true`). In `manual` mode, the popup asks each time whether to delete the duplicate.
-When the manual merge popup opens, SubMiner pauses playback and closes any open Yomitan popup first so the merge flow can take focus.
 
 <video controls playsinline preload="metadata" poster="/assets/kiku-integration-poster.jpg" style="width: 100%; max-width: 960px;">
   <source :src="'/assets/kiku-integration.webm'" type="video/webm" />

docs/architecture/README.md

@@ -3,7 +3,7 @@
 # Architecture Map
 
 Status: active
-Last verified: 2026-03-31
+Last verified: 2026-03-26
 Owner: Kyle Yasuda
 Read when: runtime ownership, composition boundaries, or layering questions
 
@@ -24,27 +24,6 @@ The desktop app keeps `src/main.ts` as composition root and pushes behavior into
 ## Current Shape
 
 - `src/main/` owns composition, runtime setup, IPC wiring, and app lifecycle adapters.
-- `src/main/*.ts` wrapper runtimes sit between `src/main.ts` and `src/main/runtime/**`
-  so the composition root stays thin while exported `createBuild*MainDepsHandler`
-  APIs remain internal plumbing. Key domain runtimes:
-  - `anilist-runtime` – AniList token management, media tracking, retry queue
-  - `cli-startup-runtime` – CLI command dispatch and initial-args handling
-  - `discord-presence-lifecycle-runtime` – Discord Rich Presence lifecycle
-  - `first-run-runtime` – first-run setup wizard
-  - `ipc-runtime` – IPC handler registration and composition
-  - `jellyfin-runtime` – Jellyfin session, playback, mpv orchestration
-  - `main-startup-runtime` – top-level startup orchestration (app-ready → CLI → headless)
-  - `main-startup-bootstrap` – wiring helper that builds startup runtime inputs
-  - `mining-runtime` – Anki card mining actions
-  - `mpv-runtime` – mpv client lifecycle
-  - `overlay-ui-runtime` – overlay window management, visibility, tray
-  - `overlay-geometry-runtime` – overlay bounds resolution
-  - `shortcuts-runtime` – global shortcut registration
-  - `startup-sequence-runtime` – headless known-word refresh and deferred startup sequencing
-  - `stats-runtime` – immersion tracker, stats server, stats CLI
-  - `subtitle-runtime` – subtitle prefetch, tokenization, caching
-  - `youtube-runtime` – YouTube playback flow
-  - `yomitan-runtime` – Yomitan extension loading and settings
 - `src/main/boot/` owns boot-phase assembly seams so `src/main.ts` can stay focused on lifecycle coordination and startup-path selection.
 - `src/core/services/` owns focused runtime services plus pure or side-effect-bounded logic.
 - `src/renderer/` owns overlay rendering and input behavior.

plugin/subminer/process.lua

@@ -34,17 +34,6 @@ function M.create(ctx)
 		return options_helper.coerce_bool(raw_pause_until_ready, false)
 	end
 
-	local function resolve_texthooker_enabled(override_value)
-		if override_value ~= nil then
-			return options_helper.coerce_bool(override_value, false)
-		end
-		local raw_texthooker_enabled = opts.texthooker_enabled
-		if raw_texthooker_enabled == nil then
-			raw_texthooker_enabled = opts["texthooker-enabled"]
-		end
-		return options_helper.coerce_bool(raw_texthooker_enabled, false)
-	end
-
 	local function resolve_pause_until_ready_timeout_seconds()
 		local raw_timeout_seconds = opts.auto_start_pause_until_ready_timeout_seconds
 		if raw_timeout_seconds == nil then
@@ -203,7 +192,10 @@ function M.create(ctx)
 				table.insert(args, "--hide-visible-overlay")
 			end
 
-			local texthooker_enabled = resolve_texthooker_enabled(overrides.texthooker_enabled)
+			local texthooker_enabled = overrides.texthooker_enabled
+			if texthooker_enabled == nil then
+				texthooker_enabled = opts.texthooker_enabled
+			end
 			if texthooker_enabled then
 				table.insert(args, "--texthooker")
 			end
@@ -304,7 +296,10 @@ function M.create(ctx)
 			return
 		end
 
-		local texthooker_enabled = resolve_texthooker_enabled(overrides.texthooker_enabled)
+		local texthooker_enabled = overrides.texthooker_enabled
+		if texthooker_enabled == nil then
+			texthooker_enabled = opts.texthooker_enabled
+		end
 		local socket_path = overrides.socket_path or opts.socket_path
 		local should_pause_until_ready = (
 			overrides.auto_start_trigger == true
@@ -503,7 +498,7 @@ function M.create(ctx)
 				end
 			end)
 
-			if resolve_texthooker_enabled(nil) then
+			if opts.texthooker_enabled then
 				ensure_texthooker_running(function() end)
 			end
 		end)

scripts/build-yomitan.mjs

@@ -51,16 +51,9 @@ function ensureSubmodulePresent() {
 }
 
 function getSourceState() {
-  try {
-    const revision = readCommand('git', ['rev-parse', 'HEAD'], submoduleDir);
-    const dirty = readCommand('git', ['status', '--short', '--untracked-files=no'], submoduleDir);
-    return { revision, dirty };
-  } catch (error) {
-    if (process.env.SUBMINER_YOMITAN_ALLOW_MISSING_GIT === '1') {
-      return { revision: 'unknown', dirty: '' };
-    }
-    throw error;
-  }
+  const revision = readCommand('git', ['rev-parse', 'HEAD'], submoduleDir);
+  const dirty = readCommand('git', ['status', '--short', '--untracked-files=no'], submoduleDir);
+  return { revision, dirty };
 }
 
 function isBuildCurrent(force) {

scripts/test-plugin-start-gate.lua

@@ -531,31 +531,6 @@ do
 	)
 end
 
-do
-	local recorded, err = run_plugin_scenario({
-		process_list = "",
-		option_overrides = {
-			binary_path = binary_path,
-			auto_start = "yes",
-			auto_start_visible_overlay = "yes",
-			auto_start_pause_until_ready = "no",
-			socket_path = "/tmp/subminer-socket",
-			texthooker_enabled = "no",
-		},
-		input_ipc_server = "/tmp/subminer-socket",
-		media_title = "Random Movie",
-		files = {
-			[binary_path] = true,
-		},
-	})
-	assert_true(recorded ~= nil, "plugin failed to load for disabled texthooker auto-start scenario: " .. tostring(err))
-	fire_event(recorded, "file-loaded")
-	local start_call = find_start_call(recorded.async_calls)
-	assert_true(start_call ~= nil, "disabled texthooker auto-start should still issue --start command")
-	assert_true(not call_has_arg(start_call, "--texthooker"), "disabled texthooker should not include --texthooker on --start")
-	assert_true(find_control_call(recorded.async_calls, "--texthooker") == nil, "disabled texthooker should not issue a helper texthooker command")
-end
-
 do
 	local recorded, err = run_plugin_scenario({
 		process_list = "",

src/anki-integration.ts

@@ -51,7 +51,6 @@ import { KnownWordCacheManager } from './anki-integration/known-word-cache';
 import { PollingRunner } from './anki-integration/polling';
 import type { AnkiConnectProxyServer } from './anki-integration/anki-connect-proxy';
 import { findDuplicateNote as findDuplicateNoteForAnkiIntegration } from './anki-integration/duplicate';
-import { findDuplicateNoteIds as findDuplicateNoteIdsForAnkiIntegration } from './anki-integration/duplicate';
 import { CardCreationService } from './anki-integration/card-creation';
 import { FieldGroupingService } from './anki-integration/field-grouping';
 import { FieldGroupingMergeCollaborator } from './anki-integration/field-grouping-merge';
@@ -149,7 +148,6 @@ export class AnkiIntegration {
   private aiConfig: AiConfig;
   private recordCardsMinedCallback: ((count: number, noteIds?: number[]) => void) | null = null;
   private noteIdRedirects = new Map<number, number>();
-  private trackedDuplicateNoteIds = new Map<number, number[]>();
 
   constructor(
     config: AnkiConnectConfig,
@@ -266,9 +264,6 @@ export class AnkiIntegration {
       recordCardsAdded: (count, noteIds) => {
         this.recordCardsMinedSafely(count, noteIds, 'proxy');
       },
-      trackAddedDuplicateNoteIds: (noteId, duplicateNoteIds) => {
-        this.trackDuplicateNoteIdsForNote(noteId, duplicateNoteIds);
-      },
       getDeck: () => this.config.deck,
       findNotes: async (query, options) =>
         (await this.client.findNotes(query, options)) as number[],
@@ -366,10 +361,6 @@ export class AnkiIntegration {
       trackLastAddedNoteId: (noteId) => {
         this.previousNoteIds.add(noteId);
       },
-      trackLastAddedDuplicateNoteIds: (noteId, duplicateNoteIds) => {
-        this.trackedDuplicateNoteIds.set(noteId, [...duplicateNoteIds]);
-      },
-      findDuplicateNoteIds: (expression, noteInfo) => this.findDuplicateNoteIds(expression, noteInfo),
       recordCardsMinedCallback: (count, noteIds) => {
         this.recordCardsMinedSafely(count, noteIds, 'card creation');
       },
@@ -391,10 +382,6 @@ export class AnkiIntegration {
       extractFields: (fields) => this.extractFields(fields),
       findDuplicateNote: (expression, noteId, noteInfo) =>
         this.findDuplicateNote(expression, noteId, noteInfo),
-      getTrackedDuplicateNoteIds: (noteId) =>
-        this.trackedDuplicateNoteIds.has(noteId)
-          ? [...(this.trackedDuplicateNoteIds.get(noteId) ?? [])]
-          : null,
       hasAllConfiguredFields: (noteInfo, configuredFieldNames) =>
         this.hasAllConfiguredFields(noteInfo, configuredFieldNames),
       processNewCard: (noteId, options) => this.processNewCard(noteId, options),
@@ -1055,10 +1042,6 @@ export class AnkiIntegration {
     );
   }
 
-  trackDuplicateNoteIdsForNote(noteId: number, duplicateNoteIds: number[]): void {
-    this.trackedDuplicateNoteIds.set(noteId, [...duplicateNoteIds]);
-  }
-
   private async findDuplicateNote(
     expression: string,
     excludeNoteId: number,
@@ -1082,28 +1065,6 @@ export class AnkiIntegration {
     });
   }
 
-  private async findDuplicateNoteIds(
-    expression: string,
-    noteInfo: NoteInfo,
-  ): Promise<number[]> {
-    return findDuplicateNoteIdsForAnkiIntegration(expression, -1, noteInfo, {
-      findNotes: async (query, options) => (await this.client.findNotes(query, options)) as unknown,
-      notesInfo: async (noteIds) => (await this.client.notesInfo(noteIds)) as unknown,
-      getDeck: () => this.config.deck,
-      getWordFieldCandidates: () => this.getConfiguredWordFieldCandidates(),
-      resolveFieldName: (info, preferredName) => this.resolveNoteFieldName(info, preferredName),
-      logInfo: (message) => {
-        log.info(message);
-      },
-      logDebug: (message) => {
-        log.debug(message);
-      },
-      logWarn: (message, error) => {
-        log.warn(message, (error as Error).message);
-      },
-    });
-  }
-
   private getPreferredSentenceAudioFieldName(): string {
     const sentenceCardConfig = this.getEffectiveSentenceCardConfig();
     return sentenceCardConfig.audioField || 'SentenceAudio';

src/anki-integration/anki-connect-proxy.test.ts

@@ -324,123 +324,6 @@ test('proxy fallback-enqueues latest note for addNote responses without note IDs
   assert.deepEqual(recordedCards, [1]);
 });
 
-test('proxy tracks duplicate note ids from addNote request metadata before enrichment', async () => {
-  const processed: number[] = [];
-  const tracked: Array<{ noteId: number; duplicateNoteIds: number[] }> = [];
-  const proxy = new AnkiConnectProxyServer({
-    shouldAutoUpdateNewCards: () => true,
-    processNewCard: async (noteId) => {
-      processed.push(noteId);
-    },
-    trackAddedDuplicateNoteIds: (noteId, duplicateNoteIds) => {
-      tracked.push({ noteId, duplicateNoteIds });
-    },
-    logInfo: () => undefined,
-    logWarn: () => undefined,
-    logError: () => undefined,
-  });
-
-  (
-    proxy as unknown as {
-      maybeEnqueueFromRequest: (request: Record<string, unknown>, responseBody: Buffer) => void;
-    }
-  ).maybeEnqueueFromRequest(
-    {
-      action: 'addNote',
-      params: {
-        note: {},
-        subminerDuplicateNoteIds: [11, -1, 40, 11, 25],
-      },
-    },
-    Buffer.from(JSON.stringify({ result: 42, error: null }), 'utf8'),
-  );
-
-  await waitForCondition(() => processed.length === 1);
-  assert.deepEqual(tracked, [{ noteId: 42, duplicateNoteIds: [11, 25, 40] }]);
-  assert.deepEqual(processed, [42]);
-});
-
-test('proxy strips SubMiner duplicate metadata before forwarding upstream addNote request', async () => {
-  let upstreamBody = '';
-  const upstream = http.createServer(async (req, res) => {
-    upstreamBody = await new Promise<string>((resolve) => {
-      const chunks: Buffer[] = [];
-      req.on('data', (chunk) => chunks.push(Buffer.from(chunk)));
-      req.on('end', () => resolve(Buffer.concat(chunks).toString('utf8')));
-    });
-    res.statusCode = 200;
-    res.setHeader('content-type', 'application/json');
-    res.end(JSON.stringify({ result: 42, error: null }));
-  });
-  upstream.listen(0, '127.0.0.1');
-  await once(upstream, 'listening');
-  const upstreamAddress = upstream.address();
-  assert.ok(upstreamAddress && typeof upstreamAddress === 'object');
-  const upstreamPort = upstreamAddress.port;
-
-  const tracked: Array<{ noteId: number; duplicateNoteIds: number[] }> = [];
-  const proxy = new AnkiConnectProxyServer({
-    shouldAutoUpdateNewCards: () => true,
-    processNewCard: async () => undefined,
-    trackAddedDuplicateNoteIds: (noteId, duplicateNoteIds) => {
-      tracked.push({ noteId, duplicateNoteIds });
-    },
-    logInfo: () => undefined,
-    logWarn: () => undefined,
-    logError: () => undefined,
-  });
-
-  try {
-    proxy.start({
-      host: '127.0.0.1',
-      port: 0,
-      upstreamUrl: `http://127.0.0.1:${upstreamPort}`,
-    });
-
-    const proxyServer = (
-      proxy as unknown as {
-        server: http.Server | null;
-      }
-    ).server;
-    assert.ok(proxyServer);
-    if (!proxyServer.listening) {
-      await once(proxyServer, 'listening');
-    }
-    const proxyAddress = proxyServer.address();
-    assert.ok(proxyAddress && typeof proxyAddress === 'object');
-    const proxyPort = proxyAddress.port;
-
-    const response = await fetch(`http://127.0.0.1:${proxyPort}`, {
-      method: 'POST',
-      headers: {
-        'content-type': 'application/json',
-      },
-      body: JSON.stringify({
-        action: 'addNote',
-        version: 6,
-        params: {
-          note: {
-            deckName: 'Mining',
-            modelName: 'Sentence',
-            fields: { Expression: '食べる' },
-          },
-          subminerDuplicateNoteIds: [18, 7],
-        },
-      }),
-    });
-
-    assert.equal(response.status, 200);
-    assert.deepEqual(await response.json(), { result: 42, error: null });
-    await waitForCondition(() => tracked.length === 1);
-    assert.equal(upstreamBody.includes('subminerDuplicateNoteIds'), false);
-    assert.deepEqual(tracked, [{ noteId: 42, duplicateNoteIds: [7, 18] }]);
-  } finally {
-    proxy.stop();
-    upstream.close();
-    await once(upstream, 'close');
-  }
-});
-
 test('proxy returns addNote response without waiting for background enrichment', async () => {
   const processed: number[] = [];
   let releaseProcessing: (() => void) | undefined;

src/anki-integration/anki-connect-proxy.ts

@@ -16,7 +16,6 @@ export interface AnkiConnectProxyServerDeps {
   shouldAutoUpdateNewCards: () => boolean;
   processNewCard: (noteId: number) => Promise<void>;
   recordCardsAdded?: (count: number, noteIds: number[]) => void;
-  trackAddedDuplicateNoteIds?: (noteId: number, duplicateNoteIds: number[]) => void;
   getDeck?: () => string | undefined;
   findNotes?: (
     query: string,
@@ -162,7 +161,6 @@ export class AnkiConnectProxyServer {
     }
 
     try {
-      const forwardedBody = req.method === 'POST' ? this.getForwardRequestBody(rawBody, requestJson) : rawBody;
       const targetUrl = new URL(req.url || '/', upstreamUrl).toString();
       const contentType =
         typeof req.headers['content-type'] === 'string'
@@ -171,7 +169,7 @@ export class AnkiConnectProxyServer {
       const upstreamResponse = await this.client.request<ArrayBuffer>({
         url: targetUrl,
         method: req.method,
-        data: req.method === 'POST' ? forwardedBody : undefined,
+        data: req.method === 'POST' ? rawBody : undefined,
         headers: {
           'content-type': contentType,
         },
@@ -221,8 +219,6 @@ export class AnkiConnectProxyServer {
       return;
     }
 
-    this.maybeTrackDuplicateNoteIds(requestJson, action, responseResult);
-
     const noteIds =
       action === 'multi'
         ? this.collectMultiResultIds(requestJson, responseResult)
@@ -235,77 +231,6 @@ export class AnkiConnectProxyServer {
     this.enqueueNotes(noteIds);
   }
 
-  private maybeTrackDuplicateNoteIds(
-    requestJson: Record<string, unknown>,
-    action: string,
-    responseResult: unknown,
-  ): void {
-    if (action !== 'addNote') {
-      return;
-    }
-    const duplicateNoteIds = this.getRequestDuplicateNoteIds(requestJson);
-    if (duplicateNoteIds.length === 0) {
-      return;
-    }
-    const noteId = this.collectSingleResultId(responseResult)[0];
-    if (!noteId) {
-      return;
-    }
-    this.deps.trackAddedDuplicateNoteIds?.(noteId, duplicateNoteIds);
-  }
-
-  private getForwardRequestBody(
-    rawBody: Buffer,
-    requestJson: Record<string, unknown> | null,
-  ): Buffer {
-    if (!requestJson) {
-      return rawBody;
-    }
-
-    const sanitized = this.sanitizeRequestJson(requestJson);
-    if (sanitized === requestJson) {
-      return rawBody;
-    }
-
-    return Buffer.from(JSON.stringify(sanitized), 'utf8');
-  }
-
-  private sanitizeRequestJson(requestJson: Record<string, unknown>): Record<string, unknown> {
-    const action =
-      typeof requestJson.action === 'string' ? requestJson.action : String(requestJson.action ?? '');
-    if (action !== 'addNote') {
-      return requestJson;
-    }
-
-    const params =
-      requestJson.params && typeof requestJson.params === 'object'
-        ? (requestJson.params as Record<string, unknown>)
-        : null;
-    if (!params || !Object.prototype.hasOwnProperty.call(params, 'subminerDuplicateNoteIds')) {
-      return requestJson;
-    }
-
-    const nextParams = { ...params };
-    delete nextParams.subminerDuplicateNoteIds;
-    return {
-      ...requestJson,
-      params: nextParams,
-    };
-  }
-
-  private getRequestDuplicateNoteIds(requestJson: Record<string, unknown>): number[] {
-    const params =
-      requestJson.params && typeof requestJson.params === 'object'
-        ? (requestJson.params as Record<string, unknown>)
-        : null;
-    const rawNoteIds = Array.isArray(params?.subminerDuplicateNoteIds)
-      ? params.subminerDuplicateNoteIds
-      : [];
-    return [...new Set(rawNoteIds.filter((entry): entry is number => {
-      return typeof entry === 'number' && Number.isInteger(entry) && entry > 0;
-    }))].sort((left, right) => left - right);
-  }
-
   private requestIncludesAddAction(action: string, requestJson: Record<string, unknown>): boolean {
     if (action === 'addNote' || action === 'addNotes') {
       return true;

src/anki-integration/card-creation.test.ts

@@ -397,178 +397,3 @@ test('CardCreationService uses stream-open-filename for remote media generation'
   assert.deepEqual(audioPaths, ['https://audio.example/videoplayback?mime=audio%2Fwebm']);
   assert.deepEqual(imagePaths, ['https://video.example/videoplayback?mime=video%2Fmp4']);
 });
-
-test('CardCreationService tracks pre-add duplicate note ids for kiku sentence cards', async () => {
-  const trackedDuplicates: Array<{ noteId: number; duplicateNoteIds: number[] }> = [];
-  const duplicateLookupExpressions: string[] = [];
-
-  const service = new CardCreationService({
-    getConfig: () =>
-      ({
-        deck: 'Mining',
-        fields: {
-          word: 'Expression',
-          sentence: 'Sentence',
-          audio: 'SentenceAudio',
-        },
-        media: {
-          generateAudio: false,
-          generateImage: false,
-        },
-        behavior: {},
-        ai: false,
-      }) as AnkiConnectConfig,
-    getAiConfig: () => ({}),
-    getTimingTracker: () => ({}) as never,
-    getMpvClient: () =>
-      ({
-        currentVideoPath: '/video.mp4',
-        currentSubText: '字幕',
-        currentSubStart: 1,
-        currentSubEnd: 2,
-        currentTimePos: 1.5,
-        currentAudioStreamIndex: 0,
-      }) as never,
-    client: {
-      addNote: async () => 42,
-      addTags: async () => undefined,
-      notesInfo: async () => [],
-      updateNoteFields: async () => undefined,
-      storeMediaFile: async () => undefined,
-      findNotes: async () => [],
-      retrieveMediaFile: async () => '',
-    },
-    mediaGenerator: {
-      generateAudio: async () => null,
-      generateScreenshot: async () => null,
-      generateAnimatedImage: async () => null,
-    },
-    showOsdNotification: () => undefined,
-    showUpdateResult: () => undefined,
-    showStatusNotification: () => undefined,
-    showNotification: async () => undefined,
-    beginUpdateProgress: () => undefined,
-    endUpdateProgress: () => undefined,
-    withUpdateProgress: async (_message, action) => action(),
-    resolveConfiguredFieldName: () => null,
-    resolveNoteFieldName: () => null,
-    getAnimatedImageLeadInSeconds: async () => 0,
-    extractFields: () => ({}),
-    processSentence: (sentence) => sentence,
-    setCardTypeFields: () => undefined,
-    mergeFieldValue: (_existing, newValue) => newValue,
-    formatMiscInfoPattern: () => '',
-    getEffectiveSentenceCardConfig: () => ({
-      model: 'Sentence',
-      sentenceField: 'Sentence',
-      audioField: 'SentenceAudio',
-      lapisEnabled: false,
-      kikuEnabled: true,
-      kikuFieldGrouping: 'manual',
-      kikuDeleteDuplicateInAuto: false,
-    }),
-    getFallbackDurationSeconds: () => 10,
-    appendKnownWordsFromNoteInfo: () => undefined,
-    isUpdateInProgress: () => false,
-    setUpdateInProgress: () => undefined,
-    trackLastAddedNoteId: () => undefined,
-    findDuplicateNoteIds: async (expression) => {
-      duplicateLookupExpressions.push(expression);
-      return [18, 7, 30, 7];
-    },
-    trackLastAddedDuplicateNoteIds: (noteId, duplicateNoteIds) => {
-      trackedDuplicates.push({ noteId, duplicateNoteIds });
-    },
-  });
-
-  const created = await service.createSentenceCard('重複文', 0, 1);
-
-  assert.equal(created, true);
-  assert.deepEqual(duplicateLookupExpressions, ['重複文']);
-  assert.deepEqual(trackedDuplicates, [{ noteId: 42, duplicateNoteIds: [7, 18, 30] }]);
-});
-
-test('CardCreationService does not track duplicate ids when pre-add lookup returns none', async () => {
-  const trackedDuplicates: Array<{ noteId: number; duplicateNoteIds: number[] }> = [];
-
-  const service = new CardCreationService({
-    getConfig: () =>
-      ({
-        deck: 'Mining',
-        fields: {
-          word: 'Expression',
-          sentence: 'Sentence',
-          audio: 'SentenceAudio',
-        },
-        media: {
-          generateAudio: false,
-          generateImage: false,
-        },
-        behavior: {},
-        ai: false,
-      }) as AnkiConnectConfig,
-    getAiConfig: () => ({}),
-    getTimingTracker: () => ({}) as never,
-    getMpvClient: () =>
-      ({
-        currentVideoPath: '/video.mp4',
-        currentSubText: '字幕',
-        currentSubStart: 1,
-        currentSubEnd: 2,
-        currentTimePos: 1.5,
-        currentAudioStreamIndex: 0,
-      }) as never,
-    client: {
-      addNote: async () => 42,
-      addTags: async () => undefined,
-      notesInfo: async () => [],
-      updateNoteFields: async () => undefined,
-      storeMediaFile: async () => undefined,
-      findNotes: async () => [],
-      retrieveMediaFile: async () => '',
-    },
-    mediaGenerator: {
-      generateAudio: async () => null,
-      generateScreenshot: async () => null,
-      generateAnimatedImage: async () => null,
-    },
-    showOsdNotification: () => undefined,
-    showUpdateResult: () => undefined,
-    showStatusNotification: () => undefined,
-    showNotification: async () => undefined,
-    beginUpdateProgress: () => undefined,
-    endUpdateProgress: () => undefined,
-    withUpdateProgress: async (_message, action) => action(),
-    resolveConfiguredFieldName: () => null,
-    resolveNoteFieldName: () => null,
-    getAnimatedImageLeadInSeconds: async () => 0,
-    extractFields: () => ({}),
-    processSentence: (sentence) => sentence,
-    setCardTypeFields: () => undefined,
-    mergeFieldValue: (_existing, newValue) => newValue,
-    formatMiscInfoPattern: () => '',
-    getEffectiveSentenceCardConfig: () => ({
-      model: 'Sentence',
-      sentenceField: 'Sentence',
-      audioField: 'SentenceAudio',
-      lapisEnabled: false,
-      kikuEnabled: true,
-      kikuFieldGrouping: 'manual',
-      kikuDeleteDuplicateInAuto: false,
-    }),
-    getFallbackDurationSeconds: () => 10,
-    appendKnownWordsFromNoteInfo: () => undefined,
-    isUpdateInProgress: () => false,
-    setUpdateInProgress: () => undefined,
-    trackLastAddedNoteId: () => undefined,
-    findDuplicateNoteIds: async () => [],
-    trackLastAddedDuplicateNoteIds: (noteId, duplicateNoteIds) => {
-      trackedDuplicates.push({ noteId, duplicateNoteIds });
-    },
-  });
-
-  const created = await service.createSentenceCard('重複なし', 0, 1);
-
-  assert.equal(created, true);
-  assert.deepEqual(trackedDuplicates, []);
-});

src/anki-integration/card-creation.ts

@@ -112,11 +112,6 @@ interface CardCreationDeps {
   isUpdateInProgress: () => boolean;
   setUpdateInProgress: (value: boolean) => void;
   trackLastAddedNoteId?: (noteId: number) => void;
-  trackLastAddedDuplicateNoteIds?: (noteId: number, duplicateNoteIds: number[]) => void;
-  findDuplicateNoteIds?: (
-    expression: string,
-    noteInfo: CardCreationNoteInfo,
-  ) => Promise<number[]>;
   recordCardsMinedCallback?: (count: number, noteIds?: number[]) => void;
 }
 
@@ -553,33 +548,6 @@ export class CardCreationService {
           fields[getConfiguredWordFieldName(this.deps.getConfig())] = sentence;
         }
 
-        const pendingNoteInfo = this.createPendingNoteInfo(fields);
-        const pendingNoteFields = Object.fromEntries(
-          Object.entries(fields).map(([name, value]) => [name.toLowerCase(), value]),
-        );
-        const pendingExpressionText = getPreferredWordValueFromExtractedFields(
-          pendingNoteFields,
-          this.deps.getConfig(),
-        ).trim();
-        let duplicateNoteIds: number[] = [];
-        if (
-          sentenceCardConfig.kikuEnabled &&
-          sentenceCardConfig.kikuFieldGrouping !== 'disabled' &&
-          pendingExpressionText &&
-          this.deps.findDuplicateNoteIds
-        ) {
-          try {
-            duplicateNoteIds = sortUniqueNoteIds(
-              await this.deps.findDuplicateNoteIds(pendingExpressionText, pendingNoteInfo),
-            );
-          } catch (error) {
-            log.warn(
-              'Failed to capture pre-add duplicate note ids:',
-              (error as Error).message,
-            );
-          }
-        }
-
         const deck = this.deps.getConfig().deck || 'Default';
         let noteId: number;
         try {
@@ -602,14 +570,6 @@ export class CardCreationService {
           log.warn('Failed to track last added note:', (error as Error).message);
         }
 
-        if (duplicateNoteIds.length > 0) {
-          try {
-            this.deps.trackLastAddedDuplicateNoteIds?.(noteId, duplicateNoteIds);
-          } catch (error) {
-            log.warn('Failed to track duplicate note ids:', (error as Error).message);
-          }
-        }
-
         try {
           this.deps.recordCardsMinedCallback?.(1, [noteId]);
         } catch (error) {
@@ -725,15 +685,6 @@ export class CardCreationService {
     );
   }
 
-  private createPendingNoteInfo(fields: Record<string, string>): CardCreationNoteInfo {
-    return {
-      noteId: -1,
-      fields: Object.fromEntries(
-        Object.entries(fields).map(([name, value]) => [name, { value }]),
-      ),
-    };
-  }
-
   private async mediaGenerateAudio(
     videoPath: string,
     startTime: number,
@@ -813,7 +764,3 @@ export class CardCreationService {
     return `image_${timestamp}.${ext}`;
   }
 }
-
-function sortUniqueNoteIds(noteIds: number[]): number[] {
-  return [...new Set(noteIds)].sort((left, right) => left - right);
-}

src/anki-integration/duplicate.test.ts

@@ -1,6 +1,6 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
-import { findDuplicateNote, findDuplicateNoteIds, type NoteInfo } from './duplicate';
+import { findDuplicateNote, type NoteInfo } from './duplicate';
 
 function createFieldResolver(noteInfo: NoteInfo, preferredName: string): string | null {
   const names = Object.keys(noteInfo.fields);
@@ -267,62 +267,3 @@ test('findDuplicateNote does not disable retries on findNotes calls', async () =
   assert.ok(seenOptions.length > 0);
   assert.ok(seenOptions.every((options) => options?.maxRetries !== 0));
 });
-
-test('findDuplicateNote stops after the first exact-match chunk', async () => {
-  const currentNote: NoteInfo = {
-    noteId: 100,
-    fields: {
-      Expression: { value: '貴様' },
-    },
-  };
-
-  let notesInfoCalls = 0;
-  const candidateIds = Array.from({ length: 51 }, (_, index) => 200 + index);
-  const duplicateId = await findDuplicateNote('貴様', 100, currentNote, {
-    findNotes: async () => candidateIds,
-    notesInfo: async (noteIds) => {
-      notesInfoCalls += 1;
-      return noteIds.map((noteId) => ({
-        noteId,
-        fields: {
-          Expression: { value: noteId === 200 ? '貴様' : `別単語-${noteId}` },
-        },
-      }));
-    },
-    getDeck: () => 'Japanese::Mining',
-    resolveFieldName: (noteInfo, preferredName) => createFieldResolver(noteInfo, preferredName),
-    logWarn: () => {},
-  });
-
-  assert.equal(duplicateId, 200);
-  assert.equal(notesInfoCalls, 1);
-});
-
-test('findDuplicateNoteIds returns no matches when maxMatches is zero', async () => {
-  const currentNote: NoteInfo = {
-    noteId: 100,
-    fields: {
-      Expression: { value: '貴様' },
-    },
-  };
-
-  let notesInfoCalls = 0;
-  const duplicateIds = await findDuplicateNoteIds('貴様', 100, currentNote, {
-    findNotes: async () => [200],
-    notesInfo: async (noteIds) => {
-      notesInfoCalls += 1;
-      return noteIds.map((noteId) => ({
-        noteId,
-        fields: {
-          Expression: { value: '貴様' },
-        },
-      }));
-    },
-    getDeck: () => 'Japanese::Mining',
-    resolveFieldName: (noteInfo, preferredName) => createFieldResolver(noteInfo, preferredName),
-    logWarn: () => {},
-  }, 0);
-
-  assert.deepEqual(duplicateIds, []);
-  assert.equal(notesInfoCalls, 0);
-});