@agentrhq/webcmd 0.1.2 → 0.2.0

package/clis/zhihu/collections.js

@@ -1,128 +0,0 @@
-import { cli, Strategy } from '@agentrhq/webcmd/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@agentrhq/webcmd/errors';
-import { log } from '@agentrhq/webcmd/logger';
-
-function validatePositiveInt(value, name) {
-  const n = Number(value);
-  if (!Number.isInteger(n) || n <= 0) {
-    throw new ArgumentError(`zhihu collections --${name} must be a positive integer`, 'Example: webcmd zhihu collections --limit 20');
-  }
-  return n;
-}
-
-async function fetchJson(page, url, errorLabel) {
-  const data = await page.evaluate(`
-    (async () => {
-      const r = await fetch(${JSON.stringify(url)}, { credentials: 'include' });
-      if (!r.ok) return { __httpError: r.status };
-      return await r.json();
-    })()
-  `);
-
-  if (!data || data.__httpError) {
-    const status = data?.__httpError;
-    if (status === 401 || status === 403) {
-      throw new AuthRequiredError('www.zhihu.com', `${errorLabel} from Zhihu failed. Please ensure you are logged in.`);
-    }
-    throw new CommandExecutionError(
-      status ? `${errorLabel} from Zhihu failed (HTTP ${status})` : `${errorLabel} from Zhihu failed`,
-      'Try again later or rerun with -v for more detail',
-    );
-  }
-  return data;
-}
-
-function collectionKey(item) {
-  return String(item?.id || item?.url || item?.title || '');
-}
-
-cli({
-  site: 'zhihu',
-  name: 'collections',
-    access: 'read',
-  description: '知乎收藏夹列表（需要登录）',
-  domain: 'www.zhihu.com',
-  strategy: Strategy.COOKIE,
-  browser: true,
-  args: [
-    { name: 'limit', type: 'int', default: 20, help: '每页数量（最大 20）' },
-  ],
-  columns: ['rank', 'title', 'item_count', 'description', 'collection_id'],
-  func: async (page, kwargs) => {
-    const { limit = 20 } = kwargs;
-    const requestedLimit = validatePositiveInt(limit, 'limit');
-
-    // 先访问知乎主页建立 session
-    await page.goto('https://www.zhihu.com');
-    // 获取当前用户的 url_token
-    const meData = await fetchJson(page, 'https://www.zhihu.com/api/v4/me?include=url_token', 'Zhihu user info request');
-
-    const urlToken = meData.url_token;
-    if (!urlToken) {
-      throw new CommandExecutionError('Failed to get user url_token from Zhihu', 'Please ensure you are logged in.');
-    }
-
-    const collected = [];
-    const seen = new Set();
-    let totals = 0;
-    let offset = 0;
-    const pageLimit = Math.min(requestedLimit, 20);
-    const maxPages = Math.ceil(requestedLimit / pageLimit) + 2;
-
-    for (let pageIndex = 0; pageIndex < maxPages && collected.length < requestedLimit; pageIndex += 1) {
-      const currentFetchLimit = Math.min(pageLimit, requestedLimit - collected.length);
-      const url = `https://www.zhihu.com/api/v4/people/${urlToken}/collections?include=data%5B*%5D.updated_time&offset=${offset}&limit=${currentFetchLimit}`;
-      const data = await fetchJson(page, url, 'Zhihu favorite collections request');
-      const items = Array.isArray(data.data) ? data.data : [];
-      const paging = data.paging || {};
-      totals = Number(paging.totals || totals || 0);
-
-      for (const item of items) {
-        const key = collectionKey(item);
-        if (key && !seen.has(key)) {
-          seen.add(key);
-          collected.push(item);
-        }
-        if (collected.length >= requestedLimit) break;
-      }
-
-      if (items.length === 0 || paging.is_end || collected.length >= requestedLimit) break;
-      if (typeof paging.next === 'string') {
-        try {
-          const nextUrl = new URL(paging.next);
-          const parsedOffset = Number(nextUrl.searchParams.get('offset'));
-          if (Number.isInteger(parsedOffset) && parsedOffset > offset) {
-            offset = parsedOffset;
-            continue;
-          }
-        } catch {}
-      }
-      if (items.length < currentFetchLimit) break;
-      const fallbackOffset = offset + items.length;
-      if (fallbackOffset <= offset) break;
-      offset = fallbackOffset;
-      if (totals && offset >= totals) break;
-    }
-
-    if (totals > 0) {
-      log.info(`共有 ${totals} 个收藏夹`);
-    }
-
-    if (collected.length === 0) {
-      throw new EmptyResultError('zhihu collections', 'No favorite collections were returned for the logged-in user.');
-    }
-
-    return collected.slice(0, requestedLimit).map((item, i) => ({
-      rank: i + 1,
-      title: item.title || '未命名',
-      item_count: item.item_count ?? item.answer_count ?? 0,
-      description: item.description || '',
-      collection_id: String(item.id || ''),
-    }));
-  },
-});
-
-export const __test__ = {
-  validatePositiveInt,
-  collectionKey,
-};

package/clis/zhihu/collections.test.js

@@ -1,182 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@agentrhq/webcmd/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@agentrhq/webcmd/errors';
-
-// Mock logger
-vi.mock('@agentrhq/webcmd/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.js

@@ -1,55 +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: 'comment',
-    access: 'write',
-    description: 'Create a top-level comment on a Zhihu answer or article',
-    domain: 'zhihu.com',
-    strategy: Strategy.COOKIE,
-    browser: true,
-    args: [
-        { name: 'target', positional: true, required: true, help: 'Zhihu target URL or typed target' },
-        { name: 'text', positional: true, help: 'Comment text' },
-        { name: 'file', help: 'Comment text file path' },
-        { name: 'execute', type: 'boolean', help: 'Actually perform the write action' },
-    ],
-    columns: ['status', 'outcome', 'message', 'target_type', 'target', 'author_identity', 'created_url'],
-    func: async (page, kwargs) => {
-        if (!page)
-            throw new CommandExecutionError('Browser session required for zhihu comment');
-        requireExecute(kwargs);
-        const rawTarget = String(kwargs.target);
-        const target = assertAllowedKinds('comment', 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 targetKind = ${JSON.stringify(target.kind)};
-            var targetId = ${JSON.stringify(target.id)};
-            var content = ${JSON.stringify(payload)};
-            var resourceType = targetKind === 'answer' ? 'answers' : 'articles';
-            var url = 'https://www.zhihu.com/api/v4/' + resourceType + '/' + targetId + '/comments';
-            var resp = await fetch(url, {
-                method: 'POST',
-                credentials: 'include',
-                headers: { 'Content-Type': 'application/json' },
-                body: JSON.stringify({ content: content }),
-            });
-            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: 'Comment API response did not include a created comment id' };
-            return { ok: true, id: data.id, url: data.url };
-        })()`);
-        if (!apiResult?.ok) {
-            throw new CliError('COMMAND_EXEC', apiResult?.message || 'Failed to create comment');
-        }
-        return buildResultRow(`Commented on ${target.kind} ${target.id}`, target.kind, rawTarget, 'created', {
-            author_identity: authorIdentity,
-            created_url: apiResult.url || '',
-        });
-    },
-});

package/clis/zhihu/comment.test.js

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

@@ -1,80 +0,0 @@
-/**
- * Zhihu download — export articles to Markdown format.
- *
- * Usage:
- *   webcmd zhihu download --url "https://zhuanlan.zhihu.com/p/xxx" --output ./zhihu
- */
-import { cli, Strategy } from '@agentrhq/webcmd/registry';
-import { downloadArticle } from '@agentrhq/webcmd/download/article-download';
-cli({
-    site: 'zhihu',
-    name: 'download',
-    access: 'read',
-    description: '导出知乎文章为 Markdown 格式',
-    domain: 'zhuanlan.zhihu.com',
-    strategy: Strategy.COOKIE,
-    args: [
-        { name: 'url', required: true, help: 'Article URL (zhuanlan.zhihu.com/p/xxx)' },
-        { name: 'output', default: './zhihu-articles', help: 'Output directory' },
-        { name: 'download-images', type: 'boolean', default: false, help: 'Download images locally' },
-    ],
-    columns: ['title', 'author', 'publish_time', 'status', 'size'],
-    func: async (page, kwargs) => {
-        const url = kwargs.url;
-        // Navigate to article page
-        await page.goto(url);
-        await page.wait(3);
-        // Extract article content
-        const data = await page.evaluate(`
-      (() => {
-        const result = {
-          title: '',
-          author: '',
-          publishTime: '',
-          contentHtml: '',
-          imageUrls: []
-        };
-
-        // Get title
-        const titleEl = document.querySelector('.Post-Title, h1.ContentItem-title, .ArticleTitle');
-        result.title = titleEl?.textContent?.trim() || 'untitled';
-
-        // Get author
-        const authorEl = document.querySelector('.AuthorInfo-name, .UserLink-link');
-        result.author = authorEl?.textContent?.trim() || '';
-
-        // Get publish time
-        const timeEl = document.querySelector('.ContentItem-time, .Post-Time');
-        result.publishTime = timeEl?.textContent?.trim() || '';
-
-        // Get content HTML
-        const contentEl = document.querySelector('.Post-RichTextContainer, .RichText, .ArticleContent');
-        if (contentEl) {
-          result.contentHtml = contentEl.innerHTML;
-
-          // Extract image URLs
-          contentEl.querySelectorAll('img').forEach(img => {
-            const src = img.getAttribute('data-original') || img.getAttribute('data-actualsrc') || img.src;
-            if (src && !src.includes('data:image')) {
-              result.imageUrls.push(src);
-            }
-          });
-        }
-
-        return result;
-      })()
-    `);
-        return downloadArticle({
-            title: data?.title || '',
-            author: data?.author,
-            publishTime: data?.publishTime,
-            sourceUrl: url,
-            contentHtml: data?.contentHtml || '',
-            imageUrls: data?.imageUrls,
-        }, {
-            output: kwargs.output,
-            downloadImages: kwargs['download-images'],
-            imageHeaders: { Referer: 'https://zhuanlan.zhihu.com/' },
-        });
-    },
-});

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');
-    });
-});

package/clis/zhihu/favorite.js

@@ -1,87 +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 } from './write-shared.js';
-function normalizeCollectionName(value) {
-    return value
-        .replace(/\s+/g, ' ')
-        .replace(/\s+\d+\s*(条内容|个内容|items?)$/i, '')
-        .replace(/\s+(公开|私密|默认)$/i, '')
-        .trim();
-}
-cli({
-    site: 'zhihu',
-    name: 'favorite',
-    access: 'write',
-    description: 'Favorite a Zhihu answer or article into a specific collection',
-    domain: 'zhihu.com',
-    strategy: Strategy.COOKIE,
-    browser: true,
-    args: [
-        { name: 'target', positional: true, required: true, help: 'Zhihu target URL or typed target' },
-        { name: 'collection', help: 'Collection name' },
-        { name: 'collection-id', help: 'Stable collection id' },
-        { name: 'execute', type: 'boolean', help: 'Actually perform the write action' },
-    ],
-    columns: ['status', 'outcome', 'message', 'target_type', 'target', 'collection_name', 'collection_id'],
-    func: async (page, kwargs) => {
-        if (!page)
-            throw new CommandExecutionError('Browser session required for zhihu favorite');
-        requireExecute(kwargs);
-        const rawTarget = String(kwargs.target);
-        const target = assertAllowedKinds('favorite', parseTarget(rawTarget));
-        const collectionName = typeof kwargs.collection === 'string' ? kwargs.collection : undefined;
-        const collectionId = typeof kwargs['collection-id'] === 'string' ? kwargs['collection-id'] : undefined;
-        if ((collectionName ? 1 : 0) + (collectionId ? 1 : 0) !== 1) {
-            throw new CliError('INVALID_INPUT', 'Use exactly one of --collection or --collection-id');
-        }
-        await page.goto('https://www.zhihu.com');
-        await page.wait(2);
-        const apiResult = await page.evaluate(`(async () => {
-            var collectionId = ${JSON.stringify(collectionId || null)};
-            var collectionName = ${JSON.stringify(collectionName || null)};
-            var targetKind = ${JSON.stringify(target.kind)};
-            var targetId = ${JSON.stringify(target.id)};
-            var normalizeCollectionName = function(value) {
-                return String(value || '')
-                    .replace(/\\s+/g, ' ')
-                    .replace(/\\s+\\d+\\s*(条内容|个内容|items?)$/i, '')
-                    .replace(/\\s+(公开|私密|默认)$/i, '')
-                    .trim()
-                    .toLowerCase();
-            };
-
-            if (!collectionId && collectionName) {
-                var listResp = await fetch('https://www.zhihu.com/api/v4/people/self/collections?limit=50', { credentials: 'include' });
-                if (!listResp.ok) return { ok: false, message: 'Failed to list collections: HTTP ' + listResp.status };
-                var listData = {};
-                try { listData = await listResp.json(); } catch(e) {
-                    return { ok: false, message: 'Failed to parse collection list' };
-                }
-                var needle = normalizeCollectionName(collectionName);
-                var matches = (listData.data || []).filter(function(c) { return normalizeCollectionName(c.title) === needle; });
-                if (matches.length === 0) return { ok: false, message: 'Collection not found: ' + collectionName };
-                if (matches.length > 1) return { ok: false, message: 'Collection name is ambiguous: ' + collectionName };
-                collectionId = String(matches[0].id);
-            }
-
-            var resp = await fetch('https://www.zhihu.com/api/v4/favlists/' + collectionId + '/items', {
-                method: 'POST',
-                credentials: 'include',
-                headers: { 'Content-Type': 'application/json' },
-                body: JSON.stringify({ item_id: targetId, item_type: targetKind }),
-            });
-            if (resp.ok || resp.status === 204) return { ok: true, collectionId: collectionId };
-            var data = {};
-            try { data = await resp.json(); } catch(e) {}
-            return { ok: false, message: data.error ? data.error.message : 'HTTP ' + resp.status };
-        })()`);
-        if (!apiResult?.ok) {
-            throw new CliError('COMMAND_EXEC', apiResult?.message || 'Failed to favorite');
-        }
-        return buildResultRow(`Favorited ${target.kind} ${target.id}`, target.kind, rawTarget, 'applied', {
-            collection_name: collectionName || '',
-            collection_id: apiResult.collectionId || collectionId || '',
-        });
-    },
-});

package/clis/zhihu/favorite.test.js

@@ -1,59 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@agentrhq/webcmd/registry';
-import './favorite.js';
-describe('zhihu favorite', () => {
-    it('registers as a cookie browser command', () => {
-        const cmd = getRegistry().get('zhihu/favorite');
-        expect(cmd).toBeDefined();
-        expect(cmd.strategy).toBe('cookie');
-    });
-    it('favorites via API with collection-id', async () => {
-        const cmd = getRegistry().get('zhihu/favorite');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValueOnce({ ok: true, collectionId: '123' }),
-        };
-        const rows = await cmd.func(page, { target: 'answer:1:2', 'collection-id': '123', execute: true });
-        expect(rows).toEqual([expect.objectContaining({ outcome: 'applied', collection_id: '123' })]);
-    });
-    it('throws on API error', async () => {
-        const cmd = getRegistry().get('zhihu/favorite');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValueOnce({ ok: false, message: 'collection not found' }),
-        };
-        await expect(cmd.func(page, { target: 'answer:1:2', 'collection-id': '123', execute: true }))
-            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
-    });
-    it('requires exact normalized collection-name matches', async () => {
-        const cmd = getRegistry().get('zhihu/favorite');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValueOnce({ ok: false, message: 'Collection not found: AI' }),
-        };
-        await expect(cmd.func(page, { target: 'answer:1:2', collection: 'AI', execute: true }))
-            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
-        expect(page.evaluate.mock.calls[0][0]).toContain('normalizeCollectionName(c.title) === needle');
-        expect(page.evaluate.mock.calls[0][0]).not.toContain('.includes(needle)');
-    });
-    it('fails fast on ambiguous collection-name matches', async () => {
-        const cmd = getRegistry().get('zhihu/favorite');
-        const page = {
-            goto: vi.fn().mockResolvedValue(undefined),
-            wait: vi.fn().mockResolvedValue(undefined),
-            evaluate: vi.fn().mockResolvedValueOnce({ ok: false, message: 'Collection name is ambiguous: 默认收藏夹' }),
-        };
-        await expect(cmd.func(page, { target: 'answer:1:2', collection: '默认收藏夹', execute: true }))
-            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
-        expect(page.evaluate.mock.calls[0][0]).toContain('matches.length > 1');
-    });
-    it('requires exactly one of --collection or --collection-id', async () => {
-        const cmd = getRegistry().get('zhihu/favorite');
-        const page = { goto: vi.fn(), wait: vi.fn(), evaluate: vi.fn() };
-        await expect(cmd.func(page, { target: 'answer:1:2', execute: true }))
-            .rejects.toMatchObject({ code: 'INVALID_INPUT' });
-    });
-});