Skip to content
This repository was archived by the owner on Jan 27, 2026. It is now read-only.

Commit 99e7cd1

Browse files
authored
chore: bump rollup to v4 (#2630)
* chore: bump rollup to v4 Signed-off-by: Kim Tsao <ktsao@redhat.com> * chore: fix ts errors Signed-off-by: Kim Tsao <ktsao@redhat.com> * chore: remove patch changeset Signed-off-by: Kim Tsao <ktsao@redhat.com> --------- Signed-off-by: Kim Tsao <ktsao@redhat.com>
1 parent 795c12e commit 99e7cd1

File tree

5 files changed

+222
-24
lines changed

5 files changed

+222
-24
lines changed

.changeset/happy-monkeys-cover.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@janus-idp/cli": minor
3+
---
4+
5+
Bump rollup to v4

packages/cli/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@
7474
"react-dev-utils": "^12.0.0-next.60",
7575
"react-refresh": "^0.14.0",
7676
"recursive-readdir": "^2.2.2",
77-
"rollup": "^2.78.0",
77+
"rollup": "^4.0.0",
7878
"rollup-plugin-dts": "^4.0.1",
7979
"rollup-plugin-esbuild": "^4.7.2",
8080
"rollup-plugin-postcss": "^4.0.0",

packages/cli/src/commands/export-dynamic-plugin/backend-embed-as-code.ts

Lines changed: 17 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -164,20 +164,22 @@ export async function backend(
164164
} = {};
165165

166166
const rollupConfig = rollupConfigs[0];
167-
rollupConfig.plugins?.push(
168-
embedModules({
169-
filter: filter,
170-
addDependency: (embeddedModule, dependencyName, newDependencyVersion) =>
171-
addToDependenciesForModule({
172-
dependency: {
173-
name: dependencyName,
174-
version: newDependencyVersion,
175-
},
176-
dependencies: dependenciesToAdd,
177-
module: embeddedModule,
178-
}),
179-
}),
180-
);
167+
if (Array.isArray(rollupConfig.plugins)) {
168+
rollupConfig.plugins?.push(
169+
embedModules({
170+
filter: filter,
171+
addDependency: (embeddedModule, dependencyName, newDependencyVersion) =>
172+
addToDependenciesForModule({
173+
dependency: {
174+
name: dependencyName,
175+
version: newDependencyVersion,
176+
},
177+
dependencies: dependenciesToAdd,
178+
module: embeddedModule,
179+
}),
180+
}),
181+
);
182+
}
181183

182184
if (Array.isArray(rollupConfig.output)) {
183185
rollupConfig.output.forEach(output => {
@@ -194,7 +196,7 @@ export async function backend(
194196
);
195197
return interopForPackage[id];
196198
}
197-
return interopForAll || true; // true is the default value in Rollup.
199+
return interopForAll || 'compat'; // compat is the default value in Rollup.
198200
};
199201
}
200202
});

packages/cli/src/lib/builder/config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import yaml from '@rollup/plugin-yaml';
2323
import svgr from '@svgr/rollup';
2424
import chalk from 'chalk';
2525
import fs from 'fs-extra';
26-
import { OutputOptions, RollupOptions, RollupWarning } from 'rollup';
26+
import { OutputOptions, RollupLog, RollupOptions } from 'rollup';
2727
import dts from 'rollup-plugin-dts';
2828
import esbuild from 'rollup-plugin-esbuild';
2929
import postcss from 'rollup-plugin-postcss';
@@ -68,7 +68,7 @@ export async function makeRollupConfigs(
6868
targetPkg = (await fs.readJson(packagePath)) as BackstagePackageJson;
6969
}
7070

71-
const onwarn = ({ code, message }: RollupWarning) => {
71+
const onwarn = ({ code, message }: RollupLog) => {
7272
if (code === 'EMPTY_BUNDLE') {
7373
return; // We don't care about this one
7474
}

0 commit comments

Comments
 (0)