@atlaspack/optimizer-inline-requires 2.12.1-dev.3567 → 2.13.1-canary.3630

package/CHANGELOG.md

@@ -0,0 +1,15 @@
+# @atlaspack/optimizer-inline-requires
+
+## 2.13.0
+
+### Minor Changes
+
+- [#335](https://github.com/atlassian-labs/atlaspack/pull/335) [`b4dbd4d`](https://github.com/atlassian-labs/atlaspack/commit/b4dbd4d5b23d1b7aa3fcdf59cc7bc8bedd3a59cf) Thanks [@yamadapc](https://github.com/yamadapc)! - Initial changeset release
+
+### Patch Changes
+
+- Updated dependencies [[`b4dbd4d`](https://github.com/atlassian-labs/atlaspack/commit/b4dbd4d5b23d1b7aa3fcdf59cc7bc8bedd3a59cf)]:
+  - @atlaspack/feature-flags@2.13.0
+  - @atlaspack/plugin@2.13.0
+  - @atlaspack/rust@2.13.0
+  - @atlaspack/types@2.13.0

package/lib/InlineRequires.js

@@ -7,13 +7,6 @@ function _plugin() {
   };
   return data;
 }
-function _featureFlags() {
-  const data = require("@atlaspack/feature-flags");
-  _featureFlags = function () {
-    return data;
-  };
-  return data;
-}
 function _rust() {
   const data = require("@atlaspack/rust");
   _rust = function () {
@@ -21,14 +14,6 @@ function _rust() {
   };
   return data;
 }
-function _core() {
-  const data = require("@swc/core");
-  _core = function () {
-    return data;
-  };
-  return data;
-}
-var _RequireInliningVisitor = require("./RequireInliningVisitor");
 function _nullthrows() {
   const data = _interopRequireDefault(require("nullthrows"));
   _nullthrows = function () {
@@ -76,11 +61,10 @@ module.exports = new (_plugin().Optimizer)({
       assetPublicIdsWithSideEffects
     };
   },
-  async optimize({
+  optimize({
     bundle,
     contents,
     map: originalMap,
-    tracer,
     logger,
     bundleConfig,
     options
@@ -92,58 +76,24 @@ module.exports = new (_plugin().Optimizer)({
       };
     }
     try {
-      if ((0, _featureFlags().getFeatureFlag)('fastOptimizeInlineRequires')) {
-        let sourceMap = null;
-        const result = (0, _rust().runInlineRequiresOptimizer)({
-          code: contents.toString(),
-          sourceMaps: !!bundle.env.sourceMap,
-          ignoreModuleIds: Array.from(bundleConfig.assetPublicIdsWithSideEffects)
-        });
-        const sourceMapResult = result.sourceMap;
-        if (sourceMapResult != null) {
-          sourceMap = new (_sourceMap().default)(options.projectRoot);
-          sourceMap.addVLQMap(JSON.parse(sourceMapResult));
-          if (originalMap) {
-            sourceMap.extends(originalMap);
-          }
-        }
-        return {
-          contents: result.code,
-          map: originalMap
-        };
-      }
-      let measurement = tracer.createMeasurement('@atlaspack/optimizer-inline-requires', 'parse', bundle.name);
-      const ast = await (0, _core().parse)(contents.toString());
-      measurement && measurement.end();
-      const visitor = new _RequireInliningVisitor.RequireInliningVisitor({
-        bundle,
-        logger,
-        assetPublicIdsWithSideEffects: bundleConfig.assetPublicIdsWithSideEffects
+      let sourceMap = null;
+      const result = (0, _rust().runInlineRequiresOptimizer)({
+        code: contents.toString(),
+        sourceMaps: !!bundle.env.sourceMap,
+        ignoreModuleIds: Array.from(bundleConfig.assetPublicIdsWithSideEffects)
       });
-      measurement = tracer.createMeasurement('@atlaspack/optimizer-inline-requires', 'visit', bundle.name);
-      visitor.visitProgram(ast);
-      measurement && measurement.end();
-      if (visitor.dirty) {
-        const measurement = tracer.createMeasurement('@atlaspack/optimizer-inline-requires', 'print', bundle.name);
-        const result = await (0, _core().print)(ast, {
-          sourceMaps: !!bundle.env.sourceMap
-        });
-        measurement && measurement.end();
-        let sourceMap = null;
-        let resultMap = result.map;
-        let contents = (0, _nullthrows().default)(result.code);
-        if (resultMap != null) {
-          sourceMap = new (_sourceMap().default)(options.projectRoot);
-          sourceMap.addVLQMap(JSON.parse(resultMap));
-          if (originalMap) {
-            sourceMap.extends(originalMap);
-          }
+      const sourceMapResult = result.sourceMap;
+      if (sourceMapResult != null) {
+        sourceMap = new (_sourceMap().default)(options.projectRoot);
+        sourceMap.addVLQMap(JSON.parse(sourceMapResult));
+        if (originalMap) {
+          sourceMap.extends(originalMap);
         }
-        return {
-          contents,
-          map: sourceMap
-        };
       }
+      return {
+        contents: result.code,
+        map: sourceMap
+      };
     } catch (err) {
       logger.warn({
         origin: 'atlaspack-optimizer-experimental-inline-requires',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/optimizer-inline-requires",
-  "version": "2.12.1-dev.3567+c06f4487e",
+  "version": "2.13.1-canary.3630+10a6f37ef",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -12,17 +12,17 @@
   "main": "lib/InlineRequires.js",
   "source": "src/InlineRequires.js",
   "engines": {
-    "atlaspack": "^2.12.1-dev.3567+c06f4487e",
+    "atlaspack": "2.13.1-canary.3630+10a6f37ef",
     "node": ">= 14.0.0"
   },
   "dependencies": {
-    "@atlaspack/feature-flags": "2.12.1-dev.3567+c06f4487e",
-    "@atlaspack/plugin": "2.12.1-dev.3567+c06f4487e",
-    "@atlaspack/rust": "2.12.1-dev.3567+c06f4487e",
-    "@atlaspack/types": "2.12.1-dev.3567+c06f4487e",
+    "@atlaspack/feature-flags": "2.13.1-canary.3630+10a6f37ef",
+    "@atlaspack/plugin": "2.13.1-canary.3630+10a6f37ef",
+    "@atlaspack/rust": "2.13.1-canary.3630+10a6f37ef",
+    "@atlaspack/types": "2.13.1-canary.3630+10a6f37ef",
     "@parcel/source-map": "^2.1.1",
-    "@swc/core": "^1.7.26",
+    "@swc/core": "^1.10.0",
     "nullthrows": "^1.1.1"
   },
-  "gitHead": "c06f4487ebddc632957147c8b585a97e149062a1"
+  "gitHead": "10a6f37ef063d0227ebb26310383e899dbd9b1e6"
 }

package/src/InlineRequires.js

@@ -1,9 +1,6 @@
 // @flow strict-local
 import {Optimizer} from '@atlaspack/plugin';
-import {getFeatureFlag} from '@atlaspack/feature-flags';
 import {runInlineRequiresOptimizer} from '@atlaspack/rust';
-import {parse, print} from '@swc/core';
-import {RequireInliningVisitor} from './RequireInliningVisitor';
 import nullthrows from 'nullthrows';
 import SourceMap from '@parcel/source-map';
 
@@ -45,11 +42,10 @@ module.exports = new Optimizer<empty, BundleConfig>({
     return {assetPublicIdsWithSideEffects};
   },
 
-  async optimize({
+  optimize({
     bundle,
     contents,
     map: originalMap,
-    tracer,
     logger,
     bundleConfig,
     options,
@@ -59,72 +55,21 @@ module.exports = new Optimizer<empty, BundleConfig>({
     }
 
     try {
-      if (getFeatureFlag('fastOptimizeInlineRequires')) {
-        let sourceMap = null;
-        const result = runInlineRequiresOptimizer({
-          code: contents.toString(),
-          sourceMaps: !!bundle.env.sourceMap,
-          ignoreModuleIds: Array.from(
-            bundleConfig.assetPublicIdsWithSideEffects,
-          ),
-        });
-        const sourceMapResult = result.sourceMap;
-        if (sourceMapResult != null) {
-          sourceMap = new SourceMap(options.projectRoot);
-          sourceMap.addVLQMap(JSON.parse(sourceMapResult));
-          if (originalMap) {
-            sourceMap.extends(originalMap);
-          }
-        }
-        return {contents: result.code, map: originalMap};
-      }
-
-      let measurement = tracer.createMeasurement(
-        '@atlaspack/optimizer-inline-requires',
-        'parse',
-        bundle.name,
-      );
-      const ast = await parse(contents.toString());
-      measurement && measurement.end();
-
-      const visitor = new RequireInliningVisitor({
-        bundle,
-        logger,
-        assetPublicIdsWithSideEffects:
-          bundleConfig.assetPublicIdsWithSideEffects,
+      let sourceMap = null;
+      const result = runInlineRequiresOptimizer({
+        code: contents.toString(),
+        sourceMaps: !!bundle.env.sourceMap,
+        ignoreModuleIds: Array.from(bundleConfig.assetPublicIdsWithSideEffects),
       });
-
-      measurement = tracer.createMeasurement(
-        '@atlaspack/optimizer-inline-requires',
-        'visit',
-        bundle.name,
-      );
-      visitor.visitProgram(ast);
-      measurement && measurement.end();
-
-      if (visitor.dirty) {
-        const measurement = tracer.createMeasurement(
-          '@atlaspack/optimizer-inline-requires',
-          'print',
-          bundle.name,
-        );
-        const result = await print(ast, {sourceMaps: !!bundle.env.sourceMap});
-        measurement && measurement.end();
-
-        let sourceMap = null;
-        let resultMap = result.map;
-        let contents: string = nullthrows(result.code);
-
-        if (resultMap != null) {
-          sourceMap = new SourceMap(options.projectRoot);
-          sourceMap.addVLQMap(JSON.parse(resultMap));
-          if (originalMap) {
-            sourceMap.extends(originalMap);
-          }
+      const sourceMapResult = result.sourceMap;
+      if (sourceMapResult != null) {
+        sourceMap = new SourceMap(options.projectRoot);
+        sourceMap.addVLQMap(JSON.parse(sourceMapResult));
+        if (originalMap) {
+          sourceMap.extends(originalMap);
         }
-
-        return {contents, map: sourceMap};
       }
+      return {contents: result.code, map: sourceMap};
     } catch (err) {
       logger.warn({
         origin: 'atlaspack-optimizer-experimental-inline-requires',

package/lib/RequireInliningVisitor.js

@@ -1,194 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.RequireInliningVisitor = void 0;
-function _Visitor() {
-  const data = require("@swc/core/Visitor");
-  _Visitor = function () {
-    return data;
-  };
-  return data;
-}
-function _nullthrows() {
-  const data = _interopRequireDefault(require("nullthrows"));
-  _nullthrows = function () {
-    return data;
-  };
-  return data;
-}
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-class RequireInliningVisitor extends _Visitor().Visitor {
-  constructor({
-    bundle,
-    logger,
-    assetPublicIdsWithSideEffects
-  }) {
-    super();
-    this.currentModuleNode = null;
-    this.moduleVariables = new Set();
-    this.moduleVariableMap = new Map();
-    this.dirty = false;
-    this.logger = logger;
-    this.bundle = bundle;
-    this.assetPublicIdsWithSideEffects = assetPublicIdsWithSideEffects;
-  }
-  visitFunctionExpression(n) {
-    // This visitor tries to find module definition functions, these are of the form:
-    //
-    // parcelRequire.register("moduleId", function (require, module, exports) { ... });
-    //
-    // We do this to set the vistior variable `inModuleDefinition` for subsequent visits,
-    // and also reset the module variable tracking data structures.
-    //
-    // (TODO: Support arrow functions if we modify the runtime to output arrow functions)
-    if (n.params.length === 3 && n.params[0].pat.type === 'Identifier' && n.params[0].pat.value === 'require' && n.params[1].pat.type === 'Identifier' && n.params[1].pat.value === 'module' && n.params[2].pat.type === 'Identifier' && n.params[2].pat.value === 'exports') {
-      // `inModuleDefinition` is either null, or the module definition node
-      this.currentModuleNode = n;
-      this.moduleVariables = new Set();
-      this.moduleVariableMap = new Map();
-    }
-
-    // Make sure we visit the function itself
-    let result = super.visitFunctionExpression(n);
-
-    // only "exit" module definition if we're exiting the module definition node
-    if (n === this.currentModuleNode) {
-      this.currentModuleNode = null;
-    }
-    return result;
-  }
-  visitVariableDeclaration(n) {
-    // We're looking for variable declarations that look like this:
-    //
-    // `var $acw62 = require("acw62");`
-    let needsReplacement = false;
-    for (let i = 0; i < n.declarations.length; i++) {
-      let decl = n.declarations[i];
-      const init = decl.init;
-      if (!init || init.type !== 'CallExpression') {
-        continue;
-      }
-      if ((init.callee.type === 'Identifier' && init.callee.value === 'require' || init.callee.value === 'parcelRequire') && decl.id.value !== 'parcelHelpers' &&
-      // ignore var parcelHelpers = require("@atlaspack/transformer-js/src/esmodule-helpers.js");
-      init.arguments[0].expression.type === 'StringLiteral' && typeof decl.id.value === 'string' && decl.id.value.startsWith('$')) {
-        const variable = decl.id.value;
-        const assetPublicId = variable.substring(1);
-
-        // We need to determine whether the asset we're require'ing has sideEffects - if it does, we
-        // shouldn't optimise it to an inline require as the side effects need to run immediately
-        //
-        // We need to use the public id of the asset (which is the variable name used for requiring it) in
-        // order to find the asset in the bundle graph, and check whether `asset.sideEffects` is true - in
-        // which case we skip optimising this asset.
-        //
-        // This won't work in dev mode, because the id used to require the asset isn't the public id
-        if (this.assetPublicIdsWithSideEffects && this.assetPublicIdsWithSideEffects.has(assetPublicId)) {
-          continue;
-        }
-
-        // The moduleVariableMap contains a mapping from (e.g. $acw62 -> the AST node `require("acw62")`)
-        this.moduleVariableMap.set(variable, init);
-        // The moduleVariables set is just the used set of modules (e.g. `$acw62`)
-        this.moduleVariables.add(variable);
-
-        // Replace this with a null declarator, we'll use the `init` where it's declared.
-        //
-        // This mutates `var $acw62 = require("acw62")` -> `var $acw62 = null`
-        //
-        // The variable will be unused and removed by optimisation
-        decl.init = undefined;
-        needsReplacement = true;
-      } else if (decl.id.type === 'Identifier' && typeof decl.id.value === 'string' && decl.id.value.endsWith('Default') && decl.id.value.startsWith('$')) {
-        // Handle modules with default values, these look like this in the source:
-        // ```
-        // var _app = require("./App");
-        // var _appDefault = parcelHelpers.interopDefault(_app);
-        // ```
-        //
-        // In this case we want to also put `_appDefault` into the `moduleVariableMap` with the initializer node,
-        // but we want to replace `_app` in there with `require("./App")`.. to summarise, this code will end up looking like:
-        //
-        // ```
-        // var _app = null;
-        // var _appDefault = null;
-        // ```
-        //
-        // .. and where `_appDefault` is used we replace that with `parcelHelpers.interopDefault(require('./App'))`
-        const variable = decl.id.value;
-        const baseId = variable.substring(0, decl.id.value.length - 'Default'.length);
-        if (!this.moduleVariables.has(baseId)) {
-          continue;
-        }
-        init.arguments[0] = {
-          spread: undefined,
-          expression: (0, _nullthrows().default)(this.moduleVariableMap.get(baseId))
-        };
-        this.moduleVariableMap.set(variable, init);
-        this.moduleVariables.add(variable);
-        decl.init = undefined;
-        needsReplacement = true;
-      }
-    }
-    if (!needsReplacement) {
-      return super.visitVariableDeclaration(n);
-    } else {
-      this.dirty = true;
-      return n;
-    }
-  }
-  visitIdentifier(n) {
-    // This does the actual replacement - for any identifier within this factory function
-    // that is in the `moduleVariables` list, replace the identifier with the original expression
-    // that was going to be used to initialise the identifier.
-    //
-    // The replacement expression uses the `(0, require(...))` pattern to allow for safe replacement
-    // in any use cases (since we're replacing a variable access with a function call) - the minifier
-    // will take care of optimising this away where possible.
-    //
-    // e.g.
-    // var $abc = require("abc");
-    // console.log($abc.foo());
-    //
-    // becomes
-    //
-    // var $abc;
-    // console.log((0, require("abc")).foo);
-    //
-    if (this.moduleVariables.has(n.value)) {
-      // $FlowFixMe the types don't allow for swapping out the node type here, might need a different approach
-      return this.getReplacementExpression(n.value);
-    }
-    return super.visitIdentifier(n);
-  }
-
-  // Helper function to wrap an existing node in a sequence expression, that is, it
-  // will take a node `n` and return the AST for `(0, n)`
-  //
-  // This ensures that the require call can be correctly used in any context - where
-  // the sequence is redundant, the minifier will optimise it away.
-  getReplacementExpression(id) {
-    return {
-      type: 'ParenthesisExpression',
-      span: RequireInliningVisitor.getEmptySpan(),
-      expression: {
-        type: 'SequenceExpression',
-        span: RequireInliningVisitor.getEmptySpan(),
-        expressions: [{
-          type: 'NumericLiteral',
-          span: RequireInliningVisitor.getEmptySpan(),
-          value: 0
-        }, (0, _nullthrows().default)(this.moduleVariableMap.get(id))]
-      }
-    };
-  }
-  static getEmptySpan() {
-    return {
-      start: 0,
-      end: 0,
-      ctxt: 0
-    };
-  }
-}
-exports.RequireInliningVisitor = RequireInliningVisitor;

package/src/RequireInliningVisitor.js

@@ -1,226 +0,0 @@
-// @flow strict-local
-import type {NamedBundle, PluginLogger} from '@atlaspack/types';
-
-import type {
-  CallExpression,
-  VariableDeclaration,
-  FunctionExpression,
-  ParenthesisExpression,
-  Span,
-  Identifier,
-} from '@swc/core';
-import {Visitor} from '@swc/core/Visitor';
-import nullthrows from 'nullthrows';
-
-type VisitorOpts = {|
-  bundle: NamedBundle,
-  logger: PluginLogger,
-  assetPublicIdsWithSideEffects: Set<string>,
-|};
-
-export class RequireInliningVisitor extends Visitor {
-  currentModuleNode: null | FunctionExpression;
-  moduleVariables: Set<string>;
-  moduleVariableMap: Map<string, CallExpression>;
-  dirty: boolean;
-  logger: PluginLogger;
-  bundle: NamedBundle;
-  assetPublicIdsWithSideEffects: Set<string>;
-
-  constructor({bundle, logger, assetPublicIdsWithSideEffects}: VisitorOpts) {
-    super();
-    this.currentModuleNode = null;
-    this.moduleVariables = new Set();
-    this.moduleVariableMap = new Map();
-    this.dirty = false;
-    this.logger = logger;
-    this.bundle = bundle;
-    this.assetPublicIdsWithSideEffects = assetPublicIdsWithSideEffects;
-  }
-
-  visitFunctionExpression(n: FunctionExpression): FunctionExpression {
-    // This visitor tries to find module definition functions, these are of the form:
-    //
-    // parcelRequire.register("moduleId", function (require, module, exports) { ... });
-    //
-    // We do this to set the vistior variable `inModuleDefinition` for subsequent visits,
-    // and also reset the module variable tracking data structures.
-    //
-    // (TODO: Support arrow functions if we modify the runtime to output arrow functions)
-    if (
-      n.params.length === 3 &&
-      n.params[0].pat.type === 'Identifier' &&
-      n.params[0].pat.value === 'require' &&
-      n.params[1].pat.type === 'Identifier' &&
-      n.params[1].pat.value === 'module' &&
-      n.params[2].pat.type === 'Identifier' &&
-      n.params[2].pat.value === 'exports'
-    ) {
-      // `inModuleDefinition` is either null, or the module definition node
-      this.currentModuleNode = n;
-      this.moduleVariables = new Set();
-      this.moduleVariableMap = new Map();
-    }
-
-    // Make sure we visit the function itself
-    let result = super.visitFunctionExpression(n);
-
-    // only "exit" module definition if we're exiting the module definition node
-    if (n === this.currentModuleNode) {
-      this.currentModuleNode = null;
-    }
-    return result;
-  }
-
-  visitVariableDeclaration(n: VariableDeclaration): VariableDeclaration {
-    // We're looking for variable declarations that look like this:
-    //
-    // `var $acw62 = require("acw62");`
-    let needsReplacement = false;
-    for (let i = 0; i < n.declarations.length; i++) {
-      let decl = n.declarations[i];
-      const init = decl.init;
-      if (!init || init.type !== 'CallExpression') {
-        continue;
-      }
-
-      if (
-        ((init.callee.type === 'Identifier' &&
-          init.callee.value === 'require') ||
-          init.callee.value === 'parcelRequire') &&
-        decl.id.value !== 'parcelHelpers' && // ignore var parcelHelpers = require("@atlaspack/transformer-js/src/esmodule-helpers.js");
-        init.arguments[0].expression.type === 'StringLiteral' &&
-        typeof decl.id.value === 'string' &&
-        decl.id.value.startsWith('$')
-      ) {
-        const variable = decl.id.value;
-        const assetPublicId = variable.substring(1);
-
-        // We need to determine whether the asset we're require'ing has sideEffects - if it does, we
-        // shouldn't optimise it to an inline require as the side effects need to run immediately
-        //
-        // We need to use the public id of the asset (which is the variable name used for requiring it) in
-        // order to find the asset in the bundle graph, and check whether `asset.sideEffects` is true - in
-        // which case we skip optimising this asset.
-        //
-        // This won't work in dev mode, because the id used to require the asset isn't the public id
-        if (
-          this.assetPublicIdsWithSideEffects &&
-          this.assetPublicIdsWithSideEffects.has(assetPublicId)
-        ) {
-          continue;
-        }
-
-        // The moduleVariableMap contains a mapping from (e.g. $acw62 -> the AST node `require("acw62")`)
-        this.moduleVariableMap.set(variable, init);
-        // The moduleVariables set is just the used set of modules (e.g. `$acw62`)
-        this.moduleVariables.add(variable);
-
-        // Replace this with a null declarator, we'll use the `init` where it's declared.
-        //
-        // This mutates `var $acw62 = require("acw62")` -> `var $acw62 = null`
-        //
-        // The variable will be unused and removed by optimisation
-        decl.init = undefined;
-        needsReplacement = true;
-      } else if (
-        decl.id.type === 'Identifier' &&
-        typeof decl.id.value === 'string' &&
-        decl.id.value.endsWith('Default') &&
-        decl.id.value.startsWith('$')
-      ) {
-        // Handle modules with default values, these look like this in the source:
-        // ```
-        // var _app = require("./App");
-        // var _appDefault = parcelHelpers.interopDefault(_app);
-        // ```
-        //
-        // In this case we want to also put `_appDefault` into the `moduleVariableMap` with the initializer node,
-        // but we want to replace `_app` in there with `require("./App")`.. to summarise, this code will end up looking like:
-        //
-        // ```
-        // var _app = null;
-        // var _appDefault = null;
-        // ```
-        //
-        // .. and where `_appDefault` is used we replace that with `parcelHelpers.interopDefault(require('./App'))`
-        const variable = decl.id.value;
-        const baseId = variable.substring(
-          0,
-          decl.id.value.length - 'Default'.length,
-        );
-        if (!this.moduleVariables.has(baseId)) {
-          continue;
-        }
-        init.arguments[0] = {
-          spread: undefined,
-          expression: nullthrows(this.moduleVariableMap.get(baseId)),
-        };
-        this.moduleVariableMap.set(variable, init);
-        this.moduleVariables.add(variable);
-
-        decl.init = undefined;
-        needsReplacement = true;
-      }
-    }
-    if (!needsReplacement) {
-      return super.visitVariableDeclaration(n);
-    } else {
-      this.dirty = true;
-      return n;
-    }
-  }
-
-  visitIdentifier(n: Identifier): Identifier {
-    // This does the actual replacement - for any identifier within this factory function
-    // that is in the `moduleVariables` list, replace the identifier with the original expression
-    // that was going to be used to initialise the identifier.
-    //
-    // The replacement expression uses the `(0, require(...))` pattern to allow for safe replacement
-    // in any use cases (since we're replacing a variable access with a function call) - the minifier
-    // will take care of optimising this away where possible.
-    //
-    // e.g.
-    // var $abc = require("abc");
-    // console.log($abc.foo());
-    //
-    // becomes
-    //
-    // var $abc;
-    // console.log((0, require("abc")).foo);
-    //
-    if (this.moduleVariables.has(n.value)) {
-      // $FlowFixMe the types don't allow for swapping out the node type here, might need a different approach
-      return this.getReplacementExpression(n.value);
-    }
-    return super.visitIdentifier(n);
-  }
-
-  // Helper function to wrap an existing node in a sequence expression, that is, it
-  // will take a node `n` and return the AST for `(0, n)`
-  //
-  // This ensures that the require call can be correctly used in any context - where
-  // the sequence is redundant, the minifier will optimise it away.
-  getReplacementExpression(id: string): ParenthesisExpression {
-    return {
-      type: 'ParenthesisExpression',
-      span: RequireInliningVisitor.getEmptySpan(),
-      expression: {
-        type: 'SequenceExpression',
-        span: RequireInliningVisitor.getEmptySpan(),
-        expressions: [
-          {
-            type: 'NumericLiteral',
-            span: RequireInliningVisitor.getEmptySpan(),
-            value: 0,
-          },
-          nullthrows(this.moduleVariableMap.get(id)),
-        ],
-      },
-    };
-  }
-
-  static getEmptySpan(): Span {
-    return {start: 0, end: 0, ctxt: 0};
-  }
-}

package/test/RequireInliningVisitor.test.js

@@ -1,81 +0,0 @@
-import {parse, print} from '@swc/core';
-import {RequireInliningVisitor} from '../src/RequireInliningVisitor';
-import assert from 'assert';
-import logger from '@atlaspack/logger';
-
-async function testRequireInliningVisitor(src, sideEffects) {
-  const ast = await parse(src, {});
-  const assetPublicIdsWithSideEffects = new Set(sideEffects);
-
-  const visitor = new RequireInliningVisitor({
-    bundle: {
-      name: 'test-bundle',
-    },
-    assetPublicIdsWithSideEffects,
-    logger,
-  });
-  visitor.visitProgram(ast);
-  return (await print(ast)).code;
-}
-
-function getModule(body) {
-  return `
-    n.register('def456', function(require, module, exports) {
-        ${body}
-    });
-    `;
-}
-
-function normaliseCode(code) {
-  return code
-    .split('\n')
-    .map((line) => line.trim())
-    .join('\n')
-    .trim();
-}
-
-function assertEqualCode(left, right) {
-  assert.equal(normaliseCode(left), normaliseCode(right));
-}
-
-describe('InliningVisitor', () => {
-  it('performs basic inlining', async () => {
-    const src = getModule(`
-        var $abc123 = require('abc123');
-        console.log($abc123);`);
-    const result = await testRequireInliningVisitor(src, []);
-    assertEqualCode(
-      result,
-      getModule(`var $abc123;
-      console.log((0, require('abc123')));`),
-    );
-  });
-
-  it('performs module default inlining', async () => {
-    const src = getModule(
-      `var $abc123 = require('abc123');
-        var $abc123Default =  parcelHelpers.interopDefault($abc123);
-        console.log($abc123Default.foo());`,
-    );
-    const expected = getModule(
-      `var $abc123;
-        var $abc123Default;
-        console.log((0, parcelHelpers.interopDefault(require('abc123'))).foo());`,
-    );
-    const result = await testRequireInliningVisitor(src, []);
-    assertEqualCode(result, expected);
-  });
-
-  it('ignores assets with sideEffects', async () => {
-    const src = getModule(`var $abc123 = require('abc123');
-        var $abc456 = require('abc456');
-        console.log($abc123);
-        console.log($abc456);`);
-    const expected = getModule(`var $abc123 = require('abc123');
-        var $abc456;
-        console.log($abc123);
-        console.log((0, require('abc456')));`);
-    const result = await testRequireInliningVisitor(src, ['abc123']);
-    assertEqualCode(result, expected);
-  });
-});