Support format vue file

[tjx666]

This extension is very useful to format ts/js file imports, but seems not support .vue yet. @daidodo

[daidodo]

Duplicate to https://github.com/daidodo/format-imports-vscode/issues/37.

[tjx666]

@daidodo @zddai any problems when implement vue support? Vue user amount is very large...

[daidodo]

I'm not familiar with Vue and don't have much time to ramp up. If someone with experience can help, that'd be great!

[tjx666]

@daidodo

1. match the script tag in vue file by https://regex101.com/r/zvwJ1B/1
2. format the script code
3. insert the result code back

This should also works for svelte

[daidodo]

Many thanks for your PR (https://github.com/daidodo/format-imports/pull/19)!

Before shipping it to production, I still have a few questions:

• What are the supported lang values? For example, if <script lang="coffee">, the plugin should skip. I assume they are default value (js?), plus ts. Are there other values (tsx, jsx)?

• How is tsconfig.json resolved? The plugin uses TypeScript API to find it for source file. I'm not sure if it's the same way in Vue? Another minor question is: From the doc, seems jsx is always preserved in Vue, which means import React from 'react' should be kept. But in your test case it's removed. Am I missing something?

• From these examples it seems names can be imported in <script> and used in <template>. How to handle such cases?

I've added some test cases (which may fail) to reflect my questions.

[tjx666]

• vue only support js/jsx/ts/tsx

• tsconfig should be resolved same like rename the .vue extension to .ts. Vue jsx needn't react, jsx is not designed for react.

• seems I didn't implement the remove unused imports complented. In fact, variables can even be used in style block

[daidodo]

• Any doc or link? (VSCode warns about jsx and tsx)
• Any doc or link?
• This can be a blocker for the feature. (The test cases are failing).

[tjx666]

• you should use volar which is a more recommend vue extension. use tsx/jsx is no warnings is my vscode + volar
• no, this is my thought, or it there any other posible?
• my work is busy recently, if you feel hard to implements it, you can just revert to last pr.

[daidodo]

• I believe here are all values for <script lang>.
• I implemented it based on your assumption. However, we can change it anytime if it's wrong.
• Many thanks for your works so far! There are no other critical features in the queue so no pressure. I tried to understand how Volar is working (because it gives correct unused warning messages) but got no luck. Will dive deeper later.

[daidodo]

@tjx666 I've released a new version to support Vue, with removing unused imports is disabled.

Please take a try and tell me if you find any issue. Thanks!

[tjx666]

simple try, and works good