test(auto-install): make npm/pnpm bare tests robust on Windows by using local input and scoped cleanup\n\n- Copy shared input into the fixture cwd so node-resolve finds installed deps\n- Run del cleanup in the fixture cwd and avoid deleting repo files\n- Keep env tweaks scoped per-test; no behavior change for npm/pnpm/yarn tests\n\nFixes CI Windows Node 20 failures introduced by 218488a.

Author: CharlieHelps

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

@@ -12,7 +12,10 @@ import nodeResolve from '@rollup/plugin-node-resolve';
 const DIR = fileURLToPath(new URL('.', import.meta.url));
 const cwd = path.join(DIR, 'fixtures/npm-bare');
 const file = path.join(cwd, 'output/bundle.js');
-const input = path.join(cwd, '../input.js');
+// Use a local input inside the cwd so Node resolution finds packages installed
+// by the test (e.g., on Windows where upward-only resolution won't see
+// `npm-bare/node_modules` from `fixtures/input.js`).
+const input = path.join(cwd, 'input.local.js');
 const pkgFile = path.join(cwd, 'package.json');
 
 // Helper to temporarily disable slow npm features during the test
@@ -48,6 +51,8 @@ it.runIf(RUN_ON_THIS_NODE)(
   async () => {
     const restoreEnv = stubNpmQuietEnv();
     const prevCwd = process.cwd();
+    // Create a local copy of the shared input so resolution starts from `cwd`.
+    fs.copyFileSync(path.join(cwd, '../input.js'), input);
     process.chdir(cwd);
     try {
       const { default: autoInstall } = await import('~package');
@@ -65,12 +70,17 @@ it.runIf(RUN_ON_THIS_NODE)(
       }
     } finally {
       process.chdir(prevCwd);
+      try {
+        fs.unlinkSync(input);
+      } catch {
+        /* ignore cleanup errors */
+      }
       restoreEnv();
     }
   },
   60000
 );
 
 afterAll(async () => {
-  await del(['node_modules', 'package.json', 'package-lock.json']);
+  await del(['node_modules', 'package.json', 'package-lock.json'], { cwd });
 });

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

@@ -12,13 +12,18 @@ import nodeResolve from '@rollup/plugin-node-resolve';
 const DIR = fileURLToPath(new URL('.', import.meta.url));
 const cwd = path.join(DIR, 'fixtures/pnpm-bare');
 const file = path.join(cwd, 'output/bundle.js');
-const input = path.join(cwd, '../input.js');
+// Use a local input inside the cwd so Node resolution finds packages installed
+// by the test (e.g., on Windows where upward-only resolution won't see
+// `pnpm-bare/node_modules` from `fixtures/input.js`).
+const input = path.join(cwd, 'input.local.js');
 
 const PREV_CWD = process.cwd();
 const [NODE_MAJOR, NODE_MINOR] = process.versions.node.split('.').map(Number);
 const RUN_ON_THIS_NODE = NODE_MAJOR > 20 || (NODE_MAJOR === 20 && NODE_MINOR >= 19);
 
 it.runIf(RUN_ON_THIS_NODE)('pnpm, bare', async () => {
+  // Create a local copy of the shared input so resolution starts from `cwd`.
+  fs.copyFileSync(path.join(cwd, '../input.js'), input);
   process.chdir(cwd);
   const { default: autoInstall } = await import('~package');
   const bundle = await rollup({
@@ -36,6 +41,12 @@ it.runIf(RUN_ON_THIS_NODE)('pnpm, bare', async () => {
 });
 
 afterAll(async () => {
-  await del(['node_modules', 'package.json', 'pnpm-lock.yaml']);
+  // Ensure cleanup runs against the fixture directory, not the repo root
+  await del(['node_modules', 'package.json', 'pnpm-lock.yaml'], { cwd });
+  try {
+    fs.unlinkSync(input);
+  } catch {
+    /* ignore cleanup errors */
+  }
   process.chdir(PREV_CWD);
 });