@jackwener/opencli 1.5.3 → 1.5.4

package/dist/build-manifest.test.js

@@ -2,65 +2,8 @@ import { afterEach, describe, expect, it } from 'vitest';
 import * as fs from 'node:fs';
 import * as os from 'node:os';
 import * as path from 'node:path';
-import { parseTsArgsBlock, scanTs, shouldReplaceManifestEntry } from './build-manifest.js';
-describe('parseTsArgsBlock', () => {
-    it('keeps args with nested choices arrays', () => {
-        const args = parseTsArgsBlock(`
-      {
-        name: 'period',
-        type: 'string',
-        default: 'seven',
-        help: 'Stats period: seven or thirty',
-        choices: ['seven', 'thirty'],
-      },
-    `);
-        expect(args).toEqual([
-            {
-                name: 'period',
-                type: 'string',
-                default: 'seven',
-                required: false,
-                positional: undefined,
-                help: 'Stats period: seven or thirty',
-                choices: ['seven', 'thirty'],
-            },
-        ]);
-    });
-    it('keeps hyphenated arg names from TS adapters', () => {
-        const args = parseTsArgsBlock(`
-      {
-        name: 'tweet-url',
-        help: 'Single tweet URL to download',
-      },
-      {
-        name: 'download-images',
-        type: 'boolean',
-        default: false,
-        help: 'Download images locally',
-      },
-    `);
-        expect(args).toEqual([
-            {
-                name: 'tweet-url',
-                type: 'str',
-                default: undefined,
-                required: false,
-                positional: undefined,
-                help: 'Single tweet URL to download',
-                choices: undefined,
-            },
-            {
-                name: 'download-images',
-                type: 'boolean',
-                default: false,
-                required: false,
-                positional: undefined,
-                help: 'Download images locally',
-                choices: undefined,
-            },
-        ]);
-    });
-});
+import { cli, getRegistry, Strategy } from './registry.js';
+import { loadTsManifestEntries, shouldReplaceManifestEntry } from './build-manifest.js';
 describe('manifest helper rules', () => {
     const tempDirs = [];
     afterEach(() => {
@@ -111,39 +54,121 @@ describe('manifest helper rules', () => {
         tempDirs.push(dir);
         const file = path.join(dir, 'utils.ts');
         fs.writeFileSync(file, `export function helper() { return 'noop'; }`);
-        expect(scanTs(file, 'demo')).toBeNull();
+        return expect(loadTsManifestEntries(file, 'demo', async () => ({}))).resolves.toEqual([]);
     });
-    it('keeps literal domain and navigateBefore for TS adapters', () => {
-        const file = path.join(process.cwd(), 'src', 'clis', 'xueqiu', 'fund-holdings.ts');
-        const entry = scanTs(file, 'xueqiu');
-        expect(entry).toMatchObject({
-            site: 'xueqiu',
-            name: 'fund-holdings',
-            domain: 'danjuanfunds.com',
-            navigateBefore: 'https://danjuanfunds.com/my-money',
-            type: 'ts',
-            modulePath: 'xueqiu/fund-holdings.js',
-        });
+    it('builds TS manifest entries from exported runtime commands', async () => {
+        const site = `manifest-hydrate-${Date.now()}`;
+        const key = `${site}/dynamic`;
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'opencli-manifest-'));
+        tempDirs.push(dir);
+        const file = path.join(dir, `${site}.ts`);
+        fs.writeFileSync(file, `export const command = cli({ site: '${site}', name: 'dynamic' });`);
+        const entries = await loadTsManifestEntries(file, site, async () => ({
+            command: cli({
+                site,
+                name: 'dynamic',
+                description: 'dynamic command',
+                strategy: Strategy.PUBLIC,
+                browser: false,
+                args: [
+                    {
+                        name: 'model',
+                        required: true,
+                        positional: true,
+                        help: 'Choose a model',
+                        choices: ['auto', 'thinking'],
+                        default: '30',
+                    },
+                ],
+                domain: 'localhost',
+                navigateBefore: 'https://example.com/session',
+                deprecated: 'legacy command',
+                replacedBy: 'opencli demo new',
+            }),
+        }));
+        expect(entries).toEqual([
+            {
+                site,
+                name: 'dynamic',
+                description: 'dynamic command',
+                domain: 'localhost',
+                strategy: 'public',
+                browser: false,
+                args: [
+                    {
+                        name: 'model',
+                        type: 'str',
+                        required: true,
+                        positional: true,
+                        help: 'Choose a model',
+                        choices: ['auto', 'thinking'],
+                        default: '30',
+                    },
+                ],
+                type: 'ts',
+                modulePath: `${site}/${site}.js`,
+                navigateBefore: 'https://example.com/session',
+                deprecated: 'legacy command',
+                replacedBy: 'opencli demo new',
+            },
+        ]);
+        getRegistry().delete(key);
     });
-    it('captures deprecated metadata for TS adapters', () => {
+    it('falls back to registry delta for side-effect-only cli modules', async () => {
+        const site = `manifest-side-effect-${Date.now()}`;
+        const key = `${site}/legacy`;
         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',
+        const file = path.join(dir, `${site}.ts`);
+        fs.writeFileSync(file, `cli({ site: '${site}', name: 'legacy' });`);
+        const entries = await loadTsManifestEntries(file, site, async () => {
+            cli({
+                site,
+                name: 'legacy',
+                description: 'legacy command',
+                deprecated: 'legacy is deprecated',
+                replacedBy: 'opencli demo new',
+            });
+            return {};
         });
+        expect(entries).toEqual([
+            {
+                site,
+                name: 'legacy',
+                description: 'legacy command',
+                strategy: 'cookie',
+                browser: true,
+                args: [],
+                type: 'ts',
+                modulePath: `${site}/${site}.js`,
+                deprecated: 'legacy is deprecated',
+                replacedBy: 'opencli demo new',
+            },
+        ]);
+        getRegistry().delete(key);
+    });
+    it('keeps every command a module exports instead of guessing by site', async () => {
+        const site = `manifest-multi-${Date.now()}`;
+        const screenKey = `${site}/screen`;
+        const statusKey = `${site}/status`;
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'opencli-manifest-'));
+        tempDirs.push(dir);
+        const file = path.join(dir, `${site}.ts`);
+        fs.writeFileSync(file, `export const screen = cli({ site: '${site}', name: 'screen' });`);
+        const entries = await loadTsManifestEntries(file, site, async () => ({
+            screen: cli({
+                site,
+                name: 'screen',
+                description: 'capture screen',
+            }),
+            status: cli({
+                site,
+                name: 'status',
+                description: 'show status',
+            }),
+        }));
+        expect(entries.map(entry => entry.name)).toEqual(['screen', 'status']);
+        getRegistry().delete(screenKey);
+        getRegistry().delete(statusKey);
     });
 });