chore: only take effect in web environment

Author: SoonIter

src/index.ts

@@ -112,6 +112,9 @@ export const pluginAssetsRetry = (
 
     if (inlineScript) {
       api.modifyHTMLTags(async ({ headTags, bodyTags }, { environment }) => {
+        if (environment.config.output.target !== 'web') {
+          return { headTags, bodyTags };
+        }
         const { minify, crossorigin } = getDefaultValueFromRsbuildConfig(
           environment.config,
         );
@@ -131,6 +134,9 @@ export const pluginAssetsRetry = (
     } else {
       api.modifyHTMLTags(
         async ({ headTags, bodyTags }, { assetPrefix, environment }) => {
+          if (environment.config.output.target !== 'web') {
+            return { headTags, bodyTags };
+          }
           const scriptPath = getScriptPath(environment);
           const url = ensureAssetPrefix(scriptPath, assetPrefix);
 
@@ -149,10 +155,13 @@ export const pluginAssetsRetry = (
       api.processAssets(
         { stage: 'additional' },
         async ({ sources, compilation, environment }) => {
+          const { config } = environment;
+          if (config.output.target !== 'web') {
+            return;
+          }
           const scriptPath = getScriptPath(environment);
-          const { crossorigin, minify } = getDefaultValueFromRsbuildConfig(
-            environment.config,
-          );
+          const { crossorigin, minify } =
+            getDefaultValueFromRsbuildConfig(config);
           const runtimeOptions = getRuntimeOptions(userOptions, crossorigin);
           const code = await getRetryCode(runtimeOptions, minify);
           compilation.emitAsset(scriptPath, new sources.RawSource(code));
@@ -161,9 +170,8 @@ export const pluginAssetsRetry = (
     }
 
     api.modifyBundlerChain(async (chain, { environment }) => {
-      const { config, htmlPaths } = environment;
-
-      if (!userOptions || Object.keys(htmlPaths).length === 0) {
+      const { config } = environment;
+      if (config.output.target !== 'web') {
         return;
       }
 

test/node/asyncChunk.js

@@ -0,0 +1 @@
+console.log('asyncChunk');

test/node/index.client.js

@@ -0,0 +1,3 @@
+import('./asyncChunk.js').then(({ default: value }) => {
+  console.log('asyncChunk', value);
+});

test/node/index.server.js

@@ -0,0 +1,5 @@
+const fs = require('node:fs');
+
+module.exports = {
+  myFs: fs,
+};

test/node/index.test.ts

@@ -0,0 +1,31 @@
+import fs from 'node:fs/promises';
+import { expect, test } from '@playwright/test';
+import { createRsbuild } from '@rsbuild/core';
+
+const build = async () => {
+  const rsbuild = await createRsbuild({
+    cwd: import.meta.dirname,
+  });
+  await rsbuild.build();
+};
+
+test('should not work in node environment', async () => {
+  await build();
+
+  // dist/server only contains one file
+  expect((await fs.readdir('dist/server')).length).toBe(1);
+  // only dist/server/index.js exists
+  expect(
+    await fs.access('dist/server/index.js').then(
+      () => true,
+      () => false,
+    ),
+  ).toBe(true);
+
+  // dist/static/js contains two files, index.js and assets-retry.js
+  expect((await fs.readdir('dist/static/js')).length).toBe(2);
+  // index.js contains "registerAsyncChunkRetry" function calling
+  expect(await fs.readFile('dist/static/js/index.js', 'utf-8')).toContain(
+    'registerAsyncChunkRetry',
+  );
+});

test/node/rsbuild.config.ts

@@ -0,0 +1,42 @@
+import { defineConfig } from '@rsbuild/core';
+import { pluginAssetsRetry } from '@rsbuild/plugin-assets-retry';
+import { pluginReact } from '@rsbuild/plugin-react';
+
+export default defineConfig({
+  plugins: [
+    pluginReact(),
+    pluginAssetsRetry({
+      domain: ['http://localhost:3000', 'http://localhost:3001'],
+      inlineScript: false,
+    }),
+  ],
+  environments: {
+    node: {
+      output: {
+        target: 'node',
+        distPath: {
+          root: 'dist/server',
+        },
+      },
+      source: {
+        entry: {
+          index: './index.server.js',
+        },
+      },
+    },
+    web: {
+      source: {
+        entry: {
+          index: './index.client.js',
+        },
+      },
+      output: {
+        minify: false,
+        filenameHash: false,
+      },
+    },
+  },
+  tools: {
+    htmlPlugin: false,
+  },
+});