@jackwener/opencli 1.7.4 → 1.7.6

package/dist/src/commanderAdapter.d.ts

@@ -11,7 +11,6 @@
  */
 import { Command } from 'commander';
 import { type CliCommand } from './registry.js';
-export declare function normalizeArgValue(argType: string | undefined, value: unknown, name: string): unknown;
 /**
  * Register a single CliCommand as a Commander subcommand.
  */

package/dist/src/commanderAdapter.js

@@ -15,22 +15,8 @@ import { fullName, getRegistry } from './registry.js';
 import { formatRegistryHelpText } from './serialization.js';
 import { render as renderOutput } from './output.js';
 import { executeCommand, prepareCommandArgs } from './execution.js';
-import { CliError, EXIT_CODES, ArgumentError, toEnvelope, } from './errors.js';
+import { CliError, EXIT_CODES, toEnvelope, } from './errors.js';
 import { isDiagnosticEnabled } from './diagnostic.js';
-export function normalizeArgValue(argType, value, name) {
-    if (argType !== 'bool' && argType !== 'boolean')
-        return value;
-    if (typeof value === 'boolean')
-        return value;
-    if (value == null || value === '')
-        return false;
-    const normalized = String(value).trim().toLowerCase();
-    if (normalized === 'true')
-        return true;
-    if (normalized === 'false')
-        return false;
-    throw new ArgumentError(`"${name}" must be either "true" or "false".`);
-}
 /**
  * Register a single CliCommand as a Commander subcommand.
  */
@@ -83,7 +69,7 @@ export function registerCommandToProgram(siteCmd, cmd) {
                 const camelName = arg.name.replace(/-([a-z])/g, (_m, ch) => ch.toUpperCase());
                 const v = optionsRecord[arg.name] ?? optionsRecord[camelName];
                 if (v !== undefined)
-                    rawKwargs[arg.name] = normalizeArgValue(arg.type, v, arg.name);
+                    rawKwargs[arg.name] = v;
             }
             const kwargs = prepareCommandArgs(cmd, rawKwargs);
             const verbose = optionsRecord.verbose === true;

package/dist/src/commanderAdapter.test.js

@@ -61,7 +61,7 @@ describe('commanderAdapter arg passing', () => {
         const siteCmd = program.command('paperreview');
         registerCommandToProgram(siteCmd, cmd);
         await program.parseAsync(['node', 'opencli', 'paperreview', 'submit', './paper.pdf', '--dry-run', 'maybe']);
-        // normalizeArgValue validates bools eagerly; executeCommand should not be reached
+        // prepareCommandArgs validates bools before dispatch; executeCommand should not be reached
         expect(mockExecuteCommand).not.toHaveBeenCalled();
     });
 });

package/dist/src/completion-shared.js

@@ -11,11 +11,8 @@ export const BUILTIN_COMMANDS = [
     'list',
     'validate',
     'verify',
-    'explore',
-    'probe', // alias for explore
-    'synthesize',
-    'generate',
-    'cascade',
+    'browser',
+    'tab',
     'doctor',
     'plugin',
     'install',

package/dist/src/daemon.js

@@ -154,6 +154,14 @@ async function handleRequest(req, res) {
             const timeoutMs = typeof body.timeout === 'number' && body.timeout > 0
                 ? body.timeout * 1000
                 : 120000;
+            if (pending.has(body.id)) {
+                jsonResponse(res, 409, {
+                    id: body.id,
+                    ok: false,
+                    error: 'Duplicate command id already pending; retry',
+                });
+                return;
+            }
             const result = await new Promise((resolve, reject) => {
                 const timer = setTimeout(() => {
                     pending.delete(body.id);

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

@@ -44,6 +44,7 @@ export interface ArticleDownloadResult {
     publish_time: string;
     status: string;
     size: string;
+    saved: string;
 }
 /**
  * Download an article to Markdown with optional image localization.

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

@@ -129,6 +129,7 @@ export async function downloadArticle(data, options) {
                 publish_time: '-',
                 status: 'failed — no title',
                 size: '-',
+                saved: '-',
             }];
     }
     if (!data.contentHtml) {
@@ -138,6 +139,7 @@ export async function downloadArticle(data, options) {
                 publish_time: data.publishTime || '-',
                 status: 'failed — no content',
                 size: '-',
+                saved: '-',
             }];
     }
     // Convert HTML to Markdown
@@ -174,5 +176,6 @@ export async function downloadArticle(data, options) {
             publish_time: data.publishTime || '-',
             status: 'success',
             size: formatBytes(size),
+            saved: filePath,
         }];
 }

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

@@ -0,0 +1 @@
+export {};

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

@@ -0,0 +1,39 @@
+import * as fs from 'node:fs';
+import * as os from 'node:os';
+import * as path from 'node:path';
+import { afterEach, describe, expect, it } from 'vitest';
+import { downloadArticle } from './article-download.js';
+const tempDirs = [];
+afterEach(() => {
+    for (const dir of tempDirs) {
+        try {
+            fs.rmSync(dir, { recursive: true, force: true });
+        }
+        catch {
+            // Ignore cleanup errors in tests.
+        }
+    }
+    tempDirs.length = 0;
+});
+describe('downloadArticle', () => {
+    it('returns the saved markdown file path on success', async () => {
+        const tempDir = await fs.promises.mkdtemp(path.join(os.tmpdir(), 'opencli-article-'));
+        tempDirs.push(tempDir);
+        const result = await downloadArticle({
+            title: 'Test Article',
+            author: 'Author',
+            publishTime: '2026-04-20 12:00:00',
+            sourceUrl: 'https://example.com/article',
+            contentHtml: '<p>Hello world</p>',
+        }, {
+            output: tempDir,
+            downloadImages: false,
+        });
+        expect(result).toHaveLength(1);
+        expect(result[0].status).toBe('success');
+        expect(result[0].saved).toMatch(new RegExp(`^${tempDir.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}`));
+        expect(path.extname(result[0].saved)).toBe('.md');
+        expect(fs.existsSync(result[0].saved)).toBe(true);
+        expect(fs.readFileSync(result[0].saved, 'utf8')).toContain('Hello world');
+    });
+});

package/dist/src/execution.js

@@ -185,6 +185,9 @@ export async function executeCommand(cmd, rawKwargs, debug = false, opts = {}) {
                         throw new CommandExecutionError(`Pre-navigation to ${preNavUrl} failed: ${err instanceof Error ? err.message : err}`, 'Check that the site is reachable and the browser extension is running.');
                     }
                 }
+                // --live / OPENCLI_LIVE=1 keeps the automation window open after the
+                // command finishes, so agents (or humans) can inspect the page state.
+                const keepOpen = process.env.OPENCLI_LIVE === '1' || process.env.OPENCLI_LIVE === 'true';
                 try {
                     const result = await runWithTimeout(runCommand(cmd, page, kwargs, debug), {
                         timeout: cmd.timeoutSeconds ?? DEFAULT_BROWSER_COMMAND_TIMEOUT,
@@ -192,7 +195,8 @@ export async function executeCommand(cmd, rawKwargs, debug = false, opts = {}) {
                     });
                     // Adapter commands are one-shot — close the automation window immediately
                     // instead of waiting for the 30s idle timeout.
-                    await page.closeWindow?.().catch(() => { });
+                    if (!keepOpen)
+                        await page.closeWindow?.().catch(() => { });
                     return result;
                 }
                 catch (err) {
@@ -206,7 +210,8 @@ export async function executeCommand(cmd, rawKwargs, debug = false, opts = {}) {
                     // Close the automation window on failure too — without this, the window
                     // lingers until the extension's idle timer fires (unreliable on Windows
                     // where MV3 service workers may be suspended before setTimeout triggers).
-                    await page.closeWindow?.().catch(() => { });
+                    if (!keepOpen)
+                        await page.closeWindow?.().catch(() => { });
                     throw err;
                 }
             }, { workspace: `site:${cmd.site}`, cdpEndpoint });

package/dist/src/execution.test.js

@@ -60,6 +60,60 @@ describe('executeCommand — non-browser timeout', () => {
         expect(closeWindow).toHaveBeenCalledTimes(1);
         vi.restoreAllMocks();
     });
+    it('skips closeWindow when OPENCLI_LIVE=1 (success path)', async () => {
+        const closeWindow = vi.fn().mockResolvedValue(undefined);
+        const mockPage = { closeWindow };
+        vi.spyOn(capRouting, 'shouldUseBrowserSession').mockReturnValue(true);
+        vi.spyOn(runtime, 'browserSession').mockImplementation(async (_Factory, fn) => fn(mockPage));
+        const prev = process.env.OPENCLI_LIVE;
+        process.env.OPENCLI_LIVE = '1';
+        try {
+            const cmd = cli({
+                site: 'test-execution',
+                name: 'browser-live-success',
+                description: 'test closeWindow skipped with --live on success',
+                browser: true,
+                strategy: Strategy.PUBLIC,
+                func: async () => [{ ok: true }],
+            });
+            await executeCommand(cmd, {});
+            expect(closeWindow).not.toHaveBeenCalled();
+        }
+        finally {
+            if (prev === undefined)
+                delete process.env.OPENCLI_LIVE;
+            else
+                process.env.OPENCLI_LIVE = prev;
+            vi.restoreAllMocks();
+        }
+    });
+    it('skips closeWindow when OPENCLI_LIVE=1 (failure path)', async () => {
+        const closeWindow = vi.fn().mockResolvedValue(undefined);
+        const mockPage = { closeWindow };
+        vi.spyOn(capRouting, 'shouldUseBrowserSession').mockReturnValue(true);
+        vi.spyOn(runtime, 'browserSession').mockImplementation(async (_Factory, fn) => fn(mockPage));
+        const prev = process.env.OPENCLI_LIVE;
+        process.env.OPENCLI_LIVE = '1';
+        try {
+            const cmd = cli({
+                site: 'test-execution',
+                name: 'browser-live-failure',
+                description: 'test closeWindow skipped with --live on failure',
+                browser: true,
+                strategy: Strategy.PUBLIC,
+                func: async () => { throw new Error('adapter failure'); },
+            });
+            await expect(executeCommand(cmd, {})).rejects.toThrow('adapter failure');
+            expect(closeWindow).not.toHaveBeenCalled();
+        }
+        finally {
+            if (prev === undefined)
+                delete process.env.OPENCLI_LIVE;
+            else
+                process.env.OPENCLI_LIVE = prev;
+            vi.restoreAllMocks();
+        }
+    });
     it('does not re-run custom validation when args are already prepared', async () => {
         const validateArgs = vi.fn();
         const cmd = {

package/dist/src/main.js

@@ -26,6 +26,22 @@ const __dirname = path.dirname(__filename);
 // Use findPackageRoot so the path works both in dev (src/main.ts) and prod (dist/src/main.js).
 const BUILTIN_CLIS = path.join(findPackageRoot(__filename), 'clis');
 const USER_CLIS = path.join(os.homedir(), '.opencli', 'clis');
+// ── Session lifecycle flags ──────────────────────────────────────────────
+// `--live` / `--focus` are top-level-ish toggles that tweak the automation
+// window's lifecycle. We strip them from argv before Commander runs so they
+// can be placed anywhere and work on any subcommand (adapter or browser).
+{
+    const liveIdx = process.argv.indexOf('--live');
+    if (liveIdx !== -1) {
+        process.env.OPENCLI_LIVE = '1';
+        process.argv.splice(liveIdx, 1);
+    }
+    const focusIdx = process.argv.indexOf('--focus');
+    if (focusIdx !== -1) {
+        process.env.OPENCLI_WINDOW_FOCUSED = '1';
+        process.argv.splice(focusIdx, 1);
+    }
+}
 // ── Ultra-fast path: lightweight commands bypass full discovery ──────────
 // These are high-frequency or trivial paths that must not pay the startup tax.
 const argv = process.argv.slice(2);

package/dist/src/plugin.d.ts

@@ -60,13 +60,6 @@ declare function resolveStoredPluginSource(lockEntry: LockEntry | undefined, plu
  */
 type MoveDirFsOps = Pick<typeof fs, 'renameSync' | 'cpSync' | 'rmSync'>;
 declare function moveDir(src: string, dest: string, fsOps?: MoveDirFsOps): void;
-type PromoteDirFsOps = MoveDirFsOps & Pick<typeof fs, 'existsSync' | 'mkdirSync'>;
-/**
- * Promote a prepared staging directory into its final location.
- * The final path is only exposed after the directory has been fully prepared.
- */
-declare function promoteDir(stagingDir: string, dest: string, fsOps?: PromoteDirFsOps): void;
-declare function replaceDir(stagingDir: string, dest: string, fsOps?: PromoteDirFsOps): void;
 export interface ValidationResult {
     valid: boolean;
     errors: string[];
@@ -149,4 +142,4 @@ declare function parseSource(source: string): ParsedSource | null;
  */
 export declare function resolveEsbuildBin(): string | null;
 declare function resolveHostOpencliRoot(startFile?: string): string;
-export { resolveHostOpencliRoot as _resolveHostOpencliRoot, resolveEsbuildBin as _resolveEsbuildBin, getCommitHash as _getCommitHash, installDependencies as _installDependencies, parseSource as _parseSource, postInstallMonorepoLifecycle as _postInstallMonorepoLifecycle, readLockFile as _readLockFile, readLockFileWithWriter as _readLockFileWithWriter, updateAllPlugins as _updateAllPlugins, validatePluginStructure as _validatePluginStructure, writeLockFile as _writeLockFile, writeLockFileWithFs as _writeLockFileWithFs, isSymlinkSync as _isSymlinkSync, getMonoreposDir as _getMonoreposDir, installLocalPlugin as _installLocalPlugin, isLocalPluginSource as _isLocalPluginSource, moveDir as _moveDir, promoteDir as _promoteDir, replaceDir as _replaceDir, resolvePluginSource as _resolvePluginSource, resolveStoredPluginSource as _resolveStoredPluginSource, toStoredPluginSource as _toStoredPluginSource, toLocalPluginSource as _toLocalPluginSource, };
+export { resolveHostOpencliRoot as _resolveHostOpencliRoot, resolveEsbuildBin as _resolveEsbuildBin, getCommitHash as _getCommitHash, installDependencies as _installDependencies, parseSource as _parseSource, postInstallMonorepoLifecycle as _postInstallMonorepoLifecycle, readLockFile as _readLockFile, readLockFileWithWriter as _readLockFileWithWriter, updateAllPlugins as _updateAllPlugins, validatePluginStructure as _validatePluginStructure, writeLockFile as _writeLockFile, writeLockFileWithFs as _writeLockFileWithFs, isSymlinkSync as _isSymlinkSync, getMonoreposDir as _getMonoreposDir, installLocalPlugin as _installLocalPlugin, isLocalPluginSource as _isLocalPluginSource, moveDir as _moveDir, resolvePluginSource as _resolvePluginSource, resolveStoredPluginSource as _resolveStoredPluginSource, toStoredPluginSource as _toStoredPluginSource, toLocalPluginSource as _toLocalPluginSource, };

package/dist/src/plugin.js

@@ -159,32 +159,6 @@ function createSiblingTempPath(dest, kind) {
     const suffix = `${process.pid}-${Date.now()}-${Math.random().toString(16).slice(2)}`;
     return path.join(path.dirname(dest), `.${path.basename(dest)}.${kind}-${suffix}`);
 }
-/**
- * Promote a prepared staging directory into its final location.
- * The final path is only exposed after the directory has been fully prepared.
- */
-function promoteDir(stagingDir, dest, fsOps = fs) {
-    if (fsOps.existsSync(dest)) {
-        throw new PluginError(`Destination already exists: ${dest}`);
-    }
-    fsOps.mkdirSync(path.dirname(dest), { recursive: true });
-    const tempDest = createSiblingTempPath(dest, 'tmp');
-    try {
-        moveDir(stagingDir, tempDest, fsOps);
-        fsOps.renameSync(tempDest, dest);
-    }
-    catch (err) {
-        try {
-            fsOps.rmSync(tempDest, { recursive: true, force: true });
-        }
-        catch { }
-        throw err;
-    }
-}
-function replaceDir(stagingDir, dest, fsOps = fs) {
-    const replacement = beginReplaceDir(stagingDir, dest, fsOps);
-    replacement.finalize();
-}
 function cloneRepoToTemp(cloneUrl) {
     const tmpCloneDir = path.join(os.tmpdir(), `opencli-clone-${process.pid}-${Date.now()}-${Math.random().toString(16).slice(2)}`);
     try {
@@ -1268,4 +1242,4 @@ function transpilePluginTs(pluginDir) {
         log.warn(`TS transpilation setup failed: ${getErrorMessage(err)}`);
     }
 }
-export { resolveHostOpencliRoot as _resolveHostOpencliRoot, resolveEsbuildBin as _resolveEsbuildBin, getCommitHash as _getCommitHash, installDependencies as _installDependencies, parseSource as _parseSource, postInstallMonorepoLifecycle as _postInstallMonorepoLifecycle, readLockFile as _readLockFile, readLockFileWithWriter as _readLockFileWithWriter, updateAllPlugins as _updateAllPlugins, validatePluginStructure as _validatePluginStructure, writeLockFile as _writeLockFile, writeLockFileWithFs as _writeLockFileWithFs, isSymlinkSync as _isSymlinkSync, getMonoreposDir as _getMonoreposDir, installLocalPlugin as _installLocalPlugin, isLocalPluginSource as _isLocalPluginSource, moveDir as _moveDir, promoteDir as _promoteDir, replaceDir as _replaceDir, resolvePluginSource as _resolvePluginSource, resolveStoredPluginSource as _resolveStoredPluginSource, toStoredPluginSource as _toStoredPluginSource, toLocalPluginSource as _toLocalPluginSource, };
+export { resolveHostOpencliRoot as _resolveHostOpencliRoot, resolveEsbuildBin as _resolveEsbuildBin, getCommitHash as _getCommitHash, installDependencies as _installDependencies, parseSource as _parseSource, postInstallMonorepoLifecycle as _postInstallMonorepoLifecycle, readLockFile as _readLockFile, readLockFileWithWriter as _readLockFileWithWriter, updateAllPlugins as _updateAllPlugins, validatePluginStructure as _validatePluginStructure, writeLockFile as _writeLockFile, writeLockFileWithFs as _writeLockFileWithFs, isSymlinkSync as _isSymlinkSync, getMonoreposDir as _getMonoreposDir, installLocalPlugin as _installLocalPlugin, isLocalPluginSource as _isLocalPluginSource, moveDir as _moveDir, resolvePluginSource as _resolvePluginSource, resolveStoredPluginSource as _resolveStoredPluginSource, toStoredPluginSource as _toStoredPluginSource, toLocalPluginSource as _toLocalPluginSource, };

package/dist/src/plugin.test.js

@@ -12,7 +12,7 @@ const { mockExecFileSync, mockExecSync } = vi.hoisted(() => ({
     mockExecFileSync: vi.fn(),
     mockExecSync: vi.fn(),
 }));
-const { _getCommitHash, _installDependencies, _postInstallMonorepoLifecycle, _promoteDir, _replaceDir, installPlugin, listPlugins, _readLockFile, _readLockFileWithWriter, _resolveEsbuildBin, _resolveHostOpencliRoot, uninstallPlugin, updatePlugin, _parseSource, _updateAllPlugins, _validatePluginStructure, _writeLockFile, _writeLockFileWithFs, _isSymlinkSync, _getMonoreposDir, getLockFilePath, _installLocalPlugin, _isLocalPluginSource, _moveDir, _resolvePluginSource, _resolveStoredPluginSource, _toStoredPluginSource, _toLocalPluginSource, } = pluginModule;
+const { _getCommitHash, _installDependencies, _postInstallMonorepoLifecycle, installPlugin, listPlugins, _readLockFile, _readLockFileWithWriter, _resolveEsbuildBin, _resolveHostOpencliRoot, uninstallPlugin, updatePlugin, _parseSource, _updateAllPlugins, _validatePluginStructure, _writeLockFile, _writeLockFileWithFs, _isSymlinkSync, _getMonoreposDir, getLockFilePath, _installLocalPlugin, _isLocalPluginSource, _moveDir, _resolvePluginSource, _resolveStoredPluginSource, _toStoredPluginSource, _toLocalPluginSource, } = pluginModule;
 describe('parseSource', () => {
     it('parses github:user/repo format', () => {
         const result = _parseSource('github:ByteYue/opencli-plugin-github-trending');
@@ -924,64 +924,6 @@ describe('moveDir', () => {
         expect(rmSync).toHaveBeenCalledWith(dest, { recursive: true, force: true });
     });
 });
-describe('promoteDir', () => {
-    it('cleans up temporary publish dir when final rename fails', () => {
-        const staging = path.join(os.tmpdir(), 'opencli-promote-stage');
-        const dest = path.join(os.tmpdir(), 'opencli-promote-dest');
-        const publishErr = new Error('publish failed');
-        const existsSync = vi.fn(() => false);
-        const mkdirSync = vi.fn(() => undefined);
-        const cpSync = vi.fn(() => undefined);
-        const rmSync = vi.fn(() => undefined);
-        const renameSync = vi.fn((src, _target) => {
-            if (String(src) === staging)
-                return;
-            throw publishErr;
-        });
-        expect(() => _promoteDir(staging, dest, { existsSync, mkdirSync, renameSync, cpSync, rmSync })).toThrow(publishErr);
-        const tempDest = renameSync.mock.calls[0][1];
-        expect(renameSync).toHaveBeenNthCalledWith(1, staging, tempDest);
-        expect(renameSync).toHaveBeenNthCalledWith(2, tempDest, dest);
-        expect(rmSync).toHaveBeenCalledWith(tempDest, { recursive: true, force: true });
-    });
-});
-describe('replaceDir', () => {
-    it('rolls back the original destination when swap fails', () => {
-        const staging = path.join(os.tmpdir(), 'opencli-replace-stage');
-        const dest = path.join(os.tmpdir(), 'opencli-replace-dest');
-        const publishErr = new Error('swap failed');
-        const existingPaths = new Set([dest]);
-        const existsSync = vi.fn((p) => existingPaths.has(String(p)));
-        const mkdirSync = vi.fn(() => undefined);
-        const cpSync = vi.fn(() => undefined);
-        const rmSync = vi.fn(() => undefined);
-        const renameSync = vi.fn((src, target) => {
-            if (String(src) === staging) {
-                existingPaths.add(String(target));
-                return;
-            }
-            if (String(src) === dest) {
-                existingPaths.delete(dest);
-                existingPaths.add(String(target));
-                return;
-            }
-            if (String(target) === dest)
-                throw publishErr;
-            if (existingPaths.has(String(src))) {
-                existingPaths.delete(String(src));
-                existingPaths.add(String(target));
-            }
-        });
-        expect(() => _replaceDir(staging, dest, { existsSync, mkdirSync, renameSync, cpSync, rmSync })).toThrow(publishErr);
-        const tempDest = renameSync.mock.calls[0][1];
-        const backupDest = renameSync.mock.calls[1][1];
-        expect(renameSync).toHaveBeenNthCalledWith(1, staging, tempDest);
-        expect(renameSync).toHaveBeenNthCalledWith(2, dest, backupDest);
-        expect(renameSync).toHaveBeenNthCalledWith(3, tempDest, dest);
-        expect(renameSync).toHaveBeenNthCalledWith(4, backupDest, dest);
-        expect(rmSync).toHaveBeenCalledWith(tempDest, { recursive: true, force: true });
-    });
-});
 describe('installPlugin transactional staging', () => {
     const standaloneSource = 'github:user/opencli-plugin-__test-transactional-standalone__';
     const standaloneName = '__test-transactional-standalone__';

package/dist/src/registry.d.ts

@@ -4,6 +4,7 @@
 import type { IPage } from './types.js';
 export declare enum Strategy {
     PUBLIC = "public",
+    LOCAL = "local",
     COOKIE = "cookie",
     HEADER = "header",
     INTERCEPT = "intercept",

package/dist/src/registry.js

@@ -4,6 +4,7 @@
 export var Strategy;
 (function (Strategy) {
     Strategy["PUBLIC"] = "public";
+    Strategy["LOCAL"] = "local";
     Strategy["COOKIE"] = "cookie";
     Strategy["HEADER"] = "header";
     Strategy["INTERCEPT"] = "intercept";
@@ -58,13 +59,13 @@ export function strategyLabel(cmd) {
  */
 function normalizeCommand(cmd) {
     const strategy = cmd.strategy ?? (cmd.browser === false ? Strategy.PUBLIC : Strategy.COOKIE);
-    const browser = cmd.browser ?? (strategy !== Strategy.PUBLIC);
+    const browser = cmd.browser ?? (strategy !== Strategy.PUBLIC && strategy !== Strategy.LOCAL);
     let navigateBefore = cmd.navigateBefore;
     if (navigateBefore === undefined) {
         if ((strategy === Strategy.COOKIE || strategy === Strategy.HEADER) && cmd.domain) {
             navigateBefore = `https://${cmd.domain}`;
         }
-        else if (strategy !== Strategy.PUBLIC) {
+        else if (strategy !== Strategy.PUBLIC && strategy !== Strategy.LOCAL) {
             // Non-PUBLIC without domain: needs authenticated browser context
             // but no specific pre-navigation URL. `true` signals this to
             // shouldUseBrowserSession without triggering resolvePreNav.

package/dist/src/registry.test.js

@@ -43,6 +43,17 @@ describe('cli() registration', () => {
         });
         expect(cmd.strategy).toBe(Strategy.PUBLIC);
     });
+    it('preserves LOCAL strategy on registration', () => {
+        const cmd = cli({
+            site: 'test-registry',
+            name: 'local-strategy',
+            description: 'reads local credentials',
+            strategy: Strategy.LOCAL,
+            browser: false,
+        });
+        expect(cmd.strategy).toBe(Strategy.LOCAL);
+        expect(cmd.browser).toBe(false);
+    });
     it('overwrites existing command on re-registration', () => {
         cli({ site: 'test-registry', name: 'overwrite', description: 'v1' });
         cli({ site: 'test-registry', name: 'overwrite', description: 'v2' });
@@ -148,6 +159,17 @@ describe('normalizeCommand (via registerCommand)', () => {
         expect(cmd.browser).toBe(false);
         expect(cmd.navigateBefore).toBeUndefined();
     });
+    it('LOCAL → browser false, navigateBefore undefined', () => {
+        registerCommand({
+            site: 'test-norm', name: 'local', description: '', args: [],
+            strategy: Strategy.LOCAL,
+        });
+        const cmd = getRegistry().get('test-norm/local');
+        expect(cmd.strategy).toBe(Strategy.LOCAL);
+        expect(strategyLabel(cmd)).toBe('local');
+        expect(cmd.browser).toBe(false);
+        expect(cmd.navigateBefore).toBeUndefined();
+    });
     it('explicit navigateBefore: false overrides COOKIE + domain', () => {
         registerCommand({
             site: 'test-norm', name: 'cookie-override', description: '', args: [],

package/dist/src/types.d.ts

@@ -51,16 +51,31 @@ export interface IPage {
         url?: string;
     }): Promise<BrowserCookie[]>;
     snapshot(opts?: SnapshotOptions): Promise<any>;
-    click(ref: string): Promise<void>;
-    typeText(ref: string, text: string): Promise<void>;
+    click(ref: string, opts?: {
+        nth?: number;
+        firstOnMulti?: boolean;
+    }): Promise<{
+        matches_n: number;
+        match_level: 'exact' | 'stable' | 'reidentified';
+    }>;
+    typeText(ref: string, text: string, opts?: {
+        nth?: number;
+        firstOnMulti?: boolean;
+    }): Promise<{
+        matches_n: number;
+        match_level: 'exact' | 'stable' | 'reidentified';
+    }>;
     pressKey(key: string): Promise<void>;
-    scrollTo(ref: string): Promise<any>;
+    scrollTo(ref: string, opts?: {
+        nth?: number;
+        firstOnMulti?: boolean;
+    }): Promise<any>;
     getFormState(): Promise<any>;
     wait(options: number | WaitOptions): Promise<void>;
     tabs(): Promise<any>;
-    closeTab?(index?: number): Promise<void>;
-    newTab?(): Promise<void>;
-    selectTab(index: number): Promise<void>;
+    closeTab?(target?: number | string): Promise<void>;
+    newTab?(url?: string): Promise<string | undefined>;
+    selectTab(target: number | string): Promise<void>;
     networkRequests(includeStatic?: boolean): Promise<any>;
     consoleMessages(level?: string): Promise<any>;
     scroll(direction?: string, amount?: number): Promise<void>;
@@ -89,10 +104,19 @@ export interface IPage {
     getCurrentUrl?(): Promise<string | null>;
     /** Returns the active page identity (targetId), or undefined if not yet resolved. */
     getActivePage?(): string | undefined;
-    /** @deprecated Use getActivePage() instead */
-    getActiveTabId?(): number | undefined;
+    /** Bind the page object to a specific page identity (targetId). */
+    setActivePage?(page?: string): void;
     /** Send a raw CDP command via chrome.debugger passthrough. */
     cdp?(method: string, params?: Record<string, unknown>): Promise<unknown>;
+    /** List cross-origin iframe targets in snapshot order. */
+    frames?(): Promise<Array<{
+        index: number;
+        frameId: string;
+        url: string;
+        name: string;
+    }>>;
+    /** Evaluate JavaScript inside a cross-origin iframe identified by its frame index. */
+    evaluateInFrame?(js: string, frameIndex: number): Promise<unknown>;
     /** Click at native coordinates via CDP Input.dispatchMouseEvent. */
     nativeClick?(x: number, y: number): Promise<void>;
     /** Type text via CDP Input.insertText. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jackwener/opencli",
-  "version": "1.7.4",
+  "version": "1.7.6",
   "publishConfig": {
     "access": "public"
   },

package/clis/twitter/lists-parser.js

@@ -1,77 +0,0 @@
-const MEMBER_PATTERNS = [
-    /([\d.,]+(?:\s?[KMB千萬万亿])?)\s*members?/i,
-    /([\d.,]+(?:\s?[KMB千萬万亿])?)\s*位成员/,
-];
-const FOLLOWER_PATTERNS = [
-    /([\d.,]+(?:\s?[KMB千萬万亿])?)\s*followers?/i,
-    /([\d.,]+(?:\s?[KMB千萬万亿])?)\s*位关注者/,
-];
-const PRIVATE_PATTERNS = [/\bprivate\b/i, /锁定列表/];
-const EMPTY_STATE_PATTERNS = [
-    /hasn't created any lists/i,
-    /has not created any lists/i,
-    /no lists yet/i,
-    /没有创建任何列表/,
-    /还没有创建任何列表/,
-];
-function normalizeText(text) {
-    return String(text || '').replace(/\s+/g, ' ').trim();
-}
-function matchMetric(text, patterns) {
-    for (const pattern of patterns) {
-        const match = text.match(pattern);
-        if (match)
-            return normalizeText(match[1]);
-    }
-    return '0';
-}
-function looksLikeMetadata(line) {
-    const text = normalizeText(line);
-    if (!text)
-        return true;
-    if (text.startsWith('@'))
-        return true;
-    if (MEMBER_PATTERNS.some((pattern) => pattern.test(text)))
-        return true;
-    if (FOLLOWER_PATTERNS.some((pattern) => pattern.test(text)))
-        return true;
-    if (PRIVATE_PATTERNS.some((pattern) => pattern.test(text)))
-        return true;
-    if (/^(public|pinned)$/i.test(text))
-        return true;
-    if (/^(lists?|你的列表)$/i.test(text))
-        return true;
-    return false;
-}
-export function parseListCards(cards) {
-    const seen = new Set();
-    const results = [];
-    for (const card of cards || []) {
-        const href = normalizeText(card?.href);
-        const rawText = String(card?.text || '');
-        if (!href || seen.has(href))
-            continue;
-        seen.add(href);
-        const text = normalizeText(rawText);
-        if (!text)
-            continue;
-        const lines = rawText
-            .split('\n')
-            .map((line) => normalizeText(line))
-            .filter(Boolean);
-        const name = lines.find((line) => !looksLikeMetadata(line));
-        if (!name)
-            continue;
-        results.push({
-            name,
-            members: matchMetric(text, MEMBER_PATTERNS),
-            followers: matchMetric(text, FOLLOWER_PATTERNS),
-            mode: PRIVATE_PATTERNS.some((pattern) => pattern.test(text)) ? 'private' : 'public',
-        });
-    }
-    return results;
-}
-export function isEmptyListsState(text) {
-    const normalized = normalizeText(text);
-    return EMPTY_STATE_PATTERNS.some((pattern) => pattern.test(normalized));
-}

package/clis/twitter/lists.d.ts

@@ -1,5 +0,0 @@
-import { Argument, Column } from '@jackwener/opencli/types';
-declare const args: Argument[];
-declare const columns: Column[];
-export { args, columns };
-export default {};