feat(notebooklm): add core workflows and downloads

Author: qiaoqiao147

findings.md

@@ -55,4 +55,16 @@ describe('Page.evaluate', () => {
     expect(value).toBe(42);
     expect(sendCommandMock).toHaveBeenCalledTimes(2);
   });
+
+  it('retries once when the daemon reports a detached target during exec', async () => {
+    sendCommandMock
+      .mockRejectedValueOnce(new Error('Detached while handling command.'))
+      .mockResolvedValueOnce(42);
+
+    const page = new Page('site:notebooklm');
+    const value = await page.evaluate('21 + 21');
+
+    expect(value).toBe(42);
+    expect(sendCommandMock).toHaveBeenCalledTimes(2);
+  });
 });

progress.md

@@ -33,6 +33,7 @@ import {
 export function isRetryableSettleError(err: unknown): boolean {
   const message = err instanceof Error ? err.message : String(err);
   return message.includes('Inspected target navigated or closed')
+    || message.includes('Detached while handling command')
     || (message.includes('-32000') && message.toLowerCase().includes('target'));
 }
 

src/browser/page.test.ts

@@ -197,4 +197,31 @@ describe('manifest helper rules', () => {
     getRegistry().delete(screenKey);
     getRegistry().delete(statusKey);
   });
+
+  it('preserves path-like command names in manifest entries', async () => {
+    const site = `manifest-nested-${Date.now()}`;
+    const key = `${site}/source/list`;
+    const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'opencli-manifest-'));
+    tempDirs.push(dir);
+    const file = path.join(dir, `${site}.ts`);
+    fs.writeFileSync(file, `export const nested = cli({ site: '${site}', name: 'source/list' });`);
+
+    const entries = await loadTsManifestEntries(file, site, async () => ({
+      nested: cli({
+        site,
+        name: 'source/list',
+        description: 'nested command',
+      }),
+    }));
+
+    expect(entries).toEqual([
+      expect.objectContaining({
+        site,
+        name: 'source/list',
+        modulePath: `${site}/${site}.js`,
+      }),
+    ]);
+
+    getRegistry().delete(key);
+  });
 });

src/browser/page.ts

@@ -7,7 +7,7 @@
 
 import { Command } from 'commander';
 import chalk from 'chalk';
-import { type CliCommand, fullName, getRegistry, strategyLabel } from './registry.js';
+import { type CliCommand, formatCommandInvocation, fullName, getRegistry, strategyLabel } from './registry.js';
 import { serializeCommand, formatArgSummary } from './serialization.js';
 import { render as renderOutput } from './output.js';
 import { getBrowserFactory, browserSession } from './runtime.js';
@@ -46,6 +46,7 @@ export function runCli(BUILTIN_CLIS: string, USER_CLIS: string): void {
           ? commands.map(serializeCommand)
           : commands.map(c => ({
               command: fullName(c),
+              invocation: formatCommandInvocation(c),
               site: c.site,
               name: c.name,
               aliases: c.aliases?.join(', ') ?? '',
@@ -56,7 +57,7 @@ export function runCli(BUILTIN_CLIS: string, USER_CLIS: string): void {
             }));
         renderOutput(rows, {
           fmt,
-          columns: ['command', 'site', 'name', 'aliases', 'description', 'strategy', 'browser', 'args',
+          columns: ['command', 'invocation', 'site', 'name', 'aliases', 'description', 'strategy', 'browser', 'args',
                      ...(isStructured ? ['columns', 'domain'] : [])],
           title: 'opencli/list',
           source: 'opencli list',
@@ -83,7 +84,7 @@ export function runCli(BUILTIN_CLIS: string, USER_CLIS: string): void {
             ? chalk.green('[public]')
             : chalk.yellow(`[${label}]`);
           const aliases = cmd.aliases?.length ? chalk.dim(` (aliases: ${cmd.aliases.join(', ')})`) : '';
-          console.log(`    ${cmd.name} ${tag}${aliases}${cmd.description ? chalk.dim(` — ${cmd.description}`) : ''}`);
+          console.log(`    ${formatCommandInvocation(cmd).slice(cmd.site.length + 1)} ${tag}${aliases}${cmd.description ? chalk.dim(` — ${cmd.description}`) : ''}`);
         }
         console.log();
       }

src/build-manifest.test.ts

@@ -50,4 +50,63 @@ describe('notebooklm automatic binding', () => {
     await expect(ensureNotebooklmNotebookBinding(page as any)).resolves.toBe(false);
     expect(mockBindCurrentTab).not.toHaveBeenCalled();
   });
+
+  it('does not rebind to another notebook when the real page is already a notebook add-source url', async () => {
+    const page = {
+      getCurrentUrl: async () => 'https://notebooklm.google.com/',
+      evaluate: vi.fn(async () => ({
+        url: 'https://notebooklm.google.com/notebook/nb-demo?addSource=true',
+        title: 'NotebookLM',
+        hostname: 'notebooklm.google.com',
+        kind: 'notebook',
+        notebookId: 'nb-demo',
+        loginRequired: false,
+        notebookCount: 1,
+        path: '/notebook/nb-demo',
+      })),
+      goto: vi.fn(async () => undefined),
+      wait: vi.fn(async () => undefined),
+    };
+
+    await expect(ensureNotebooklmNotebookBinding(page as any)).resolves.toBe(false);
+    expect(mockBindCurrentTab).not.toHaveBeenCalled();
+    expect(page.goto).toHaveBeenCalledWith('https://notebooklm.google.com/notebook/nb-demo');
+  });
+
+  it('canonicalizes the bound notebook page after bind-current lands on add-source', async () => {
+    const page = {
+      getCurrentUrl: async () => 'https://notebooklm.google.com/',
+      evaluate: vi.fn()
+        .mockResolvedValueOnce({
+          url: 'https://notebooklm.google.com/',
+          title: 'NotebookLM',
+          hostname: 'notebooklm.google.com',
+          kind: 'home',
+          notebookId: '',
+          loginRequired: false,
+          notebookCount: 1,
+          path: '/',
+        })
+        .mockResolvedValueOnce({
+          url: 'https://notebooklm.google.com/notebook/nb-live?addSource=true',
+          title: 'NotebookLM',
+          hostname: 'notebooklm.google.com',
+          kind: 'notebook',
+          notebookId: 'nb-live',
+          loginRequired: false,
+          notebookCount: 1,
+          path: '/notebook/nb-live',
+        }),
+      goto: vi.fn(async () => undefined),
+      wait: vi.fn(async () => undefined),
+    };
+
+    mockBindCurrentTab.mockResolvedValue({});
+    await expect(ensureNotebooklmNotebookBinding(page as any)).resolves.toBe(true);
+    expect(mockBindCurrentTab).toHaveBeenCalledWith('site:notebooklm', {
+      matchDomain: 'notebooklm.google.com',
+      matchPathPrefix: '/notebook/',
+    });
+    expect(page.goto).toHaveBeenCalledWith('https://notebooklm.google.com/notebook/nb-live');
+  });
 });

src/cli.ts

@@ -2,7 +2,15 @@ import { describe, expect, it } from 'vitest';
 import { getRegistry } from '../../registry.js';
 import './bind-current.js';
 import './get.js';
+import './language-get.js';
+import './language-list.js';
+import './language-set.js';
 import './note-list.js';
+import './notes-get.js';
+import './source-fulltext.js';
+import './source-get.js';
+import './source-guide.js';
+import './source-list.js';
 
 describe('notebooklm compatibility aliases', () => {
   it('registers use as a compatibility alias for bind-current', () => {
@@ -16,4 +24,20 @@ describe('notebooklm compatibility aliases', () => {
   it('registers notes-list as a compatibility alias for note-list', () => {
     expect(getRegistry().get('notebooklm/notes-list')).toBe(getRegistry().get('notebooklm/note-list'));
   });
+
+  it('remounts source commands onto nested canonical paths while keeping flat aliases', () => {
+    expect(getRegistry().get('notebooklm/source/list')).toBe(getRegistry().get('notebooklm/source-list'));
+    expect(getRegistry().get('notebooklm/source/get')).toBe(getRegistry().get('notebooklm/source-get'));
+    expect(getRegistry().get('notebooklm/source/fulltext')).toBe(getRegistry().get('notebooklm/source-fulltext'));
+    expect(getRegistry().get('notebooklm/source/guide')).toBe(getRegistry().get('notebooklm/source-guide'));
+  });
+
+  it('remounts note and language commands onto nested canonical paths while keeping flat aliases', () => {
+    expect(getRegistry().get('notebooklm/notes/list')).toBe(getRegistry().get('notebooklm/note-list'));
+    expect(getRegistry().get('notebooklm/notes/list')).toBe(getRegistry().get('notebooklm/notes-list'));
+    expect(getRegistry().get('notebooklm/notes/get')).toBe(getRegistry().get('notebooklm/notes-get'));
+    expect(getRegistry().get('notebooklm/language/list')).toBe(getRegistry().get('notebooklm/language-list'));
+    expect(getRegistry().get('notebooklm/language/get')).toBe(getRegistry().get('notebooklm/language-get'));
+    expect(getRegistry().get('notebooklm/language/set')).toBe(getRegistry().get('notebooklm/language-set'));
+  });
 });

src/clis/notebooklm/binding.test.ts

@@ -0,0 +1,61 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+const {
+  mockCreateNotebooklmNotebookViaRpc,
+  mockEnsureNotebooklmHome,
+  mockRequireNotebooklmSession,
+} = vi.hoisted(() => ({
+  mockCreateNotebooklmNotebookViaRpc: vi.fn(),
+  mockEnsureNotebooklmHome: vi.fn(),
+  mockRequireNotebooklmSession: vi.fn(),
+}));
+
+vi.mock('./utils.js', async () => {
+  const actual = await vi.importActual<typeof import('./utils.js')>('./utils.js');
+  return {
+    ...actual,
+    createNotebooklmNotebookViaRpc: mockCreateNotebooklmNotebookViaRpc,
+    ensureNotebooklmHome: mockEnsureNotebooklmHome,
+    requireNotebooklmSession: mockRequireNotebooklmSession,
+  };
+});
+
+import { getRegistry } from '../../registry.js';
+import './create.js';
+
+describe('notebooklm create', () => {
+  const command = getRegistry().get('notebooklm/create');
+
+  beforeEach(() => {
+    mockCreateNotebooklmNotebookViaRpc.mockReset();
+    mockEnsureNotebooklmHome.mockReset();
+    mockRequireNotebooklmSession.mockReset();
+    mockEnsureNotebooklmHome.mockResolvedValue(undefined);
+    mockRequireNotebooklmSession.mockResolvedValue(undefined);
+  });
+
+  it('creates a new notebook via rpc and returns the created notebook row', async () => {
+    mockCreateNotebooklmNotebookViaRpc.mockResolvedValue({
+      id: 'nb-created',
+      title: '新建 Notebook',
+      url: 'https://notebooklm.google.com/notebook/nb-created',
+      source: 'rpc',
+      is_owner: true,
+      created_at: '2026-03-31T09:12:00.000Z',
+      updated_at: '2026-03-31T09:12:00.000Z',
+      emoji: null,
+      source_count: 0,
+    });
+
+    const result = await command!.func!({} as any, { title: '新建 Notebook' });
+
+    expect(mockCreateNotebooklmNotebookViaRpc).toHaveBeenCalledWith(expect.anything(), '新建 Notebook');
+    expect(result).toEqual([
+      expect.objectContaining({
+        id: 'nb-created',
+        title: '新建 Notebook',
+        source: 'rpc',
+      }),
+    ]);
+  });
+});

src/clis/notebooklm/compat.test.ts

@@ -0,0 +1,45 @@
+import { ArgumentError, EmptyResultError } from '../../errors.js';
+import { cli, Strategy } from '../../registry.js';
+import type { IPage } from '../../types.js';
+import { NOTEBOOKLM_DOMAIN, NOTEBOOKLM_SITE } from './shared.js';
+import {
+  createNotebooklmNotebookViaRpc,
+  ensureNotebooklmHome,
+  requireNotebooklmSession,
+} from './utils.js';
+
+cli({
+  site: NOTEBOOKLM_SITE,
+  name: 'create',
+  description: 'Create a new NotebookLM notebook with the given title',
+  domain: NOTEBOOKLM_DOMAIN,
+  strategy: Strategy.COOKIE,
+  browser: true,
+  navigateBefore: false,
+  args: [
+    {
+      name: 'title',
+      positional: true,
+      required: true,
+      help: 'Title for the new notebook',
+    },
+  ],
+  columns: ['id', 'title', 'created_at', 'updated_at', 'source_count', 'url', 'source'],
+  func: async (page: IPage, kwargs) => {
+    await requireNotebooklmSession(page);
+    await ensureNotebooklmHome(page);
+
+    const title = typeof kwargs.title === 'string' ? kwargs.title.trim() : String(kwargs.title ?? '').trim();
+    if (!title) {
+      throw new ArgumentError('The notebook title cannot be empty.');
+    }
+
+    const notebook = await createNotebooklmNotebookViaRpc(page, title);
+    if (notebook) return [notebook];
+
+    throw new EmptyResultError(
+      'opencli notebooklm create',
+      'NotebookLM did not return the created notebook row.',
+    );
+  },
+});