import-js icon indicating copy to clipboard operation
import-js copied to clipboard

Discontinue `importjs` cli tool

Open trotzig opened this issue 8 years ago • 5 comments

I'm currently working on an index of all files and their exports. An important part of that is a full scan of the file system (inside a project) to find all js/jsx/json files. We then check each of those files for exports that we store in an index.

This is time-consuming, which means we probably can't afford it in simple importjs calls. I'm looking into using a cache of some sort, but we'll still have to deal with cold starts.

Since the daemon is used by all plugins except the emacs one (although I know @kevinkehl might be working on moving it over), I want to discontinue support for plain importjs. This probably means discontinuing support for rewrite as well, which I actually think is better handled by tools like jscodeshift.

trotzig avatar Dec 26 '16 16:12 trotzig

The export finder feature might also lead to having a hard dependency on watchman. Or at least some watcher at least.

trotzig avatar Dec 26 '16 16:12 trotzig

I am still planning on bringing back the daemon version soon. I'll probably have time the first week or two of 2017.

kevinkehl avatar Dec 26 '16 16:12 kevinkehl

No rush, I have no immediate plan of cutting it.

trotzig avatar Dec 27 '16 10:12 trotzig

an index of all files and their exports

... would be awesome!

a full importjs rewrite on a large project takes forever

milahu avatar May 14 '21 19:05 milahu

@milahu I'm not actively working on anything import-js related right now but I know that the index was implemented. I didn't drop support for the CLI though, but there's an underlying SQLite database that holds an up-to-date index of all exports.

trotzig avatar May 17 '21 06:05 trotzig