fix(alias): prepare for Rollup 3, handle latest node-resolve (rollup#1269)

BREAKING CHANGES: Requires Node 14

Author: lukastaegert

packages/alias/README.md

@@ -31,7 +31,7 @@ This plugin will work for any file type that Rollup natively supports, or those
 
 ## Requirements
 
-This plugin requires an [LTS](https://github.com/nodejs/Release) Node version (v8.0.0+) and Rollup v1.20.0+.
+This plugin requires an [LTS](https://github.com/nodejs/Release) Node version (v14.0.0+) and Rollup v1.20.0+.
 
 ## Install
 

packages/alias/package.json

@@ -13,10 +13,15 @@
   "author": "Johannes Stein",
   "homepage": "https://github.com/rollup/plugins/tree/master/packages/alias#readme",
   "bugs": "https://github.com/rollup/plugins/issues",
-  "main": "dist/index.js",
-  "module": "dist/index.es.js",
+  "main": "./dist/cjs/index.js",
+  "module": "./dist/es/index.js",
+  "exports": {
+    "import": "./dist/es/index.js",
+    "types": "./types/index.d.ts",
+    "default": "./dist/cjs/index.js"
+  },
   "engines": {
-    "node": ">=8.0.0"
+    "node": ">=14.0.0"
   },
   "scripts": {
     "build": "rollup -c",
@@ -34,6 +39,7 @@
   },
   "files": [
     "dist",
+    "!dist/**/*.map",
     "types",
     "README.md",
     "LICENSE"
@@ -45,19 +51,24 @@
     "alias"
   ],
   "peerDependencies": {
-    "rollup": "^1.20.0||^2.0.0"
+    "rollup": "^1.20.0||^2.0.0||^3.0.0"
+  },
+  "peerDependenciesMeta": {
+    "rollup": {
+      "optional": true
+    }
   },
   "dependencies": {
-    "slash": "^3.0.0"
+    "slash": "^4.0.0"
   },
   "devDependencies": {
-    "@rollup/plugin-node-resolve": "^8.4.0",
-    "@rollup/plugin-typescript": "^5.0.2",
-    "del-cli": "^3.0.1",
-    "rollup": "2.67.3",
-    "typescript": "^4.1.2"
+    "@rollup/plugin-node-resolve": "^14.1.0",
+    "@rollup/plugin-typescript": "^8.5.0",
+    "del-cli": "^5.0.0",
+    "rollup": "^3.0.0-7",
+    "typescript": "^4.8.3"
   },
-  "types": "types/index.d.ts",
+  "types": "./types/index.d.ts",
   "ava": {
     "files": [
       "!**/fixtures/**",

packages/alias/rollup.config.js

@@ -0,0 +1,7 @@
+import { readFileSync } from 'fs';
+
+import { createConfig } from '../../shared/rollup.config.mjs';
+
+export default createConfig({
+  pkg: JSON.parse(readFileSync(new URL('./package.json', import.meta.url), 'utf8'))
+});

packages/alias/rollup.config.mjs

@@ -1,6 +1,6 @@
 import { Plugin } from 'rollup';
 
-import { ResolvedAlias, ResolverFunction, ResolverObject, RollupAliasOptions } from '../types';
+import type { ResolvedAlias, ResolverFunction, ResolverObject, RollupAliasOptions } from '../types';
 
 function matches(pattern: string | RegExp, importee: string) {
   if (pattern instanceof RegExp) {
@@ -38,16 +38,24 @@ function getEntries({ entries, customResolver }: RollupAliasOptions): readonly R
   });
 }
 
+function getHookFunction<T extends Function>(hook: T | { handler?: T }): T | null {
+  if (typeof hook === 'function') {
+    return hook;
+  }
+  if (hook && 'handler' in hook && typeof hook.handler === 'function') {
+    return hook.handler;
+  }
+  return null;
+}
+
 function resolveCustomResolver(
   customResolver: ResolverFunction | ResolverObject | null | undefined
 ): ResolverFunction | null {
+  if (typeof customResolver === 'function') {
+    return customResolver;
+  }
   if (customResolver) {
-    if (typeof customResolver === 'function') {
-      return customResolver;
-    }
-    if (typeof customResolver.resolveId === 'function') {
-      return customResolver.resolveId;
-    }
+    return getHookFunction(customResolver.resolveId);
   }
   return null;
 }
@@ -68,10 +76,7 @@ export default function alias(options: RollupAliasOptions = {}): Plugin {
       await Promise.all(
         [...(Array.isArray(options.entries) ? options.entries : []), options].map(
           ({ customResolver }) =>
-            customResolver &&
-            typeof customResolver === 'object' &&
-            typeof customResolver.buildStart === 'function' &&
-            customResolver.buildStart.call(this, inputOptions)
+            customResolver && getHookFunction(customResolver.buildStart)?.call(this, inputOptions)
         )
       );
     },

packages/alias/src/index.ts

@@ -1,6 +1,8 @@
 import { Plugin, PluginHooks } from 'rollup';
 
-export type ResolverFunction = PluginHooks['resolveId'];
+type MapToFunction<T> = T extends Function ? T : never;
+
+export type ResolverFunction = MapToFunction<PluginHooks['resolveId']>;
 
 export interface ResolverObject {
   buildStart?: PluginHooks['buildStart'];