@atlaspack/packager-js 2.12.1-dev.3401 → 2.12.1-dev.3450

package/lib/ScopeHoistingPackager.js

@@ -53,6 +53,13 @@ function _path() {
   };
   return data;
 }
+function _featureFlags() {
+  const data = require("@atlaspack/feature-flags");
+  _featureFlags = function () {
+    return data;
+  };
+  return data;
+}
 var _ESMOutputFormat = require("./ESMOutputFormat");
 var _CJSOutputFormat = require("./CJSOutputFormat");
 var _GlobalOutputFormat = require("./GlobalOutputFormat");
@@ -63,7 +70,10 @@ function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e;
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 // General regex used to replace imports with the resolved code, references with resolutions,
 // and count the number of newlines in the file for source maps.
-const REPLACEMENT_RE = /\n|import\s+"([0-9a-f]{16}:.+?)";|(?:\$[0-9a-f]{16}\$exports)|(?:\$[0-9a-f]{16}\$(?:import|importAsync|require)\$[0-9a-f]+(?:\$[0-9a-f]+)?)/g;
+//
+// For conditional bundling the only difference in this regex is adding `importCond` where we have `importAsync` etc..
+const REPLACEMENT_RE_CONDITIONAL = /\n|import\s+"([0-9a-f]{16,20}:.+?)";|(?:\$[0-9a-f]{16,20}\$exports)|(?:\$[0-9a-f]{16,20}\$(?:import|importAsync|require|importCond)\$[0-9a-f]+(?:\$[0-9a-f]+)?)/g;
+const REPLACEMENT_RE = /\n|import\s+"([0-9a-f]{16,20}:.+?)";|(?:\$[0-9a-f]{16,20}\$exports)|(?:\$[0-9a-f]{16,20}\$(?:import|importAsync|require)\$[0-9a-f]+(?:\$[0-9a-f]+)?)/g;
 const BUILTINS = Object.keys(_globals().default.builtin);
 const GLOBALS_BY_CONTEXT = {
   browser: new Set([...BUILTINS, ...Object.keys(_globals().default.browser)]),
@@ -89,12 +99,15 @@ class ScopeHoistingPackager {
   needsPrelude = false;
   usedHelpers = new Set();
   externalAssets = new Set();
-  constructor(options, bundleGraph, bundle, parcelRequireName, useAsyncBundleRuntime) {
+  forceSkipWrapAssets = [];
+  constructor(options, bundleGraph, bundle, parcelRequireName, useAsyncBundleRuntime, forceSkipWrapAssets, logger) {
     this.options = options;
     this.bundleGraph = bundleGraph;
     this.bundle = bundle;
     this.parcelRequireName = parcelRequireName;
     this.useAsyncBundleRuntime = useAsyncBundleRuntime;
+    this.forceSkipWrapAssets = forceSkipWrapAssets ?? [];
+    this.logger = logger;
     let OutputFormat = OUTPUT_FORMATS[this.bundle.env.outputFormat];
     this.outputFormat = new OutputFormat(this);
     this.isAsyncBundle = this.bundleGraph.hasParentBundleOfType(this.bundle, 'js') && !this.bundle.env.isIsolated() && this.bundle.bundleBehavior !== 'isolated';
@@ -257,6 +270,18 @@ class ScopeHoistingPackager {
           actions.skipChildren();
           return;
         }
+        // This prevents children of a wrapped asset also being wrapped - it's an "unsafe" optimisation
+        // that should only be used when you know (or think you know) what you're doing.
+        //
+        // In particular this can force an async bundle to be scope hoisted where it previously would not be
+        // due to the entry asset being wrapped.
+        if (this.forceSkipWrapAssets.length > 0 && this.forceSkipWrapAssets.some(p => p === _path().default.relative(this.options.projectRoot, asset.filePath))) {
+          this.logger.verbose({
+            message: `Force skipping wrapping of ${_path().default.relative(this.options.projectRoot, asset.filePath)}`
+          });
+          actions.skipChildren();
+          return;
+        }
         if (!asset.meta.isConstantModule) {
           this.wrappedAssets.add(asset.id);
           wrapped.push(asset);
@@ -412,7 +437,7 @@ class ScopeHoistingPackager {
       // in a single regex so that we only do one pass over the whole code.
       let offset = 0;
       let columnStartIndex = 0;
-      code = code.replace(REPLACEMENT_RE, (m, d, i) => {
+      code = code.replace((0, _featureFlags().getFeatureFlag)('conditionalBundlingApi') ? REPLACEMENT_RE_CONDITIONAL : REPLACEMENT_RE, (m, d, i) => {
         if (m === '\n') {
           columnStartIndex = i + offset + 1;
           lineCount++;
@@ -559,7 +584,7 @@ ${code}
       // Async dependencies need a namespace object even if all used symbols were statically analyzed.
       // This is recorded in the promiseSymbol meta property set by the transformer rather than in
       // symbols so that we don't mark all symbols as used.
-      if (dep.priority === 'lazy' && dep.meta.promiseSymbol) {
+      if ((dep.priority === 'lazy' || dep.priority === 'conditional') && dep.meta.promiseSymbol) {
         let promiseSymbol = dep.meta.promiseSymbol;
         (0, _assert().default)(typeof promiseSymbol === 'string');
         let symbol = this.getSymbolResolution(asset, resolved, '*', dep);
@@ -993,7 +1018,9 @@ ${code}
       // Add the prelude if this is potentially the first JS bundle to load in a
       // particular context (e.g. entry scripts in HTML, workers, etc.).
       let parentBundles = this.bundleGraph.getParentBundles(this.bundle);
-      let mightBeFirstJS = parentBundles.length === 0 || parentBundles.some(b => b.type !== 'js') || this.bundleGraph.getBundleGroupsContainingBundle(this.bundle).some(g => this.bundleGraph.isEntryBundleGroup(g)) || this.bundle.env.isIsolated() || this.bundle.bundleBehavior === 'isolated';
+      let mightBeFirstJS = parentBundles.length === 0 || parentBundles.some(b => b.type !== 'js') || this.bundleGraph.getBundleGroupsContainingBundle(this.bundle).some(g => this.bundleGraph.isEntryBundleGroup(g)) || this.bundle.env.isIsolated() || this.bundle.bundleBehavior === 'isolated' ||
+      // Conditional deps may be loaded before entrypoints on the server
+      this.hasConditionalDependency();
       if (mightBeFirstJS) {
         let preludeCode = (0, _helpers.prelude)(this.parcelRequireName);
         res += preludeCode;
@@ -1036,10 +1063,7 @@ ${code}
   }
   needsDefaultInterop(asset) {
     if (asset.symbols.hasExportSymbol('*') && !asset.symbols.hasExportSymbol('default')) {
-      let deps = this.bundleGraph.getIncomingDependencies(asset);
-      return deps.some(dep => this.bundle.hasDependency(dep) &&
-      // dep.meta.isES6Module &&
-      dep.symbols.hasExportSymbol('default'));
+      return true;
     }
     return false;
   }
@@ -1055,5 +1079,11 @@ ${code}
   buildFunctionExpression(args, expr) {
     return this.bundle.env.supports('arrow-functions', true) ? `(${args.join(', ')}) => ${expr}` : `function (${args.join(', ')}) { return ${expr}; }`;
   }
+  hasConditionalDependency() {
+    if (!(0, _featureFlags().getFeatureFlag)('conditionalBundlingApi')) {
+      return false;
+    }
+    return this.bundle.getEntryAssets().some(entry => this.bundleGraph.getIncomingDependencies(entry).some(dep => dep.priority === 'conditional'));
+  }
 }
 exports.ScopeHoistingPackager = ScopeHoistingPackager;

package/lib/index.js

@@ -47,6 +47,12 @@ const CONFIG_SCHEMA = {
   properties: {
     unstable_asyncBundleRuntime: {
       type: 'boolean'
+    },
+    unstable_forceSkipWrapAssets: {
+      type: 'array',
+      items: {
+        type: 'string'
+      }
     }
   },
   additionalProperties: false
@@ -56,7 +62,7 @@ var _default = exports.default = new (_plugin().Packager)({
     config,
     options
   }) {
-    var _packageName$contents, _conf$contents;
+    var _packageName$contents, _conf$contents, _conf$contents2;
     let packageKey = '@atlaspack/packager-js';
     let conf = await config.getConfigFrom(options.projectRoot + '/index', [], {
       packageKey
@@ -78,7 +84,8 @@ var _default = exports.default = new (_plugin().Packager)({
     let name = (packageName === null || packageName === void 0 || (_packageName$contents = packageName.contents) === null || _packageName$contents === void 0 ? void 0 : _packageName$contents.name) ?? '';
     return {
       parcelRequireName: 'parcelRequire' + (0, _rust().hashString)(name).slice(-4),
-      unstable_asyncBundleRuntime: Boolean(conf === null || conf === void 0 || (_conf$contents = conf.contents) === null || _conf$contents === void 0 ? void 0 : _conf$contents.unstable_asyncBundleRuntime)
+      unstable_asyncBundleRuntime: Boolean(conf === null || conf === void 0 || (_conf$contents = conf.contents) === null || _conf$contents === void 0 ? void 0 : _conf$contents.unstable_asyncBundleRuntime),
+      unstable_forceSkipWrapAssets: (conf === null || conf === void 0 || (_conf$contents2 = conf.contents) === null || _conf$contents2 === void 0 ? void 0 : _conf$contents2.unstable_forceSkipWrapAssets) ?? []
     };
   },
   async package({
@@ -87,7 +94,8 @@ var _default = exports.default = new (_plugin().Packager)({
     getInlineBundleContents,
     getSourceMapReference,
     config,
-    options
+    options,
+    logger
   }) {
     // If this is a non-module script, and there is only one asset with no dependencies,
     // then we don't need to package at all and can pass through the original code un-wrapped.
@@ -100,7 +108,7 @@ var _default = exports.default = new (_plugin().Packager)({
       }
     }
     if (contents == null) {
-      let packager = bundle.env.shouldScopeHoist ? new _ScopeHoistingPackager.ScopeHoistingPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName, (0, _nullthrows().default)(config).unstable_asyncBundleRuntime) : new _DevPackager.DevPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName);
+      let packager = bundle.env.shouldScopeHoist ? new _ScopeHoistingPackager.ScopeHoistingPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName, (0, _nullthrows().default)(config).unstable_asyncBundleRuntime, (0, _nullthrows().default)(config).unstable_forceSkipWrapAssets, logger) : new _DevPackager.DevPackager(options, bundleGraph, bundle, (0, _nullthrows().default)(config).parcelRequireName);
       ({
         contents,
         map

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@atlaspack/packager-js",
-  "version": "2.12.1-dev.3401+b483af77f",
-  "license": "MIT",
+  "version": "2.12.1-dev.3450+58845ef87",
+  "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
   },
@@ -13,17 +13,18 @@
   "source": "src/index.js",
   "engines": {
     "node": ">= 16.0.0",
-    "parcel": "^2.12.1-dev.3401+b483af77f"
+    "parcel": "^2.12.1-dev.3450+58845ef87"
   },
   "dependencies": {
-    "@atlaspack/diagnostic": "2.12.1-dev.3401+b483af77f",
-    "@atlaspack/plugin": "2.12.1-dev.3401+b483af77f",
-    "@atlaspack/rust": "2.12.1-dev.3401+b483af77f",
-    "@atlaspack/types": "2.12.1-dev.3401+b483af77f",
-    "@atlaspack/utils": "2.12.1-dev.3401+b483af77f",
+    "@atlaspack/diagnostic": "2.12.1-dev.3450+58845ef87",
+    "@atlaspack/feature-flags": "2.12.1-dev.3450+58845ef87",
+    "@atlaspack/plugin": "2.12.1-dev.3450+58845ef87",
+    "@atlaspack/rust": "2.12.1-dev.3450+58845ef87",
+    "@atlaspack/types": "2.12.1-dev.3450+58845ef87",
+    "@atlaspack/utils": "2.12.1-dev.3450+58845ef87",
     "@parcel/source-map": "^2.1.1",
     "globals": "^13.2.0",
     "nullthrows": "^1.1.1"
   },
-  "gitHead": "b483af77f02d1258c8dad156e097b94f83671d8e"
+  "gitHead": "58845ef87446fcedb7d7d8876440c64184645cbb"
 }

package/src/ScopeHoistingPackager.js

@@ -6,6 +6,7 @@ import type {
   Dependency,
   PluginOptions,
   NamedBundle,
+  PluginLogger,
 } from '@atlaspack/types';
 
 import {
@@ -23,6 +24,7 @@ import ThrowableDiagnostic, {
 } from '@atlaspack/diagnostic';
 import globals from 'globals';
 import path from 'path';
+import {getFeatureFlag} from '@atlaspack/feature-flags';
 
 import {ESMOutputFormat} from './ESMOutputFormat';
 import {CJSOutputFormat} from './CJSOutputFormat';
@@ -34,10 +36,15 @@ import {
   isValidIdentifier,
   makeValidIdentifier,
 } from './utils';
+
 // General regex used to replace imports with the resolved code, references with resolutions,
 // and count the number of newlines in the file for source maps.
+//
+// For conditional bundling the only difference in this regex is adding `importCond` where we have `importAsync` etc..
+const REPLACEMENT_RE_CONDITIONAL =
+  /\n|import\s+"([0-9a-f]{16,20}:.+?)";|(?:\$[0-9a-f]{16,20}\$exports)|(?:\$[0-9a-f]{16,20}\$(?:import|importAsync|require|importCond)\$[0-9a-f]+(?:\$[0-9a-f]+)?)/g;
 const REPLACEMENT_RE =
-  /\n|import\s+"([0-9a-f]{16}:.+?)";|(?:\$[0-9a-f]{16}\$exports)|(?:\$[0-9a-f]{16}\$(?:import|importAsync|require)\$[0-9a-f]+(?:\$[0-9a-f]+)?)/g;
+  /\n|import\s+"([0-9a-f]{16,20}:.+?)";|(?:\$[0-9a-f]{16,20}\$exports)|(?:\$[0-9a-f]{16,20}\$(?:import|importAsync|require)\$[0-9a-f]+(?:\$[0-9a-f]+)?)/g;
 
 const BUILTINS = Object.keys(globals.builtin);
 const GLOBALS_BY_CONTEXT = {
@@ -95,6 +102,8 @@ export class ScopeHoistingPackager {
   needsPrelude: boolean = false;
   usedHelpers: Set<string> = new Set();
   externalAssets: Set<Asset> = new Set();
+  forceSkipWrapAssets: Array<string> = [];
+  logger: PluginLogger;
 
   constructor(
     options: PluginOptions,
@@ -102,12 +111,16 @@ export class ScopeHoistingPackager {
     bundle: NamedBundle,
     parcelRequireName: string,
     useAsyncBundleRuntime: boolean,
+    forceSkipWrapAssets: Array<string>,
+    logger: PluginLogger,
   ) {
     this.options = options;
     this.bundleGraph = bundleGraph;
     this.bundle = bundle;
     this.parcelRequireName = parcelRequireName;
     this.useAsyncBundleRuntime = useAsyncBundleRuntime;
+    this.forceSkipWrapAssets = forceSkipWrapAssets ?? [];
+    this.logger = logger;
 
     let OutputFormat = OUTPUT_FORMATS[this.bundle.env.outputFormat];
     this.outputFormat = new OutputFormat(this);
@@ -340,6 +353,26 @@ export class ScopeHoistingPackager {
           actions.skipChildren();
           return;
         }
+        // This prevents children of a wrapped asset also being wrapped - it's an "unsafe" optimisation
+        // that should only be used when you know (or think you know) what you're doing.
+        //
+        // In particular this can force an async bundle to be scope hoisted where it previously would not be
+        // due to the entry asset being wrapped.
+        if (
+          this.forceSkipWrapAssets.length > 0 &&
+          this.forceSkipWrapAssets.some(
+            p => p === path.relative(this.options.projectRoot, asset.filePath),
+          )
+        ) {
+          this.logger.verbose({
+            message: `Force skipping wrapping of ${path.relative(
+              this.options.projectRoot,
+              asset.filePath,
+            )}`,
+          });
+          actions.skipChildren();
+          return;
+        }
         if (!asset.meta.isConstantModule) {
           this.wrappedAssets.add(asset.id);
           wrapped.push(asset);
@@ -536,92 +569,100 @@ export class ScopeHoistingPackager {
       // in a single regex so that we only do one pass over the whole code.
       let offset = 0;
       let columnStartIndex = 0;
-      code = code.replace(REPLACEMENT_RE, (m, d, i) => {
-        if (m === '\n') {
-          columnStartIndex = i + offset + 1;
-          lineCount++;
-          return '\n';
-        }
-
-        // If we matched an import, replace with the source code for the dependency.
-        if (d != null) {
-          let deps = depMap.get(d);
-          if (!deps) {
-            return m;
+      code = code.replace(
+        getFeatureFlag('conditionalBundlingApi')
+          ? REPLACEMENT_RE_CONDITIONAL
+          : REPLACEMENT_RE,
+        (m, d, i) => {
+          if (m === '\n') {
+            columnStartIndex = i + offset + 1;
+            lineCount++;
+            return '\n';
           }
 
-          let replacement = '';
-
-          // A single `${id}:${specifier}:esm` might have been resolved to multiple assets due to
-          // reexports.
-          for (let dep of deps) {
-            let resolved = this.bundleGraph.getResolvedAsset(dep, this.bundle);
-            let skipped = this.bundleGraph.isDependencySkipped(dep);
-            if (resolved && !skipped) {
-              // Hoist variable declarations for the referenced parcelRequire dependencies
-              // after the dependency is declared. This handles the case where the resulting asset
-              // is wrapped, but the dependency in this asset is not marked as wrapped. This means
-              // that it was imported/required at the top-level, so its side effects should run immediately.
-              let [res, lines] = this.getHoistedParcelRequires(
-                asset,
+          // If we matched an import, replace with the source code for the dependency.
+          if (d != null) {
+            let deps = depMap.get(d);
+            if (!deps) {
+              return m;
+            }
+
+            let replacement = '';
+
+            // A single `${id}:${specifier}:esm` might have been resolved to multiple assets due to
+            // reexports.
+            for (let dep of deps) {
+              let resolved = this.bundleGraph.getResolvedAsset(
                 dep,
-                resolved,
+                this.bundle,
               );
-              let map;
-              if (
-                this.bundle.hasAsset(resolved) &&
-                !this.seenAssets.has(resolved.id)
-              ) {
-                // If this asset is wrapped, we need to hoist the code for the dependency
-                // outside our parcelRequire.register wrapper. This is safe because all
-                // assets referenced by this asset will also be wrapped. Otherwise, inline the
-                // asset content where the import statement was.
-                if (shouldWrap) {
-                  depContent.push(this.visitAsset(resolved));
-                } else {
-                  let [depCode, depMap, depLines] = this.visitAsset(resolved);
-                  res = depCode + '\n' + res;
-                  lines += 1 + depLines;
-                  map = depMap;
+              let skipped = this.bundleGraph.isDependencySkipped(dep);
+              if (resolved && !skipped) {
+                // Hoist variable declarations for the referenced parcelRequire dependencies
+                // after the dependency is declared. This handles the case where the resulting asset
+                // is wrapped, but the dependency in this asset is not marked as wrapped. This means
+                // that it was imported/required at the top-level, so its side effects should run immediately.
+                let [res, lines] = this.getHoistedParcelRequires(
+                  asset,
+                  dep,
+                  resolved,
+                );
+                let map;
+                if (
+                  this.bundle.hasAsset(resolved) &&
+                  !this.seenAssets.has(resolved.id)
+                ) {
+                  // If this asset is wrapped, we need to hoist the code for the dependency
+                  // outside our parcelRequire.register wrapper. This is safe because all
+                  // assets referenced by this asset will also be wrapped. Otherwise, inline the
+                  // asset content where the import statement was.
+                  if (shouldWrap) {
+                    depContent.push(this.visitAsset(resolved));
+                  } else {
+                    let [depCode, depMap, depLines] = this.visitAsset(resolved);
+                    res = depCode + '\n' + res;
+                    lines += 1 + depLines;
+                    map = depMap;
+                  }
                 }
-              }
 
-              // Push this asset's source mappings down by the number of lines in the dependency
-              // plus the number of hoisted parcelRequires. Then insert the source map for the dependency.
-              if (sourceMap) {
-                if (lines > 0) {
-                  sourceMap.offsetLines(lineCount + 1, lines);
-                }
+                // Push this asset's source mappings down by the number of lines in the dependency
+                // plus the number of hoisted parcelRequires. Then insert the source map for the dependency.
+                if (sourceMap) {
+                  if (lines > 0) {
+                    sourceMap.offsetLines(lineCount + 1, lines);
+                  }
 
-                if (map) {
-                  sourceMap.addSourceMap(map, lineCount);
+                  if (map) {
+                    sourceMap.addSourceMap(map, lineCount);
+                  }
                 }
-              }
 
-              replacement += res;
-              lineCount += lines;
+                replacement += res;
+                lineCount += lines;
+              }
             }
+            return replacement;
           }
-          return replacement;
-        }
 
-        // If it wasn't a dependency, then it was an inline replacement (e.g. $id$import$foo -> $id$export$foo).
-        let replacement = replacements.get(m) ?? m;
-        if (sourceMap) {
-          // Offset the source map columns for this line if the replacement was a different length.
-          // This assumes that the match and replacement both do not contain any newlines.
-          let lengthDifference = replacement.length - m.length;
-          if (lengthDifference !== 0) {
-            sourceMap.offsetColumns(
-              lineCount + 1,
-              i + offset - columnStartIndex + m.length,
-              lengthDifference,
-            );
-            offset += lengthDifference;
+          // If it wasn't a dependency, then it was an inline replacement (e.g. $id$import$foo -> $id$export$foo).
+          let replacement = replacements.get(m) ?? m;
+          if (sourceMap) {
+            // Offset the source map columns for this line if the replacement was a different length.
+            // This assumes that the match and replacement both do not contain any newlines.
+            let lengthDifference = replacement.length - m.length;
+            if (lengthDifference !== 0) {
+              sourceMap.offsetColumns(
+                lineCount + 1,
+                i + offset - columnStartIndex + m.length,
+                lengthDifference,
+              );
+              offset += lengthDifference;
+            }
           }
-        }
-        return replacement;
-      });
+          return replacement;
+        },
+      );
     }
 
     // If the asset is wrapped, we need to insert the dependency code outside the parcelRequire.register
@@ -743,7 +784,10 @@ ${code}
       // Async dependencies need a namespace object even if all used symbols were statically analyzed.
       // This is recorded in the promiseSymbol meta property set by the transformer rather than in
       // symbols so that we don't mark all symbols as used.
-      if (dep.priority === 'lazy' && dep.meta.promiseSymbol) {
+      if (
+        (dep.priority === 'lazy' || dep.priority === 'conditional') &&
+        dep.meta.promiseSymbol
+      ) {
         let promiseSymbol = dep.meta.promiseSymbol;
         invariant(typeof promiseSymbol === 'string');
         let symbol = this.getSymbolResolution(asset, resolved, '*', dep);
@@ -1376,7 +1420,9 @@ ${code}
           .getBundleGroupsContainingBundle(this.bundle)
           .some(g => this.bundleGraph.isEntryBundleGroup(g)) ||
         this.bundle.env.isIsolated() ||
-        this.bundle.bundleBehavior === 'isolated';
+        this.bundle.bundleBehavior === 'isolated' ||
+        // Conditional deps may be loaded before entrypoints on the server
+        this.hasConditionalDependency();
 
       if (mightBeFirstJS) {
         let preludeCode = prelude(this.parcelRequireName);
@@ -1430,13 +1476,7 @@ ${code}
       asset.symbols.hasExportSymbol('*') &&
       !asset.symbols.hasExportSymbol('default')
     ) {
-      let deps = this.bundleGraph.getIncomingDependencies(asset);
-      return deps.some(
-        dep =>
-          this.bundle.hasDependency(dep) &&
-          // dep.meta.isES6Module &&
-          dep.symbols.hasExportSymbol('default'),
-      );
+      return true;
     }
 
     return false;
@@ -1467,4 +1507,18 @@ ${code}
       ? `(${args.join(', ')}) => ${expr}`
       : `function (${args.join(', ')}) { return ${expr}; }`;
   }
+
+  hasConditionalDependency(): boolean {
+    if (!getFeatureFlag('conditionalBundlingApi')) {
+      return false;
+    }
+
+    return this.bundle
+      .getEntryAssets()
+      .some(entry =>
+        this.bundleGraph
+          .getIncomingDependencies(entry)
+          .some(dep => dep.priority === 'conditional'),
+      );
+  }
 }

package/src/index.js

@@ -17,6 +17,7 @@ import {ScopeHoistingPackager} from './ScopeHoistingPackager';
 type JSPackagerConfig = {|
   parcelRequireName: string,
   unstable_asyncBundleRuntime: boolean,
+  unstable_forceSkipWrapAssets: Array<string>,
 |};
 
 const CONFIG_SCHEMA: SchemaEntity = {
@@ -25,6 +26,12 @@ const CONFIG_SCHEMA: SchemaEntity = {
     unstable_asyncBundleRuntime: {
       type: 'boolean',
     },
+    unstable_forceSkipWrapAssets: {
+      type: 'array',
+      items: {
+        type: 'string',
+      },
+    },
   },
   additionalProperties: false,
 };
@@ -66,6 +73,8 @@ export default (new Packager({
       unstable_asyncBundleRuntime: Boolean(
         conf?.contents?.unstable_asyncBundleRuntime,
       ),
+      unstable_forceSkipWrapAssets:
+        conf?.contents?.unstable_forceSkipWrapAssets ?? [],
     };
   },
   async package({
@@ -75,6 +84,7 @@ export default (new Packager({
     getSourceMapReference,
     config,
     options,
+    logger,
   }) {
     // If this is a non-module script, and there is only one asset with no dependencies,
     // then we don't need to package at all and can pass through the original code un-wrapped.
@@ -98,6 +108,8 @@ export default (new Packager({
             bundle,
             nullthrows(config).parcelRequireName,
             nullthrows(config).unstable_asyncBundleRuntime,
+            nullthrows(config).unstable_forceSkipWrapAssets,
+            logger,
           )
         : new DevPackager(
             options,