feat(auto-install)!: ESM only; tsc build; migrate tests to Vitest per playbook\n\n- Use type: module and ESM-only exports at . with types/import/default\n- engines.node >=20.19.0; peerDependencies.rollup >=4.0.0\n- tsc build via shared .config/tsconfig.plugin.json (symlink); remove rollup config and hand-authored types/\n- Update source to use node: specifiers and inline public types\n- Convert AVA tests to Vitest; close bundles; restore CWD in teardown\n- Keep changes scoped to packages/auto-install only

Author: CharlieHelps

packages/auto-install/package.json

@@ -13,34 +13,33 @@
   "author": "Rich Harris",
   "homepage": "https://github.com/rollup/plugins/tree/master/packages/auto-install/#readme",
   "bugs": "https://github.com/rollup/plugins/issues",
-  "main": "./dist/cjs/index.js",
-  "module": "./dist/es/index.js",
+  "type": "module",
   "exports": {
-    "types": "./types/index.d.ts",
-    "import": "./dist/es/index.js",
-    "default": "./dist/cjs/index.js"
+    ".": {
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.js",
+      "default": "./dist/index.js"
+    }
   },
   "engines": {
-    "node": ">=14.0.0"
+    "node": ">=20.19.0"
   },
   "scripts": {
-    "build": "rollup -c",
+    "build": "tsc --project tsconfig.json",
     "ci:coverage": "nyc pnpm test && nyc report --reporter=text-lcov > coverage.lcov",
     "ci:lint": "pnpm build && pnpm lint",
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
-    "ci:test": "pnpm test -- --verbose",
+    "ci:test": "pnpm test -- --reporter=verbose",
     "prebuild": "del-cli dist",
     "prepare": "if [ ! -d 'dist' ]; then pnpm build; fi",
     "prerelease": "pnpm build",
     "pretest": "pnpm build",
     "release": "pnpm --workspace-root package:release $(pwd)",
-    "test": "ava",
+    "test": "vitest --config ../../.config/vitest.config.mts run",
     "test:ts": "tsc --noEmit"
   },
   "files": [
     "dist",
-    "!dist/**/*.map",
-    "types",
     "README.md",
     "LICENSE"
   ],
@@ -53,7 +52,7 @@
     "modules"
   ],
   "peerDependencies": {
-    "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0"
+    "rollup": ">=4.0.0"
   },
   "peerDependenciesMeta": {
     "rollup": {
@@ -62,13 +61,13 @@
   },
   "devDependencies": {
     "@rollup/plugin-node-resolve": "^15.0.0",
-    "@rollup/plugin-typescript": "^9.0.1",
     "del": "^6.1.1",
+    "del-cli": "^5.0.0",
     "node-noop": "^1.0.0",
-    "rollup": "^4.0.0-24",
-    "typescript": "^4.8.3"
+    "rollup": "^4.0.0",
+    "typescript": "catalog:"
   },
-  "types": "./types/index.d.ts",
+  "types": "./dist/index.d.ts",
   "ava": {
     "workerThreads": false,
     "files": [

packages/auto-install/rollup.config.mjs

@@ -1,30 +1,47 @@
-import * as fs from 'fs';
-import * as path from 'path';
-import mod from 'module';
-import { exec } from 'child_process';
-import { promisify } from 'util';
+import fs from 'node:fs';
+import path from 'node:path';
+import { builtinModules } from 'node:module';
+import { exec } from 'node:child_process';
+import { promisify } from 'node:util';
 
 import type { Plugin } from 'rollup';
 
-import type { RollupAutoInstallOptions } from '../types';
+export interface RollupAutoInstallOptions {
+  /**
+   * Specifies the location on disk of the target `package.json` file.
+   * If the file doesn't exist, it will be created by the plugin,
+   * as package managers need to populate the `dependencies` property.
+   * @default '{cwd}/package.json'
+   */
+  pkgFile?: string;
+
+  /**
+   * Specifies the package manager to use.
+   * If not specified, the plugin will default to `yarn` if `yarn.lock` exists,
+   * to `pnpm` if `pnpm-lock.yaml` exists, or `npm` otherwise.
+   */
+  manager?: 'npm' | 'yarn' | 'pnpm';
+}
 
 const execAsync = promisify(exec);
 
 export default function autoInstall(opts: RollupAutoInstallOptions = {}): Plugin {
-  const defaults = {
-    // intentionally undocumented options. used for tests
-    commands: {
-      npm: 'npm install',
-      pnpm: 'pnpm install',
-      yarn: 'yarn add'
-    },
-    manager: fs.existsSync('yarn.lock') ? 'yarn' : fs.existsSync('pnpm-lock.yaml') ? 'pnpm' : 'npm',
-    pkgFile: path.resolve(opts.pkgFile || 'package.json')
+  const commands: Record<'npm' | 'yarn' | 'pnpm', string> = {
+    npm: 'npm install',
+    pnpm: 'pnpm install',
+    yarn: 'yarn add'
   };
 
-  const options = Object.assign({}, defaults, opts);
-  const { manager, pkgFile } = options;
-  const validManagers = ['npm', 'yarn', 'pnpm'];
+  const manager = opts.manager
+    ? opts.manager
+    : fs.existsSync('yarn.lock')
+    ? 'yarn'
+    : fs.existsSync('pnpm-lock.yaml')
+    ? 'pnpm'
+    : 'npm';
+
+  const pkgFile = path.resolve(opts.pkgFile || 'package.json');
+  const validManagers: Array<RollupAutoInstallOptions['manager']> = ['npm', 'yarn', 'pnpm'];
 
   if (!validManagers.includes(manager)) {
     throw new RangeError(
@@ -41,8 +58,8 @@ export default function autoInstall(opts: RollupAutoInstallOptions = {}): Plugin
     pkg = {};
   }
 
-  const installed = new Set(Object.keys(pkg.dependencies || {}).concat(mod.builtinModules));
-  const cmd = options.commands[manager];
+  const installed = new Set(Object.keys(pkg.dependencies || {}).concat(builtinModules));
+  const cmd = commands[manager];
 
   return {
     name: 'auto-install',

packages/auto-install/src/index.ts

@@ -0,0 +1,38 @@
+import fs from 'node:fs';
+import path from 'node:path';
+
+import del from 'del';
+import { it, afterAll } from 'vitest';
+import { rollup } from 'rollup';
+import nodeResolve from '@rollup/plugin-node-resolve';
+
+import autoInstall from '~package';
+
+const DIR = import.meta.dirname;
+const cwd = path.join(DIR, 'fixtures/npm-bare');
+const file = path.join(cwd, 'output/bundle.js');
+const input = path.join(cwd, '../input.js');
+const pkgFile = path.join(cwd, 'package.json');
+
+const PREV_CWD = process.cwd();
+
+it('npm, bare', async () => {
+  process.chdir(cwd);
+  const bundle = await rollup({
+    input,
+    // @ts-expect-error - rollup() ignores output here but tests kept it historically
+    output: { file, format: 'es' },
+    plugins: [autoInstall({ pkgFile, manager: 'npm' }), nodeResolve()]
+  });
+  await bundle.close();
+
+  const json = JSON.parse(fs.readFileSync(pkgFile, 'utf-8'));
+  if (!json.dependencies || !json.dependencies['node-noop']) {
+    throw new Error('Expected node-noop to be added to dependencies');
+  }
+});
+
+afterAll(async () => {
+  await del(['node_modules', 'package.json', 'package-lock.json']);
+  process.chdir(PREV_CWD);
+});

packages/auto-install/test/npm-bare.js

@@ -0,0 +1,41 @@
+import fs from 'node:fs';
+import path from 'node:path';
+
+import del from 'del';
+import { it, expect, afterAll } from 'vitest';
+import { rollup } from 'rollup';
+import nodeResolve from '@rollup/plugin-node-resolve';
+
+import autoInstall from '~package';
+
+const DIR = import.meta.dirname;
+const cwd = path.join(DIR, 'fixtures/npm');
+const file = path.join(cwd, 'output/bundle.js');
+const input = path.join(cwd, '../input.js');
+const pkgFile = path.join(cwd, 'package.json');
+
+const PREV_CWD = process.cwd();
+
+it('invalid manager', () => {
+  expect(() => autoInstall({ pkgFile, manager: 'foo' as any })).toThrow(RangeError);
+});
+
+it('npm', async () => {
+  process.chdir(cwd);
+  const bundle = await rollup({
+    input,
+    // @ts-expect-error - rollup() ignores output here but tests kept it historically
+    output: { file, format: 'es' },
+    plugins: [autoInstall({ pkgFile, manager: 'npm' }), nodeResolve()]
+  });
+  await bundle.close();
+
+  const json = JSON.parse(fs.readFileSync(pkgFile, 'utf-8'));
+  expect(json.dependencies && json.dependencies['node-noop']).toBeDefined();
+});
+
+afterAll(async () => {
+  await del(['node_modules', 'package-lock.json']);
+  fs.writeFileSync(pkgFile, '{}');
+  process.chdir(PREV_CWD);
+});

packages/auto-install/test/npm-bare.test.ts

@@ -0,0 +1,37 @@
+import fs from 'node:fs';
+import path from 'node:path';
+
+import del from 'del';
+import { it, afterAll } from 'vitest';
+import { rollup } from 'rollup';
+import nodeResolve from '@rollup/plugin-node-resolve';
+
+import autoInstall from '~package';
+
+const DIR = import.meta.dirname;
+const cwd = path.join(DIR, 'fixtures/pnpm-bare');
+const file = path.join(cwd, 'output/bundle.js');
+const input = path.join(cwd, '../input.js');
+
+const PREV_CWD = process.cwd();
+
+it('pnpm, bare', async () => {
+  process.chdir(cwd);
+  const bundle = await rollup({
+    input,
+    // @ts-expect-error - rollup() ignores output here but tests kept it historically
+    output: { file, format: 'es' },
+    plugins: [autoInstall({ manager: 'pnpm' }), nodeResolve()]
+  });
+  await bundle.close();
+
+  const json = JSON.parse(fs.readFileSync(path.join(cwd, 'package.json'), 'utf-8'));
+  if (!json.dependencies || !json.dependencies['node-noop']) {
+    throw new Error('Expected node-noop to be added to dependencies');
+  }
+});
+
+afterAll(async () => {
+  await del(['node_modules', 'package.json', 'pnpm-lock.yaml']);
+  process.chdir(PREV_CWD);
+});