@atlaspack/core 2.16.2-canary.33 → 2.16.2-canary.330

package/lib/public/Environment.js

@@ -29,11 +29,11 @@ function _semver() {
 var _utils = require("../utils");
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 const inspect = Symbol.for('nodejs.util.inspect.custom');
-const BROWSER_ENVS = exports.BROWSER_ENVS = new Set(['browser', 'web-worker', 'service-worker', 'worklet', 'electron-renderer']);
+const BROWSER_ENVS = exports.BROWSER_ENVS = new Set(['browser', 'web-worker', 'service-worker', 'worklet', 'tesseract', 'electron-renderer']);
 const ELECTRON_ENVS = new Set(['electron-main', 'electron-renderer']);
 const NODE_ENVS = new Set(['node', ...ELECTRON_ENVS]);
 const WORKER_ENVS = new Set(['web-worker', 'service-worker']);
-const ISOLATED_ENVS = exports.ISOLATED_ENVS = new Set([...WORKER_ENVS, 'worklet']);
+const ISOLATED_ENVS = exports.ISOLATED_ENVS = new Set([...WORKER_ENVS, 'worklet', 'tesseract']);
 const ALL_BROWSERS = ['chrome', 'and_chr', 'edge', 'firefox', 'and_ff', 'safari', 'ios', 'samsung', 'opera', 'ie', 'op_mini', 'blackberry', 'op_mob', 'ie_mob', 'and_uc', 'and_qq', 'baidu', 'kaios'];
 
 // See require("caniuse-api").getSupport(<feature name>)
@@ -127,9 +127,10 @@ function environmentToInternalEnvironment(environment) {
   return (0, _nullthrows().default)(_environmentToInternalEnvironment.get(environment));
 }
 class Environment {
-  #environment /*: InternalEnvironment */;
-  #options /*: AtlaspackOptions */;
-
+  // @ts-expect-error TS2564
+  #environment;
+  // @ts-expect-error TS2564
+  #options;
   constructor(env, options) {
     let existing = internalEnvironmentToEnvironment.get(env);
     if (existing != null) {
@@ -177,8 +178,9 @@ class Environment {
   get unstableSingleFileOutput() {
     return this.#environment.unstableSingleFileOutput;
   }
-
-  // $FlowFixMe[unsupported-syntax]
+  get customEnv() {
+    return this.#environment.customEnv;
+  }
   [inspect]() {
     return `Env(${this.#environment.context})`;
   }
@@ -200,6 +202,9 @@ class Environment {
   isWorklet() {
     return this.#environment.context === 'worklet';
   }
+  isTesseract() {
+    return this.#environment.context === 'tesseract';
+  }
   matchesEngines(minVersions, defaultValue = false) {
     // Determine if the environment matches some minimum version requirements.
     // For browsers, we run a browserslist query with and without the minimum

package/lib/public/MutableBundleGraph.js

@@ -26,7 +26,7 @@ function _rust() {
   return data;
 }
 var _BundleGraph = _interopRequireDefault(require("./BundleGraph"));
-var _BundleGraph2 = _interopRequireWildcard(require("../BundleGraph"));
+var _BundleGraph2 = require("../BundleGraph");
 var _Bundle = require("./Bundle");
 var _Asset = require("./Asset");
 var _utils = require("../utils");
@@ -38,6 +38,14 @@ var _projectPath = require("../projectPath");
 var _types = require("../types");
 var _BundleGroup = _interopRequireWildcard(require("./BundleGroup"));
 var _IdentifierRegistry = require("../IdentifierRegistry");
+var _EnvironmentManager = require("../EnvironmentManager");
+function _featureFlags() {
+  const data = require("@atlaspack/feature-flags");
+  _featureFlags = function () {
+    return data;
+  };
+  return data;
+}
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
@@ -53,9 +61,9 @@ function createBundleId(data) {
   return id;
 }
 class MutableBundleGraph extends _BundleGraph.default {
-  #graph /*: InternalBundleGraph */;
-  #options /*: AtlaspackOptions */;
-  #bundlePublicIds /*: Set<string> */ = new Set();
+  #graph;
+  #options;
+  #bundlePublicIds = new Set();
   constructor(graph, options) {
     super(graph, _Bundle.Bundle.get.bind(_Bundle.Bundle), options);
     this.#graph = graph;
@@ -65,10 +73,18 @@ class MutableBundleGraph extends _BundleGraph.default {
     this.#graph.addAssetToBundle((0, _Asset.assetToAssetValue)(asset), (0, _Bundle.bundleToInternalBundle)(bundle));
   }
   addAssetGraphToBundle(asset, bundle, shouldSkipDependency) {
-    this.#graph.addAssetGraphToBundle((0, _Asset.assetToAssetValue)(asset), (0, _Bundle.bundleToInternalBundle)(bundle), shouldSkipDependency ? d => shouldSkipDependency(new _Dependency.default(d, this.#options)) : undefined);
+    this.#graph.addAssetGraphToBundle((0, _Asset.assetToAssetValue)(asset), (0, _Bundle.bundleToInternalBundle)(bundle),
+    // @ts-expect-error TS2345
+    shouldSkipDependency ? d =>
+    // @ts-expect-error TS2345
+    shouldSkipDependency(new _Dependency.default(d, this.#options)) : undefined);
   }
   addEntryToBundle(asset, bundle, shouldSkipDependency) {
-    this.#graph.addEntryToBundle((0, _Asset.assetToAssetValue)(asset), (0, _Bundle.bundleToInternalBundle)(bundle), shouldSkipDependency ? d => shouldSkipDependency(new _Dependency.default(d, this.#options)) : undefined);
+    this.#graph.addEntryToBundle((0, _Asset.assetToAssetValue)(asset), (0, _Bundle.bundleToInternalBundle)(bundle),
+    // @ts-expect-error TS2345
+    shouldSkipDependency ? d =>
+    // @ts-expect-error TS2345
+    shouldSkipDependency(new _Dependency.default(d, this.#options)) : undefined);
   }
   createBundleGroup(dependency, target) {
     let dependencyNode = this.#graph._graph.getNodeByContentKey(dependency.id);
@@ -121,10 +137,14 @@ class MutableBundleGraph extends _BundleGraph.default {
     this.#graph.internalizeAsyncDependency((0, _Bundle.bundleToInternalBundle)(bundle), (0, _Dependency.dependencyToInternalDependency)(dependency));
   }
   createBundle(opts) {
-    let entryAsset = opts.entryAsset ? (0, _Asset.assetToAssetValue)(opts.entryAsset) : null;
+    // @ts-expect-error TS2339
+    let entryAsset = opts.entryAsset ?
+    // @ts-expect-error TS2339
+    (0, _Asset.assetToAssetValue)(opts.entryAsset) : null;
     let target = (0, _Target.targetToInternalTarget)(opts.target);
     let bundleId = createBundleId({
       entryAssetId: (entryAsset === null || entryAsset === void 0 ? void 0 : entryAsset.id) ?? null,
+      // @ts-expect-error TS2339
       uniqueKey: opts.uniqueKey ?? null,
       distDir: target.distDir,
       bundleBehavior: opts.bundleBehavior ?? null
@@ -142,20 +162,42 @@ class MutableBundleGraph extends _BundleGraph.default {
       (0, _assert().default)((entryAssetNode === null || entryAssetNode === void 0 ? void 0 : entryAssetNode.type) === 'asset', 'Entry asset does not exist');
       isPlaceholder = entryAssetNode.requested === false;
     }
+    let entryAssetIds = [];
+    if (entryAsset) {
+      entryAssetIds = [entryAsset.id];
+      if ((0, _featureFlags().getFeatureFlag)('supportWebpackChunkName')) {
+        // @ts-expect-error TS2339
+        let bundleRoots = opts.bundleRoots ?? [entryAsset];
+        let bundleRootIds = bundleRoots.map(({
+          id
+        }) => id).filter(assetId => assetId !== entryAsset.id);
+        entryAssetIds.push(...bundleRootIds);
+      }
+    }
     let bundleNode = {
       type: 'bundle',
       id: bundleId,
       value: {
         id: bundleId,
         hashReference: this.#options.shouldContentHash ? _constants.HASH_REF_PREFIX + bundleId : bundleId.slice(-8),
+        // @ts-expect-error TS2339
         type: opts.entryAsset ? opts.entryAsset.type : opts.type,
-        env: opts.env ? (0, _Environment.environmentToInternalEnvironment)(opts.env) : (0, _nullthrows().default)(entryAsset).env,
-        entryAssetIds: entryAsset ? [entryAsset.id] : [],
+        // @ts-expect-error TS2339
+        env: opts.env ?
+        // @ts-expect-error TS2339
+        (0, _EnvironmentManager.toEnvironmentRef)((0, _Environment.environmentToInternalEnvironment)(opts.env)) : (0, _nullthrows().default)(entryAsset).env,
+        entryAssetIds,
         mainEntryId: entryAsset === null || entryAsset === void 0 ? void 0 : entryAsset.id,
+        // @ts-expect-error TS2339
         pipeline: opts.entryAsset ? opts.entryAsset.pipeline : opts.pipeline,
         needsStableName: opts.needsStableName,
         bundleBehavior: opts.bundleBehavior != null ? _types.BundleBehavior[opts.bundleBehavior] : null,
-        isSplittable: opts.entryAsset ? opts.entryAsset.isBundleSplittable : opts.isSplittable,
+        // @ts-expect-error TS2339
+        isSplittable: opts.entryAsset ?
+        // @ts-expect-error TS2339
+        opts.entryAsset.isBundleSplittable :
+        // @ts-expect-error TS2339
+        opts.isSplittable,
         isPlaceholder,
         target,
         name: null,
@@ -165,8 +207,8 @@ class MutableBundleGraph extends _BundleGraph.default {
       }
     };
     let bundleNodeId = this.#graph._graph.addNodeByContentKey(bundleId, bundleNode);
-    if (opts.entryAsset) {
-      this.#graph._graph.addEdge(bundleNodeId, this.#graph._graph.getNodeIdByContentKey(opts.entryAsset.id));
+    for (let assetId of entryAssetIds) {
+      this.#graph._graph.addEdge(bundleNodeId, this.#graph._graph.getNodeIdByContentKey(assetId));
     }
     return _Bundle.Bundle.get(bundleNode.value, this.#graph, this.#options);
   }

package/lib/public/PluginOptions.js

@@ -6,8 +6,8 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 let parcelOptionsToPluginOptions = new WeakMap();
 class PluginOptions {
-  #options /*: AtlaspackOptions */;
-
+  // @ts-expect-error TS2564
+  #options;
   constructor(options) {
     let existing = parcelOptionsToPluginOptions.get(options);
     if (existing != null) {

package/lib/public/Symbols.js

@@ -31,7 +31,9 @@ class AssetSymbols {
   /*::
   @@iterator(): Iterator<[ISymbol, {|local: ISymbol, loc: ?SourceLocation, meta?: ?Meta|}]> { return ({}: any); }
   */
+  // @ts-expect-error TS2564
   #value;
+  // @ts-expect-error TS2564
   #options;
   constructor(options, asset) {
     let existing = valueToSymbols.get(asset);
@@ -67,12 +69,10 @@ class AssetSymbols {
     var _this$value$symbols3;
     return ((_this$value$symbols3 = this.#value.symbols) === null || _this$value$symbols3 === void 0 ? void 0 : _this$value$symbols3.keys()) ?? [];
   }
-  // $FlowFixMe
+  // @ts-expect-error TS2416
   [Symbol.iterator]() {
     return this.#value.symbols ? this.#value.symbols[Symbol.iterator]() : EMPTY_ITERATOR;
   }
-
-  // $FlowFixMe
   [inspect]() {
     return `AssetSymbols(${this.#value.symbols ? [...this.#value.symbols].map(([s, {
       local
@@ -85,7 +85,9 @@ class MutableAssetSymbols {
   /*::
   @@iterator(): Iterator<[ISymbol, {|local: ISymbol, loc: ?SourceLocation, meta?: ?Meta|}]> { return ({}: any); }
   */
+  // @ts-expect-error TS2564
   #value;
+  // @ts-expect-error TS2564
   #options;
   constructor(options, asset) {
     let existing = valueToMutableAssetSymbols.get(asset);
@@ -120,15 +122,13 @@ class MutableAssetSymbols {
     return this.#value.symbols == null;
   }
   exportSymbols() {
-    // $FlowFixMe
+    // @ts-expect-error TS2322
     return this.#value.symbols.keys();
   }
-  // $FlowFixMe
+  // @ts-expect-error TS2416
   [Symbol.iterator]() {
     return this.#value.symbols ? this.#value.symbols[Symbol.iterator]() : EMPTY_ITERATOR;
   }
-
-  // $FlowFixMe
   [inspect]() {
     return `MutableAssetSymbols(${this.#value.symbols ? [...this.#value.symbols].map(([s, {
       local
@@ -159,7 +159,9 @@ class MutableDependencySymbols {
   /*::
   @@iterator(): Iterator<[ISymbol, {|local: ISymbol, loc: ?SourceLocation, isWeak: boolean, meta?: ?Meta|}]> { return ({}: any); }
   */
+  // @ts-expect-error TS2564
   #value;
+  // @ts-expect-error TS2564
   #options;
   constructor(options, dep) {
     let existing = valueToMutableDependencySymbols.get(dep);
@@ -192,16 +194,14 @@ class MutableDependencySymbols {
     return this.#value.symbols == null;
   }
   exportSymbols() {
-    // $FlowFixMe
+    // @ts-expect-error TS2322
     return this.#value.symbols ? this.#value.symbols.keys() : EMPTY_ITERABLE;
   }
 
-  // $FlowFixMe
+  // @ts-expect-error TS2416
   [Symbol.iterator]() {
     return this.#value.symbols ? this.#value.symbols[Symbol.iterator]() : EMPTY_ITERATOR;
   }
-
-  // $FlowFixMe
   [inspect]() {
     return `MutableDependencySymbols(${this.#value.symbols ? [...this.#value.symbols].map(([s, {
       local,

package/lib/public/Target.js

@@ -15,6 +15,7 @@ function _nullthrows() {
 var _Environment = _interopRequireDefault(require("./Environment"));
 var _projectPath = require("../projectPath");
 var _utils = require("../utils");
+var _EnvironmentManager = require("../EnvironmentManager");
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 const inspect = Symbol.for('nodejs.util.inspect.custom');
 const internalTargetToTarget = new WeakMap();
@@ -23,9 +24,10 @@ function targetToInternalTarget(target) {
   return (0, _nullthrows().default)(_targetToInternalTarget.get(target));
 }
 class Target {
-  #target /*: TargetValue */;
-  #options /*: AtlaspackOptions */;
-
+  // @ts-expect-error TS2564
+  #target;
+  // @ts-expect-error TS2564
+  #options;
   constructor(target, options) {
     let existing = internalTargetToTarget.get(target);
     if (existing != null) {
@@ -44,7 +46,7 @@ class Target {
     return (0, _projectPath.fromProjectPath)(this.#options.projectRoot, this.#target.distDir);
   }
   get env() {
-    return new _Environment.default(this.#target.env, this.#options);
+    return new _Environment.default((0, _EnvironmentManager.fromEnvironmentId)(this.#target.env), this.#options);
   }
   get name() {
     return this.#target.name;
@@ -55,9 +57,8 @@ class Target {
   get loc() {
     return (0, _utils.fromInternalSourceLocation)(this.#options.projectRoot, this.#target.loc);
   }
-
-  // $FlowFixMe[unsupported-syntax]
   [inspect]() {
+    // @ts-expect-error TS7053
     return `Target(${this.name} - ${this.env[inspect]()})`;
   }
 }

package/lib/registerCoreWithSerializer.js

@@ -25,8 +25,13 @@ var _BundleGraph = _interopRequireDefault(require("./BundleGraph"));
 var _Config = _interopRequireDefault(require("./public/Config"));
 var _RequestTracker = require("./RequestTracker");
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+// flow-to-ts helpers
+
+// /flow-to-ts helpers
+// @ts-expect-error TS7034
 let coreRegistered;
 function registerCoreWithSerializer() {
+  // @ts-expect-error TS7005
   if (coreRegistered) {
     return;
   }
@@ -34,8 +39,6 @@ function registerCoreWithSerializer() {
   if (typeof packageVersion !== 'string') {
     throw new Error('Expected package version to be a string');
   }
-
-  // $FlowFixMe[incompatible-cast]
   for (let [name, ctor] of Object.entries({
     AssetGraph: _AssetGraph.default,
     Config: _Config.default,
@@ -43,7 +46,6 @@ function registerCoreWithSerializer() {
     Graph: _graph().Graph,
     AtlaspackConfig: _AtlaspackConfig.AtlaspackConfig,
     RequestGraph: _RequestTracker.RequestGraph
-    // $FlowFixMe[unclear-type]
   })) {
     (0, _buildCache().registerSerializableClass)(packageVersion + ':' + name, ctor);
   }

package/lib/requests/AssetGraphRequest.js

@@ -26,6 +26,13 @@ function _nullthrows() {
   };
   return data;
 }
+function _featureFlags() {
+  const data = require("@atlaspack/feature-flags");
+  _featureFlags = function () {
+    return data;
+  };
+  return data;
+}
 function _utils() {
   const data = require("@atlaspack/utils");
   _utils = function () {
@@ -66,9 +73,10 @@ function createAssetGraphRequest(requestInput) {
     run: async input => {
       let prevResult = await input.api.getPreviousResult();
       let builder = new AssetGraphBuilder(input, prevResult);
-      let assetGraphRequest = await await builder.build();
+      let assetGraphRequest = await builder.build();
 
       // early break for incremental bundling if production or flag is off;
+      assetGraphRequest.assetGraph.setDisableIncrementalBundling(!input.options.shouldBundleIncrementally || input.options.mode === 'production');
       if (!input.options.shouldBundleIncrementally || input.options.mode === 'production') {
         assetGraphRequest.assetGraph.safeToIncrementallyBundle = false;
       }
@@ -116,8 +124,16 @@ class AssetGraphBuilder {
     this.shouldBuildLazily = shouldBuildLazily ?? false;
     this.lazyIncludes = lazyIncludes ?? [];
     this.lazyExcludes = lazyExcludes ?? [];
-    this.cacheKey = (0, _rust().hashString)(`${_constants.ATLASPACK_VERSION}${name}${JSON.stringify(entries) ?? ''}${options.mode}${options.shouldBuildLazily ? 'lazy' : 'eager'}`) + '-AssetGraph';
-    this.isSingleChangeRebuild = api.getInvalidSubRequests().filter(req => req.requestType === 'asset_request').length === 1;
+    this.skipSymbolProp = input.skipSymbolProp ?? false;
+    if ((0, _featureFlags().getFeatureFlag)('cachePerformanceImprovements')) {
+      const key = (0, _rust().hashString)(`${_constants.ATLASPACK_VERSION}${name}${JSON.stringify(entries) ?? ''}${options.mode}${options.shouldBuildLazily ? 'lazy' : 'eager'}`);
+      this.cacheKey = `AssetGraph/${_constants.ATLASPACK_VERSION}/${options.mode}/${key}`;
+    } else {
+      this.cacheKey = (0, _rust().hashString)(`${_constants.ATLASPACK_VERSION}${name}${JSON.stringify(entries) ?? ''}${options.mode}${options.shouldBuildLazily ? 'lazy' : 'eager'}`) + '-AssetGraph';
+    }
+    this.isSingleChangeRebuild = api.getInvalidSubRequests()
+    // @ts-expect-error TS2367
+    .filter(req => req.requestType === 'asset_request').length === 1;
     this.queue = new (_utils().PromiseQueue)();
     assetGraph.onNodeRemoved = nodeId => {
       this.assetGroupsWithRemovedParents.delete(nodeId);
@@ -156,7 +172,9 @@ class AssetGraphBuilder {
     const visitChildren = nodeId => {
       for (let childNodeId of this.assetGraph.getNodeIdsConnectedFrom(nodeId)) {
         let child = (0, _nullthrows().default)(this.assetGraph.getNode(childNodeId));
-        if ((!visited.has(childNodeId) || child.hasDeferred) && this.shouldVisitChild(nodeId, childNodeId)) {
+        if (
+        // @ts-expect-error TS2339
+        (!visited.has(childNodeId) || child.hasDeferred) && this.shouldVisitChild(nodeId, childNodeId)) {
           if (child.type === 'asset_group') {
             visitedAssetGroups.add(childNodeId);
           }
@@ -199,34 +217,43 @@ class AssetGraphBuilder {
     }
     if (this.assetGraph.nodes.length > 1) {
       await (0, _dumpGraphToGraphViz.default)(this.assetGraph, 'AssetGraph_' + this.name + '_before_prop');
-      try {
-        let errors = (0, _SymbolPropagation.propagateSymbols)({
-          options: this.options,
-          assetGraph: this.assetGraph,
-          changedAssetsPropagation: this.changedAssetsPropagation,
-          assetGroupsWithRemovedParents: this.assetGroupsWithRemovedParents,
-          previousErrors: this.previousSymbolPropagationErrors
+
+      // Skip symbol propagation for runtime assets - they have pre-computed symbol data
+      if (this.skipSymbolProp) {
+        _logger().default.verbose({
+          origin: '@atlaspack/core',
+          message: 'Skipping symbol propagation for runtime asset graph'
         });
-        this.changedAssetsPropagation.clear();
-        if (errors.size > 0) {
-          this.api.storeResult({
+      } else {
+        try {
+          let errors = (0, _SymbolPropagation.propagateSymbols)({
+            options: this.options,
             assetGraph: this.assetGraph,
-            changedAssets: this.changedAssets,
             changedAssetsPropagation: this.changedAssetsPropagation,
             assetGroupsWithRemovedParents: this.assetGroupsWithRemovedParents,
-            previousSymbolPropagationErrors: errors,
-            assetRequests: []
-          }, this.cacheKey);
-
-          // Just throw the first error. Since errors can bubble (e.g. reexporting a reexported symbol also fails),
-          // determining which failing export is the root cause is nontrivial (because of circular dependencies).
-          throw new (_diagnostic().default)({
-            diagnostic: [...errors.values()][0]
+            previousErrors: this.previousSymbolPropagationErrors
           });
+          this.changedAssetsPropagation.clear();
+          if (errors.size > 0) {
+            this.api.storeResult({
+              assetGraph: this.assetGraph,
+              changedAssets: this.changedAssets,
+              changedAssetsPropagation: this.changedAssetsPropagation,
+              assetGroupsWithRemovedParents: this.assetGroupsWithRemovedParents,
+              previousSymbolPropagationErrors: errors,
+              assetRequests: []
+            }, this.cacheKey);
+
+            // Just throw the first error. Since errors can bubble (e.g. reexporting a reexported symbol also fails),
+            // determining which failing export is the root cause is nontrivial (because of circular dependencies).
+            throw new (_diagnostic().default)({
+              diagnostic: [...errors.values()][0]
+            });
+          }
+        } catch (e) {
+          await (0, _dumpGraphToGraphViz.default)(this.assetGraph, 'AssetGraph_' + this.name + '_failed');
+          throw e;
         }
-      } catch (e) {
-        await (0, _dumpGraphToGraphViz.default)(this.assetGraph, 'AssetGraph_' + this.name + '_failed');
-        throw e;
       }
     }
     await (0, _dumpGraphToGraphViz.default)(this.assetGraph, 'AssetGraph_' + this.name);
@@ -317,7 +344,14 @@ class AssetGraphBuilder {
   }
   shouldSkipRequest(nodeId) {
     let node = (0, _nullthrows().default)(this.assetGraph.getNode(nodeId));
-    return node.complete === true || !typesWithRequests.has(node.type) || node.correspondingRequest != null && this.api.canSkipSubrequest(node.correspondingRequest);
+    return (
+      // @ts-expect-error TS2339
+      node.complete === true || !typesWithRequests.has(node.type) ||
+      // @ts-expect-error TS2339
+      node.correspondingRequest != null &&
+      // @ts-expect-error TS2339
+      this.api.canSkipSubrequest(node.correspondingRequest)
+    );
   }
   queueCorrespondingRequest(nodeId, errors) {
     let promise;
@@ -356,6 +390,7 @@ class AssetGraphBuilder {
       let didEntriesChange = prevEntries.length !== currentEntries.length || prevEntries.every((entryId, index) => entryId === currentEntries[index]);
       if (didEntriesChange) {
         this.assetGraph.safeToIncrementallyBundle = false;
+        this.assetGraph.setNeedsBundling();
       }
     }
   }
@@ -378,6 +413,7 @@ class AssetGraphBuilder {
   }
   async runAssetRequest(input) {
     this.assetRequests.push(input);
+    // @ts-expect-error TS2345
     let request = (0, _AssetRequest.default)({
       ...input,
       name: this.name,
@@ -389,16 +425,26 @@ class AssetGraphBuilder {
     });
     if (assets != null) {
       for (let asset of assets) {
+        // Pass the runtimeAssetRequiringExecutionOnLoad flag from the asset
+        // group down to the asset itself, for reading in the packager.
+        if (input.runtimeAssetRequiringExecutionOnLoad) {
+          asset.meta = {
+            ...(asset.meta ?? {}),
+            runtimeAssetRequiringExecutionOnLoad: input.runtimeAssetRequiringExecutionOnLoad
+          };
+        }
         if (this.assetGraph.safeToIncrementallyBundle) {
           let otherAsset = this.assetGraph.getNodeByContentKey(asset.id);
           if (otherAsset != null) {
             (0, _assert().default)(otherAsset.type === 'asset');
             if (!this._areDependenciesEqualForAssets(asset, otherAsset.value)) {
               this.assetGraph.safeToIncrementallyBundle = false;
+              this.assetGraph.setNeedsBundling();
             }
           } else {
             // adding a new entry or dependency
             this.assetGraph.safeToIncrementallyBundle = false;
+            this.assetGraph.setNeedsBundling();
           }
         }
         this.changedAssets.set(asset.id, asset);
@@ -407,6 +453,7 @@ class AssetGraphBuilder {
       this.assetGraph.resolveAssetGroup(input, assets, request.id);
     } else {
       this.assetGraph.safeToIncrementallyBundle = false;
+      this.assetGraph.setNeedsBundling();
     }
     this.isSingleChangeRebuild = false;
   }