@agentrhq/webcmd 0.1.2 → 0.2.1

package/dist/src/constants.js

@@ -1,16 +1,9 @@
 /**
  * Shared constants used across explore, synthesize, and pipeline modules.
  */
-import { CLI_COMMAND, DAEMON_HEADER_NAME, ENV_PREFIX, PRODUCT_DISPLAY_NAME } from './brand.js';
-/** Default daemon port for HTTP/WebSocket communication with browser extension */
-export const DEFAULT_DAEMON_PORT = 19825;
-export function unsupportedDaemonPortEnvMessage(value) {
-    const envName = `${ENV_PREFIX}_DAEMON_PORT`;
-    const suffix = value ? ` (received ${value})` : '';
-    return `${envName} is no longer supported${suffix}. ` +
-        `The ${PRODUCT_DISPLAY_NAME} Chrome extension can only connect to localhost:${DEFAULT_DAEMON_PORT}. ` +
-        `Unset ${envName} and rerun ${CLI_COMMAND}.`;
-}
+import { DAEMON_HEADER_NAME } from './brand.js';
+/** Default daemon port for HTTP communication with the browser runtime. */
+export const DEFAULT_DAEMON_PORT = 9777;
 export { DAEMON_HEADER_NAME };
 /** URL query params that are volatile/ephemeral and should be stripped from patterns */
 export const VOLATILE_PARAMS = new Set([

package/dist/src/convention-audit.js

@@ -244,7 +244,7 @@ function auditTypedErrorPatterns(command, source, sourcePath, projectRoot) {
                 details: { text: line.trim() },
             });
         }
-        const sentinel = /(?:\?\?|\|\|)\s*(['"])(unknown|Unknown|UNKNOWN|N\/A|n\/a|NA|未知|-)\1/.exec(line);
+        const sentinel = /(?:\?\?|\|\|)\s*(['"])(unknown|Unknown|UNKNOWN|N\/A|n\/a|NA|-)\1/.exec(line);
         if (sentinel) {
             if (!isThrowMessageLine(line)) {
                 violations.push({

package/dist/src/daemon/server.js

@@ -1,6 +1,6 @@
 import { createServer } from 'node:http';
 import { DAEMON_HEADER_NAME, DEFAULT_DAEMON_PORT } from '../constants.js';
-import { getResponseCorsHeaders } from '../daemon-utils.js';
+import { buildCommandTimeoutFailure, getResponseCorsHeaders } from '../daemon-utils.js';
 const MAX_BODY = 1024 * 1024;
 const LOG_BUFFER_SIZE = 200;
 function readBody(req) {
@@ -36,7 +36,7 @@ export function createDaemonServer(provider, opts) {
     const port = opts.port ?? DEFAULT_DAEMON_PORT;
     const host = opts.host ?? '127.0.0.1';
     const logBuffer = [];
-    const pending = new Set();
+    const pending = new Map();
     let shutdownStarted = false;
     function pushLog(level, msg) {
         logBuffer.push({ level, msg, ts: Date.now() });
@@ -115,35 +115,40 @@ export function createDaemonServer(provider, opts) {
                     jsonResponse(res, 400, { ok: false, error: 'Missing command id' });
                     return;
                 }
-                if (pending.has(body.id)) {
-                    jsonResponse(res, 409, { id: body.id, ok: false, error: 'Duplicate command id already pending; retry' });
+                const existing = pending.get(body.id);
+                if (existing) {
+                    const result = await existing;
+                    jsonResponse(res, result.ok ? 200 : result.errorCode === 'command_result_unknown' ? 408 : 400, result);
                     return;
                 }
-                pending.add(body.id);
-                const timeoutMs = typeof body.timeout === 'number' && body.timeout > 0 ? body.timeout * 1000 : 120_000;
+                const timeoutMs = typeof body.deadlineAt === 'number' && body.deadlineAt > 0
+                    ? Math.max(1000, body.deadlineAt - Date.now())
+                    : (typeof body.timeout === 'number' && body.timeout > 0 ? body.timeout * 1000 : 120_000);
                 let timeoutId;
-                try {
-                    const result = await Promise.race([
-                        provider.dispatch(body),
-                        new Promise((resolve) => {
-                            timeoutId = setTimeout(() => resolve({
+                const commandPromise = Promise.race([
+                    provider.dispatch(body),
+                    new Promise((resolve) => {
+                        timeoutId = setTimeout(() => {
+                            const failure = buildCommandTimeoutFailure(body.action, timeoutMs);
+                            resolve({
                                 id: body.id,
                                 ok: false,
-                                errorCode: 'command_result_unknown',
-                                error: `Command timeout (${timeoutMs / 1000}s)`,
-                                errorHint: 'Inspect the browser state before retrying non-idempotent commands.',
-                            }), timeoutMs);
-                        }),
-                    ]);
-                    if (!result.ok)
-                        pushLog('warn', `Command ${body.id} failed: ${result.error ?? result.errorCode ?? 'unknown error'}`);
-                    jsonResponse(res, result.ok ? 200 : result.errorCode === 'command_result_unknown' ? 408 : 400, result);
-                }
-                finally {
+                                errorCode: failure.errorCode,
+                                error: failure.message,
+                                errorHint: failure.errorHint,
+                            });
+                        }, timeoutMs);
+                    }),
+                ]).finally(() => {
                     if (timeoutId)
                         clearTimeout(timeoutId);
                     pending.delete(body.id);
-                }
+                });
+                pending.set(body.id, commandPromise);
+                const result = await commandPromise;
+                if (!result.ok)
+                    pushLog('warn', `Command ${body.id} failed: ${result.error ?? result.errorCode ?? 'unknown error'}`);
+                jsonResponse(res, result.ok ? 200 : result.errorCode === 'command_result_unknown' ? 408 : 400, result);
             }
             catch (err) {
                 jsonResponse(res, 400, { ok: false, error: err instanceof Error ? err.message : 'Invalid request' });

package/dist/src/daemon/server.test.js

@@ -93,6 +93,58 @@ describe('createDaemonServer', () => {
             clearTimeoutSpy.mockRestore();
         }
     });
+    it('uses deadlineAt as the command timeout budget when present', async () => {
+        const now = 1_763_000_000_000;
+        const dateSpy = vi.spyOn(Date, 'now').mockReturnValue(now);
+        const setTimeoutSpy = vi.spyOn(globalThis, 'setTimeout');
+        try {
+            const { baseUrl } = await start();
+            const res = await fetch(`${baseUrl}/command`, {
+                method: 'POST',
+                headers: { [DAEMON_HEADER_NAME]: '1', 'Content-Type': 'application/json' },
+                body: JSON.stringify({
+                    id: 'deadline-command',
+                    action: 'exec',
+                    session: 'work',
+                    timeout: 120,
+                    deadlineAt: now + 12_345,
+                }),
+            });
+            expect(res.status).toBe(200);
+            await expect(res.json()).resolves.toMatchObject({
+                id: 'deadline-command',
+                ok: true,
+            });
+            expect(setTimeoutSpy.mock.calls.some(([, delay]) => delay === 12_345)).toBe(true);
+        }
+        finally {
+            setTimeoutSpy.mockRestore();
+            dateSpy.mockRestore();
+        }
+    });
+    it('attaches duplicate command ids to the in-flight dispatch instead of re-dispatching', async () => {
+        const provider = new FakeProvider();
+        provider.delayMs = 50;
+        const { baseUrl } = await start(provider);
+        const body = JSON.stringify({ id: 'same-id', action: 'exec', session: 'work', code: '1' });
+        const [first, second] = await Promise.all([
+            fetch(`${baseUrl}/command`, {
+                method: 'POST',
+                headers: { [DAEMON_HEADER_NAME]: '1', 'Content-Type': 'application/json' },
+                body,
+            }),
+            fetch(`${baseUrl}/command`, {
+                method: 'POST',
+                headers: { [DAEMON_HEADER_NAME]: '1', 'Content-Type': 'application/json' },
+                body,
+            }),
+        ]);
+        expect(first.status).toBe(200);
+        expect(second.status).toBe(200);
+        await expect(first.json()).resolves.toMatchObject({ id: 'same-id', ok: true });
+        await expect(second.json()).resolves.toMatchObject({ id: 'same-id', ok: true });
+        expect(provider.commands).toHaveLength(1);
+    });
     it('requires X-Webcmd on non-ping endpoints', async () => {
         const { baseUrl } = await start();
         const res = await fetch(`${baseUrl}/status`);

package/dist/src/daemon-utils.d.ts

@@ -14,5 +14,6 @@ export declare function buildExtensionDisconnectFailure(input: {
     action: string;
     dispatched: boolean;
 }): DaemonFailureContract;
+export declare function buildCommandTimeoutFailure(action: string, timeoutMs: number): DaemonFailureContract;
 export declare function buildCommandDispatchFailure(contextId: string): DaemonFailureContract;
 export declare function getResponseCorsHeaders(pathname: string, origin?: string): Record<string, string> | undefined;

package/dist/src/daemon-utils.js

@@ -16,6 +16,15 @@ export function buildExtensionDisconnectFailure(input) {
     }
     return buildCommandDispatchFailure(input.contextId);
 }
+export function buildCommandTimeoutFailure(action, timeoutMs) {
+    return {
+        message: `Browser ${action} command timed out after ${Math.round(timeoutMs / 1000)}s; it may still complete in the browser.`,
+        errorCode: COMMAND_RESULT_UNKNOWN_CODE,
+        errorHint: COMMAND_RESULT_UNKNOWN_HINT,
+        status: 408,
+        countAsCommandResultUnknown: true,
+    };
+}
 export function buildCommandDispatchFailure(contextId) {
     return {
         message: `Browser profile "${contextId}" disconnected before command dispatch`,

package/dist/src/daemon.js

@@ -1,13 +1,9 @@
-import { DEFAULT_DAEMON_PORT, unsupportedDaemonPortEnvMessage } from './constants.js';
+import { DEFAULT_DAEMON_PORT } from './constants.js';
 import { EXIT_CODES } from './errors.js';
 import { log } from './logger.js';
 import { PKG_VERSION } from './version.js';
 import { createDaemonServer } from './daemon/server.js';
 import { LocalCloakRuntimeProvider } from './browser/runtime/local-cloak/provider.js';
-if (process.env.WEBCMD_DAEMON_PORT) {
-    log.error(unsupportedDaemonPortEnvMessage(process.env.WEBCMD_DAEMON_PORT));
-    process.exit(EXIT_CODES.USAGE_ERROR);
-}
 const provider = new LocalCloakRuntimeProvider();
 const daemon = createDaemonServer(provider, { port: DEFAULT_DAEMON_PORT, host: '127.0.0.1', version: PKG_VERSION });
 daemon.listen().then(() => {

package/dist/src/daemon.test.js

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { COMMAND_RESULT_UNKNOWN_CODE, COMMAND_RESULT_UNKNOWN_HINT, buildCommandDispatchFailure, buildExtensionDisconnectFailure, commandResultUnknownMessage, getResponseCorsHeaders, } from './daemon-utils.js';
+import { COMMAND_RESULT_UNKNOWN_CODE, COMMAND_RESULT_UNKNOWN_HINT, buildCommandDispatchFailure, buildCommandTimeoutFailure, buildExtensionDisconnectFailure, commandResultUnknownMessage, getResponseCorsHeaders, } from './daemon-utils.js';
 describe('getResponseCorsHeaders', () => {
     it('allows the packaged runtime origin to read /ping', () => {
         expect(getResponseCorsHeaders('/ping', 'chrome-extension://abc123')).toEqual({
@@ -57,4 +57,13 @@ describe('daemon command dispatch', () => {
             countAsCommandResultUnknown: false,
         });
     });
+    it('classifies command timeouts as command_result_unknown', () => {
+        expect(buildCommandTimeoutFailure('exec', 12_345)).toEqual({
+            message: 'Browser exec command timed out after 12s; it may still complete in the browser.',
+            errorCode: 'command_result_unknown',
+            errorHint: COMMAND_RESULT_UNKNOWN_HINT,
+            status: 408,
+            countAsCommandResultUnknown: true,
+        });
+    });
 });

package/dist/src/doctor.js

@@ -5,6 +5,7 @@
  */
 import { DEFAULT_DAEMON_PORT } from './constants.js';
 import { BrowserBridge } from './browser/index.js';
+import { setDaemonCommandTimeoutSeconds } from './browser/daemon-client.js';
 import { getDaemonHealth } from './browser/daemon-transport.js';
 import { getErrorMessage } from './errors.js';
 import { getRuntimeLabel } from './runtime-detect.js';
@@ -18,10 +19,12 @@ const DOCTOR_SESSION = '__doctor__';
  */
 export async function checkConnectivity(opts) {
     const start = Date.now();
+    const timeoutSeconds = opts?.timeout ?? DOCTOR_LIVE_TIMEOUT_SECONDS;
+    setDaemonCommandTimeoutSeconds(timeoutSeconds);
     try {
         const bridge = new BrowserBridge();
         const page = await bridge.connect({
-            timeout: opts?.timeout ?? DOCTOR_LIVE_TIMEOUT_SECONDS,
+            timeout: timeoutSeconds,
             session: DOCTOR_SESSION,
             surface: 'browser',
         });
@@ -38,6 +41,9 @@ export async function checkConnectivity(opts) {
     catch (err) {
         return { ok: false, error: getErrorMessage(err), durationMs: Date.now() - start };
     }
+    finally {
+        setDaemonCommandTimeoutSeconds(null);
+    }
 }
 export async function runBrowserDoctor(opts = {}) {
     // Live connectivity check is the core of doctor — it doubles as auto-start
@@ -87,6 +93,18 @@ export async function runBrowserDoctor(opts = {}) {
     if (!connectivity.ok) {
         issues.push(`Browser connectivity test failed: ${connectivity.error ?? 'unknown'}`);
     }
+    const profileConfig = loadProfileConfig();
+    const staleDefault = profileConfig.defaultContextId;
+    if (staleDefault && profiles?.length && !profiles.some((p) => p.contextId === staleDefault)) {
+        const alias = aliasForContextId(profileConfig, staleDefault);
+        const label = alias ? `${alias} (${staleDefault})` : staleDefault;
+        const fallbackNote = profiles.length === 1
+            ? `Commands currently fall back to the only active profile: ${profiles[0].contextId}.`
+            : 'Multiple profiles are active, so commands will ask you to choose.';
+        issues.push(`Default Cloak profile is not active: ${label}.\n` +
+            `  ${fallbackNote}\n` +
+            '  Refresh it with: webcmd profile list, then webcmd profile use <name>.');
+    }
     if (adapterShadows.length > 0) {
         issues.push(formatAdapterShadowIssue(adapterShadows));
     }

package/dist/src/doctor.test.js

@@ -1,9 +1,10 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest';
-const { mockGetDaemonHealth, mockConnect, mockClose, mockFindShadowedUserAdapters } = vi.hoisted(() => ({
+const { mockGetDaemonHealth, mockConnect, mockClose, mockFindShadowedUserAdapters, mockSetDaemonCommandTimeoutSeconds, } = vi.hoisted(() => ({
     mockGetDaemonHealth: vi.fn(),
     mockConnect: vi.fn(),
     mockClose: vi.fn(),
     mockFindShadowedUserAdapters: vi.fn(),
+    mockSetDaemonCommandTimeoutSeconds: vi.fn(),
 }));
 vi.mock('./browser/daemon-transport.js', () => ({
     getDaemonHealth: mockGetDaemonHealth,
@@ -14,6 +15,9 @@ vi.mock('./browser/index.js', () => ({
         close = mockClose;
     },
 }));
+vi.mock('./browser/daemon-client.js', () => ({
+    setDaemonCommandTimeoutSeconds: mockSetDaemonCommandTimeoutSeconds,
+}));
 vi.mock('./adapter-shadow.js', async () => {
     const actual = await vi.importActual('./adapter-shadow.js');
     return {
@@ -27,6 +31,7 @@ describe('doctor report rendering', () => {
     beforeEach(() => {
         vi.clearAllMocks();
         mockFindShadowedUserAdapters.mockReturnValue([]);
+        mockSetDaemonCommandTimeoutSeconds.mockClear();
         // Doctor always runs live connectivity. Tests that want connect to fail override.
         mockConnect.mockResolvedValue({
             evaluate: vi.fn().mockResolvedValue(2),
@@ -44,7 +49,7 @@ describe('doctor report rendering', () => {
             runtimeVersion: '1.6.8',
             issues: [],
         }));
-        expect(text).toContain('[OK] Daemon: running on port 19825');
+        expect(text).toContain('[OK] Daemon: running on port 9777');
         expect(text).toContain('(v1.7.9)');
         expect(text).toContain('[OK] Runtime: Cloak connected (v1.6.8)');
         expect(text).toContain('Everything looks good!');
@@ -61,7 +66,7 @@ describe('doctor report rendering', () => {
             runtimeVersion: '1.0.3',
             issues: ['Stale daemon detected: daemon v1.7.6 != CLI v1.7.9.\n  Run: webcmd daemon restart'],
         }));
-        expect(text).toContain('[WARN] Daemon: running on port 19825 (v1.7.6, stale; CLI v1.7.9)');
+        expect(text).toContain('[WARN] Daemon: running on port 9777 (v1.7.6, stale; CLI v1.7.9)');
         expect(text).toContain('Run: webcmd daemon restart');
         expect(text).not.toContain('Everything looks good!');
     });
@@ -81,7 +86,7 @@ describe('doctor report rendering', () => {
             runtimeConnected: false,
             issues: ['Daemon is running but the Cloak runtime is not connected.'],
         }));
-        expect(text).toContain('[OK] Daemon: running on port 19825');
+        expect(text).toContain('[OK] Daemon: running on port 9777');
         expect(text).toContain('[MISSING] Runtime: Cloak not connected');
     });
     it('renders OK when the connected Cloak runtime version is unknown', () => {
@@ -152,6 +157,33 @@ describe('doctor report rendering', () => {
             expect.stringContaining('Daemon is not running'),
         ]));
     });
+    it('reports a stale default Cloak profile when it is not active', async () => {
+        const fs = await import('node:fs');
+        const os = await import('node:os');
+        const path = await import('node:path');
+        const configDir = fs.mkdtempSync(path.join(os.tmpdir(), 'webcmd-doctor-profile-'));
+        fs.writeFileSync(path.join(configDir, 'browser-profiles.json'), JSON.stringify({ version: 1, aliases: { work: 'profile-default' }, defaultContextId: 'profile-default' }));
+        vi.stubEnv('WEBCMD_CONFIG_DIR', configDir);
+        try {
+            mockGetDaemonHealth.mockResolvedValueOnce({
+                state: 'ready',
+                status: {
+                    runtimeConnected: true,
+                    runtimeName: 'Cloak',
+                    profiles: [{ contextId: 'active-profile', runtimeConnected: true, pending: 0 }],
+                },
+            });
+            const report = await runBrowserDoctor();
+            expect(report.issues).toEqual(expect.arrayContaining([
+                expect.stringContaining('Default Cloak profile is not active: work (profile-default)'),
+            ]));
+            expect(report.issues.join('\n')).toContain('fall back to the only active profile: active-profile');
+        }
+        finally {
+            vi.unstubAllEnvs();
+            fs.rmSync(configDir, { recursive: true, force: true });
+        }
+    });
     it('reports flapping when live check succeeds but final status shows runtime disconnected', async () => {
         mockGetDaemonHealth.mockResolvedValueOnce({ state: 'no-runtime', status: { runtimeConnected: false, runtimeName: 'Cloak' } });
         const report = await runBrowserDoctor();
@@ -202,6 +234,8 @@ describe('doctor report rendering', () => {
         await runBrowserDoctor();
         expect(timeoutSeen).toBe(8);
         expect(closeWindow).toHaveBeenCalledTimes(1);
+        expect(mockSetDaemonCommandTimeoutSeconds).toHaveBeenNthCalledWith(1, 8);
+        expect(mockSetDaemonCommandTimeoutSeconds).toHaveBeenLastCalledWith(null);
     });
     it('does not report an issue when the connected Cloak runtime does not report a version', async () => {
         const status = {

package/dist/src/download/article-download.d.ts

@@ -1,7 +1,7 @@
 /**
  * Article download helper — shared logic for downloading articles as Markdown.
  *
- * Used by: zhihu/download, weixin/download, and future article adapters.
+ * Used by article adapters that export readable web pages as Markdown.
  *
  * Flow: ArticleData → TurndownService → image download → frontmatter → .md file
  */
@@ -35,12 +35,12 @@ export interface ArticleDownloadOptions {
     configureTurndown?: (td: TurndownService) => void;
     /** Custom image extension detector (default: infer from URL extension) */
     detectImageExt?: (url: string) => string;
-    /** Custom frontmatter labels (default: Chinese labels) */
+    /** Custom frontmatter labels (default: English labels) */
     frontmatterLabels?: FrontmatterLabels;
     /**
      * Extra CSS selectors removed from the article before Turndown conversion.
      * Use this to drop site-specific noise the adapter can't always trim upstream
-     * (e.g. zhihu 折叠卡, weixin 赞赏栏, wiki infobox).
+     * (e.g. social widgets, reward panels, wiki infoboxes).
      */
     cleanSelectors?: string[];
     /**

package/dist/src/download/article-download.js

@@ -1,7 +1,7 @@
 /**
  * Article download helper — shared logic for downloading articles as Markdown.
  *
- * Used by: zhihu/download, weixin/download, and future article adapters.
+ * Used by article adapters that export readable web pages as Markdown.
  *
  * Flow: ArticleData → TurndownService → image download → frontmatter → .md file
  */
@@ -13,9 +13,9 @@ import { httpDownload, sanitizeFilename } from './index.js';
 import { formatBytes } from './progress.js';
 const IMAGE_CONCURRENCY = 5;
 const DEFAULT_LABELS = {
-    author: '作者',
-    publishTime: '发布时间',
-    sourceUrl: '原文链接',
+    author: 'Author',
+    publishTime: 'Published',
+    sourceUrl: 'Source URL',
 };
 // ============================================================
 // Markdown Conversion
@@ -112,8 +112,8 @@ function createTurndown(configure, cleanSelectors) {
         },
     });
     // Per-adapter dirty-node removal. Adapters know their site's specific noise
-    // (zhihu 折叠卡, weixin 赞赏栏, wiki 折叠 infobox …); we keep the default set
-    // empty so the generic converter stays untouched.
+    // (social widgets, reward panels, wiki infoboxes, and similar chrome);
+    // we keep the default set empty so the generic converter stays untouched.
     const selectorRules = (cleanSelectors ?? [])
         .map(sel => sel.trim())
         .filter(Boolean);
@@ -152,7 +152,7 @@ function convertToMarkdown(contentHtml, codeBlocks, configure, cleanSelectors) {
     md = md.replace(/\u00a0/g, ' ');
     // Turndown leaves behind lone dashes / middle dots when list bullets or
     // decorative separators lose their surrounding inline context.
-    md = md.replace(/^[ \t]*[-·][ \t]*$/gm, '');
+    md = md.replace(/^[ \t]*[-\u00b7][ \t]*$/gm, '');
     md = md.replace(/^[ \t]+$/gm, '');
     md = md.replace(/[ \t]+$/gm, '');
     md = md.replace(/\n{3,}/g, '\n\n');

package/dist/src/download/article-download.test.js

@@ -96,11 +96,11 @@ describe('downloadArticle', () => {
         it('collapses 3+ blank lines and strips lone bullet / middle-dot residue', async () => {
             const md = await runAndRead('<p>top</p>' +
                 '<p>-</p>' +
-                '<p>·</p>' +
+                '<p>\u00b7</p>' +
                 '<p>bottom</p>');
             expect(md).not.toMatch(/\n{3,}/);
             expect(md).not.toMatch(/^\s*-\s*$/m);
-            expect(md).not.toMatch(/^\s*·\s*$/m);
+            expect(md).not.toMatch(/^\s*\u00b7\s*$/m);
             expect(md).toContain('top');
             expect(md).toContain('bottom');
         });
@@ -118,13 +118,13 @@ describe('downloadArticle', () => {
         });
         it('cleanSelectors removes matching nodes before conversion', async () => {
             const md = await runAndRead('<p>keep-me</p>' +
-                '<div class="vote-card">折叠卡</div>' +
-                '<section class="reward-panel">赞赏栏</section>' +
+                '<div class="vote-card">Collapsed card</div>' +
+                '<section class="reward-panel">Reward panel</section>' +
                 '<p>also-keep</p>', { cleanSelectors: ['.vote-card', '.reward-panel'] });
             expect(md).toContain('keep-me');
             expect(md).toContain('also-keep');
-            expect(md).not.toContain('折叠卡');
-            expect(md).not.toContain('赞赏栏');
+            expect(md).not.toContain('Collapsed card');
+            expect(md).not.toContain('Reward panel');
         });
         it('cleanSelectors silently ignores invalid selectors', async () => {
             const md = await runAndRead('<p>survives</p><div class="x">and-this-too</div>', { cleanSelectors: ['!!!not-a-valid-selector', '.missing'] });

package/dist/src/download/index.js

@@ -18,7 +18,7 @@ export function checkYtdlp() {
 }
 /** Domains that host video content and can be downloaded via yt-dlp. */
 const VIDEO_PLATFORM_DOMAINS = [
-    'youtube.com', 'youtu.be', 'bilibili.com', 'twitter.com',
+    'youtube.com', 'youtu.be', 'twitter.com',
     'x.com', 'tiktok.com', 'vimeo.com', 'twitch.tv',
 ];
 /**

package/dist/src/download/index.test.js

@@ -143,11 +143,10 @@ describe('video-platform host detection', () => {
         for (const url of [
             'https://www.youtube.com/watch?v=abc',
             'https://youtu.be/abc',
-            'https://m.bilibili.com/video/BV1xx',
+            'https://vimeo.com/12345',
             'https://twitter.com/u/status/1',
             'https://x.com/u/status/1',
             'https://www.tiktok.com/@u/video/1',
-            'https://vimeo.com/12345',
             'https://www.twitch.tv/u',
         ]) {
             expect(requiresYtdlp(url)).toBe(true);

package/dist/src/download/media-download.d.ts

@@ -1,7 +1,7 @@
 /**
  * Media download helper — shared logic for batch downloading images/videos.
  *
- * Used by: xiaohongshu/download, twitter/download, bilibili/download,
+ * Used by: twitter/download, youtube/download,
  * and future media adapters.
  *
  * Flow: MediaItem[] → DownloadProgressTracker → httpDownload/ytdlpDownload → results

package/dist/src/download/media-download.js

@@ -1,7 +1,7 @@
 /**
  * Media download helper — shared logic for batch downloading images/videos.
  *
- * Used by: xiaohongshu/download, twitter/download, bilibili/download,
+ * Used by: twitter/download, youtube/download,
  * and future media adapters.
  *
  * Flow: MediaItem[] → DownloadProgressTracker → httpDownload/ytdlpDownload → results

package/dist/src/electron-apps.js

@@ -13,7 +13,6 @@ export const builtinApps = {
     codex: { port: 9238, processName: 'Codex', bundleId: 'com.openai.codex', displayName: 'Codex' },
     chatwise: { port: 9228, processName: 'ChatWise', bundleId: 'com.chatwise.app', displayName: 'ChatWise' },
     'discord-app': { port: 9232, processName: 'Discord', bundleId: 'com.discord.app', displayName: 'Discord' },
-    'doubao-app': { port: 9225, processName: 'Doubao', bundleId: 'com.volcengine.doubao', displayName: 'Doubao' },
     antigravity: {
         port: 9234,
         processName: 'Antigravity',
@@ -36,13 +35,6 @@ export const builtinApps = {
         bundleId: 'com.trae.solo.app',
         displayName: 'Trae SOLO',
     },
-    'trae-cn': {
-        port: 39240,
-        processName: 'Trae CN',
-        executableNames: ['Electron'],
-        bundleId: 'cn.trae.app',
-        displayName: 'Trae CN',
-    },
 };
 /** Merge builtin + user-defined apps. User entries are additive only. */
 export function loadApps(userApps) {

package/dist/src/electron-apps.test.js

@@ -17,16 +17,8 @@ describe('electron-apps registry', () => {
         expect(new Set(ports).size).toBe(ports.length);
         expect(ports).not.toContain(9222);
     });
-    it('returns builtin app entry for Trae CN', () => {
-        const app = getElectronApp('trae-cn');
-        expect(app).toBeDefined();
-        expect(app.port).toBe(39240);
-        expect(app.processName).toBe('Trae CN');
-        expect(app.bundleId).toBe('cn.trae.app');
-        expect(app.executableNames).toEqual(['Electron']);
-    });
     it('returns undefined for non-Electron sites', () => {
-        expect(getElectronApp('bilibili')).toBeUndefined();
+        expect(getElectronApp('youtube')).toBeUndefined();
         expect(getElectronApp('hackernews')).toBeUndefined();
     });
     it('isElectronApp returns true for registered apps', () => {
@@ -35,7 +27,6 @@ describe('electron-apps registry', () => {
         expect(isElectronApp('chatwise')).toBe(true);
         expect(isElectronApp('qoder')).toBe(true);
         expect(isElectronApp('trae-solo')).toBe(true);
-        expect(isElectronApp('trae-cn')).toBe(true);
     });
     it('registers Qoder on its own CDP port', () => {
         const app = getElectronApp('qoder');
@@ -54,7 +45,7 @@ describe('electron-apps registry', () => {
         expect(app.bundleId).toBe('com.trae.solo.app');
     });
     it('isElectronApp returns false for non-Electron sites', () => {
-        expect(isElectronApp('bilibili')).toBe(false);
+        expect(isElectronApp('youtube')).toBe(false);
         expect(isElectronApp('notion')).toBe(false);
         expect(isElectronApp('unknown-app')).toBe(false);
     });

package/dist/src/errors.test.js

@@ -19,11 +19,11 @@ describe('Error type hierarchy', () => {
         }
     });
     it('AuthRequiredError has correct code, domain, and auto-generated hint', () => {
-        const err = new AuthRequiredError('bilibili.com');
+        const err = new AuthRequiredError('youtube.com');
         expect(err.code).toBe('AUTH_REQUIRED');
-        expect(err.domain).toBe('bilibili.com');
-        expect(err.message).toBe('Not logged in to bilibili.com');
-        expect(err.hint).toContain('https://bilibili.com');
+        expect(err.domain).toBe('youtube.com');
+        expect(err.message).toBe('Not logged in to youtube.com');
+        expect(err.hint).toContain('https://youtube.com');
     });
     it('AuthRequiredError accepts custom message', () => {
         const err = new AuthRequiredError('x.com', 'No ct0 cookie found');
@@ -31,9 +31,9 @@ describe('Error type hierarchy', () => {
         expect(err.hint).toContain('https://x.com');
     });
     it('TimeoutError has correct code and hint', () => {
-        const err = new TimeoutError('bilibili/hot', 60);
+        const err = new TimeoutError('youtube/search', 60);
         expect(err.code).toBe('TIMEOUT');
-        expect(err.message).toBe('bilibili/hot timed out after 60s');
+        expect(err.message).toBe('youtube/search timed out after 60s');
         expect(err.hint).toContain('timeout');
     });
     it('ArgumentError has correct code', () => {
@@ -59,14 +59,14 @@ describe('Error type hierarchy', () => {
 });
 describe('toEnvelope', () => {
     it('converts CliError to structured envelope', () => {
-        const err = new AuthRequiredError('bilibili.com');
+        const err = new AuthRequiredError('youtube.com');
         const envelope = toEnvelope(err);
         expect(envelope).toEqual({
             ok: false,
             error: {
                 code: 'AUTH_REQUIRED',
-                message: 'Not logged in to bilibili.com',
-                help: expect.stringContaining('https://bilibili.com'),
+                message: 'Not logged in to youtube.com',
+                help: expect.stringContaining('https://youtube.com'),
                 exitCode: 77,
             },
         });