test ci

Author: fi3ework

packages/core/src/constant.ts

@@ -14,7 +14,7 @@ export const RSLIB_ENTRY_QUERY = '__rslib_entry__';
 export const SHEBANG_PREFIX = '#!';
 export const SHEBANG_REGEX: RegExp = /#!.*[\s\n\r]*/;
 export const REACT_DIRECTIVE_REGEX: RegExp =
-  /^['"]use (client|server)['"](;?)$/;
+  /^['"]use (client|server)['"](;?)[\s\n\r]*$/;
 
 export const JS_EXTENSIONS: string[] = [
   'js',

packages/core/src/plugins/PostEntryChunkPlugin.ts

@@ -1,4 +1,5 @@
 import { createRequire } from 'node:module';
+import os from 'node:os';
 import {
   type RsbuildConfig,
   type RsbuildPlugin,
@@ -17,7 +18,7 @@ const require = createRequire(import.meta.url);
 const PLUGIN_NAME = 'rsbuild:entry';
 
 const matchFirstLine = (source: string, regex: RegExp) => {
-  const [firstLine] = source.split('\n');
+  const [firstLine] = source.split(os.EOL);
   if (!firstLine) {
     return false;
   }
@@ -154,7 +155,7 @@ class PostEntryPlugin {
         {
           name: PLUGIN_NAME,
           // Just after minify stage, to avoid from being minified.
-          stage: rspack.Compilation.PROCESS_ASSETS_STAGE_OPTIMIZE_SIZE + 1,
+          stage: rspack.Compilation.PROCESS_ASSETS_STAGE_DEV_TOOLING - 1,
         },
         (assets) => {
           const chunkAsset = Object.keys(assets);

packages/core/src/plugins/entryModuleLoader.ts

@@ -1,3 +1,4 @@
+import os from 'node:os';
 import type { LoaderDefinition } from '@rspack/core';
 import {
   REACT_DIRECTIVE_REGEX,
@@ -14,7 +15,7 @@ const loader: LoaderDefinition = function loader(source) {
       result = rest1.join('\n');
     }
 
-    const [firstLine, ...rest2] = result.split('\n');
+    const [firstLine, ...rest2] = result.split(os.EOL);
     if (REACT_DIRECTIVE_REGEX.test(firstLine!)) {
       result = rest2.join('\n');
     }

tests/integration/directive/index.test.ts

@@ -7,11 +7,12 @@ describe('shebang', async () => {
   const { entries } = await buildAndGetResults({ fixturePath });
 
   test('shebang at the beginning', async () => {
-    expect(entries.esm0!.startsWith('#!/usr/bin/env node\n')).toBe(true);
+    console.log('😧1', entries.esm0!);
+    expect(entries.esm0!.startsWith('#!/usr/bin/env node')).toBe(true);
   });
 
   test('shebang at the beginning even if minified', async () => {
-    expect(entries.esm1!.startsWith('#!/usr/bin/env node\n')).toBe(true);
+    expect(entries.esm1!.startsWith('#!/usr/bin/env node')).toBe(true);
   });
 });
 
@@ -23,6 +24,7 @@ describe('react', async () => {
       return path.basename(key) === 'bar.js';
     })?.[1];
 
+    console.log('😧2', bar!);
     expect(bar!.startsWith(`'use server';`)).toBe(true);
 
     const foo = Object.entries(contents.esm0!).find(([key]) => {

tests/playwright.config.ts

@@ -7,6 +7,9 @@ export default defineConfig({
   retries: process.env.CI ? 3 : 0,
   // Print line for each test being run in CI
   reporter: 'list',
+  expect: {
+    timeout: process.env.CI ? 10_000 : 5_000,
+  },
   webServer: [
     {
       command: 'cd ../examples/module-federation && npm run dev:host',