@jackwener/opencli 1.1.1 → 1.2.1

package/dist/build-manifest.test.js

@@ -1,5 +1,8 @@
-import { describe, expect, it } from 'vitest';
-import { parseTsArgsBlock } from './build-manifest.js';
+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(`
@@ -58,3 +61,56 @@ describe('parseTsArgsBlock', () => {
         ]);
     });
 });
+describe('manifest helper rules', () => {
+    const tempDirs = [];
+    afterEach(() => {
+        for (const dir of tempDirs.splice(0)) {
+            fs.rmSync(dir, { recursive: true, force: true });
+        }
+    });
+    it('prefers TS adapters over duplicate YAML adapters', () => {
+        expect(shouldReplaceManifestEntry({
+            site: 'demo',
+            name: 'search',
+            description: 'yaml',
+            strategy: 'public',
+            browser: false,
+            args: [],
+            type: 'yaml',
+        }, {
+            site: 'demo',
+            name: 'search',
+            description: 'ts',
+            strategy: 'public',
+            browser: false,
+            args: [],
+            type: 'ts',
+            modulePath: 'demo/search.js',
+        })).toBe(true);
+        expect(shouldReplaceManifestEntry({
+            site: 'demo',
+            name: 'search',
+            description: 'ts',
+            strategy: 'public',
+            browser: false,
+            args: [],
+            type: 'ts',
+            modulePath: 'demo/search.js',
+        }, {
+            site: 'demo',
+            name: 'search',
+            description: 'yaml',
+            strategy: 'public',
+            browser: false,
+            args: [],
+            type: 'yaml',
+        })).toBe(false);
+    });
+    it('skips TS files that do not register a cli', () => {
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'opencli-manifest-'));
+        tempDirs.push(dir);
+        const file = path.join(dir, 'utils.ts');
+        fs.writeFileSync(file, `export function helper() { return 'noop'; }`);
+        expect(scanTs(file, 'demo')).toBeNull();
+    });
+});