@meteorjs/rspack 2.0.1 → 2.1.0

package/lib/swc.js

@@ -14,7 +14,13 @@ function getMeteorAppSwcrc(file = '.swcrc') {
       
       if (file.endsWith('.ts')) {
         try {
-          const swc = require('@swc/core');
+          let swc;
+          try {
+            swc = require(require.resolve('@swc/core', { paths: [process.cwd()] }));
+          } catch (e) {
+            if (e.code !== 'MODULE_NOT_FOUND') throw e;
+            swc = require('@swc/core');
+          }
           const result = swc.transformSync(content, {
             jsc: {
               parser: {

package/lib/testClientNodePolyfills.js

@@ -0,0 +1,66 @@
+const { ProvidePlugin } = require("@rspack/core");
+
+let meteorNodeStubs;
+try {
+  const stubPath = require.resolve("meteor-node-stubs", { paths: [process.cwd()] });
+  meteorNodeStubs = require(stubPath);
+} catch (e) {
+  if (e.code !== 'MODULE_NOT_FOUND') throw e;
+  try {
+    meteorNodeStubs = require("meteor-node-stubs");
+  } catch (fallbackError) {
+    if (fallbackError.code !== 'MODULE_NOT_FOUND') throw fallbackError;
+    // It's optional. If not installed, meteorNodeStubs remains undefined.
+  }
+}
+
+const TEST_CLIENT_NODE_POLYFILLS = [
+  "assert",
+  "buffer",
+  "constants",
+  "crypto",
+  "events",
+  "fs",
+  "http",
+  "https",
+  "os",
+  "path",
+  "querystring",
+  "stream",
+  "string_decoder",
+  "sys",
+  "timers",
+  "tty",
+  "url",
+  "util",
+  "vm",
+  "zlib",
+];
+
+function createTestClientNodePolyfillConfig() {
+  const fallback = Object.fromEntries(
+    TEST_CLIENT_NODE_POLYFILLS.map((name) => [name, meteorNodeStubs?.[name] ?? false]),
+  );
+
+  const plugins = [];
+  
+  if (meteorNodeStubs?.buffer) {
+    plugins.push(
+      new ProvidePlugin({
+        Buffer: [meteorNodeStubs.buffer, "Buffer"],
+      })
+    );
+  }
+
+  return {
+    resolve: {
+      alias: {
+        "timers/promises$": require.resolve("isomorphic-timers-promises"),
+      },
+      fallback,
+    },
+    plugins,
+  };
+}
+
+module.exports = { createTestClientNodePolyfillConfig };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@meteorjs/rspack",
-  "version": "2.0.1",
+  "version": "2.1.0",
   "description": "Configuration logic for using Rspack in Meteor projects",
   "main": "index.js",
   "type": "commonjs",
@@ -13,7 +13,7 @@
   "dependencies": {
     "fast-deep-equal": "^3.1.3",
     "ignore-loader": "^0.1.2",
-    "node-polyfill-webpack-plugin": "^4.1.0",
+    "isomorphic-timers-promises": "^1.0.1",
     "webpack-merge": "^6.0.1"
   },
   "devDependencies": {
@@ -22,6 +22,12 @@
   "peerDependencies": {
     "@rspack/cli": ">=1.3.0",
     "@rspack/core": ">=1.3.0",
-    "@swc/core": ">=1.3.0"
+    "@swc/core": ">=1.3.0",
+    "meteor-node-stubs": ">=1.2.0"
+  },
+  "peerDependenciesMeta": {
+    "meteor-node-stubs": {
+      "optional": true
+    }
   }
 }

package/plugins/RequireExtenalsPlugin.js

@@ -13,6 +13,18 @@ const path = require('path');
 // Normalize a path to always use forward slashes (POSIX style).
 // Module identifiers in bundled JS must use '/' regardless of OS.
 const toPosix = (p) => p.replace(/\\/g, '/');
+const escapeRegExp = (value) => value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+const buildStandaloneRequireRegex = (pkg) =>
+  new RegExp(`^\\s*require\\('${escapeRegExp(pkg)}'\\);?.*(\\r?\\n)?`, 'gm');
+const buildStandaloneImportRegex = (pkg) =>
+  new RegExp(
+    `^[\\t ]*import[\\t ]+'${escapeRegExp(pkg)}';?[^\\r\\n]*(?:\\r?\\n|$)`,
+    'gm'
+  );
+const MANAGED_IMPORT_BLOCK_REGEX =
+  /^\/\/\s*\(function\s+(?:eagerExternalImports\d+|lastImports(?:\d+)?)\s*\(\)\s*\{\s*\r?\n[\s\S]*?^\/\/\s*\}\)\s*(?:\r?\n|$)/gm;
+const STANDALONE_REQUIRE_REGEX = /^\s*require\('([^']+)'\)/gm;
+const STANDALONE_IMPORT_REGEX = /^\s*import\s+'([^']+)'/gm;
 
 class RequireExternalsPlugin {
   constructor({
@@ -173,15 +185,22 @@ class RequireExternalsPlugin {
         }
       }
 
-      // 2b) Remove any requires that are no longer in `current`
+      // 2b) Remove any dependencies that are no longer in `current`
       const toRemove = [...existing].filter(p => !current.has(p));
       if (toRemove.length) {
         let content = fs.readFileSync(this.filePath, 'utf-8');
 
-        // Strip stale require(...) lines
+        // Strip stale requires and imports generated by this plugin. Other
+        // standalone imports link Meteor entry and Rspack output files and
+        // must remain intact.
         for (const pkg of toRemove) {
-          const re = new RegExp(`^.*require\\('${pkg}'\\);?.*(\\r?\\n)?`, 'gm');
-          content = content.replace(re, '');
+          const requireRe = buildStandaloneRequireRegex(pkg);
+          const importRe = buildStandaloneImportRegex(pkg);
+          content = content.replace(requireRe, '');
+          content = content.replace(
+            MANAGED_IMPORT_BLOCK_REGEX,
+            (block) => block.replace(importRe, '')
+          );
         }
 
         // Strip out any now-empty helper functions:
@@ -206,11 +225,11 @@ class RequireExternalsPlugin {
         // Re-populate `existing` so the add-diff is accurate
         existing.clear();
         // Check for require statements
-        for (const match of content.matchAll(/require\('([^']+)'\)/g)) {
+        for (const match of content.matchAll(STANDALONE_REQUIRE_REGEX)) {
           existing.add(match[1]);
         }
         // Also check for import statements (used in the new format)
-        for (const match of content.matchAll(/import\s+'([^']+)'/g)) {
+        for (const match of content.matchAll(STANDALONE_IMPORT_REGEX)) {
           existing.add(match[1]);
         }
       }
@@ -481,15 +500,13 @@ class RequireExternalsPlugin {
     try {
       const content = fs.readFileSync(this.filePath, 'utf-8');
       // Check for require statements
-      const requireRegex = /require\('([^']+)'\)/g;
       let match;
-      while ((match = requireRegex.exec(content)) !== null) {
+      while ((match = STANDALONE_REQUIRE_REGEX.exec(content)) !== null) {
         existing.add(match[1]);
       }
 
       // Also check for import statements (used in the new format)
-      const importRegex = /import\s+'([^']+)'/g;
-      while ((match = importRegex.exec(content)) !== null) {
+      while ((match = STANDALONE_IMPORT_REGEX.exec(content)) !== null) {
         existing.add(match[1]);
       }
     } catch {

package/rspack.config.js

@@ -3,7 +3,6 @@ const fs = require('fs');
 const { inspect } = require('node:util');
 const path = require('path');
 const { merge } = require('webpack-merge');
-const NodePolyfillPlugin = require('node-polyfill-webpack-plugin');
 
 const { cleanOmittedPaths, mergeSplitOverlap } = require("./lib/mergeRulesSplitOverlap.js");
 const { getMeteorAppSwcConfig } = require('./lib/swc.js');
@@ -30,7 +29,7 @@ const {
 const { loadUserAndOverrideConfig } = require('./lib/meteorRspackConfigHelpers.js');
 const { prepareMeteorRspackConfig } = require("./lib/meteorRspackConfigFactory");
 const { extractLocalDependencies } = require('./lib/localDependenciesHelpers.js');
-
+const { createTestClientNodePolyfillConfig } = require("./lib/testClientNodePolyfills.js");
 
 // Safe require that doesn't throw if the module isn't found
 function safeRequire(moduleName) {
@@ -127,15 +126,13 @@ function createSwcConfig({
 }) {
   const defaultConfig = {
     jsc: {
-      baseUrl: process.cwd(),
-      paths: { '/*': ['*', '/*'] },
       parser: {
         syntax: isTypescriptEnabled ? 'typescript' : 'ecmascript',
         ...(isTsxEnabled && { tsx: true }),
         ...(isJsxEnabled && { jsx: true }),
         ...(isAngularEnabled && { decorators: true }),
       },
-      target: 'es2015',
+      target: isClient ? 'es2015' : 'es2022',
       ...(isReactEnabled && {
         transform: {
           react: {
@@ -426,7 +423,7 @@ module.exports = async function (inMeteor = {}, argv = {}) {
   // Additional ignore entries
   const additionalEntries = [
     "**/.meteor/local/**",
-    "**/dist/**",
+    path.join(projectDir, "dist", "**").replace(/\\/g, "/"),
     ...(isTest && isTestEager
       ? [`**/${buildContext}/**`, "**/.meteor/local/**", "node_modules/**"]
       : []),
@@ -650,7 +647,7 @@ module.exports = async function (inMeteor = {}, argv = {}) {
         ...extraRules,
       ],
     },
-    resolve: { extensions, alias, fallback },
+    resolve: { extensions, alias, fallback, roots: [path.resolve(projectDir)] },
     externals,
     plugins: [
       ...[
@@ -774,6 +771,7 @@ module.exports = async function (inMeteor = {}, argv = {}) {
       alias,
       modules: ["node_modules", path.resolve(projectDir)],
       conditionNames: ["import", "require", "node", "default"],
+      roots: [path.resolve(projectDir)],
     },
     externals,
     externalsPresets: { node: true },
@@ -841,7 +839,7 @@ module.exports = async function (inMeteor = {}, argv = {}) {
           optimization: {
             splitChunks: false,
           },
-          plugins: [new NodePolyfillPlugin()],
+          ...createTestClientNodePolyfillConfig(),
         }
       : {};
 

package/.claude/settings.local.json

@@ -1,10 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(chmod +x:*)",
-      "Bash(bash:*)",
-      "Bash(npm install:*)",
-      "Bash(node:*)"
-    ]
-  }
-}