Improve constant module bundling error message (#9803)

* Improve constant module bundling error message * Avoid work when not throwing error
parcel-bundler · Jun 18, 2024 · 53161b4 · 53161b4
1 parent 2ce4efe
commit 53161b4
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/packages/packagers/js/src/ScopeHoistingPackager.js b/packages/packagers/js/src/ScopeHoistingPackager.js
@@ -17,7 +17,7 @@ import {
 } from '@parcel/utils';
 import SourceMap from '@parcel/source-map';
 import nullthrows from 'nullthrows';
-import invariant from 'assert';
+import invariant, {AssertionError} from 'assert';
 import ThrowableDiagnostic, {
  convertSourceLocationToHighlight,
 } from '@parcel/diagnostic';
@@ -922,10 +922,17 @@ ${code}
 
  isWrapped(resolved: Asset, parentAsset: Asset): boolean {
  if (resolved.meta.isConstantModule) {
- invariant(
- this.bundle.hasAsset(resolved),
- 'Constant module not found in bundle',
- );
+ if (!this.bundle.hasAsset(resolved)) {
+ throw new AssertionError({
+ message: `Constant module ${path.relative(
+ this.options.projectRoot,
+ resolved.filePath,
+ )} referenced from ${path.relative(
+ this.options.projectRoot,
+ parentAsset.filePath,
+ )} not found in bundle ${this.bundle.name}`,
+ });
+ }
  return false;
  }
  return (