fix(auto-install): restore pnpm lockfile, make TS config TS4/5-compatible; type hooks; normalize built-ins; fix tests CWD

- Regenerate workspace pnpm-lock.yaml (v9) to fix CI install step
- tsconfig: use moduleResolution "nodenext" for TS 4.8 compatibility; stripInternal in plugin config
- src: mark test-only "commands" as @internal; normalize core module names; add resolveId param types
- tests: restore process.cwd() in teardown to avoid cross-test coupling
- package: constrain peer rollup to ^4

Author: CharlieHelps

.config/tsconfig.base.json

@@ -4,7 +4,9 @@
     "esModuleInterop": true,
     "lib": ["ESNext"],
     "module": "esnext",
-    "moduleResolution": "bundler",
+    // Use NodeNext for broad compatibility with both TS 4.8 and 5.x
+    // (TS 4.8 does not support "bundler" resolution yet.)
+    "moduleResolution": "nodenext",
     "noEmit": true,
     "noUnusedLocals": true,
     "noUnusedParameters": true,

.config/tsconfig.plugin.json

@@ -7,7 +7,9 @@
     "outDir": "dist",
     "rootDir": "src",
     "declaration": true,
-    "declarationMap": true
+    "declarationMap": true,
+    // Ensure TSDoc `@internal` members aren't emitted in published types
+    "stripInternal": true
   },
   "include": ["src/**/*"]
 }

packages/auto-install/package.json

@@ -48,7 +48,7 @@
     "modules"
   ],
   "peerDependencies": {
-    "rollup": ">=4.0.0"
+    "rollup": "^4"
   },
   "peerDependenciesMeta": {
     "rollup": {

packages/auto-install/src/index.ts

@@ -28,7 +28,10 @@ export interface RollupAutoInstallOptions {
    */
   manager?: 'npm' | 'yarn' | 'pnpm';
 
-  /** Test-only override of package manager commands. */
+  /**
+   * Test-only override of package manager commands.
+   * @internal
+   */
   commands?: Partial<Record<'npm' | 'yarn' | 'pnpm', string>>;
 }
 
@@ -65,13 +68,20 @@ export default function autoInstall(opts: RollupAutoInstallOptions = {}): Plugin
     pkg = {};
   }
 
-  const installed = new Set(Object.keys(pkg.dependencies || {}).concat(builtinModules));
+  // Normalize core module names to include both prefixed and unprefixed variants
+  const coreModules = new Set<string>([
+    ...builtinModules,
+    ...builtinModules.filter((m) => m.startsWith('node:')).map((m) => m.slice(5)),
+    ...builtinModules.filter((m) => !m.startsWith('node:')).map((m) => `node:${m}`)
+  ]);
+
+  const installed = new Set<string>([...Object.keys(pkg.dependencies || {}), ...coreModules]);
   const cmd = options.commands[manager];
 
   return {
     name: 'auto-install',
 
-    async resolveId(importee, importer) {
+    async resolveId(importee: string, importer: string | undefined) {
       // entry module
       if (!importer) return null;
 

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

@@ -10,6 +10,7 @@ import nodeResolve from '@rollup/plugin-node-resolve';
 import autoInstall from '~package';
 
 const DIRNAME = fileURLToPath(new URL('.', import.meta.url));
+const PREV_CWD = process.cwd();
 const cwd = path.join(DIRNAME, 'fixtures/npm-bare');
 const file = path.join(cwd, 'output/bundle.js');
 const input = path.join(cwd, '../input.js');
@@ -31,4 +32,5 @@ it('npm, bare', async () => {
 
 afterAll(async () => {
   await del(['node_modules', 'package.json', 'package-lock.json']);
+  process.chdir(PREV_CWD);
 });

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

@@ -10,6 +10,7 @@ import nodeResolve from '@rollup/plugin-node-resolve';
 import autoInstall from '~package';
 
 const DIRNAME = fileURLToPath(new URL('.', import.meta.url));
+const PREV_CWD = process.cwd();
 const cwd = path.join(DIRNAME, 'fixtures/npm');
 const file = path.join(cwd, 'output/bundle.js');
 const input = path.join(cwd, '../input.js');
@@ -36,4 +37,5 @@ it('npm', async () => {
 afterAll(async () => {
   await del(['node_modules', 'package-lock.json']);
   fs.writeFileSync(pkgFile, '{}');
+  process.chdir(PREV_CWD);
 });

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

@@ -10,6 +10,7 @@ import nodeResolve from '@rollup/plugin-node-resolve';
 import autoInstall from '~package';
 
 const DIRNAME = fileURLToPath(new URL('.', import.meta.url));
+const PREV_CWD = process.cwd();
 const cwd = path.join(DIRNAME, 'fixtures/pnpm-bare');
 const file = path.join(cwd, 'output/bundle.js');
 const input = path.join(cwd, '../input.js');
@@ -30,4 +31,5 @@ it('pnpm, bare', async () => {
 
 afterAll(async () => {
   await del(['node_modules', 'package.json', 'pnpm-lock.yaml']);
+  process.chdir(PREV_CWD);
 });

packages/auto-install/test/pnpm.test.ts

@@ -10,6 +10,7 @@ import nodeResolve from '@rollup/plugin-node-resolve';
 import autoInstall from '~package';
 
 const DIRNAME = fileURLToPath(new URL('.', import.meta.url));
+const PREV_CWD = process.cwd();
 const cwd = path.join(DIRNAME, 'fixtures/pnpm');
 const file = path.join(cwd, 'output/bundle.js');
 const input = path.join(cwd, '../input.js');
@@ -31,4 +32,5 @@ it('pnpm', async () => {
 afterAll(async () => {
   await del(['node_modules', 'package.json']);
   fs.writeFileSync(path.join(cwd, 'pnpm-lock.yaml'), '');
+  process.chdir(PREV_CWD);
 });

packages/auto-install/test/yarn.test.ts

@@ -10,6 +10,7 @@ import nodeResolve from '@rollup/plugin-node-resolve';
 import autoInstall from '~package';
 
 const DIRNAME = fileURLToPath(new URL('.', import.meta.url));
+const PREV_CWD = process.cwd();
 const cwd = path.join(DIRNAME, 'fixtures/yarn');
 const file = path.join(cwd, 'output/bundle.js');
 const input = path.join(cwd, '../input.js');
@@ -30,4 +31,5 @@ it('yarn', async () => {
 afterAll(async () => {
   await del(['node_modules', 'package.json']);
   fs.writeFileSync(path.join(cwd, 'yarn.lock'), '');
+  process.chdir(PREV_CWD);
 });