@jackwener/opencli 1.6.4 → 1.6.5

package/dist/src/package-exports.test.js

@@ -0,0 +1,75 @@
+/**
+ * Regression tests for package exports.
+ *
+ * Ensures adapter files use @jackwener/opencli/... package imports
+ * (not fragile relative paths) and that all declared exports resolve
+ * to real files. Prevents regressions like #788 / #791.
+ */
+import { describe, it, expect } from 'vitest';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import { fileURLToPath } from 'node:url';
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const ROOT = path.resolve(__dirname, '..');
+const CLIS_DIR = path.join(ROOT, 'clis');
+/** Recursively collect all .ts files in a directory. */
+function collectTsFiles(dir) {
+    const results = [];
+    for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
+        const full = path.join(dir, entry.name);
+        if (entry.isDirectory()) {
+            results.push(...collectTsFiles(full));
+        }
+        else if (entry.name.endsWith('.ts') && !entry.name.endsWith('.d.ts')) {
+            results.push(full);
+        }
+    }
+    return results;
+}
+/** Forbidden relative import patterns that should have been replaced.
+ * Uses (?:\.\./)+ to catch any depth of ../ traversal.
+ * Covers: import/export from, vi.mock(), vi.importActual(). */
+const FORBIDDEN_PATTERNS = [
+    /(?:from|mock|importActual)\s*\(?['"](?:\.\.\/)+src\//,
+    /(?:from|mock|importActual)\s*\(?['"](?:\.\.\/)+browser\//,
+    /(?:from|mock|importActual)\s*\(?['"](?:\.\.\/)+download\//,
+    /(?:from|mock|importActual)\s*\(?['"](?:\.\.\/)+pipeline\//,
+];
+describe('adapter imports use package exports', () => {
+    const adapterFiles = collectTsFiles(CLIS_DIR);
+    it('found adapter files to check', () => {
+        expect(adapterFiles.length).toBeGreaterThan(100);
+    });
+    it('no adapter uses relative imports to src/, browser/, download/, or pipeline/', () => {
+        const violations = [];
+        for (const file of adapterFiles) {
+            const content = fs.readFileSync(file, 'utf-8');
+            for (const pattern of FORBIDDEN_PATTERNS) {
+                if (pattern.test(content)) {
+                    const rel = path.relative(ROOT, file);
+                    const match = content.match(pattern)?.[0];
+                    violations.push(`${rel}: ${match}`);
+                }
+            }
+        }
+        expect(violations).toEqual([]);
+    });
+});
+describe('package.json exports resolve to real files', () => {
+    const pkgJson = JSON.parse(fs.readFileSync(path.join(ROOT, 'package.json'), 'utf-8'));
+    const exports = pkgJson.exports;
+    it('has exports defined', () => {
+        expect(Object.keys(exports).length).toBeGreaterThan(5);
+    });
+    for (const [exportPath, target] of Object.entries(exports)) {
+        it(`export "${exportPath}" → ${target} has a source file`, () => {
+            // Export targets point to dist/ (compiled). Verify the source .ts exists.
+            // dist/src/foo.js → src/foo.ts
+            const sourcePath = target
+                .replace(/^\.\/dist\//, './')
+                .replace(/\.js$/, '.ts');
+            const fullPath = path.join(ROOT, sourcePath);
+            expect(fs.existsSync(fullPath), `Missing source: ${sourcePath}`).toBe(true);
+        });
+    }
+});

package/dist/src/registry-api.d.ts

@@ -7,7 +7,7 @@
  * plugins are dynamically imported during discoverPlugins().
  */
 export { cli, Strategy, getRegistry, fullName, registerCommand } from './registry.js';
-export type { CliCommand, Arg, CliOptions } from './registry.js';
+export type { CliCommand, Arg, CliOptions, CommandArgs } from './registry.js';
 export type { IPage } from './types.js';
 export { onStartup, onBeforeExecute, onAfterExecute } from './hooks.js';
 export type { HookFn, HookContext, HookName } from './hooks.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jackwener/opencli",
-  "version": "1.6.4",
+  "version": "1.6.5",
   "publishConfig": {
     "access": "public"
   },
@@ -15,21 +15,25 @@
   },
   "exports": {
     ".": "./dist/src/main.js",
-    "./registry": "./dist/src/registry-api.js"
+    "./registry": "./dist/src/registry-api.js",
+    "./errors": "./dist/src/errors.js",
+    "./types": "./dist/src/types.js",
+    "./utils": "./dist/src/utils.js",
+    "./logger": "./dist/src/logger.js",
+    "./launcher": "./dist/src/launcher.js",
+    "./browser/cdp": "./dist/src/browser/cdp.js",
+    "./browser/page": "./dist/src/browser/page.js",
+    "./browser/utils": "./dist/src/browser/utils.js",
+    "./download": "./dist/src/download/index.js",
+    "./download/article-download": "./dist/src/download/article-download.js",
+    "./download/media-download": "./dist/src/download/media-download.js",
+    "./download/progress": "./dist/src/download/progress.js",
+    "./pipeline": "./dist/src/pipeline/index.js"
   },
   "files": [
     "dist/src/",
     "dist/clis/",
     "dist/cli-manifest.json",
-    "dist/registry.*",
-    "dist/errors.*",
-    "dist/types.*",
-    "dist/utils.*",
-    "dist/logger.*",
-    "dist/launcher.*",
-    "dist/browser/",
-    "dist/download/",
-    "dist/pipeline/",
     "scripts/",
     "README.md",
     "LICENSE"

package/dist/browser/cdp.d.ts

@@ -1 +0,0 @@
-export * from '../src/browser/cdp.js';

package/dist/browser/cdp.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/browser/ for adapter imports
-export * from '../src/browser/cdp.js';

package/dist/browser/page.d.ts

@@ -1 +0,0 @@
-export * from '../src/browser/page.js';

package/dist/browser/page.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/browser/ for adapter imports
-export * from '../src/browser/page.js';

package/dist/browser/utils.d.ts

@@ -1 +0,0 @@
-export * from '../src/browser/utils.js';

package/dist/browser/utils.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/browser/ for adapter imports
-export * from '../src/browser/utils.js';

package/dist/download/article-download.d.ts

@@ -1 +0,0 @@
-export * from '../src/download/article-download.js';

package/dist/download/article-download.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/download/ for adapter imports
-export * from '../src/download/article-download.js';

package/dist/download/index.d.ts

@@ -1 +0,0 @@
-export * from '../src/download/index.js';

package/dist/download/index.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/download/ for adapter imports
-export * from '../src/download/index.js';

package/dist/download/media-download.d.ts

@@ -1 +0,0 @@
-export * from '../src/download/media-download.js';

package/dist/download/media-download.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/download/ for adapter imports
-export * from '../src/download/media-download.js';

package/dist/download/progress.d.ts

@@ -1 +0,0 @@
-export * from '../src/download/progress.js';

package/dist/download/progress.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/download/ for adapter imports
-export * from '../src/download/progress.js';

package/dist/pipeline/index.d.ts

@@ -1 +0,0 @@
-export * from '../src/pipeline/index.js';

package/dist/pipeline/index.js

@@ -1,2 +0,0 @@
-// Shim: re-exports from src/pipeline/ for adapter imports
-export * from '../src/pipeline/index.js';