@sovovs/bycli 1.8.0 → 2.1.0

package/clis/zhihu/answer.test.js

@@ -1,54 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import './answer.js';
-describe('zhihu answer', () => {
-    it('registers as a cookie browser command', () => {
-        const cmd = getRegistry().get('zhihu/answer');
-        expect(cmd).toBeDefined();
-        expect(cmd.strategy).toBe('cookie');
-        expect(cmd.browser).toBe(true);
-    });
-    it('creates an answer via API and returns result', async () => {
-        const cmd = getRegistry().get('zhihu/answer');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn()
-                .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ ok: true, id: '42', url: 'https://www.zhihu.com/question/1/answer/42' }),
-        };
-        const rows = await cmd.func(page, { target: 'question:1', text: 'hello', execute: true });
-        expect(rows).toEqual([
-            expect.objectContaining({
-                outcome: 'created',
-                created_target: 'answer:1:42',
-                author_identity: 'alice',
-            }),
-        ]);
-    });
-    it('throws on API error', async () => {
-        const cmd = getRegistry().get('zhihu/answer');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn()
-                .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ ok: false, status: 400, message: 'already answered' }),
-        };
-        await expect(cmd.func(page, { target: 'question:1', text: 'hello', execute: true }))
-            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
-    });
-    it('requires the answer API response to include the created id', async () => {
-        const cmd = getRegistry().get('zhihu/answer');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn()
-                .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ ok: false, status: 200, message: 'Answer API response did not include a created answer id' }),
-        };
-        await expect(cmd.func(page, { target: 'question:1', text: 'hello', execute: true }))
-            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
-        expect(page.evaluate.mock.calls[1][0]).toContain('Answer API response did not include a created answer id');
-    });
-});

package/clis/zhihu/collection.test.js

@@ -1,337 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@sovovs/bycli/errors';
-
-// Mock logger
-vi.mock('@sovovs/bycli/logger', () => ({
-  log: {
-    info: vi.fn(),
-    status: vi.fn(),
-    success: vi.fn(),
-    warn: vi.fn(),
-    error: vi.fn(),
-    verbose: vi.fn(),
-    debug: vi.fn(),
-    step: vi.fn(),
-    stepResult: vi.fn(),
-  },
-}));
-
-import './collection.js';
-
-describe('zhihu collection', () => {
-  it('returns collection items from the Zhihu API', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    expect(cmd?.func).toBeTypeOf('function');
-
-    const goto = vi.fn().mockResolvedValue(undefined);
-    const evaluate = vi.fn().mockImplementation(async (js) => {
-      expect(js).toContain('collections/83283292/items');
-      expect(js).toContain("credentials: 'include'");
-      return {
-        data: [
-          {
-            content: {
-              type: 'answer',
-              id: 123456,
-              question: { id: 789012, title: '"Test" & Question' },
-              author: { name: 'test_author' },
-              voteup_count: 42,
-              content: '<p>&#34;Test&#34; &#x26; answer content</p>',
-              url: 'https://www.zhihu.com/question/789012/answer/123456',
-            },
-          },
-        ],
-        paging: { totals: 100 },
-      };
-    });
-
-    const page = { goto, evaluate };
-
-    const result = await cmd.func(page, { id: '83283292', offset: 0, limit: 20 });
-    
-    expect(result).toHaveLength(1);
-    expect(result[0]).toMatchObject({
-      rank: 1,
-      type: 'answer',
-      title: '"Test" & Question',
-      author: 'test_author',
-      votes: 42,
-      excerpt: '"Test" & answer content',
-      url: 'https://www.zhihu.com/question/789012/answer/123456',
-    });
-
-    expect(goto).toHaveBeenCalledWith('https://www.zhihu.com');
-    expect(evaluate).toHaveBeenCalledTimes(1);
-  });
-
-  it('handles article type items', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const evaluate = vi.fn().mockResolvedValue({
-      data: [
-        {
-          content: {
-            type: 'article',
-            id: 987654,
-            title: 'Test Article',
-            author: { name: 'article_author' },
-            voteup_count: 100,
-            content: '<p>Article content</p>',
-            url: 'https://zhuanlan.zhihu.com/p/987654',
-          },
-        },
-      ],
-      paging: { totals: 50 },
-    });
-
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-
-    const result = await cmd.func(page, { id: '83283292', offset: 0, limit: 20 });
-    
-    expect(result[0]).toMatchObject({
-      type: 'article',
-      title: 'Test Article',
-      author: 'article_author',
-      votes: 100,
-    });
-  });
-
-  it('handles pin type items', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const evaluate = vi.fn().mockResolvedValue({
-      data: [
-        {
-          content: {
-            type: 'pin',
-            id: 111222,
-            author: { name: 'pin_author' },
-            reaction_count: 25,
-            content: [{ content: 'Pin content here' }],
-            url: 'https://www.zhihu.com/pin/111222',
-          },
-        },
-      ],
-      paging: { totals: 30 },
-    });
-
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-
-    const result = await cmd.func(page, { id: '83283292', offset: 0, limit: 20 });
-    
-    expect(result[0]).toMatchObject({
-      type: 'pin',
-      title: '想法',
-      author: 'pin_author',
-      votes: 25,
-    });
-  });
-
-  it('maps auth failures to AuthRequiredError', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockResolvedValue({ __httpError: 401 }),
-    };
-
-    await expect(
-      cmd.func(page, { id: '83283292', offset: 0, limit: 20 }),
-    ).rejects.toBeInstanceOf(AuthRequiredError);
-  });
-
-  it('maps 403 errors to AuthRequiredError', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockResolvedValue({ __httpError: 403 }),
-    };
-
-    await expect(
-      cmd.func(page, { id: '83283292', offset: 0, limit: 20 }),
-    ).rejects.toBeInstanceOf(AuthRequiredError);
-  });
-
-  it('preserves non-auth fetch failures as CommandExecutionError', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockResolvedValue({ __httpError: 500 }),
-    };
-
-    await expect(
-      cmd.func(page, { id: '83283292', offset: 0, limit: 20 }),
-    ).rejects.toBeInstanceOf(CommandExecutionError);
-  });
-
-  it('handles null evaluate response as fetch error', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockResolvedValue(null),
-    };
-
-    await expect(
-      cmd.func(page, { id: '83283292', offset: 0, limit: 20 }),
-    ).rejects.toBeInstanceOf(CommandExecutionError);
-  });
-
-  it('rejects non-numeric collection IDs', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = { goto: vi.fn(), evaluate: vi.fn() };
-
-    await expect(
-      cmd.func(page, { id: "abc'; alert(1); //", offset: 0, limit: 20 }),
-    ).rejects.toBeInstanceOf(ArgumentError);
-
-    expect(page.goto).not.toHaveBeenCalled();
-    expect(page.evaluate).not.toHaveBeenCalled();
-  });
-
-  it('respects pagination offset', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const evaluate = vi.fn().mockResolvedValue({
-      data: [
-        {
-          content: {
-            type: 'answer',
-            id: 1,
-            question: { id: 1, title: 'Test' },
-            author: { name: 'author' },
-            voteup_count: 10,
-            content: 'Content',
-          },
-        },
-      ],
-      paging: { totals: 100 },
-    });
-
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-
-    const result = await cmd.func(page, { id: '83283292', offset: 40, limit: 20 });
-    
-    expect(result[0].rank).toBe(41); // offset 40 + index 0 + 1
-    expect(evaluate).toHaveBeenCalledWith(
-      expect.stringContaining('offset=40'),
-    );
-  });
-
-  it('rejects invalid offset and limit before navigation', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = { goto: vi.fn(), evaluate: vi.fn() };
-
-    await expect(cmd.func(page, { id: '83283292', offset: -1, limit: 20 }))
-      .rejects.toBeInstanceOf(ArgumentError);
-    await expect(cmd.func(page, { id: '83283292', offset: 0, limit: 0 }))
-      .rejects.toBeInstanceOf(ArgumentError);
-    expect(page.goto).not.toHaveBeenCalled();
-  });
-
-  it('paginates until requested limit and deduplicates items', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const evaluate = vi.fn()
-      .mockResolvedValueOnce({
-        data: [
-          {
-            content: {
-              type: 'answer',
-              id: 1,
-              question: { id: 1, title: 'A' },
-              author: { name: 'alice' },
-              content: 'A',
-            },
-          },
-        ],
-        paging: { totals: 3, is_end: false, next: 'https://www.zhihu.com/api/v4/collections/83283292/items?offset=1&limit=1' },
-      })
-      .mockResolvedValueOnce({
-        data: [
-          {
-            content: {
-              type: 'answer',
-              id: 1,
-              question: { id: 1, title: 'A duplicate' },
-              author: { name: 'alice' },
-              content: 'A',
-            },
-          },
-          {
-            content: {
-              type: 'answer',
-              id: 2,
-              question: { id: 2, title: 'B' },
-              author: { name: 'bob' },
-              content: 'B',
-            },
-          },
-        ],
-        paging: { totals: 3, is_end: true },
-      });
-
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-
-    const result = await cmd.func(page, { id: '83283292', offset: 0, limit: 2 });
-
-    expect(result.map((row) => row.title)).toEqual(['A', 'B']);
-    expect(evaluate).toHaveBeenCalledTimes(2);
-    expect(evaluate.mock.calls[1][0]).toContain('offset=1');
-  });
-
-  it('throws EmptyResultError for empty collection', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockResolvedValue({
-        data: [],
-        paging: { totals: 0 },
-      }),
-    };
-
-    await expect(cmd.func(page, { id: '83283292', offset: 0, limit: 20 }))
-      .rejects.toBeInstanceOf(EmptyResultError);
-  });
-
-  it('fails typed for missing content.type instead of emitting a blank identity row', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const evaluate = vi.fn().mockResolvedValue({
-      data: [
-        {
-          content: {
-            id: 555,
-            question: { id: 666, title: 'No-type Question' },
-            author: { name: 'a' },
-            voteup_count: 1,
-            content: '<p>x</p>',
-            url: 'https://www.zhihu.com/question/666',
-          },
-        },
-      ],
-      paging: { totals: 1 },
-    });
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-    await expect(cmd.func(page, { id: '83283292', offset: 0, limit: 20 }))
-      .rejects.toBeInstanceOf(CommandExecutionError);
-  });
-
-  it('fails typed for supported collection items missing title/url identity', async () => {
-    const cmd = getRegistry().get('zhihu/collection');
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockResolvedValue({
-        data: [
-          {
-            content: {
-              type: 'answer',
-              id: 555,
-              question: { id: 666, title: '' },
-              author: { name: 'a' },
-              voteup_count: 1,
-              content: '<p>x</p>',
-            },
-          },
-        ],
-        paging: { totals: 1 },
-      }),
-    };
-    await expect(cmd.func(page, { id: '83283292', offset: 0, limit: 20 }))
-      .rejects.toBeInstanceOf(CommandExecutionError);
-  });
-});

package/clis/zhihu/collections.test.js

@@ -1,182 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@sovovs/bycli/errors';
-
-// Mock logger
-vi.mock('@sovovs/bycli/logger', () => ({
-  log: {
-    info: vi.fn(),
-    status: vi.fn(),
-    success: vi.fn(),
-    warn: vi.fn(),
-    error: vi.fn(),
-    verbose: vi.fn(),
-    debug: vi.fn(),
-    step: vi.fn(),
-    stepResult: vi.fn(),
-  },
-}));
-
-import './collections.js';
-
-describe('zhihu collections', () => {
-  it('returns list of collections', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    expect(cmd?.func).toBeTypeOf('function');
-
-    const goto = vi.fn().mockResolvedValue(undefined);
-    let callCount = 0;
-    const evaluate = vi.fn().mockImplementation(async (js) => {
-      callCount++;
-      if (callCount === 1) {
-        expect(js).toContain('api/v4/me');
-        return { url_token: 'testuser', id: 'abc123' };
-      }
-      expect(js).toContain('people/testuser/collections');
-      return {
-        data: [
-          { id: 123456, title: '我的收藏夹', item_count: 42, description: '待读' },
-          { id: 789012, title: '技术文章', item_count: 100, description: '' },
-        ],
-        paging: { totals: 2 },
-      };
-    });
-
-    const page = { goto, evaluate };
-    const result = await cmd.func(page, { limit: 20 });
-
-    expect(result).toHaveLength(2);
-    expect(result[0]).toMatchObject({
-      rank: 1,
-      title: '我的收藏夹',
-      item_count: 42,
-      description: '待读',
-      collection_id: '123456',
-    });
-    expect(result[1]).toMatchObject({
-      rank: 2,
-      title: '技术文章',
-      item_count: 100,
-      description: '',
-      collection_id: '789012',
-    });
-    expect(evaluate).toHaveBeenCalledTimes(2);
-  });
-
-  it('returns list of collections with answer_count fallback', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    let callCount = 0;
-    const evaluate = vi.fn().mockImplementation(async () => {
-      callCount++;
-      if (callCount === 1) {
-        return { url_token: 'testuser', id: 'abc123' };
-      }
-      return {
-        data: [{ id: 111, title: '默认收藏夹', answer_count: 15, description: 'test desc' }],
-        paging: { totals: 1 },
-      };
-    });
-
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-    const result = await cmd.func(page, { limit: 20 });
-
-    expect(result).toHaveLength(1);
-    expect(result[0]).toMatchObject({
-      title: '默认收藏夹',
-      item_count: 15,
-      description: 'test desc',
-      collection_id: '111',
-    });
-  });
-
-  it('maps auth failures to AuthRequiredError', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockResolvedValue({ __httpError: 401 }),
-    };
-
-    await expect(cmd.func(page, { limit: 20 }))
-      .rejects.toBeInstanceOf(AuthRequiredError);
-  });
-
-  it('handles missing url_token', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    let callCount = 0;
-    const page = {
-      goto: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn().mockImplementation(async () => {
-        callCount++;
-        if (callCount === 1) return { id: 'abc123' };
-        return {};
-      }),
-    };
-
-    await expect(cmd.func(page, { limit: 20 }))
-      .rejects.toBeInstanceOf(CommandExecutionError);
-  });
-
-  it('respects limit parameter', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    let callCount = 0;
-    const evaluate = vi.fn().mockImplementation(async (js) => {
-      callCount++;
-      if (callCount === 1) {
-        return { url_token: 'testuser', id: 'abc123' };
-      }
-      expect(js).toContain('limit=10');
-      return {
-        data: [{ id: 1, title: 'Test', answer_count: 0, description: '' }],
-        paging: { totals: 1 },
-      };
-    });
-
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-    const result = await cmd.func(page, { limit: 10 });
-
-    expect(result).toHaveLength(1);
-    expect(evaluate).toHaveBeenCalledTimes(2);
-  });
-
-  it('rejects invalid limits before navigation', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    const page = { goto: vi.fn(), evaluate: vi.fn() };
-
-    await expect(cmd.func(page, { limit: 0 })).rejects.toBeInstanceOf(ArgumentError);
-    expect(page.goto).not.toHaveBeenCalled();
-  });
-
-  it('paginates collection list and deduplicates by collection id', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    const evaluate = vi.fn()
-      .mockResolvedValueOnce({ url_token: 'testuser', id: 'abc123' })
-      .mockResolvedValueOnce({
-        data: [{ id: 1, title: 'A', item_count: 1, description: '' }],
-        paging: { totals: 3, is_end: false, next: 'https://www.zhihu.com/api/v4/people/testuser/collections?offset=1&limit=1' },
-      })
-      .mockResolvedValueOnce({
-        data: [
-          { id: 1, title: 'A duplicate', item_count: 1, description: '' },
-          { id: 2, title: 'B', item_count: 2, description: '' },
-        ],
-        paging: { totals: 3, is_end: true },
-      });
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-
-    const result = await cmd.func(page, { limit: 2 });
-
-    expect(result.map((row) => row.title)).toEqual(['A', 'B']);
-    expect(evaluate).toHaveBeenCalledTimes(3);
-    expect(evaluate.mock.calls[2][0]).toContain('offset=1');
-  });
-
-  it('throws EmptyResultError when no collections are returned', async () => {
-    const cmd = getRegistry().get('zhihu/collections');
-    const evaluate = vi.fn()
-      .mockResolvedValueOnce({ url_token: 'testuser', id: 'abc123' })
-      .mockResolvedValueOnce({ data: [], paging: { totals: 0, is_end: true } });
-    const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-
-    await expect(cmd.func(page, { limit: 20 })).rejects.toBeInstanceOf(EmptyResultError);
-  });
-});

package/clis/zhihu/comment.test.js

@@ -1,50 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import './comment.js';
-describe('zhihu comment', () => {
-    it('registers as a cookie browser command', () => {
-        const cmd = getRegistry().get('zhihu/comment');
-        expect(cmd).toBeDefined();
-        expect(cmd.strategy).toBe('cookie');
-        expect(cmd.browser).toBe(true);
-    });
-    it('creates a comment via API and returns result', async () => {
-        const cmd = getRegistry().get('zhihu/comment');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn()
-                .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ ok: true, id: 99, url: 'https://www.zhihu.com/api/v4/comments/99' }),
-        };
-        const rows = await cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true });
-        expect(rows).toEqual([
-            expect.objectContaining({ outcome: 'created', author_identity: 'alice' }),
-        ]);
-    });
-    it('throws on API error', async () => {
-        const cmd = getRegistry().get('zhihu/comment');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn()
-                .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ ok: false, status: 403, message: 'forbidden' }),
-        };
-        await expect(cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true }))
-            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
-    });
-    it('requires the comment API response to include the created id', async () => {
-        const cmd = getRegistry().get('zhihu/comment');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn()
-                .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ ok: false, status: 200, message: 'Comment API response did not include a created comment id' }),
-        };
-        await expect(cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true }))
-            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
-        expect(page.evaluate.mock.calls[1][0]).toContain('Comment API response did not include a created comment id');
-    });
-});

package/clis/zhihu/download.test.js

@@ -1,12 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import TurndownService from 'turndown';
-describe('article markdown conversion', () => {
-    it('renders ordered lists with the original list item content', () => {
-        const html = '<ol><li>First item</li><li>Second item</li></ol>';
-        const td = new TurndownService({ headingStyle: 'atx', bulletListMarker: '-' });
-        const md = td.turndown(html);
-        expect(md).toMatch(/1\.\s+First item/);
-        expect(md).toMatch(/2\.\s+Second item/);
-        expect(md).not.toContain('$1');
-    });
-});