@jackwener/opencli 1.3.1 → 1.3.3

package/src/runtime.ts

@@ -1,12 +1,13 @@
 import { BrowserBridge, CDPBridge } from './browser/index.js';
 import type { IPage } from './types.js';
+import { TimeoutError } from './errors.js';
 
 /**
  * Returns the appropriate browser factory based on environment config.
  * Uses CDPBridge when OPENCLI_CDP_ENDPOINT is set, otherwise BrowserBridge.
  */
 export function getBrowserFactory(): new () => IBrowserFactory {
-  return (process.env.OPENCLI_CDP_ENDPOINT ? CDPBridge : BrowserBridge) as any;
+  return (process.env.OPENCLI_CDP_ENDPOINT ? CDPBridge : BrowserBridge) as unknown as new () => IBrowserFactory;
 }
 
 export const DEFAULT_BROWSER_CONNECT_TIMEOUT = parseInt(process.env.OPENCLI_BROWSER_CONNECT_TIMEOUT ?? '30', 10);
@@ -21,15 +22,26 @@ export async function runWithTimeout<T>(
   promise: Promise<T>,
   opts: { timeout: number; label?: string },
 ): Promise<T> {
-  return withTimeoutMs(promise, opts.timeout * 1000, `${opts.label ?? 'Operation'} timed out after ${opts.timeout}s`);
+  const label = opts.label ?? 'Operation';
+  return withTimeoutMs(promise, opts.timeout * 1000,
+    () => new TimeoutError(label, opts.timeout));
 }
 
 /**
  * Timeout with milliseconds unit. Used for low-level internal timeouts.
+ * Accepts a factory function to create the rejection error, keeping this
+ * utility decoupled from specific error types.
  */
-export function withTimeoutMs<T>(promise: Promise<T>, timeoutMs: number, message: string): Promise<T> {
+export function withTimeoutMs<T>(
+  promise: Promise<T>,
+  timeoutMs: number,
+  makeError: string | (() => Error) = 'Operation timed out',
+): Promise<T> {
+  const reject_ = typeof makeError === 'string'
+    ? () => new Error(makeError)
+    : makeError;
   return new Promise<T>((resolve, reject) => {
-    const timer = setTimeout(() => reject(new Error(message)), timeoutMs);
+    const timer = setTimeout(() => reject(reject_()), timeoutMs);
     promise.then(
       (value) => { clearTimeout(timer); resolve(value); },
       (error) => { clearTimeout(timer); reject(error); },

package/src/validate.ts

@@ -2,6 +2,7 @@
 import * as fs from 'node:fs';
 import * as path from 'node:path';
 import yaml from 'js-yaml';
+import { getErrorMessage } from './errors.js';
 
 /** All recognized pipeline step names */
 const KNOWN_STEP_NAMES = new Set([
@@ -37,9 +38,6 @@ function isRecord(value: unknown): value is Record<string, unknown> {
   return typeof value === 'object' && value !== null && !Array.isArray(value);
 }
 
-function getErrorMessage(error: unknown): string {
-  return error instanceof Error ? error.message : String(error);
-}
 
 export function validateClisWithTarget(dirs: string[], target?: string): ValidationReport {
   const results: FileValidationResult[] = [];

package/tests/e2e/browser-auth.test.ts

@@ -101,7 +101,7 @@ describe('login-required commands — graceful failure', () => {
   }, 60_000);
 
   it('linux-do search fails gracefully without login', async () => {
-    await expectGracefulAuthFailure(['linux-do', 'search', '--keyword', 'test', '--limit', '3', '-f', 'json'], 'linux-do search');
+    await expectGracefulAuthFailure(['linux-do', 'search', 'test', '--limit', '3', '-f', 'json'], 'linux-do search');
   }, 60_000);
 
   // ── xiaohongshu (requires login) ──
@@ -112,4 +112,13 @@ describe('login-required commands — graceful failure', () => {
   it('xiaohongshu notifications fails gracefully without login', async () => {
     await expectGracefulAuthFailure(['xiaohongshu', 'notifications', '--limit', '3', '-f', 'json'], 'xiaohongshu notifications');
   }, 60_000);
+
+  // ── yollomi (requires login session) ──
+  it('yollomi generate fails gracefully without login', async () => {
+    await expectGracefulAuthFailure(['yollomi', 'generate', 'a cute cat', '--no-download', '-f', 'json'], 'yollomi generate');
+  }, 60_000);
+
+  it('yollomi video fails gracefully without login', async () => {
+    await expectGracefulAuthFailure(['yollomi', 'video', 'a sunset over the ocean', '--no-download', '-f', 'json'], 'yollomi video');
+  }, 60_000);
 });

package/tests/e2e/browser-public.test.ts

@@ -92,7 +92,7 @@ describe('browser public-data commands E2E', () => {
   }, 60_000);
 
   it('bilibili search returns results', async () => {
-    const data = await tryBrowserCommand(['bilibili', 'search', '--keyword', 'typescript', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['bilibili', 'search', 'typescript', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'bilibili search');
   }, 60_000);
 
@@ -102,6 +102,11 @@ describe('browser public-data commands E2E', () => {
     expectDataOrSkip(data, 'weibo hot');
   }, 60_000);
 
+  it('weibo search returns results', async () => {
+    const data = await tryBrowserCommand(['weibo', 'search', 'openai', '--limit', '3', '-f', 'json']);
+    expectDataOrSkip(data, 'weibo search');
+  }, 60_000);
+
   // ── zhihu (browser: true, cookie strategy) ──
   it('zhihu hot returns trending questions', async () => {
     const data = await tryBrowserCommand(['zhihu', 'hot', '--limit', '5', '-f', 'json']);
@@ -112,7 +117,7 @@ describe('browser public-data commands E2E', () => {
   }, 60_000);
 
   it('zhihu search returns results', async () => {
-    const data = await tryBrowserCommand(['zhihu', 'search', '--keyword', 'playwright', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['zhihu', 'search', 'playwright', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'zhihu search');
   }, 60_000);
 
@@ -146,25 +151,25 @@ describe('browser public-data commands E2E', () => {
 
   // ── reuters (browser: true) ──
   it('reuters search returns articles', async () => {
-    const data = await tryBrowserCommand(['reuters', 'search', '--keyword', 'technology', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['reuters', 'search', 'technology', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'reuters search');
   }, 60_000);
 
   // ── youtube (browser: true) ──
   it('youtube search returns videos', async () => {
-    const data = await tryBrowserCommand(['youtube', 'search', '--keyword', 'typescript tutorial', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['youtube', 'search', 'typescript tutorial', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'youtube search');
   }, 60_000);
 
   // ── smzdm (browser: true) ──
   it('smzdm search returns deals', async () => {
-    const data = await tryBrowserCommand(['smzdm', 'search', '--keyword', '键盘', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['smzdm', 'search', '键盘', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'smzdm search');
   }, 60_000);
 
   // ── boss (browser: true) ──
   it('boss search returns jobs', async () => {
-    const data = await tryBrowserCommand(['boss', 'search', '--keyword', 'golang', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['boss', 'search', 'golang', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'boss search');
   }, 60_000);
 
@@ -176,13 +181,13 @@ describe('browser public-data commands E2E', () => {
 
   // ── coupang (browser: true) ──
   it('coupang search returns products', async () => {
-    const data = await tryBrowserCommand(['coupang', 'search', '--keyword', 'laptop', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['coupang', 'search', 'laptop', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'coupang search');
   }, 60_000);
 
   // ── xiaohongshu (browser: true) ──
   it('xiaohongshu search returns notes', async () => {
-    const data = await tryBrowserCommand(['xiaohongshu', 'search', '--keyword', '美食', '--limit', '3', '-f', 'json']);
+    const data = await tryBrowserCommand(['xiaohongshu', 'search', '美食', '--limit', '3', '-f', 'json']);
     expectDataOrSkip(data, 'xiaohongshu search');
   }, 60_000);
 

package/tests/e2e/public-commands.test.ts

@@ -3,12 +3,15 @@
  * These commands use Node.js fetch directly — no browser needed.
  */
 
-import { describe, it, expect } from 'vitest';
-import { runCli, parseJsonOutput } from './helpers.js';
+import { describe, expect, it } from 'vitest';
+import { parseJsonOutput, runCli } from './helpers.js';
 
 function isExpectedChineseSiteRestriction(code: number, stderr: string): boolean {
   if (code === 0) return false;
-  return /Error \[FETCH_ERROR\]: HTTP (403|429|451|503)\b/.test(stderr);
+  // Overseas CI runners may get HTTP errors, geo-blocks, DNS failures,
+  // or receive mangled HTML that fails parsing.
+  return /Error \[(FETCH_ERROR|PARSE_ERROR|NOT_FOUND)\]/.test(stderr)
+    || /fetch failed/.test(stderr);
 }
 
 function isExpectedApplePodcastsRestriction(code: number, stderr: string): boolean {
@@ -40,21 +43,25 @@ describe('public commands E2E', () => {
     expect(Array.isArray(data[0].mediaLinks)).toBe(true);
   }, 30_000);
 
-  it.each(['markets', 'economics', 'industries', 'tech', 'politics', 'businessweek', 'opinions'])(
-    'bloomberg %s returns structured RSS items',
-    async (section) => {
-      const { stdout, code } = await runCli(['bloomberg', section, '--limit', '1', '-f', 'json']);
-      expect(code).toBe(0);
-      const data = parseJsonOutput(stdout);
-      expect(Array.isArray(data)).toBe(true);
-      expect(data.length).toBe(1);
-      expect(data[0]).toHaveProperty('title');
-      expect(data[0]).toHaveProperty('summary');
-      expect(data[0]).toHaveProperty('link');
-      expect(data[0]).toHaveProperty('mediaLinks');
-    },
-    30_000,
-  );
+  it.each([
+    'markets',
+    'economics',
+    'industries',
+    'tech',
+    'politics',
+    'businessweek',
+    'opinions',
+  ])('bloomberg %s returns structured RSS items', async (section) => {
+    const { stdout, code } = await runCli(['bloomberg', section, '--limit', '1', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBe(1);
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('summary');
+    expect(data[0]).toHaveProperty('link');
+    expect(data[0]).toHaveProperty('mediaLinks');
+  }, 30_000);
 
   it('bloomberg feeds lists the supported RSS aliases', async () => {
     const { stdout, code } = await runCli(['bloomberg', 'feeds', '-f', 'json']);
@@ -95,7 +102,16 @@ describe('public commands E2E', () => {
   }, 30_000);
 
   it('apple-podcasts top returns ranked podcasts', async () => {
-    const { stdout, stderr, code } = await runCli(['apple-podcasts', 'top', '--limit', '3', '--country', 'us', '-f', 'json']);
+    const { stdout, stderr, code } = await runCli([
+      'apple-podcasts',
+      'top',
+      '--limit',
+      '3',
+      '--country',
+      'us',
+      '-f',
+      'json',
+    ]);
     if (isExpectedApplePodcastsRestriction(code, stderr)) {
       console.warn(`apple-podcasts top skipped: ${stderr.trim()}`);
       return;
@@ -128,6 +144,76 @@ describe('public commands E2E', () => {
     expect(data.length).toBe(1);
   }, 30_000);
 
+  it('hackernews new returns newest stories', async () => {
+    const { stdout, code } = await runCli(['hackernews', 'new', '--limit', '3', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBeGreaterThanOrEqual(1);
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('score');
+    expect(data[0]).toHaveProperty('rank');
+  }, 30_000);
+
+  it('hackernews best returns best stories', async () => {
+    const { stdout, code } = await runCli(['hackernews', 'best', '--limit', '3', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBeGreaterThanOrEqual(1);
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('score');
+  }, 30_000);
+
+  it('hackernews ask returns Ask HN posts', async () => {
+    const { stdout, code } = await runCli(['hackernews', 'ask', '--limit', '3', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBeGreaterThanOrEqual(1);
+    expect(data[0]).toHaveProperty('title');
+  }, 30_000);
+
+  it('hackernews show returns Show HN posts', async () => {
+    const { stdout, code } = await runCli(['hackernews', 'show', '--limit', '3', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBeGreaterThanOrEqual(1);
+    expect(data[0]).toHaveProperty('title');
+  }, 30_000);
+
+  it('hackernews jobs returns job postings', async () => {
+    const { stdout, code } = await runCli(['hackernews', 'jobs', '--limit', '3', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBeGreaterThanOrEqual(1);
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('url');
+  }, 30_000);
+
+  it('hackernews search returns results for query', async () => {
+    const { stdout, code } = await runCli(['hackernews', 'search', 'typescript', '--limit', '3', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBe(3);
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('score');
+    expect(data[0]).toHaveProperty('author');
+  }, 30_000);
+
+  it('hackernews user returns user profile', async () => {
+    const { stdout, code } = await runCli(['hackernews', 'user', 'pg', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBe(1);
+    expect(data[0]).toHaveProperty('username', 'pg');
+    expect(data[0]).toHaveProperty('karma');
+  }, 30_000);
+
   // ── v2ex (public API, browser: false) ──
   it('v2ex hot returns topics', async () => {
     const { stdout, code } = await runCli(['v2ex', 'hot', '--limit', '3', '-f', 'json']);
@@ -156,6 +242,69 @@ describe('public commands E2E', () => {
     }
   }, 30_000);
 
+  it('v2ex node returns topics for a given node', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'node', 'python', '--limit', '3', '-f', 'json']);
+    // V2EX may rate-limit; only assert when successful
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(Array.isArray(data)).toBe(true);
+      expect(data.length).toBeGreaterThanOrEqual(1);
+      expect(data.length).toBeLessThanOrEqual(3);
+      expect(data[0]).toHaveProperty('title');
+      expect(data[0]).toHaveProperty('author');
+      expect(data[0]).toHaveProperty('url');
+    }
+  }, 30_000);
+
+  it('v2ex user returns topics by username', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'user', 'Livid', '--limit', '3', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(Array.isArray(data)).toBe(true);
+      expect(data.length).toBeGreaterThanOrEqual(1);
+      expect(data.length).toBeLessThanOrEqual(3);
+      expect(data[0]).toHaveProperty('title');
+      expect(data[0]).toHaveProperty('node');
+      expect(data[0]).toHaveProperty('url');
+    }
+  }, 30_000);
+
+  it('v2ex member returns user profile', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'member', 'Livid', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(Array.isArray(data)).toBe(true);
+      expect(data.length).toBe(1);
+      expect(data[0].username).toBe('Livid');
+    }
+  }, 30_000);
+
+  it('v2ex replies returns topic replies', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'replies', '1000', '--limit', '3', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(Array.isArray(data)).toBe(true);
+      expect(data.length).toBeGreaterThanOrEqual(1);
+      expect(data.length).toBeLessThanOrEqual(3);
+      expect(data[0]).toHaveProperty('author');
+      expect(data[0]).toHaveProperty('content');
+    }
+  }, 30_000);
+
+  it('v2ex nodes returns node list sorted by topics', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'nodes', '--limit', '5', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(Array.isArray(data)).toBe(true);
+      expect(data.length).toBe(5);
+      expect(data[0]).toHaveProperty('name');
+      expect(data[0]).toHaveProperty('title');
+      expect(data[0]).toHaveProperty('topics');
+      // Verify descending sort by topic count
+      expect(Number(data[0].topics)).toBeGreaterThanOrEqual(Number(data[data.length - 1].topics));
+    }
+  }, 30_000);
+
   // ── xiaoyuzhou (Chinese site — may return empty on overseas CI runners) ──
   it('xiaoyuzhou podcast returns podcast profile', async () => {
     const { stdout, stderr, code } = await runCli(['xiaoyuzhou', 'podcast', '6013f9f58e2f7ee375cf4216', '-f', 'json']);
@@ -173,7 +322,13 @@ describe('public commands E2E', () => {
   }, 30_000);
 
   it('xiaoyuzhou podcast-episodes returns episode list', async () => {
-    const { stdout, stderr, code } = await runCli(['xiaoyuzhou', 'podcast-episodes', '6013f9f58e2f7ee375cf4216', '-f', 'json']);
+    const { stdout, stderr, code } = await runCli([
+      'xiaoyuzhou',
+      'podcast-episodes',
+      '6013f9f58e2f7ee375cf4216',
+      '-f',
+      'json',
+    ]);
     if (isExpectedXiaoyuzhouRestriction(code, stderr)) {
       console.warn(`xiaoyuzhou podcast-episodes skipped: ${stderr.trim()}`);
       return;
@@ -204,7 +359,15 @@ describe('public commands E2E', () => {
   }, 30_000);
 
   it('xiaoyuzhou podcast-episodes rejects invalid limit', async () => {
-    const { stderr, code } = await runCli(['xiaoyuzhou', 'podcast-episodes', '6013f9f58e2f7ee375cf4216', '--limit', 'abc', '-f', 'json']);
+    const { stderr, code } = await runCli([
+      'xiaoyuzhou',
+      'podcast-episodes',
+      '6013f9f58e2f7ee375cf4216',
+      '--limit',
+      'abc',
+      '-f',
+      'json',
+    ]);
     if (isExpectedXiaoyuzhouRestriction(code, stderr)) {
       console.warn(`xiaoyuzhou invalid-limit skipped: ${stderr.trim()}`);
       return;
@@ -300,4 +463,29 @@ describe('public commands E2E', () => {
     expect(data[0]).toHaveProperty('readingCount');
     expect(data[0]).toHaveProperty('bookId');
   }, 30_000);
+
+  // ── yollomi (browser: false, hardcoded data) ──
+  it('yollomi models returns model list with all types', async () => {
+    const { stdout, code } = await runCli(['yollomi', 'models', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBeGreaterThan(10);
+    expect(data[0]).toHaveProperty('type');
+    expect(data[0]).toHaveProperty('model');
+    expect(data[0]).toHaveProperty('credits');
+    expect(data[0]).toHaveProperty('description');
+    const types = new Set(data.map((d: any) => d.type));
+    expect(types.has('image')).toBe(true);
+    expect(types.has('video')).toBe(true);
+    expect(types.has('tool')).toBe(true);
+  }, 30_000);
+
+  it('yollomi models --type image filters correctly', async () => {
+    const { stdout, code } = await runCli(['yollomi', 'models', '--type', 'image', '-f', 'json']);
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(data.length).toBeGreaterThan(0);
+    expect(data.every((d: any) => d.type === 'image')).toBe(true);
+  }, 30_000);
 });

package/tests/smoke/api-health.test.ts

@@ -4,11 +4,10 @@
  * These verify that external APIs haven't changed their structure.
  */
 
-import { describe, it, expect } from 'vitest';
-import { runCli, parseJsonOutput } from '../e2e/helpers.js';
+import { describe, expect, it } from 'vitest';
+import { parseJsonOutput, runCli } from '../e2e/helpers.js';
 
 describe('API health smoke tests', () => {
-
   // ── Public API commands (should always work) ──
   it('hackernews API is responsive and returns expected structure', async () => {
     const { stdout, code } = await runCli(['hackernews', 'top', '--limit', '5', '-f', 'json']);
@@ -46,6 +45,53 @@ describe('API health smoke tests', () => {
     }
   }, 30_000);
 
+  it('v2ex node API is responsive', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'node', 'python', '--limit', '3', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(data.length).toBeGreaterThanOrEqual(1);
+      expect(data[0]).toHaveProperty('title');
+      expect(data[0]).toHaveProperty('author');
+    }
+  }, 30_000);
+
+  it('v2ex user API is responsive', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'user', 'Livid', '--limit', '3', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(data.length).toBeGreaterThanOrEqual(1);
+      expect(data[0]).toHaveProperty('title');
+      expect(data[0]).toHaveProperty('url');
+    }
+  }, 30_000);
+
+  it('v2ex member API is responsive', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'member', 'Livid', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(data.length).toBe(1);
+      expect(data[0].username).toBe('Livid');
+    }
+  }, 30_000);
+
+  it('v2ex replies API is responsive', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'replies', '1000', '--limit', '3', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(data.length).toBeGreaterThanOrEqual(1);
+      expect(data[0]).toHaveProperty('author');
+    }
+  }, 30_000);
+
+  it('v2ex nodes API is responsive', async () => {
+    const { stdout, code } = await runCli(['v2ex', 'nodes', '--limit', '5', '-f', 'json']);
+    if (code === 0) {
+      const data = parseJsonOutput(stdout);
+      expect(data.length).toBe(5);
+      expect(data[0]).toHaveProperty('topics');
+    }
+  }, 30_000);
+
   // ── Validate all adapters ──
   it('all adapter definitions are valid', async () => {
     const { stdout, code } = await runCli(['validate']);
@@ -61,9 +107,22 @@ describe('API health smoke tests', () => {
     const sites = new Set(data.map((d: any) => d.site));
     // Verify all 17 sites are present
     for (const expected of [
-      'hackernews', 'bbc', 'bilibili', 'v2ex', 'weibo', 'zhihu',
-      'twitter', 'reddit', 'xueqiu', 'reuters', 'youtube',
-      'smzdm', 'boss', 'ctrip', 'coupang', 'xiaohongshu',
+      'hackernews',
+      'bbc',
+      'bilibili',
+      'v2ex',
+      'weibo',
+      'zhihu',
+      'twitter',
+      'reddit',
+      'xueqiu',
+      'reuters',
+      'youtube',
+      'smzdm',
+      'boss',
+      'ctrip',
+      'coupang',
+      'xiaohongshu',
       'yahoo-finance',
     ]) {
       expect(sites.has(expected)).toBe(true);