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>
101 lines
3.0 KiB
TypeScript
101 lines
3.0 KiB
TypeScript
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
|
|
const {
|
|
mockListNotebooklmSourcesViaRpc,
|
|
mockListNotebooklmSourcesFromPage,
|
|
mockGetNotebooklmPageState,
|
|
mockRequireNotebooklmSession,
|
|
} = vi.hoisted(() => ({
|
|
mockListNotebooklmSourcesViaRpc: vi.fn(),
|
|
mockListNotebooklmSourcesFromPage: 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,
|
|
getNotebooklmPageState: mockGetNotebooklmPageState,
|
|
listNotebooklmSourcesViaRpc: mockListNotebooklmSourcesViaRpc,
|
|
listNotebooklmSourcesFromPage: mockListNotebooklmSourcesFromPage,
|
|
requireNotebooklmSession: mockRequireNotebooklmSession,
|
|
};
|
|
});
|
|
|
|
import { getRegistry } from '../../src/registry.js';
|
|
import './source-get.js';
|
|
|
|
describe('notebooklm source-get', () => {
|
|
const command = getRegistry().get('notebooklm/source-get');
|
|
|
|
beforeEach(() => {
|
|
mockListNotebooklmSourcesViaRpc.mockReset();
|
|
mockListNotebooklmSourcesFromPage.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 a source by exact id from rpc results', async () => {
|
|
mockListNotebooklmSourcesViaRpc.mockResolvedValue([
|
|
{
|
|
id: 'src-1',
|
|
notebook_id: 'nb-demo',
|
|
title: 'Release Notes',
|
|
url: 'https://notebooklm.google.com/notebook/nb-demo',
|
|
source: 'rpc',
|
|
type: 'web',
|
|
},
|
|
]);
|
|
|
|
const result = await command!.func!({} as any, { source: 'src-1' });
|
|
|
|
expect(result).toEqual([
|
|
{
|
|
id: 'src-1',
|
|
notebook_id: 'nb-demo',
|
|
title: 'Release Notes',
|
|
url: 'https://notebooklm.google.com/notebook/nb-demo',
|
|
source: 'rpc',
|
|
type: 'web',
|
|
},
|
|
]);
|
|
expect(mockListNotebooklmSourcesFromPage).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('falls back to page results and matches by title when rpc is empty', async () => {
|
|
mockListNotebooklmSourcesViaRpc.mockResolvedValue([]);
|
|
mockListNotebooklmSourcesFromPage.mockResolvedValue([
|
|
{
|
|
id: 'Meeting Notes',
|
|
notebook_id: 'nb-demo',
|
|
title: 'Meeting Notes',
|
|
url: 'https://notebooklm.google.com/notebook/nb-demo',
|
|
source: 'current-page',
|
|
},
|
|
]);
|
|
|
|
const result = await command!.func!({} as any, { source: 'meeting notes' });
|
|
|
|
expect(result).toEqual([
|
|
{
|
|
id: 'Meeting Notes',
|
|
notebook_id: 'nb-demo',
|
|
title: 'Meeting Notes',
|
|
url: 'https://notebooklm.google.com/notebook/nb-demo',
|
|
source: 'current-page',
|
|
},
|
|
]);
|
|
});
|
|
});
|