@rsdoctor/core 1.5.1 → 1.5.2

package/dist/build-utils/build/utils/parseBundle.cjs

@@ -41,10 +41,12 @@ const compat_namespaceObject = require("es-toolkit/compat");
 const external_filesize_namespaceObject = require("filesize");
 const ruleUtils_namespaceObject = require("@rsdoctor/utils/ruleUtils");
 const external_path_namespaceObject = require("path");
+const types_namespaceObject = require("@rsdoctor/types");
 const logger_namespaceObject = require("@rsdoctor/utils/logger");
 const parseBundle = (bundlePath, modulesData)=>{
     if (bundlePath.indexOf('.worker.') > 0) return {};
-    if ('.js' !== (0, external_path_namespaceObject.extname)(bundlePath)) return {};
+    const ext = (0, external_path_namespaceObject.extname)(bundlePath);
+    if (!types_namespaceObject.Constants.JSExtensions.includes(ext)) return {};
     let content = external_fs_default().readFileSync(bundlePath, 'utf8');
     const tagCache = new Map();
     const sourceMapRegex = /\/\/# sourceMappingURL=[^\n]*$/m;

package/dist/build-utils/build/utils/parseBundle.js

@@ -5,10 +5,12 @@ import { find, mapValues } from "es-toolkit/compat";
 import { filesize } from "filesize";
 import { parser } from "@rsdoctor/utils/ruleUtils";
 import { extname } from "path";
+import { Constants } from "@rsdoctor/types";
 import { logger } from "@rsdoctor/utils/logger";
 const parseBundle = (bundlePath, modulesData)=>{
     if (bundlePath.indexOf('.worker.') > 0) return {};
-    if ('.js' !== extname(bundlePath)) return {};
+    const ext = extname(bundlePath);
+    if (!Constants.JSExtensions.includes(ext)) return {};
     let content = fs.readFileSync(bundlePath, 'utf8');
     const tagCache = new Map();
     const sourceMapRegex = /\/\/# sourceMappingURL=[^\n]*$/m;

package/dist/inner-plugins/plugins/sourcemapTool.cjs

@@ -154,7 +154,7 @@ async function handleAfterEmitAssets(compilation, _this, sourceMapFilenameRegex,
                         if (outputPath && 'string' == typeof outputPath) sourceMapPath = (0, external_path_namespaceObject.resolve)(outputPath, sourceMapAsset.name);
                     }
                 } else {
-                    if (assetName && 'string' == typeof assetName && (assetName.endsWith('.js') || assetName.endsWith('.css'))) _this.assetsWithoutSourceMap.add(assetName);
+                    if (assetName && 'string' == typeof assetName && (assetName.endsWith('.js') || assetName.endsWith('.bundle') || assetName.endsWith('.css'))) _this.assetsWithoutSourceMap.add(assetName);
                     continue;
                 }
             }
@@ -237,7 +237,7 @@ function parseAsset(asset, assets, type) {
                 };
             }
             assetLinesCodeList = bundledCode.split(/\r?\n/);
-        } else if ('js/css' === type && (assetName.endsWith('.js') || assetName.endsWith('.css'))) {
+        } else if ('js/css' === type && (assetName.endsWith('.js') || assetName.endsWith('.bundle') || assetName.endsWith('.css'))) {
             assetContent = asset.source?.source?.() || '';
             assetLinesCodeList = assetContent.split(/\r?\n/);
             map = asset.source?.sourceAndMap?.()?.map || null;

package/dist/inner-plugins/plugins/sourcemapTool.js

@@ -124,7 +124,7 @@ async function handleAfterEmitAssets(compilation, _this, sourceMapFilenameRegex,
                         if (outputPath && 'string' == typeof outputPath) sourceMapPath = resolve(outputPath, sourceMapAsset.name);
                     }
                 } else {
-                    if (assetName && 'string' == typeof assetName && (assetName.endsWith('.js') || assetName.endsWith('.css'))) _this.assetsWithoutSourceMap.add(assetName);
+                    if (assetName && 'string' == typeof assetName && (assetName.endsWith('.js') || assetName.endsWith('.bundle') || assetName.endsWith('.css'))) _this.assetsWithoutSourceMap.add(assetName);
                     continue;
                 }
             }
@@ -207,7 +207,7 @@ function parseAsset(asset, assets, type) {
                 };
             }
             assetLinesCodeList = bundledCode.split(/\r?\n/);
-        } else if ('js/css' === type && (assetName.endsWith('.js') || assetName.endsWith('.css'))) {
+        } else if ('js/css' === type && (assetName.endsWith('.js') || assetName.endsWith('.bundle') || assetName.endsWith('.css'))) {
             assetContent = asset.source?.source?.() || '';
             assetLinesCodeList = assetContent.split(/\r?\n/);
             map = asset.source?.sourceAndMap?.()?.map || null;

package/dist/rules/rules/ecma-version-check/index.cjs

@@ -55,7 +55,7 @@ const rule = (0, external_rule_cjs_namespaceObject.defineRule)(()=>({
         },
         async check ({ chunkGraph, report, ruleConfig, root, configs }) {
             for (const asset of chunkGraph.getAssets()){
-                if ('.js' !== external_path_default().extname(asset.path)) continue;
+                if ('.js' !== external_path_default().extname(asset.path) && '.bundle' !== external_path_default().extname(asset.path)) continue;
                 const browserslistConfig = (0, external_browserslist_load_config_namespaceObject.loadConfig)({
                     path: root,
                     env: 'production'

package/dist/rules/rules/ecma-version-check/index.js

@@ -19,7 +19,7 @@ const rule = defineRule(()=>({
         },
         async check ({ chunkGraph, report, ruleConfig, root, configs }) {
             for (const asset of chunkGraph.getAssets()){
-                if ('.js' !== path.extname(asset.path)) continue;
+                if ('.js' !== path.extname(asset.path) && '.bundle' !== path.extname(asset.path)) continue;
                 const browserslistConfig = loadConfig({
                     path: root,
                     env: 'production'

package/dist/rules/rules/index.cjs

@@ -31,12 +31,14 @@ const external_default_import_check_index_cjs_namespaceObject = require("./defau
 const external_loader_performance_optimization_index_cjs_namespaceObject = require("./loader-performance-optimization/index.cjs");
 const external_ecma_version_check_index_cjs_namespaceObject = require("./ecma-version-check/index.cjs");
 const external_cross_chunks_package_index_cjs_namespaceObject = require("./cross-chunks-package/index.cjs");
+const external_module_mixed_chunks_index_cjs_namespaceObject = require("./module-mixed-chunks/index.cjs");
 const rules = [
     index_cjs_namespaceObject.rule,
     external_default_import_check_index_cjs_namespaceObject.rule,
     external_loader_performance_optimization_index_cjs_namespaceObject.rule,
     external_ecma_version_check_index_cjs_namespaceObject.rule,
-    external_cross_chunks_package_index_cjs_namespaceObject.rule
+    external_cross_chunks_package_index_cjs_namespaceObject.rule,
+    external_module_mixed_chunks_index_cjs_namespaceObject.rule
 ];
 exports.rules = __webpack_exports__.rules;
 for(var __rspack_i in __webpack_exports__)if (-1 === [

package/dist/rules/rules/index.d.ts

@@ -1 +1 @@
-export declare const rules: (import("@rsdoctor/types/dist/linter").RuleData<import("./duplicate-package/index.js").Config, "duplicate-package"> | import("@rsdoctor/types/dist/linter").RuleData<import("./default-import-check/index.js").Config, "default-import-check"> | import("@rsdoctor/types/dist/linter").RuleData<import("./loader-performance-optimization/index.js").Config, "loader-performance-optimization"> | import("@rsdoctor/types/dist/linter").RuleData<import("./ecma-version-check/index.js").Config, "ecma-version-check"> | import("@rsdoctor/types/dist/linter").RuleData<import("./cross-chunks-package/index.js").Config, "cross-chunks-package">)[];
+export declare const rules: (import("@rsdoctor/types/dist/linter").RuleData<import("./duplicate-package/index.js").Config, "duplicate-package"> | import("@rsdoctor/types/dist/linter").RuleData<import("./default-import-check/index.js").Config, "default-import-check"> | import("@rsdoctor/types/dist/linter").RuleData<import("./loader-performance-optimization/index.js").Config, "loader-performance-optimization"> | import("@rsdoctor/types/dist/linter").RuleData<import("./ecma-version-check/index.js").Config, "ecma-version-check"> | import("@rsdoctor/types/dist/linter").RuleData<import("./cross-chunks-package/index.js").Config, "cross-chunks-package"> | import("@rsdoctor/types/dist/linter").RuleData<import("./module-mixed-chunks/index.js").Config, "module-mixed-chunks">)[];

package/dist/rules/rules/index.js

@@ -5,11 +5,13 @@ import { rule as index_js_rule } from "./default-import-check/index.js";
 import { rule as external_loader_performance_optimization_index_js_rule } from "./loader-performance-optimization/index.js";
 import { rule as external_ecma_version_check_index_js_rule } from "./ecma-version-check/index.js";
 import { rule as external_cross_chunks_package_index_js_rule } from "./cross-chunks-package/index.js";
+import { rule as external_module_mixed_chunks_index_js_rule } from "./module-mixed-chunks/index.js";
 const rules = [
     rule,
     index_js_rule,
     external_loader_performance_optimization_index_js_rule,
     external_ecma_version_check_index_js_rule,
-    external_cross_chunks_package_index_js_rule
+    external_cross_chunks_package_index_js_rule,
+    external_module_mixed_chunks_index_js_rule
 ];
 export { rules };

package/dist/rules/rules/loader-performance-optimization/index.cjs

@@ -63,7 +63,7 @@ const rule = (0, external_rule_cjs_namespaceObject.defineRule)(()=>({
             const cwd = config.context || root;
             const nodeModulesPathRegexp = /\/node_modules\//;
             const resultMap = new Map();
-            const loaders = loader.map((el)=>el.loaders).reduce((t, c)=>t.concat(c));
+            const loaders = loader.flatMap((el)=>el.loaders);
             for (const item of loader){
                 const { path, ext } = item.resource;
                 if ((0, external_utils_cjs_namespaceObject.match)(ext, extensions)) {

package/dist/rules/rules/loader-performance-optimization/index.js

@@ -37,7 +37,7 @@ const rule = defineRule(()=>({
             const cwd = config.context || root;
             const nodeModulesPathRegexp = /\/node_modules\//;
             const resultMap = new Map();
-            const loaders = loader.map((el)=>el.loaders).reduce((t, c)=>t.concat(c));
+            const loaders = loader.flatMap((el)=>el.loaders);
             for (const item of loader){
                 const { path, ext } = item.resource;
                 if (match(ext, extensions)) {

package/dist/rules/rules/module-mixed-chunks/index.cjs

@@ -0,0 +1,85 @@
+"use strict";
+var __webpack_require__ = {};
+(()=>{
+    __webpack_require__.d = (exports1, definition)=>{
+        for(var key in definition)if (__webpack_require__.o(definition, key) && !__webpack_require__.o(exports1, key)) Object.defineProperty(exports1, key, {
+            enumerable: true,
+            get: definition[key]
+        });
+    };
+})();
+(()=>{
+    __webpack_require__.o = (obj, prop)=>Object.prototype.hasOwnProperty.call(obj, prop);
+})();
+(()=>{
+    __webpack_require__.r = (exports1)=>{
+        if ("u" > typeof Symbol && Symbol.toStringTag) Object.defineProperty(exports1, Symbol.toStringTag, {
+            value: 'Module'
+        });
+        Object.defineProperty(exports1, '__esModule', {
+            value: true
+        });
+    };
+})();
+var __webpack_exports__ = {};
+__webpack_require__.r(__webpack_exports__);
+__webpack_require__.d(__webpack_exports__, {
+    rule: ()=>rule
+});
+const types_namespaceObject = require("@rsdoctor/types");
+const external_rule_cjs_namespaceObject = require("../../rule.cjs");
+const title = 'module-mixed-chunks';
+const rule = (0, external_rule_cjs_namespaceObject.defineRule)(()=>({
+        meta: {
+            code: 'E1006',
+            title,
+            category: 'bundle',
+            severity: types_namespaceObject.Linter.Severity.Warn,
+            defaultConfig: {
+                ignore: []
+            }
+        },
+        check ({ moduleGraph, report, ruleConfig }) {
+            const modules = moduleGraph.getModules();
+            for (const module of modules){
+                if (ruleConfig.ignore.some((pattern)=>module.path.includes(pattern))) continue;
+                const moduleChunks = module.getChunks();
+                if (0 === moduleChunks.length) continue;
+                const hasInitial = moduleChunks.some((chunk)=>chunk.initial);
+                const hasAsync = moduleChunks.some((chunk)=>!chunk.initial);
+                if (hasInitial && hasAsync) {
+                    const initialChunks = moduleChunks.filter((chunk)=>chunk.initial);
+                    const asyncChunks = moduleChunks.filter((chunk)=>!chunk.initial);
+                    const detail = {
+                        type: title,
+                        module: {
+                            id: module.id,
+                            path: module.path,
+                            webpackId: module.webpackId
+                        },
+                        initialChunks: initialChunks.map((chunk)=>({
+                                id: chunk.id,
+                                name: chunk.name
+                            })),
+                        asyncChunks: asyncChunks.map((chunk)=>({
+                                id: chunk.id,
+                                name: chunk.name
+                            }))
+                    };
+                    const chunkNames = moduleChunks.map((chunk)=>chunk.name || `chunk-${chunk.id}`).join(', ');
+                    const message = `Module "${module.path}" is included in both initial and async chunks: ${chunkNames}. This should be optimized to avoid code duplication.`;
+                    report({
+                        message,
+                        detail
+                    });
+                }
+            }
+        }
+    }));
+exports.rule = __webpack_exports__.rule;
+for(var __rspack_i in __webpack_exports__)if (-1 === [
+    "rule"
+].indexOf(__rspack_i)) exports[__rspack_i] = __webpack_exports__[__rspack_i];
+Object.defineProperty(exports, '__esModule', {
+    value: true
+});

package/dist/rules/rules/module-mixed-chunks/index.d.ts

@@ -0,0 +1,4 @@
+import { Linter } from '@rsdoctor/types';
+import { Config } from './types.js';
+export type { Config } from './types.js';
+export declare const rule: Linter.RuleData<Config, "module-mixed-chunks">;

package/dist/rules/rules/module-mixed-chunks/index.js

@@ -0,0 +1,53 @@
+import 'module';
+/*#__PURE__*/ import.meta.url;
+import { Linter } from "@rsdoctor/types";
+import { defineRule } from "../../rule.js";
+const title = 'module-mixed-chunks';
+const rule = defineRule(()=>({
+        meta: {
+            code: 'E1006',
+            title,
+            category: 'bundle',
+            severity: Linter.Severity.Warn,
+            defaultConfig: {
+                ignore: []
+            }
+        },
+        check ({ moduleGraph, report, ruleConfig }) {
+            const modules = moduleGraph.getModules();
+            for (const module of modules){
+                if (ruleConfig.ignore.some((pattern)=>module.path.includes(pattern))) continue;
+                const moduleChunks = module.getChunks();
+                if (0 === moduleChunks.length) continue;
+                const hasInitial = moduleChunks.some((chunk)=>chunk.initial);
+                const hasAsync = moduleChunks.some((chunk)=>!chunk.initial);
+                if (hasInitial && hasAsync) {
+                    const initialChunks = moduleChunks.filter((chunk)=>chunk.initial);
+                    const asyncChunks = moduleChunks.filter((chunk)=>!chunk.initial);
+                    const detail = {
+                        type: title,
+                        module: {
+                            id: module.id,
+                            path: module.path,
+                            webpackId: module.webpackId
+                        },
+                        initialChunks: initialChunks.map((chunk)=>({
+                                id: chunk.id,
+                                name: chunk.name
+                            })),
+                        asyncChunks: asyncChunks.map((chunk)=>({
+                                id: chunk.id,
+                                name: chunk.name
+                            }))
+                    };
+                    const chunkNames = moduleChunks.map((chunk)=>chunk.name || `chunk-${chunk.id}`).join(', ');
+                    const message = `Module "${module.path}" is included in both initial and async chunks: ${chunkNames}. This should be optimized to avoid code duplication.`;
+                    report({
+                        message,
+                        detail
+                    });
+                }
+            }
+        }
+    }));
+export { rule };

package/dist/rules/rules/module-mixed-chunks/types.cjs

@@ -0,0 +1,18 @@
+"use strict";
+var __webpack_require__ = {};
+(()=>{
+    __webpack_require__.r = (exports1)=>{
+        if ("u" > typeof Symbol && Symbol.toStringTag) Object.defineProperty(exports1, Symbol.toStringTag, {
+            value: 'Module'
+        });
+        Object.defineProperty(exports1, '__esModule', {
+            value: true
+        });
+    };
+})();
+var __webpack_exports__ = {};
+__webpack_require__.r(__webpack_exports__);
+for(var __rspack_i in __webpack_exports__)exports[__rspack_i] = __webpack_exports__[__rspack_i];
+Object.defineProperty(exports, '__esModule', {
+    value: true
+});

package/dist/rules/rules/module-mixed-chunks/types.d.ts

@@ -0,0 +1,3 @@
+export interface Config {
+    ignore: string[];
+}

package/dist/rules/rules/module-mixed-chunks/types.js

@@ -0,0 +1,2 @@
+import 'module';
+/*#__PURE__*/ import.meta.url;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rsdoctor/core",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "repository": {
     "type": "git",
     "url": "https://github.com/web-infra-dev/rsdoctor",
@@ -69,10 +69,10 @@
     "fs-extra": "^11.1.1",
     "semver": "^7.7.3",
     "source-map": "^0.7.6",
-    "@rsdoctor/graph": "1.5.1",
-    "@rsdoctor/sdk": "1.5.1",
-    "@rsdoctor/utils": "1.5.1",
-    "@rsdoctor/types": "1.5.1"
+    "@rsdoctor/graph": "1.5.2",
+    "@rsdoctor/types": "1.5.2",
+    "@rsdoctor/utils": "1.5.2",
+    "@rsdoctor/sdk": "1.5.2"
   },
   "devDependencies": {
     "axios": "^1.13.4",