bunchee 6.3.2 → 6.3.4

package/README.md

@@ -41,12 +41,12 @@ Add the exports in `package.json`.
 
 ```json5
 {
-  name: 'coffee',
-  type: 'module',
-  main: './dist/index.js',
-  scripts: {
-    build: 'bunchee',
-  },
+  "name": "coffee",
+  "type": "module",
+  "main": "./dist/index.js",
+  "scripts": {
+    "build": "bunchee"
+  }
 }
 ```
 
@@ -73,23 +73,23 @@ Here's a example of entry files and exports configuration:
 
 ```json5
 {
-  name: 'coffee',
-  scripts: {
-    build: 'bunchee',
+  "name": "coffee",
+  "scripts": {
+    "build: "bunchee",
   },
-  type: 'module',
-  exports: {
+  "type": "module",
+  "exports": {
     // entry: ./src/index.ts
-    '.': {
-      import: './dist/index.js',
-      require: './dist/index.cjs',
+    ".": {
+      "import": "./dist/index.js",
+      "require": "./dist/index.cjs",
     },
 
     // entry: ./src/lite.ts
-    './lite': './dist/lite.js',
+    "./lite": "./dist/lite.js",
 
     // entry: ./src/react/index.ts
-    './react': './dist/react.js',
+    "./react": "./dist/react.js",
   },
 }
 ```
@@ -124,15 +124,15 @@ The `dependencies` and `peerDependencies` will be marked as externalized and won
 ```json5
 {
   // Externalized
-  dependencies: {
+  "dependencies": {
     /* ... */
   },
-  peerDependencies: {
+  "peerDependencies": {
     /* ... */
   },
 
   // Bundled
-  devDependencies: {
+  "devDependencies": {
     /* ... */
   },
 }
@@ -146,10 +146,10 @@ For instance:
 
 ```json5
 {
-  exports: {
-    'react-server': './dist/react-server.mjs',
-    'edge-light': './dist/edge-light.mjs',
-    import: './dist/index.mjs',
+  "exports": {
+    "react-server": "./dist/react-server.mjs",
+    "edge-light": "./dist/edge-light.mjs",
+    "import": "./dist/index.mjs",
   },
 }
 ```
@@ -161,8 +161,8 @@ For instance:
 ```json5
 // package.json
 {
-  imports: {
-    '#util': './src/utils.ts',
+  "imports": {
+    "#util": "./src/utils.ts",
   },
 }
 ```
@@ -183,7 +183,7 @@ This will match the `bin` field in package.json as:
 
 ```json5
 {
-  bin: './dist/bin.js',
+  "bin": "./dist/bin.js",
 }
 ```
 
@@ -204,9 +204,9 @@ This will match the `bin` field in package.json as:
 
 ```json5
 {
-  bin: {
-    foo: './dist/bin/a.js',
-    bar: './dist/bin/b.js',
+  "bin": {
+    "foo": "./dist/bin/a.js",
+    "bar": "./dist/bin/b.js",
   },
 }
 ```
@@ -338,14 +338,14 @@ Then use use the [exports field in package.json](https://nodejs.org/api/packages
 
 ```json5
 {
-  files: ['dist'],
-  type: 'module',
-  exports: {
-    '.': './dist/es/index.js',
-    './react': './dist/es/react.js',
+  "files": ["dist"],
+  "type": "module",
+  "exports": {
+    ".": "./dist/es/index.js",
+    "./react": "./dist/es/react.js",
   },
-  scripts: {
-    build: 'bunchee',
+  "scripts": {
+    "build": "bunchee",
   },
 }
 ```
@@ -359,21 +359,21 @@ If you're building a TypeScript library, separate the types from the main entry
 
 ```json5
 {
-  files: ['dist'],
-  type: 'module',
-  main: './dist/index.js',
-  exports: {
-    '.': {
-      types: './dist/index.d.ts',
-      default: './dist/index.js',
+  "files": ["dist"],
+  "type": "module",
+  "main": "./dist/index.js",
+  "exports": {
+    ".": {
+      "types": "./dist/index.d.ts",
+      "default": "./dist/index.js",
     },
-    './react': {
-      types: './dist/react/index.d.ts',
-      default: './dist/react/index.js',
+    "./react": {
+      "types": "./dist/react/index.d.ts",
+      "default": "./dist/react/index.js",
     },
   },
-  scripts: {
-    build: 'bunchee',
+  "scripts": {
+    "build": "bunchee",
   },
 }
 ```
@@ -388,23 +388,23 @@ _NOTE_: When you're using `.mjs` or `.cjs` extensions with TypeScript and modern
 
 ```json5
 {
-  files: ['dist'],
-  type: 'module',
-  main: './dist/index.js',
-  module: './dist/index.js',
-  types: './dist/index.d.ts',
-  exports: {
-    import: {
-      types: './dist/index.d.ts',
-      default: './dist/index.js',
+  "files": ["dist"],
+  "type": "module",
+  "main": "./dist/index.js",
+  "module": "./dist/index.js",
+  "types": "./dist/index.d.ts",
+  "exports": {
+    "import": {
+      "types": "./dist/index.d.ts",
+      "default": "./dist/index.js",
     },
-    require: {
-      types: './dist/index.d.cts',
-      default: './dist/index.cjs',
+    "require": {
+      "types": "./dist/index.d.cts",
+      "default": "./dist/index.cjs",
     },
   },
-  scripts: {
-    build: 'bunchee',
+  "scripts": {
+    "build": "bunchee",
   },
 }
 ```
@@ -450,9 +450,9 @@ This will match the export name `"react-server"` and `"edge-light"` then use the
 
 ```json5
 {
-  exports: {
-    development: './dist/index.development.js',
-    production: './dist/index.production.js',
+  "exports": {
+    "development": './dist/index.development.js',
+    "production": './dist/index.production.js',
   },
 }
 ```
@@ -499,8 +499,8 @@ hello world
 
 output
 
-```
-export default "hello world"
+```js
+export default 'hello world'
 ```
 
 ### Node.js API
@@ -538,11 +538,11 @@ For example:
 
 ```json5
 {
-  browserslist: [
-    'last 1 version',
-    '> 1%',
-    'maintained node versions',
-    'not dead',
+  "browserslist": [
+    "last 1 version",
+    "> 1%",
+    "maintained node versions",
+    "not dead",
   ],
 }
 ```

package/dist/bin/cli.js

@@ -176,9 +176,10 @@ function validateEntryFiles(entryFiles) {
         // Check if there are multiple files with the same base name
         const filePathWithoutExt = filePath.slice(0, -path__default.default.extname(filePath).length).replace(/\\/g, '/');
         const segments = filePathWithoutExt.split('/');
-        const lastSegment = segments.pop() || '';
-        if (lastSegment !== 'index' && lastSegment !== '') {
-            segments.push(lastSegment);
+        let lastSegment = segments[segments.length - 1];
+        while(lastSegment && (lastSegment === 'index' || lastSegment === '')){
+            segments.pop();
+            lastSegment = segments[segments.length - 1];
         }
         const fileBasePath = segments.join('/');
         if (fileBasePaths.has(fileBasePath)) {
@@ -646,7 +647,7 @@ function lint$1(pkg) {
     }
 }
 
-var version = "6.3.2";
+var version = "6.3.4";
 
 async function writeDefaultTsconfig(tsConfigPath) {
     await fs.promises.writeFile(tsConfigPath, JSON.stringify(DEFAULT_TS_CONFIG, null, 2), 'utf-8');
@@ -1049,6 +1050,16 @@ function logOutputState(stats) {
     });
 }
 
+function normalizeError(error) {
+    // Remove the noise from rollup plugin error
+    if (error.code === 'PLUGIN_ERROR') {
+        const normalizedError = new Error(error.message);
+        normalizedError.stack = error.stack;
+        error = normalizedError;
+    }
+    return error;
+}
+
 const helpMessage = `
 Usage: bunchee [options]
 
@@ -1265,10 +1276,14 @@ async function run(args) {
             }
         }
     }
+    function onBuildError(error) {
+        logError(error);
+    }
     let buildError;
     bundleConfig._callbacks = {
         onBuildStart,
-        onBuildEnd
+        onBuildEnd,
+        onBuildError
     };
     if (watch) {
         logger.log(`Watching changes in the project directory...`);
@@ -1354,11 +1369,8 @@ function logWatcherBuildTime(result, spinner) {
         });
     });
 }
-function logError(error) {
-    if (!error) return;
-    // logging source code in format
-    if (error.frame) {
-        process.stderr.write(error.frame + '\n');
-    }
+function logError(err) {
+    const error = normalizeError(err);
+    logger.error(error);
 }
 main().catch(exit);

package/dist/index.d.ts

@@ -24,6 +24,7 @@ type BundleConfig = {
     _callbacks?: {
         onBuildStart?: (state: any) => void;
         onBuildEnd?: (assetJobs: any) => void;
+        onBuildError?: (assetJob: any) => void;
     };
 };
 type PackageMetadata = {

package/dist/index.js

@@ -277,9 +277,10 @@ function validateEntryFiles(entryFiles) {
         // Check if there are multiple files with the same base name
         const filePathWithoutExt = filePath.slice(0, -path__default.default.extname(filePath).length).replace(/\\/g, '/');
         const segments = filePathWithoutExt.split('/');
-        const lastSegment = segments.pop() || '';
-        if (lastSegment !== 'index' && lastSegment !== '') {
-            segments.push(lastSegment);
+        let lastSegment = segments[segments.length - 1];
+        while(lastSegment && (lastSegment === 'index' || lastSegment === '')){
+            segments.pop();
+            lastSegment = segments[segments.length - 1];
         }
         const fileBasePath = segments.join('/');
         if (fileBasePaths.has(fileBasePath)) {
@@ -1368,10 +1369,19 @@ function aliasEntries({ entry: sourceFilePath, conditionNames, entries, format,
                 })) == null ? undefined : _exportMapEntries_find1.bundlePath;
             }
         } else {
-            var _exportMapEntries_find2;
-            matchedBundlePath = (_exportMapEntries_find2 = exportMapEntries.find((item)=>{
+            var _exportMapEntries_sort_find;
+            matchedBundlePath = (_exportMapEntries_sort_find = exportMapEntries.sort(// always put special condition after the general condition (default, cjs, esm)
+            (a, b)=>{
+                if (a.conditionNames.has(specialCondition)) {
+                    return -1;
+                }
+                if (b.conditionNames.has(specialCondition)) {
+                    return 1;
+                }
+                return 0;
+            }).find((item)=>{
                 return findJsBundlePathCallback(item, specialCondition);
-            })) == null ? undefined : _exportMapEntries_find2.bundlePath;
+            })) == null ? undefined : _exportMapEntries_sort_find.bundlePath;
         }
         if (matchedBundlePath) {
             if (!sourceToRelativeBundleMap.has(exportCondition.source)) sourceToRelativeBundleMap.set(exportCondition.source, matchedBundlePath);
@@ -1499,6 +1509,7 @@ async function buildInputConfig(entry, bundleConfig, exportCondition, buildConte
         decorators: true
     };
     const hasBrowserslistConfig = !!(browserslistConfig && !hasSpecifiedTsTarget);
+    const sourceMap = bundleConfig.sourcemap;
     const swcOptions = {
         jsc: {
             ...!hasSpecifiedTsTarget && !hasBrowserslistConfig && {
@@ -1513,11 +1524,11 @@ async function buildInputConfig(entry, bundleConfig, exportCondition, buildConte
             ...shouldMinify && {
                 minify: {
                     ...swcMinifyOptions,
-                    sourceMap: bundleConfig.sourcemap
+                    sourceMap: sourceMap
                 }
             }
         },
-        sourceMaps: bundleConfig.sourcemap,
+        sourceMaps: sourceMap,
         inlineSourcesContent: false,
         isModule: true,
         ...hasBrowserslistConfig && {
@@ -1714,7 +1725,7 @@ async function buildOutputConfigs(bundleConfig, exportCondition, buildContext, d
         interop: 'auto',
         freeze: false,
         strict: false,
-        sourcemap: bundleConfig.sourcemap,
+        sourcemap: bundleConfig.sourcemap && !dts,
         manualChunks: createSplitChunks(pluginContext.moduleDirectiveLayerMap, entryFiles),
         chunkFileNames () {
             const isCjsFormat = format === 'cjs';
@@ -1828,6 +1839,16 @@ async function buildConfig(bundleConfig, exportCondition, pluginContext, bundleE
     return Promise.all(outputConfigs);
 }
 
+function normalizeError(error) {
+    // Remove the noise from rollup plugin error
+    if (error.code === 'PLUGIN_ERROR') {
+        const normalizedError = new Error(error.message);
+        normalizedError.stack = error.stack;
+        error = normalizedError;
+    }
+    return error;
+}
+
 async function createAssetRollupJobs(options, buildContext, bundleJobOptions) {
     const { isFromCli, generateTypes } = bundleJobOptions;
     const assetsConfigs = await buildEntryConfig(options, buildContext, {
@@ -1845,7 +1866,12 @@ async function createAssetRollupJobs(options, buildContext, bundleJobOptions) {
         }
     }
     const rollupJobs = allConfigs.map((rollupConfig)=>bundleOrWatch(options, rollupConfig));
-    return await Promise.all(rollupJobs);
+    try {
+        return await Promise.all(rollupJobs);
+    } catch (err) {
+        const error = normalizeError(err);
+        throw error;
+    }
 }
 async function bundleOrWatch(options, rollupConfig) {
     if (options.watch) {
@@ -1891,7 +1917,7 @@ function hasMultiEntryExport(exportPaths) {
     return exportKeys.length > 0 && exportKeys.every((name)=>name.startsWith('.'));
 }
 async function bundle(cliEntryPath, { cwd: _cwd, ...options } = {}) {
-    var _options__callbacks_onBuildStart, _options__callbacks, _options__callbacks_onBuildEnd, _options__callbacks1;
+    var _options__callbacks_onBuildStart, _options__callbacks;
     const cwd = path.resolve(process.cwd(), _cwd || '');
     assignDefault(options, 'format', 'esm');
     assignDefault(options, 'minify', false);
@@ -1995,8 +2021,15 @@ async function bundle(cliEntryPath, { cwd: _cwd, ...options } = {}) {
         isFromCli,
         generateTypes
     };
-    const assetJobs = await createAssetRollupJobs(options, buildContext, rollupJobsOptions);
-    (_options__callbacks1 = options._callbacks) == null ? undefined : (_options__callbacks_onBuildEnd = _options__callbacks1.onBuildEnd) == null ? undefined : _options__callbacks_onBuildEnd.call(_options__callbacks1, assetJobs);
+    try {
+        var _options__callbacks_onBuildEnd, _options__callbacks1;
+        const assetJobs = await createAssetRollupJobs(options, buildContext, rollupJobsOptions);
+        (_options__callbacks1 = options._callbacks) == null ? void 0 : (_options__callbacks_onBuildEnd = _options__callbacks1.onBuildEnd) == null ? void 0 : _options__callbacks_onBuildEnd.call(_options__callbacks1, assetJobs);
+    } catch (error) {
+        var _options__callbacks_onBuildError, _options__callbacks2;
+        (_options__callbacks2 = options._callbacks) == null ? undefined : (_options__callbacks_onBuildError = _options__callbacks2.onBuildError) == null ? undefined : _options__callbacks_onBuildError.call(_options__callbacks2, error);
+        return Promise.reject(error);
+    }
 }
 
 exports.bundle = bundle;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bunchee",
-  "version": "6.3.2",
+  "version": "6.3.4",
   "description": "zero config bundler for js/ts/jsx libraries",
   "bin": "./dist/bin/cli.js",
   "main": "./dist/index.js",
@@ -72,10 +72,8 @@
   "devDependencies": {
     "@huozhi/testing-package": "1.0.0",
     "@swc-node/register": "^1.10.9",
-    "@swc/jest": "^0.2.37",
     "@swc/types": "^0.1.17",
     "@types/clean-css": "^4.2.11",
-    "@types/jest": "29.0.0",
     "@types/node": "^22.9.3",
     "@types/picomatch": "^3.0.1",
     "@types/react": "19.0.1",
@@ -83,42 +81,22 @@
     "bunchee": "link:./",
     "cross-env": "^7.0.3",
     "husky": "^9.0.11",
-    "jest": "29.7.0",
     "lint-staged": "^15.2.2",
     "next": "^15.0.4",
     "picocolors": "^1.0.0",
-    "prettier": "2.8.8",
+    "prettier": "3.4.2",
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
-    "typescript": "^5.7.2"
+    "typescript": "^5.7.2",
+    "vitest": "^3.0.4"
   },
   "lint-staged": {
     "*.{js,jsx,ts,tsx,md,json,yml,yaml}": "prettier --write"
   },
-  "jest": {
-    "moduleDirectories": [
-      "node_modules"
-    ],
-    "moduleNameMapper": {
-      "^bunchee$": "<rootDir>/src/index.ts"
-    },
-    "transform": {
-      "^.+\\.(t|j)sx?$": [
-        "@swc/jest"
-      ]
-    },
-    "testPathIgnorePatterns": [
-      "/node_modules/",
-      "<rootDir>/test/integration/.*/*src",
-      "<rootDir>/test/fixtures"
-    ],
-    "testTimeout": 60000
-  },
   "scripts": {
-    "test": "jest --env node",
-    "test:ci": "pnpm test -- --ci",
+    "test": "vitest run",
     "test:update": "TEST_UPDATE_SNAPSHOT=1 pnpm test",
-    "test:post": "cross-env POST_BUILD=1 pnpm jest test/compile.test.ts test/integration.test.ts",
+    "test:post": "cross-env POST_BUILD=1 pnpm test test/compile.test.ts test/integration.test.ts",
     "docs:dev": "next dev docs",
     "docs:build": "next build docs",
     "clean": "rm -rf ./dist",