@ngtools/webpack 12.1.0-next.6 → 12.1.0

package/README.md

@@ -37,4 +37,4 @@ The loader works with webpack plugin to compile the application's TypeScript. It
 - `jitMode` [default: `false`] - Enables JIT compilation and do not refactor the code to bootstrap. This replaces `templateUrl: "string"` with `template: require("string")` (and similar for styles) to allow for webpack to properly link the resources.
 - `directTemplateLoading` [default: `true`] - Causes the plugin to load component templates (HTML) directly from the filesystem. This is more efficient if only using the `raw-loader` to load component templates. Do not enable this option if additional loaders are configured for component templates.
 - `fileReplacements` [default: none] - Allows replacing TypeScript files with other TypeScript files in the build. This option acts on fully resolved file paths.
-- `inlineStyleMimeType` [default: none] - When set to a valid MIME type, enables conversion of an Angular Component's inline styles into data URIs. This allows a Webpack 5 configuration rule to use the `mimetype` condition to process the inline styles. A valid MIME type is a string starting with `text/` (Example for CSS: `text/css`).
+- `inlineStyleFileExtension` [default: none] - When set inline component styles will be processed by Webpack as files with the provided extension.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngtools/webpack",
-  "version": "12.1.0-next.6",
+  "version": "12.1.0",
   "description": "Webpack plugin that AoT compiles your Angular components and modules.",
   "main": "./src/index.js",
   "typings": "src/index.d.ts",

package/src/ivy/host.js

@@ -30,7 +30,6 @@ exports.augmentHostWithCaching = exports.augmentProgramWithVersioning = exports.
 const crypto_1 = require("crypto");
 const path = __importStar(require("path"));
 const ts = __importStar(require("typescript"));
-const transformers_1 = require("../transformers");
 const paths_1 = require("./paths");
 function augmentHostWithResources(host, resourceLoader, options = {}) {
     const resourceHost = host;
@@ -248,10 +247,6 @@ function augmentHostWithCaching(host, cache) {
         }
         const file = baseGetSourceFile.call(host, fileName, languageVersion, onError, true, ...parameters);
         if (file) {
-            // Temporary workaround for upstream transform resource defect
-            if (file && !file.isDeclarationFile && file.text.includes('@Component')) {
-                transformers_1.workaroundStylePreprocessing(file);
-            }
             cache.set(fileName, file);
         }
         return file;

package/src/ivy/plugin.d.ts

@@ -16,6 +16,7 @@ export interface AngularWebpackPluginOptions {
     emitClassMetadata: boolean;
     emitNgModuleScope: boolean;
     jitMode: boolean;
+    /** @deprecated use `inlineStyleFileExtension` instead. */
     inlineStyleMimeType?: string;
     inlineStyleFileExtension?: string;
 }

package/src/transformers/replace_resources.d.ts

@@ -8,4 +8,3 @@
 import * as ts from 'typescript';
 export declare function replaceResources(shouldTransform: (fileName: string) => boolean, getTypeChecker: () => ts.TypeChecker, directTemplateLoading?: boolean, inlineStyleMimeType?: string, inlineStyleFileExtension?: string): ts.TransformerFactory<ts.SourceFile>;
 export declare function getResourceUrl(node: ts.Node, loader?: string): string | null;
-export declare function workaroundStylePreprocessing(sourceFile: ts.SourceFile): void;

package/src/transformers/replace_resources.js

@@ -26,7 +26,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
     return result;
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.workaroundStylePreprocessing = exports.getResourceUrl = exports.replaceResources = void 0;
+exports.getResourceUrl = exports.replaceResources = void 0;
 const ts = __importStar(require("typescript"));
 const inlineDataLoaderPath = require.resolve('../inline-data-loader');
 function replaceResources(shouldTransform, getTypeChecker, directTemplateLoading = false, inlineStyleMimeType, inlineStyleFileExtension) {
@@ -222,62 +222,3 @@ function getDecoratorOrigin(decorator, typeChecker) {
     }
     return null;
 }
-function workaroundStylePreprocessing(sourceFile) {
-    const visitNode = (node) => {
-        var _a;
-        if (ts.isClassDeclaration(node) && ((_a = node.decorators) === null || _a === void 0 ? void 0 : _a.length)) {
-            for (const decorator of node.decorators) {
-                visitDecoratorWorkaround(decorator);
-            }
-        }
-        return ts.forEachChild(node, visitNode);
-    };
-    ts.forEachChild(sourceFile, visitNode);
-}
-exports.workaroundStylePreprocessing = workaroundStylePreprocessing;
-function visitDecoratorWorkaround(node) {
-    if (!ts.isCallExpression(node.expression)) {
-        return;
-    }
-    const decoratorFactory = node.expression;
-    if (!ts.isIdentifier(decoratorFactory.expression) ||
-        decoratorFactory.expression.text !== 'Component') {
-        return;
-    }
-    const args = decoratorFactory.arguments;
-    if (args.length !== 1 || !ts.isObjectLiteralExpression(args[0])) {
-        // Unsupported component metadata
-        return;
-    }
-    const objectExpression = args[0];
-    // check if a `styles` property is present
-    let hasStyles = false;
-    for (const element of objectExpression.properties) {
-        if (!ts.isPropertyAssignment(element) || ts.isComputedPropertyName(element.name)) {
-            continue;
-        }
-        if (element.name.text === 'styles') {
-            hasStyles = true;
-            break;
-        }
-    }
-    if (hasStyles) {
-        return;
-    }
-    const nodeFactory = ts.factory;
-    // add a `styles` property to workaround upstream compiler defect
-    const emptyArray = nodeFactory.createArrayLiteralExpression();
-    const stylePropertyName = nodeFactory.createIdentifier('styles');
-    const styleProperty = nodeFactory.createPropertyAssignment(stylePropertyName, emptyArray);
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    stylePropertyName.parent = styleProperty;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    emptyArray.parent = styleProperty;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    styleProperty.parent = objectExpression;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    objectExpression.properties = nodeFactory.createNodeArray([
-        ...objectExpression.properties,
-        styleProperty,
-    ]);
-}