Support options in quick fixes/refactoring

[ghost]

We should support editor configuration of services behavior like we do for formatting.

Would fix:

• Configure implement interface quick fix quote type #18169
• Add file extensions when auto-completing import paths #19019 (if there's no existing import to infer from)
• [Discussion] Run install @types quick fix with --no-shrinkwrap #19997
• We could prefer to create a namespace import instead of a named import for import completions and import fixes

[mjbvz]

As covered in #21768, I see a lot of requests against VS Code for configuring quotes specifically. Cases where quotes should be configurable include:

• Imports
• Bracket accessors
• Implement interface / implement abstract class or method
• Others?

[mhegazy]

Should also address:

• autoimport should care about semicolons #19882
• Configure implement interface quick fix quote type #18169
• TS AutoImport style config enhancement #20161
• Make adding semicolon configurable for "Extract function" #18780
• make the quote symbol configurable for import codefix #13270

[zpdDG4gta8XKpMCd]

please add an option to always use absolute paths (relative to baseUri) for imports at completions, per: #21860

[srcspider]

+1

If possible I would like an option to NEVER even see the relative path suggestions; or spaghetti imports as I like to think of them.

Any explanation on why it suddenly doesn't work at all? I've reverted to the 1.19.3 just because it's unusable. In 1.19.3 spaghetti paths aren't even suggested, only the correct absolute paths (as per the tsconfig)

[srcspider]

@mjbvz please add an option to omit the semicolon on import statements

[alextreppass]

Thanks - please consider #22249 to allow the old behaviour of full import paths (rather than ../../...), which I gather may be the same as #21860

[zpdDG4gta8XKpMCd]

how did it even come to mind to make relatives paths a default choice? what makes the design team think they are more convenient?

[srcspider]

@andy-ms why did you label this "Fixed"? What's the fix?

I've looked though 2.8.1, 2.8.2 and the roadmap for 2.9 not sure what you're implying fixes it.

[mhegazy]

The fix went in #22236. It is scheduled to release in Typescript 2.8 (added to the road-map).

VSCode /VS still need to expose these settings in a way you can interact with. microsoft/vscode#45948 is a good issue to watch for VSCode.

[mhegazy]

For posterity,

• Semicolon in quickfixes and refactorings tracked by autoimport should care about semicolons #19882, Make adding semicolon configurable for "Extract function" #18780 and TS AutoImport style config enhancement #20161
• Configure relative vs non-relative path in imports in quickfixes and completion tracked by auto import always uses relative (to the current file) module paths in presense of baseUri #21860 and TS AutoImport style config enhancement #20161
• Additional arguments to npm for installing missing @types packages tracked by [Discussion] Run install @types quick fix with --no-shrinkwrap #19997