Update import/export specifier sort on rename about typescript HOT 7 OPEN

@abarghoud according to the repository CONTRIBUTING any issue with the label "Help wanted" is allowed to be worked by anyone.

Also, is important to avoid Issue Claiming.

Please feel free to start working in this, you don't have to ask permission as it's labelled with "Help wanted"

from typescript.

@TulioPintoNeto Thanks for pointing it out, I did not know about the issue claiming thing. I forgot to read the contribution guide 🤦‍♂️

from typescript.

Sorry, I was supposed to mention @andrewbranch.

But to answer you, @abarghoud :

• This is a experience enhancement, not a bug (not that the nomenclature for this case matters too much hehe);
• Probably it is possible to achieve this using the TypeScript compiler. The responsibility for intellisense refactorings/formatting/codefixes is within the TypeScript compiler itself. You may find most of the intellisense things in the src/services/ folder;

But the main thing I'm calling out here is that reordering lines when renaming will not always give a consistent output with Eslint (which is the desired according to the use cases provided in the description of this issue). This happens due to the fact that each codebase can set up its own rules for ordering imports.

from typescript.

@RyanCavanaugh I'm interested in tackling this. Is this something I can jump in on?

from typescript.

Context

@abarghoud I was checking about the way we sort when adding an import (the suggestion on this issue)

Seems like we are sorting by the folder structure, not by the variable name, while Eslint rule is sorting by variable name.

TypeScript import (please note that we have an error in line 2 due to sort-import rule from eslint):

Next steps?

With that in mind:

• Should we use the same order the import command does?;
  OR
• Should we use the same order as the Eslint rule?

I think it's pertinent, so I'm also bringing this:

Eslint team chose not to auto-fix the order of imports in --fix because this change the way the code could behaviour:
eslint/eslint#11542

I mean, by implementing this behaviour (of reording lines) we can even increase the amount of conflicts with Eslint, as the order of lines could be defined differently for each project on the eslint config. So I'm advocating against it based on the use cases given

from typescript.

@TulioPintoNeto I support the notion of retaining TypeScript's team's design for auto-import ordering. This is especially pertinent considering the availability of multiple ESLint rules that can be configured to alter the sorting method.

I believe the bug that needs fixing is the failure to reorder file names when renaming them. However, I'm concerned that it may not be possible because I think IDEs handle it rather than TypeScript itself.

Wdyt?

from typescript.

There are TS Server preferences that control the sort order, and a lot of utilities under ts.OrganizeImports to detect and apply those preferences.

from typescript.