@jackwener/opencli 1.0.0 → 1.0.1

package/src/doctor.ts

@@ -7,7 +7,7 @@
 
 import chalk from 'chalk';
 import { checkDaemonStatus } from './browser/discover.js';
-import { PlaywrightMCP } from './browser/index.js';
+import { BrowserBridge } from './browser/index.js';
 import { browserSession } from './runtime.js';
 
 export type DoctorOptions = {
@@ -37,7 +37,7 @@ export type DoctorReport = {
 export async function checkConnectivity(opts?: { timeout?: number }): Promise<ConnectivityResult> {
   const start = Date.now();
   try {
-    const mcp = new PlaywrightMCP();
+    const mcp = new BrowserBridge();
     const page = await mcp.connect({ timeout: opts?.timeout ?? 8 });
     // Try a simple eval to verify end-to-end connectivity
     await page.evaluate('1 + 1');
@@ -116,20 +116,3 @@ export function renderBrowserDoctorReport(report: DoctorReport): string {
   return lines.join('\n');
 }
 
-// Backward compatibility exports (no-ops for things that no longer exist)
-export const PLAYWRIGHT_TOKEN_ENV = 'PLAYWRIGHT_MCP_EXTENSION_TOKEN';
-export function discoverExtensionToken(): string | null { return null; }
-export function checkExtensionInstalled(): { installed: boolean; browsers: string[] } { return { installed: false, browsers: [] }; }
-export function applyBrowserDoctorFix(): Promise<string[]> { return Promise.resolve([]); }
-export function getDefaultShellRcPath(): string { return ''; }
-export function getDefaultMcpConfigPaths(): string[] { return []; }
-export function readTokenFromShellContent(_content: string): string | null { return null; }
-export function upsertShellToken(content: string): string { return content; }
-export function upsertJsonConfigToken(content: string): string { return content; }
-export function readTomlConfigToken(_content: string): string | null { return null; }
-export function upsertTomlConfigToken(content: string): string { return content; }
-export function shortenPath(p: string): string { return p; }
-export function toolName(_p: string): string { return ''; }
-export function fileExists(filePath: string): boolean { try { return require('node:fs').existsSync(filePath); } catch { return false; } }
-export function writeFileWithMkdir(_p: string, _c: string): void {}
-export async function checkTokenConnectivity(opts?: { timeout?: number }): Promise<ConnectivityResult> { return checkConnectivity(opts); }

package/src/main.ts

@@ -11,7 +11,7 @@ import chalk from 'chalk';
 import { discoverClis, executeCommand } from './engine.js';
 import { Strategy, type CliCommand, fullName, getRegistry, strategyLabel } from './registry.js';
 import { render as renderOutput } from './output.js';
-import { PlaywrightMCP } from './browser/index.js';
+import { BrowserBridge } from './browser/index.js';
 import { browserSession, DEFAULT_BROWSER_COMMAND_TIMEOUT, runWithTimeout } from './runtime.js';
 import { PKG_VERSION } from './version.js';
 import { getCompletions, printCompletionScript } from './completion.js';
@@ -99,18 +99,18 @@ program.command('verify').description('Validate + smoke test').argument('[target
   });
 
 program.command('explore').alias('probe').description('Explore a website: discover APIs, stores, and recommend strategies').argument('<url>').option('--site <name>').option('--goal <text>').option('--wait <s>', '', '3').option('--auto', 'Enable interactive fuzzing (simulate clicks to trigger lazy APIs)').option('--click <labels>', 'Comma-separated labels to click before fuzzing (e.g. "字幕,CC,评论")')
-  .action(async (url, opts) => { const { exploreUrl, renderExploreSummary } = await import('./explore.js'); const clickLabels = opts.click ? opts.click.split(',').map((s: string) => s.trim()) : undefined; console.log(renderExploreSummary(await exploreUrl(url, { BrowserFactory: PlaywrightMCP, site: opts.site, goal: opts.goal, waitSeconds: parseFloat(opts.wait), auto: opts.auto, clickLabels }))); });
+  .action(async (url, opts) => { const { exploreUrl, renderExploreSummary } = await import('./explore.js'); const clickLabels = opts.click ? opts.click.split(',').map((s: string) => s.trim()) : undefined; console.log(renderExploreSummary(await exploreUrl(url, { BrowserFactory: BrowserBridge, site: opts.site, goal: opts.goal, waitSeconds: parseFloat(opts.wait), auto: opts.auto, clickLabels }))); });
 
 program.command('synthesize').description('Synthesize CLIs from explore').argument('<target>').option('--top <n>', '', '3')
   .action(async (target, opts) => { const { synthesizeFromExplore, renderSynthesizeSummary } = await import('./synthesize.js'); console.log(renderSynthesizeSummary(synthesizeFromExplore(target, { top: parseInt(opts.top) }))); });
 
 program.command('generate').description('One-shot: explore → synthesize → register').argument('<url>').option('--goal <text>').option('--site <name>')
-  .action(async (url, opts) => { const { generateCliFromUrl, renderGenerateSummary } = await import('./generate.js'); const r = await generateCliFromUrl({ url, BrowserFactory: PlaywrightMCP, builtinClis: BUILTIN_CLIS, userClis: USER_CLIS, goal: opts.goal, site: opts.site }); console.log(renderGenerateSummary(r)); process.exitCode = r.ok ? 0 : 1; });
+  .action(async (url, opts) => { const { generateCliFromUrl, renderGenerateSummary } = await import('./generate.js'); const r = await generateCliFromUrl({ url, BrowserFactory: BrowserBridge, builtinClis: BUILTIN_CLIS, userClis: USER_CLIS, goal: opts.goal, site: opts.site }); console.log(renderGenerateSummary(r)); process.exitCode = r.ok ? 0 : 1; });
 
 program.command('cascade').description('Strategy cascade: find simplest working strategy').argument('<url>').option('--site <name>')
   .action(async (url, opts) => {
     const { cascadeProbe, renderCascadeResult } = await import('./cascade.js');
-    const result = await browserSession(PlaywrightMCP, async (page) => {
+    const result = await browserSession(BrowserBridge, async (page) => {
       // Navigate to the site first for cookie context
       try { const siteUrl = new URL(url); await page.goto(`${siteUrl.protocol}//${siteUrl.host}`); await page.wait(2); } catch {}
       return cascadeProbe(page, url);
@@ -192,13 +192,7 @@ for (const [, cmd] of registry) {
       if (actionOpts.verbose) process.env.OPENCLI_VERBOSE = '1';
       let result: any;
       if (cmd.browser) {
-        result = await browserSession(PlaywrightMCP, async (page) => {
-          // Cookie/header strategies require same-origin context for credentialed fetch.
-          // In CDP mode the active tab may be on an unrelated domain, causing CORS failures.
-          // Navigate to the command's domain first (mirrors cascade command behavior).
-          if ((cmd.strategy === Strategy.COOKIE || cmd.strategy === Strategy.HEADER) && cmd.domain) {
-            try { await page.goto(`https://${cmd.domain}`); await page.wait(2); } catch {}
-          }
+        result = await browserSession(BrowserBridge, async (page) => {
           return runWithTimeout(executeCommand(cmd, page, kwargs, actionOpts.verbose), { timeout: cmd.timeoutSeconds ?? DEFAULT_BROWSER_COMMAND_TIMEOUT, label: fullName(cmd) });
         });
       } else { result = await executeCommand(cmd, null, kwargs, actionOpts.verbose); }

package/src/runtime.ts

@@ -1,11 +1,7 @@
-/**
- * Runtime utilities: timeouts and browser session management.
- */
-
 import type { IPage } from './types.js';
 
 export const DEFAULT_BROWSER_CONNECT_TIMEOUT = parseInt(process.env.OPENCLI_BROWSER_CONNECT_TIMEOUT ?? '30', 10);
-export const DEFAULT_BROWSER_COMMAND_TIMEOUT = parseInt(process.env.OPENCLI_BROWSER_COMMAND_TIMEOUT ?? '45', 10);
+export const DEFAULT_BROWSER_COMMAND_TIMEOUT = parseInt(process.env.OPENCLI_BROWSER_COMMAND_TIMEOUT ?? '60', 10);
 export const DEFAULT_BROWSER_EXPLORE_TIMEOUT = parseInt(process.env.OPENCLI_BROWSER_EXPLORE_TIMEOUT ?? '120', 10);
 export const DEFAULT_BROWSER_SMOKE_TIMEOUT = parseInt(process.env.OPENCLI_BROWSER_SMOKE_TIMEOUT ?? '60', 10);
 
@@ -32,7 +28,7 @@ export function withTimeoutMs<T>(promise: Promise<T>, timeoutMs: number, message
   });
 }
 
-/** Interface for browser factory (PlaywrightMCP or test mocks) */
+/** Interface for browser factory (BrowserBridge or test mocks) */
 export interface IBrowserFactory {
   connect(opts?: { timeout?: number }): Promise<IPage>;
   close(): Promise<void>;

package/src/setup.ts

@@ -8,7 +8,7 @@
 import chalk from 'chalk';
 import { checkDaemonStatus } from './browser/discover.js';
 import { checkConnectivity } from './doctor.js';
-import { PlaywrightMCP } from './browser/index.js';
+import { BrowserBridge } from './browser/index.js';
 
 export async function runSetup(opts: { cliVersion?: string; token?: string } = {}) {
   console.log();
@@ -27,7 +27,7 @@ export async function runSetup(opts: { cliVersion?: string; token?: string } = {
     console.log(chalk.dim('    Starting daemon now...'));
 
     // Try to spawn daemon
-    const mcp = new PlaywrightMCP();
+    const mcp = new BrowserBridge();
     try {
       await mcp.connect({ timeout: 5 });
       await mcp.close();

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

@@ -6,12 +6,49 @@
 import { describe, it, expect } from 'vitest';
 import { runCli, parseJsonOutput } from './helpers.js';
 
-function isExpectedXiaoyuzhouRestriction(code: number, stderr: string): boolean {
+function isExpectedChineseSiteRestriction(code: number, stderr: string): boolean {
   if (code === 0) return false;
   return /Error \[FETCH_ERROR\]: HTTP (403|429|451|503)\b/.test(stderr);
 }
 
+// Keep old name as alias for existing tests
+const isExpectedXiaoyuzhouRestriction = isExpectedChineseSiteRestriction;
+
 describe('public commands E2E', () => {
+  // ── apple-podcasts ──
+  it('apple-podcasts search returns structured podcast results', async () => {
+    const { stdout, code } = await runCli(['apple-podcasts', 'search', 'technology', '--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('id');
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('author');
+  }, 30_000);
+
+  it('apple-podcasts episodes returns episode list from a known show', async () => {
+    const { stdout, code } = await runCli(['apple-podcasts', 'episodes', '275699983', '--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('duration');
+    expect(data[0]).toHaveProperty('date');
+  }, 30_000);
+
+  it('apple-podcasts top returns ranked podcasts', async () => {
+    const { stdout, code } = await runCli(['apple-podcasts', 'top', '--limit', '3', '--country', 'us', '-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('rank');
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('id');
+  }, 30_000);
+
   // ── hackernews ──
   it('hackernews top returns structured data', async () => {
     const { stdout, code } = await runCli(['hackernews', 'top', '--limit', '3', '-f', 'json']);
@@ -115,4 +152,34 @@ describe('public commands E2E', () => {
     expect(code).not.toBe(0);
     expect(stderr).toMatch(/limit must be a positive integer|Argument "limit" must be a valid number/);
   }, 30_000);
+
+  // ── weread (Chinese site — may return empty on overseas CI runners) ──
+  it('weread search returns books', async () => {
+    const { stdout, stderr, code } = await runCli(['weread', 'search', 'python', '--limit', '3', '-f', 'json']);
+    if (isExpectedChineseSiteRestriction(code, stderr)) {
+      console.warn(`weread search skipped: ${stderr.trim()}`);
+      return;
+    }
+    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('bookId');
+  }, 30_000);
+
+  it('weread ranking returns books', async () => {
+    const { stdout, stderr, code } = await runCli(['weread', 'ranking', 'all', '--limit', '3', '-f', 'json']);
+    if (isExpectedChineseSiteRestriction(code, stderr)) {
+      console.warn(`weread ranking skipped: ${stderr.trim()}`);
+      return;
+    }
+    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('readingCount');
+    expect(data[0]).toHaveProperty('bookId');
+  }, 30_000);
 });

package/dist/clis/grok/debug.d.ts

@@ -1 +0,0 @@
-export declare const debugCommand: import("../../registry.js").CliCommand;

package/dist/clis/grok/debug.js

@@ -1,45 +0,0 @@
-import { cli, Strategy } from '../../registry.js';
-export const debugCommand = cli({
-    site: 'grok',
-    name: 'debug',
-    description: 'Debug grok page structure',
-    domain: 'grok.com',
-    strategy: Strategy.COOKIE,
-    browser: true,
-    columns: ['data'],
-    func: async (page, _kwargs) => {
-        await page.goto('https://grok.com');
-        await page.wait(3);
-        // Get all button-like elements near textarea
-        const debug = await page.evaluate(`(() => {
-      const ta = document.querySelector('textarea');
-      if (!ta) return { error: 'no textarea' };
-
-      // Get parent containers
-      let parent = ta.parentElement;
-      const parents = [];
-      for (let i = 0; i < 5 && parent; i++) {
-        parents.push({
-          tag: parent.tagName,
-          class: parent.className?.substring(0, 80),
-          childCount: parent.children.length,
-        });
-        parent = parent.parentElement;
-      }
-
-      // Find buttons in the form/container near textarea
-      const form = ta.closest('form') || ta.closest('[class*="composer"]') || ta.closest('[class*="input"]') || ta.parentElement?.parentElement;
-      const buttons = form ? [...form.querySelectorAll('button')].map(b => ({
-        testid: b.getAttribute('data-testid'),
-        type: b.type,
-        disabled: b.disabled,
-        text: (b.textContent || '').substring(0, 30),
-        html: b.outerHTML.substring(0, 200),
-        rect: b.getBoundingClientRect().toJSON(),
-      })) : [];
-
-      return { parents, buttons, formTag: form?.tagName, formClass: form?.className?.substring(0, 80) };
-    })()`);
-        return [{ data: JSON.stringify(debug, null, 2) }];
-    },
-});

package/src/clis/grok/debug.ts

@@ -1,49 +0,0 @@
-import { cli, Strategy } from '../../registry.js';
-import type { IPage } from '../../types.js';
-
-export const debugCommand = cli({
-  site: 'grok',
-  name: 'debug',
-  description: 'Debug grok page structure',
-  domain: 'grok.com',
-  strategy: Strategy.COOKIE,
-  browser: true,
-  columns: ['data'],
-  func: async (page: IPage, _kwargs: Record<string, any>) => {
-    await page.goto('https://grok.com');
-    await page.wait(3);
-
-    // Get all button-like elements near textarea
-    const debug = await page.evaluate(`(() => {
-      const ta = document.querySelector('textarea');
-      if (!ta) return { error: 'no textarea' };
-
-      // Get parent containers
-      let parent = ta.parentElement;
-      const parents = [];
-      for (let i = 0; i < 5 && parent; i++) {
-        parents.push({
-          tag: parent.tagName,
-          class: parent.className?.substring(0, 80),
-          childCount: parent.children.length,
-        });
-        parent = parent.parentElement;
-      }
-
-      // Find buttons in the form/container near textarea
-      const form = ta.closest('form') || ta.closest('[class*="composer"]') || ta.closest('[class*="input"]') || ta.parentElement?.parentElement;
-      const buttons = form ? [...form.querySelectorAll('button')].map(b => ({
-        testid: b.getAttribute('data-testid'),
-        type: b.type,
-        disabled: b.disabled,
-        text: (b.textContent || '').substring(0, 30),
-        html: b.outerHTML.substring(0, 200),
-        rect: b.getBoundingClientRect().toJSON(),
-      })) : [];
-
-      return { parents, buttons, formTag: form?.tagName, formClass: form?.className?.substring(0, 80) };
-    })()`);
-
-    return [{ data: JSON.stringify(debug, null, 2) }];
-  },
-});