Files
jackwener__opencli/clis/notebooklm/summary.test.ts
jackwener 314bc6f231 refactor: simplify core modules — remove root shims, consolidate error classification, streamline cascade/interceptor, clean up synthesize
1. Remove root-level shim files (errors.ts, logger.ts, registry.ts, types.ts, utils.ts, launcher.ts) — update all ~840 adapter imports to reference src/ directly
2. Consolidate interceptor: reuse shared DISGUISE_FN in tap interceptor instead of reimplementing
3. Unify error classification: single ClassifiedError type with icon/exitCode/hint lookup table, eliminating duplicated pattern matching between resolveExitCode and renderError
4. Simplify cascade probe: replace repetitive switch cases with PROBE_OPTIONS lookup map
12. Clean up synthesize.ts: remove deprecated snake_case field aliases (recommended_args, recommended_columns, recommendedColumnsLegacy) and unnecessary constant aliases

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-05 03:06:47 +08:00

95 lines
3.0 KiB
TypeScript

import { beforeEach, describe, expect, it, vi } from 'vitest';
const {
mockReadNotebooklmSummaryFromPage,
mockGetNotebooklmSummaryViaRpc,
mockGetNotebooklmPageState,
mockRequireNotebooklmSession,
} = vi.hoisted(() => ({
mockReadNotebooklmSummaryFromPage: vi.fn(),
mockGetNotebooklmSummaryViaRpc: vi.fn(),
mockGetNotebooklmPageState: vi.fn(),
mockRequireNotebooklmSession: vi.fn(),
}));
vi.mock('./utils.js', async () => {
const actual = await vi.importActual<typeof import('./utils.js')>('./utils.js');
return {
...actual,
readNotebooklmSummaryFromPage: mockReadNotebooklmSummaryFromPage,
getNotebooklmSummaryViaRpc: mockGetNotebooklmSummaryViaRpc,
getNotebooklmPageState: mockGetNotebooklmPageState,
requireNotebooklmSession: mockRequireNotebooklmSession,
};
});
import { getRegistry } from '../../src/registry.js';
import './summary.js';
describe('notebooklm summary', () => {
const command = getRegistry().get('notebooklm/summary');
beforeEach(() => {
mockReadNotebooklmSummaryFromPage.mockReset();
mockGetNotebooklmSummaryViaRpc.mockReset();
mockGetNotebooklmPageState.mockReset();
mockRequireNotebooklmSession.mockReset();
mockRequireNotebooklmSession.mockResolvedValue(undefined);
mockGetNotebooklmPageState.mockResolvedValue({
url: 'https://notebooklm.google.com/notebook/nb-demo',
title: 'Browser Automation',
hostname: 'notebooklm.google.com',
kind: 'notebook',
notebookId: 'nb-demo',
loginRequired: false,
notebookCount: 1,
});
});
it('returns the current notebook summary from the visible page first', async () => {
mockReadNotebooklmSummaryFromPage.mockResolvedValue({
notebook_id: 'nb-demo',
title: 'Browser Automation',
summary: 'A concise notebook summary.',
url: 'https://notebooklm.google.com/notebook/nb-demo',
source: 'summary-dom',
});
const result = await command!.func!({} as any, {});
expect(result).toEqual([
{
notebook_id: 'nb-demo',
title: 'Browser Automation',
summary: 'A concise notebook summary.',
url: 'https://notebooklm.google.com/notebook/nb-demo',
source: 'summary-dom',
},
]);
expect(mockGetNotebooklmSummaryViaRpc).not.toHaveBeenCalled();
});
it('falls back to rpc summary extraction when no visible summary block is found', async () => {
mockReadNotebooklmSummaryFromPage.mockResolvedValue(null);
mockGetNotebooklmSummaryViaRpc.mockResolvedValue({
notebook_id: 'nb-demo',
title: 'Browser Automation',
summary: 'Summary recovered from rpc.',
url: 'https://notebooklm.google.com/notebook/nb-demo',
source: 'rpc',
});
const result = await command!.func!({} as any, {});
expect(result).toEqual([
{
notebook_id: 'nb-demo',
title: 'Browser Automation',
summary: 'Summary recovered from rpc.',
url: 'https://notebooklm.google.com/notebook/nb-demo',
source: 'rpc',
},
]);
});
});