Don't add file extensions automatically about plugins HOT 10 CLOSED

This is preventing me from using this in a TypeScript build setup because it is automatically adding .js when I just want to put .ts

from plugins.

@Rich-Harris I'd love to see this integrated, even if it requires a major bump. Otherwise, an alternative or workaround would be appreciated!

from plugins.

This also break my import './some-my-style.css';.

from plugins.

@Rich-Harris is this one still relevant?

from plugins.

I just ran into this problem too, when I want to alias a JSON file and then load it with rollup-plugin-json, it fails because this configuration:

{find: 'aliased-json', replacement: './test.json'},

makes it look for test.json.js.

So I have to create a dummy JS test.js next to test.json that just imports and exports test.json, and then alias to that rather than my JSON file.

If a non-breaking change was desired, an option could be added to disable the automatic file extension.

EDIT: Sorry, I just realized what the resolve option does. Still seems like it'd be better to change the default behavior :)

from plugins.

webpack has a similar thing that allows us to alias directories so we can even alias to asset files that might be images.

Currently my config is

alias({
            '@assets': __dirname + '/src/assets'
        }),

And i'm doing
import png from "@assets/icon.png";

but this adds .js to the end of the file.. which is not ideal. it seems alias shouldnt be bound to a file type assumption.

from plugins.

Another approach could be to use the this.resolve context function to resolve this. If someone were to use rollup-plugin-node-resolve, they get the resolution algorithm they know and love, otherwise just what is built-in. @Acionyx has been contacting me because he is working on a fork to do just that but I did not yet find time to have a look.

from plugins.

@lukastaegert could you give a small example of using this.resolve?

Cheers.

from plugins.

Here is a real-world example, though it is probably not a simple one: https://github.com/rollup/rollup-plugin-commonjs/blob/535e8117607731e1e8688be84c3aeb6e5c3b6fa5/src/resolve-id.js#L54

Here it is taking an import of the form commonjs-proxy:./some-imports.js, removes the prefix and uses this.resolve to find out what path the import of ./some-import.js resolves to.

from plugins.

#34 was merged and published a little while back. It now uses this.resolve() and should alleviate this issue. If it still pops up with the new version, please respond and we'll reopen.

from plugins.