feat(alias,auto-install,beep,buble,commonjs,data-uri,dsv,dynamic-import-vars,eslint,esm-shim,graphql,image,inject,legacy,multi-entry,node-resolve,pluginutils,replace,run,strip,sucrase,swc,terser,typescript,url,virtual,yaml): support rollup 5

.config/vitest.config.mts

@@ -7,6 +7,7 @@ export default defineConfig({
   test: {
     // Enable global APIs for CommonJS test files.
     globals: true,
+    testTimeout: 20_000,
     // Phase 1/2/3 packages use runtime-style, top-level test files, *.test, and a few named entrypoints.
     include: [
       'test/*.{js,mjs,cjs,ts,mts,cts}',

packages/alias/package.json

@@ -48,7 +48,7 @@
     "rollup-plugin"
   ],
   "peerDependencies": {
-    "rollup": ">=4.0.0"
+    "rollup": "^4.0.0||^5.0.0"
   },
   "peerDependenciesMeta": {
     "rollup": {
@@ -58,7 +58,7 @@
   "devDependencies": {
     "@rollup/plugin-node-resolve": "^15.0.0",
     "del-cli": "^5.0.0",
-    "rollup": "^4.0.0",
+    "rollup": "catalog:rollup-next",
     "typescript": "catalog:"
   },
   "types": "./dist/index.d.ts",

packages/alias/test/alias.test.ts

@@ -414,11 +414,27 @@ describe('plugin-alias', () => {
       ]
     );
     expect(resolverCalls).toEqual([
-      ['entry-point', '/src/importer.js', { attributes: {}, custom: void 0, isEntry: true }],
+      [
+        'entry-point',
+        '/src/importer.js',
+        {
+          attributes: {},
+          custom: void 0,
+          importerAttributes: {},
+          importerRawId: '/src/importer.js',
+          isEntry: true
+        }
+      ],
       [
         'non-entry-point',
         '/src/importer.js',
-        { attributes: {}, custom: { test: 42 }, isEntry: false }
+        {
+          attributes: {},
+          custom: { test: 42 },
+          importerAttributes: {},
+          importerRawId: '/src/importer.js',
+          isEntry: false
+        }
       ]
     ]);
     expect(result).toEqual(['entry-point', 'non-entry-point']);
@@ -451,11 +467,27 @@ describe('plugin-alias', () => {
       ]
     );
     expect(resolverCalls).toEqual([
-      ['entry-point', '/src/importer.js', { attributes: {}, custom: void 0, isEntry: true }],
+      [
+        'entry-point',
+        '/src/importer.js',
+        {
+          attributes: {},
+          custom: void 0,
+          importerAttributes: {},
+          importerRawId: '/src/importer.js',
+          isEntry: true
+        }
+      ],
       [
         'non-entry-point',
         '/src/importer.js',
-        { attributes: {}, custom: { test: 42 }, isEntry: false }
+        {
+          attributes: {},
+          custom: { test: 42 },
+          importerAttributes: {},
+          importerRawId: '/src/importer.js',
+          isEntry: false
+        }
       ]
     ]);
     expect(result).toEqual(['entry-point', 'non-entry-point']);

packages/auto-install/package.json

@@ -54,7 +54,7 @@
     "rollup-plugin"
   ],
   "peerDependencies": {
-    "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0"
+    "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0||^5.0.0"
   },
   "peerDependenciesMeta": {
     "rollup": {
@@ -66,7 +66,7 @@
     "@rollup/plugin-typescript": "^9.0.1",
     "del": "^6.1.1",
     "node-noop": "^1.0.0",
-    "rollup": "^4.0.0-24",
+    "rollup": "catalog:rollup-next",
     "typescript": "^4.8.3"
   },
   "types": "./types/index.d.ts"

packages/auto-install/test/fixtures/pnpm-bare/.npmrc

@@ -0,0 +1,3 @@
+# pnpm refuses to add dependencies at a workspace root without an explicit opt-in;
+# this also means the root .npmrc no longer applies here, which is fine for this fixture
+ignore-workspace-root-check=true

packages/auto-install/test/fixtures/pnpm-bare/pnpm-workspace.yaml

@@ -0,0 +1,2 @@
+# Anchor pnpm workspace detection to this fixture so that package installs spawned here do not
+# operate on the monorepo workspace root and regenerate its lockfile.

packages/auto-install/test/fixtures/pnpm/.npmrc

@@ -0,0 +1,3 @@
+# pnpm refuses to add dependencies at a workspace root without an explicit opt-in;
+# this also means the root .npmrc no longer applies here, which is fine for this fixture
+ignore-workspace-root-check=true

packages/auto-install/test/fixtures/pnpm/pnpm-workspace.yaml

@@ -0,0 +1,2 @@
+# Anchor pnpm workspace detection to this fixture so that package installs spawned here do not
+# operate on the monorepo workspace root and regenerate its lockfile.

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

@@ -13,6 +13,12 @@ const input = join(cwd, '../input.js');
 
 process.chdir(cwd);
 
+// pnpm would otherwise discover the monorepo workspace root above the fixture and regenerate
+// its lockfile. The fixture pnpm-workspace.yaml anchors workspace detection to the fixture
+// directory; ensure the fixture installs leave the workspace root lockfile untouched.
+const rootLockfile = join(__dirname, '../../../pnpm-lock.yaml');
+const originalLockfileContent = readFileSync(rootLockfile);
+
 test('pnpm, bare', async () => {
   await rollup({
     input,
@@ -28,5 +34,12 @@ test('pnpm, bare', async () => {
 }, 50000);
 
 afterAll(async () => {
-  await del(['node_modules', 'package.json', 'pnpm-lock.yaml']);
+  try {
+    expect(
+      readFileSync(rootLockfile).equals(originalLockfileContent),
+      'the workspace root lockfile was modified by a fixture install'
+    ).toBe(true);
+  } finally {
+    await del(['node_modules', 'package.json', 'pnpm-lock.yaml']);
+  }
 });

packages/auto-install/test/pnpm.js

@@ -13,6 +13,12 @@ const input = join(cwd, '../input.js');
 
 process.chdir(cwd);
 
+// pnpm would otherwise discover the monorepo workspace root above the fixture and regenerate
+// its lockfile. The fixture pnpm-workspace.yaml anchors workspace detection to the fixture
+// directory; ensure the fixture installs leave the workspace root lockfile untouched.
+const rootLockfile = join(__dirname, '../../../pnpm-lock.yaml');
+const originalLockfileContent = readFileSync(rootLockfile);
+
 test('pnpm', async () => {
   await rollup({
     input,
@@ -29,6 +35,13 @@ test('pnpm', async () => {
 }, 50000);
 
 afterAll(async () => {
-  await del(['node_modules', 'package.json']);
-  writeFileSync('pnpm-lock.yaml', '');
+  try {
+    expect(
+      readFileSync(rootLockfile).equals(originalLockfileContent),
+      'the workspace root lockfile was modified by a fixture install'
+    ).toBe(true);
+  } finally {
+    await del(['node_modules', 'package.json']);
+    writeFileSync('pnpm-lock.yaml', '');
+  }
 });

packages/beep/package.json

@@ -38,15 +38,15 @@
     "rollup-plugin"
   ],
   "peerDependencies": {
-    "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0"
+    "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0||^5.0.0"
   },
   "peerDependenciesMeta": {
     "rollup": {
       "optional": true
     }
   },
   "devDependencies": {
-    "rollup": "^4.0.0-24",
+    "rollup": "catalog:rollup-next",
     "strip-ansi": "^7.0.1"
   },
   "types": "types/index.d.ts"

packages/buble/package.json

@@ -54,7 +54,7 @@
     "rollup-plugin"
   ],
   "peerDependencies": {
-    "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0"
+    "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0||^5.0.0"
   },
   "peerDependenciesMeta": {
     "rollup": {
@@ -69,7 +69,7 @@
   "devDependencies": {
     "@rollup/plugin-typescript": "^9.0.1",
     "del-cli": "^5.0.0",
-    "rollup": "^4.0.0-24",
+    "rollup": "catalog:rollup-next",
     "source-map": "^0.7.4",
     "typescript": "^4.8.3"
   },

packages/commonjs/package.json

@@ -55,7 +55,7 @@
     "rollup-plugin"
   ],
   "peerDependencies": {
-    "rollup": "^2.68.0||^3.0.0||^4.0.0"
+    "rollup": "^2.68.0||^3.0.0||^4.0.0||^5.0.0"
   },
   "peerDependenciesMeta": {
     "rollup": {
@@ -76,7 +76,7 @@
     "@rollup/plugin-node-resolve": "^15.0.0",
     "locate-character": "^2.0.5",
     "require-relative": "^0.8.7",
-    "rollup": "^4.0.0-24",
+    "rollup": "catalog:rollup-next",
     "source-map": "^0.7.4",
     "source-map-support": "^0.5.21",
     "typescript": "^4.8.3"

packages/commonjs/test/fixtures/function/custom-options/_config.js

@@ -21,8 +21,28 @@ module.exports = {
         },
         buildEnd() {
           assert.deepStrictEqual(resolveIdArgs, [
-            ['other.js', 'main.js', { attributes: {}, custom: { test: 42 }, isEntry: true }],
-            ['main.js', void 0, { attributes: {}, custom: {}, isEntry: true }]
+            [
+              'other.js',
+              'main.js',
+              {
+                attributes: {},
+                custom: { test: 42 },
+                importerAttributes: {},
+                importerRawId: ID_MAIN,
+                isEntry: true
+              }
+            ],
+            [
+              'main.js',
+              void 0,
+              {
+                attributes: {},
+                custom: {},
+                importerAttributes: {},
+                importerRawId: void 0,
+                isEntry: true
+              }
+            ]
           ]);
         },
         resolveId(source, importer, options) {

packages/commonjs/test/fixtures/function/module-side-effects-late-entry/_config.js

@@ -2,7 +2,7 @@ const path = require('path');
 const assert = require('assert');
 
 let referenceId;
-const testEntry = 'generated-foo2.js';
+const testEntry = 'generated-foo.js';
 
 module.exports = {
   description:

packages/commonjs/test/fixtures/function/strict-requires-debug-none/_config.js

@@ -18,6 +18,9 @@ module.exports = {
   exports() {
     assert.strictEqual(warnings.length, 1);
     assert.deepStrictEqual(warnings[0].ids, []);
-    assert.strictEqual(warnings[0].message, 'The commonjs plugin did not wrap any files.');
+    assert.strictEqual(
+      warnings[0].message,
+      '[plugin commonjs] The commonjs plugin did not wrap any files.'
+    );
   }
 };

packages/commonjs/test/fixtures/function/warn-this-resolve-without-options/_config.js

@@ -18,7 +18,7 @@ module.exports = {
           assert.strictEqual(warnings.length, 1);
           assert.strictEqual(
             warnings[0].message,
-            'It appears a plugin has implemented a "resolveId" hook that uses "this.resolve" without forwarding the third "options" parameter of "resolveId". This is problematic as it can lead to wrong module resolutions especially for the node-resolve plugin and in certain cases cause early exit errors for the commonjs plugin.\nIn rare cases, this warning can appear if the same file is both imported and required from the same mixed ES/CommonJS module, in which case it can be ignored.'
+            '[plugin commonjs--resolver] It appears a plugin has implemented a "resolveId" hook that uses "this.resolve" without forwarding the third "options" parameter of "resolveId". This is problematic as it can lead to wrong module resolutions especially for the node-resolve plugin and in certain cases cause early exit errors for the commonjs plugin.\nIn rare cases, this warning can appear if the same file is both imported and required from the same mixed ES/CommonJS module, in which case it can be ignored.'
           );
           assert.strictEqual(warnings[0].pluginCode, 'THIS_RESOLVE_WITHOUT_OPTIONS');
           assert.strictEqual(warnings[0].url, 'https://rollupjs.org/guide/en/#resolveid');