feat: restore commonjs bundle (#10984)

auto/auto.cjs

@@ -0,0 +1,6 @@
+const exports = require('../dist/chart.cjs');
+const {Chart, registerables} = exports;
+
+Chart.register(...registerables);
+
+module.exports = Object.assign(Chart, exports);

helpers/helpers.cjs

@@ -0,0 +1 @@
+module.exports = require('../dist/helpers.cjs');

package.json

@@ -15,15 +15,18 @@
     "exports": {
         ".": {
             "types": "./dist/types.d.ts",
-            "import": "./dist/chart.js"
+            "import": "./dist/chart.js",
+            "require": "./dist/chart.cjs"
         },
         "./auto": {
             "types": "./auto/auto.d.ts",
-            "import": "./auto/auto.js"
+            "import": "./auto/auto.js",
+            "require": "./auto/auto.cjs"
         },
         "./helpers": {
             "types": "./helpers/helpers.d.ts",
-            "import": "./helpers/helpers.js"
+            "import": "./helpers/helpers.js",
+            "require": "./helpers/helpers.cjs"
         }
     },
     "types": "./dist/types.d.ts",

pnpm-lock.yaml

@@ -152,6 +152,12 @@ importers:
     dependencies:
       chart.js: link:../../..
 
+  test/integration/node-commonjs:
+    specifiers:
+      chart.js: workspace:*
+    dependencies:
+      chart.js: link:../../..
+
   test/integration/react-browser:
     specifiers:
       '@babel/core': ^7.0.0

rollup.config.js

@@ -77,5 +77,26 @@ export default [
       indent: false,
       sourcemap: true,
     },
+  },
+
+  // CommonJS builds
+  // dist/chart.js
+  // helpers/*.js
+  {
+    input: {
+      'dist/chart': 'src/index.ts',
+      'dist/helpers': 'src/helpers/index.ts'
+    },
+    plugins: plugins(),
+    external: _ => (/node_modules/).test(_),
+    output: {
+      dir: './',
+      chunkFileNames: 'dist/chunks/[name].cjs',
+      entryFileNames: '[name].cjs',
+      banner,
+      format: 'commonjs',
+      indent: false,
+      sourcemap: true,
+    },
   }
 ];

test/integration/node-commonjs/package.json

@@ -0,0 +1,10 @@
+{
+  "private": true,
+  "description": "chart.js should work in Node",
+  "scripts": {
+    "test": "node test.js"
+  },
+  "dependencies": {
+    "chart.js": "workspace:*"
+  }
+}

test/integration/node-commonjs/test.js

@@ -0,0 +1,7 @@
+const {Chart} = require('chart.js');
+const {valueOrDefault} = require('chart.js/helpers');
+
+Chart.register({
+  id: 'TEST_PLUGIN',
+  dummyValue: valueOrDefault(0, 1)
+});

test/integration/node/test.cjs

@@ -1,10 +1,7 @@
-/* eslint-disable es/no-dynamic-import */
-Promise.all([
-  import('chart.js'),
-  import('chart.js/helpers')
-]).then(([{Chart}, {valueOrDefault}]) => {
-  Chart.register({
-    id: 'TEST_PLUGIN',
-    dummyValue: valueOrDefault(0, 1)
-  });
+const {Chart} = require('chart.js');
+const {valueOrDefault} = require('chart.js/helpers');
+
+Chart.register({
+  id: 'TEST_PLUGIN',
+  dummyValue: valueOrDefault(0, 1)
 });