[LS] Suggested import returns string with CRLFs, double quotes and no indent

[non25]

svelteserver screenshots

Contrary to tsserver, which inserts imports without CRLFs and uses quote variant that is prevalent in this file.

tsserver screenshots

Edit: I've also tried converting Promise-based function to an async in svelte file, it inserts CRLFs too, indent is messed up, but at least it doesn't touch my single quotes: 😄

I guess vcsode just autoformats everything after code actions so it goes unnoticed?
My autoformatter doesn't strip CRs from LFs. 🤔

[dummdidumm]

These options are passed into the typescript language service by us, and it works in VS Code. You likely need to pass TypeScript's user preferences into the language server similar to this.

The indent is a different issue indepent of the configuration.

[jasonlyu123]

With 106.0.0. The behaviour now is:

1. If the current file contains LF but not CRLF, use LF otherwise use the system default. i.e. require('os').EOL
2. base indent for the module and instance script tag is now 4 by default. If you have a prettier config it would use the tabWidth config. If you don't want the base indent set svelteIndentScriptAndStyle to false in your prettier config.
3. quoteStyle can already be configured with typescript.preferences.quoteStyle or javascript.preferences.quoteStyle

That check all the boxes. So I think we can close this now. If there's anything I missed, feel free to reopen it.