@atlaspack/packager-js 2.14.5-canary.99 → 2.14.5-dev.1c70d50f9.99

package/CHANGELOG.md

@@ -1,17 +1,5 @@
 # @atlaspack/packager-js
 
-## 2.16.2
-
-### Patch Changes
-
-- [#654](https://github.com/atlassian-labs/atlaspack/pull/654) [`4d7abde`](https://github.com/atlassian-labs/atlaspack/commit/4d7abde0a6faa148bcf330b08d820f171f853b76) Thanks [@marcins](https://github.com/marcins)! - Fixes constant inlining to work correctly with scope hoisting optimisations
-
-- Updated dependencies [[`5ded263`](https://github.com/atlassian-labs/atlaspack/commit/5ded263c7f11b866e8885b81c73e20dd060b25be)]:
-  - @atlaspack/feature-flags@2.18.3
-  - @atlaspack/utils@2.15.3
-  - @atlaspack/types@2.15.5
-  - @atlaspack/plugin@2.14.15
-
 ## 2.16.1
 
 ### Patch Changes

package/lib/ScopeHoistingPackager.js

@@ -60,13 +60,6 @@ function _featureFlags() {
   };
   return data;
 }
-function _outdent() {
-  const data = require("outdent");
-  _outdent = function () {
-    return data;
-  };
-  return data;
-}
 var _ESMOutputFormat = require("./ESMOutputFormat");
 var _CJSOutputFormat = require("./CJSOutputFormat");
 var _GlobalOutputFormat = require("./GlobalOutputFormat");
@@ -438,9 +431,6 @@ class ScopeHoistingPackager {
     } = (0, _nullthrows().default)(this.assetOutputs.get(asset.id));
     return this.buildAsset(asset, code, map);
   }
-  getAssetFilePath(asset) {
-    return _path().default.relative(this.options.projectRoot, asset.filePath);
-  }
   buildAsset(asset, code, map) {
     let shouldWrap = this.wrappedAssets.has(asset.id);
     let deps = this.bundleGraph.getDependencies(asset);
@@ -538,19 +528,8 @@ class ScopeHoistingPackager {
                 if ((0, _featureFlags().getFeatureFlag)('applyScopeHoistingImprovement')) {
                   if (!resolved.meta.isConstantModule && !this.wrappedAssets.has(resolved.id)) {
                     let [depCode, depMap, depLines] = this.visitAsset(resolved);
-                    if (_utils().debugTools['asset-file-names-in-output']) {
-                      let resolvedPath = this.getAssetFilePath(resolved);
-                      res = (0, _outdent().outdent)`
-                          /* Scope hoisted asset: ${resolvedPath} */
-                          ${depCode}
-                          /* End: ${resolvedPath} */
-                          ${res}
-                        `;
-                      lines += 3 + depLines;
-                    } else {
-                      res = depCode + '\n' + res;
-                      lines += 1 + depLines;
-                    }
+                    res = depCode + '\n' + res;
+                    lines += 1 + depLines;
                     map = depMap;
                   }
                 } else {
@@ -608,10 +587,6 @@ ${code}
 });
 `;
       lineCount += 2;
-      if (_utils().debugTools['asset-file-names-in-output']) {
-        code = `/* ${this.getAssetFilePath(asset)} */\n` + code;
-        lineCount += 1;
-      }
       for (let [depCode, map, lines] of depContent) {
         if (!depCode) continue;
         code += depCode + '\n';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/packager-js",
-  "version": "2.14.5-canary.99+30ee2cfcd",
+  "version": "2.14.5-dev.1c70d50f9.99+1c70d50f9",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -15,17 +15,16 @@
     "node": ">= 16.0.0"
   },
   "dependencies": {
-    "@atlaspack/diagnostic": "2.14.1-canary.167+30ee2cfcd",
-    "@atlaspack/feature-flags": "2.14.1-canary.167+30ee2cfcd",
-    "@atlaspack/plugin": "2.14.5-canary.99+30ee2cfcd",
-    "@atlaspack/rust": "3.2.1-canary.99+30ee2cfcd",
-    "@atlaspack/types": "2.14.5-canary.99+30ee2cfcd",
-    "@atlaspack/utils": "2.14.5-canary.99+30ee2cfcd",
+    "@atlaspack/diagnostic": "2.14.1-dev.1c70d50f9.167+1c70d50f9",
+    "@atlaspack/feature-flags": "2.14.1-dev.1c70d50f9.167+1c70d50f9",
+    "@atlaspack/plugin": "2.14.5-dev.1c70d50f9.99+1c70d50f9",
+    "@atlaspack/rust": "3.2.1-dev.1c70d50f9.99+1c70d50f9",
+    "@atlaspack/types": "2.14.5-dev.1c70d50f9.99+1c70d50f9",
+    "@atlaspack/utils": "2.14.5-dev.1c70d50f9.99+1c70d50f9",
     "@parcel/source-map": "^2.1.1",
     "globals": "^13.2.0",
-    "nullthrows": "^1.1.1",
-    "outdent": "^0.8.0"
+    "nullthrows": "^1.1.1"
   },
   "type": "commonjs",
-  "gitHead": "30ee2cfcd34cf2646ded0eda13fdb80a2a5de529"
+  "gitHead": "1c70d50f914cb662515b0b61053e51a06f3af234"
 }

package/src/ScopeHoistingPackager.js

@@ -15,7 +15,6 @@ import {
   relativeBundlePath,
   countLines,
   normalizeSeparators,
-  debugTools,
 } from '@atlaspack/utils';
 import SourceMap from '@parcel/source-map';
 import nullthrows from 'nullthrows';
@@ -26,7 +25,6 @@ import ThrowableDiagnostic, {
 import globals from 'globals';
 import path from 'path';
 import {getFeatureFlag} from '@atlaspack/feature-flags';
-import {outdent} from 'outdent';
 
 import {ESMOutputFormat} from './ESMOutputFormat';
 import {CJSOutputFormat} from './CJSOutputFormat';
@@ -581,10 +579,6 @@ export class ScopeHoistingPackager {
     return this.buildAsset(asset, code, map);
   }
 
-  getAssetFilePath(asset: Asset): string {
-    return path.relative(this.options.projectRoot, asset.filePath);
-  }
-
   buildAsset(
     asset: Asset,
     code: string,
@@ -726,19 +720,8 @@ export class ScopeHoistingPackager {
                     ) {
                       let [depCode, depMap, depLines] =
                         this.visitAsset(resolved);
-                      if (debugTools['asset-file-names-in-output']) {
-                        let resolvedPath = this.getAssetFilePath(resolved);
-                        res = outdent`
-                          /* Scope hoisted asset: ${resolvedPath} */
-                          ${depCode}
-                          /* End: ${resolvedPath} */
-                          ${res}
-                        `;
-                        lines += 3 + depLines;
-                      } else {
-                        res = depCode + '\n' + res;
-                        lines += 1 + depLines;
-                      }
+                      res = depCode + '\n' + res;
+                      lines += 1 + depLines;
                       map = depMap;
                     }
                   } else {
@@ -809,11 +792,6 @@ ${code}
 
       lineCount += 2;
 
-      if (debugTools['asset-file-names-in-output']) {
-        code = `/* ${this.getAssetFilePath(asset)} */\n` + code;
-        lineCount += 1;
-      }
-
       for (let [depCode, map, lines] of depContent) {
         if (!depCode) continue;
         code += depCode + '\n';