@jackwener/opencli 1.7.4 → 1.7.5

package/dist/src/cli.test.js

@@ -1,97 +1,22 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest';
+import * as fs from 'node:fs';
+import * as os from 'node:os';
 import * as path from 'node:path';
-const { mockExploreUrl, mockRenderExploreSummary, mockGenerateVerifiedFromUrl, mockRenderGenerateVerifiedSummary, mockRecordSession, mockRenderRecordSummary, mockCascadeProbe, mockRenderCascadeResult, mockGetBrowserFactory, mockBrowserSession, } = vi.hoisted(() => ({
-    mockExploreUrl: vi.fn(),
-    mockRenderExploreSummary: vi.fn(),
-    mockGenerateVerifiedFromUrl: vi.fn(),
-    mockRenderGenerateVerifiedSummary: vi.fn(),
-    mockRecordSession: vi.fn(),
-    mockRenderRecordSummary: vi.fn(),
-    mockCascadeProbe: vi.fn(),
-    mockRenderCascadeResult: vi.fn(),
-    mockGetBrowserFactory: vi.fn(() => ({ name: 'BrowserFactory' })),
-    mockBrowserSession: vi.fn(),
+const { mockBrowserConnect, mockBrowserClose, browserState, } = vi.hoisted(() => ({
+    mockBrowserConnect: vi.fn(),
+    mockBrowserClose: vi.fn(),
+    browserState: { page: null },
 }));
-vi.mock('./explore.js', () => ({
-    exploreUrl: mockExploreUrl,
-    renderExploreSummary: mockRenderExploreSummary,
-}));
-vi.mock('./generate-verified.js', () => ({
-    generateVerifiedFromUrl: mockGenerateVerifiedFromUrl,
-    renderGenerateVerifiedSummary: mockRenderGenerateVerifiedSummary,
-}));
-vi.mock('./record.js', () => ({
-    recordSession: mockRecordSession,
-    renderRecordSummary: mockRenderRecordSummary,
-}));
-vi.mock('./cascade.js', () => ({
-    cascadeProbe: mockCascadeProbe,
-    renderCascadeResult: mockRenderCascadeResult,
-}));
-vi.mock('./runtime.js', () => ({
-    getBrowserFactory: mockGetBrowserFactory,
-    browserSession: mockBrowserSession,
-}));
-import { createProgram, findPackageRoot, resolveBrowserVerifyInvocation } from './cli.js';
-describe('built-in browser commands verbose wiring', () => {
-    const consoleLogSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
-    beforeEach(() => {
-        delete process.env.OPENCLI_VERBOSE;
-        process.exitCode = undefined;
-        mockExploreUrl.mockReset().mockResolvedValue({ ok: true });
-        mockRenderExploreSummary.mockReset().mockReturnValue('explore-summary');
-        mockGenerateVerifiedFromUrl.mockReset().mockResolvedValue({ status: 'success' });
-        mockRenderGenerateVerifiedSummary.mockReset().mockReturnValue('generate-summary');
-        mockRecordSession.mockReset().mockResolvedValue({ candidateCount: 1 });
-        mockRenderRecordSummary.mockReset().mockReturnValue('record-summary');
-        mockCascadeProbe.mockReset().mockResolvedValue({ ok: true });
-        mockRenderCascadeResult.mockReset().mockReturnValue('cascade-summary');
-        mockGetBrowserFactory.mockClear();
-        mockBrowserSession.mockReset().mockImplementation(async (_factory, fn) => {
-            const page = {
-                goto: vi.fn(),
-                wait: vi.fn(),
-            };
-            return fn(page);
-        });
-    });
-    it('enables OPENCLI_VERBOSE for explore via the real CLI command', async () => {
-        const program = createProgram('', '');
-        await program.parseAsync(['node', 'opencli', 'explore', 'https://example.com', '-v']);
-        expect(process.env.OPENCLI_VERBOSE).toBe('1');
-        expect(mockExploreUrl).toHaveBeenCalledWith('https://example.com', expect.objectContaining({ workspace: 'explore:example.com' }));
-    });
-    it('enables OPENCLI_VERBOSE for generate via the real CLI command', async () => {
-        const program = createProgram('', '');
-        await program.parseAsync(['node', 'opencli', 'generate', 'https://example.com', '-v']);
-        expect(process.env.OPENCLI_VERBOSE).toBe('1');
-        expect(mockGenerateVerifiedFromUrl).toHaveBeenCalledWith(expect.objectContaining({ url: 'https://example.com', workspace: 'generate:example.com', noRegister: false }));
-    });
-    it('passes --no-register through the real CLI command', async () => {
-        const program = createProgram('', '');
-        await program.parseAsync(['node', 'opencli', 'generate', 'https://example.com', '--no-register']);
-        expect(mockGenerateVerifiedFromUrl).toHaveBeenCalledWith(expect.objectContaining({ url: 'https://example.com', workspace: 'generate:example.com', noRegister: true }));
-    });
-    it('enables OPENCLI_VERBOSE for record via the real CLI command', async () => {
-        const program = createProgram('', '');
-        await program.parseAsync(['node', 'opencli', 'record', 'https://example.com', '-v']);
-        expect(process.env.OPENCLI_VERBOSE).toBe('1');
-        expect(mockRecordSession).toHaveBeenCalledWith(expect.objectContaining({ url: 'https://example.com' }));
-    });
-    it('enables OPENCLI_VERBOSE for cascade via the real CLI command', async () => {
-        const program = createProgram('', '');
-        await program.parseAsync(['node', 'opencli', 'cascade', 'https://example.com', '-v']);
-        expect(process.env.OPENCLI_VERBOSE).toBe('1');
-        expect(mockBrowserSession).toHaveBeenCalled();
-        expect(mockCascadeProbe).toHaveBeenCalledWith(expect.any(Object), 'https://example.com');
-    });
-    it('leaves OPENCLI_VERBOSE unset when verbose is omitted', async () => {
-        const program = createProgram('', '');
-        await program.parseAsync(['node', 'opencli', 'explore', 'https://example.com']);
-        expect(process.env.OPENCLI_VERBOSE).toBeUndefined();
-    });
-    consoleLogSpy.mockClear();
+vi.mock('./browser/index.js', () => {
+    mockBrowserConnect.mockImplementation(async () => browserState.page);
+    return {
+        BrowserBridge: class {
+            connect = mockBrowserConnect;
+            close = mockBrowserClose;
+        },
+    };
 });
+import { createProgram, findPackageRoot, resolveBrowserVerifyInvocation } from './cli.js';
 describe('resolveBrowserVerifyInvocation', () => {
     it('prefers the built entry declared in package metadata', () => {
         const projectRoot = path.join('repo-root');
@@ -156,6 +81,158 @@ describe('resolveBrowserVerifyInvocation', () => {
         });
     });
 });
+describe('browser tab targeting commands', () => {
+    const consoleLogSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
+    const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
+    function getBrowserStateFile(cacheDir) {
+        return path.join(cacheDir, 'browser-state', 'browser_default.json');
+    }
+    beforeEach(() => {
+        process.exitCode = undefined;
+        process.env.OPENCLI_CACHE_DIR = fs.mkdtempSync(path.join(os.tmpdir(), 'opencli-browser-tab-state-'));
+        consoleLogSpy.mockClear();
+        stderrSpy.mockClear();
+        mockBrowserConnect.mockClear();
+        mockBrowserClose.mockReset().mockResolvedValue(undefined);
+        browserState.page = {
+            goto: vi.fn().mockResolvedValue(undefined),
+            wait: vi.fn().mockResolvedValue(undefined),
+            setActivePage: vi.fn(),
+            getActivePage: vi.fn().mockReturnValue('tab-1'),
+            getCurrentUrl: vi.fn().mockResolvedValue('https://one.example'),
+            startNetworkCapture: vi.fn().mockResolvedValue(true),
+            evaluate: vi.fn().mockResolvedValue({ ok: true }),
+            tabs: vi.fn().mockResolvedValue([
+                { index: 0, page: 'tab-1', url: 'https://one.example', title: 'one', active: true },
+                { index: 1, page: 'tab-2', url: 'https://two.example', title: 'two', active: false },
+            ]),
+            selectTab: vi.fn().mockResolvedValue(undefined),
+            newTab: vi.fn().mockResolvedValue('tab-3'),
+            closeTab: vi.fn().mockResolvedValue(undefined),
+            frames: vi.fn().mockResolvedValue([
+                { index: 0, frameId: 'frame-1', url: 'https://x.example/embed', name: 'x-embed' },
+            ]),
+            evaluateInFrame: vi.fn().mockResolvedValue('inside frame'),
+            readNetworkCapture: vi.fn().mockResolvedValue([]),
+        };
+    });
+    it('binds browser commands to an explicit target tab via --tab', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'eval', '--tab', 'tab-2', 'document.title']);
+        expect(browserState.page?.setActivePage).toHaveBeenCalledWith('tab-2');
+        expect(browserState.page?.evaluate).toHaveBeenCalledWith('document.title');
+    });
+    it('rejects an explicit --tab target that is no longer in the current session', async () => {
+        browserState.page = {
+            setActivePage: vi.fn(),
+            getActivePage: vi.fn(),
+            tabs: vi.fn().mockResolvedValue([]),
+            evaluate: vi.fn(),
+        };
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'eval', '--tab', 'tab-stale', 'document.title']);
+        expect(process.exitCode).toBeDefined();
+        expect(browserState.page?.setActivePage).not.toHaveBeenCalled();
+        expect(browserState.page?.evaluate).not.toHaveBeenCalled();
+        expect(stderrSpy.mock.calls.flat().join('\n')).toContain('Target tab tab-stale is not part of the current browser session');
+    });
+    it('lists tabs with target IDs via browser tab list', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'list']);
+        expect(browserState.page?.tabs).toHaveBeenCalledTimes(1);
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"page": "tab-1"');
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"page": "tab-2"');
+    });
+    it('creates a new tab and prints its target ID', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'new', 'https://three.example']);
+        expect(browserState.page?.newTab).toHaveBeenCalledWith('https://three.example');
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"page": "tab-3"');
+    });
+    it('prints the resolved target ID when browser open creates or navigates a tab', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'open', 'https://example.com']);
+        expect(browserState.page?.goto).toHaveBeenCalledWith('https://example.com');
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"url": "https://one.example"');
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"page": "tab-1"');
+    });
+    it('lists cross-origin frames via browser frames', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'frames']);
+        expect(browserState.page?.frames).toHaveBeenCalledTimes(1);
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"frameId": "frame-1"');
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"url": "https://x.example/embed"');
+    });
+    it('routes browser eval --frame through frame-targeted evaluation', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'eval', '--frame', '0', 'document.title']);
+        expect(browserState.page?.evaluateInFrame).toHaveBeenCalledWith('document.title', 0);
+        expect(browserState.page?.evaluate).not.toHaveBeenCalled();
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('inside frame');
+    });
+    it('does not promote a newly created tab to the persisted default target', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'new', 'https://three.example']);
+        await program.parseAsync(['node', 'opencli', 'browser', 'eval', 'document.title']);
+        expect(browserState.page?.newTab).toHaveBeenCalledWith('https://three.example');
+        expect(browserState.page?.setActivePage).not.toHaveBeenCalled();
+        expect(browserState.page?.evaluate).toHaveBeenCalledWith('document.title');
+    });
+    it('persists an explicitly selected tab as the default target for later untargeted commands', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'select', 'tab-2']);
+        await program.parseAsync(['node', 'opencli', 'browser', 'eval', 'document.title']);
+        expect(browserState.page?.selectTab).toHaveBeenCalledWith('tab-2');
+        expect(browserState.page?.setActivePage).toHaveBeenCalledWith('tab-2');
+        expect(browserState.page?.evaluate).toHaveBeenCalledWith('document.title');
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"selected": "tab-2"');
+    });
+    it('clears a saved default target when it is no longer present in the current session', async () => {
+        const cacheDir = String(process.env.OPENCLI_CACHE_DIR);
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'select', 'tab-2']);
+        expect(fs.existsSync(getBrowserStateFile(cacheDir))).toBe(true);
+        browserState.page = {
+            setActivePage: vi.fn(),
+            getActivePage: vi.fn(),
+            tabs: vi.fn().mockResolvedValue([]),
+            evaluate: vi.fn().mockResolvedValue({ ok: true }),
+            readNetworkCapture: vi.fn().mockResolvedValue([]),
+        };
+        await program.parseAsync(['node', 'opencli', 'browser', 'eval', 'document.title']);
+        expect(browserState.page?.setActivePage).not.toHaveBeenCalled();
+        expect(browserState.page?.evaluate).toHaveBeenCalledWith('document.title');
+        expect(fs.existsSync(getBrowserStateFile(cacheDir))).toBe(false);
+    });
+    it('clears the persisted default target when that tab is closed', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'select', 'tab-2']);
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'close', 'tab-2']);
+        vi.mocked(browserState.page?.setActivePage).mockClear();
+        vi.mocked(browserState.page?.evaluate).mockClear();
+        await program.parseAsync(['node', 'opencli', 'browser', 'eval', 'document.title']);
+        expect(browserState.page?.closeTab).toHaveBeenCalledWith('tab-2');
+        expect(browserState.page?.setActivePage).not.toHaveBeenCalled();
+        expect(browserState.page?.evaluate).toHaveBeenCalledWith('document.title');
+    });
+    it('closes a tab by target ID', async () => {
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'close', 'tab-2']);
+        expect(browserState.page?.closeTab).toHaveBeenCalledWith('tab-2');
+        expect(consoleLogSpy.mock.calls.flat().join('\n')).toContain('"closed": "tab-2"');
+    });
+    it('rejects closing a stale tab target ID that is no longer in the current session', async () => {
+        browserState.page = {
+            tabs: vi.fn().mockResolvedValue([]),
+            closeTab: vi.fn(),
+        };
+        const program = createProgram('', '');
+        await program.parseAsync(['node', 'opencli', 'browser', 'tab', 'close', 'tab-stale']);
+        expect(process.exitCode).toBeDefined();
+        expect(browserState.page?.closeTab).not.toHaveBeenCalled();
+        expect(stderrSpy.mock.calls.flat().join('\n')).toContain('Target tab tab-stale is not part of the current browser session');
+    });
+});
 describe('findPackageRoot', () => {
     it('walks up from dist/src to the package root', () => {
         const packageRoot = path.join('repo-root');

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.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/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: [],