@jackwener/opencli 1.8.0 → 1.8.1

package/clis/notebooklm/generate-audio.test.js

@@ -0,0 +1,63 @@
+import { describe, expect, it, vi } from 'vitest';
+import { ArgumentError } from '@jackwener/opencli/errors';
+import { getRegistry } from '@jackwener/opencli/registry';
+import { __test__ } from './generate-audio.js';
+
+const { AUDIO_OVERVIEW_CONFIG_BLOCK, buildCreateAudioArgs, parseAudioIdFromResult } = __test__;
+
+describe('notebooklm generate-audio', () => {
+    it('AUDIO_OVERVIEW_CONFIG_BLOCK matches the live-captured wire shape', () => {
+        expect(AUDIO_OVERVIEW_CONFIG_BLOCK).toEqual([
+            2, null, null,
+            [1, null, null, null, null, null, null, null, null, null, [1]],
+            [[1, 4, 2, 3, 6]],
+        ]);
+    });
+
+    it('buildCreateAudioArgs matches the byte-perfect wire format captured from the UI', () => {
+        const projectId = '42ad744e-477d-4198-97b6-a9ae6a663165';
+        const sourceId = '7c7666bd-59e1-42ab-879d-bacfe33325eb';
+        expect(buildCreateAudioArgs(projectId, [sourceId])).toEqual([
+            [2, null, null, [1, null, null, null, null, null, null, null, null, null, [1]], [[1, 4, 2, 3, 6]]],
+            projectId,
+            [null, null, 1, [[[sourceId]]], null, null, [null, [null, null, null, [[sourceId]]]]],
+        ]);
+    });
+
+    it('buildCreateAudioArgs threads multiple sources into both the nested and tail blocks', () => {
+        const a = '11111111-1111-4111-8111-111111111111';
+        const b = '22222222-2222-4222-8222-222222222222';
+        const args = buildCreateAudioArgs('pid', [a, b]);
+        expect(args[2][3]).toEqual([[[a]], [[b]]]);
+        expect(args[2][6][1][3]).toEqual([[a], [b]]);
+    });
+
+    it('parseAudioIdFromResult walks the tree for a UUID-shaped audio id', () => {
+        const id = '38da0e55-2360-4d3e-8573-61b5a6c0c219';
+        expect(parseAudioIdFromResult([[id, 'opencli-audio-benjaminliu', 1]])).toBe(id);
+        expect(parseAudioIdFromResult({ result: { audioId: id } })).toBe(id);
+    });
+
+    it('parseAudioIdFromResult ignores non-UUID strings', () => {
+        expect(parseAudioIdFromResult([[null, 'opencli-audio-benjaminliu', 1]])).toBe('');
+        expect(parseAudioIdFromResult({})).toBe('');
+        expect(parseAudioIdFromResult([])).toBe('');
+        expect(parseAudioIdFromResult(null)).toBe('');
+    });
+
+    it('parseAudioIdFromResult skips notebook/source ids before selecting the generated audio id', () => {
+        const notebookId = '17e2b882-6a01-4c6c-9262-0738dfa2abee';
+        const sourceId = '7c7666bd-59e1-42ab-879d-bacfe33325eb';
+        const audioId = '38da0e55-2360-4d3e-8573-61b5a6c0c219';
+        expect(parseAudioIdFromResult([notebookId, [[sourceId]], [audioId]], [notebookId, sourceId])).toBe(audioId);
+    });
+
+    it('refuses to trigger remote audio generation without --execute', async () => {
+        const command = getRegistry().get('notebooklm/generate-audio');
+        const page = { goto: vi.fn() };
+        await expect(command.func(page, {
+            notebook: '17e2b882-6a01-4c6c-9262-0738dfa2abee',
+        })).rejects.toThrow(ArgumentError);
+        expect(page.goto).not.toHaveBeenCalled();
+    });
+});

package/clis/notebooklm/generate-slides.js

@@ -0,0 +1,106 @@
+import { cli, Strategy } from '@jackwener/opencli/registry';
+import { ArgumentError, CommandExecutionError, EmptyResultError } from '@jackwener/opencli/errors';
+import { NOTEBOOKLM_DOMAIN, NOTEBOOKLM_SITE } from './shared.js';
+import { callNotebooklmRpc } from './rpc.js';
+import { buildNotebooklmNotebookUrl, listNotebooklmSourcesViaRpc, parseNotebooklmNotebookTarget, requireNotebooklmExecute, requireNotebooklmSession } from './utils.js';
+
+const NOTEBOOKLM_CREATE_ARTIFACT_RPC_ID = 'R7cb6c';
+const SLIDE_DECK_CONFIG_BLOCK = [2, null, null, [1, null, null, null, null, null, null, null, null, null, [1]], [[1, 4, 2, 3, 6]]];
+const ARTIFACT_UUID_RE = /^[a-f0-9]{8}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{12}$/i;
+
+function toExcludedUuidSet(excludedIds) {
+    return new Set(excludedIds.map((id) => String(id ?? '').toLowerCase()).filter(Boolean));
+}
+
+export function buildCreateSlidesArgs(projectId, sourceIds, options = {}) {
+    const sourceTuples = sourceIds.map((id) => [[id]]);
+    const language = options.language || 'en';
+    const length = Number.isInteger(options.length) ? options.length : 3;
+    return [
+        SLIDE_DECK_CONFIG_BLOCK,
+        projectId,
+        [
+            null,
+            null,
+            8,
+            sourceTuples,
+            null,
+            null,
+            null, null, null, null, null, null, null, null, null, null,
+            [[null, language, 1, length]],
+        ],
+    ];
+}
+
+export function parseSlidesIdFromResult(result, excludedIds = []) {
+    const excluded = toExcludedUuidSet(excludedIds);
+    if (typeof result === 'string' && ARTIFACT_UUID_RE.test(result) && !excluded.has(result.toLowerCase())) return result;
+    const stack = [result];
+    while (stack.length) {
+        const node = stack.shift();
+        if (typeof node === 'string' && ARTIFACT_UUID_RE.test(node) && !excluded.has(node.toLowerCase())) return node;
+        if (Array.isArray(node)) for (const child of node) stack.push(child);
+        else if (node && typeof node === 'object') for (const v of Object.values(node)) stack.push(v);
+    }
+    return '';
+}
+
+export function parseSlideDeckLength(value) {
+    if (value === undefined || value === '') return 3;
+    const length = Number(value);
+    if (!Number.isInteger(length) || length <= 0) {
+        throw new ArgumentError('--length must be a positive integer');
+    }
+    return length;
+}
+
+cli({
+    site: NOTEBOOKLM_SITE,
+    name: 'generate-slides',
+    access: 'write',
+    description: 'Trigger a Slide Deck (AI presentation) generation for a NotebookLM notebook, using all of its sources',
+    domain: NOTEBOOKLM_DOMAIN,
+    strategy: Strategy.COOKIE,
+    browser: true,
+    navigateBefore: false,
+    args: [
+        { name: 'notebook', positional: true, required: true, help: 'Notebook id from `notebooklm list` or full notebook URL' },
+        { name: 'length', help: 'Slide deck length: 1=Short, 3=Default (default 3)' },
+        { name: 'language', help: 'Language code (default en)' },
+        { name: 'execute', type: 'boolean', help: 'Actually trigger remote NotebookLM slide deck generation' },
+    ],
+    columns: ['notebook_id', 'slides_id', 'source_count', 'status', 'notebook_url'],
+    func: async (page, kwargs) => {
+        const notebookId = parseNotebooklmNotebookTarget(String(kwargs.notebook ?? ''));
+        const length = parseSlideDeckLength(kwargs.length);
+        const language = String(kwargs.language ?? 'en').trim() || 'en';
+        requireNotebooklmExecute(kwargs.execute, 'generate NotebookLM slides');
+        try {
+            await page.goto(buildNotebooklmNotebookUrl(notebookId));
+            await page.wait(2);
+        }
+        catch (error) {
+            throw new CommandExecutionError(`Failed to open NotebookLM notebook ${notebookId}: ${error?.message || error}`);
+        }
+        await requireNotebooklmSession(page);
+        const sources = await listNotebooklmSourcesViaRpc(page);
+        const sourceIds = sources.map((s) => s.id).filter((id) => typeof id === 'string' && id);
+        if (sourceIds.length === 0) {
+            throw new EmptyResultError('notebooklm generate-slides', 'The notebook has no sources; add a source before generating a slide deck.');
+        }
+        const rpc = await callNotebooklmRpc(page, NOTEBOOKLM_CREATE_ARTIFACT_RPC_ID, buildCreateSlidesArgs(notebookId, sourceIds, { length, language }));
+        const slidesId = parseSlidesIdFromResult(rpc.result, [notebookId, ...sourceIds]);
+        if (!slidesId) {
+            throw new CommandExecutionError('NotebookLM CreateArtifact (slides) RPC returned no slide-deck id; the server may have rejected the request.');
+        }
+        return [{
+            notebook_id: notebookId,
+            slides_id: slidesId,
+            source_count: sourceIds.length,
+            status: 'pending',
+            notebook_url: buildNotebooklmNotebookUrl(notebookId),
+        }];
+    },
+});
+
+export const __test__ = { SLIDE_DECK_CONFIG_BLOCK, buildCreateSlidesArgs, parseSlideDeckLength, parseSlidesIdFromResult };

package/clis/notebooklm/generate-slides.test.js

@@ -0,0 +1,75 @@
+import { describe, expect, it, vi } from 'vitest';
+import { ArgumentError } from '@jackwener/opencli/errors';
+import { getRegistry } from '@jackwener/opencli/registry';
+import { __test__ } from './generate-slides.js';
+
+const { SLIDE_DECK_CONFIG_BLOCK, buildCreateSlidesArgs, parseSlideDeckLength, parseSlidesIdFromResult } = __test__;
+
+describe('notebooklm generate-slides', () => {
+    it('SLIDE_DECK_CONFIG_BLOCK reuses the same R7cb6c config envelope as audio', () => {
+        expect(SLIDE_DECK_CONFIG_BLOCK).toEqual([
+            2, null, null,
+            [1, null, null, null, null, null, null, null, null, null, [1]],
+            [[1, 4, 2, 3, 6]],
+        ]);
+    });
+
+    it('buildCreateSlidesArgs matches the live-captured slide-deck wire format', () => {
+        const projectId = '17e2b882-6a01-4c6c-9262-0738dfa2abee';
+        const sourceA = '493b9ddd-453b-4523-b638-bb560b37723c';
+        const sourceB = '182508cf-9afd-4db5-8640-2f8cafcc7111';
+        expect(buildCreateSlidesArgs(projectId, [sourceA, sourceB])).toEqual([
+            [2, null, null, [1, null, null, null, null, null, null, null, null, null, [1]], [[1, 4, 2, 3, 6]]],
+            projectId,
+            [
+                null, null, 8,
+                [[[sourceA]], [[sourceB]]],
+                null, null,
+                null, null, null, null, null, null, null, null, null, null,
+                [[null, 'en', 1, 3]],
+            ],
+        ]);
+    });
+
+    it('buildCreateSlidesArgs honours --language and --length overrides', () => {
+        const args = buildCreateSlidesArgs('pid', ['s1'], { language: 'zh', length: 1 });
+        expect(args[2][16]).toEqual([[null, 'zh', 1, 1]]);
+    });
+
+    it('parseSlideDeckLength defaults empty values and rejects invalid input', () => {
+        expect(parseSlideDeckLength(undefined)).toBe(3);
+        expect(parseSlideDeckLength('')).toBe(3);
+        expect(parseSlideDeckLength('1')).toBe(1);
+        expect(() => parseSlideDeckLength('many')).toThrow(ArgumentError);
+        expect(() => parseSlideDeckLength(0)).toThrow(ArgumentError);
+    });
+
+    it('parseSlidesIdFromResult finds a UUID-shaped slides id anywhere in the tree', () => {
+        const id = '1f8ada7d-cb33-49a4-8498-c5b81c1a899d';
+        expect(parseSlidesIdFromResult([[id, 'opencli-slides-test']])).toBe(id);
+        expect(parseSlidesIdFromResult({ artifactId: id })).toBe(id);
+    });
+
+    it('parseSlidesIdFromResult ignores non-UUID strings and empty inputs', () => {
+        expect(parseSlidesIdFromResult([[null, 'opencli-slides-test']])).toBe('');
+        expect(parseSlidesIdFromResult({})).toBe('');
+        expect(parseSlidesIdFromResult([])).toBe('');
+        expect(parseSlidesIdFromResult(null)).toBe('');
+    });
+
+    it('parseSlidesIdFromResult skips notebook/source ids before selecting the generated deck id', () => {
+        const notebookId = '17e2b882-6a01-4c6c-9262-0738dfa2abee';
+        const sourceId = '493b9ddd-453b-4523-b638-bb560b37723c';
+        const slidesId = '1f8ada7d-cb33-49a4-8498-c5b81c1a899d';
+        expect(parseSlidesIdFromResult([notebookId, [[sourceId]], [slidesId]], [notebookId, sourceId])).toBe(slidesId);
+    });
+
+    it('refuses to trigger remote slide generation without --execute', async () => {
+        const command = getRegistry().get('notebooklm/generate-slides');
+        const page = { goto: vi.fn() };
+        await expect(command.func(page, {
+            notebook: '17e2b882-6a01-4c6c-9262-0738dfa2abee',
+        })).rejects.toThrow(ArgumentError);
+        expect(page.goto).not.toHaveBeenCalled();
+    });
+});

package/clis/notebooklm/open.test.js

@@ -23,18 +23,18 @@ describe('notebooklm open', () => {
         mockRequireNotebooklmSession.mockReset();
         mockRequireNotebooklmSession.mockResolvedValue(undefined);
         mockGetNotebooklmPageState.mockResolvedValue({
-            url: 'https://notebooklm.google.com/notebook/nb-demo',
+            url: 'https://notebooklm.google.com/notebook/17e2b882-1234-1234-1234-abcdef012345',
             title: 'Browser Automation',
             hostname: 'notebooklm.google.com',
             kind: 'notebook',
-            notebookId: 'nb-demo',
+            notebookId: '17e2b882-1234-1234-1234-abcdef012345',
             loginRequired: false,
             notebookCount: 1,
         });
         mockReadCurrentNotebooklm.mockResolvedValue({
-            id: 'nb-demo',
+            id: '17e2b882-1234-1234-1234-abcdef012345',
             title: 'Browser Automation',
-            url: 'https://notebooklm.google.com/notebook/nb-demo',
+            url: 'https://notebooklm.google.com/notebook/17e2b882-1234-1234-1234-abcdef012345',
             source: 'current-page',
         });
     });
@@ -43,12 +43,12 @@ describe('notebooklm open', () => {
             goto: vi.fn(async () => { }),
             wait: vi.fn(async () => { }),
         };
-        const result = await command.func(page, { notebook: 'nb-demo' });
-        expect(page.goto).toHaveBeenCalledWith('https://notebooklm.google.com/notebook/nb-demo');
+        const result = await command.func(page, { notebook: '17e2b882-1234-1234-1234-abcdef012345' });
+        expect(page.goto).toHaveBeenCalledWith('https://notebooklm.google.com/notebook/17e2b882-1234-1234-1234-abcdef012345');
         expect(result).toEqual([{
-                id: 'nb-demo',
+                id: '17e2b882-1234-1234-1234-abcdef012345',
                 title: 'Browser Automation',
-                url: 'https://notebooklm.google.com/notebook/nb-demo',
+                url: 'https://notebooklm.google.com/notebook/17e2b882-1234-1234-1234-abcdef012345',
                 source: 'current-page',
             }]);
     });
@@ -57,7 +57,7 @@ describe('notebooklm open', () => {
             goto: vi.fn(async () => { }),
             wait: vi.fn(async () => { }),
         };
-        await command.func(page, { notebook: 'https://notebooklm.google.com/notebook/nb-demo?pli=1' });
-        expect(page.goto).toHaveBeenCalledWith('https://notebooklm.google.com/notebook/nb-demo');
+        await command.func(page, { notebook: 'https://notebooklm.google.com/notebook/17e2b882-1234-1234-1234-abcdef012345?pli=1' });
+        expect(page.goto).toHaveBeenCalledWith('https://notebooklm.google.com/notebook/17e2b882-1234-1234-1234-abcdef012345');
     });
 });

package/clis/notebooklm/rpc.js

@@ -1,5 +1,13 @@
 import { AuthRequiredError, CliError } from '@jackwener/opencli/errors';
 import { NOTEBOOKLM_DOMAIN } from './shared.js';
+
+export function unwrapNotebooklmEvaluateResult(payload) {
+    if (payload && typeof payload === 'object' && !Array.isArray(payload) && 'session' in payload && 'data' in payload) {
+        return payload.data;
+    }
+    return payload;
+}
+
 export function extractNotebooklmPageAuthFromHtml(html, sourcePath = '/', preferredTokens) {
     const csrfMatch = html.match(/"SNlM0e":"([^"]+)"/);
     const sessionMatch = html.match(/"FdrFJe":"([^"]+)"/);
@@ -8,26 +16,30 @@ export function extractNotebooklmPageAuthFromHtml(html, sourcePath = '/', prefer
     if (!csrfToken || !sessionId) {
         throw new CliError('NOTEBOOKLM_TOKENS', 'NotebookLM page tokens were not found in the current page HTML', 'Open the NotebookLM notebook page in Chrome, wait for it to finish loading, then retry with --verbose if it still fails.');
     }
-    return { csrfToken, sessionId, sourcePath: sourcePath || '/' };
+    return { csrfToken, sessionId, sourcePath: sourcePath || '/', authuser: preferredTokens?.authuser ?? '' };
 }
 async function probeNotebooklmPageAuth(page) {
-    const raw = await page.evaluate(`(() => {
+    const raw = unwrapNotebooklmEvaluateResult(await page.evaluate(`(() => {
     const wiz = window.WIZ_global_data || {};
     const html = document.documentElement.innerHTML;
+    const authMatch = (location.search || '').match(/[?&]authuser=(\\d+)/);
+    const pathMatch = (location.pathname || '').match(/^\\/u\\/(\\d+)\\//);
     return {
       html,
       sourcePath: location.pathname || '/',
       readyState: document.readyState || '',
       csrfToken: typeof wiz.SNlM0e === 'string' ? wiz.SNlM0e : '',
       sessionId: typeof wiz.FdrFJe === 'string' ? wiz.FdrFJe : '',
+      authuser: authMatch ? authMatch[1] : (pathMatch ? pathMatch[1] : ''),
     };
-  })()`);
+  })()`));
     return {
         html: String(raw?.html ?? ''),
         sourcePath: String(raw?.sourcePath ?? '/'),
         readyState: String(raw?.readyState ?? ''),
         csrfToken: String(raw?.csrfToken ?? ''),
         sessionId: String(raw?.sessionId ?? ''),
+        authuser: String(raw?.authuser ?? ''),
     };
 }
 export async function getNotebooklmPageAuth(page) {
@@ -35,7 +47,7 @@ export async function getNotebooklmPageAuth(page) {
     for (let attempt = 0; attempt < 2; attempt += 1) {
         const probe = await probeNotebooklmPageAuth(page);
         try {
-            return extractNotebooklmPageAuthFromHtml(probe.html, probe.sourcePath, { csrfToken: probe.csrfToken, sessionId: probe.sessionId });
+            return extractNotebooklmPageAuthFromHtml(probe.html, probe.sourcePath, { csrfToken: probe.csrfToken, sessionId: probe.sessionId, authuser: probe.authuser });
         }
         catch (error) {
             lastError = error;
@@ -130,7 +142,7 @@ export async function fetchNotebooklmInPage(page, url, options = {}) {
     const method = options.method ?? 'GET';
     const headers = options.headers ?? {};
     const body = options.body ?? '';
-    const raw = await page.evaluate(`(async () => {
+    const raw = unwrapNotebooklmEvaluateResult(await page.evaluate(`(async () => {
     const request = {
       url: ${JSON.stringify(url)},
       method: ${JSON.stringify(method)},
@@ -151,7 +163,7 @@ export async function fetchNotebooklmInPage(page, url, options = {}) {
       body: await response.text(),
       finalUrl: response.url,
     };
-  })()`);
+  })()`));
     return {
         ok: Boolean(raw?.ok),
         status: Number(raw?.status ?? 0),
@@ -162,8 +174,10 @@ export async function fetchNotebooklmInPage(page, url, options = {}) {
 export async function callNotebooklmRpc(page, rpcId, params, options = {}) {
     const auth = await getNotebooklmPageAuth(page);
     const requestBody = buildNotebooklmRpcBody(rpcId, params, auth.csrfToken);
+    const authuser = auth.authuser || '';
     const url = `https://${NOTEBOOKLM_DOMAIN}/_/LabsTailwindUi/data/batchexecute` +
         `?rpcids=${rpcId}&source-path=${encodeURIComponent(auth.sourcePath)}` +
+        (authuser ? `&authuser=${encodeURIComponent(authuser)}` : '') +
         `&hl=${encodeURIComponent(options.hl ?? 'en')}` +
         `&f.sid=${encodeURIComponent(auth.sessionId)}&rt=c`;
     const response = await fetchNotebooklmInPage(page, url, {

package/clis/notebooklm/rpc.test.js

@@ -1,7 +1,13 @@
 import { describe, expect, it, vi } from 'vitest';
 import { AuthRequiredError } from '@jackwener/opencli/errors';
-import { buildNotebooklmRpcBody, extractNotebooklmRpcResult, getNotebooklmPageAuth, parseNotebooklmChunkedResponse, } from './rpc.js';
+import { buildNotebooklmRpcBody, extractNotebooklmRpcResult, getNotebooklmPageAuth, parseNotebooklmChunkedResponse, unwrapNotebooklmEvaluateResult, } from './rpc.js';
 describe('notebooklm rpc transport', () => {
+    it('unwraps Browser Bridge evaluate envelopes', () => {
+        const data = { ok: true };
+        expect(unwrapNotebooklmEvaluateResult({ session: 'site:notebooklm:abc', data })).toBe(data);
+        expect(unwrapNotebooklmEvaluateResult(data)).toBe(data);
+    });
+
     it('extracts auth tokens from the page html via page evaluation', async () => {
         const page = {
             evaluate: vi.fn(async (script) => {
@@ -16,9 +22,27 @@ describe('notebooklm rpc transport', () => {
             csrfToken: 'csrf-123',
             sessionId: 'sess-456',
             sourcePath: '/',
+            authuser: '',
         });
         expect(page.evaluate).toHaveBeenCalledTimes(1);
     });
+    it('extracts auth tokens when page evaluation is wrapped in a Browser Bridge envelope', async () => {
+        const page = {
+            evaluate: vi.fn(async () => ({
+                session: 'site:notebooklm:abc',
+                data: {
+                    html: '<html>"SNlM0e":"csrf-123","FdrFJe":"sess-456"</html>',
+                    sourcePath: '/notebook/nb-demo',
+                },
+            })),
+        };
+        await expect(getNotebooklmPageAuth(page)).resolves.toEqual({
+            csrfToken: 'csrf-123',
+            sessionId: 'sess-456',
+            sourcePath: '/notebook/nb-demo',
+            authuser: '',
+        });
+    });
     it('falls back to WIZ_global_data tokens when html regex data is missing', async () => {
         const page = {
             evaluate: vi.fn(async () => ({
@@ -33,6 +57,7 @@ describe('notebooklm rpc transport', () => {
             csrfToken: 'csrf-wiz',
             sessionId: 'sess-wiz',
             sourcePath: '/notebook/nb-demo',
+            authuser: '',
         });
     });
     it('retries token extraction once when the first probe returns no tokens', async () => {
@@ -58,6 +83,7 @@ describe('notebooklm rpc transport', () => {
             csrfToken: 'csrf-123',
             sessionId: 'sess-456',
             sourcePath: '/notebook/nb-demo',
+            authuser: '',
         });
         expect(page.evaluate).toHaveBeenCalledTimes(2);
     });