@meteorjs/rspack 3.0.0-beta.1 → 3.0.0-beta.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@meteorjs/rspack",
-  "version": "3.0.0-beta.1",
+  "version": "3.0.0-beta.2",
   "description": "Configuration logic for using Rspack in Meteor projects",
   "main": "index.js",
   "type": "commonjs",

package/plugins/RequireExtenalsPlugin.js

@@ -164,6 +164,13 @@ class RequireExternalsPlugin {
     }
 
     compiler.hooks.done.tap({ name: this.pluginName, stage: -10 }, (stats) => {
+      // Preserve the last known-good Meteor runtime entry when compilation
+      // fails. Rewriting externals from incomplete stats would make Meteor
+      // restart against a broken Rspack bundle.
+      if (stats.hasErrors()) {
+        return;
+      }
+
       // 1) Ensure globalThis.module / exports block is present if enabled
       if (this._enableGlobalPolyfill) {
         this._ensureGlobalThisModule();

package/rspack.config.js

@@ -441,7 +441,7 @@ module.exports = async function (inMeteor = {}, argv = {}) {
     ],
   };
 
-  if (Meteor.isDebug || Meteor.isVerbose) {
+  if (Meteor.isVerbose) {
     console.log("[i] Rspack mode:", mode);
     console.log("[i] Meteor flags:", Meteor);
   }
@@ -932,8 +932,8 @@ module.exports = async function (inMeteor = {}, argv = {}) {
 
   delete config["meteor.enablePortableBuild"];
 
-  if (Meteor.isDebug || Meteor.isVerbose) {
-  console.log("Config:", inspect(config, { depth: null, colors: true }));
+  if (Meteor.isVerbose) {
+    console.log("Config:", inspect(config, { depth: null, colors: true }));
   }
 
   // Check if lazyCompilation is enabled and warn the user
@@ -950,19 +950,42 @@ module.exports = async function (inMeteor = {}, argv = {}) {
 
   // Add MeteorRspackOutputPlugin as the last plugin to output compilation info
   const meteorRspackOutputPlugin = new MeteorRspackOutputPlugin({
-    getData: (stats, { isRebuild, compilationCount, compiler }) => ({
-      name: config.name,
-      mode: config.mode,
-      hasErrors: stats.hasErrors(),
-      hasWarnings: stats.hasWarnings(),
-      timestamp: Date.now(),
-      statsOverrided,
-      compilationCount,
-      isRebuild,
-      ...(!isRebuild && compiler && {
-        delegatedExtensions: extractDelegatedExtensions(stats, compiler),
-      }),
-    }),
+    getData: (stats, { isRebuild, compilationCount, compiler }) => {
+      const generatedAppPaths = [
+        path.join('.meteor', 'local'),
+        path.join('.meteor', 'types'),
+        buildContext,
+      ];
+      const changedFiles = [
+        ...(compiler?.modifiedFiles || []),
+        ...(compiler?.removedFiles || []),
+      ];
+      const hasAppSourceChanges =
+        changedFiles.length === 0 ||
+        changedFiles.some(file => {
+          const relativePath = path.relative(projectDir, file);
+          return !generatedAppPaths.some(generatedPath =>
+            relativePath === generatedPath ||
+            relativePath.startsWith(`${generatedPath}${path.sep}`)
+          );
+        });
+
+      return {
+        name: config.name,
+        mode: config.mode,
+        hasErrors: stats.hasErrors(),
+        hasWarnings: stats.hasWarnings(),
+        hasAppSourceChanges,
+        hash: stats.hash,
+        timestamp: Date.now(),
+        statsOverrided,
+        compilationCount,
+        isRebuild,
+        ...(!isRebuild && compiler && {
+          delegatedExtensions: extractDelegatedExtensions(stats, compiler),
+        }),
+      };
+    },
   });
   config.plugins = [meteorRspackOutputPlugin, ...(config.plugins || [])];