fix: make PostCSS CJS entries callable

.changeset/bright-cjs-plugins.md

@@ -0,0 +1,8 @@
+---
+'postcss-rem-to-responsive-pixel': patch
+'postcss-rem-to-viewport': patch
+'postcss-rule-unit-converter': patch
+'postcss-units-to-px': patch
+---
+
+Fix CommonJS plugin entries so `require()` returns a callable PostCSS plugin while preserving the default and named exports.

.gitattributes

@@ -59,14 +59,14 @@ Dockerfile        text
 
 # Documentation
 *.ipynb           text eol=lf
-*.markdown        text diff=markdown
-*.md              text diff=markdown
-*.mdwn            text diff=markdown
-*.mdown           text diff=markdown
-*.mkd             text diff=markdown
-*.mkdn            text diff=markdown
+*.markdown        text eol=lf diff=markdown
+*.md              text eol=lf diff=markdown
+*.mdwn            text eol=lf diff=markdown
+*.mdown           text eol=lf diff=markdown
+*.mkd             text eol=lf diff=markdown
+*.mkdn            text eol=lf diff=markdown
 *.mdtxt           text
-*.mdtext          text
+*.mdtext          text eol=lf
 *.txt             text
 AUTHORS           text
 CHANGELOG         text
@@ -215,4 +215,4 @@ Procfile          text
 *.*ignore         text
 
 # Prevents massive diffs from built files
-dist/*            binary
+dist/*            binary

.github/workflows/ci.yml

@@ -18,7 +18,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
-        node-version: [20, 22, 24]
+        node-version: [22, 24]
     runs-on: ${{ matrix.os }}
     # Remote Caching enabled - configure TURBO_TOKEN and TURBO_TEAM in repository settings
     env:

README.md

@@ -15,7 +15,7 @@ Each package has its own README with usage and configuration details.
 
 ## Requirements
 
-- Node.js >= 20
+- Node.js >= 22.12.0
 - pnpm (workspace managed)
 
 ## Development

package.json

@@ -13,7 +13,7 @@
     "url": "https://github.com/sonofmagic/postcss-plugins/issues"
   },
   "engines": {
-    "node": ">=20.0.0"
+    "node": ">=22.12.0"
   },
   "scripts": {
     "build": "turbo run build",
@@ -23,6 +23,7 @@
     "bench": "node scripts/bench/generate-benchmarks.mjs",
     "bench:baseline": "node scripts/bench/generate-benchmarks.mjs",
     "bench:report": "node scripts/bench/generate-benchmarks.mjs",
+    "check:cjs": "tsx scripts/check-cjs-plugins.mjs",
     "lint": "turbo run lint",
     "release": "changeset",
     "publish-packages": "turbo run build lint test && changeset version && changeset publish",

packages/postcss-rem-to-responsive-pixel/package.json

@@ -41,14 +41,14 @@
     ".": {
       "types": {
         "import": "./dist/index.d.mts",
-        "require": "./dist/index.d.cts"
+        "require": "./dist/index.require.d.cts"
       },
       "import": "./dist/index.mjs",
-      "require": "./dist/index.cjs"
+      "require": "./dist/index.require.cjs"
     },
     "./*": "./*"
   },
-  "main": "./dist/index.cjs",
+  "main": "./dist/index.require.cjs",
   "module": "./dist/index.mjs",
   "types": "./dist/index.d.mts",
   "files": [

packages/postcss-rem-to-responsive-pixel/src/index-require.cts

@@ -0,0 +1,8 @@
+type RemToResponsivePixelModule = typeof import('./index')
+
+// eslint-disable-next-line import/newline-after-import, ts/no-require-imports
+const remToResponsivePixelModule = require('./index') as RemToResponsivePixelModule
+const requireEntry = Object.assign(remToResponsivePixelModule.default, remToResponsivePixelModule)
+
+// eslint-disable-next-line no-restricted-syntax
+export = requireEntry

packages/postcss-rem-to-responsive-pixel/tsdown.config.mts

@@ -1,10 +1,22 @@
 import { defineConfig } from 'tsdown'
 
-export default defineConfig({
-  entry: ['./src/index.ts'],
-  format: ['esm', 'cjs'],
-  dts: true,
-  clean: true,
-  target: 'node16',
-  failOnWarn: false,
-})
+export default defineConfig([
+  {
+    entry: ['./src/index.ts'],
+    format: ['esm'],
+    dts: true,
+    clean: true,
+    target: 'node16',
+    failOnWarn: false,
+  },
+  {
+    entry: {
+      'index.require': './src/index-require.cts',
+    },
+    format: ['cjs'],
+    dts: true,
+    clean: false,
+    target: 'node16',
+    failOnWarn: false,
+  },
+])

packages/postcss-rem-to-viewport/package.json

@@ -51,13 +51,13 @@
       ".": {
         "types": {
           "import": "./dist/index.d.mts",
-          "require": "./dist/index.d.cts"
+          "require": "./dist/index.require.d.cts"
         },
         "import": "./dist/index.mjs",
-        "require": "./dist/index.cjs"
+        "require": "./dist/index.require.cjs"
       }
     },
-    "main": "./dist/index.cjs",
+    "main": "./dist/index.require.cjs",
     "module": "./dist/index.mjs",
     "types": "./dist/index.d.mts"
   }

packages/postcss-rem-to-viewport/src/index-require.cts

@@ -0,0 +1,8 @@
+type RemToViewportModule = typeof import('./index')
+
+// eslint-disable-next-line import/newline-after-import, ts/no-require-imports
+const remToViewportModule = require('./index') as RemToViewportModule
+const requireEntry = Object.assign(remToViewportModule.default, remToViewportModule)
+
+// eslint-disable-next-line no-restricted-syntax
+export = requireEntry

packages/postcss-rem-to-viewport/tsdown.config.mts

@@ -1,10 +1,22 @@
 import { defineConfig } from 'tsdown'
 
-export default defineConfig({
-  entry: ['./src/index.ts'],
-  format: ['esm', 'cjs'],
-  dts: true,
-  clean: true,
-  target: 'node16',
-  failOnWarn: false,
-})
+export default defineConfig([
+  {
+    entry: ['./src/index.ts'],
+    format: ['esm'],
+    dts: true,
+    clean: true,
+    target: 'node16',
+    failOnWarn: false,
+  },
+  {
+    entry: {
+      'index.require': './src/index-require.cts',
+    },
+    format: ['cjs'],
+    dts: true,
+    clean: false,
+    target: 'node16',
+    failOnWarn: false,
+  },
+])

packages/postcss-rule-unit-converter/package.json

@@ -30,10 +30,10 @@
     ".": {
       "types": {
         "import": "./dist/index.d.mts",
-        "require": "./dist/index.d.cts"
+        "require": "./dist/index.require.d.cts"
       },
       "import": "./dist/index.mjs",
-      "require": "./dist/index.cjs"
+      "require": "./dist/index.require.cjs"
     },
     "./presets": {
       "types": {
@@ -44,7 +44,7 @@
       "require": "./dist/presets.cjs"
     }
   },
-  "main": "./dist/index.cjs",
+  "main": "./dist/index.require.cjs",
   "module": "./dist/index.mjs",
   "types": "./dist/index.d.mts",
   "files": [

packages/postcss-rule-unit-converter/src/index-require.cts

@@ -0,0 +1,8 @@
+type UnitConverterModule = typeof import('./index')
+
+// eslint-disable-next-line import/newline-after-import, ts/no-require-imports
+const unitConverterModule = require('./index') as UnitConverterModule
+const requireEntry = Object.assign(unitConverterModule.default, unitConverterModule)
+
+// eslint-disable-next-line no-restricted-syntax
+export = requireEntry

packages/postcss-rule-unit-converter/tsdown.config.ts

@@ -1,14 +1,35 @@
 import { defineConfig } from 'tsdown'
 
-export default defineConfig({
-  entry: {
-    index: './src/index.ts',
-    presets: './src/presets.ts',
+export default defineConfig([
+  {
+    entry: {
+      index: './src/index.ts',
+      presets: './src/presets.ts',
+    },
+    format: ['esm'],
+    dts: true,
+    clean: true,
+    target: 'node18',
+    failOnWarn: false,
   },
-  format: ['esm', 'cjs'],
-  cjsDefault: false,
-  dts: true,
-  clean: true,
-  target: 'node18',
-  failOnWarn: false,
-})
+  {
+    entry: {
+      'index.require': './src/index-require.cts',
+    },
+    format: ['cjs'],
+    dts: true,
+    clean: false,
+    target: 'node18',
+    failOnWarn: false,
+  },
+  {
+    entry: {
+      presets: './src/presets.ts',
+    },
+    format: ['cjs'],
+    dts: true,
+    clean: false,
+    target: 'node18',
+    failOnWarn: false,
+  },
+])

packages/postcss-units-to-px/package.json

@@ -27,10 +27,10 @@
     ".": {
       "types": {
         "import": "./dist/index.d.mts",
-        "require": "./dist/index.d.cts"
+        "require": "./dist/index.require.d.cts"
       },
       "import": "./dist/index.mjs",
-      "require": "./dist/index.cjs"
+      "require": "./dist/index.require.cjs"
     },
     "./defaults": {
       "types": {
@@ -41,7 +41,7 @@
       "require": "./dist/defaults.cjs"
     }
   },
-  "main": "./dist/index.cjs",
+  "main": "./dist/index.require.cjs",
   "module": "./dist/index.mjs",
   "types": "./dist/index.d.mts",
   "files": [

packages/postcss-units-to-px/src/index-require.cts

@@ -0,0 +1,8 @@
+type UnitsToPxModule = typeof import('./index')
+
+// eslint-disable-next-line import/newline-after-import, ts/no-require-imports
+const unitsToPxModule = require('./index') as UnitsToPxModule
+const requireEntry = Object.assign(unitsToPxModule.default, unitsToPxModule)
+
+// eslint-disable-next-line no-restricted-syntax
+export = requireEntry

packages/postcss-units-to-px/tsdown.config.ts

@@ -1,13 +1,35 @@
 import { defineConfig } from 'tsdown'
 
-export default defineConfig({
-  entry: {
-    index: './src/index.ts',
-    defaults: './src/defaults-entry.ts',
+export default defineConfig([
+  {
+    entry: {
+      index: './src/index.ts',
+      defaults: './src/defaults-entry.ts',
+    },
+    format: ['esm'],
+    dts: true,
+    clean: true,
+    target: 'node18',
+    failOnWarn: false,
   },
-  format: ['esm', 'cjs'],
-  dts: true,
-  clean: true,
-  target: 'node18',
-  failOnWarn: false,
-})
+  {
+    entry: {
+      'index.require': './src/index-require.cts',
+    },
+    format: ['cjs'],
+    dts: true,
+    clean: false,
+    target: 'node18',
+    failOnWarn: false,
+  },
+  {
+    entry: {
+      defaults: './src/defaults-entry.ts',
+    },
+    format: ['cjs'],
+    dts: true,
+    clean: false,
+    target: 'node18',
+    failOnWarn: false,
+  },
+])