@atlaspack/packager-js 2.23.8-dev-compiled-hash-e5f8a1735.0 → 2.24.0

package/CHANGELOG.md

@@ -1,5 +1,36 @@
 # @atlaspack/packager-js
 
+## 2.24.0
+
+### Minor Changes
+
+- [#858](https://github.com/atlassian-labs/atlaspack/pull/858) [`8eb84ee`](https://github.com/atlassian-labs/atlaspack/commit/8eb84ee61a42bfe87c58079b610802b07a6a13e4) Thanks [@marcins](https://github.com/marcins)! - Introduce a new `getReferencedAssets(bundle)` method to the BundleGraph to pre-compute referenced assets, this is used by the scope hoisting packager behind a new `precomputeReferencedAssets` feature flag.
+
+- [#789](https://github.com/atlassian-labs/atlaspack/pull/789) [`73168c2`](https://github.com/atlassian-labs/atlaspack/commit/73168c275a5d9abff9907bcf536b340bca1ed5f0) Thanks [@marcins](https://github.com/marcins)! - Use an in-sourced version of @parcel/source-map, as @atlaspack/source-map
+
+### Patch Changes
+
+- Updated dependencies [[`7dd5bd8`](https://github.com/atlassian-labs/atlaspack/commit/7dd5bd89ff60c7b324347d772729eb4c3314f01e), [`c8a0e51`](https://github.com/atlassian-labs/atlaspack/commit/c8a0e51b1096d038a64644ad02fe68383f58682a), [`8eb84ee`](https://github.com/atlassian-labs/atlaspack/commit/8eb84ee61a42bfe87c58079b610802b07a6a13e4), [`71900f3`](https://github.com/atlassian-labs/atlaspack/commit/71900f3508d53c3882ca8ea71501ba3069687d4e), [`73168c2`](https://github.com/atlassian-labs/atlaspack/commit/73168c275a5d9abff9907bcf536b340bca1ed5f0), [`617a318`](https://github.com/atlassian-labs/atlaspack/commit/617a318ddc9419b38360257353fec50b9051ee13), [`2c4953e`](https://github.com/atlassian-labs/atlaspack/commit/2c4953e20dc1a6b012792081667aefeb9340505e), [`3fe7d4e`](https://github.com/atlassian-labs/atlaspack/commit/3fe7d4e821bee162fcbbaf9278b107a5eead944f)]:
+  - @atlaspack/rust@3.11.0
+  - @atlaspack/feature-flags@2.27.0
+  - @atlaspack/source-map@3.1.0
+  - @atlaspack/utils@3.2.0
+  - @atlaspack/plugin@2.14.39
+  - @atlaspack/types@2.15.29
+
+## 2.23.8
+
+### Patch Changes
+
+- [#882](https://github.com/atlassian-labs/atlaspack/pull/882) [`13d23e8`](https://github.com/atlassian-labs/atlaspack/commit/13d23e8a23091f92eb1b8dd7c9a452fe33cf1aac) Thanks [@mattcompiles](https://github.com/mattcompiles)! - Fix an issue where in rare cases, scope hoisting could cause 'Cannot find module'
+
+- Updated dependencies [[`ec3abe4`](https://github.com/atlassian-labs/atlaspack/commit/ec3abe4dffc98560a850fd2f71fb566577e6c99c), [`c7fe3f7`](https://github.com/atlassian-labs/atlaspack/commit/c7fe3f76f247e9e20299e205e2df0a16c418eaf2), [`cfb4707`](https://github.com/atlassian-labs/atlaspack/commit/cfb4707087498e4fa4dcf10753fe984a248d196b), [`1468695`](https://github.com/atlassian-labs/atlaspack/commit/1468695fc0c9d06f060a6da9e9b0e154f11dff34), [`f31b041`](https://github.com/atlassian-labs/atlaspack/commit/f31b04107e9077c9946aadb99f6f91bb69703bb7), [`8180981`](https://github.com/atlassian-labs/atlaspack/commit/8180981be14c00f9570adb70d3f350bd91d6ec0a), [`ae77e74`](https://github.com/atlassian-labs/atlaspack/commit/ae77e7452a466b43b3fa5bed24d4ba26345ed765), [`f0a496f`](https://github.com/atlassian-labs/atlaspack/commit/f0a496f70fc8652e090cf1b3f6260e8cfbb796e2), [`7d7a55d`](https://github.com/atlassian-labs/atlaspack/commit/7d7a55dd6395ec391a2e4c33b3dec0d1ea477d4c), [`540f253`](https://github.com/atlassian-labs/atlaspack/commit/540f253dfdcd1a5caebbdc0b197319d439404aae)]:
+  - @atlaspack/feature-flags@2.26.2
+  - @atlaspack/rust@3.10.0
+  - @atlaspack/utils@3.1.2
+  - @atlaspack/plugin@2.14.38
+  - @atlaspack/types@2.15.28
+
 ## 2.23.7
 
 ### Patch Changes

package/dist/DevPackager.js

@@ -5,7 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.DevPackager = void 0;
 const utils_1 = require("@atlaspack/utils");
-const source_map_1 = __importDefault(require("@parcel/source-map"));
+const source_map_1 = __importDefault(require("@atlaspack/source-map"));
 const feature_flags_1 = require("@atlaspack/feature-flags");
 const assert_1 = __importDefault(require("assert"));
 const path_1 = __importDefault(require("path"));
@@ -162,7 +162,6 @@ class DevPackager {
             // @ts-expect-error TS2339
             script.code, entryMap, this.parcelRequireName);
             if (this.bundle.env.sourceMap && entryMap) {
-                // @ts-expect-error TS2551
                 map.addSourceMap(entryMap, lineOffset);
             }
         }

package/dist/ScopeHoistingPackager.js

@@ -38,7 +38,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ScopeHoistingPackager = void 0;
 const utils_1 = require("@atlaspack/utils");
-const source_map_1 = __importDefault(require("@parcel/source-map"));
+const source_map_1 = __importDefault(require("@atlaspack/source-map"));
 const nullthrows_1 = __importDefault(require("nullthrows"));
 const assert_1 = __importStar(require("assert"));
 const diagnostic_1 = __importStar(require("@atlaspack/diagnostic"));
@@ -95,6 +95,7 @@ class ScopeHoistingPackager {
         this.externalAssets = new Set();
         this.useBothScopeHoistingImprovements = (0, feature_flags_1.getFeatureFlag)('applyScopeHoistingImprovementV2') ||
             (0, feature_flags_1.getFeatureFlag)('applyScopeHoistingImprovement');
+        this.referencedAssetsCache = new Map();
         this.options = options;
         this.bundleGraph = bundleGraph;
         this.bundle = bundle;
@@ -135,7 +136,6 @@ class ScopeHoistingPackager {
             this.seenHoistedRequires.clear();
             let [content, map, lines] = this.visitAsset(asset);
             if (sourceMap && map) {
-                // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
                 sourceMap.addSourceMap(map, lineCount);
             }
             else if (this.bundle.env.sourceMap) {
@@ -301,6 +301,36 @@ class ScopeHoistingPackager {
         ];
         return `$parcel$global.rwr(${params.join(', ')});`;
     }
+    // Helper to check if an asset is referenced, with cache-first + fast-check hybrid approach
+    isAssetReferencedInBundle(bundle, asset) {
+        // STEP 1: Check expensive computation cache first (fastest when it hits)
+        if ((0, feature_flags_1.getFeatureFlag)('precomputeReferencedAssets')) {
+            let bundleId = bundle.id;
+            let referencedAssets = this.referencedAssetsCache.get(bundleId);
+            if (referencedAssets) {
+                // Cache hit - fastest path (~0.001ms)
+                return referencedAssets.has(asset);
+            }
+        }
+        // STEP 2: Cache miss - try fast checks (~0.01ms)
+        let fastCheckResult = this.bundleGraph.isAssetReferencedFastCheck(bundle, asset);
+        if (fastCheckResult === true) {
+            // Fast check succeeded - asset is referenced
+            return true;
+        }
+        // STEP 3: Need expensive computation (~20-2000ms)
+        if ((0, feature_flags_1.getFeatureFlag)('precomputeReferencedAssets')) {
+            // Compute and cache expensive results for this bundle
+            let bundleId = bundle.id;
+            let referencedAssets = this.bundleGraph.getReferencedAssets(bundle);
+            this.referencedAssetsCache.set(bundleId, referencedAssets);
+            return referencedAssets.has(asset);
+        }
+        else {
+            // No caching - fall back to original expensive method
+            return this.bundleGraph.isAssetReferenced(bundle, asset);
+        }
+    }
     async loadAssets() {
         let queue = new utils_1.PromiseQueue({
             maxConcurrent: 32,
@@ -315,7 +345,7 @@ class ScopeHoistingPackager {
             });
             if (asset.meta.shouldWrap ||
                 this.bundle.env.sourceType === 'script' ||
-                this.bundleGraph.isAssetReferenced(this.bundle, asset) ||
+                this.isAssetReferencedInBundle(this.bundle, asset) ||
                 this.bundleGraph
                     .getIncomingDependencies(asset)
                     .some((dep) => dep.meta.shouldWrap && dep.specifierType !== 'url')) {
@@ -515,6 +545,11 @@ class ScopeHoistingPackager {
                 if (this.bundle.hasAsset(resolved) && !this.seenAssets.has(resolved)) {
                     if (this.useBothScopeHoistingImprovements &&
                         this.wrappedAssets.has(resolved)) {
+                        if (this.wrappedAssets.has(asset)) {
+                            // If both the asset and the dep are wrapped there's no need to
+                            // drop a side-effect require. This is an extremely rare case.
+                            continue;
+                        }
                         // When the dep is wrapped then we just need to drop a side effect
                         // require instead of inlining
                         depCode += `parcelRequire("${this.bundleGraph.getAssetPublicId(resolved)}");\n`;
@@ -524,7 +559,6 @@ class ScopeHoistingPackager {
                         let [code, map, lines] = this.visitAsset(resolved);
                         depCode += code + '\n';
                         if (sourceMap && map) {
-                            // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
                             sourceMap.addSourceMap(map, lineCount);
                         }
                         lineCount += lines + 1;
@@ -647,7 +681,6 @@ class ScopeHoistingPackager {
                                     sourceMap.offsetLines(lineCount + 1, lines);
                                 }
                                 if (map) {
-                                    // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
                                     sourceMap.addSourceMap(map, lineCount);
                                 }
                             }
@@ -691,7 +724,6 @@ ${code}
                     continue;
                 code += depCode + '\n';
                 if (sourceMap && map) {
-                    // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
                     sourceMap.addSourceMap(map, lineCount);
                 }
                 lineCount += lines + 1;
@@ -737,7 +769,7 @@ ${code}
                 if (referencedBundle &&
                     referencedBundle.getMainEntry() === resolved &&
                     referencedBundle.type === 'js' &&
-                    !this.bundleGraph.isAssetReferenced(referencedBundle, resolved)) {
+                    !this.isAssetReferencedInBundle(referencedBundle, resolved)) {
                     this.addExternal(dep, replacements, referencedBundle);
                     this.externalAssets.add(resolved);
                     continue;
@@ -1343,7 +1375,7 @@ ${code}
         }
         return (asset.sideEffects === false &&
             (0, nullthrows_1.default)(this.bundleGraph.getUsedSymbols(asset)).size == 0 &&
-            !this.bundleGraph.isAssetReferenced(this.bundle, asset));
+            !this.isAssetReferencedInBundle(this.bundle, asset));
     }
     isScriptEntry(asset) {
         return (this.bundle.env.outputFormat === 'global' &&

package/lib/DevPackager.js

@@ -12,7 +12,7 @@ function _utils() {
   return data;
 }
 function _sourceMap() {
-  const data = _interopRequireDefault(require("@parcel/source-map"));
+  const data = _interopRequireDefault(require("@atlaspack/source-map"));
   _sourceMap = function () {
     return data;
   };
@@ -182,7 +182,6 @@ class DevPackager {
       // @ts-expect-error TS2339
       script.code, entryMap, this.parcelRequireName);
       if (this.bundle.env.sourceMap && entryMap) {
-        // @ts-expect-error TS2551
         map.addSourceMap(entryMap, lineOffset);
       }
     }

package/lib/ScopeHoistingPackager.js

@@ -12,7 +12,7 @@ function _utils() {
   return data;
 }
 function _sourceMap2() {
-  const data = _interopRequireDefault(require("@parcel/source-map"));
+  const data = _interopRequireDefault(require("@atlaspack/source-map"));
   _sourceMap2 = function () {
     return data;
   };
@@ -110,6 +110,7 @@ class ScopeHoistingPackager {
   usedHelpers = new Set();
   externalAssets = new Set();
   useBothScopeHoistingImprovements = (0, _featureFlags().getFeatureFlag)('applyScopeHoistingImprovementV2') || (0, _featureFlags().getFeatureFlag)('applyScopeHoistingImprovement');
+  referencedAssetsCache = new Map();
   constructor(options, bundleGraph, bundle, parcelRequireName, useAsyncBundleRuntime, manualStaticBindingExports, logger) {
     this.options = options;
     this.bundleGraph = bundleGraph;
@@ -147,7 +148,6 @@ class ScopeHoistingPackager {
       this.seenHoistedRequires.clear();
       let [content, map, lines] = this.visitAsset(asset);
       if (sourceMap && map) {
-        // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
         sourceMap.addSourceMap(map, lineCount);
       } else if (this.bundle.env.sourceMap) {
         sourceMap = map;
@@ -300,6 +300,38 @@ class ScopeHoistingPackager {
     let params = [JSON.stringify(this.bundle.publicId), (0, _helpers.fnExpr)(this.bundle.env, [], [codeToRun]), `${JSON.stringify(deps)}${conditions.length > 0 ? `.concat([${conditions.map(conditions => `...${conditions}`).join(',')}])` : ''}`];
     return `$parcel$global.rwr(${params.join(', ')});`;
   }
+
+  // Helper to check if an asset is referenced, with cache-first + fast-check hybrid approach
+  isAssetReferencedInBundle(bundle, asset) {
+    // STEP 1: Check expensive computation cache first (fastest when it hits)
+    if ((0, _featureFlags().getFeatureFlag)('precomputeReferencedAssets')) {
+      let bundleId = bundle.id;
+      let referencedAssets = this.referencedAssetsCache.get(bundleId);
+      if (referencedAssets) {
+        // Cache hit - fastest path (~0.001ms)
+        return referencedAssets.has(asset);
+      }
+    }
+
+    // STEP 2: Cache miss - try fast checks (~0.01ms)
+    let fastCheckResult = this.bundleGraph.isAssetReferencedFastCheck(bundle, asset);
+    if (fastCheckResult === true) {
+      // Fast check succeeded - asset is referenced
+      return true;
+    }
+
+    // STEP 3: Need expensive computation (~20-2000ms)
+    if ((0, _featureFlags().getFeatureFlag)('precomputeReferencedAssets')) {
+      // Compute and cache expensive results for this bundle
+      let bundleId = bundle.id;
+      let referencedAssets = this.bundleGraph.getReferencedAssets(bundle);
+      this.referencedAssetsCache.set(bundleId, referencedAssets);
+      return referencedAssets.has(asset);
+    } else {
+      // No caching - fall back to original expensive method
+      return this.bundleGraph.isAssetReferenced(bundle, asset);
+    }
+  }
   async loadAssets() {
     let queue = new (_utils().PromiseQueue)({
       maxConcurrent: 32
@@ -312,7 +344,7 @@ class ScopeHoistingPackager {
           map
         }];
       });
-      if (asset.meta.shouldWrap || this.bundle.env.sourceType === 'script' || this.bundleGraph.isAssetReferenced(this.bundle, asset) || this.bundleGraph.getIncomingDependencies(asset).some(dep => dep.meta.shouldWrap && dep.specifierType !== 'url')) {
+      if (asset.meta.shouldWrap || this.bundle.env.sourceType === 'script' || this.isAssetReferencedInBundle(this.bundle, asset) || this.bundleGraph.getIncomingDependencies(asset).some(dep => dep.meta.shouldWrap && dep.specifierType !== 'url')) {
         // Don't wrap constant "entry" modules _except_ if they are referenced by any lazy dependency
         if (!asset.meta.isConstantModule || this.bundleGraph.getIncomingDependencies(asset).some(dep => dep.priority === 'lazy')) {
           this.wrappedAssets.add(asset);
@@ -506,6 +538,12 @@ class ScopeHoistingPackager {
         }
         if (this.bundle.hasAsset(resolved) && !this.seenAssets.has(resolved)) {
           if (this.useBothScopeHoistingImprovements && this.wrappedAssets.has(resolved)) {
+            if (this.wrappedAssets.has(asset)) {
+              // If both the asset and the dep are wrapped there's no need to
+              // drop a side-effect require. This is an extremely rare case.
+              continue;
+            }
+
             // When the dep is wrapped then we just need to drop a side effect
             // require instead of inlining
             depCode += `parcelRequire("${this.bundleGraph.getAssetPublicId(resolved)}");\n`;
@@ -514,7 +552,6 @@ class ScopeHoistingPackager {
             let [code, map, lines] = this.visitAsset(resolved);
             depCode += code + '\n';
             if (sourceMap && map) {
-              // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
               sourceMap.addSourceMap(map, lineCount);
             }
             lineCount += lines + 1;
@@ -633,7 +670,6 @@ class ScopeHoistingPackager {
                   sourceMap.offsetLines(lineCount + 1, lines);
                 }
                 if (map) {
-                  // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
                   sourceMap.addSourceMap(map, lineCount);
                 }
               }
@@ -678,7 +714,6 @@ ${code}
         if (!depCode) continue;
         code += depCode + '\n';
         if (sourceMap && map) {
-          // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
           sourceMap.addSourceMap(map, lineCount);
         }
         lineCount += lines + 1;
@@ -716,7 +751,7 @@ ${code}
       // Handle imports from other bundles in libraries.
       if (this.bundle.env.isLibrary && !this.bundle.hasAsset(resolved)) {
         let referencedBundle = this.bundleGraph.getReferencedBundle(dep, this.bundle);
-        if (referencedBundle && referencedBundle.getMainEntry() === resolved && referencedBundle.type === 'js' && !this.bundleGraph.isAssetReferenced(referencedBundle, resolved)) {
+        if (referencedBundle && referencedBundle.getMainEntry() === resolved && referencedBundle.type === 'js' && !this.isAssetReferencedInBundle(referencedBundle, resolved)) {
           this.addExternal(dep, replacements, referencedBundle);
           this.externalAssets.add(resolved);
           continue;
@@ -1252,7 +1287,7 @@ ${code}
     if (this.isScriptEntry(asset)) {
       return true;
     }
-    return asset.sideEffects === false && (0, _nullthrows().default)(this.bundleGraph.getUsedSymbols(asset)).size == 0 && !this.bundleGraph.isAssetReferenced(this.bundle, asset);
+    return asset.sideEffects === false && (0, _nullthrows().default)(this.bundleGraph.getUsedSymbols(asset)).size == 0 && !this.isAssetReferencedInBundle(this.bundle, asset);
   }
   isScriptEntry(asset) {
     return this.bundle.env.outputFormat === 'global' && this.bundle.env.sourceType === 'script' && asset === this.bundle.getMainEntry();

package/lib/types/DevPackager.d.ts

@@ -1,5 +1,5 @@
 import type { BundleGraph, PluginOptions, NamedBundle } from '@atlaspack/types';
-import SourceMap from '@parcel/source-map';
+import SourceMap from '@atlaspack/source-map';
 export declare class DevPackager {
     options: PluginOptions;
     bundleGraph: BundleGraph<NamedBundle>;

package/lib/types/ScopeHoistingPackager.d.ts

@@ -1,5 +1,5 @@
 import type { Asset, BundleGraph, Dependency, PluginOptions, NamedBundle, PluginLogger } from '@atlaspack/types';
-import SourceMap from '@parcel/source-map';
+import SourceMap from '@atlaspack/source-map';
 export interface OutputFormat {
     buildBundlePrelude(): [string, number];
     buildBundlePostlude(): [string, number];
@@ -44,10 +44,12 @@ export declare class ScopeHoistingPackager {
     externalAssets: Set<Asset>;
     logger: PluginLogger;
     useBothScopeHoistingImprovements: boolean;
+    referencedAssetsCache: Map<string, Set<Asset>>;
     constructor(options: PluginOptions, bundleGraph: BundleGraph<NamedBundle>, bundle: NamedBundle, parcelRequireName: string, useAsyncBundleRuntime: boolean, manualStaticBindingExports: string[] | null, logger: PluginLogger);
     package(): Promise<PackageResult>;
     shouldBundleQueue(bundle: NamedBundle): boolean;
     runWhenReady(bundle: NamedBundle, codeToRun: string): string;
+    isAssetReferencedInBundle(bundle: NamedBundle, asset: Asset): boolean;
     loadAssets(): Promise<void>;
     isReExported(asset: Asset): boolean;
     buildExportedSymbols(): void;

package/lib/types/utils.d.ts

@@ -1,5 +1,5 @@
 import type { BundleGraph, Dependency, NamedBundle } from '@atlaspack/types';
-import type SourceMap from '@parcel/source-map';
+import type SourceMap from '@atlaspack/source-map';
 export declare function replaceScriptDependencies(bundleGraph: BundleGraph<NamedBundle>, bundle: NamedBundle, code: string, map: SourceMap | null | undefined, parcelRequireName: string): string;
 export declare function getSpecifier(dep: Dependency): string;
 export declare function isValidIdentifier(id: string): boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/packager-js",
-  "version": "2.23.8-dev-compiled-hash-e5f8a1735.0",
+  "version": "2.24.0",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -16,13 +16,13 @@
     "node": ">= 16.0.0"
   },
   "dependencies": {
-    "@atlaspack/diagnostic": "2.14.5-dev-compiled-hash-e5f8a1735.0",
-    "@atlaspack/feature-flags": "2.26.2-dev-compiled-hash-e5f8a1735.0",
-    "@atlaspack/plugin": "2.14.38-dev-compiled-hash-e5f8a1735.0",
-    "@atlaspack/rust": "3.9.2-dev-compiled-hash-e5f8a1735.0",
-    "@atlaspack/types": "2.15.28-dev-compiled-hash-e5f8a1735.0",
-    "@atlaspack/utils": "3.1.2-dev-compiled-hash-e5f8a1735.0",
-    "@parcel/source-map": "^2.1.1",
+    "@atlaspack/diagnostic": "2.14.4",
+    "@atlaspack/feature-flags": "2.27.0",
+    "@atlaspack/plugin": "2.14.39",
+    "@atlaspack/rust": "3.11.0",
+    "@atlaspack/source-map": "3.1.0",
+    "@atlaspack/types": "2.15.29",
+    "@atlaspack/utils": "3.2.0",
     "globals": "^13.2.0",
     "nullthrows": "^1.1.1",
     "outdent": "^0.8.0"
@@ -30,6 +30,5 @@
   "type": "commonjs",
   "scripts": {
     "build:lib": "gulp build --gulpfile ../../../gulpfile.js --cwd ."
-  },
-  "gitHead": "e5f8a173505611c1fafafd6e7dddb2f6b483f67c"
+  }
 }

package/src/DevPackager.ts

@@ -6,7 +6,7 @@ import {
   countLines,
   normalizeSeparators,
 } from '@atlaspack/utils';
-import SourceMap from '@parcel/source-map';
+import SourceMap from '@atlaspack/source-map';
 import {getFeatureFlag} from '@atlaspack/feature-flags';
 import invariant from 'assert';
 import path from 'path';
@@ -229,7 +229,6 @@ export class DevPackager {
         this.parcelRequireName,
       );
       if (this.bundle.env.sourceMap && entryMap) {
-        // @ts-expect-error TS2551
         map.addSourceMap(entryMap, lineOffset);
       }
     }

package/src/ScopeHoistingPackager.ts

@@ -16,7 +16,7 @@ import {
   debugTools,
   globToRegex,
 } from '@atlaspack/utils';
-import SourceMap from '@parcel/source-map';
+import SourceMap from '@atlaspack/source-map';
 import nullthrows from 'nullthrows';
 import invariant, {AssertionError} from 'assert';
 import ThrowableDiagnostic, {
@@ -131,6 +131,7 @@ export class ScopeHoistingPackager {
   useBothScopeHoistingImprovements: boolean =
     getFeatureFlag('applyScopeHoistingImprovementV2') ||
     getFeatureFlag('applyScopeHoistingImprovement');
+  referencedAssetsCache: Map<string, Set<Asset>> = new Map();
 
   constructor(
     options: PluginOptions,
@@ -190,7 +191,6 @@ export class ScopeHoistingPackager {
       let [content, map, lines] = this.visitAsset(asset);
 
       if (sourceMap && map) {
-        // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
         sourceMap.addSourceMap(map, lineCount);
       } else if (this.bundle.env.sourceMap) {
         sourceMap = map;
@@ -412,6 +412,44 @@ export class ScopeHoistingPackager {
     return `$parcel$global.rwr(${params.join(', ')});`;
   }
 
+  // Helper to check if an asset is referenced, with cache-first + fast-check hybrid approach
+  isAssetReferencedInBundle(bundle: NamedBundle, asset: Asset): boolean {
+    // STEP 1: Check expensive computation cache first (fastest when it hits)
+    if (getFeatureFlag('precomputeReferencedAssets')) {
+      let bundleId = bundle.id;
+      let referencedAssets = this.referencedAssetsCache.get(bundleId);
+
+      if (referencedAssets) {
+        // Cache hit - fastest path (~0.001ms)
+        return referencedAssets.has(asset);
+      }
+    }
+
+    // STEP 2: Cache miss - try fast checks (~0.01ms)
+    let fastCheckResult = this.bundleGraph.isAssetReferencedFastCheck(
+      bundle,
+      asset,
+    );
+
+    if (fastCheckResult === true) {
+      // Fast check succeeded - asset is referenced
+      return true;
+    }
+
+    // STEP 3: Need expensive computation (~20-2000ms)
+    if (getFeatureFlag('precomputeReferencedAssets')) {
+      // Compute and cache expensive results for this bundle
+      let bundleId = bundle.id;
+      let referencedAssets = this.bundleGraph.getReferencedAssets(bundle);
+      this.referencedAssetsCache.set(bundleId, referencedAssets);
+
+      return referencedAssets.has(asset);
+    } else {
+      // No caching - fall back to original expensive method
+      return this.bundleGraph.isAssetReferenced(bundle, asset);
+    }
+  }
+
   async loadAssets() {
     type QueueItem = [Asset, {code: string; map: Buffer | undefined | null}];
     let queue = new PromiseQueue<QueueItem>({
@@ -431,7 +469,7 @@ export class ScopeHoistingPackager {
       if (
         asset.meta.shouldWrap ||
         this.bundle.env.sourceType === 'script' ||
-        this.bundleGraph.isAssetReferenced(this.bundle, asset) ||
+        this.isAssetReferencedInBundle(this.bundle, asset) ||
         this.bundleGraph
           .getIncomingDependencies(asset)
           .some((dep) => dep.meta.shouldWrap && dep.specifierType !== 'url')
@@ -692,6 +730,12 @@ export class ScopeHoistingPackager {
             this.useBothScopeHoistingImprovements &&
             this.wrappedAssets.has(resolved)
           ) {
+            if (this.wrappedAssets.has(asset)) {
+              // If both the asset and the dep are wrapped there's no need to
+              // drop a side-effect require. This is an extremely rare case.
+              continue;
+            }
+
             // When the dep is wrapped then we just need to drop a side effect
             // require instead of inlining
             depCode += `parcelRequire("${this.bundleGraph.getAssetPublicId(resolved)}");\n`;
@@ -700,7 +744,6 @@ export class ScopeHoistingPackager {
             let [code, map, lines] = this.visitAsset(resolved);
             depCode += code + '\n';
             if (sourceMap && map) {
-              // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
               sourceMap.addSourceMap(map, lineCount);
             }
             lineCount += lines + 1;
@@ -856,7 +899,6 @@ export class ScopeHoistingPackager {
                   }
 
                   if (map) {
-                    // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
                     sourceMap.addSourceMap(map, lineCount);
                   }
                 }
@@ -913,7 +955,6 @@ ${code}
         if (!depCode) continue;
         code += depCode + '\n';
         if (sourceMap && map) {
-          // @ts-expect-error TS2551 - addSourceMap method exists but missing from @parcel/source-map type definitions
           sourceMap.addSourceMap(map, lineCount);
         }
         lineCount += lines + 1;
@@ -987,7 +1028,7 @@ ${code}
           referencedBundle &&
           referencedBundle.getMainEntry() === resolved &&
           referencedBundle.type === 'js' &&
-          !this.bundleGraph.isAssetReferenced(referencedBundle, resolved)
+          !this.isAssetReferencedInBundle(referencedBundle, resolved)
         ) {
           this.addExternal(dep, replacements, referencedBundle);
           this.externalAssets.add(resolved);
@@ -1786,7 +1827,7 @@ ${code}
     return (
       asset.sideEffects === false &&
       nullthrows(this.bundleGraph.getUsedSymbols(asset)).size == 0 &&
-      !this.bundleGraph.isAssetReferenced(this.bundle, asset)
+      !this.isAssetReferencedInBundle(this.bundle, asset)
     );
   }
 

package/src/index.ts

@@ -1,5 +1,5 @@
 import type {Async, BundleResult} from '@atlaspack/types';
-import type SourceMap from '@parcel/source-map';
+import type SourceMap from '@atlaspack/source-map';
 import {Packager} from '@atlaspack/plugin';
 import {
   replaceInlineReferences,

package/src/utils.ts

@@ -1,5 +1,5 @@
 import type {BundleGraph, Dependency, NamedBundle} from '@atlaspack/types';
-import type SourceMap from '@parcel/source-map';
+import type SourceMap from '@atlaspack/source-map';
 import nullthrows from 'nullthrows';
 
 // This replaces __parcel__require__ references left by the transformer with

package/tsconfig.json

@@ -22,6 +22,9 @@
     },
     {
       "path": "../../core/utils/tsconfig.json"
+    },
+    {
+      "path": "../../core/source-map/tsconfig.json"
     }
   ]
 }