[Bug] Importing .vue file in .jsx file added .js extension

[wenfangdu]

Repro: https://github.com/wenfangdu/volar-jsx-import-vue-repro

[johnsoncodehk]

I can't reproduce, please provide some informations:

• VSCode info from Help: About command
• volar version
• Is TS plugin enabled?

[wenfangdu]

• VSCode Info
  Version: 1.56.2 (user setup)
  Commit: 054a9295330880ed74ceaedda236253b4f39a335
  Date: 2021-05-12T17:13:13.157Z
  Electron: 12.0.4
  Chrome: 89.0.4389.114
  Node.js: 14.16.0
  V8: 8.9.255.24-electron.0
  OS: Windows_NT x64 10.0.19043
• Volar Version
  v0.25.17
• TS plugin is enabled

[wenfangdu]

@johnsoncodehk
I see why now, to reproduce, please turn on "javascript.preferences.importModuleSpecifierEnding": "js" in your VSCode settings. For importing .vue files in .js/.jsx files, I think this setting should be ignored.

Just for completeness, I just tried enabling "typescript.preferences.importModuleSpecifierEnding": "js", then this issue appears for importing .vue files in .ts/.tsx files as well.

[johnsoncodehk]

I see, this is expected behavior. Will check is it suitable for patching or not.

[wenfangdu]

@johnsoncodehk
Since .vue files import auto-completion can only be enabled by enabling Volar, I think it makes sense to remove the .js suffix when auto-completing, could you please reconsider patching?

[johnsoncodehk]

Since patching involves too much hacking, and this is expected behavior, it is temporarily decided not to patch. I want to avoid hacking to achieve unexpected but correct behavior.