Expose JS API to combine multiple outputs into one

[aklinker1]

I'm working on a framework for building chrome extensions, WXT, that includes this plugin when running wxt build --analyze. Chrome extensions contain multiple bulid steps, so each step outputs a separate stats JSON file, and then the rollup-plugin-visualizer CLI is used to combine them.

However, this doesn't work out of the box when using pnpm and shamefully-hoist=true (the default). This is because the binary isn't included in node_modules/.bin, so it's not in the path. Instead, the binary is stored at node_modules/wxt/node_modules/.bin/rollup-plugin-visualizer. There might be a similar problem for yarn PnP, haven't tested it.

See wxt-dev/wxt#360 for more details.

I could add node_modules/wxt/node_modules/.bin to the path while calling spawn("rollup-plugin-visualizer") (and I likely will do that as a workaround for now), but I'd also like to just call a function instead of spawning a new process, something like this:

import { mergeStats } from 'rollup-plugin-visualizer';

await mergeStats({
  files: ["stats1.json", "stats2.json", ...],
  // other options...
});

After looking at the code, I think this is a pretty simple change. Just involves exporting runForPluginJson from plugin/index.ts, or another entrypoint if that doesn't make sense.

rollup-plugin-visualizer/bin/cli.ts

Line 56 in dec8398

const runForPluginJson = async ({ title, template, filename, open }: CliArgs, files: string[]) => {

If that sounds OK, I'm willing to work on this and open a PR!

[btd]

There is a CLI tool for this https://github.com/btd/rollup-plugin-visualizer#cli already. Not sure if i want to support merge as API

[btd]

Basically merge happen there https://github.com/btd/rollup-plugin-visualizer/blob/master/bin/cli.ts#L70C3-L102C5
I can extract this to the function, if it works for you

[aklinker1]

I would prefer the entire runForPluginJson function be exported. Or else please include the call to renderTemplate, since that's not exported either.

rollup-plugin-visualizer/bin/cli.ts

Lines 104 to 107 in dec8398

	const fileContent = await renderTemplate(template, {
	title,
	data: JSON.stringify(data),
	});

[btd]

I was looking on this again. I do not really want to keep this as public interface as is. I am fine with merge itself, but i am not ok to expose template rendering function. I do not want to maintain its public interface between versions.

Unless you find some ideas how to remove this requirement i am closing this and PR