@jackwener/opencli 1.7.3 → 1.7.5

package/dist/src/doctor.test.js

@@ -25,10 +25,11 @@ describe('doctor report rendering', () => {
         const text = strip(renderBrowserDoctorReport({
             daemonRunning: true,
             extensionConnected: true,
+            extensionVersion: '1.6.8',
             issues: [],
         }));
         expect(text).toContain('[OK] Daemon: running on port 19825');
-        expect(text).toContain('[OK] Extension: connected');
+        expect(text).toContain('[OK] Extension: connected (v1.6.8)');
         expect(text).toContain('Everything looks good!');
     });
     it('renders MISSING when daemon not running', () => {
@@ -50,6 +51,16 @@ describe('doctor report rendering', () => {
         expect(text).toContain('[OK] Daemon: running on port 19825');
         expect(text).toContain('[MISSING] Extension: not connected');
     });
+    it('renders a warning when the extension version is unknown', () => {
+        const text = strip(renderBrowserDoctorReport({
+            daemonRunning: true,
+            extensionConnected: true,
+            issues: ['Extension is connected but did not report a version.'],
+        }));
+        expect(text).toContain('[WARN] Extension: connected (version unknown)');
+        expect(text).toContain('Extension is connected but did not report a version.');
+        expect(text).not.toContain('Everything looks good!');
+    });
     it('renders connectivity OK when live test succeeds', () => {
         const text = strip(renderBrowserDoctorReport({
             daemonRunning: true,
@@ -90,12 +101,8 @@ describe('doctor report rendering', () => {
         expect(text).toContain('Daemon connectivity is unstable.');
     });
     it('reports daemon not running when no-live and auto-start fails', async () => {
-        // no-live mode: getDaemonHealth called twice (initial check + final status)
-        // Initial: stopped → triggers auto-start attempt
         mockGetDaemonHealth.mockResolvedValueOnce({ state: 'stopped', status: null });
-        // Auto-start fails
         mockConnect.mockRejectedValueOnce(new Error('Could not start daemon'));
-        // Final: still stopped
         mockGetDaemonHealth.mockResolvedValueOnce({ state: 'stopped', status: null });
         const report = await runBrowserDoctor({ live: false });
         expect(report.daemonRunning).toBe(false);
@@ -106,12 +113,10 @@ describe('doctor report rendering', () => {
         ]));
     });
     it('reports flapping when live check succeeds but final status shows extension disconnected', async () => {
-        // Live check succeeds
         mockConnect.mockResolvedValueOnce({
             evaluate: vi.fn().mockResolvedValue(2),
         });
         mockClose.mockResolvedValueOnce(undefined);
-        // After live check, getDaemonHealth shows no-extension
         mockGetDaemonHealth.mockResolvedValueOnce({ state: 'no-extension', status: { extensionConnected: false } });
         const report = await runBrowserDoctor({ live: true });
         expect(report.daemonRunning).toBe(true);
@@ -122,12 +127,10 @@ describe('doctor report rendering', () => {
         ]));
     });
     it('reports daemon flapping when live check succeeds but daemon disappears afterward', async () => {
-        // Live check succeeds
         mockConnect.mockResolvedValueOnce({
             evaluate: vi.fn().mockResolvedValue(2),
         });
         mockClose.mockResolvedValueOnce(undefined);
-        // After live check, getDaemonHealth shows stopped
         mockGetDaemonHealth.mockResolvedValueOnce({ state: 'stopped', status: null });
         const report = await runBrowserDoctor({ live: true });
         expect(report.daemonRunning).toBe(false);
@@ -151,14 +154,27 @@ describe('doctor report rendering', () => {
         expect(timeoutSeen).toBe(8);
     });
     it('skips auto-start in no-live mode when daemon is already running', async () => {
-        // no-live mode but daemon already running (no-extension)
         mockGetDaemonHealth.mockResolvedValueOnce({ state: 'no-extension', status: { extensionConnected: false } });
-        // Final status: same
         mockGetDaemonHealth.mockResolvedValueOnce({ state: 'no-extension', status: { extensionConnected: false } });
         const report = await runBrowserDoctor({ live: false });
-        // Should NOT have tried auto-start since daemon was already running
         expect(mockConnect).not.toHaveBeenCalled();
         expect(report.daemonRunning).toBe(true);
         expect(report.extensionConnected).toBe(false);
     });
+    it('reports an issue when the extension is connected but does not report a version', async () => {
+        const status = {
+            state: 'ready',
+            status: {
+                extensionConnected: true,
+                extensionVersion: undefined,
+            },
+        };
+        mockGetDaemonHealth
+            .mockResolvedValueOnce(status)
+            .mockResolvedValueOnce(status);
+        const report = await runBrowserDoctor({ live: false });
+        expect(report.issues).toEqual(expect.arrayContaining([
+            expect.stringContaining('did not report a version'),
+        ]));
+    });
 });

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/external-clis.yaml

@@ -36,8 +36,8 @@
   homepage: "https://github.com/DingTalk-Real-AI/dingtalk-workspace-cli"
   tags: [dingtalk, collaboration, productivity, ai-agent]
   install:
-    mac: "curl -fsSL https://raw.githubusercontent.com/DingTalk-Real-AI/dingtalk-workspace-cli/main/scripts/install.sh | sh"
-    linux: "curl -fsSL https://raw.githubusercontent.com/DingTalk-Real-AI/dingtalk-workspace-cli/main/scripts/install.sh | sh"
+    mac: "npm install -g dingtalk-workspace-cli"
+    linux: "npm install -g dingtalk-workspace-cli"
 
 - name: wecom-cli
   binary: wecom-cli

package/dist/src/logger.d.ts

@@ -15,9 +15,9 @@ export declare const log: {
     warn(msg: string): void;
     /** Error (always shown) */
     error(msg: string): void;
-    /** Verbose output (shown when -v flag, OPENCLI_VERBOSE, or DEBUG=opencli is set) */
+    /** Verbose output (shown when -v flag or OPENCLI_VERBOSE is set) */
     verbose(msg: string): void;
-    /** @deprecated Use log.verbose() instead. Kept as alias for backward compatibility. */
+    /** Alias for verbose output. */
     debug(msg: string): void;
     /** Step-style debug (for pipeline steps, etc.) */
     step(stepNum: number, total: number, op: string, preview?: string): void;

package/dist/src/logger.js

@@ -6,7 +6,7 @@
  */
 import { styleText } from 'node:util';
 function isVerbose() {
-    return !!process.env.OPENCLI_VERBOSE || !!process.env.DEBUG?.includes('opencli');
+    return !!process.env.OPENCLI_VERBOSE;
 }
 export const log = {
     /** Informational message (always shown) */
@@ -29,13 +29,13 @@ export const log = {
     error(msg) {
         process.stderr.write(`${styleText('red', '✖')}  ${msg}\n`);
     },
-    /** Verbose output (shown when -v flag, OPENCLI_VERBOSE, or DEBUG=opencli is set) */
+    /** Verbose output (shown when -v flag or OPENCLI_VERBOSE is set) */
     verbose(msg) {
         if (isVerbose()) {
             process.stderr.write(`${styleText('dim', '[verbose]')} ${msg}\n`);
         }
     },
-    /** @deprecated Use log.verbose() instead. Kept as alias for backward compatibility. */
+    /** Alias for verbose output. */
     debug(msg) {
         this.verbose(msg);
     },

package/dist/src/output.js

@@ -17,12 +17,8 @@ function resolveColumns(rows, opts) {
 export function render(data, opts = {}) {
     let fmt = opts.fmt ?? 'table';
     // Non-TTY auto-downgrade only when format was NOT explicitly passed by user.
-    // Priority: explicit -f (any value) > OUTPUT env var > TTY auto-detect > table
     if (!opts.fmtExplicit) {
-        const envFmt = process.env.OUTPUT?.trim().toLowerCase();
-        if (envFmt)
-            fmt = envFmt;
-        else if (fmt === 'table' && !process.stdout.isTTY)
+        if (fmt === 'table' && !process.stdout.isTTY)
             fmt = 'yaml';
     }
     if (data === null || data === undefined) {

package/dist/src/output.test.js

@@ -2,17 +2,12 @@ import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';
 import { render } from './output.js';
 describe('output TTY detection', () => {
     const originalIsTTY = process.stdout.isTTY;
-    const originalEnv = process.env.OUTPUT;
     let logSpy;
     beforeEach(() => {
         logSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
     });
     afterEach(() => {
         Object.defineProperty(process.stdout, 'isTTY', { value: originalIsTTY, writable: true });
-        if (originalEnv === undefined)
-            delete process.env.OUTPUT;
-        else
-            process.env.OUTPUT = originalEnv;
         logSpy.mockRestore();
     });
     it('outputs YAML in non-TTY when format is default table', () => {
@@ -35,22 +30,6 @@ describe('output TTY detection', () => {
         const out = logSpy.mock.calls.map((c) => c[0]).join('\n');
         expect(JSON.parse(out)).toEqual([{ name: 'alice' }]);
     });
-    it('OUTPUT env var overrides default table in non-TTY', () => {
-        Object.defineProperty(process.stdout, 'isTTY', { value: false, writable: true });
-        process.env.OUTPUT = 'json';
-        render([{ name: 'alice' }], { fmt: 'table' });
-        const out = logSpy.mock.calls.map((c) => c[0]).join('\n');
-        expect(JSON.parse(out)).toEqual([{ name: 'alice' }]);
-    });
-    it('explicit -f flag takes precedence over OUTPUT env var', () => {
-        Object.defineProperty(process.stdout, 'isTTY', { value: false, writable: true });
-        process.env.OUTPUT = 'json';
-        render([{ name: 'alice' }], { fmt: 'csv', fmtExplicit: true });
-        const out = logSpy.mock.calls.map((c) => c[0]).join('\n');
-        expect(out).toContain('name');
-        expect(out).toContain('alice');
-        expect(out).not.toContain('"name"'); // not JSON
-    });
     it('explicit -f table overrides non-TTY auto-downgrade', () => {
         Object.defineProperty(process.stdout, 'isTTY', { value: false, writable: true });
         render([{ name: 'alice' }], { fmt: 'table', fmtExplicit: true, columns: ['name'] });

package/dist/src/pipeline/steps/transform.js

@@ -40,7 +40,7 @@ export async function stepMap(_page, params, data, args) {
         for (const [key, template] of Object.entries(templateParams)) {
             if (key === 'select')
                 continue;
-            row[key] = render(template, { args, data: source, item, index: i });
+            row[key] = render(template, { args, data: source, root: data, item, index: i });
         }
         result.push(row);
     }

package/dist/src/pipeline/template.d.ts

@@ -4,6 +4,7 @@
 export interface RenderContext {
     args?: Record<string, unknown>;
     data?: unknown;
+    root?: unknown;
     item?: unknown;
     index?: number;
 }

package/dist/src/pipeline/template.js

@@ -26,6 +26,7 @@ export function evalExpr(expr, ctx) {
     const args = ctx.args ?? {};
     const item = ctx.item ?? {};
     const data = ctx.data;
+    const root = ctx.root;
     const index = ctx.index ?? 0;
     // ── Pipe filters: expr | filter1(arg) | filter2 ──
     // Split on single | (not ||) so "item.a || item.b | upper" works correctly.
@@ -45,12 +46,12 @@ export function evalExpr(expr, ctx) {
     if (/^\d+(\.\d+)?$/.test(expr))
         return Number(expr);
     // Try resolving as a simple dotted path (item.foo.bar, args.limit, index)
-    const resolved = resolvePath(expr, { args, item, data, index });
+    const resolved = resolvePath(expr, { args, item, data, root, index });
     if (resolved !== null && resolved !== undefined)
         return resolved;
     // Fallback: evaluate as JS in a sandboxed VM.
     // Handles ||, ??, arithmetic, ternary, method calls, etc. natively.
-    return evalJsExpr(expr, { args, item, data, index });
+    return evalJsExpr(expr, { args, item, data, root, index });
 }
 /**
  * Apply a named filter to a value.
@@ -143,6 +144,7 @@ export function resolvePath(pathStr, ctx) {
     const args = ctx.args ?? {};
     const item = ctx.item ?? {};
     const data = ctx.data;
+    const root = ctx.root;
     const index = ctx.index ?? 0;
     const parts = pathStr.split('.');
     const rootName = parts[0];
@@ -160,6 +162,10 @@ export function resolvePath(pathStr, ctx) {
         obj = data;
         rest = parts.slice(1);
     }
+    else if (rootName === 'root') {
+        obj = root;
+        rest = parts.slice(1);
+    }
     else if (rootName === 'index')
         return index;
     else {
@@ -261,6 +267,7 @@ function getReusableContext() {
         args: {},
         item: {},
         data: null,
+        root: null,
         index: 0,
         encodeURIComponent,
         decodeURIComponent,
@@ -279,7 +286,7 @@ function getReusableContext() {
 }
 /** Properties that are part of the sandbox's initial shape and safe to keep. */
 const SANDBOX_WHITELIST = new Set([
-    'args', 'item', 'data', 'index',
+    'args', 'item', 'data', 'root', 'index',
     'encodeURIComponent', 'decodeURIComponent',
     'JSON', 'Math', 'Number', 'String', 'Boolean', 'Array', 'Date',
 ]);
@@ -304,6 +311,7 @@ function evalJsExpr(expr, ctx) {
         sandbox.args = sanitizeContext(ctx.args ?? {});
         sandbox.item = sanitizeContext(ctx.item ?? {});
         sandbox.data = sanitizeContext(ctx.data);
+        sandbox.root = sanitizeContext(ctx.root);
         sandbox.index = ctx.index ?? 0;
         return script.runInContext(context, { timeout: 50 });
     }

package/dist/src/pipeline/template.test.js

@@ -22,6 +22,9 @@ describe('resolvePath', () => {
     it('resolves data path', () => {
         expect(resolvePath('data.items', { data: { items: [1, 2, 3] } })).toEqual([1, 2, 3]);
     });
+    it('resolves root path', () => {
+        expect(resolvePath('root.items', { root: { items: [1, 2, 3] } })).toEqual([1, 2, 3]);
+    });
     it('returns null for missing path', () => {
         expect(resolvePath('args.missing', { args: {} })).toBeUndefined();
     });

package/dist/src/pipeline/transform.test.js

@@ -60,6 +60,20 @@ describe('stepMap', () => {
             { title: 'Two', rank: 2 },
         ]);
     });
+    it('keeps data bound to the selected source and exposes root separately', async () => {
+        const result = await stepMap(null, {
+            select: 'bids',
+            bid_price: '${{ data[index][0] }}',
+            ask_price: '${{ root.asks[index][0] }}',
+        }, {
+            bids: [['100', '2'], ['99', '3']],
+            asks: [['101', '1'], ['102', '4']],
+        }, {});
+        expect(result).toEqual([
+            { bid_price: '100', ask_price: '101' },
+            { bid_price: '99', ask_price: '102' },
+        ]);
+    });
 });
 describe('stepFilter', () => {
     it('filters by expression', async () => {

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[];
@@ -86,7 +79,13 @@ export declare function getCommitHash(dir: string): string | undefined;
 export declare function validatePluginStructure(pluginDir: string): ValidationResult;
 declare function installDependencies(dir: string): void;
 /**
- * Monorepo lifecycle: install shared deps once at repo root, then finalize each sub-plugin.
+ * Monorepo lifecycle: install shared deps at repo root, then install and finalize each sub-plugin.
+ *
+ * The root install covers monorepos that use npm workspaces to hoist dependencies.
+ * For monorepos that do NOT use workspaces, sub-plugins may declare their own
+ * production dependencies in their package.json.  We install those per sub-plugin
+ * so that runtime imports (e.g. `undici`) can be resolved from the sub-plugin
+ * directory.  When the root already satisfies all deps this is a fast no-op.
  */
 declare function postInstallMonorepoLifecycle(repoDir: string, pluginDirs: string[]): void;
 /**
@@ -143,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 {
@@ -492,6 +466,19 @@ export function validatePluginStructure(pluginDir) {
     }
     return { valid: errors.length === 0, errors };
 }
+/** Check whether a directory has its own production dependencies in package.json. */
+function hasOwnDependencies(dir) {
+    const pkgPath = path.join(dir, 'package.json');
+    if (!fs.existsSync(pkgPath))
+        return false;
+    try {
+        const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
+        return pkg.dependencies != null && Object.keys(pkg.dependencies).length > 0;
+    }
+    catch {
+        return false;
+    }
+}
 function installDependencies(dir) {
     const pkgJsonPath = path.join(dir, 'package.json');
     if (!fs.existsSync(pkgJsonPath))
@@ -523,11 +510,20 @@ function postInstallLifecycle(pluginDir) {
     finalizePluginRuntime(pluginDir);
 }
 /**
- * Monorepo lifecycle: install shared deps once at repo root, then finalize each sub-plugin.
+ * Monorepo lifecycle: install shared deps at repo root, then install and finalize each sub-plugin.
+ *
+ * The root install covers monorepos that use npm workspaces to hoist dependencies.
+ * For monorepos that do NOT use workspaces, sub-plugins may declare their own
+ * production dependencies in their package.json.  We install those per sub-plugin
+ * so that runtime imports (e.g. `undici`) can be resolved from the sub-plugin
+ * directory.  When the root already satisfies all deps this is a fast no-op.
  */
 function postInstallMonorepoLifecycle(repoDir, pluginDirs) {
     installDependencies(repoDir);
     for (const pluginDir of pluginDirs) {
+        if (pluginDir !== repoDir && hasOwnDependencies(pluginDir)) {
+            installDependencies(pluginDir);
+        }
         finalizePluginRuntime(pluginDir);
     }
 }
@@ -1246,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');
@@ -545,13 +545,27 @@ describe('postInstallMonorepoLifecycle', () => {
     afterEach(() => {
         fs.rmSync(repoDir, { recursive: true, force: true });
     });
-    it('installs dependencies once at the monorepo root, not in each sub-plugin', () => {
+    it('installs dependencies at the monorepo root and skips sub-plugins without own dependencies', () => {
         _postInstallMonorepoLifecycle(repoDir, [subDir]);
         const npmCalls = mockExecFileSync.mock.calls.filter(([cmd, args]) => cmd === 'npm' && Array.isArray(args) && args[0] === 'install');
         expect(npmCalls).toHaveLength(1);
         expect(npmCalls[0][2]).toMatchObject({ cwd: repoDir });
         expect(npmCalls.some(([, , opts]) => opts?.cwd === subDir)).toBe(false);
     });
+    it('also installs dependencies in sub-plugins that declare their own production dependencies', () => {
+        // Give the sub-plugin its own production dependencies
+        fs.writeFileSync(path.join(subDir, 'package.json'), JSON.stringify({
+            name: 'opencli-plugin-alpha',
+            version: '1.0.0',
+            type: 'module',
+            dependencies: { undici: '^8.0.0' },
+        }));
+        _postInstallMonorepoLifecycle(repoDir, [subDir]);
+        const npmCalls = mockExecFileSync.mock.calls.filter(([cmd, args]) => cmd === 'npm' && Array.isArray(args) && args[0] === 'install');
+        expect(npmCalls).toHaveLength(2);
+        expect(npmCalls[0][2]).toMatchObject({ cwd: repoDir });
+        expect(npmCalls[1][2]).toMatchObject({ cwd: subDir });
+    });
 });
 describe('updateAllPlugins', () => {
     const testDirA = path.join(PLUGINS_DIR, 'plugin-a');
@@ -910,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.