fix: more CJS compatibility (#4)

- Change package.json type to CJS for more compatibility.
- Replace `unbuild` with `tsup` for more default export ergonomics.
- `unbuild` was fine as long as we added a patch script to export
`default` in CJS. However that's a bit unexpected in CJS; you would
expect to just do `const rxjsX = require('eslint-plugin-rxjs-x');`
without `.default` tacked onto the end. `tsup` lets us build different
output for CJS that's more ergonomic for CJS users.
- Had to remove the `assert { type: 'json' }` since `esbuild` doesn't
support it combined with named exports.
- Only support decamelize v5.
- v6 is ESM-only, which would be fine if users use this plugin in an ESM
eslint config. However if they're using a CJS eslint config and they
install this package, their package manager will likely pull in
decamelize v6 and cause a error trying to import it. Downgrading helps
avoid that confusion; advanced users who need v6 can use something like
yarn resolutions to override.

Author: JasonWeinzierl

build.config.ts

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-rxjs-x",
-  "type": "module",
+  "type": "commonjs",
   "version": "0.1.0",
   "packageManager": "[email protected]+sha512.341db9396b6e289fecc30cd7ab3af65060e05ebff4b3b47547b278b9e67b08f485ecd8c79006b405446262142c7a38154445ef7f17c1d5d1de7d90bf9ce7054d",
   "description": "ESLint v9+ plugin for RxJS",
@@ -28,20 +28,20 @@
         "default": "./dist/index.mjs"
       },
       "require": {
-        "types": "./dist/index.d.cts",
-        "default": "./dist/index.cjs"
+        "types": "./dist/index.d.ts",
+        "default": "./dist/index.js"
       }
     }
   },
-  "main": "./dist/index.mjs",
+  "main": "./dist/index.js",
   "module": "./dist/index.mjs",
   "types": "./dist/index.d.ts",
   "files": [
     "dist",
     "docs"
   ],
   "scripts": {
-    "build": "unbuild && tsx scripts/postbuild.ts",
+    "build": "tsup",
     "lint": "eslint",
     "release": "bumpp && yarn run build",
     "test": "vitest",
@@ -52,7 +52,7 @@
     "@typescript-eslint/scope-manager": "^8.12.2",
     "@typescript-eslint/utils": "^8.12.2",
     "common-tags": "^1.8.0",
-    "decamelize": "^5.0.0 || ^6.0.0",
+    "decamelize": "^5.0.1",
     "ts-api-utils": "^1.3.0",
     "tslib": "^2.1.0"
   },
@@ -77,10 +77,10 @@
     "eslint-plugin-import-x": "^4.4.0",
     "eslint-plugin-n": "^17.12.0",
     "rxjs": "^7.0.0",
+    "tsup": "^8.3.5",
     "tsx": "^4.19.2",
     "typescript": "~5.6.3",
     "typescript-eslint": "^8.12.2",
-    "unbuild": "^2.0.0",
     "vitest": "^2.1.4"
   },
   "engines": {

package.json

@@ -1,5 +1,5 @@
 import { TSESLint } from '@typescript-eslint/utils';
-import { name, version } from '../package.json' assert { type: 'json' };
+import { name, version } from '../package.json';
 import { createRecommendedConfig } from './configs/recommended';
 
 import { banObservablesRule } from './rules/ban-observables';

scripts/postbuild.ts

@@ -0,0 +1,12 @@
+import { defineConfig } from 'tsup';
+
+export default defineConfig({
+  entry: [
+    'src/index.ts',
+  ],
+  clean: true,
+  dts: true,
+  cjsInterop: true,
+  splitting: true,
+  format: ['cjs', 'esm'],
+});