@atlaspack/core 2.12.1-canary.3448 → 2.12.1-canary.3450

package/lib/AssetGraph.js

@@ -153,7 +153,11 @@ class AssetGraph extends _graph().ContentGraph {
         nodes.push(node);
       }
     } else if (assetGroups) {
-      nodes.push(...assetGroups.map(assetGroup => nodeFromAssetGroup(assetGroup)));
+      for (const assetGroup of assetGroups) {
+        // Adding nodes individually ensures we do not encounter a range stack size exceeded error
+        // when there are >100000 asset groups
+        nodes.push(nodeFromAssetGroup(assetGroup));
+      }
     }
     this.replaceNodeIdsConnectedTo((0, _nullthrows().default)(this.rootNodeId), nodes.map(node => this.addNode(node)));
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/core",
-  "version": "2.12.1-canary.3448+ea8d2373e",
+  "version": "2.12.1-canary.3450+d9092cb8e",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -20,20 +20,20 @@
     "check-ts": "tsc --noEmit index.d.ts"
   },
   "dependencies": {
-    "@atlaspack/cache": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/diagnostic": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/events": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/feature-flags": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/fs": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/graph": "3.2.1-canary.3448+ea8d2373e",
-    "@atlaspack/logger": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/package-manager": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/plugin": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/profiler": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/rust": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/types": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/utils": "2.12.1-canary.3448+ea8d2373e",
-    "@atlaspack/workers": "2.12.1-canary.3448+ea8d2373e",
+    "@atlaspack/cache": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/diagnostic": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/events": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/feature-flags": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/fs": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/graph": "3.2.1-canary.3450+d9092cb8e",
+    "@atlaspack/logger": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/package-manager": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/plugin": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/profiler": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/rust": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/types": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/utils": "2.12.1-canary.3450+d9092cb8e",
+    "@atlaspack/workers": "2.12.1-canary.3450+d9092cb8e",
     "@mischnic/json-sourcemap": "^0.1.0",
     "@parcel/source-map": "^2.1.1",
     "base-x": "^3.0.8",
@@ -57,5 +57,5 @@
   "browser": {
     "./src/serializerCore.js": "./src/serializerCore.browser.js"
   },
-  "gitHead": "ea8d2373eeead6e3b0dc4326582800cd84addf38"
+  "gitHead": "d9092cb8efbaed752761ddba4d11603533e2e063"
 }

package/src/AssetGraph.js

@@ -168,13 +168,15 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
         nodes.push(node);
       }
     } else if (assetGroups) {
-      nodes.push(
-        ...assetGroups.map(assetGroup => nodeFromAssetGroup(assetGroup)),
-      );
+      for (const assetGroup of assetGroups) {
+        // Adding nodes individually ensures we do not encounter a range stack size exceeded error
+        // when there are >100000 asset groups
+        nodes.push(nodeFromAssetGroup(assetGroup));
+      }
     }
     this.replaceNodeIdsConnectedTo(
       nullthrows(this.rootNodeId),
-      nodes.map(node => this.addNode(node)),
+      nodes.map((node) => this.addNode(node)),
     );
   }
 
@@ -213,7 +215,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
 
     this.replaceNodeIdsConnectedTo(
       entrySpecifierNodeId,
-      resolved.map(file => this.addNode(nodeFromEntryFile(file))),
+      resolved.map((file) => this.addNode(nodeFromEntryFile(file))),
     );
   }
 
@@ -222,7 +224,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
     targets: Array<Target>,
     correspondingRequest: string,
   ) {
-    let depNodes = targets.map(target => {
+    let depNodes = targets.map((target) => {
       let node = nodeFromDep(
         // The passed project path is ignored in this case, because there is no `loc`
         createDependency('', {
@@ -254,7 +256,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
 
     this.replaceNodeIdsConnectedTo(
       entryNodeId,
-      depNodes.map(node => this.addNode(node)),
+      depNodes.map((node) => this.addNode(node)),
     );
   }
 
@@ -340,7 +342,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
       let traversedNode = nullthrows(this.getNode(traversedNodeId));
       if (traversedNode.type === 'asset') {
         let hasDeferred = this.getNodeIdsConnectedFrom(traversedNodeId).some(
-          childNodeId => {
+          (childNodeId) => {
             let childNode = nullthrows(this.getNode(childNodeId));
             return childNode.hasDeferred == null
               ? false
@@ -417,7 +419,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
 
     let deps = this.getIncomingDependencies(resolvedAsset);
 
-    return deps.every(d => {
+    return deps.every((d) => {
       // If this dependency has already been through this process, and we
       // know it's not deferrable, then there's no need to re-check
       if (this.undeferredDependencies.has(d)) {
@@ -428,7 +430,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
         d.symbols &&
         !(d.env.isLibrary && d.isEntry) &&
         !d.symbols.has('*') &&
-        ![...d.symbols.keys()].some(symbol => {
+        ![...d.symbols.keys()].some((symbol) => {
           let assetSymbol = resolvedAsset.symbols?.get(symbol)?.local;
           return assetSymbol != null && symbols.has(assetSymbol);
         });
@@ -532,7 +534,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
         };
       }
       let dependentAsset = dependentAssets.find(
-        a => a.uniqueKey === dep.specifier,
+        (a) => a.uniqueKey === dep.specifier,
       );
       if (dependentAsset) {
         depNode.complete = true;
@@ -593,7 +595,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
     startNodeId: ?NodeId,
   ): ?TContext {
     return this.filteredTraverse(
-      nodeId => {
+      (nodeId) => {
         let node = nullthrows(this.getNode(nodeId));
         return node.type === 'asset' ? node.value : null;
       },
@@ -631,7 +633,7 @@ export default class AssetGraph extends ContentGraph<AssetGraphNode> {
 
     let hash = new Hash();
     // TODO: sort??
-    this.traverse(nodeId => {
+    this.traverse((nodeId) => {
       let node = nullthrows(this.getNode(nodeId));
       if (node.type === 'asset') {
         hash.writeString(nullthrows(node.value.outputHash));

package/src/Atlaspack.js

@@ -401,7 +401,7 @@ export default class Atlaspack {
           options,
         ),
         buildTime: Date.now() - startTime,
-        requestBundle: async bundle => {
+        requestBundle: async (bundle) => {
           let bundleNode = bundleGraph._graph.getNodeByContentKey(bundle.id);
           invariant(bundleNode?.type === 'bundle', 'Bundle does not exist');
 
@@ -580,7 +580,7 @@ export default class Atlaspack {
     let res = await this.#requestTracker.runRequest(request, {
       force: true,
     });
-    return res.map(asset =>
+    return res.map((asset) =>
       assetFromValue(asset, nullthrows(this.#resolvedOptions)),
     );
   }

package/src/AtlaspackConfig.js

@@ -154,7 +154,7 @@ export default class AtlaspackConfig {
   loadPlugins<T>(
     plugins: PureAtlaspackConfigPipeline,
   ): Promise<Array<LoadedPlugin<T>>> {
-    return Promise.all(plugins.map(p => this.loadPlugin<T>(p)));
+    return Promise.all(plugins.map((p) => this.loadPlugin<T>(p)));
   }
 
   async getResolvers(): Promise<Array<LoadedPlugin<Resolver<mixed>>>> {
@@ -181,7 +181,7 @@ export default class AtlaspackConfig {
   getValidatorNames(filePath: ProjectPath): Array<string> {
     let validators: PureAtlaspackConfigPipeline =
       this._getValidatorNodes(filePath);
-    return validators.map(v => v.packageName);
+    return validators.map((v) => v.packageName);
   }
 
   getValidators(
@@ -193,8 +193,8 @@ export default class AtlaspackConfig {
 
   getNamedPipelines(): $ReadOnlyArray<string> {
     return Object.keys(this.transformers)
-      .filter(glob => glob.includes(':'))
-      .map(glob => glob.split(':')[0]);
+      .filter((glob) => glob.includes(':'))
+      .map((glob) => glob.split(':')[0]);
   }
 
   async getTransformers(
@@ -278,7 +278,9 @@ export default class AtlaspackConfig {
     // Pipelines for bundles come from their entry assets, so the pipeline likely exists in transformers.
     if (pipeline) {
       let prefix = pipeline + ':';
-      if (!Object.keys(this.optimizers).some(glob => glob.startsWith(prefix))) {
+      if (
+        !Object.keys(this.optimizers).some((glob) => glob.startsWith(prefix))
+      ) {
         pipeline = null;
       }
     }
@@ -294,7 +296,7 @@ export default class AtlaspackConfig {
 
   getOptimizerNames(filePath: FilePath, pipeline: ?string): Array<string> {
     let optimizers = this._getOptimizerNodes(filePath, pipeline);
-    return optimizers.map(o => o.packageName);
+    return optimizers.map((o) => o.packageName);
   }
 
   getOptimizers(
@@ -435,7 +437,7 @@ export default class AtlaspackConfig {
       configsWithPlugin = new Set(getConfigPaths(this.options, plugins));
     } else {
       configsWithPlugin = new Set(
-        Object.keys(plugins).flatMap(k =>
+        Object.keys(plugins).flatMap((k) =>
           Array.isArray(plugins[k])
             ? getConfigPaths(this.options, plugins[k])
             : [getConfigPath(this.options, plugins[k])],
@@ -451,7 +453,7 @@ export default class AtlaspackConfig {
 
     let seenKey = false;
     let codeFrames = await Promise.all(
-      [...configsWithPlugin].map(async filePath => {
+      [...configsWithPlugin].map(async (filePath) => {
         let configContents = await this.options.inputFS.readFile(
           filePath,
           'utf8',
@@ -481,7 +483,7 @@ export default class AtlaspackConfig {
 
 function getConfigPaths(options, nodes) {
   return nodes
-    .map(node => (node !== '...' ? getConfigPath(options, node) : null))
+    .map((node) => (node !== '...' ? getConfigPath(options, node) : null))
     .filter(Boolean);
 }
 

package/src/AtlaspackConfig.schema.js

@@ -96,7 +96,7 @@ export default {
     },
     bundler: {
       type: 'string',
-      __validate: (validatePluginName('bundler', 'bundler'): string => void),
+      __validate: (validatePluginName('bundler', 'bundler'): (string) => void),
     },
     resolvers: (pipelineSchema('resolver', 'resolvers'): SchemaEntity),
     transformers: (mapPipelineSchema(