kiku-agent 1.3.1 → 1.3.3

package/server/agent/tests/suggestionsTui.test.js

@@ -0,0 +1,251 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { EventEmitter } from 'node:events';
+
+import {
+  getCategoryBadge,
+  formatWorkerProgressMatrix,
+  formatExecutiveFindings,
+  stripAnsi,
+} from '../ui.js';
+import {
+  renderSuggestionStudioView,
+  promptInteractiveSuggestion,
+  SUGGESTION_STUDIO_TABS,
+} from '../suggestionTui.js';
+import { routeCommand } from '../commands.js';
+
+const SAMPLE_SUGGESTIONS = [
+  {
+    id: 1,
+    category: 'DRAMA',
+    question: "What are Ju Jingyi's upcoming 2026/2025 drama projects?",
+    description: "Detailed breakdown of upcoming dramas and broadcasting platforms.",
+    tools: ['search_works', 'web_search'],
+    depth: 'Deep Analysis',
+    estimatedSeconds: 3,
+  },
+  {
+    id: 2,
+    category: 'MUSIC',
+    question: "What are the most popular OSTs and solo singles released by Ju Jingyi?",
+    description: "Discography highlights from In Blossom and Legend of Yunxi.",
+    tools: ['search_discography', 'search_videos'],
+    depth: 'Media Scan',
+    estimatedSeconds: 2,
+  },
+  {
+    id: 3,
+    category: 'FASHION',
+    question: "What are Ju Jingyi's major luxury brand ambassadorships in 2025-2026?",
+    description: "Overview of Versace Global Brand Ambassador role and endorsements.",
+    tools: ['get_profile_and_fandom'],
+    depth: 'Portfolio Scan',
+    estimatedSeconds: 2,
+  },
+];
+
+test('getCategoryBadge returns distinctive icons and colors for all 6 categories', () => {
+  const drama = getCategoryBadge('DRAMA');
+  assert.equal(drama.icon, '🌸');
+  assert.equal(drama.label, 'DRAMA');
+  assert.ok(drama.color);
+
+  const music = getCategoryBadge('MUSIC');
+  assert.equal(music.icon, '🎵');
+  assert.equal(music.label, 'MUSIC');
+
+  const fashion = getCategoryBadge('FASHION');
+  assert.equal(fashion.icon, '✨');
+  assert.equal(fashion.label, 'FASHION');
+
+  const awards = getCategoryBadge('MILESTONE');
+  assert.equal(awards.icon, '🌸');
+  assert.equal(awards.label, 'AWARDS');
+
+  const analysis = getCategoryBadge('ANALYSIS');
+  assert.equal(analysis.icon, '📖');
+  assert.equal(analysis.label, 'ANALYSIS');
+
+  const trivia = getCategoryBadge('TRIVIA');
+  assert.equal(trivia.icon, '💬');
+  assert.equal(trivia.label, 'TRIVIA');
+
+  const unknown = getCategoryBadge('OTHER');
+  assert.equal(unknown.icon, '✨');
+  assert.equal(unknown.label, 'OTHER');
+});
+
+test('renderSuggestionStudioView renders complete interactive studio frame', () => {
+  const rows = renderSuggestionStudioView({
+    suggestions: SAMPLE_SUGGESTIONS,
+    selectedIndex: 0,
+    selectedIds: new Set([1]),
+    activeCategory: 'ALL',
+    termWidth: 80,
+  });
+
+  assert.ok(rows.length > 5);
+  const plainText = rows.map(r => stripAnsi(r)).join('\n');
+
+
+  assert.ok(plainText.includes('🌸 KIKU SUGGESTION STUDIO'));
+  assert.ok(plainText.includes('DRAMA'));
+  assert.ok(plainText.includes('MUSIC'));
+  assert.ok(plainText.includes('FASHION'));
+
+
+  assert.ok(plainText.includes('❯ [◉] 1.'));
+  assert.ok(plainText.includes('[ ] 2.'));
+
+
+  assert.ok(plainText.includes('Focus:'));
+  assert.ok(plainText.includes('Tools:'));
+  assert.ok(plainText.includes('Est:'));
+
+
+  assert.ok(plainText.includes('[↑/↓] Nav'));
+  assert.ok(plainText.includes('[Enter] Launch Batch (1)'));
+});
+
+test('renderSuggestionStudioView correctly filters suggestions by category tab', () => {
+  const dramaRows = renderSuggestionStudioView({
+    suggestions: SAMPLE_SUGGESTIONS,
+    selectedIndex: 0,
+    selectedIds: new Set(),
+    activeCategory: 'DRAMA',
+    termWidth: 100,
+  });
+
+  const plainText = dramaRows.map(r => stripAnsi(r)).join('\n');
+  assert.ok(plainText.includes('upcoming 2026/2025 drama'));
+  assert.ok(!plainText.includes('popular OSTs and solo singles'));
+
+
+  const emptyRows = renderSuggestionStudioView({
+    suggestions: SAMPLE_SUGGESTIONS,
+    selectedIndex: 0,
+    selectedIds: new Set(),
+    activeCategory: 'TRIVIA',
+    termWidth: 80,
+  });
+  const emptyText = emptyRows.map(r => stripAnsi(r)).join('\n');
+  assert.ok(emptyText.includes('No suggestions found under TRIVIA'));
+});
+
+test('promptInteractiveSuggestion exits immediately on non-TTY without hanging', async () => {
+  const mockStdin = new EventEmitter();
+  mockStdin.isTTY = false;
+  const mockStdout = { isTTY: false, columns: 80, write: () => {} };
+
+  const res = await promptInteractiveSuggestion(SAMPLE_SUGGESTIONS, {
+    stdin: mockStdin,
+    stdout: mockStdout,
+  });
+
+  assert.equal(res.action, 'display');
+  assert.equal(res.targets, null);
+});
+
+test('formatWorkerProgressMatrix renders unified concurrent progress table', () => {
+  const workers = [
+    { index: 1, category: 'DRAMA', state: 'running', currentTool: 'search_works', question: 'Upcoming dramas?' },
+    { index: 2, category: 'MUSIC', state: 'complete', currentTool: '', question: 'Popular OSTs?' },
+    { index: 3, category: 'FASHION', state: 'error', currentTool: '', question: 'Versace role?' },
+  ];
+
+  const rows = formatWorkerProgressMatrix({
+    workers,
+    batchId: 'batch_test_1',
+    elapsed: '1.8',
+    termWidth: 80,
+  });
+
+  assert.ok(rows.length >= 5);
+  const plainText = rows.map(r => stripAnsi(r)).join('\n');
+
+  assert.ok(plainText.includes('CONCURRENT RESEARCH SUBAGENTS'));
+  assert.ok(plainText.includes('3 Workers • 1.8s'));
+  assert.ok(plainText.includes('W1 [🌸 DRAMA]'));
+  assert.ok(plainText.includes('Working'));
+  assert.ok(plainText.includes('W2 [🎵 MUSIC]'));
+  assert.ok(plainText.includes('Done'));
+  assert.ok(plainText.includes('W3 [✨ FASHION]'));
+  assert.ok(plainText.includes('Error'));
+});
+
+test('formatExecutiveFindings synthesizes findings, takeaways, and grounded citations', () => {
+  const batchResult = {
+    tasksCount: 2,
+    successfulCount: 2,
+    elapsed: '2.4',
+    results: [
+      {
+        index: 1,
+        category: 'DRAMA',
+        question: 'Upcoming dramas?',
+        answer: 'Ju Jingyi has multiple upcoming drama productions announced.',
+        toolCalls: [{ name: 'search_works' }],
+        iterations: 2,
+      },
+      {
+        index: 2,
+        category: 'FASHION',
+        question: 'Versace role?',
+        answer: 'Ju Jingyi was named Versace Global Brand Ambassador.',
+        toolCalls: [{ name: 'get_profile_and_fandom' }],
+        iterations: 1,
+      },
+    ],
+  };
+
+  const rows = formatExecutiveFindings({ batchResult, termWidth: 80 });
+  assert.ok(rows.length > 5);
+  const plainText = rows.map(r => stripAnsi(r)).join('\n');
+
+  assert.ok(plainText.includes('KIKU EXECUTIVE RESEARCH FINDINGS'));
+  assert.ok(plainText.includes('2/2 Succeeded • 2.4s'));
+  assert.ok(plainText.includes('[Worker 1] DRAMA: Upcoming dramas?'));
+  assert.ok(plainText.includes('Grounded Evidence: 1 tool calls executed'));
+  assert.ok(plainText.includes('[Worker 2] FASHION: Versace role?'));
+  assert.ok(plainText.includes('/session export'));
+});
+
+test('Commands: /suggestion executes cleanly in non-interactive routeCommand', async () => {
+  let generateCalled = false;
+  let batchCalled = false;
+
+  const mockAgent = {
+    async generateSuggestions() {
+      generateCalled = true;
+      return SAMPLE_SUGGESTIONS;
+    },
+    async runSuggestionBatch(targets) {
+      batchCalled = true;
+      return {
+        batchId: 'batch_test',
+        tasksCount: 1,
+        successfulCount: 1,
+        elapsed: '0.5',
+        results: [
+          {
+            index: 1,
+            category: 'DRAMA',
+            question: 'Upcoming dramas?',
+            answer: 'Yang Caiwei in In Blossom received critical acclaim.',
+            toolCalls: [{ name: 'search_works', args: {} }],
+            iterations: 1,
+          },
+        ],
+      };
+    },
+  };
+
+  const listRes = await routeCommand('/suggestion', { agent: mockAgent, sub: '', extra: '' });
+  assert.equal(listRes, 'HANDLED');
+  assert.equal(generateCalled, true);
+
+  const runRes = await routeCommand('/suggestion run 1', { agent: mockAgent, sub: 'run', extra: '1' });
+  assert.equal(runRes, 'HANDLED');
+  assert.equal(batchCalled, true);
+});

package/server/agent/tests/telemetryAndProfiles.test.js

@@ -0,0 +1,179 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { spawnSync } from 'node:child_process';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+import {
+  estimateTokens,
+  calculateTokenCost,
+  formatCostUsd,
+  formatTokenCount,
+  MODEL_PRICING
+} from '../telemetry.js';
+import {
+  BUILTIN_PROFILES,
+  loadProfiles,
+  getProfile,
+  getActiveProfile,
+  setActiveProfile,
+  createOrUpdateProfile,
+  deleteProfile,
+  listProfiles
+} from '../profiles.js';
+import { parseCliArgs } from '../cli.js';
+import { routeCommand } from '../commands.js';
+import { PACKAGE_VERSION } from '../version.js';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const cliPath = path.resolve(__dirname, '../cli.js');
+
+test('Telemetry: estimateTokens calculates token heuristics for various scripts', () => {
+  assert.equal(estimateTokens(''), 0);
+  assert.equal(estimateTokens(null), 0);
+
+  const enTokens = estimateTokens('Ju Jingyi is an accomplished artist and singer.');
+  assert.ok(enTokens >= 8 && enTokens <= 20, `enTokens ${enTokens} out of expected range`);
+
+  const zhTokens = estimateTokens('鞠婧祎中国女演员歌手');
+  assert.ok(zhTokens >= 10 && zhTokens <= 25, `zhTokens ${zhTokens} out of expected range`);
+
+  const thTokens = estimateTokens('จวีจิ้งอี นักแสดงและนักร้องสาว');
+  assert.ok(thTokens >= 15 && thTokens <= 40, `thTokens ${thTokens} out of expected range`);
+});
+
+test('Telemetry: calculateTokenCost calculates pricing accurately across providers', () => {
+  const lmCost = calculateTokenCost('lmstudio', 'google/gemma-4-e4b', 1000, 500);
+  assert.equal(lmCost, 0);
+
+  const groqCost = calculateTokenCost('groq', 'openai/gpt-oss-20b', 1000, 1000);
+  assert.ok(groqCost > 0 && groqCost < 0.01, `Groq cost: ${groqCost}`);
+
+  const deepseekCost = calculateTokenCost('deepseek', 'deepseek-v4-flash', 10000, 5000);
+  assert.ok(deepseekCost > 0 && deepseekCost < 0.05, `DeepSeek cost: ${deepseekCost}`);
+
+  const openaiCost = calculateTokenCost('openai', 'gpt-5.6-luna', 1000, 1000);
+  assert.ok(openaiCost > 0, `OpenAI cost: ${openaiCost}`);
+
+  const geminiCost = calculateTokenCost('gemini', 'gemini-3.7-flash', 1000, 1000);
+  assert.ok(geminiCost > 0, `Gemini cost: ${geminiCost}`);
+});
+
+test('Telemetry: formatCostUsd and formatTokenCount format numbers cleanly', () => {
+  assert.equal(formatCostUsd(0), '$0.00 (Local)');
+  assert.equal(formatCostUsd(0.00005), '<$0.0001');
+  assert.equal(formatCostUsd(0.0012), '$0.0012');
+  assert.equal(formatCostUsd(1.5), '$1.5000');
+
+  assert.equal(formatTokenCount(500), '500');
+  assert.equal(formatTokenCount(1500), '1,500');
+  assert.equal(formatTokenCount(2500000), '2,500,000');
+});
+
+test('Profiles: BUILTIN_PROFILES has required production profiles', () => {
+  const expectedProfiles = ['default', 'fast', 'deep-reasoning', 'cloud', 'gemini'];
+  for (const name of expectedProfiles) {
+    assert.ok(BUILTIN_PROFILES[name], `Missing built-in profile: ${name}`);
+    assert.ok(BUILTIN_PROFILES[name].provider);
+    assert.ok(BUILTIN_PROFILES[name].model);
+    assert.ok(BUILTIN_PROFILES[name].description);
+  }
+});
+
+test('Profiles: lifecycle methods support create, activate, query, and delete', () => {
+  const initialList = listProfiles();
+  assert.ok(initialList.length >= 5);
+
+  const active = getActiveProfile();
+  assert.ok(active && active.name);
+
+  const switched = setActiveProfile('fast');
+  assert.equal(switched.name, 'fast');
+  assert.equal(getActiveProfile().name, 'fast');
+
+  const custom = createOrUpdateProfile('unit-test-profile', {
+    provider: 'openai',
+    model: 'gpt-5.6-luna',
+    temperature: 0.5,
+    description: 'Temporary profile for unit tests'
+  });
+  assert.equal(custom.name, 'unit-test-profile');
+  assert.equal(custom.provider, 'openai');
+  assert.equal(custom.model, 'gpt-5.6-luna');
+
+  const retrieved = getProfile('unit-test-profile');
+  assert.equal(retrieved.name, 'unit-test-profile');
+
+  assert.throws(() => {
+    deleteProfile('default');
+  }, /Cannot delete built-in profile/);
+
+  const deleted = deleteProfile('unit-test-profile');
+  assert.equal(deleted, true);
+  assert.equal(getProfile('unit-test-profile'), null);
+
+  setActiveProfile('default');
+  assert.equal(getActiveProfile().name, 'default');
+});
+
+test('CLI: parseCliArgs handles --profile flag', () => {
+  const opts1 = parseCliArgs(['--profile', 'fast']);
+  assert.equal(opts1.profile, 'fast');
+
+  const opts2 = parseCliArgs(['-q', 'what is in blossom', '--profile', 'deep-reasoning', '--output', 'json']);
+  assert.equal(opts2.profile, 'deep-reasoning');
+  assert.equal(opts2.output, 'json');
+  assert.equal(opts2.query, 'what is in blossom');
+});
+
+test('CLI: headless JSON output contains token metrics and cost accounting', () => {
+  const res = spawnSync(process.execPath, [
+    cliPath,
+    '--url', 'http://127.0.0.1:9999/v1',
+    '-q', 'test query',
+    '--output', 'json',
+    '--profile', 'fast'
+  ], { encoding: 'utf8' });
+
+  assert.equal(res.status, 2);
+  const payload = JSON.parse(res.stdout);
+  assert.equal(payload.version, PACKAGE_VERSION);
+  assert.equal(payload.status, 'error');
+  assert.ok(payload.metrics, 'Metrics object must be present');
+  assert.ok(payload.metrics.tokens !== undefined, 'Tokens object must be present in metrics');
+  assert.ok(typeof payload.metrics.costUsd === 'number', 'costUsd must be a number in metrics');
+});
+
+test('Commands: /profile and /config execute cleanly in routeCommand', async () => {
+  const logs = [];
+  const fakeContext = {
+    sub: 'list',
+    extra: '',
+    parts: ['/profile', 'list'],
+    agent: null,
+    client: {
+      provider: 'lmstudio',
+      providerName: 'LM Studio (Local)',
+      model: 'google/gemma-4-e4b',
+      baseUrl: 'http://localhost:1234/v1'
+    }
+  };
+
+  const oldLog = console.log;
+  try {
+    console.log = (msg) => logs.push(msg);
+
+    await routeCommand('/profile', fakeContext);
+    assert.ok(logs.length > 0);
+
+    logs.length = 0;
+    await routeCommand('/profile', { ...fakeContext, sub: 'show', extra: 'fast', parts: ['/profile', 'show', 'fast'] });
+    assert.ok(logs.length > 0);
+
+    logs.length = 0;
+    await routeCommand('/config', { ...fakeContext, sub: '', extra: '', parts: ['/config'] });
+    assert.ok(logs.length > 0);
+  } finally {
+    console.log = oldLog;
+  }
+});

package/server/agent/tests/terminalImage.test.js

@@ -0,0 +1,80 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+import {
+  createTerminalImageGallery,
+  renderJpegFile,
+  supportsTerminalImages,
+} from '../terminalImage.js';
+import { getStringDisplayWidth, stripAnsi } from '../ui.js';
+
+const testDirectory = path.dirname(fileURLToPath(import.meta.url));
+const imageDirectory = path.resolve(testDirectory, '../../../src/assets/images');
+const squareImage = path.join(imageDirectory, '0040jbadly8icq3if7twwj60qy0qy0uc02.jpg');
+
+test('renders a JPEG as bounded true-color quadrant rows', () => {
+  const lines = renderJpegFile(squareImage, { columns: 30, rows: 20, maxWidth: 20, maxRows: 6 });
+
+  assert.equal(lines.length, 6);
+  assert.match(lines[0], /\x1b\[38;2;\d+;\d+;\d+m/);
+  assert.match(lines[0], /\x1b\[48;2;\d+;\d+;\d+m/);
+  assert.match(stripAnsi(lines[0]), /^  [ ▘▝▀▖▌▞▛▗▚▐▜▄▙▟█]+$/);
+  assert.ok(getStringDisplayWidth(lines[0]) <= 22);
+  assert.match(stripAnsi(lines.join('')), /[▘▝▖▗▚▞▛▜▙▟]/);
+});
+
+test('default gallery rendering uses most of a standard TUI viewport', () => {
+  const lines = renderJpegFile(squareImage, { columns: 80, rows: 24 });
+
+  assert.equal(lines.length, 17);
+  assert.equal(getStringDisplayWidth(lines[0]), 36);
+  assert.ok(new Set(lines).size > 12, 'photo detail should survive downsampling');
+});
+
+test('terminal image detection respects TTY, NO_COLOR, and explicit overrides', () => {
+  assert.equal(supportsTerminalImages({ env: {}, isTTY: true }), true);
+  assert.equal(supportsTerminalImages({ env: {}, isTTY: false }), false);
+  assert.equal(supportsTerminalImages({ env: { NO_COLOR: '1' }, isTTY: true }), false);
+  assert.equal(supportsTerminalImages({ env: { KIKU_TUI_IMAGES: 'on' }, isTTY: false }), true);
+  assert.equal(supportsTerminalImages({ env: { KIKU_TUI_IMAGES: 'off' }, isTTY: true }), false);
+});
+
+test('gallery renders on its cadence, supports manual display, and cycles assets', () => {
+  const gallery = createTerminalImageGallery({
+    directory: imageDirectory,
+    frequency: 3,
+    env: { KIKU_TUI_IMAGES: 'on' },
+    isTTY: false,
+    getColumns: () => 40,
+    getRows: () => 24,
+  });
+
+  assert.equal(gallery.enabled, true);
+  assert.equal(gallery.imageCount, 8);
+  assert.deepEqual(gallery.maybeRender(1), []);
+  assert.match(stripAnsi(gallery.maybeRender(3)[1]), /Gallery 1\/8/);
+  assert.match(stripAnsi(gallery.maybeRender(4, { force: true })[1]), /Gallery 2\/8/);
+});
+
+test('renderTwoImagesSideBySide and getWelcomeGalleryLines render 2 photos', async () => {
+  const { renderTwoImagesSideBySide, getWelcomeGalleryLines, discoverImages } = await import('../terminalImage.js');
+  const images = discoverImages(imageDirectory);
+  assert.ok(images.length >= 2);
+
+  const pair = renderTwoImagesSideBySide(images[0], images[1], { columns: 100, rows: 24, maxRows: 10 });
+  assert.ok(pair.length > 0);
+  assert.ok(pair.every(line => typeof line === 'string'));
+
+  const welcomeGallery = getWelcomeGalleryLines(2, {
+    directory: imageDirectory,
+    env: { KIKU_TUI_IMAGES: 'on' },
+    isTTY: false,
+    columns: 100,
+    rows: 30,
+  });
+
+  assert.ok(welcomeGallery.length > 0);
+  assert.match(stripAnsi(welcomeGallery[1]), /JU JINGYI • 鞠婧祎 GALLERY/);
+});
+

package/server/agent/tests/terminalMarkdown.test.js

@@ -0,0 +1,166 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+
+import {
+  expandMarkdownLinks,
+  formatMarkdownLines,
+  formatSolidPanelLines,
+  isMarkdownTableSeparator,
+  parseMarkdownTableRow,
+} from '../ui.js';
+
+test('parses Markdown table rows without retaining outer pipes', () => {
+  assert.deepEqual(
+    parseMarkdownTableRow('| Brand | Role & Highlight | Media Link |'),
+    ['Brand', 'Role & Highlight', 'Media Link']
+  );
+  assert.deepEqual(parseMarkdownTableRow('Brand | Category'), ['Brand', 'Category']);
+});
+
+test('recognizes Markdown table alignment separators', () => {
+  assert.equal(isMarkdownTableSeparator([':---', '---:', ':---:']), true);
+  assert.equal(isMarkdownTableSeparator(['Brand', 'Category']), false);
+});
+
+test('expands Markdown links before terminal line wrapping', () => {
+  assert.equal(
+    expandMarkdownLinks('[Search Commercial](https://example.com/a-very-long-link)'),
+    'Search Commercial: https://example.com/a-very-long-link'
+  );
+});
+
+test('formatMarkdownLines converts markdown headers, bold, and tables to formatted terminal rows', () => {
+  const sample = `### Evidence Report: Ju Jingyi Brand Endorsements
+
+**Summary:**
+Ju Jingyi has secured multiple ambassador roles.
+
+| Brand Name | Category | Role/Title |
+| :--- | :--- | :--- |
+| **PRSR** | Luxury Eyewear | Global Ambassador |
+| **Lafang** | Haircare | Ambassador |`;
+
+  const rows = formatMarkdownLines(sample, 80);
+  assert.ok(rows.length > 0);
+  assert.ok(rows.some(r => r.includes('Evidence Report')));
+  assert.ok(rows.some(r => r.includes('┌') && r.includes('┐')));
+  assert.ok(rows.some(r => r.includes('PRSR')));
+  assert.ok(rows.some(r => r.includes('└') && r.includes('┘')));
+});
+
+test('formatSolidPanelLines preserves ANSI-colored box table lines without destroying structure', () => {
+  const tableSample = `| Year | Award Title | Ceremony |
+| :--- | :--- | :--- |
+| 2024 | Popular Actress | Weibo TV |
+| 2021 | Outstanding Actress | Tencent Video |`;
+
+  const tableRows = formatMarkdownLines(tableSample, 80);
+  const panelRows = formatSolidPanelLines({
+    title: 'Awards Panel',
+    lines: tableRows,
+    width: 80,
+  });
+
+  assert.ok(panelRows.length > 0);
+  assert.ok(panelRows.some(r => r.includes('┌') && r.includes('┐')));
+  assert.ok(panelRows.some(r => r.includes('Popular Actress')));
+  assert.ok(panelRows.some(r => r.includes('Outstanding Actress')));
+  assert.ok(panelRows.some(r => r.includes('└') && r.includes('┘')));
+});
+
+test('formatMarkdownLines handles markdown tables with URLs without column breakage', () => {
+  const tableWithUrls = `| Detail | Finding | Source URLs |
+| :--- | :--- | :--- |
+| SNH48 Debut | Joined SNH48 Team NII in November 2013 | [https://snh48.fandom.com/wiki/Ju_Jingyi](https://snh48.fandom.com/wiki/Ju_Jingyi) |
+| Solo Debut | Established studio in December 2017 | [https://baike.baidu.com/item/Ju_Jingyi](https://baike.baidu.com/item/Ju_Jingyi) |`;
+
+  const rows = formatMarkdownLines(tableWithUrls, 120);
+  assert.ok(rows.length > 0);
+
+  assert.ok(rows.some(r => r.includes('┌') && r.includes('┐')));
+  assert.ok(rows.some(r => r.includes('SNH48 Debut')));
+  assert.ok(rows.some(r => r.includes('Solo Debut')));
+  assert.ok(rows.some(r => r.includes('└') && r.includes('┘')));
+});
+
+test('formatMarkdownLines handles markdown tables with short separators and unequal columns', () => {
+  const unevenTable = `| Title | Year | Category |
+| -- | :-: | -- |
+| In Blossom | 2024 | Drama |
+| Everyday | 2016 |
+| Rain Love | 2024 | Single | Extra Info |`;
+
+  const rows = formatMarkdownLines(unevenTable, 100);
+  assert.ok(rows.length > 0);
+  assert.ok(rows.some(r => r.includes('┌') && r.includes('┐')));
+  assert.ok(rows.some(r => r.includes('In Blossom')));
+  assert.ok(rows.some(r => r.includes('Everyday')));
+  assert.ok(rows.some(r => r.includes('Rain Love')));
+  assert.ok(rows.some(r => r.includes('└') && r.includes('┘')));
+});
+
+test('formatTerminalMarkdown formats headings h1-h5 and numbered list items', () => {
+  const markdownText = `#### Subheading 4
+##### Subheading 5
+1. First point
+2. Second point`;
+
+  const rows = formatMarkdownLines(markdownText, 80);
+  assert.ok(rows.some(r => r.includes('Subheading 4')));
+  assert.ok(rows.some(r => r.includes('Subheading 5')));
+  assert.ok(rows.some(r => r.includes('1.') && r.includes('First point')));
+  assert.ok(rows.some(r => r.includes('2.') && r.includes('Second point')));
+});
+
+test('formatMarkdownLines wraps long cell contents across multiple lines without truncating with ellipsis', () => {
+  const table = `| Detail | Finding | Source URL(s) | Uncertainty |
+| :--- | :--- | :--- | :--- |
+| SNH48 General Election Success | Elected No.4 (2014), No.2 (2015), No.1 (2016), No.1 (2017) | [Wikipedia](https://en.wikipedia.org/wiki/Ju_Jingyi) | Low |
+| Solo Transition | Established personal studio on December 15, 2017 | [carrd.co](https://ju-jingyi.carrd.co/) | Low |`;
+
+  const rows = formatMarkdownLines(table, 90);
+  assert.ok(rows.length > 0);
+  
+  const tableBorderRows = rows.filter(r => r.includes('│'));
+  assert.ok(tableBorderRows.length > 0);
+  assert.ok(!tableBorderRows.some(r => r.includes('…')));
+ 
+  assert.ok(rows.some(r => r.includes('Uncertainty')));
+
+  assert.ok(rows.some(r => r.includes('Elected No.4')));
+  assert.ok(rows.some(r => r.includes('(2017)')));
+});
+
+test('cleanTableCell and formatMarkdownLines handle <br> tags cleanly without raw HTML tags', () => {
+  const tableWithBr = `| Service | Links |
+| :--- | :--- |
+| Portals | [carrd.co](https://ju-jingyi.carrd.co/)<br>[Wikipedia](https://en.wikipedia.org/wiki/Ju_Jingyi) |`;
+
+  const rows = formatMarkdownLines(tableWithBr, 80);
+  assert.ok(rows.length > 0);
+  assert.ok(!rows.some(r => r.includes('<br')));
+  assert.ok(!rows.some(r => r.includes('<b…')));
+  assert.ok(rows.some(r => r.includes('carrd.co')));
+  assert.ok(rows.some(r => r.includes('Wikipedia')));
+});
+
+test('formatMarkdownLines does not corrupt bullet points with pipes into artificial tables', () => {
+  const markdownText = [
+    'Sources:',
+    '  • DuckDuckGo - Ju Jingyi Drama Tops Charts: https://www.youtube.com/watch?v=lkFoSXBARkQ',
+    '* DuckDuckGo - Best Chinese Dramas of 2026 So Far | Updated Rankings: https://mijudrama.com/blog/best-chinese-dramas-2026-updated-rankings',
+    '  • DuckDuckGo - Ju Jing Yi - DramaWiki: https://wiki.d-addicts.com/Ju_Jing_Yi',
+  ].join('\n');
+
+  const rows = formatMarkdownLines(markdownText, 100);
+  assert.equal(rows.some(r => r.startsWith('|') && r.endsWith('|')), false, 'Bullet with pipe should not become a table');
+  assert.ok(rows.some(r => r.includes('Best Chinese Dramas of 2026 So Far | Updated Rankings')));
+});
+
+test('formatTerminalMarkdown divider lines do not exceed maxWidth', () => {
+  const rows = formatMarkdownLines('---\n***\n___', 80);
+  for (const r of rows) {
+    const plain = r.replace(/\x1b\[[0-9;]*m/g, '');
+    assert.ok(plain.length <= 80, `Divider line should not exceed maxWidth (got ${plain.length})`);
+  }
+});