metro 0.76.1 → 0.76.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "metro",
-  "version": "0.76.1",
+  "version": "0.76.3",
   "description": "🚇 The JavaScript bundler for React Native.",
   "main": "src/index.js",
   "bin": "src/cli.js",
@@ -34,22 +34,22 @@
     "invariant": "^2.2.4",
     "jest-worker": "^27.2.0",
     "lodash.throttle": "^4.1.1",
-    "metro-babel-transformer": "0.76.1",
-    "metro-cache": "0.76.1",
-    "metro-cache-key": "0.76.1",
-    "metro-config": "0.76.1",
-    "metro-core": "0.76.1",
-    "metro-file-map": "0.76.1",
-    "metro-inspector-proxy": "0.76.1",
-    "metro-minify-terser": "0.76.1",
-    "metro-minify-uglify": "0.76.1",
-    "metro-react-native-babel-preset": "0.76.1",
-    "metro-resolver": "0.76.1",
-    "metro-runtime": "0.76.1",
-    "metro-source-map": "0.76.1",
-    "metro-symbolicate": "0.76.1",
-    "metro-transform-plugins": "0.76.1",
-    "metro-transform-worker": "0.76.1",
+    "metro-babel-transformer": "0.76.3",
+    "metro-cache": "0.76.3",
+    "metro-cache-key": "0.76.3",
+    "metro-config": "0.76.3",
+    "metro-core": "0.76.3",
+    "metro-file-map": "0.76.3",
+    "metro-inspector-proxy": "0.76.3",
+    "metro-minify-terser": "0.76.3",
+    "metro-minify-uglify": "0.76.3",
+    "metro-react-native-babel-preset": "0.76.3",
+    "metro-resolver": "0.76.3",
+    "metro-runtime": "0.76.3",
+    "metro-source-map": "0.76.3",
+    "metro-symbolicate": "0.76.3",
+    "metro-transform-plugins": "0.76.3",
+    "metro-transform-worker": "0.76.3",
     "mime-types": "^2.1.27",
     "node-fetch": "^2.2.0",
     "nullthrows": "^1.1.1",
@@ -63,16 +63,14 @@
   },
   "devDependencies": {
     "@babel/plugin-transform-flow-strip-types": "^7.0.0",
-    "@types/babel__code-frame": "^7.0.3",
-    "@types/ws": "^8.5.4",
     "babel-jest": "^29.2.1",
     "dedent": "^0.7.0",
     "jest-snapshot": "^26.5.2",
     "jest-snapshot-serializer-raw": "^1.2.0",
-    "metro-babel-register": "0.76.1",
-    "metro-memory-fs": "0.76.1",
-    "metro-react-native-babel-preset": "0.76.1",
-    "metro-react-native-babel-transformer": "0.76.1",
+    "metro-babel-register": "0.76.3",
+    "metro-memory-fs": "0.76.3",
+    "metro-react-native-babel-preset": "0.76.3",
+    "metro-react-native-babel-transformer": "0.76.3",
     "mock-req": "^0.2.0",
     "mock-res": "^0.6.0",
     "stack-trace": "^0.0.10"

package/src/DeltaBundler/Graph.js

@@ -86,17 +86,11 @@ const nullthrows = require("nullthrows");
 
 // TODO: Convert to a Flow enum
 
-function getInternalOptions({
-  transform,
-  resolve,
-  onProgress,
-  experimentalImportBundleSupport,
-  shallow,
-}) {
+function getInternalOptions({ transform, resolve, onProgress, lazy, shallow }) {
   let numProcessed = 0;
   let total = 0;
   return {
-    experimentalImportBundleSupport,
+    lazy,
     transform,
     resolve,
     onDependencyAdd: () => onProgress && onProgress(numProcessed, ++total),
@@ -286,10 +280,7 @@ class Graph {
       // Don't add a node for the module if the graph is shallow (single-module).
     } else if (dependency.data.data.asyncType === "weak") {
       // Exclude weak dependencies from the bundle.
-    } else if (
-      options.experimentalImportBundleSupport &&
-      dependency.data.data.asyncType != null
-    ) {
+    } else if (options.lazy && dependency.data.data.asyncType != null) {
       // Don't add a node for the module if we are traversing async dependencies
       // lazily (and this is an async dependency). Instead, record it in
       // importBundleNodes.
@@ -338,10 +329,7 @@ class Graph {
       // Weak dependencies are excluded from the bundle.
       return;
     }
-    if (
-      options.experimentalImportBundleSupport &&
-      dependency.data.data.asyncType != null
-    ) {
+    if (options.lazy && dependency.data.data.asyncType != null) {
       this._decrementImportBundleReference(dependency, parentModule);
     }
     const module = this.dependencies.get(absolutePath);
@@ -515,7 +503,7 @@ class Graph {
     );
     invariant(
       importBundleNode.inverseDependencies.has(parentModule.path),
-      "experimentalImportBundleSupport: import bundle inverse references"
+      "lazy: import bundle inverse references"
     );
     importBundleNode.inverseDependencies.delete(parentModule.path);
     if (importBundleNode.inverseDependencies.size === 0) {
@@ -667,8 +655,7 @@ function dependenciesEqual(a, b, options) {
   return (
     a === b ||
     (a.absolutePath === b.absolutePath &&
-      (!options.experimentalImportBundleSupport ||
-        a.data.data.asyncType === b.data.data.asyncType) &&
+      (!options.lazy || a.data.data.asyncType === b.data.data.asyncType) &&
       contextParamsEqual(a.data.data.contextParams, b.data.data.contextParams))
   );
 }

package/src/DeltaBundler/Graph.js.flow

@@ -94,7 +94,7 @@ type Delta = $ReadOnly<{
 }>;
 
 type InternalOptions<T> = $ReadOnly<{
-  experimentalImportBundleSupport: boolean,
+  lazy: boolean,
   onDependencyAdd: () => mixed,
   onDependencyAdded: () => mixed,
   resolve: Options<T>['resolve'],
@@ -106,14 +106,14 @@ function getInternalOptions<T>({
   transform,
   resolve,
   onProgress,
-  experimentalImportBundleSupport,
+  lazy,
   shallow,
 }: Options<T>): InternalOptions<T> {
   let numProcessed = 0;
   let total = 0;
 
   return {
-    experimentalImportBundleSupport,
+    lazy,
     transform,
     resolve,
     onDependencyAdd: () => onProgress && onProgress(numProcessed, ++total),
@@ -357,10 +357,7 @@ export class Graph<T = MixedOutput> {
       // Don't add a node for the module if the graph is shallow (single-module).
     } else if (dependency.data.data.asyncType === 'weak') {
       // Exclude weak dependencies from the bundle.
-    } else if (
-      options.experimentalImportBundleSupport &&
-      dependency.data.data.asyncType != null
-    ) {
+    } else if (options.lazy && dependency.data.data.asyncType != null) {
       // Don't add a node for the module if we are traversing async dependencies
       // lazily (and this is an async dependency). Instead, record it in
       // importBundleNodes.
@@ -421,10 +418,7 @@ export class Graph<T = MixedOutput> {
       return;
     }
 
-    if (
-      options.experimentalImportBundleSupport &&
-      dependency.data.data.asyncType != null
-    ) {
+    if (options.lazy && dependency.data.data.asyncType != null) {
       this._decrementImportBundleReference(dependency, parentModule);
     }
 
@@ -629,7 +623,7 @@ export class Graph<T = MixedOutput> {
     );
     invariant(
       importBundleNode.inverseDependencies.has(parentModule.path),
-      'experimentalImportBundleSupport: import bundle inverse references',
+      'lazy: import bundle inverse references',
     );
     importBundleNode.inverseDependencies.delete(parentModule.path);
     if (importBundleNode.inverseDependencies.size === 0) {
@@ -784,13 +778,12 @@ export class Graph<T = MixedOutput> {
 function dependenciesEqual(
   a: Dependency,
   b: Dependency,
-  options: $ReadOnly<{experimentalImportBundleSupport: boolean, ...}>,
+  options: $ReadOnly<{lazy: boolean, ...}>,
 ): boolean {
   return (
     a === b ||
     (a.absolutePath === b.absolutePath &&
-      (!options.experimentalImportBundleSupport ||
-        a.data.data.asyncType === b.data.data.asyncType) &&
+      (!options.lazy || a.data.data.asyncType === b.data.data.asyncType) &&
       contextParamsEqual(a.data.data.contextParams, b.data.data.contextParams))
   );
 }

package/src/DeltaBundler/Serializers/baseJSBundle.js

@@ -24,6 +24,7 @@ function baseJSBundle(entryPoint, preModules, graph, options) {
     includeAsyncPaths: options.includeAsyncPaths,
     projectRoot: options.projectRoot,
     serverRoot: options.serverRoot,
+    sourceUrl: options.sourceUrl,
   };
 
   // Do not prepend polyfills or the require runtime when only modules are requested

package/src/DeltaBundler/Serializers/baseJSBundle.js.flow

@@ -39,6 +39,7 @@ function baseJSBundle(
     includeAsyncPaths: options.includeAsyncPaths,
     projectRoot: options.projectRoot,
     serverRoot: options.serverRoot,
+    sourceUrl: options.sourceUrl,
   };
 
   // Do not prepend polyfills or the require runtime when only modules are requested

package/src/DeltaBundler/Serializers/helpers/js.js

@@ -31,16 +31,32 @@ function getModuleParams(module, options) {
       const id = options.createModuleId(dependency.absolutePath);
       if (options.includeAsyncPaths && dependency.data.data.asyncType != null) {
         hasPaths = true;
+        invariant(
+          options.sourceUrl != null,
+          "sourceUrl is required when includeAsyncPaths is true"
+        );
+
+        // TODO: Only include path if the target is not in the bundle
+
+        // Construct a server-relative URL for the split bundle, propagating
+        // most parameters from the main bundle's URL.
+
+        const { searchParams } = new URL(options.sourceUrl);
+        searchParams.set("modulesOnly", "true");
+        searchParams.set("runModule", "false");
         const bundlePath = path.relative(
           options.serverRoot,
           dependency.absolutePath
         );
-        // TODO: Eventually this slicing should be asyncRequire's responsibility
-        // Strip the file extension
-        paths[id] = path.join(
-          path.dirname(bundlePath),
-          path.basename(bundlePath, path.extname(bundlePath))
-        );
+        paths[id] =
+          "/" +
+          path.join(
+            path.dirname(bundlePath),
+            // Strip the file extension
+            path.basename(bundlePath, path.extname(bundlePath))
+          ) +
+          ".bundle?" +
+          searchParams.toString();
       }
       return id;
     }

package/src/DeltaBundler/Serializers/helpers/js.js.flow

@@ -24,6 +24,7 @@ export type Options = $ReadOnly<{
   includeAsyncPaths: boolean,
   projectRoot: string,
   serverRoot: string,
+  sourceUrl: ?string,
   ...
 }>;
 
@@ -48,16 +49,33 @@ function getModuleParams(module: Module<>, options: Options): Array<mixed> {
       const id = options.createModuleId(dependency.absolutePath);
       if (options.includeAsyncPaths && dependency.data.data.asyncType != null) {
         hasPaths = true;
+        invariant(
+          options.sourceUrl != null,
+          'sourceUrl is required when includeAsyncPaths is true',
+        );
+
+        // TODO: Only include path if the target is not in the bundle
+
+        // Construct a server-relative URL for the split bundle, propagating
+        // most parameters from the main bundle's URL.
+
+        const {searchParams} = new URL(options.sourceUrl);
+        searchParams.set('modulesOnly', 'true');
+        searchParams.set('runModule', 'false');
+
         const bundlePath = path.relative(
           options.serverRoot,
           dependency.absolutePath,
         );
-        // TODO: Eventually this slicing should be asyncRequire's responsibility
-        // Strip the file extension
-        paths[id] = path.join(
-          path.dirname(bundlePath),
-          path.basename(bundlePath, path.extname(bundlePath)),
-        );
+        paths[id] =
+          '/' +
+          path.join(
+            path.dirname(bundlePath),
+            // Strip the file extension
+            path.basename(bundlePath, path.extname(bundlePath)),
+          ) +
+          '.bundle?' +
+          searchParams.toString();
       }
       return id;
     },

package/src/DeltaBundler/Serializers/helpers/processModules.js

@@ -21,6 +21,7 @@ function processModules(
     includeAsyncPaths,
     projectRoot,
     serverRoot,
+    sourceUrl,
   }
 ) {
   return [...modules]
@@ -34,6 +35,7 @@ function processModules(
         includeAsyncPaths,
         projectRoot,
         serverRoot,
+        sourceUrl,
       }),
     ]);
 }

package/src/DeltaBundler/Serializers/helpers/processModules.js.flow

@@ -24,6 +24,7 @@ function processModules(
     includeAsyncPaths,
     projectRoot,
     serverRoot,
+    sourceUrl,
   }: $ReadOnly<{
     filter?: (module: Module<>) => boolean,
     createModuleId: string => number,
@@ -31,6 +32,7 @@ function processModules(
     includeAsyncPaths: boolean,
     projectRoot: string,
     serverRoot: string,
+    sourceUrl: ?string,
   }>,
 ): $ReadOnlyArray<[Module<>, string]> {
   return [...modules]
@@ -44,6 +46,7 @@ function processModules(
         includeAsyncPaths,
         projectRoot,
         serverRoot,
+        sourceUrl,
       }),
     ]);
 }

package/src/DeltaBundler/Serializers/hmrJSBundle.js

@@ -50,6 +50,7 @@ function generateModules(sourceModules, graph, options) {
 function prepareModule(module, graph, options) {
   const code = wrapModule(module, {
     ...options,
+    sourceUrl: url.format(options.clientUrl),
     dev: true,
   });
   const inverseDependencies = getInverseDependencies(module.path, graph);

package/src/DeltaBundler/Serializers/hmrJSBundle.js.flow

@@ -77,6 +77,7 @@ function prepareModule(
 ): string {
   const code = wrapModule(module, {
     ...options,
+    sourceUrl: url.format(options.clientUrl),
     dev: true,
   });
 

package/src/DeltaBundler/types.d.ts

@@ -8,7 +8,6 @@
  * @oncall react_native
  */
 
-import type {SourceLocation} from '@babel/code-frame';
 import type {JsTransformOptions} from 'metro-transform-worker';
 import type {RequireContextParams} from '../ModuleGraph/worker/collectDependencies';
 import type {RequireContext} from '../lib/contextModule';
@@ -55,7 +54,10 @@ export interface TransformResultDependency {
      */
     readonly isOptional?: boolean;
 
-    readonly locs: ReadonlyArray<SourceLocation>;
+    readonly locs: ReadonlyArray<{
+      readonly start: {readonly line: number; readonly column: number};
+      readonly end: {readonly line: number; readonly column: number};
+    }>;
 
     /** Context for requiring a collection of modules. */
     readonly contextParams?: RequireContextParams;
@@ -137,7 +139,7 @@ export interface Options<T = MixedOutput> {
   readonly onProgress:
     | ((numProcessed: number, total: number) => unknown)
     | null;
-  readonly experimentalImportBundleSupport: boolean;
+  readonly lazy: boolean;
   readonly unstable_allowRequireContext: boolean;
   readonly shallow: boolean;
 }

package/src/DeltaBundler/types.flow.js.flow

@@ -130,7 +130,7 @@ export type Options<T = MixedOutput> = {
   +transform: TransformFn<T>,
   +transformOptions: TransformInputOptions,
   +onProgress: ?(numProcessed: number, total: number) => mixed,
-  +experimentalImportBundleSupport: boolean,
+  +lazy: boolean,
   +unstable_allowRequireContext: boolean,
   +unstable_enablePackageExports: boolean,
   +shallow: boolean,

package/src/HmrServer.js

@@ -79,8 +79,7 @@ class HmrServer {
     const graphId = getGraphId(resolvedEntryFilePath, transformOptions, {
       resolverOptions,
       shallow: graphOptions.shallow,
-      experimentalImportBundleSupport:
-        this._config.server.experimentalImportBundleSupport,
+      lazy: graphOptions.lazy,
       unstable_allowRequireContext:
         this._config.transformer.unstable_allowRequireContext,
     });
@@ -120,6 +119,7 @@ class HmrServer {
         clients: new Set([client]),
         clientUrl,
         revisionId: id,
+        graphOptions,
         unlisten: () => unlisten(),
       };
       this._clientGroups.set(id, clientGroup);
@@ -279,7 +279,7 @@ class HmrServer {
       const hmrUpdate = hmrJSBundle(delta, revision.graph, {
         clientUrl: group.clientUrl,
         createModuleId: this._createModuleId,
-        includeAsyncPaths: this._config.server.experimentalImportBundleSupport,
+        includeAsyncPaths: group.graphOptions.lazy,
         projectRoot: this._config.projectRoot,
         serverRoot:
           this._config.server.unstable_serverRoot ?? this._config.projectRoot,

package/src/HmrServer.js.flow

@@ -11,6 +11,7 @@
 'use strict';
 
 import type IncrementalBundler, {RevisionId} from './IncrementalBundler';
+import type {GraphOptions} from './shared/types.flow';
 import type {ConfigT, RootPerfLogger} from 'metro-config';
 import type {
   HmrClientMessage,
@@ -48,6 +49,7 @@ type ClientGroup = {
   clientUrl: EntryPointURL,
   revisionId: RevisionId,
   +unlisten: () => void,
+  +graphOptions: GraphOptions,
 };
 
 function send(sendFns: Array<(string) => void>, message: HmrMessage): void {
@@ -123,8 +125,7 @@ class HmrServer<TClient: Client> {
     const graphId = getGraphId(resolvedEntryFilePath, transformOptions, {
       resolverOptions,
       shallow: graphOptions.shallow,
-      experimentalImportBundleSupport:
-        this._config.server.experimentalImportBundleSupport,
+      lazy: graphOptions.lazy,
       unstable_allowRequireContext:
         this._config.transformer.unstable_allowRequireContext,
     });
@@ -167,6 +168,7 @@ class HmrServer<TClient: Client> {
         clients: new Set([client]),
         clientUrl,
         revisionId: id,
+        graphOptions,
         unlisten: (): void => unlisten(),
       };
 
@@ -356,7 +358,7 @@ class HmrServer<TClient: Client> {
       const hmrUpdate = hmrJSBundle(delta, revision.graph, {
         clientUrl: group.clientUrl,
         createModuleId: this._createModuleId,
-        includeAsyncPaths: this._config.server.experimentalImportBundleSupport,
+        includeAsyncPaths: group.graphOptions.lazy,
         projectRoot: this._config.projectRoot,
         serverRoot:
           this._config.server.unstable_serverRoot ?? this._config.projectRoot,

package/src/IncrementalBundler.js

@@ -59,6 +59,7 @@ class IncrementalBundler {
     otherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     }
   ) {
     const absoluteEntryFiles = await this._getAbsoluteEntryFiles(entryFiles);
@@ -78,8 +79,7 @@ class IncrementalBundler {
       ),
       transformOptions,
       onProgress: otherOptions.onProgress,
-      experimentalImportBundleSupport:
-        this._config.server.experimentalImportBundleSupport,
+      lazy: otherOptions.lazy,
       unstable_allowRequireContext:
         this._config.transformer.unstable_allowRequireContext,
       unstable_enablePackageExports:
@@ -101,6 +101,7 @@ class IncrementalBundler {
     otherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     }
   ) {
     const absoluteEntryFiles = await this._getAbsoluteEntryFiles(entryFiles);
@@ -122,8 +123,7 @@ class IncrementalBundler {
         ),
         transformOptions,
         onProgress: otherOptions.onProgress,
-        experimentalImportBundleSupport:
-          this._config.server.experimentalImportBundleSupport,
+        lazy: otherOptions.lazy,
         unstable_allowRequireContext:
           this._config.transformer.unstable_allowRequireContext,
         unstable_enablePackageExports:
@@ -140,6 +140,7 @@ class IncrementalBundler {
     otherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     }
   ) {
     const graph = await this.buildGraphForEntries(
@@ -171,13 +172,13 @@ class IncrementalBundler {
     otherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     }
   ) {
     const graphId = getGraphId(entryFile, transformOptions, {
       resolverOptions,
       shallow: otherOptions.shallow,
-      experimentalImportBundleSupport:
-        this._config.server.experimentalImportBundleSupport,
+      lazy: otherOptions.lazy,
       unstable_allowRequireContext:
         this._config.transformer.unstable_allowRequireContext,
     });

package/src/IncrementalBundler.js.flow

@@ -38,6 +38,7 @@ export type OutputGraph = Graph<>;
 type OtherOptions = $ReadOnly<{
   onProgress: $PropertyType<DeltaBundlerOptions<>, 'onProgress'>,
   shallow: boolean,
+  lazy: boolean,
 }>;
 
 export type GraphRevision = {
@@ -107,6 +108,7 @@ class IncrementalBundler {
     otherOptions?: OtherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     },
   ): Promise<OutputGraph> {
     const absoluteEntryFiles = await this._getAbsoluteEntryFiles(entryFiles);
@@ -127,8 +129,7 @@ class IncrementalBundler {
       ),
       transformOptions,
       onProgress: otherOptions.onProgress,
-      experimentalImportBundleSupport:
-        this._config.server.experimentalImportBundleSupport,
+      lazy: otherOptions.lazy,
       unstable_allowRequireContext:
         this._config.transformer.unstable_allowRequireContext,
       unstable_enablePackageExports:
@@ -153,6 +154,7 @@ class IncrementalBundler {
     otherOptions?: OtherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     },
   ): Promise<ReadOnlyDependencies<>> {
     const absoluteEntryFiles = await this._getAbsoluteEntryFiles(entryFiles);
@@ -175,8 +177,7 @@ class IncrementalBundler {
         ),
         transformOptions,
         onProgress: otherOptions.onProgress,
-        experimentalImportBundleSupport:
-          this._config.server.experimentalImportBundleSupport,
+        lazy: otherOptions.lazy,
         unstable_allowRequireContext:
           this._config.transformer.unstable_allowRequireContext,
         unstable_enablePackageExports:
@@ -195,6 +196,7 @@ class IncrementalBundler {
     otherOptions?: OtherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     },
   ): Promise<{+graph: OutputGraph, +prepend: $ReadOnlyArray<Module<>>}> {
     const graph = await this.buildGraphForEntries(
@@ -229,6 +231,7 @@ class IncrementalBundler {
     otherOptions?: OtherOptions = {
       onProgress: null,
       shallow: false,
+      lazy: false,
     },
   ): Promise<{
     delta: DeltaResult<>,
@@ -238,8 +241,7 @@ class IncrementalBundler {
     const graphId = getGraphId(entryFile, transformOptions, {
       resolverOptions,
       shallow: otherOptions.shallow,
-      experimentalImportBundleSupport:
-        this._config.server.experimentalImportBundleSupport,
+      lazy: otherOptions.lazy,
       unstable_allowRequireContext:
         this._config.transformer.unstable_allowRequireContext,
     });

package/src/Server/symbolicate.js

@@ -34,7 +34,7 @@ function createFunctionNameGetter(module) {
       source: "dummy",
     });
 }
-async function symbolicate(stack, maps, config) {
+async function symbolicate(stack, maps, config, extraData) {
   const mapsByUrl = new Map();
   for (const [url, map] of maps) {
     mapsByUrl.set(url, map);
@@ -108,7 +108,9 @@ async function symbolicate(stack, maps, config) {
   function symbolicateFrame(frame) {
     const module = findModule(frame);
     if (!module) {
-      return frame;
+      return {
+        ...frame,
+      };
     }
     if (!Array.isArray(module.map)) {
       throw new Error(
@@ -117,7 +119,9 @@ async function symbolicate(stack, maps, config) {
     }
     const originalPos = findOriginalPos(frame, module);
     if (!originalPos) {
-      return frame;
+      return {
+        ...frame,
+      };
     }
     const methodName =
       findFunctionName(originalPos, module) ?? frame.methodName;
@@ -129,15 +133,39 @@ async function symbolicate(stack, maps, config) {
       column: originalPos.column0Based,
     };
   }
+
+  /**
+   * `customizeFrame` allows for custom modifications of the symbolicated frame in a stack.
+   * It can be used to collapse stack frames that are not relevant to users, pointing them
+   * to more relevant product code instead.
+   *
+   * An example usecase is a library throwing an error while sanitizing inputs from product code.
+   * In some cases, it's more useful to point the developer looking at the error towards the product code directly.
+   */
   async function customizeFrame(frame) {
     const customizations =
       (await config.symbolicator.customizeFrame(frame)) || {};
     return {
       ...frame,
-      collapse: false,
       ...customizations,
     };
   }
-  return Promise.all(stack.map(symbolicateFrame).map(customizeFrame));
+
+  /**
+   * `customizeStack` allows for custom modifications of a symbolicated stack.
+   * Where `customizeFrame` operates on individual frames, this hook can process the entire stack in context.
+   *
+   * Note: `customizeStack` has access to an `extraData` object which can be used to attach metadata
+   * to the error coming in, to be used by the customizeStack hook.
+   */
+  async function customizeStack(symbolicatedStack) {
+    return await config.symbolicator.customizeStack(
+      symbolicatedStack,
+      extraData
+    );
+  }
+  return Promise.all(stack.map(symbolicateFrame).map(customizeFrame)).then(
+    customizeStack
+  );
 }
 module.exports = symbolicate;