@jackwener/opencli 1.4.0 → 1.5.0

package/dist/browser/errors.js

@@ -9,20 +9,12 @@ import { DEFAULT_DAEMON_PORT } from '../constants.js';
 export function formatBrowserConnectError(kind, detail) {
     switch (kind) {
         case 'daemon-not-running':
-            return new BrowserConnectError('Cannot connect to opencli daemon.' +
-                (detail ? `\n\n${detail}` : ''), 'The daemon should start automatically. If it doesn\'t, try:\n' +
-                '  node dist/daemon.js\n' +
-                `Make sure port ${DEFAULT_DAEMON_PORT} is available.`);
+            return new BrowserConnectError('Cannot connect to opencli daemon.' + (detail ? `\n\n${detail}` : ''), `The daemon should auto-start. If it keeps failing, make sure port ${DEFAULT_DAEMON_PORT} is available.`, kind);
         case 'extension-not-connected':
-            return new BrowserConnectError('opencli Browser Bridge extension is not connected.' +
-                (detail ? `\n\n${detail}` : ''), 'Please install the extension:\n' +
-                '  1. Download from GitHub Releases\n' +
-                '  2. Open chrome://extensions/ → Enable Developer Mode\n' +
-                '  3. Click "Load unpacked" → select the extension folder\n' +
-                '  4. Make sure Chrome is running');
+            return new BrowserConnectError('Browser Bridge extension is not connected.' + (detail ? `\n\n${detail}` : ''), 'Install the extension from GitHub Releases, then reload.', kind);
         case 'command-failed':
-            return new BrowserConnectError(`Browser command failed: ${detail ?? 'unknown error'}`);
+            return new BrowserConnectError(`Browser command failed: ${detail ?? 'unknown error'}`, undefined, kind);
         default:
-            return new BrowserConnectError(detail ?? 'Failed to connect to browser');
+            return new BrowserConnectError(detail ?? 'Failed to connect to browser', undefined, kind);
     }
 }

package/dist/browser/mcp.d.ts

@@ -2,11 +2,12 @@
  * Browser session manager — auto-spawns daemon and provides IPage.
  */
 import type { IPage } from '../types.js';
+import type { IBrowserFactory } from '../runtime.js';
 export type BrowserBridgeState = 'idle' | 'connecting' | 'connected' | 'closing' | 'closed';
 /**
  * Browser factory: manages daemon lifecycle and provides IPage instances.
  */
-export declare class BrowserBridge {
+export declare class BrowserBridge implements IBrowserFactory {
     private _state;
     private _page;
     private _daemonProc;

package/dist/build-manifest.d.ts

@@ -27,6 +27,8 @@ export interface ManifestEntry {
     columns?: string[];
     pipeline?: Record<string, unknown>[];
     timeout?: number;
+    deprecated?: boolean | string;
+    replacedBy?: string;
     /** 'yaml' or 'ts' — determines how executeCommand loads the handler */
     type: 'yaml' | 'ts';
     /** Relative path from clis/ dir, e.g. 'bilibili/hot.yaml' or 'bilibili/search.js' */

package/dist/build-manifest.js

@@ -16,6 +16,7 @@ import { getErrorMessage } from './errors.js';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const CLIS_DIR = path.resolve(__dirname, 'clis');
 const OUTPUT = path.resolve(__dirname, '..', 'dist', 'cli-manifest.json');
+import { parseYamlArgs } from './yaml-schema.js';
 import { isRecord } from './utils.js';
 function extractBalancedBlock(source, startIndex, openChar, closeChar) {
     let depth = 0;
@@ -126,20 +127,7 @@ function scanYaml(filePath, site) {
         const strategyStr = cliDef.strategy ?? (cliDef.browser === false ? 'public' : 'cookie');
         const strategy = strategyStr.toUpperCase();
         const browser = cliDef.browser ?? (strategy !== 'PUBLIC');
-        const args = [];
-        if (cliDef.args && typeof cliDef.args === 'object') {
-            for (const [argName, argDef] of Object.entries(cliDef.args)) {
-                args.push({
-                    name: argName,
-                    type: argDef?.type ?? 'str',
-                    default: argDef?.default,
-                    required: argDef?.required ?? false,
-                    positional: argDef?.positional === true || undefined,
-                    help: argDef?.description ?? argDef?.help ?? '',
-                    choices: argDef?.choices,
-                });
-            }
-        }
+        const args = parseYamlArgs(cliDef.args);
         return {
             site: cliDef.site ?? site,
             name: cliDef.name ?? path.basename(filePath, path.extname(filePath)),
@@ -151,6 +139,8 @@ function scanYaml(filePath, site) {
             columns: cliDef.columns,
             pipeline: cliDef.pipeline,
             timeout: cliDef.timeout,
+            deprecated: cliDef.deprecated,
+            replacedBy: cliDef.replacedBy,
             type: 'yaml',
             navigateBefore: cliDef.navigateBefore,
         };
@@ -218,6 +208,18 @@ export function scanTs(filePath, site) {
             if (navStringMatch)
                 entry.navigateBefore = navStringMatch[1];
         }
+        const deprecatedBoolMatch = src.match(/deprecated\s*:\s*(true|false)/);
+        if (deprecatedBoolMatch) {
+            entry.deprecated = deprecatedBoolMatch[1] === 'true';
+        }
+        else {
+            const deprecatedStringMatch = src.match(/deprecated\s*:\s*['"`]([^'"`]+)['"`]/);
+            if (deprecatedStringMatch)
+                entry.deprecated = deprecatedStringMatch[1];
+        }
+        const replacedByMatch = src.match(/replacedBy\s*:\s*['"`]([^'"`]+)['"`]/);
+        if (replacedByMatch)
+            entry.replacedBy = replacedByMatch[1];
         return entry;
     }
     catch (err) {
@@ -283,6 +285,29 @@ function main() {
     const yamlCount = manifest.filter(e => e.type === 'yaml').length;
     const tsCount = manifest.filter(e => e.type === 'ts').length;
     console.log(`✅ Manifest compiled: ${manifest.length} entries (${yamlCount} YAML, ${tsCount} TS) → ${OUTPUT}`);
+    // Restore executable permissions on bin entries.
+    // tsc does not preserve the +x bit, so after a clean rebuild the CLI
+    // entry-point loses its executable permission, causing "Permission denied".
+    // See: https://github.com/jackwener/opencli/issues/446
+    if (process.platform !== 'win32') {
+        const pkgPath = path.resolve(__dirname, '..', 'package.json');
+        try {
+            const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
+            const bins = typeof pkg.bin === 'string'
+                ? { [pkg.name ?? 'cli']: pkg.bin }
+                : pkg.bin ?? {};
+            for (const binPath of Object.values(bins)) {
+                const abs = path.resolve(__dirname, '..', binPath);
+                if (fs.existsSync(abs)) {
+                    fs.chmodSync(abs, 0o755);
+                    console.log(`✅ Restored executable permission: ${binPath}`);
+                }
+            }
+        }
+        catch {
+            // Best-effort; never break the build for a permission fix.
+        }
+    }
 }
 const entrypoint = process.argv[1] ? pathToFileURL(path.resolve(process.argv[1])).href : null;
 if (entrypoint === import.meta.url) {

package/dist/build-manifest.test.js

@@ -125,4 +125,25 @@ describe('manifest helper rules', () => {
             modulePath: 'xueqiu/fund-holdings.js',
         });
     });
+    it('captures deprecated metadata for TS adapters', () => {
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'opencli-manifest-'));
+        tempDirs.push(dir);
+        const file = path.join(dir, 'legacy.ts');
+        fs.writeFileSync(file, `
+      import { cli } from '../../registry.js';
+      cli({
+        site: 'demo',
+        name: 'legacy',
+        description: 'legacy command',
+        deprecated: 'legacy is deprecated',
+        replacedBy: 'opencli demo new',
+      });
+    `);
+        expect(scanTs(file, 'demo')).toMatchObject({
+            site: 'demo',
+            name: 'legacy',
+            deprecated: 'legacy is deprecated',
+            replacedBy: 'opencli demo new',
+        });
+    });
 });

package/dist/capabilityRouting.d.ts

@@ -1,2 +1,4 @@
 import { type CliCommand } from './registry.js';
+/** Pipeline steps that require a live browser session. */
+export declare const BROWSER_ONLY_STEPS: Set<string>;
 export declare function shouldUseBrowserSession(cmd: CliCommand): boolean;

package/dist/capabilityRouting.js

@@ -1,5 +1,6 @@
 import { Strategy } from './registry.js';
-const BROWSER_ONLY_STEPS = new Set([
+/** Pipeline steps that require a live browser session. */
+export const BROWSER_ONLY_STEPS = new Set([
     'navigate',
     'click',
     'type',