fix: more dumb cjs/esm issues

Author: jedwards1211

package.json

@@ -152,7 +152,7 @@
     "@jedwards1211/eslint-config": "^2.0.2",
     "@jedwards1211/eslint-config-typescript": "^2.0.2",
     "@types/babel__code-frame": "^7.0.3",
-    "@types/babel__generator": "^7.6.3",
+    "@types/babel__generator": "^7.6.5",
     "@types/babel__parser": "^7.1.1",
     "@types/chai": "^4.2.14",
     "@types/chai-as-promised": "^7.1.5",

pnpm-lock.yaml

@@ -1,15 +1,16 @@
 import { Node, File, Statement, Expression } from '@babel/types'
 import { Backend } from '../backend/Backend'
-import * as defaultParser from '@babel/parser'
+import defaultParser from '@babel/parser'
 import { ParserOptions } from '@babel/parser'
 import * as defaultTypes from '@babel/types'
 import * as defaultGenerator from '@babel/generator'
-import * as AstTypes from 'ast-types'
+import type * as AstTypes from 'ast-types'
 import babelAstTypes from './babelAstTypes'
 import { Comment, Location } from '../types'
 import reprint from './reprint'
 import detectChangedNodes from '../util/detectChangedNodes'
 import { cloneAstWithOriginals } from '../util/cloneAstWithOriginals'
+import babelInterop from './babelInterop'
 
 interface Parser {
   parse(code: string, parserOpts?: ParserOptions): File
@@ -34,8 +35,8 @@ export default class BabelBackend extends Backend<Node> {
   constructor({
     parser = defaultParser,
     parserOptions,
-    generator = defaultGenerator,
-    types = defaultTypes,
+    generator = babelInterop(defaultGenerator),
+    types = babelInterop(defaultTypes),
     preserveFormat,
   }: {
     parser?: Parser
@@ -46,6 +47,10 @@ export default class BabelBackend extends Backend<Node> {
   } = {}) {
     super()
 
+    if (typeof generator.default !== 'function') {
+      throw new Error(`typeof generator.default should === 'function'`)
+    }
+
     const t = babelAstTypes(types)
 
     this.t = t

src/babel/BabelBackend.ts

@@ -0,0 +1,6 @@
+export default function babelInterop<T>(module: T): T {
+  return (module as any).default instanceof Object &&
+    Object.getPrototypeOf(module) == null
+    ? (module as any).default
+    : module
+}

src/babel/babelInterop.ts

@@ -1,10 +1,11 @@
 import path from 'path'
 import _resolve from 'resolve'
 import BabelBackend from './BabelBackend'
-import * as defaultBabelTypes from '@babel/types'
-import * as defaultBabelGenerator from '@babel/generator'
+import * as defaultTypes from '@babel/types'
+import * as defaultGenerator from '@babel/generator'
 import { getParserAsync } from 'babel-parse-wild-code'
 import { promisify } from 'util'
+import babelInterop from './babelInterop'
 
 const resolve: (
   id: string,
@@ -27,13 +28,12 @@ export default async function getBabelAutoBackend(
   const basedir = path.dirname(file)
   const [_parser, types, generator] = await Promise.all([
     getParserAsync(file, options),
-    importLocal<typeof defaultBabelTypes>('@babel/types', basedir).catch(
-      () => defaultBabelTypes
+    importLocal<typeof defaultTypes>('@babel/types', basedir).catch(
+      () => defaultTypes
+    ),
+    importLocal<typeof defaultGenerator>('@babel/generator', basedir).catch(
+      () => defaultGenerator
     ),
-    importLocal<typeof defaultBabelGenerator>(
-      '@babel/generator',
-      basedir
-    ).catch(() => defaultBabelGenerator),
   ])
   const parser = (
     _parser.parserOpts.sourceType
@@ -44,7 +44,7 @@ export default async function getBabelAutoBackend(
     parser: parser as any,
     parserOptions: options,
     preserveFormat,
-    generator,
-    types,
+    generator: babelInterop(generator),
+    types: babelInterop(types),
   })
 }

src/babel/getBabelAutoBackend.ts

@@ -1,4 +1,4 @@
-import * as babelGenerator from '@babel/generator'
+import type * as babelGenerator from '@babel/generator'
 import { Comment, Node } from '../types'
 import { original, rangeWithWhitespace, source } from '../util/symbols'
 

src/babel/reprint.ts

@@ -66,7 +66,10 @@ export default async function runTransformOnFile({
 }: RunTransformOnFileOptions): Promise<TransformResult> {
   const transform: Transform = transformFile
     ? await import(transformFile)
-    : _transform
+    : _transform ??
+      ((): Transform => {
+        throw new Error('transformFile or transform is required')
+      })()
 
   const baseConfig = (await astxCosmiconfig.search(Path.dirname(file)))
     ?.config as AstxConfig | undefined