Seems that this.info wouldn't be called by rollup itself until the plugins call it

Hello!

When I use some rollup plugins, only warning log at the plugin for rollup is treated by the vite.

I propose the following three modifications.

1. Use onLog instead of onwarn. (Rollup Configuration guide - onLog)

   vite/packages/vite/src/node/build.ts

   Lines 537 to 552 in 6d5be5c

   	const rollupOptions: RollupOptions = {
   	context: 'globalThis',
   	preserveEntrySignatures: ssr
   	? 'allow-extension'
   	: libOptions
   	? 'strict'
   	: false,
   	cache: config.build.watch ? undefined : false,
   	...options.rollupOptions,
   	input,
   	plugins,
   	external,
   	onwarn(warning, warn) {
   	onRollupWarning(warning, warn, config)
   	},
   	}

2. Following codes would be modified to accept all log levels(info,warn,error,debug).
   

   vite/packages/vite/src/node/build.ts

   Lines 869 to 936 in 6d5be5c

   	export function onRollupWarning(
   	warning: RollupWarning,
   	warn: LoggingFunction,
   	config: ResolvedConfig,
   	): void {
   	const viteWarn: LoggingFunction = (warnLog) => {
   	let warning: string | RollupLog
   	if (typeof warnLog === 'function') {
   	warning = warnLog()
   	} else {
   	warning = warnLog
   	}
   	if (typeof warning === 'object') {
   	if (warning.code === 'UNRESOLVED_IMPORT') {
   	const id = warning.id
   	const exporter = warning.exporter
   	// throw unless it's commonjs external...
   	if (!id || !/\?commonjs-external$/.test(id)) {
   	throw new Error(
   	`[vite]: Rollup failed to resolve import "${exporter}" from "${id}".\n` +
   	`This is most likely unintended because it can break your application at runtime.\n` +
   	`If you do want to externalize this module explicitly add it to\n` +
   	`\`build.rollupOptions.external\``,
   	)
   	}
   	}
   	if (
   	warning.plugin === 'rollup-plugin-dynamic-import-variables' &&
   	dynamicImportWarningIgnoreList.some((msg) =>
   	// @ts-expect-error warning is RollupLog
   	warning.message.includes(msg),
   	)
   	) {
   	return
   	}
   	if (warningIgnoreList.includes(warning.code!)) {
   	return
   	}
   	if (warning.code === 'PLUGIN_WARNING') {
   	config.logger.warn(
   	`${colors.bold(
   	colors.yellow(`[plugin:${warning.plugin}]`),
   	)} ${colors.yellow(warning.message)}`,
   	)
   	return
   	}
   	}
   	warn(warnLog)
   	}
   	const tty = process.stdout.isTTY && !process.env.CI
   	if (tty) {
   	process.stdout.clearLine(0)
   	process.stdout.cursorTo(0)
   	}
   	const userOnWarn = config.build.rollupOptions?.onwarn
   	if (userOnWarn) {
   	userOnWarn(warning, viteWarn)
   	} else {
   	viteWarn(warning)
   	}
   	}

3. Also change the options where Rollup is called in different code.

   vite/packages/vite/src/node/plugins/worker.ts

   Lines 75 to 83 in ef4602b

   	const bundle = await rollup({
   	...rollupOptions,
   	input: cleanUrl(id),
   	plugins,
   	onwarn(warning, warn) {
   	onRollupWarning(warning, warn, config)
   	},
   	preserveEntrySignatures: false,
   	})

Thank you!

+1
Seems onwarn and onLog defined in vite.config.mts are not called when running npx vite build