feat: monorepo migration and Netflix/YouTube video overlay

[conoremclaughlin]

Summary

• Monorepo migration: Restructures the repo into packages/extension and packages/web using npm workspaces
• Full Netflix/YouTube video overlay: Center subtitles, right panel transcript, settings menu, progress bar, auto-pause, seek, keyboard shortcuts
• Video subtitle features: Greedy longest-match tokenization, transliteration (pinyin/zhuyin ruby annotations), double subtitles with auto-translation, subtitle pre-caching for instant hover
• Hover UX: Subtitle hover always shows definitions regardless of hover key setting; regular web page text respects the hover key
• Hover popup overhaul: Tone-colored characters + transliteration, stable positioning, dark mode, semicolon-separated definitions
• Popup/Settings/Bookmarks pages: Full React UI with popup, bookmarks (words/sentences tabs, CSV export), and settings (hover key, tone colors, transliteration)
• New message types: search/batch for single-round-trip subtitle pre-caching
• YouTube auto-translation: Falls back to tlang parameter for translated subtitles
• Debug builds: DEBUG=true npm run build enables inline source maps
• Marketing site skeleton: Next.js static export at packages/web
• Tests: E2E tests for hover popup and popup features, unit tests for settings/VTT parser/tone coloring

Test plan

• Install extension from dist/chrome-mv3, verify hover popup on regular Chinese/Korean web pages
• Netflix: verify subtitles render, settings menu, right panel, auto-pause, transliteration toggle, double subtitles
• YouTube: verify subtitles render, settings menu, auto-translation language selector
• Hover key setting (ctrl/alt/etc) gates web page hover but NOT subtitle hover
• npm test passes
• npm run build produces clean output without source maps
• DEBUG=true npm run build includes inline source maps

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
inkah	Error		Apr 14, 2026 3:04am
inkah-api	Error		Apr 14, 2026 3:04am
inkah-awkn	Error		Apr 14, 2026 3:04am

[conoremclaughlin]

Changes requested.

I found a few blockers before this is mergeable:

1. The YouTube page-world XHR hook drops optional XMLHttpRequest.open() arguments, changing semantics for page requests that pass async/credentials. Inline comment on video-injector.content.ts.
2. Korean transliteration is wired to a nonexistent transliteration.romanization property, so the setting cannot render Korean romanization and TypeScript fails. Inline comment on content.ts.
3. Validation is not green:
   • npm run typecheck fails after WXT preparation with the new TS errors above plus custom-event/mock typing errors.
   • npm run test:e2e fails 2/34 in netflix-subtitles.spec.ts because the tests still assert the old inkah-subs marker while the new controller injects inkahsubs styles. I confirmed runtime style injection is present on both Netflix and YouTube, so this looks like stale test assertions.
   • npm run build:web fails in an isolated checkout during Next’s TypeScript worker step: Unexpected response from worker: undefined; therefore npm run build:all also fails after the extension build.

Positive signal: npm run build for the extension passes, and the new video styles do inject on Netflix/YouTube pages when checked manually. No GitHub checks are configured on this PR at head cd9883d.

— Lumen

[conoremclaughlin]

LGTM — the prior review blockers are addressed at c293075.

Verified:

• XHR hook now forwards the original XMLHttpRequest.open() arguments via origOpen.apply(this, arguments as any).
• Korean popup transliteration now reads the existing transliteration.pinyin field.
• Stale E2E style assertions now check the new inkahsubs marker.
• Resolved both prior inline threads.
• npm run typecheck passes.
• npm run build passes.
• DEBUG=true npm run build passes.
• Targeted video E2E passes 3/3.
• Full E2E passes 34/34.
• Dry-run merge into current origin/main succeeds, so GitHub’s earlier mergeable: false looks stale.

Notes, not blockers: this Codex runner still fails worker-based Node commands before useful project output (vitest with a Tinypool IPC error, and next build/build:web with Unexpected response from worker: undefined, reproduced in an isolated checkout on Node 20/22). Given the matching worker/IPC failure mode and Wren’s reported 96/96 unit + build:all success, I’m treating those as local runner issues rather than PR code blockers. git diff --check also reports trailing whitespace inside committed generated packages/web/out chunks; worth cleaning/ignoring generated output in a follow-up, but not blocking this feature PR.

— Lumen