agentic-qe 3.7.11 → 3.7.13

package/scripts/build-mcp.mjs

@@ -52,7 +52,6 @@ const nativeModules = [
 
 // Pure JS externals that work fine with ESM import
 const esmExternals = [
-  'typescript',
   'fast-glob',
   'fast-json-patch',
   'yaml',
@@ -64,19 +63,62 @@ const esmExternals = [
 ];
 
 /**
- * esbuild plugin: Rewrite bare "typescript" import to explicit subpath.
+ * esbuild plugin: Lazy-load typescript via createRequire().
  *
- * TypeScript's package.json has no "exports" field — only "main": "./lib/typescript.js".
- * Node.js 22+ ESM legacyMainResolve fails to resolve the bare specifier in some
- * environments (devcontainers, NVM-managed Node). Rewriting to the explicit subpath
- * bypasses legacyMainResolve entirely (see issue #267).
+ * typescript is a devDependency — it won't be available when users install
+ * agentic-qe globally. A top-level ESM `import` would crash the entire CLI
+ * (even `aqe --version`). This plugin replaces the static import with a
+ * lazy createRequire() call that only executes when the TypeScript parser
+ * is actually used, and throws a helpful error if typescript is missing.
  */
-const typescriptResolvePlugin = {
-  name: 'typescript-resolve',
+/**
+ * esbuild plugin: Lazy-load typescript via createRequire().
+ *
+ * typescript is a devDependency — it won't be available when users install
+ * agentic-qe globally. A top-level ESM `import` would crash the entire CLI
+ * (even `aqe --version`). This plugin replaces the static import with a
+ * lazy createRequire() call that only executes when the TypeScript parser
+ * is actually used, and throws a helpful error if typescript is missing.
+ */
+const typescriptLazyPlugin = {
+  name: 'typescript-lazy',
   setup(build) {
     build.onResolve({ filter: /^typescript$/ }, () => ({
-      path: 'typescript/lib/typescript.js',
-      external: true,
+      path: 'typescript',
+      namespace: 'typescript-lazy',
+    }));
+
+    build.onLoad({ filter: /.*/, namespace: 'typescript-lazy' }, () => ({
+      contents: [
+        'import { createRequire } from "module";',
+        'let _ts;',
+        'function _load() {',
+        '  if (!_ts) {',
+        '    const req = createRequire(import.meta.url);',
+        '    try { _ts = req("typescript"); }',
+        '    catch {',
+        '      try { _ts = req("typescript/lib/typescript.js"); }',
+        '      catch {',
+        '        _ts = new Proxy({}, { get(_, p) {',
+        '          if (p === "__esModule" || typeof p === "symbol") return undefined;',
+        '          throw new Error("TypeScript is required for code analysis. Install it: npm install -g typescript");',
+        '        }});',
+        '      }',
+        '    }',
+        '  }',
+        '  return _ts;',
+        '}',
+        'export default new Proxy({}, {',
+        '  get(_, p) { return _load()[p]; },',
+        '  has(_, p) { return p in _load(); },',
+        '  ownKeys() { return Object.keys(_load()); },',
+        '  getOwnPropertyDescriptor(_, p) {',
+        '    const v = _load()[p];',
+        '    if (v !== undefined) return { configurable: true, enumerable: true, value: v };',
+        '  },',
+        '});',
+      ].join('\n'),
+      loader: 'js',
     }));
   },
 };
@@ -124,7 +166,7 @@ try {
     platform: 'node',
     format: 'esm',
     external: esmExternals,
-    plugins: [typescriptResolvePlugin, nativeRequirePlugin],
+    plugins: [typescriptLazyPlugin, nativeRequirePlugin],
     outfile: join(__dirname, '..', 'dist/mcp/bundle.js'),
     define: {
       '__CLI_VERSION__': JSON.stringify(version),