Extend Node.js support to include ^10

Author: lahmatiy

.github/workflows/build.yml

@@ -43,6 +43,7 @@ jobs:
             matrix:
                 # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
                 node_version:
+                    - 10
                     - 12.20.0
                     - 14.13.0
                     - 16
@@ -56,6 +57,7 @@ jobs:
                 cache: "npm"
             - run: npm ci
             - run: npm run test
+              if: ${{ matrix.node_version != '10' }}
             - run: npm run esm-to-cjs-and-test
 
             - run: npm run coverage

CHANGELOG.md

@@ -1,5 +1,6 @@
 ## next
 
+- Extended Node.js support to include `^10`
 - Fixed `main` field in `package.json` to refer to CommonJS module
 - Bumped `css-tree` to `~2.0.4` (fixed CSS serialization issue in IE11)
 
@@ -13,7 +14,7 @@
 
 - Added support for [CSS Selectors Level 4](https://www.w3.org/TR/selectors-4/) and legacy pseudos in specificity calculation, i.e. `:has()`, `:not(<selector-list>)`, `:is()`, `:matches()`, `:-moz-any()`, `:-webkit-any()`, `:where()`, `:nth-child(... of <selector-list>)` and `:nth-last-child(... of <selector-list>)`
 - Package
-    - Changed supported versions of Node.js to `^12.20.0`, `^14.13.0` and `>=15.0.0`
+    - Changed supported versions of Node.js to `^12.20.0`, `^14.13.0` and `>=15.0.0` (extended in `5.0.2` to include `^10`)
     - Converted to ES modules. However, CommonJS is supported as well (dual module)
     - Changed bundle set to provide `dist/csso.js` (an IIFE version with `csso` as a global name) and `dist/csso.esm.js` (as ES module). Both are minified
     - Bumped `css-tree` to [`2.0`](https://github.com/csstree/csstree/releases/tag/v2.0.0)

package-lock.json

@@ -51,7 +51,7 @@
     "build-and-test": "npm run build && npm run test:dist && npm run test:cjs",
     "bundle": "node scripts/bundle",
     "bundle-and-test": "npm run bundle && npm run test:dist",
-    "esm-to-cjs": "node scripts/esm-to-cjs",
+    "esm-to-cjs": "node scripts/esm-to-cjs.cjs",
     "esm-to-cjs-and-test": "npm run esm-to-cjs && npm run test:cjs",
     "coverage": "c8 --reporter=lcovonly npm test",
     "prepublishOnly": "npm run lint-and-test && npm run build-and-test",
@@ -69,7 +69,7 @@
     "source-map-js": "^1.0.1"
   },
   "engines": {
-    "node": "^12.20.0 || ^14.13.0 || >=15.0.0",
+    "node": "^10 || ^12.20.0 || ^14.13.0 || >=15.0.0",
     "npm": ">=7.0.0"
   },
   "files": [

package.json

@@ -0,0 +1,121 @@
+// This script is written using CommonJS since it should run
+// on Node.js versions which don't support for ESM
+
+const fs = require('fs');
+const path = require('path');
+const { rollup } = require('rollup');
+
+const { name: packageName } = require('../package.json');
+
+const external = [
+    'module',
+    'fs',
+    'path',
+    'url',
+    'assert',
+    'json-to-ast',
+    'css-tree',
+    'csso',
+    /^source-map/
+];
+
+function removeCreateRequire(id) {
+    return fs.readFileSync(id, 'utf8')
+        .replace(/import .+ from 'module';/, '')
+        .replace(/const require = .+;/, '');
+}
+
+function replaceContent(map) {
+    return {
+        name: 'file-content-replacement',
+        load(id) {
+            const key = path.relative('', id);
+
+            if (map.hasOwnProperty(key)) {
+                return map[key](id);
+            }
+        }
+    };
+}
+
+function patchTests() {
+    // If Node.js doesn't support for `exports` it doesn't support for import/require
+    // by package name inside the package itself, so this resolving will fail.
+    // We can't use just `require(packageName)` here since CJS modules are not generated yet,
+    // and Node.js will fail on resolving it either disregarding of `exports` support.
+    // In this case we need to replace import/require using a package name with
+    // a relative path to a module.
+    try {
+        require(`${packageName}/package.json`);
+        return;
+    } catch (e) {}
+
+    const pathToIndex = path.resolve(__dirname, '../cjs/index.cjs');
+
+    // Make replacement for relative path only for tests since we need to check everything
+    // is work on old Node.js version. The rest of code should be unchanged since will run
+    // on any Node.js version.
+    console.log(`Fixing CommonJS tests by replacing "${packageName}" for a relative paths`);
+
+    return {
+        name: 'cjs-tests-fix',
+        resolveId(source) {
+            if (/^..\/cjs/.test(source)) {
+                return { id: source, external: true };
+            }
+            return null;
+        },
+        transform(code, id) {
+            return code.replace(
+                new RegExp(`from (['"])${packageName}\\1;`, 'g'),
+                `from '${path.relative(path.dirname(id), pathToIndex)}'`
+            );
+        }
+    };
+}
+
+function readDir(dir) {
+    return fs.readdirSync(dir)
+        .filter(fn => fn.endsWith('.js'))
+        .map(fn => `${dir}/${fn}`);
+}
+
+async function build(outputDir, patch, ...entryPoints) {
+    const startTime = Date.now();
+
+    console.log();
+    console.log(`Convert ESM to CommonJS (output: ${outputDir})`);
+
+    const res = await rollup({
+        external,
+        input: entryPoints,
+        plugins: [
+            replaceContent({
+                'lib/version.js': removeCreateRequire
+            }),
+            patch && patchTests()
+        ]
+    });
+    await res.write({
+        dir: outputDir,
+        entryFileNames: '[name].cjs',
+        format: 'cjs',
+        exports: 'auto',
+        preserveModules: true,
+        interop: false,
+        esModule: false,
+        generatedCode: {
+            constBindings: true
+        }
+    });
+    await res.close();
+
+    console.log(`Done in ${Date.now() - startTime}ms`);
+}
+
+async function buildAll() {
+    await build('./cjs', false, 'lib/index.js');
+    await build('./cjs-test', true, ...readDir('test'));
+}
+
+buildAll();