@agentrhq/webcmd 0.1.2 → 0.2.1

package/clis/zhihu/answer-detail.js

@@ -1,216 +0,0 @@
-import { cli, Strategy } from '@agentrhq/webcmd/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@agentrhq/webcmd/errors';
-import { stripHtml as stripHtmlText } from './text.js';
-
-function stripHtml(html) {
-    return stripHtmlText(html, { preserveBlocks: true });
-}
-
-const ANSWER_ID_RE = /^\d+$/;
-const ANSWER_TYPED_RE = /^answer:(\d+):(\d+)$/;
-const ANSWER_PATH_RE = /^\/question\/(\d+)\/answer\/(\d+)\/?$/;
-const BARE_ANSWER_PATH_RE = /^\/answer\/(\d+)\/?$/;
-const QUESTION_PATH_RE = /^\/question\/(\d+)\/?$/;
-const QUESTION_API_PATH_RE = /^\/api\/v4\/questions\/(\d+)\/?$/;
-
-// Accepts: bare numeric id (`1937205528846655537`), the typed
-// target form used by the existing zhihu write adapters
-// (`answer:<qid>:<aid>`), or the full Zhihu URL pasted from a
-// browser (`https://www.zhihu.com/question/<qid>/answer/<aid>`).
-// Returns string-safe ids, or null when the input does not resolve to
-// any of those exact shapes.
-function parseAnswerTarget(input) {
-    const value = String(input ?? '').trim();
-    if (!value) return null;
-    if (ANSWER_ID_RE.test(value)) return { answerId: value, questionId: '' };
-    const typed = value.match(ANSWER_TYPED_RE);
-    if (typed) return { questionId: typed[1], answerId: typed[2] };
-    try {
-        const url = new URL(value);
-        if (
-            url.protocol !== 'https:' ||
-            url.username ||
-            url.password ||
-            url.port ||
-            (url.hostname !== 'www.zhihu.com' && url.hostname !== 'zhihu.com')
-        ) {
-            return null;
-        }
-        let m = url.pathname.match(ANSWER_PATH_RE);
-        if (m) return { questionId: m[1], answerId: m[2] };
-        m = url.pathname.match(BARE_ANSWER_PATH_RE);
-        if (m) return { answerId: m[1], questionId: '' };
-    } catch {
-        return null;
-    }
-    return null;
-}
-
-function extractAnswerId(input) {
-    return parseAnswerTarget(input)?.answerId ?? null;
-}
-
-function extractQuestionIdFromAnswerUrl(input) {
-    const value = String(input ?? '').trim();
-    if (!value) return '';
-    try {
-        const url = new URL(value);
-        if (url.protocol !== 'https:' || (url.hostname !== 'www.zhihu.com' && url.hostname !== 'zhihu.com')) {
-            return '';
-        }
-        return url.pathname.match(ANSWER_PATH_RE)?.[1]
-            || url.pathname.match(QUESTION_PATH_RE)?.[1]
-            || url.pathname.match(QUESTION_API_PATH_RE)?.[1]
-            || '';
-    } catch {
-        return '';
-    }
-}
-
-function normalizeCount(value) {
-    return Number.isInteger(value) && value >= 0 ? value : 0;
-}
-
-function normalizeUnixSeconds(value) {
-    return typeof value === 'number' && Number.isFinite(value) && value > 0
-        ? new Date(value * 1000).toISOString()
-        : '';
-}
-
-cli({
-    site: 'zhihu',
-    name: 'answer-detail',
-    access: 'read',
-    description: '知乎单个回答完整内容（按 answer ID 获取）',
-    domain: 'www.zhihu.com',
-    strategy: Strategy.COOKIE,
-    args: [
-        { name: 'id', required: true, positional: true, help: 'Answer ID, full Zhihu answer URL, or typed target (answer:<qid>:<aid>)' },
-        { name: 'max-content', type: 'int', default: 0, help: 'Optional cap on stripped content length in characters (0 = no truncation, return the full answer)' },
-    ],
-    columns: ['id', 'author', 'votes', 'comments', 'question_id', 'question_title', 'url', 'created_at', 'updated_at', 'content'],
-    func: async (page, kwargs) => {
-        const target = parseAnswerTarget(kwargs.id);
-        if (!target) {
-            throw new ArgumentError(
-                'Answer ID must be a numeric id, a Zhihu answer URL, or answer:<qid>:<aid>',
-                'Example: webcmd zhihu answer-detail 1937205528846655537',
-            );
-        }
-        const { answerId } = target;
-        // `--max-content 0` (the default) means "no cap, return the
-        // full stripped answer". Any positive value is an opt-in user
-        // cap, mirroring the wikipedia `page` pattern — we never
-        // silently truncate behind the user's back.
-        const rawMaxContent = kwargs['max-content'];
-        const maxContent = rawMaxContent == null ? 0 : Number(rawMaxContent);
-        if (!Number.isInteger(maxContent) || maxContent < 0) {
-            throw new ArgumentError(
-                '--max-content must be a non-negative integer (0 = no cap, full content)',
-                'Example: --max-content 2000',
-            );
-        }
-        // Navigate to the answer page itself: this both seeds the
-        // cookie/anti-bot context and works even when the caller did
-        // not supply the parent question id (Zhihu redirects from
-        // `/answer/<aid>` to the canonical `/question/<qid>/answer/<aid>`).
-        try {
-            await page.goto(`https://www.zhihu.com/answer/${answerId}`);
-        } catch (err) {
-            throw new CommandExecutionError(
-                `Failed to open Zhihu answer ${answerId}: ${err instanceof Error ? err.message : String(err)}`,
-                'Open the answer URL in Chrome and retry after the page is reachable.',
-            );
-        }
-        const currentQuestionId = page.getCurrentUrl
-            ? extractQuestionIdFromAnswerUrl(await page.getCurrentUrl().catch(() => ''))
-            : '';
-        const apiUrl = `https://www.zhihu.com/api/v4/answers/${answerId}?include=content,voteup_count,comment_count,author,created_time,updated_time,question`;
-        const data = await page.evaluate(`
-      (async () => {
-        const r = await fetch(${JSON.stringify(apiUrl)}, { credentials: 'include' });
-        if (!r.ok) return { __httpError: r.status };
-        try {
-          return await r.json();
-        } catch (error) {
-          return { __malformedJson: error instanceof Error ? error.message : String(error) };
-        }
-      })()
-    `).catch((err) => {
-            throw new CommandExecutionError(
-                `Zhihu answer detail request failed: ${err instanceof Error ? err.message : String(err)}`,
-                'Try again later or rerun with -v for more detail.',
-            );
-        });
-        if (!data || data.__httpError) {
-            const status = data?.__httpError;
-            if (status === 401 || status === 403) {
-                throw new AuthRequiredError('www.zhihu.com', 'Failed to fetch Zhihu answer detail');
-            }
-            if (status === 404) {
-                throw new EmptyResultError('zhihu answer-detail', `No Zhihu answer was found for ${answerId}.`);
-            }
-            throw new CommandExecutionError(
-                status
-                    ? `Zhihu answer detail request failed (HTTP ${status})`
-                    : 'Zhihu answer detail request failed',
-                'Try again later or rerun with -v for more detail',
-            );
-        }
-        if (data.__malformedJson) {
-            throw new CommandExecutionError(
-                `Zhihu answer detail returned malformed JSON: ${data.__malformedJson}`,
-                'Try again later or rerun with -v for more detail',
-            );
-        }
-        if (typeof data !== 'object' || Array.isArray(data)) {
-            throw new CommandExecutionError(
-                'Zhihu answer detail returned a malformed payload',
-                'Try again later or rerun with -v for more detail',
-            );
-        }
-        if (data.error || data.error_msg || data.message) {
-            throw new CommandExecutionError(
-                `Zhihu answer detail returned an error payload: ${data.error?.message || data.error_msg || data.message}`,
-                'Try again later or rerun with -v for more detail',
-            );
-        }
-        if (!Object.prototype.hasOwnProperty.call(data, 'content')) {
-            throw new CommandExecutionError(
-                'Zhihu answer detail payload did not include answer content',
-                'Try again later or rerun with -v for more detail',
-            );
-        }
-        const question = data.question || {};
-        // Answer ids and newer question ids can exceed
-        // Number.MAX_SAFE_INTEGER. Prefer ids parsed from user input or
-        // the canonical redirected URL; only fall back to API numeric ids
-        // when no string-safe source is available.
-        const questionId = target.questionId
-            || currentQuestionId
-            || extractQuestionIdFromAnswerUrl(question.url)
-            || (question.id == null ? '' : String(question.id));
-        const stripped = stripHtml(data.content || '');
-        // Truncation is opt-in only; default `maxContent === 0` short-
-        // circuits the conditional so the full stripped body is returned.
-        const content = maxContent > 0 && stripped.length > maxContent
-            ? stripped.substring(0, maxContent)
-            : stripped;
-        return [{
-            id: answerId,
-            author: data.author?.name || 'anonymous',
-            votes: normalizeCount(data.voteup_count),
-            comments: normalizeCount(data.comment_count),
-            question_id: questionId,
-            question_title: question.title || '',
-            url: questionId
-                ? `https://www.zhihu.com/question/${questionId}/answer/${answerId}`
-                : `https://www.zhihu.com/answer/${answerId}`,
-            created_at: normalizeUnixSeconds(data.created_time),
-            updated_at: normalizeUnixSeconds(data.updated_time),
-            content,
-        }];
-    },
-});
-
-export const __test__ = { stripHtml, extractAnswerId, parseAnswerTarget, extractQuestionIdFromAnswerUrl };

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

@@ -1,338 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@agentrhq/webcmd/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@agentrhq/webcmd/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();
-    });
-});

package/clis/zhihu/answer.js

@@ -1,54 +0,0 @@
-import { CliError, CommandExecutionError } from '@agentrhq/webcmd/errors';
-import { cli, Strategy } from '@agentrhq/webcmd/registry';
-import { assertAllowedKinds, parseTarget } from './target.js';
-import { buildResultRow, requireExecute, resolveCurrentUserIdentity, resolvePayload } from './write-shared.js';
-cli({
-    site: 'zhihu',
-    name: 'answer',
-    access: 'write',
-    description: 'Answer a Zhihu question',
-    domain: 'www.zhihu.com',
-    strategy: Strategy.COOKIE,
-    browser: true,
-    args: [
-        { name: 'target', positional: true, required: true, help: 'Zhihu question URL or typed target' },
-        { name: 'text', positional: true, help: 'Answer text' },
-        { name: 'file', help: 'Answer text file path' },
-        { name: 'execute', type: 'boolean', help: 'Actually perform the write action' },
-    ],
-    columns: ['status', 'outcome', 'message', 'target_type', 'target', 'created_target', 'created_url', 'author_identity'],
-    func: async (page, kwargs) => {
-        if (!page)
-            throw new CommandExecutionError('Browser session required for zhihu answer');
-        requireExecute(kwargs);
-        const rawTarget = String(kwargs.target);
-        const target = assertAllowedKinds('answer', parseTarget(rawTarget));
-        const payload = await resolvePayload(kwargs);
-        await page.goto(target.url);
-        await page.wait(3);
-        const authorIdentity = await resolveCurrentUserIdentity(page);
-        const apiResult = await page.evaluate(`(async () => {
-            var questionId = ${JSON.stringify(target.id)};
-            var content = ${JSON.stringify(payload)};
-            var url = 'https://www.zhihu.com/api/v4/questions/' + questionId + '/answers';
-            var resp = await fetch(url, {
-                method: 'POST',
-                credentials: 'include',
-                headers: { 'Content-Type': 'application/json' },
-                body: JSON.stringify({ content: content, reshipment_settings: 'disallowed' }),
-            });
-            var data = await resp.json();
-            if (!resp.ok) return { ok: false, status: resp.status, message: data.error ? data.error.message : 'unknown error' };
-            if (!data || !data.id) return { ok: false, status: resp.status, message: 'Answer API response did not include a created answer id' };
-            return { ok: true, id: String(data.id), url: data.url || ('https://www.zhihu.com/question/' + questionId + '/answer/' + data.id) };
-        })()`);
-        if (!apiResult?.ok) {
-            throw new CliError('COMMAND_EXEC', apiResult?.message || 'Failed to create answer');
-        }
-        return buildResultRow(`Answered question ${target.id}`, target.kind, rawTarget, 'created', {
-            created_target: 'answer:' + target.id + ':' + apiResult.id,
-            created_url: apiResult.url,
-            author_identity: authorIdentity,
-        });
-    },
-});

package/clis/zhihu/answer.test.js

@@ -1,54 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@agentrhq/webcmd/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');
-    });
-});