fix: validate entry type

Author: Timeless0911

packages/core/src/config.ts

@@ -890,7 +890,7 @@ const composeEntryConfig = async (
   root: string,
   cssModulesAuto: CssLoaderOptionsAuto,
 ): Promise<{ entryConfig: EnvironmentConfig; lcp: string | null }> => {
-  let entries = rawEntry;
+  let entries: RsbuildConfigEntry = rawEntry;
 
   if (!entries) {
     // In bundle mode, return directly to let Rsbuild apply default entry to './src/index.ts'
@@ -904,6 +904,15 @@ const composeEntryConfig = async (
     };
   }
 
+  // Type check to ensure entries is of the expected type in bundleless mode
+  if (typeof entries !== 'object') {
+    throw new Error(
+      `The ${color.cyan('source.entry')} configuration should be an object, but received ${typeof entries}: ${color.cyan(
+        entries,
+      )}. Please check your configuration and ensure it is correctly configured.`,
+    );
+  }
+
   if (bundle !== false) {
     const entryErrorReasons: string[] = [];
     traverseEntryQuery(entries, (entry) => {
@@ -955,6 +964,7 @@ const composeEntryConfig = async (
   const scanGlobEntries = async (calcLcp: boolean) => {
     // In bundleless mode, resolve glob patterns and convert them to entry object.
     const resolvedEntries: Record<string, string> = {};
+    console.log('entries: ', entries);
     for (const key of Object.keys(entries)) {
       const entry = entries[key];
 
@@ -1478,6 +1488,7 @@ export async function composeCreateRsbuildConfig(
     // Reset certain fields because they will be completely overridden by the upcoming merge.
     // We don't want to retain them in the final configuration.
     // The reset process should occur after merging the library configuration.
+    // The reset process should occur after merging the library configuration.
     userConfig.source ??= {};
     userConfig.source.entry = {};
 

tests/integration/entry/index.test.ts

@@ -154,4 +154,17 @@ test('validate entry and throw errors', async () => {
   expect(stripAnsi(errMsg)).toMatchInlineSnapshot(
     `"Error: Can't resolve the entry "./src/main.ts" at the location <ROOT>/tests/integration/entry/validate/src/main.ts. Please ensure that the file exists."`,
   );
+
+  try {
+    await buildAndGetResults({
+      fixturePath,
+      configPath: 'bundlelessWithString.config.ts',
+    });
+  } catch (e) {
+    errMsg = (e as Error).message;
+  }
+
+  expect(stripAnsi(errMsg)).toMatchInlineSnapshot(
+    `"The source.entry configuration should be an object, but received string: ./src/**. Please check your configuration and ensure it is correctly configured."`,
+  );
 });

tests/integration/entry/validate/bundlelessWithString.config.ts

@@ -0,0 +1,14 @@
+import { defineConfig } from '@rslib/core';
+import { generateBundleEsmConfig } from 'test-helper';
+
+export default defineConfig({
+  lib: [
+    generateBundleEsmConfig({
+      bundle: false,
+      source: {
+        // @ts-expect-error test error config
+        entry: './src/**',
+      },
+    }),
+  ],
+});