@sovovs/bycli 1.8.0 → 2.1.0

package/clis/xiaoyuzhou/download.test.js

@@ -1,135 +0,0 @@
-import path from 'node:path';
-import { beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-
-const { mockRequestJson, mockLoadCredentials, mockHttpDownload, mockMkdirSync } = vi.hoisted(() => ({
-    mockRequestJson: vi.fn(),
-    mockLoadCredentials: vi.fn(),
-    mockHttpDownload: vi.fn(),
-    mockMkdirSync: vi.fn(),
-}));
-
-vi.mock('./auth.js', async () => {
-    const actual = await vi.importActual('./auth.js');
-    return {
-        ...actual,
-        requestXiaoyuzhouJson: mockRequestJson,
-        loadXiaoyuzhouCredentials: mockLoadCredentials,
-    };
-});
-
-vi.mock('@sovovs/bycli/download', () => ({
-    httpDownload: mockHttpDownload,
-    sanitizeFilename: vi.fn((value) => value.replace(/\s+/g, '_')),
-}));
-
-vi.mock('@sovovs/bycli/download/progress', () => ({
-    formatBytes: vi.fn((size) => `${size} B`),
-}));
-
-vi.mock('node:fs', () => ({
-    mkdirSync: mockMkdirSync,
-}));
-
-await import('./download.js');
-
-let cmd;
-
-function toPosixPath(value) {
-    return value.replaceAll(path.sep, '/');
-}
-
-beforeAll(() => {
-    cmd = getRegistry().get('xiaoyuzhou/download');
-    expect(cmd?.func).toBeTypeOf('function');
-});
-
-describe('xiaoyuzhou download', () => {
-    beforeEach(() => {
-        mockRequestJson.mockReset();
-        mockLoadCredentials.mockReset();
-        mockHttpDownload.mockReset();
-        mockMkdirSync.mockReset();
-        mockLoadCredentials.mockReturnValue({});
-    });
-
-    it('downloads audio from media.source.url into an episode subdirectory', async () => {
-        mockRequestJson.mockResolvedValue({
-            credentials: {},
-            data: {
-                title: 'Hello World',
-                podcast: { title: 'byCLI FM' },
-                media: {
-                    source: {
-                        url: 'https://media.xyzcdn.net/audio/hello-world.mp3?sign=abc',
-                    },
-                },
-            },
-        });
-        mockHttpDownload.mockResolvedValue({ success: true, size: 1234 });
-
-        const result = await cmd.func({
-            id: 'ep123',
-            output: '/tmp/xiaoyuzhou-test',
-        });
-
-        expect(mockRequestJson).toHaveBeenCalledWith('/v1/episode/get', {
-            query: { eid: 'ep123' },
-            credentials: {},
-        });
-        expect(toPosixPath(mockMkdirSync.mock.calls[0][0])).toBe('/tmp/xiaoyuzhou-test/ep123');
-        expect(mockMkdirSync.mock.calls[0][1]).toEqual({ recursive: true });
-        expect(mockHttpDownload).toHaveBeenCalledWith('https://media.xyzcdn.net/audio/hello-world.mp3?sign=abc', expect.stringContaining('/tmp/xiaoyuzhou-test/ep123/ep123_Hello_World.mp3'), {
-            timeout: 60000,
-        });
-        expect(result).toEqual([{
-                title: 'Hello World',
-                podcast: 'byCLI FM',
-                status: 'success',
-                size: '1234 B',
-                file: '/tmp/xiaoyuzhou-test/ep123/ep123_Hello_World.mp3',
-            }]);
-    });
-
-    it('preserves non-mp3 extensions from media.source.url', async () => {
-        mockRequestJson.mockResolvedValue({
-            credentials: {},
-            data: {
-                title: 'Lossless Episode',
-                podcast: { title: 'byCLI FM' },
-                media: {
-                    source: {
-                        url: 'https://media.xyzcdn.net/audio/lossless.m4a',
-                    },
-                },
-            },
-        });
-        mockHttpDownload.mockResolvedValue({ success: true, size: 2048 });
-
-        const result = await cmd.func({
-            id: 'ep456',
-            output: '/tmp/xiaoyuzhou-test',
-        });
-
-        expect(mockHttpDownload.mock.calls[0][1]).toContain('ep456_Lossless_Episode.m4a');
-        expect(result[0].file).toBe('/tmp/xiaoyuzhou-test/ep456/ep456_Lossless_Episode.m4a');
-    });
-
-    it('throws when media.source.url is missing', async () => {
-        mockRequestJson.mockResolvedValue({
-            credentials: {},
-            data: {
-                title: 'No Audio',
-                podcast: { title: 'byCLI FM' },
-                media: {},
-            },
-        });
-
-        await expect(cmd.func({ id: 'ep789', output: '/tmp/xiaoyuzhou-test' })).rejects.toMatchObject({
-            code: 'PARSE_ERROR',
-            message: 'Audio URL not found in episode payload',
-            hint: 'Episode payload does not expose media.source.url',
-        });
-        expect(mockHttpDownload).not.toHaveBeenCalled();
-    });
-});

package/clis/xiaoyuzhou/podcast-episodes.test.js

@@ -1,78 +0,0 @@
-import { beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-
-const { mockRequestJson, mockLoadCredentials } = vi.hoisted(() => ({
-    mockRequestJson: vi.fn(),
-    mockLoadCredentials: vi.fn(),
-}));
-
-vi.mock('./auth.js', async () => {
-    const actual = await vi.importActual('./auth.js');
-    return {
-        ...actual,
-        requestXiaoyuzhouJson: mockRequestJson,
-        loadXiaoyuzhouCredentials: mockLoadCredentials,
-    };
-});
-
-await import('./podcast-episodes.js');
-
-let cmd;
-
-beforeAll(() => {
-    cmd = getRegistry().get('xiaoyuzhou/podcast-episodes');
-    expect(cmd?.func).toBeTypeOf('function');
-});
-
-describe('xiaoyuzhou podcast-episodes', () => {
-    beforeEach(() => {
-        mockRequestJson.mockReset();
-        mockLoadCredentials.mockReset();
-        mockLoadCredentials.mockReturnValue({ access_token: 'access', refresh_token: 'refresh' });
-    });
-
-    it('calls the fixed episode list endpoint with desc ordering', async () => {
-        mockRequestJson.mockResolvedValue({
-            data: [
-                {
-                    eid: 'ep-1',
-                    title: 'Episode 1',
-                    duration: 3661,
-                    playCount: 42,
-                    pubDate: '2026-04-20T10:00:00.000Z',
-                },
-            ],
-        });
-
-        const result = await cmd.func({
-            id: 'podcast-1',
-            limit: 3,
-        });
-
-        expect(mockRequestJson).toHaveBeenCalledWith('/v1/episode/list', {
-            method: 'POST',
-            body: { pid: 'podcast-1', order: 'desc', limit: 3 },
-            credentials: { access_token: 'access', refresh_token: 'refresh' },
-        });
-        expect(result).toEqual([
-            {
-                eid: 'ep-1',
-                title: 'Episode 1',
-                duration: '61:01',
-                plays: 42,
-                date: '2026-04-20',
-            },
-        ]);
-    });
-
-    it('rejects non-positive limits before hitting the API', async () => {
-        await expect(cmd.func({
-            id: 'podcast-1',
-            limit: 0,
-        })).rejects.toMatchObject({
-            code: 'INVALID_ARGUMENT',
-            message: 'limit must be a positive integer',
-        });
-        expect(mockRequestJson).not.toHaveBeenCalled();
-    });
-});

package/clis/xiaoyuzhou/transcript.test.js

@@ -1,195 +0,0 @@
-import path from 'node:path';
-import { beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-
-const { mockLoadCredentials, mockRequestJson, mockFetchTranscriptBody, mockMkdirSync, mockWriteFileSync } = vi.hoisted(() => ({
-    mockLoadCredentials: vi.fn(),
-    mockRequestJson: vi.fn(),
-    mockFetchTranscriptBody: vi.fn(),
-    mockMkdirSync: vi.fn(),
-    mockWriteFileSync: vi.fn(),
-}));
-
-vi.mock('./auth.js', async () => {
-    const actual = await vi.importActual('./auth.js');
-    return {
-        ...actual,
-        loadXiaoyuzhouCredentials: mockLoadCredentials,
-        requestXiaoyuzhouJson: mockRequestJson,
-        fetchXiaoyuzhouTranscriptBody: mockFetchTranscriptBody,
-    };
-});
-
-vi.mock('node:fs', () => ({
-    mkdirSync: mockMkdirSync,
-    writeFileSync: mockWriteFileSync,
-}));
-
-await import('./transcript.js');
-
-let cmd;
-
-function toPosixPath(value) {
-    return value.replaceAll(path.sep, '/');
-}
-
-beforeAll(() => {
-    cmd = getRegistry().get('xiaoyuzhou/transcript');
-    expect(cmd?.func).toBeTypeOf('function');
-});
-
-describe('xiaoyuzhou transcript', () => {
-    beforeEach(() => {
-        mockLoadCredentials.mockReset();
-        mockRequestJson.mockReset();
-        mockFetchTranscriptBody.mockReset();
-        mockMkdirSync.mockReset();
-        mockWriteFileSync.mockReset();
-        mockLoadCredentials.mockReturnValue({ access_token: 'access', refresh_token: 'refresh' });
-    });
-
-    it('downloads transcript json and extracted text files', async () => {
-        mockRequestJson
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {
-                title: 'Transcript Episode',
-                podcast: { title: 'byCLI FM' },
-                transcript: { mediaId: 'media-123' },
-            },
-        })
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {
-                transcriptUrl: 'https://cdn.example.com/transcript.json',
-            },
-        });
-        mockFetchTranscriptBody.mockResolvedValue(JSON.stringify({
-            segments: [{ text: 'hello' }, { text: 'world' }],
-        }));
-        const result = await cmd.func({
-            id: 'ep123',
-            output: '/tmp/xiaoyuzhou-transcripts',
-            json: true,
-            text: true,
-        });
-        expect(mockRequestJson).toHaveBeenNthCalledWith(1, '/v1/episode/get', {
-            query: { eid: 'ep123' },
-            credentials: { access_token: 'access', refresh_token: 'refresh' },
-        });
-        expect(mockRequestJson).toHaveBeenNthCalledWith(2, '/v1/episode-transcript/get', {
-            method: 'POST',
-            body: { eid: 'ep123', mediaId: 'media-123' },
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-        });
-        expect(mockMkdirSync).toHaveBeenCalledWith('/tmp/xiaoyuzhou-transcripts/ep123', { recursive: true });
-        expect(mockWriteFileSync).toHaveBeenNthCalledWith(1, '/tmp/xiaoyuzhou-transcripts/ep123/transcript.json', expect.any(String), 'utf-8');
-        expect(mockWriteFileSync).toHaveBeenNthCalledWith(2, '/tmp/xiaoyuzhou-transcripts/ep123/transcript.txt', 'hello\nworld', 'utf-8');
-        expect(result).toEqual([{
-                title: 'Transcript Episode',
-                podcast: 'byCLI FM',
-                status: 'success',
-                segments: '2',
-                json_file: '/tmp/xiaoyuzhou-transcripts/ep123/transcript.json',
-                text_file: '/tmp/xiaoyuzhou-transcripts/ep123/transcript.txt',
-            }]);
-    });
-
-    it('derives mediaId from episode.media.id when transcript.mediaId is absent', async () => {
-        mockRequestJson
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {
-                title: 'Transcript Episode',
-                podcast: { title: 'byCLI FM' },
-                media: { id: 'media-456' },
-            },
-        })
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {
-                transcriptUrl: 'https://cdn.example.com/transcript.json',
-            },
-        });
-        mockFetchTranscriptBody.mockResolvedValue(JSON.stringify({ text: 'hello' }));
-        await cmd.func({
-            id: 'ep456',
-            output: '/tmp/xiaoyuzhou-transcripts',
-            json: false,
-            text: true,
-        });
-        expect(mockRequestJson.mock.calls[1][1].body.mediaId).toBe('media-456');
-        expect(mockWriteFileSync).toHaveBeenCalledTimes(1);
-        expect(mockWriteFileSync).toHaveBeenCalledWith('/tmp/xiaoyuzhou-transcripts/ep456/transcript.txt', 'hello', 'utf-8');
-    });
-
-    it('throws when transcript url is missing', async () => {
-        mockRequestJson
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {
-                title: 'Transcript Episode',
-                podcast: { title: 'byCLI FM' },
-                transcript: { mediaId: 'media-123' },
-            },
-        })
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {},
-        });
-        await expect(cmd.func({
-            id: 'ep123',
-            output: '/tmp/xiaoyuzhou-transcripts',
-            json: true,
-            text: true,
-        })).rejects.toMatchObject({
-            code: 'EMPTY_RESULT',
-            message: 'Transcript URL not found',
-        });
-        expect(mockWriteFileSync).not.toHaveBeenCalled();
-    });
-
-    it('throws parse_error when transcript text extraction fails', async () => {
-        mockRequestJson
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {
-                title: 'Transcript Episode',
-                podcast: { title: 'byCLI FM' },
-                transcript: { mediaId: 'media-123' },
-            },
-        })
-            .mockResolvedValueOnce({
-            credentials: { access_token: 'access-1', refresh_token: 'refresh-1' },
-            data: {
-                transcriptUrl: 'https://cdn.example.com/transcript.json',
-            },
-        });
-        mockFetchTranscriptBody.mockResolvedValue(JSON.stringify({
-            segments: [{ startAt: 0, endAt: 1 }],
-        }));
-        await expect(cmd.func({
-            id: 'ep123',
-            output: '/tmp/xiaoyuzhou-transcripts',
-            json: true,
-            text: true,
-        })).rejects.toMatchObject({
-            code: 'PARSE_ERROR',
-            message: 'Failed to extract transcript text',
-        });
-        expect(mockWriteFileSync).not.toHaveBeenCalled();
-    });
-
-    it('rejects disabling both json and text outputs', async () => {
-        await expect(cmd.func({
-            id: 'ep123',
-            output: '/tmp/xiaoyuzhou-transcripts',
-            json: false,
-            text: false,
-        })).rejects.toMatchObject({
-            code: 'ARGUMENT',
-            message: 'At least one of --json or --text must be enabled',
-        });
-        expect(mockRequestJson).not.toHaveBeenCalled();
-    });
-});

package/clis/xiaoyuzhou/utils.test.js

@@ -1,39 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { formatDuration, formatDate } from './utils.js';
-describe('formatDuration', () => {
-    it('formats typical duration', () => {
-        expect(formatDuration(3890)).toBe('64:50');
-    });
-    it('formats zero seconds', () => {
-        expect(formatDuration(0)).toBe('0:00');
-    });
-    it('pads single-digit seconds', () => {
-        expect(formatDuration(62)).toBe('1:02');
-    });
-    it('handles exact minutes', () => {
-        expect(formatDuration(120)).toBe('2:00');
-    });
-    it('rounds fractional seconds', () => {
-        expect(formatDuration(65.7)).toBe('1:06');
-    });
-    it('returns dash for NaN', () => {
-        expect(formatDuration(NaN)).toBe('-');
-    });
-    it('returns dash for negative', () => {
-        expect(formatDuration(-1)).toBe('-');
-    });
-    it('returns dash for Infinity', () => {
-        expect(formatDuration(Infinity)).toBe('-');
-    });
-});
-describe('formatDate', () => {
-    it('slices ISO string to date', () => {
-        expect(formatDate('2026-03-15T12:00:00Z')).toBe('2026-03-15');
-    });
-    it('returns dash for empty string', () => {
-        expect(formatDate('')).toBe('-');
-    });
-    it('returns dash for undefined', () => {
-        expect(formatDate(undefined)).toBe('-');
-    });
-});