adding an option to suffix the file extension about vite-plugin-require-transform HOT 14 CLOSED

Hey again 👋 Thanks for your help The main problem I think is that since there are two files with the same, but different extensions, this plugin converts them to the same variable 👇

I've moved away from the require statements and am trying a solution with imports, I'll lay down a repo where the issue can be replicated.

I think the latest version can solve the problem of same file name,different extensions,I just wrote some test code,the default variable will be turn into fileName_extension_default format.

from vite-plugin-require-transform.

//same path,different extension
const testCaseA = require("caseA.extA?aaa");
const testCaseB = require("caseA.extB?bbb").default;

console.log("caseA", testCaseA)
console.log("caseB", testCaseB)`

will be turn into:

import { default as _vite_plugin_require_transform_caseA_extB_bbb_default } from "caseA.extB?bbb";
import _vite_plugin_require_transform_caseA_extA_aaa from "caseA.extA?aaa";
//same path,different extension
const testCaseA = _vite_plugin_require_transform_caseA_extA_aaa;
const testCaseB = _vite_plugin_require_transform_caseA_extB_bbb_default;
console.log("caseA", testCaseA);
console.log("caseB", testCaseB);

from vite-plugin-require-transform.

Can you take a screenshot about the error? maybe the error is not about the extensions I think, it's about the format require(...).default, let's see what the real problem is and I'm glad to fix it

from vite-plugin-require-transform.

Hey 👋
Thanks for replying.

From what I've seen, this started to happen with .css & .ttf files with the same name, here's my case:


& here's the error

from vite-plugin-require-transform.

In the case of audiosprites, which require to have the same name across different formats, this is also a blocker:

Basically, if we could add an option to suffix the variables with the file format, I think it would solve the issue

from vite-plugin-require-transform.

checkout the newest version,with readme about case 3,it's supported now!

from vite-plugin-require-transform.

Hey 👋
Thanks for your switf action 👍
I gave it a try and now it seems to be losing the file extension when importing:

Relevant code:

EDIT: Just posting my config

    requireTransform(
      (fileRegex = /.ts$|.tsx$|.png$|.css$|.json$|.ttf$|.ac3$|.ogg$|.m4a$|.mp3$/),
      (prefix = "vite_require_transform_"),
      (importPathHandler = (requirePath) => {
        return requirePath.replace(".", "_");
      })
    ),

from vite-plugin-require-transform.

Thanks for your patient,I think this time would be fixed,just check the newest version again

from vite-plugin-require-transform.

Hey man 👋
Thanks again for comming up with a solution.
I gave it a shot and while it did indeed fixed the required errors, I now have the wrong file being loaded.

I'm still not 100% clear if this is something I'm doing wrong, but then again, I'm migrating from webpack, so I expected this to work almost out of the box 😆
I'll post here if I manage to get a handle on the transpiled files

from vite-plugin-require-transform.

I've tried to move the .default to another part of the app, and I'm back to the first error.
Here's the relevant code:


ImagesHudShiftKeyFrames is an empty enum

The error

And my config:

 defineConfig({
  plugins: [
    requireTransform(
      (fileRegex = /.ts$|.tsx$|.png$|.css$|.json$|.ttf$|.ac3$|.ogg$|.m4a$|.mp3$/),
      (prefix = "vite_require_transform_"),
      (importPathHandler = (requirePath) => {
        return requirePath.replace(".", "_");
      })
    ),
  ],
...})

from vite-plugin-require-transform.

Thanks for your patitent again and I'm glad to solve the problem.I am not sure if it is the cause the problem of the plugin,Actually I can not understand the real problem that you propose this time,maybe you can provide a minimal repo link that can reproduce the problem you ran into.

from vite-plugin-require-transform.

Hey again 👋
Thanks for your help
The main problem I think is that since there are two files with the same, but different extensions, this plugin converts them to the same variable 👇

I've moved away from the require statements and am trying a solution with imports, I'll lay down a repo where the issue can be replicated.

from vite-plugin-require-transform.

Thanks for the help :)

from vite-plugin-require-transform.

Thanks for the help :)

you're welcome.It's my pleasure hahah to make sth to help anyone in the world,my friend

from vite-plugin-require-transform.