Patching internal plugins does not work when native plugins are enabled

[sapphi-red]

Describe the regression

unplugin-vue-i18n patches the internal json plugin.
https://github.com/intlify/bundle-tools/blob/d3724e19cae256219f57ac7bc364ff2e07d05502/packages/unplugin-vue-i18n/src/core/resource.ts#L176-L198
This won't work when native plugins as the handler does not exist on JS side.

Reproduction

https://stackblitz.com/edit/vitejs-vite-kgm68otf?file=package.json,src%2FApp.vue,vite.config.js&terminal=build

Expected Behavior

There's a way to skip the internal json plugin for some modules.
related: vitejs#12168

Actual Behavior

There's no way to skip the internal json plugin for some modules.

Steps to Reproduce

No response

System Info

For all environments

Used Package Manager

npm

Logs

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs and the Rolldown-related guide.
• Check that there isn't already an issue that reports the same regression to avoid creating a duplicate.
• Check that this is a concrete regression. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the regression.

[sapphi-red]

The approaches I can think of are:

• add json.exclude option
  • in this case, unplugin-vue-i18n would set that option
  • it might be difficult to control it by multiple plugins?
  • one more config option
    • if we need other internal plugins to implement a similar feature, there would be many options...
  • easy to implement
• skip modules with meta.vite.skipPlugin: ['vite:json'] for the json plugin (ref1, ref2)
  • in this case, unplugin-vue-i18n would set meta.vite.skipPlugin: ['vite:json'] for the transformed modules
  • easy to implement
  • it should be more easier to compose multiple plugins options than json.exclude
  • reading meta on the native side is still an unresolved problem
• add moduleType support in Vite and skip modules without moduleType: 'json' for the json plugin
  • in this case, unplugin-vue-i18n would set moduleType: 'js' for the transformed modules
  • harder to implement as it requires moduleType support
  • probably more future-proof

[sapphi-red]

cc @kazupon

[kazupon]

I am working on this issue.
Fundamentally, I would like a solution to this problem on the native side, as sapphi-red suggested. 🙏

[sapphi-red]

Oh, I wasn't aware that this isn't working even without native plugins. The cause is that unplugin-vue-i18n transforms json to JS, but doesn't set the moduleType. When moduleType is set to json (which rolldown does automatically if the id ends with .json), rolldown tries to transform the module to JS.

@kazupon This can be fixed by adding moduleType: 'js' to the return value of the transform hook.
Example:

rolldown-vite/packages/vite/src/node/plugins/json.ts

Lines 84 to 88 in b642f48

	return {
	code,
	map: { mappings: '' },
	moduleType: 'js',
	}