this.addWatchFile rollup compatibility

[DylanPiercey]

Clear and concise description of the problem

There is an API incompatibility with rollup for the addWatchFile api.
Specifically the first note after the addWatchFile documentation mentions the following:

Using this.addWatchFile from within the transform hook will make sure the transform hook is also reevaluated for this module if the watched file changes.

In Vite this.addWatchFile does nothing special in this case and just tracks the new file independently.

Suggested solution

Ideally we'd mirror the rollup behavior.

[dominikg]

I recently added a workaround for that in vite-plugin-svelte by triggering virtual change events on the watcher. Could be useful here aswell? But i guess internally in vite it would be easier to just call the handler directly and skip the watch event.

https://github.com/sveltejs/vite-plugin-svelte/blob/main/packages/vite-plugin-svelte/src/utils/watch.ts

[github-actions]

Hello @DylanPiercey. We like your proposal/feedback and would appreciate a contribution via a Pull Request by you or another community member. We thank you in advance for your contribution and are looking forward to reviewing it!

[mattcompiles]

Just dropping in my use-case for this feature as requested.

Our CSS-in-JS library vanilla-extract runs JS/TS files (e.g. styles.css.ts) at build time to generate CSS files. The JS file you write is not sent to the client as it is only executed at build time. Each .css.ts file can import other files. e.g. styles.css.ts imports colors.ts. In this example, if the colors.ts file is updated, we want to re-run the transform for styles.css.ts.

Other prior art here is the webpack addDependency API.