@knighted/module 1.1.0 → 1.2.0

package/README.md

@@ -135,6 +135,7 @@ type ModuleOptions = {
   idiomaticExports?: 'off' | 'safe' | 'aggressive'
   topLevelAwait?: 'error' | 'wrap' | 'preserve'
   out?: string
+  cwd?: string
   inPlace?: boolean
 }
 ```
@@ -159,7 +160,7 @@ type ModuleOptions = {
 - `cjsDefault` (`auto`): bundler-style default interop vs direct `module.exports`.
 - `idiomaticExports` (`safe`): when raising CJS to ESM, attempt to synthesize `export` statements directly when it is safe. `off` always uses the helper bag; `aggressive` currently matches `safe` heuristics.
 - `out`/`inPlace`: write the transformed code to a file; otherwise the function returns the transformed string only.
-- CommonJS → ESM lowering will throw on `with` statements and unshadowed `eval` calls to avoid unsound rewrites.
+- `cwd` (`process.cwd()`): Base directory used to resolve relative `out` paths.
 
 > [!NOTE]
 > Package-level metadata (`package.json` updates such as setting `"type": "module"` or authoring `exports`) is not edited by this tool today; plan that change outside the per-file transform.
@@ -229,9 +230,3 @@ for (const file of files) {
 ```
 
 This pre-`tsc` step removes the flagged globals in the compiled orientation; runtime semantics still match the target build.
-
-## Roadmap
-
-- Emit source maps and clearer diagnostics for transform choices.
-- Broaden fixtures covering live-binding and top-level await edge cases across Node versions.
-- Benchmark scope analysis choices: compare `periscopic`, `scope-analyzer`, and `eslint-scope` on fixtures and pick the final adapter.

package/dist/ast.d.ts

@@ -0,0 +1,39 @@
+import type { Node } from 'oxc-parser';
+import type { CjsExport } from '../types.js';
+export type IdentifierNode = Extract<Node, {
+    type: 'Identifier';
+}>;
+export type LiteralNode = Extract<Node, {
+    type: 'Literal';
+    value?: unknown;
+}>;
+export type MemberExpressionNode = Extract<Node, {
+    type: 'MemberExpression';
+}>;
+export type CallExpressionNode = Extract<Node, {
+    type: 'CallExpression';
+}>;
+export type ProgramNode = Extract<Node, {
+    type: 'Program';
+    body: Node[];
+}>;
+export type ModuleExportNameNode = Extract<Node, {
+    type: 'Identifier' | 'Literal';
+}>;
+export type ImportDefaultSpecifierNode = Extract<Node, {
+    type: 'ImportDefaultSpecifier';
+}>;
+export type ImportNamespaceSpecifierNode = Extract<Node, {
+    type: 'ImportNamespaceSpecifier';
+}>;
+export type ImportSpecifierNode = Extract<Node, {
+    type: 'ImportSpecifier';
+}>;
+export type ExportsMap = Map<string, CjsExport> & {
+    hasUnsupportedExportWrite?: boolean;
+};
+export declare const isAstNode: (value: unknown) => value is Node;
+export declare const isIdentifierNode: (node: Node | null | undefined) => node is IdentifierNode;
+export declare const isMemberExpressionNode: (node: Node | null | undefined) => node is MemberExpressionNode;
+export declare const isCallExpressionNode: (node: Node | null | undefined) => node is CallExpressionNode;
+export declare const getModuleExportName: (name: ModuleExportNameNode | null | undefined) => string | null;

package/dist/cjs/ast.d.cts

@@ -0,0 +1,39 @@
+import type { Node } from 'oxc-parser';
+import type { CjsExport } from '../types.cjs';
+export type IdentifierNode = Extract<Node, {
+    type: 'Identifier';
+}>;
+export type LiteralNode = Extract<Node, {
+    type: 'Literal';
+    value?: unknown;
+}>;
+export type MemberExpressionNode = Extract<Node, {
+    type: 'MemberExpression';
+}>;
+export type CallExpressionNode = Extract<Node, {
+    type: 'CallExpression';
+}>;
+export type ProgramNode = Extract<Node, {
+    type: 'Program';
+    body: Node[];
+}>;
+export type ModuleExportNameNode = Extract<Node, {
+    type: 'Identifier' | 'Literal';
+}>;
+export type ImportDefaultSpecifierNode = Extract<Node, {
+    type: 'ImportDefaultSpecifier';
+}>;
+export type ImportNamespaceSpecifierNode = Extract<Node, {
+    type: 'ImportNamespaceSpecifier';
+}>;
+export type ImportSpecifierNode = Extract<Node, {
+    type: 'ImportSpecifier';
+}>;
+export type ExportsMap = Map<string, CjsExport> & {
+    hasUnsupportedExportWrite?: boolean;
+};
+export declare const isAstNode: (value: unknown) => value is Node;
+export declare const isIdentifierNode: (node: Node | null | undefined) => node is IdentifierNode;
+export declare const isMemberExpressionNode: (node: Node | null | undefined) => node is MemberExpressionNode;
+export declare const isCallExpressionNode: (node: Node | null | undefined) => node is CallExpressionNode;
+export declare const getModuleExportName: (name: ModuleExportNameNode | null | undefined) => string | null;

package/dist/cjs/exports.d.cts

@@ -2,5 +2,8 @@ import type { Node } from 'oxc-parser';
 import type { CjsExport } from '../types.cjs';
 declare const exportsRename = "__exports";
 declare const requireMainRgx: RegExp;
-declare const collectCjsExports: (ast: Node) => Promise<Map<string, CjsExport>>;
+type ExportsMap = Map<string, CjsExport> & {
+    hasUnsupportedExportWrite?: boolean;
+};
+declare const collectCjsExports: (ast: Node) => Promise<ExportsMap>;
 export { exportsRename, requireMainRgx, collectCjsExports };

package/dist/cjs/format.cjs

@@ -4,33 +4,35 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.format = void 0;
+var _ast = require("./helpers/ast.cjs");
 var _magicString = _interopRequireDefault(require("magic-string"));
-var _identifier = require("#formatters/identifier.js");
-var _metaProperty = require("#formatters/metaProperty.js");
-var _memberExpression = require("#formatters/memberExpression.js");
-var _assignmentExpression = require("#formatters/assignmentExpression.js");
-var _url = require("#utils/url.js");
-var _exports = require("#utils/exports.js");
-var _identifiers = require("#utils/identifiers.js");
-var _identifier2 = require("#helpers/identifier.js");
-var _walk = require("#walk");
+var _identifier = require("./formatters/identifier.cjs");
+var _metaProperty = require("./formatters/metaProperty.cjs");
+var _memberExpression = require("./formatters/memberExpression.cjs");
+var _assignmentExpression = require("./formatters/assignmentExpression.cjs");
+var _url = require("./utils/url.cjs");
+var _exports = require("./utils/exports.cjs");
+var _identifiers = require("./utils/identifiers.cjs");
+var _identifier2 = require("./helpers/identifier.cjs");
+var _walk = require("./walk.cjs");
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const isValidIdent = name => /^[$A-Z_a-z][$\w]*$/.test(name);
 const expressionHasRequireCall = (node, shadowed) => {
   let found = false;
   const walkNode = n => {
-    if (!n || found) return;
-    if (n.type === 'CallExpression' && n.callee?.type === 'Identifier' && n.callee.name === 'require' && !shadowed.has('require')) {
+    if (!(0, _ast.isAstNode)(n) || found) return;
+    if ((0, _ast.isCallExpressionNode)(n) && (0, _ast.isIdentifierNode)(n.callee) && n.callee.name === 'require' && !shadowed.has('require')) {
       found = true;
       return;
     }
-    if (n.type === 'CallExpression' && n.callee?.type === 'MemberExpression' && n.callee.object?.type === 'Identifier' && n.callee.object.name === 'require' && !shadowed.has('require')) {
+    if ((0, _ast.isCallExpressionNode)(n) && (0, _ast.isMemberExpressionNode)(n.callee) && (0, _ast.isIdentifierNode)(n.callee.object) && n.callee.object.name === 'require' && !shadowed.has('require')) {
       found = true;
       return;
     }
-    const keys = Object.keys(n);
+    const record = n;
+    const keys = Object.keys(record);
     for (const key of keys) {
-      const value = n[key];
+      const value = record[key];
       if (!value) continue;
       if (Array.isArray(value)) {
         for (const item of value) {
@@ -87,6 +89,10 @@ const lowerCjsRequireToImports = (program, code, shadowed) => {
       if (allStatic) {
         for (const decl of decls) {
           const init = decl.init;
+          if (!init || !(0, _ast.isCallExpressionNode)(init)) {
+            needsCreateRequire = true;
+            continue;
+          }
           const arg = init.arguments[0];
           const source = code.slice(arg.start, arg.end);
           const value = arg.value;
@@ -123,6 +129,10 @@ const lowerCjsRequireToImports = (program, code, shadowed) => {
     if (stmt.type === 'ExpressionStatement') {
       const expr = stmt.expression;
       if (expr && isStaticRequire(expr, shadowed)) {
+        if (!(0, _ast.isCallExpressionNode)(expr)) {
+          needsCreateRequire = true;
+          continue;
+        }
         const arg = expr.arguments[0];
         const source = code.slice(arg.start, arg.end);
         const value = arg.value;
@@ -149,11 +159,12 @@ const lowerCjsRequireToImports = (program, code, shadowed) => {
     needsInteropHelper
   };
 };
-const isRequireMainMember = (node, shadowed) => node && node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'require' && !shadowed.has('require') && node.property.type === 'Identifier' && node.property.name === 'main';
+const isRequireMainMember = (node, shadowed) => node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'require' && !shadowed.has('require') && node.property.type === 'Identifier' && node.property.name === 'main';
 const hasTopLevelAwait = program => {
   let found = false;
   const walkNode = (node, inFunction) => {
     if (found) return;
+    if (!(0, _ast.isAstNode)(node)) return;
     switch (node.type) {
       case 'FunctionDeclaration':
       case 'FunctionExpression':
@@ -167,9 +178,10 @@ const hasTopLevelAwait = program => {
       found = true;
       return;
     }
-    const keys = Object.keys(node);
+    const record = node;
+    const keys = Object.keys(record);
     for (const key of keys) {
-      const value = node[key];
+      const value = record[key];
       if (!value) continue;
       if (Array.isArray(value)) {
         for (const item of value) {
@@ -250,7 +262,8 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
       }
       if (namedSpecs.length) {
         const pairs = namedSpecs.map(s => {
-          const imported = s.imported.name;
+          const imported = (0, _ast.getModuleExportName)(s.imported);
+          if (!imported) return s.local.name;
           const local = s.local.name;
           return imported === local ? imported : `${imported}: ${local}`;
         });
@@ -275,7 +288,7 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
               exportedNames.push(d.id.name);
             }
           }
-        } else if (decl.id?.type === 'Identifier') {
+        } else if ('id' in decl && decl.id?.type === 'Identifier') {
           exportedNames.push(decl.id.name);
         }
         const exportLines = exportedNames.map(name => exportAssignment(name, name, live));
@@ -295,8 +308,9 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
           const lines = [`const ${modIdent} = require(${srcLiteral});`];
           for (const spec of node.specifiers) {
             if (spec.type !== 'ExportSpecifier') continue;
-            const exported = spec.exported.name;
-            const imported = spec.local.name;
+            const exported = (0, _ast.getModuleExportName)(spec.exported);
+            const imported = (0, _ast.getModuleExportName)(spec.local);
+            if (!exported || !imported) continue;
             let rhs = `${modIdent}.${imported}`;
             if (imported === 'default') {
               rhs = `${defaultInteropName}(${modIdent})`;
@@ -314,8 +328,9 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
           const lines = [];
           for (const spec of node.specifiers) {
             if (spec.type !== 'ExportSpecifier') continue;
-            const exported = spec.exported.name;
-            const local = spec.local.name;
+            const exported = (0, _ast.getModuleExportName)(spec.exported);
+            const local = (0, _ast.getModuleExportName)(spec.local);
+            if (!exported || !local) continue;
             lines.push(exportAssignment(exported, local, live));
           }
           exportTransforms.push({
@@ -359,8 +374,11 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
     }
     if (node.type === 'ExportAllDeclaration') {
       const srcLiteral = code.slice(node.source.start, node.source.end);
-      if (node.exported) {
-        const exported = node.exported.name;
+      if ('exported' in node && node.exported) {
+        const exported = (0, _ast.getModuleExportName)(node.exported);
+        if (!exported) {
+          continue;
+        }
         const modIdent = `__mod${importIndex++}`;
         const lines = [`const ${modIdent} = require(${srcLiteral});`, exportAssignment(exported, modIdent, live)];
         exportTransforms.push({

package/dist/cjs/formatters/assignmentExpression.cjs

@@ -4,7 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.assignmentExpression = void 0;
-var _walk = require("#walk");
+var _walk = require("../walk.cjs");
 const assignmentExpression = async ({
   node,
   parent: _parent,

package/dist/cjs/formatters/identifier.cjs

@@ -4,8 +4,8 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.identifier = void 0;
-var _exports = require("#utils/exports.js");
-var _identifier = require("#helpers/identifier.js");
+var _exports = require("../utils/exports.cjs");
+var _identifier = require("../helpers/identifier.cjs");
 const identifier = ({
   node,
   ancestors,

package/dist/cjs/formatters/memberExpression.cjs

@@ -4,7 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.memberExpression = void 0;
-var _exports = require("#utils/exports.js");
+var _exports = require("../utils/exports.cjs");
 const memberExpression = (node, parent, src, options, shadowed, extras, useExportsBag = true, rewriteExports = true) => {
   if (options.target === 'module') {
     if (rewriteExports && !useExportsBag) {

package/dist/cjs/helpers/ast.cjs

@@ -0,0 +1,29 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.isMemberExpressionNode = exports.isIdentifierNode = exports.isCallExpressionNode = exports.isAstNode = exports.getModuleExportName = void 0;
+const isAstNode = value => {
+  return Boolean(value) && typeof value === 'object' && 'type' in value;
+};
+exports.isAstNode = isAstNode;
+const isIdentifierNode = node => {
+  return node?.type === 'Identifier';
+};
+exports.isIdentifierNode = isIdentifierNode;
+const isMemberExpressionNode = node => {
+  return node?.type === 'MemberExpression';
+};
+exports.isMemberExpressionNode = isMemberExpressionNode;
+const isCallExpressionNode = node => {
+  return node?.type === 'CallExpression';
+};
+exports.isCallExpressionNode = isCallExpressionNode;
+const getModuleExportName = name => {
+  if (!name) return null;
+  if (name.type === 'Identifier') return name.name;
+  if (name.type === 'Literal' && typeof name.value === 'string') return name.value;
+  return null;
+};
+exports.getModuleExportName = getModuleExportName;

package/dist/cjs/helpers/identifier.cjs

@@ -127,14 +127,16 @@ const identifier = exports.identifier = {
     const node = ancestors[ancestors.length - 1];
     const varBoundScopes = ['ClassDeclaration', 'ClassExpression', 'FunctionDeclaration', 'FunctionExpression', 'ArrowFunctionExpression'];
     const declaratorIndex = ancestors.findIndex(ancestor => {
-      return ancestor.type === 'VariableDeclarator' && (ancestor === node || isInBindingPattern(ancestor.id, node));
+      if (ancestor.type !== 'VariableDeclarator') return false;
+      return ancestor === node || isInBindingPattern(ancestor.id, node);
     });
     if (declaratorIndex === -1) return false;
-    const declarator = ancestors[declaratorIndex];
+    const declaratorNode = ancestors[declaratorIndex];
     const declaration = ancestors[declaratorIndex - 1];
+    if (declaratorNode?.type !== 'VariableDeclarator') return false;
     return declaration?.type === 'VariableDeclaration' && declaration.kind === 'var' && ancestors.every(ancestor => {
       return !varBoundScopes.includes(ancestor.type);
-    }) && (declarator.id === node || isInBindingPattern(declarator.id, node));
+    }) && (declaratorNode.id === node || isInBindingPattern(declaratorNode.id, node));
   },
   isIife(ancestors) {
     const parent = ancestors[ancestors.length - 2];

package/dist/cjs/module.cjs

@@ -7,11 +7,11 @@ exports.transform = void 0;
 var _nodePath = require("node:path");
 var _promises = require("node:fs/promises");
 var _specifier = require("./specifier.cjs");
-var _parse = require("#parse");
-var _format = require("#format");
-var _lang = require("#utils/lang.js");
+var _parse = require("./parse.cjs");
+var _format = require("./format.cjs");
+var _lang = require("./utils/lang.cjs");
 var _nodeModule = require("node:module");
-var _walk = require("#walk");
+var _walk = require("./walk.cjs");
 const collapseSpecifier = value => value.replace(/['"`+)\s]|new String\(/g, '');
 const builtinSpecifiers = new Set(_nodeModule.builtinModules.map(mod => mod.startsWith('node:') ? mod.slice(5) : mod).flatMap(mod => {
   const parts = mod.split('/');
@@ -163,6 +163,7 @@ const defaultOptions = {
   idiomaticExports: 'safe',
   importMetaPrelude: 'auto',
   topLevelAwait: 'error',
+  cwd: undefined,
   out: undefined,
   inPlace: false
 };
@@ -172,10 +173,11 @@ const transform = async (filename, options = defaultOptions) => {
     ...options,
     filePath: filename
   };
+  const cwdBase = opts.cwd ? (0, _nodePath.resolve)(opts.cwd) : process.cwd();
   const appendMode = options?.appendJsExtension ?? (opts.target === 'module' ? 'relative-only' : 'off');
   const dirIndex = opts.appendDirectoryIndex === undefined ? 'index.js' : opts.appendDirectoryIndex;
   const detectCycles = opts.detectCircularRequires ?? 'off';
-  const file = (0, _nodePath.resolve)(filename);
+  const file = (0, _nodePath.resolve)(cwdBase, filename);
   const code = (await (0, _promises.readFile)(file)).toString();
   const ast = (0, _parse.parse)(filename, code);
   let source = await (0, _format.format)(code, ast, opts);
@@ -192,7 +194,7 @@ const transform = async (filename, options = defaultOptions) => {
   if (detectCycles !== 'off' && opts.target === 'module' && opts.transformSyntax) {
     await detectCircularRequireGraph(file, detectCycles, dirIndex || 'index.js');
   }
-  const outputPath = opts.inPlace ? file : opts.out ? (0, _nodePath.resolve)(opts.out) : undefined;
+  const outputPath = opts.inPlace ? file : opts.out ? (0, _nodePath.resolve)(cwdBase, opts.out) : undefined;
   if (outputPath) {
     await (0, _promises.writeFile)(outputPath, source);
   }

package/dist/cjs/specifier.cjs

@@ -8,7 +8,7 @@ var _nodePath = require("node:path");
 var _promises = require("node:fs/promises");
 var _magicString = _interopRequireDefault(require("magic-string"));
 var _oxcParser = require("oxc-parser");
-var _walk = require("#walk");
+var _walk = require("./walk.cjs");
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const isStringLiteral = node => {
   return node.type === 'Literal' && typeof node.value === 'string';

package/dist/cjs/types.d.cts

@@ -48,6 +48,8 @@ export type ModuleOptions = {
     diagnostics?: (diag: Diagnostic) => void;
     /** Optional source file path used for diagnostics context. */
     filePath?: string;
+    /** Base directory used to resolve relative `out` paths; defaults to process.cwd(). */
+    cwd?: string;
     /** Output directory or file path when writing. */
     out?: string;
     /** Overwrite input files instead of writing to out. */

package/dist/cjs/utils/exports.cjs

@@ -4,7 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.requireMainRgx = exports.exportsRename = exports.collectCjsExports = void 0;
-var _walk = require("#walk");
+var _walk = require("../walk.cjs");
 const exportsRename = exports.exportsRename = '__exports';
 const requireMainRgx = exports.requireMainRgx = /(require\.main\s*===\s*module|module\s*===\s*require\.main)/g;
 const literalPropName = (prop, literals) => {

package/dist/cjs/utils/identifiers.cjs

@@ -4,8 +4,8 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.collectScopeIdentifiers = exports.collectModuleIdentifiers = void 0;
-var _walk = require("#walk");
-var _identifier = require("#helpers/identifier.js");
+var _walk = require("../walk.cjs");
+var _identifier = require("../helpers/identifier.cjs");
 var _scopeNodes = require("./scopeNodes.cjs");
 const addBindingNames = (pattern, into) => {
   if (!pattern) return;

package/dist/exports.d.ts

@@ -2,5 +2,8 @@ import type { Node } from 'oxc-parser';
 import type { CjsExport } from '../types.js';
 declare const exportsRename = "__exports";
 declare const requireMainRgx: RegExp;
-declare const collectCjsExports: (ast: Node) => Promise<Map<string, CjsExport>>;
+type ExportsMap = Map<string, CjsExport> & {
+    hasUnsupportedExportWrite?: boolean;
+};
+declare const collectCjsExports: (ast: Node) => Promise<ExportsMap>;
 export { exportsRename, requireMainRgx, collectCjsExports };

package/dist/format.js

@@ -1,29 +1,31 @@
+import { getModuleExportName, isAstNode, isCallExpressionNode, isIdentifierNode, isMemberExpressionNode } from './helpers/ast.js';
 import MagicString from 'magic-string';
-import { identifier } from '#formatters/identifier.js';
-import { metaProperty } from '#formatters/metaProperty.js';
-import { memberExpression } from '#formatters/memberExpression.js';
-import { assignmentExpression } from '#formatters/assignmentExpression.js';
-import { isValidUrl } from '#utils/url.js';
-import { exportsRename, collectCjsExports } from '#utils/exports.js';
-import { collectModuleIdentifiers } from '#utils/identifiers.js';
-import { isIdentifierName } from '#helpers/identifier.js';
-import { ancestorWalk } from '#walk';
+import { identifier } from './formatters/identifier.js';
+import { metaProperty } from './formatters/metaProperty.js';
+import { memberExpression } from './formatters/memberExpression.js';
+import { assignmentExpression } from './formatters/assignmentExpression.js';
+import { isValidUrl } from './utils/url.js';
+import { exportsRename, collectCjsExports } from './utils/exports.js';
+import { collectModuleIdentifiers } from './utils/identifiers.js';
+import { isIdentifierName } from './helpers/identifier.js';
+import { ancestorWalk } from './walk.js';
 const isValidIdent = name => /^[$A-Z_a-z][$\w]*$/.test(name);
 const expressionHasRequireCall = (node, shadowed) => {
   let found = false;
   const walkNode = n => {
-    if (!n || found) return;
-    if (n.type === 'CallExpression' && n.callee?.type === 'Identifier' && n.callee.name === 'require' && !shadowed.has('require')) {
+    if (!isAstNode(n) || found) return;
+    if (isCallExpressionNode(n) && isIdentifierNode(n.callee) && n.callee.name === 'require' && !shadowed.has('require')) {
       found = true;
       return;
     }
-    if (n.type === 'CallExpression' && n.callee?.type === 'MemberExpression' && n.callee.object?.type === 'Identifier' && n.callee.object.name === 'require' && !shadowed.has('require')) {
+    if (isCallExpressionNode(n) && isMemberExpressionNode(n.callee) && isIdentifierNode(n.callee.object) && n.callee.object.name === 'require' && !shadowed.has('require')) {
       found = true;
       return;
     }
-    const keys = Object.keys(n);
+    const record = n;
+    const keys = Object.keys(record);
     for (const key of keys) {
-      const value = n[key];
+      const value = record[key];
       if (!value) continue;
       if (Array.isArray(value)) {
         for (const item of value) {
@@ -80,6 +82,10 @@ const lowerCjsRequireToImports = (program, code, shadowed) => {
       if (allStatic) {
         for (const decl of decls) {
           const init = decl.init;
+          if (!init || !isCallExpressionNode(init)) {
+            needsCreateRequire = true;
+            continue;
+          }
           const arg = init.arguments[0];
           const source = code.slice(arg.start, arg.end);
           const value = arg.value;
@@ -116,6 +122,10 @@ const lowerCjsRequireToImports = (program, code, shadowed) => {
     if (stmt.type === 'ExpressionStatement') {
       const expr = stmt.expression;
       if (expr && isStaticRequire(expr, shadowed)) {
+        if (!isCallExpressionNode(expr)) {
+          needsCreateRequire = true;
+          continue;
+        }
         const arg = expr.arguments[0];
         const source = code.slice(arg.start, arg.end);
         const value = arg.value;
@@ -142,11 +152,12 @@ const lowerCjsRequireToImports = (program, code, shadowed) => {
     needsInteropHelper
   };
 };
-const isRequireMainMember = (node, shadowed) => node && node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'require' && !shadowed.has('require') && node.property.type === 'Identifier' && node.property.name === 'main';
+const isRequireMainMember = (node, shadowed) => node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'require' && !shadowed.has('require') && node.property.type === 'Identifier' && node.property.name === 'main';
 const hasTopLevelAwait = program => {
   let found = false;
   const walkNode = (node, inFunction) => {
     if (found) return;
+    if (!isAstNode(node)) return;
     switch (node.type) {
       case 'FunctionDeclaration':
       case 'FunctionExpression':
@@ -160,9 +171,10 @@ const hasTopLevelAwait = program => {
       found = true;
       return;
     }
-    const keys = Object.keys(node);
+    const record = node;
+    const keys = Object.keys(record);
     for (const key of keys) {
-      const value = node[key];
+      const value = record[key];
       if (!value) continue;
       if (Array.isArray(value)) {
         for (const item of value) {
@@ -243,7 +255,8 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
       }
       if (namedSpecs.length) {
         const pairs = namedSpecs.map(s => {
-          const imported = s.imported.name;
+          const imported = getModuleExportName(s.imported);
+          if (!imported) return s.local.name;
           const local = s.local.name;
           return imported === local ? imported : `${imported}: ${local}`;
         });
@@ -268,7 +281,7 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
               exportedNames.push(d.id.name);
             }
           }
-        } else if (decl.id?.type === 'Identifier') {
+        } else if ('id' in decl && decl.id?.type === 'Identifier') {
           exportedNames.push(decl.id.name);
         }
         const exportLines = exportedNames.map(name => exportAssignment(name, name, live));
@@ -288,8 +301,9 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
           const lines = [`const ${modIdent} = require(${srcLiteral});`];
           for (const spec of node.specifiers) {
             if (spec.type !== 'ExportSpecifier') continue;
-            const exported = spec.exported.name;
-            const imported = spec.local.name;
+            const exported = getModuleExportName(spec.exported);
+            const imported = getModuleExportName(spec.local);
+            if (!exported || !imported) continue;
             let rhs = `${modIdent}.${imported}`;
             if (imported === 'default') {
               rhs = `${defaultInteropName}(${modIdent})`;
@@ -307,8 +321,9 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
           const lines = [];
           for (const spec of node.specifiers) {
             if (spec.type !== 'ExportSpecifier') continue;
-            const exported = spec.exported.name;
-            const local = spec.local.name;
+            const exported = getModuleExportName(spec.exported);
+            const local = getModuleExportName(spec.local);
+            if (!exported || !local) continue;
             lines.push(exportAssignment(exported, local, live));
           }
           exportTransforms.push({
@@ -352,8 +367,11 @@ const lowerEsmToCjs = (program, code, opts, containsTopLevelAwait) => {
     }
     if (node.type === 'ExportAllDeclaration') {
       const srcLiteral = code.slice(node.source.start, node.source.end);
-      if (node.exported) {
-        const exported = node.exported.name;
+      if ('exported' in node && node.exported) {
+        const exported = getModuleExportName(node.exported);
+        if (!exported) {
+          continue;
+        }
         const modIdent = `__mod${importIndex++}`;
         const lines = [`const ${modIdent} = require(${srcLiteral});`, exportAssignment(exported, modIdent, live)];
         exportTransforms.push({

package/dist/formatters/assignmentExpression.js

@@ -1,4 +1,4 @@
-import { walk } from '#walk';
+import { walk } from '../walk.js';
 export const assignmentExpression = async ({
   node,
   parent: _parent,

package/dist/formatters/identifier.js

@@ -1,5 +1,5 @@
-import { exportsRename } from '#utils/exports.js';
-import { identifier as ident } from '#helpers/identifier.js';
+import { exportsRename } from '../utils/exports.js';
+import { identifier as ident } from '../helpers/identifier.js';
 export const identifier = ({
   node,
   ancestors,

package/dist/formatters/memberExpression.js

@@ -1,4 +1,4 @@
-import { exportsRename } from '#utils/exports.js';
+import { exportsRename } from '../utils/exports.js';
 export const memberExpression = (node, parent, src, options, shadowed, extras, useExportsBag = true, rewriteExports = true) => {
   if (options.target === 'module') {
     if (rewriteExports && !useExportsBag) {