Implement support for external: "all" in @nx/vite:build executor

[belaczek]

Description

In @nx/rollup:rollup executor, it is possible to use external: "all" property. That way, all dependencies (including other publishable buildable libs) are automatically marked as external and not bundled.
While trying to migrate some libraries to Vite, I found out this executor property is missing and such functionality not implemented for @nx/vite:build executor.

Motivation

It would be convenient not having to care about keeping package.json and build.rollupOptions.external in sync.

Suggested Implementation

It should be possible to reuse the existing implementation from rollup plugin

nx/packages/rollup/src/executors/rollup/rollup.impl.ts

Line 169 in 867a5a8

export function createRollupOptions(

.

[github-actions[bot]]

This issue has been automatically marked as stale because it hasn't had any activity for 6 months.
Many things may have changed within this time. The issue may have already been fixed or it may not be relevant anymore.
If at this point, this is still an issue, please respond with updated information.
It will be closed in 21 days if no further activity occurs.
Thanks for being a part of the Nx community! 🙏

[alexstanbury]

@belaczek did you ever manage to get anywhere with this? All of my buildable libraries are huge because of this, and there doesn't seem to be a simple way of making all dependencies external.