mirror of
https://github.com/jackwener/OpenCLI.git
synced 2026-09-14 18:25:42 +08:00
314bc6f231
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>
95 lines
3.0 KiB
TypeScript
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',
|
|
},
|
|
]);
|
|
});
|
|
});
|