Merge pull request #60 from ghiscoding/chore/hybrid-exports

fix: improve hybrid exports CJS/ESM

Author: ghiscoding

packages/excel-builder-vanilla-types/dist/excel-builder.js renamed to packages/excel-builder-vanilla-types/dist/excel-builder.mjs

@@ -21,12 +21,15 @@
   "publishConfig": {
     "access": "public"
   },
-  "main": "./dist/excel-builder.js",
+  "type": "module",
+  "main": "./dist/excel-builder.mjs",
   "types": "./dist/index.d.ts",
   "exports": {
     ".": {
-      "types": "./dist/index.d.ts",
-      "default": "./dist/excel-builder.js"
+      "import": {
+        "types": "./dist/index.d.ts",
+        "default": "./dist/excel-builder.mjs"
+      }
     },
     "./package.json": "./package.json"
   },

packages/excel-builder-vanilla-types/package.json

@@ -10,7 +10,7 @@ copyfiles([source, destination], {}, err => {
   } else {
     // all good, next step, create JS entry file
     const content = `'use strict';`;
-    writeFile(`${destination}/dist/excel-builder.js`, content, err => {
+    writeFile(`${destination}/dist/excel-builder.mjs`, content, err => {
       if (err) {
         console.error(err);
       }

packages/excel-builder-vanilla/copy-types.mjs

@@ -27,17 +27,22 @@
   "publishConfig": {
     "access": "public"
   },
+  "type": "module",
   "main": "./dist/excel-builder.cjs",
   "types": "./dist/index.d.ts",
   "exports": {
     ".": {
-      "types": "./dist/index.d.ts",
-      "import": "./dist/excel-builder.js",
-      "require": "./dist/excel-builder.cjs"
+      "import": {
+        "types": "./dist/index.d.ts",
+        "default": "./dist/excel-builder.mjs"
+      },
+      "require": {
+        "types": "./dist/index.d.cts",
+        "default": "./dist/excel-builder.cjs"
+      }
     },
     "./package.json": "./package.json"
   },
-  "module": "./dist/excel-builder.js",
   "sideEffects": false,
   "funding": {
     "type": "ko_fi",

packages/excel-builder-vanilla/package.json

@@ -1,5 +1,6 @@
 import { dirname, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
+import { writeFileSync } from 'node:fs';
 import { defineConfig } from 'vite';
 import dts from 'vite-plugin-dts';
 
@@ -11,11 +12,10 @@ export default defineConfig({
       entry: resolve(__dirname, 'src/index.ts'),
       formats: ['es', 'cjs', 'iife'],
       name: 'ExcelBuilder',
-      // fileName: format => (format === 'es' ? 'excel-builder.js' : `excel-builder.${format}.js`),
       fileName: format => {
         switch (format) {
           case 'es':
-            return 'excel-builder.js';
+            return 'excel-builder.mjs';
           case 'cjs':
             return 'excel-builder.cjs';
           default:
@@ -41,6 +41,22 @@ export default defineConfig({
         declaration: true,
         declarationMap: true,
       },
+      beforeWriteFile: (filePath, content) => {
+        let safeContent = content;
+        if (filePath.endsWith('dist/index.d.ts')) {
+          if (!safeContent) {
+            safeContent = 'export {};';
+          }
+
+          const ctsFile = filePath.replace('d.ts', 'd.cts');
+          writeFileSync(ctsFile, safeContent);
+        }
+
+        return {
+          filePath,
+          content: safeContent,
+        };
+      },
     }),
   ],
 });