@angular-devkit/build-optimizer 0.1001.0-rc.0 → 0.1001.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-devkit/build-optimizer",
-  "version": "0.1001.0-rc.0",
+  "version": "0.1001.3",
   "description": "Angular Build Optimizer",
   "experimental": true,
   "main": "src/index.js",

package/src/build-optimizer/build-optimizer.js

@@ -15,7 +15,7 @@ const prefix_functions_1 = require("../transforms/prefix-functions");
 const scrub_file_1 = require("../transforms/scrub-file");
 const wrap_enums_1 = require("../transforms/wrap-enums");
 // Angular packages are known to have no side effects.
-const whitelistedAngularModules = [
+const knownSideEffectFreeAngularModules = [
     /[\\/]node_modules[\\/]@angular[\\/]animations[\\/]/,
     /[\\/]node_modules[\\/]@angular[\\/]common[\\/]/,
     /[\\/]node_modules[\\/]@angular[\\/]compiler[\\/]/,
@@ -49,7 +49,7 @@ function isKnownCoreFile(filePath) {
 }
 function isKnownSideEffectFree(filePath) {
     return ngFactories.some((re) => re.test(filePath)) ||
-        whitelistedAngularModules.some((re) => re.test(filePath));
+        knownSideEffectFreeAngularModules.some((re) => re.test(filePath));
 }
 function buildOptimizer(options) {
     const { inputFilePath, isAngularCoreFile } = options;
@@ -82,7 +82,7 @@ function buildOptimizer(options) {
         getTransforms.push(
         // getPrefixFunctionsTransformer is rather dangerous, apply only to known pure es5 modules.
         // It will mark both `require()` calls and `console.log(stuff)` as pure.
-        // We only apply it to whitelisted modules, since we know they are safe.
+        // We only apply it to modules known to be side effect free, since we know they are safe.
         // getPrefixFunctionsTransformer needs to be before getFoldFileTransformer.
         prefix_functions_1.getPrefixFunctionsTransformer, selectedGetScrubFileTransformer);
         typeCheck = true;

package/src/transforms/scrub-file.js

@@ -127,7 +127,7 @@ function isAngularCoreImport(node, isAngularCoreFile) {
         return true;
     }
     // Relative imports from a Angular core file are also core imports.
-    if (isAngularCoreFile && (importText.startsWith('./') || importText.startsWith('../'))) {
+    if (isAngularCoreFile && importText.startsWith('.')) {
         return true;
     }
     return false;
@@ -456,11 +456,13 @@ function isTslibHelper(callExpr, helper, tslibImports, checker) {
     if (!((_a = symbol === null || symbol === void 0 ? void 0 : symbol.declarations) === null || _a === void 0 ? void 0 : _a.length)) {
         return false;
     }
-    for (const name of tslibImports) {
-        for (const dec of symbol.declarations) {
-            if (ts.isImportSpecifier(dec) && name.elements.includes(dec)) {
-                return true;
-            }
+    for (const dec of symbol.declarations) {
+        if (ts.isImportSpecifier(dec) && tslibImports.some(name => name.elements.includes(dec))) {
+            return true;
+        }
+        // Handle inline helpers `var __decorate = (this...`
+        if (ts.isVariableDeclaration(dec)) {
+            return true;
         }
     }
     return false;