@jackwener/opencli 0.7.11 → 0.9.0

package/src/pipeline/registry.ts

@@ -0,0 +1,60 @@
+/**
+ * Dynamic registry for pipeline steps.
+ * Allows core and third-party plugins to register custom YAML operations.
+ */
+
+import type { IPage } from '../types.js';
+
+// Import core steps
+import { stepNavigate, stepClick, stepType, stepWait, stepPress, stepSnapshot, stepEvaluate } from './steps/browser.js';
+import { stepFetch } from './steps/fetch.js';
+import { stepSelect, stepMap, stepFilter, stepSort, stepLimit } from './steps/transform.js';
+import { stepIntercept } from './steps/intercept.js';
+import { stepTap } from './steps/tap.js';
+
+/** 
+ * Step handler: all pipeline steps conform to this generic interface.
+ * TData is the type of the `data` state flowing into the step.
+ * TResult is the expected return type.
+ */
+export type StepHandler<TData = any, TResult = any> = (
+  page: IPage | null,
+  params: any,
+  data: TData,
+  args: Record<string, any>
+) => Promise<TResult>;
+
+const _stepRegistry = new Map<string, StepHandler>();
+
+/**
+ * Get a registered step handler by name.
+ */
+export function getStep(name: string): StepHandler | undefined {
+  return _stepRegistry.get(name);
+}
+
+/**
+ * Register a new custom step handler for the YAML pipeline.
+ */
+export function registerStep(name: string, handler: StepHandler): void {
+  _stepRegistry.set(name, handler);
+}
+
+// -------------------------------------------------------------
+// Auto-Register Core Steps
+// -------------------------------------------------------------
+registerStep('navigate', stepNavigate);
+registerStep('fetch', stepFetch);
+registerStep('select', stepSelect);
+registerStep('evaluate', stepEvaluate);
+registerStep('snapshot', stepSnapshot);
+registerStep('click', stepClick);
+registerStep('type', stepType);
+registerStep('wait', stepWait);
+registerStep('press', stepPress);
+registerStep('map', stepMap);
+registerStep('filter', stepFilter);
+registerStep('sort', stepSort);
+registerStep('limit', stepLimit);
+registerStep('intercept', stepIntercept);
+registerStep('tap', stepTap);

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

@@ -6,6 +6,11 @@
 import { describe, it, expect } from 'vitest';
 import { runCli, parseJsonOutput } from './helpers.js';
 
+function isExpectedXiaoyuzhouRestriction(code: number, stderr: string): boolean {
+  if (code === 0) return false;
+  return /Error \[FETCH_ERROR\]: HTTP (403|429|451|503)\b/.test(stderr);
+}
+
 describe('public commands E2E', () => {
   // ── hackernews ──
   it('hackernews top returns structured data', async () => {
@@ -53,4 +58,61 @@ describe('public commands E2E', () => {
       expect(data).toBeDefined();
     }
   }, 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']);
+    if (isExpectedXiaoyuzhouRestriction(code, stderr)) {
+      console.warn(`xiaoyuzhou podcast skipped: ${stderr.trim()}`);
+      return;
+    }
+    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('subscribers');
+    expect(data[0]).toHaveProperty('episodes');
+  }, 30_000);
+
+  it('xiaoyuzhou podcast-episodes returns episode list', async () => {
+    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;
+    }
+    expect(code).toBe(0);
+    const data = parseJsonOutput(stdout);
+    expect(Array.isArray(data)).toBe(true);
+    expect(data.length).toBeGreaterThanOrEqual(1);
+    expect(data[0]).toHaveProperty('eid');
+    expect(data[0]).toHaveProperty('title');
+    expect(data[0]).toHaveProperty('duration');
+  }, 30_000);
+
+  it('xiaoyuzhou episode returns episode detail', async () => {
+    const { stdout, stderr, code } = await runCli(['xiaoyuzhou', 'episode', '69b3b675772ac2295bfc01d0', '-f', 'json']);
+    if (isExpectedXiaoyuzhouRestriction(code, stderr)) {
+      console.warn(`xiaoyuzhou episode skipped: ${stderr.trim()}`);
+      return;
+    }
+    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('podcast');
+    expect(data[0]).toHaveProperty('plays');
+    expect(data[0]).toHaveProperty('comments');
+  }, 30_000);
+
+  it('xiaoyuzhou podcast-episodes rejects invalid limit', async () => {
+    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;
+    }
+    expect(code).not.toBe(0);
+    expect(stderr).toMatch(/limit must be a positive integer|Argument "limit" must be a valid number/);
+  }, 30_000);
 });