Skip to content

fix(notebooklm): support redirected domain - #2228

Open
lingjiuu wants to merge 1 commit into
jackwener:mainfrom
lingjiuu:fix/notebooklm-redirected-domain
Open

fix(notebooklm): support redirected domain#2228
lingjiuu wants to merge 1 commit into
jackwener:mainfrom
lingjiuu:fix/notebooklm-redirected-domain

Conversation

@lingjiuu

@lingjiuu lingjiuu commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

NotebookLM now redirects notebooklm.google.com to notebook.google.com. The adapter previously treated the redirected page as unavailable before notebooklm list could use its existing RPC or DOM paths.

This change:

  • accepts both NotebookLM hosts while keeping notebooklm.google.com canonical for generated URLs;
  • uses a same-origin relative RPC path so requests follow the active page origin;
  • preserves session validation before the RPC-first, DOM-fallback flow;
  • updates notebook URL parsing and status for the redirected host;
  • adds regression coverage for both transport and fallback behavior.

Related issue: Closes #2214

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🌐 New site adapter
  • 📝 Documentation
  • ♻️ Refactor
  • 🔧 CI / build / tooling

Checklist

  • I ran the checks relevant to this PR
  • I updated tests or docs if needed
  • I included live CLI output when useful

Documentation (if adding/modifying an adapter)

  • Added a new adapter documentation page
  • Updated adapter indexes or sidebar
  • Updated README command discoverability
  • No command argument changes were needed
  • Expected adapter failures use typed OpenCLI errors

Screenshots / Output

The failure was reproduced with an authenticated Chrome session: https://notebooklm.google.com/ redirected to https://notebook.google.com/?pli=1.

After the fix:

$ node dist/src/main.js notebooklm list -f json
# authenticated Chrome session redirected to notebook.google.com
# exit code: 0; returned 18 notebook rows through RPC
# no NOTEBOOKLM_UNAVAILABLE error

Checks completed:

  • npx vitest run --project adapter clis/notebooklm — 18 files / 105 tests
  • npx vitest run --project adapter — 501 files / 5206 tests
  • npm run typecheck
  • git diff --check
  • Live verification with an authenticated Chrome session

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[autofix] notebooklm/list: NOTEBOOKLM_UNAVAILABLE

1 participant