@next-core/build-next-bricks 1.13.0 → 1.13.1

package/index.d.ts

@@ -2,7 +2,9 @@ import type { Compiler, Configuration, RuleSetRule, container } from "webpack";
 
 export declare function build(config: BuildNextBricksConfig): Compiler;
 export declare function getSvgrLoaders(options?: {
-  /** Set it to true for font icons */
+  /** Set it to true for icon font, defaults to false. */
+  icon?: boolean | string | number;
+  /** Defaults to true when `icon` is truthy, otherwise defaults to false. */
   convertCurrentColor?: boolean;
 }): RuleSetRule["use"];
 
@@ -77,7 +79,7 @@ export interface BuildNextBricksConfig {
   svgAsReactComponent?: boolean;
   /** Customize rules for svg, this will take precedence over `svgAsReactComponent` */
   svgRules?: RuleSetRule[];
-  /** By default the image assets are named `images/[hash][ext][query]` */
+  /** By default the image assets are named `images/[hash][ext]` */
   imageAssetFilename?: string | ((pathData: any, assetInfo: any) => string);
   plugins?: Configuration["plugins"];
   moduleRules?: RuleSetRule[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@next-core/build-next-bricks",
-  "version": "1.13.0",
+  "version": "1.13.1",
   "description": "Build next bricks",
   "homepage": "https://github.com/easyops-cn/next-core/tree/v3/packages/build-next-bricks",
   "license": "GPL-3.0",
@@ -51,5 +51,5 @@
   "devDependencies": {
     "@next-core/brick-manifest": "^0.5.0"
   },
-  "gitHead": "06fca79a3a3b63a995d92729cfbed8e312cfdccd"
+  "gitHead": "ade903abfe7858ec69518eed83a91c43b54bdd4a"
 }

package/src/build.js

@@ -241,6 +241,8 @@ async function getWebpackConfig(config) {
   const outputPath = path.join(packageDir, config.outputPath ?? "dist");
   const chunksDir = isBricks ? "chunks/" : "";
 
+  const imageAssetFilename = config.imageAssetFilename ?? "images/[hash][ext]";
+
   return {
     entry: config.entry || {
       main: "./src/index",
@@ -321,7 +323,7 @@ async function getWebpackConfig(config) {
           ),
           type: "asset/resource",
           generator: {
-            filename: config.imageAssetFilename ?? "images/[hash][ext][query]",
+            filename: imageAssetFilename,
           },
         },
         ...(config.svgRules ??
@@ -329,6 +331,29 @@ async function getWebpackConfig(config) {
             ? [
                 {
                   test: /\.svg$/i,
+                  type: "asset/resource",
+                  // Match `xxx.svg?url`
+                  resourceQuery: /url/,
+                  generator: {
+                    filename: imageAssetFilename,
+                  },
+                },
+                {
+                  test: /\.svg$/i,
+                  // Exclude issuer of js files
+                  issuer: {
+                    not: /\.[jt]sx?$/,
+                  },
+                  type: "asset/resource",
+                  generator: {
+                    filename: imageAssetFilename,
+                  },
+                },
+                {
+                  test: /\.svg$/i,
+                  issuer: /\.[jt]sx?$/,
+                  // Exclude `xxx.svg?url`
+                  resourceQuery: { not: /url/ },
                   use: getSvgrLoaders(),
                 },
               ]

package/src/getSvgrLoaders.js

@@ -10,7 +10,7 @@ export default function getSvgrLoaders(options = {}) {
       loader: "@svgr/webpack",
       options: {
         babel: false,
-        icon: true,
+        icon: options.icon ?? false,
         svgoConfig: {
           plugins: [
             {
@@ -21,7 +21,7 @@ export default function getSvgrLoaders(options = {}) {
                   removeViewBox: false,
 
                   convertColors: {
-                    currentColor: options.convertCurrentColor ?? false,
+                    currentColor: options.convertCurrentColor ?? !!options.icon,
                   },
                 },
               },