@sovovs/bycli 1.8.0 → 2.1.0

package/clis/zhihu/answer-comments.test.js

@@ -1,287 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@sovovs/bycli/errors';
-import './answer-comments.js';
-import { __test__ as helpers } from './answer-comments.js';
-
-describe('zhihu answer-comments', () => {
-    it('registers as a cookie read command', () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        expect(cmd).toBeDefined();
-        expect(cmd.access).toBe('read');
-        expect(cmd.strategy).toBe('cookie');
-    });
-
-    it('returns flattened comments while limiting only top-level comments', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const goto = vi.fn().mockResolvedValue(undefined);
-        const evaluate = vi.fn().mockImplementation(async (js) => {
-            expect(js).toContain('/api/v4/answers/2036567240334653053/comments?order=normal&limit=20');
-            expect(js).toContain("credentials: 'include'");
-            return {
-                data: [
-                    {
-                        id: 'c1',
-                        author: { member: { id: 'u1', name: 'alice' } },
-                        vote_count: 3,
-                        created_time: 1700000000,
-                        content: '<p>top &#34;one&#34;</p>',
-                    },
-                    {
-                        id: 'r1',
-                        author: { member: { id: 'u2', name: 'bob' } },
-                        reply_to_author: { member: { id: 'u1', name: 'alice' } },
-                        vote_count: 1,
-                        created_time: 1700000100,
-                        content: '<p>reply one</p>',
-                    },
-                    {
-                        id: 'r2',
-                        author: { member: { id: 'u3', name: 'carol' } },
-                        reply_to_author: { member: { id: 'u1', name: 'alice' } },
-                        vote_count: 2,
-                        created_time: 1700000200,
-                        content: '<p>reply two should be capped</p>',
-                    },
-                    {
-                        id: 'c2',
-                        author: { member: { id: 'u4', name: 'dave' } },
-                        vote_count: 4,
-                        created_time: 1700000300,
-                        content: '<p>top two</p>',
-                    },
-                    {
-                        id: 'c3',
-                        author: { member: { id: 'u5', name: 'erin' } },
-                        vote_count: 5,
-                        content: '<p>top three should stop the page walk</p>',
-                    },
-                ],
-                paging: { is_end: false, next: 'https://www.zhihu.com/api/v4/answers/2036567240334653053/comments?offset=20' },
-            };
-        });
-        const page = {
-            goto,
-            getCurrentUrl: vi.fn().mockResolvedValue('https://www.zhihu.com/question/2022852734622114542/answer/2036567240334653053'),
-            evaluate,
-        };
-        await expect(cmd.func(page, {
-            id: 'https://www.zhihu.com/question/2022852734622114542/answer/2036567240334653053',
-            limit: 2,
-            'replies-limit': 1,
-        })).resolves.toEqual([
-            {
-                rank: 1,
-                comment_rank: 1,
-                reply_rank: 0,
-                depth: 0,
-                id: 'c1',
-                parent_id: '',
-                author: 'alice',
-                reply_to: '',
-                likes: 3,
-                created_at: '2023-11-14T22:13:20.000Z',
-                url: 'https://www.zhihu.com/question/2022852734622114542/answer/2036567240334653053#comment-c1',
-                content: 'top "one"',
-            },
-            {
-                rank: 2,
-                comment_rank: 1,
-                reply_rank: 1,
-                depth: 0,
-                id: 'r1',
-                parent_id: '',
-                author: 'bob',
-                reply_to: 'alice',
-                likes: 1,
-                created_at: '2023-11-14T22:15:00.000Z',
-                url: 'https://www.zhihu.com/question/2022852734622114542/answer/2036567240334653053#comment-r1',
-                content: 'reply one',
-            },
-            {
-                rank: 3,
-                comment_rank: 2,
-                reply_rank: 0,
-                depth: 0,
-                id: 'c2',
-                parent_id: '',
-                author: 'dave',
-                reply_to: '',
-                likes: 4,
-                created_at: '2023-11-14T22:18:20.000Z',
-                url: 'https://www.zhihu.com/question/2022852734622114542/answer/2036567240334653053#comment-c2',
-                content: 'top two',
-            },
-        ]);
-        expect(goto).toHaveBeenCalledWith('https://www.zhihu.com/answer/2036567240334653053');
-        expect(evaluate).toHaveBeenCalledTimes(1);
-    });
-
-    it('follows paging.next until enough top-level comments are collected', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const evaluate = vi.fn()
-            .mockResolvedValueOnce({
-                data: [
-                    { id: 'c1', author: { member: { id: 'u1', name: 'alice' } }, content: 'first' },
-                    { id: 'r1', author: { member: { id: 'u2', name: 'bob' } }, reply_to_author: { member: { id: 'u1', name: 'alice' } }, content: 'reply' },
-                ],
-                paging: { is_end: false, next: 'https://www.zhihu.com/api/v4/answers/1/comments?offset=20' },
-            })
-            .mockResolvedValueOnce({
-                data: [
-                    { id: 'c2', author: { member: { id: 'u3', name: 'carol' } }, content: 'second' },
-                ],
-                paging: { is_end: true },
-            });
-        const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-        const rows = await cmd.func(page, { id: '1', limit: 2, 'replies-limit': 0 });
-        expect(rows.map((row) => row.id)).toEqual(['c1', 'c2']);
-        expect(evaluate).toHaveBeenCalledTimes(2);
-        expect(evaluate.mock.calls[1][0]).toContain('offset=20');
-    });
-
-    it('supports typed answer targets', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({
-                data: [
-                    { id: 'c1', author: { member: { id: 'u1', name: 'alice' } }, content: 'typed target comment' },
-                ],
-                paging: { is_end: true },
-            }),
-        };
-        await expect(cmd.func(page, { id: 'answer:2022852734622114542:2036567240334653053', limit: 1, 'replies-limit': 0 }))
-            .resolves.toMatchObject([{ id: 'c1', url: 'https://www.zhihu.com/question/2022852734622114542/answer/2036567240334653053#comment-c1' }]);
-        expect(page.goto).toHaveBeenCalledWith('https://www.zhihu.com/answer/2036567240334653053');
-    });
-
-    it('maps auth failures to AuthRequiredError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ __httpError: 403 }),
-        };
-        await expect(cmd.func(page, { id: '1', limit: 1, 'replies-limit': 0 })).rejects.toBeInstanceOf(AuthRequiredError);
-    });
-
-    it('maps 404 not found to EmptyResultError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ __httpError: 404 }),
-        };
-        await expect(cmd.func(page, { id: '1', limit: 1, 'replies-limit': 0 })).rejects.toBeInstanceOf(EmptyResultError);
-    });
-
-    it('maps malformed responses to CommandExecutionError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ data: {} }),
-        };
-        await expect(cmd.func(page, { id: '1', limit: 1, 'replies-limit': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('maps valid empty comments to EmptyResultError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ data: [], paging: { is_end: true } }),
-        };
-        await expect(cmd.func(page, { id: '1', limit: 1, 'replies-limit': 0 })).rejects.toBeInstanceOf(EmptyResultError);
-    });
-
-    it('rejects malformed pagination next URLs and repeated next URLs', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const malformedNextPage = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({
-                data: [{ id: 'c1', author: { member: { id: 'u1', name: 'alice' } }, content: 'first' }],
-                paging: { is_end: false, next: 'https://evil.example/api/v4/answers/1/comments?offset=20' },
-            }),
-        };
-        await expect(cmd.func(malformedNextPage, { id: '1', limit: 2, 'replies-limit': 0 }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-
-        const repeatedNextPage = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({
-                data: [{ id: 'c1', author: { member: { id: 'u1', name: 'alice' } }, content: 'first' }],
-                paging: { is_end: false, next: 'https://www.zhihu.com/api/v4/answers/1/comments?order=normal&limit=20&offset=0&status=open' },
-            }),
-        };
-        await expect(cmd.func(repeatedNextPage, { id: '1', limit: 2, 'replies-limit': 0 }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('rejects comment rows without stable comment id anchors', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({
-                data: [{ author: { member: { id: 'u1', name: 'alice' } }, content: 'missing id' }],
-                paging: { is_end: true },
-            }),
-        };
-        await expect(cmd.func(page, { id: '1', limit: 1, 'replies-limit': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('rejects null comment items and non-primitive comment ids', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const basePage = (data) => ({
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ data, paging: { is_end: true } }),
-        });
-        await expect(cmd.func(basePage([null]), { id: '1', limit: 1, 'replies-limit': 0 }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(cmd.func(basePage([{ id: { value: 'c1' }, content: 'object id' }]), { id: '1', limit: 1, 'replies-limit': 0 }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(cmd.func(basePage([{ id: true, content: 'boolean id' }]), { id: '1', limit: 1, 'replies-limit': 0 }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('rejects invalid inputs before navigation', async () => {
-        const cmd = getRegistry().get('zhihu/answer-comments');
-        const page = { goto: vi.fn(), evaluate: vi.fn() };
-        await expect(cmd.func(page, { id: 'not-an-answer', limit: 1, 'replies-limit': 0 })).rejects.toBeInstanceOf(ArgumentError);
-        await expect(cmd.func(page, { id: '1', limit: 0, 'replies-limit': 0 })).rejects.toBeInstanceOf(ArgumentError);
-        await expect(cmd.func(page, { id: '1', limit: 1001, 'replies-limit': 0 })).rejects.toBeInstanceOf(ArgumentError);
-        await expect(cmd.func(page, { id: '1', limit: 1, 'replies-limit': -1 })).rejects.toBeInstanceOf(ArgumentError);
-        await expect(cmd.func(page, { id: '1', limit: 1, 'replies-limit': 101 })).rejects.toBeInstanceOf(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-});
-
-describe('zhihu answer-comments helpers', () => {
-    it('parseAnswerTarget handles exact input shapes', () => {
-        expect(helpers.parseAnswerTarget('123')).toEqual({ answerId: '123', questionId: '' });
-        expect(helpers.parseAnswerTarget('answer:10:123')).toEqual({ answerId: '123', questionId: '10' });
-        expect(helpers.parseAnswerTarget('https://www.zhihu.com/question/10/answer/123')).toEqual({ answerId: '123', questionId: '10' });
-        expect(helpers.parseAnswerTarget('https://zhihu.com/answer/123?utm=1')).toEqual({ answerId: '123', questionId: '' });
-        expect(helpers.parseAnswerTarget('https://example.com/question/10/answer/123')).toBeNull();
-    });
-
-    it('normalizeCommentsApiUrl only accepts same-answer Zhihu comments API URLs', () => {
-        expect(helpers.normalizeCommentsApiUrl('https://www.zhihu.com/api/v4/answers/123/comments?offset=20', '123'))
-            .toBe('https://www.zhihu.com/api/v4/answers/123/comments?offset=20');
-        expect(helpers.normalizeCommentsApiUrl('https://api.zhihu.com/answers/123/comments?offset=20', '123'))
-            .toBe('https://www.zhihu.com/api/v4/answers/123/comments?offset=20');
-        expect(helpers.normalizeCommentsApiUrl('https://www.zhihu.com/api/v4/answers/999/comments?offset=20', '123')).toBe('');
-        expect(helpers.normalizeCommentsApiUrl('https://evil.example/api/v4/answers/123/comments?offset=20', '123')).toBe('');
-    });
-
-    it('buildRows keeps replies flat without guessing parent comment ids', () => {
-        const rows = helpers.buildRows([
-            { id: 'c1', author: { member: { id: 'u1', name: 'alice' } }, content: 'top' },
-            { id: 'r1', author: { member: { id: 'u2', name: 'bob' } }, reply_to_author: { member: { id: 'u1', name: 'alice' } }, content: 'reply' },
-            { id: 'r2', author: { member: { id: 'u3', name: 'carol' } }, reply_to_author: { member: { id: 'u2', name: 'bob' } }, content: 'nested' },
-        ], { answerId: 'a1', questionId: 'q1', topLevelLimit: 1, repliesLimit: 5 }).rows;
-        expect(rows.map((row) => [row.id, row.parent_id, row.depth, row.comment_rank, row.reply_rank])).toEqual([
-            ['c1', '', 0, 1, 0],
-            ['r1', '', 0, 1, 1],
-            ['r2', '', 0, 1, 2],
-        ]);
-    });
-});

package/clis/zhihu/answer-detail.test.js

@@ -1,338 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@sovovs/bycli/errors';
-import './answer-detail.js';
-import { __test__ as helpers } from './answer-detail.js';
-
-describe('zhihu answer-detail', () => {
-    it('registers as a cookie read command', () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        expect(cmd).toBeDefined();
-        expect(cmd.access).toBe('read');
-        expect(cmd.strategy).toBe('cookie');
-    });
-
-    it('fetches a single answer by numeric id and returns one row', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const goto = vi.fn().mockResolvedValue(undefined);
-        const evaluate = vi.fn().mockImplementation(async (js) => {
-            // The adapter must call the `/api/v4/answers/<id>` endpoint
-            // (not the question→answers listing) and request the rich
-            // include set so the row carries content + counts + question.
-            expect(js).toContain('/api/v4/answers/1937205528846655537?include=content');
-            expect(js).toContain('voteup_count');
-            expect(js).toContain('comment_count');
-            expect(js).toContain('question');
-            expect(js).toContain("credentials: 'include'");
-            return {
-                // Real Zhihu API returns `id` as a JSON number, which
-                // *loses precision* in browser JSON.parse for ids
-                // above 2^53 (Number.MAX_SAFE_INTEGER). The adapter
-                // must not trust this field for the canonical id —
-                // it must anchor the row id to the parsed input
-                // instead. We pass a deliberately wrong value below
-                // to lock that contract in.
-                id: 0,
-                author: { name: 'Ricky' },
-                voteup_count: 1234,
-                comment_count: 56,
-                created_time: 1700000000,
-                updated_time: 1700001000,
-                content: '<p>这是<strong>第一段</strong></p><br/><p>第二段。</p>',
-                question: { id: 630517537, title: '回想自己的人生阅历，你最想教给孩子们的一个道理是什么？' },
-            };
-        });
-        const page = { goto, evaluate };
-        const rows = await cmd.func(page, { id: '1937205528846655537', 'max-content': 0 });
-        expect(rows).toHaveLength(1);
-        expect(rows[0]).toMatchObject({
-            id: '1937205528846655537',
-            author: 'Ricky',
-            votes: 1234,
-            comments: 56,
-            question_id: '630517537',
-            question_title: '回想自己的人生阅历，你最想教给孩子们的一个道理是什么？',
-            url: 'https://www.zhihu.com/question/630517537/answer/1937205528846655537',
-            created_at: '2023-11-14T22:13:20.000Z',
-            updated_at: '2023-11-14T22:30:00.000Z',
-        });
-        // Block-level tags should become real newlines, not be collapsed flat.
-        expect(rows[0].content).toBe('这是第一段\n\n第二段。');
-        expect(goto).toHaveBeenCalledWith('https://www.zhihu.com/answer/1937205528846655537');
-    });
-
-    it('accepts a full Zhihu answer URL as id, preserving full id precision', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const evaluate = vi.fn().mockResolvedValue({
-            // Same precision-loss trap as above: `data.id` from the
-            // real API would round to `1937205528846655500`. Pass a
-            // wrong value here to assert the adapter ignores it and
-            // anchors to the parsed URL instead.
-            id: 0,
-            author: { name: 'Ricky' },
-            voteup_count: 1,
-            comment_count: 0,
-            content: '<p>hello</p>',
-            // The input question id is the string-safe source of truth
-            // when API JSON numeric ids have already lost precision.
-            question: { id: 2021881398772981800, title: 'Q' },
-        });
-        const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-        const rows = await cmd.func(page, {
-            id: 'https://www.zhihu.com/question/2021881398772981878/answer/1937205528846655537',
-            'max-content': 0,
-        });
-        expect(rows[0].id).toBe('1937205528846655537');
-        expect(rows[0].question_id).toBe('2021881398772981878');
-        expect(rows[0].url).toBe('https://www.zhihu.com/question/2021881398772981878/answer/1937205528846655537');
-        expect(evaluate.mock.calls[0][0]).toContain('/api/v4/answers/1937205528846655537?');
-    });
-
-    it('accepts the typed-target form answer:<qid>:<aid>', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const evaluate = vi.fn().mockResolvedValue({
-            id: 999,
-            author: { name: 'bob' },
-            voteup_count: 0,
-            comment_count: 0,
-            content: '<p>x</p>',
-            question: { id: 0, title: 'Q' },
-        });
-        const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-        const rows = await cmd.func(page, { id: 'answer:2021881398772981878:999', 'max-content': 0 });
-        expect(rows[0].id).toBe('999');
-        expect(rows[0].question_id).toBe('2021881398772981878');
-        expect(evaluate.mock.calls[0][0]).toContain('/api/v4/answers/999?');
-    });
-
-    it('uses the redirected canonical URL as question id source for bare answer ids', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            getCurrentUrl: vi.fn().mockResolvedValue('https://www.zhihu.com/question/2021881398772981878/answer/999'),
-            evaluate: vi.fn().mockResolvedValue({
-                id: 999,
-                author: { name: 'bob' },
-                voteup_count: 0,
-                comment_count: 0,
-                content: '<p>x</p>',
-                question: { id: 2021881398772981800, title: 'Q' },
-            }),
-        };
-        const rows = await cmd.func(page, { id: '999', 'max-content': 0 });
-        expect(rows[0].question_id).toBe('2021881398772981878');
-        expect(rows[0].url).toBe('https://www.zhihu.com/question/2021881398772981878/answer/999');
-    });
-
-    it('uses API question url as a string-safe fallback when the browser URL is unavailable', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({
-                id: 999,
-                author: { name: 'bob' },
-                voteup_count: 0,
-                comment_count: 0,
-                content: '<p>x</p>',
-                question: {
-                    id: 2021881398772981800,
-                    url: 'https://www.zhihu.com/api/v4/questions/2021881398772981878',
-                    title: 'Q',
-                },
-            }),
-        };
-        const rows = await cmd.func(page, { id: '999', 'max-content': 0 });
-        expect(rows[0].question_id).toBe('2021881398772981878');
-    });
-
-    it('returns the full stripped body when --max-content is 0 (default)', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const longBody = 'x'.repeat(5000);
-        const evaluate = vi.fn().mockResolvedValue({
-            id: 1,
-            author: { name: 'a' },
-            voteup_count: 0,
-            comment_count: 0,
-            content: `<p>${longBody}</p>`,
-            question: { id: 2, title: 'Q' },
-        });
-        const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-        const rows = await cmd.func(page, { id: '1', 'max-content': 0 });
-        expect(rows[0].content.length).toBe(5000);
-    });
-
-    it('respects --max-content as an opt-in cap', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const longBody = 'x'.repeat(5000);
-        const evaluate = vi.fn().mockResolvedValue({
-            id: 1,
-            author: { name: 'a' },
-            voteup_count: 0,
-            comment_count: 0,
-            content: `<p>${longBody}</p>`,
-            question: { id: 2, title: 'Q' },
-        });
-        const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-        const rows = await cmd.func(page, { id: '1', 'max-content': 100 });
-        expect(rows[0].content.length).toBe(100);
-    });
-
-    it('falls back to bare /answer/<id> URL when the response is missing question metadata', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const evaluate = vi.fn().mockResolvedValue({
-            id: 42,
-            author: { name: 'alice' },
-            voteup_count: 0,
-            comment_count: 0,
-            content: '<p>orphan answer</p>',
-            // no `question` field at all
-        });
-        const page = { goto: vi.fn().mockResolvedValue(undefined), evaluate };
-        const rows = await cmd.func(page, { id: '42', 'max-content': 0 });
-        expect(rows[0].question_id).toBe('');
-        expect(rows[0].question_title).toBe('');
-        expect(rows[0].url).toBe('https://www.zhihu.com/answer/42');
-    });
-
-    it('maps 401/403 to AuthRequiredError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ __httpError: 403 }),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(AuthRequiredError);
-    });
-
-    it('maps 404 to EmptyResultError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ __httpError: 404 }),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(EmptyResultError);
-    });
-
-    it('maps other HTTP failures to CommandExecutionError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ __httpError: 500 }),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('treats a null evaluate response as a fetch error', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue(null),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('wraps browser navigation failures as CommandExecutionError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockRejectedValue(new Error('navigation failed')),
-            evaluate: vi.fn(),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-
-    it('wraps malformed JSON responses as CommandExecutionError', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ __malformedJson: 'Unexpected token <' }),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('rejects in-band error payloads instead of returning empty success rows', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ error: { message: 'not found' } }),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('rejects payloads missing answer content instead of fabricating a row', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValue({ id: 1, author: { name: 'ghost' } }),
-        };
-        await expect(cmd.func(page, { id: '1', 'max-content': 0 })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('rejects non-numeric answer ids before navigation', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = { goto: vi.fn(), evaluate: vi.fn() };
-        await expect(cmd.func(page, { id: "abc'; alert(1); //", 'max-content': 0 })).rejects.toBeInstanceOf(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-
-    it('rejects negative --max-content before navigation', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = { goto: vi.fn(), evaluate: vi.fn() };
-        await expect(cmd.func(page, { id: '1', 'max-content': -5 })).rejects.toBeInstanceOf(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-    });
-
-    it('rejects invalid URL identities before navigation', async () => {
-        const cmd = getRegistry().get('zhihu/answer-detail');
-        const page = { goto: vi.fn(), evaluate: vi.fn() };
-        for (const id of [
-            'https://example.com/foo/bar',
-            'http://www.zhihu.com/question/10/answer/123',
-            'https://www.zhihu.com/question/10/answer/123/extra',
-            'https://www.zhihu.com.evil.com/question/10/answer/123',
-            'https://user:pass@www.zhihu.com/question/10/answer/123',
-        ]) {
-            await expect(cmd.func(page, { id, 'max-content': 0 })).rejects.toBeInstanceOf(ArgumentError);
-        }
-        expect(page.goto).not.toHaveBeenCalled();
-    });
-});
-
-describe('zhihu answer-detail helpers', () => {
-    it('stripHtml drops tags and decodes common entities', () => {
-        const out = helpers.stripHtml('<p>hi&nbsp;there &amp; you</p><p>second</p>');
-        expect(out).toBe('hi there & you\n\nsecond');
-    });
-
-    it('stripHtml decodes numeric entities', () => {
-        expect(helpers.stripHtml('&#34;中文&#34; &#x26; &#39;test&#39;')).toBe('"中文" & \'test\'');
-    });
-
-    it('stripHtml keeps invalid numeric entities unchanged', () => {
-        expect(helpers.stripHtml('bad &#9999999999; entity')).toBe('bad &#9999999999; entity');
-    });
-
-    it('stripHtml maps <br> to single newline', () => {
-        expect(helpers.stripHtml('a<br>b<br/>c')).toBe('a\nb\nc');
-    });
-
-    it('parseAnswerTarget handles exact input shapes', () => {
-        expect(helpers.parseAnswerTarget('123')).toEqual({ answerId: '123', questionId: '' });
-        expect(helpers.parseAnswerTarget('answer:10:123')).toEqual({ answerId: '123', questionId: '10' });
-        expect(helpers.parseAnswerTarget('https://www.zhihu.com/question/10/answer/123')).toEqual({ answerId: '123', questionId: '10' });
-        expect(helpers.parseAnswerTarget('https://zhihu.com/answer/123?utm=1#x')).toEqual({ answerId: '123', questionId: '' });
-        expect(helpers.parseAnswerTarget('http://www.zhihu.com/question/10/answer/123')).toBeNull();
-        expect(helpers.parseAnswerTarget('https://www.zhihu.com/question/10/answer/123/extra')).toBeNull();
-    });
-
-    it('extractAnswerId keeps the legacy helper contract for tests', () => {
-        expect(helpers.extractAnswerId('123')).toBe('123');
-        expect(helpers.extractAnswerId('answer:10:123')).toBe('123');
-        expect(helpers.extractAnswerId('https://www.zhihu.com/question/10/answer/123')).toBe('123');
-        expect(helpers.extractAnswerId('https://www.zhihu.com/answer/123')).toBe('123');
-        expect(helpers.extractAnswerId('  123  ')).toBe('123');
-        expect(helpers.extractAnswerId('')).toBeNull();
-        expect(helpers.extractAnswerId('not-an-id')).toBeNull();
-        expect(helpers.extractAnswerId('https://example.com/answer/123')).toBeNull();
-    });
-});