`@sentry/bundler-plugins` imports types from `rollup`, which causes a TS error because it's an optional dependency

[aguo-ntech]

@sentry/bundler-plugins declares rollup as an optional peer dependency, but its built type definitions for the ./rollup export import TransformResult directly from rollup.

Because the ./vite export transitively imports from the rollup module, consumers of @sentry/vite-plugin (or @sentry/bundler-plugins/vite) who do not have rollup installed hit a TS2307 error.

Reproduction

import { sentryVitePlugin } from '@sentry/vite-plugin';

TypeScript reports:

node_modules/@sentry/bundler-plugins/build/types/rollup/index.d.ts:5:38 - error TS2307: Cannot find module 'rollup' or its corresponding type declarations.

5 import type { TransformResult } from 'rollup';

Findings

• packages/bundler-plugins/package.json has rollup in peerDependencies with optional: true.
• packages/bundler-plugins/src/rollup/index.ts line 23: import type { TransformResult } from 'rollup';.
• packages/bundler-plugins/src/vite/index.ts imports from ../rollup, so the Vite type path depends on the Rollup type file.

[linear-code]

JS-3054

[sentry-pierre]

I cleaned up the issue body to remove the template noise and highlight the actual bug.

What I checked:

• packages/bundler-plugins/package.json marks rollup as an optional peer dependency.
• packages/bundler-plugins/src/rollup/index.ts directly imports TransformResult from rollup.
• packages/bundler-plugins/src/vite/index.ts imports from the rollup module, which is why the Vite path also pulls in the missing type dependency.

This is a confirmed bundler-plugin type dependency mismatch. A maintainer can take it from here.

[isaacs]

Thanks for reporting! This is a straightforward bug. Should be addressed by #22304.