Minor changes

Author: Septh

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rollup-plugin-node-externals",
-  "version": "3.0.0",
+  "version": "2.2.0",
   "description": "Automatically declare NodeJS built-in modules and npm dependencies as 'external' in Rollup config",
   "author": "Stephan Schreiber <[email protected]>",
   "contributors": [
@@ -30,18 +30,16 @@
     "url": "https://github.com/Septh/rollup-plugin-node-externals"
   },
   "homepage": "https://github.com/Septh/rollup-plugin-node-externals",
-  "type": "module",
+  "type": "commonjs",
   "main": "./dist/cjs/index.js",
   "module": "./dist/esm/index.js",
   "exports": {
-    "import": "./dist/cjs/index.js",
-    "require": "./dist/esm/index.js"
+    "require": "./dist/cjs/index.js",
+    "import": "./dist/esm/index.js"
   },
   "types": "./dist/index.d.ts",
   "files": [
-    "dist/cjs/index.js",
-    "dist/esm/index.js",
-    "dist/index.d.ts",
+    "dist",
     "README.md"
   ],
   "ava": {
@@ -55,11 +53,10 @@
   "scripts": {
     "build": "rollup -c",
     "watch": "rollup -cw",
-    "types": "tsc src/index.ts --declaration --emitDeclarationOnly --outFile dist/index.d.ts --stripInternal",
     "test": "ava",
     "watch-tests": "ava -w",
     "clean": "rimraf dist",
-    "prepublishOnly": "npm run clean && npm run build && npm run types"
+    "prepublishOnly": "npm run clean && npm run build"
   },
   "dependencies": {
     "find-up": "^6.2.0"

rollup.config.mjs

@@ -8,22 +8,37 @@ import ts from 'rollup-plugin-ts'
 // @ts-ignore
 const pkg = createRequire(import.meta.url)('./package.json')
 
-/** @type { import('rollup').PluginImpl } */
-const nodeColon = () => {
+/**
+ * A mini-plugin that resolves `node:` imports to their unprefixed equivalent.
+ * @type { import('rollup').PluginImpl }
+ */
+function nodeColon() {
     return {
         name: 'node-colon',
 
         resolveId(id) {
             for (const scheme of [ 'node:', 'nodejs:' ]) {
                 if (id.startsWith(scheme)) {
-                    id = id.slice(scheme.length)
-                    return {
-                        id,
-                        external: external.includes(id)
-                    }
+                    return id.slice(scheme.length)
                 }
             }
-            return null
+        }
+    }
+}
+
+/**
+ * A mini-plugin that generates a package.json file next to the bundle.
+ * @type { import('rollup').PluginImpl }
+ */
+function emitModulePackageFile() {
+    return {
+        name: 'emit-module-package-file',
+        generateBundle() {
+            this.emitFile({
+                type: 'asset',
+                fileName: 'package.json',
+                source: JSON.stringify({ type: 'module' }, undefined, 2)
+            })
         }
     }
 }
@@ -35,13 +50,16 @@ const config = {
         {
             format: 'commonjs',
             file: pkg.main,
-            exports: 'default',
+            exports: 'named',
             sourcemap: false
         },
         {
             format: 'module',
             file: pkg.module,
-            sourcemap: false
+            sourcemap: false,
+            plugins: [
+                emitModulePackageFile()
+            ]
         },
     ],
     plugins: [

src/index.ts

@@ -70,7 +70,7 @@ export interface ExternalsOptions {
     )
 
     // Build a function to filter out unwanted dependencies
-    const filterFn: (dep: string) => boolean = dep => !exclude.some(rx => rx.test(dep))
+    const filterFn = (dep: string) => !exclude.some(rx => rx.test(dep))
 
     // Filter NodeJS builtins
     const builtins = (opts.builtins ? builtinModules : []).filter(filterFn)
@@ -118,10 +118,12 @@ export interface ExternalsOptions {
             }
         },
 
-        resolveId(source, importer) {
+        resolveId(id, importer) {
             // Return `false` if importee should be treated as an external module,
             // otherwise return `null` to let Rollup and other plugins handle it.
-            return importer && !/\0/.test(source) && externals.some(deps => deps.test(source)) ? false : null
+            return importer && !/\0/.test(id) && externals.some(deps => deps.test(id)) && filterFn(id)
+                ? false
+                : null
         }
     }
 }