Merge pull request #25 from metaskills/DualPublish

Use esbuild for Dual Package Approach

.github/workflows/publish.yml

@@ -14,6 +14,10 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: 18
+      - name: Install dependencies
+        run: npm ci
+      - name: Build
+        run: ./bin/build
       - name: Publish
         uses: JS-DevTools/npm-publish@v3
         with:

CHANGELOG.md

@@ -2,6 +2,16 @@
 
 See this http://keepachangelog.com link for information on how we want this document formatted.
 
+## v1.5.0
+
+### Removed
+
+Subpath imports. Please import or require only "experts".
+
+### Added/Changed
+
+Use dual package approach. Now supports both ES6 import syntax and CommonJS require statements.
+
 ## v1.4.3
 
 ### Added 

README.md

@@ -35,6 +35,8 @@ Install via npm. Usage is very simple, there are only three objects to import.
 npm install experts
 ```
 
+Experts.js supports both ES6 import syntax and CommonJS require statements.
+
 ```javascript
 import { Assistant, Tool, Thread } from "experts";
 ```

bin/build

@@ -0,0 +1,4 @@
+#!/bin/sh
+set -e
+
+node build.js

bin/setup

@@ -1,6 +1,11 @@
 #!/bin/sh
 set -e
 
-npm install
+if [ "$CI" = "true" ]; then
+  npm ci
+else
+  npm install
+fi
+
 node test/products/create.js
 node test/products/index.js

build.js

@@ -0,0 +1,27 @@
+import * as esbuild from "esbuild";
+
+async function build() {
+  // Build ESM version
+  await esbuild.build({
+    entryPoints: ["src/index.js"],
+    outfile: "dist/index.js",
+    format: "esm",
+    platform: "node",
+    target: "node14",
+    bundle: true,
+  });
+
+  // Build CommonJS version
+  await esbuild.build({
+    entryPoints: ["src/index.js"],
+    outfile: "dist/index.cjs",
+    format: "cjs",
+    platform: "node",
+    target: "node14",
+    bundle: true,
+  });
+
+  console.log("Build complete");
+}
+
+build().catch(console.error);