vite-react-jsx icon indicating copy to clipboard operation
vite-react-jsx copied to clipboard

Pass filename to the babel transform call

Open ishmidt-pp opened this issue 4 years ago • 1 comments

Thanks for the plugin!

Problem

Calls to babel transform without passing the filename causes babel to fail. (According to the error stack and https://babeljs.io/docs/en/options#filename)

Environment

@babel/core: ^7.14.6 NodeJS: v14.17.1 vite: ^2.3.7 vite-react-jsx: ^1.1.1

Error stack

[vite:react-jsx] [BABEL] unknown: Preset /* your preset */ requires a filename to be set when babel is called directly,

babel.transform(code, { filename: 'file.ts', presets: [/* your preset */] });

See https://babeljs.io/docs/en/options#filename for more information.
file: src/index.tsx
error during build:
Error: [BABEL] unknown: Preset /* your preset */ requires a filename to be set when babel is called directly,

babel.transform(code, { filename: 'file.ts', presets: [/* your preset */] });

See https://babeljs.io/docs/en/options#filename for more information.
    at validateIfOptionNeedsFilename (node_modules/vite-react-jsx/node_modules/@babel/core/lib/config/full.js:295:11)
    at node_modules/vite-react-jsx/node_modules/@babel/core/lib/config/full.js:307:52
    at Array.forEach (<anonymous>)
    at validatePreset (node_modules/vite-react-jsx/node_modules/@babel/core/lib/config/full.js:307:25)
    at loadPresetDescriptor (node_modules/vite-react-jsx/node_modules/@babel/core/lib/config/full.js:314:3)
    at loadPresetDescriptor.next (<anonymous>)
    at recursePresetDescriptors (node_modules/vite-react-jsx/node_modules/@babel/core/lib/config/full.js:114:30)
    at recursePresetDescriptors.next (<anonymous>)
    at node_modules/vite-react-jsx/node_modules/@babel/core/lib/config/full.js:191:21
    at Generator.next (<anonymous>)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

ishmidt-pp avatar Jul 29 '21 22:07 ishmidt-pp

Hey @aleclarson, could you take a look at this PR please?

ishmidt-pp avatar Aug 02 '21 11:08 ishmidt-pp