fix(prefer-import-from-vue): skip side-effect import in .d.ts files (#1907)

* fix(prefer-import-from-vue): skip side-effect import

* fix: only skip in .d.ts

* Update tests/lib/rules/prefer-import-from-vue.js

Co-authored-by: Yosuke Ota <[email protected]>

Co-authored-by: Yosuke Ota <[email protected]>

Author: antfu

lib/rules/prefer-import-from-vue.js

@@ -94,6 +94,13 @@ module.exports = {
 
     return {
       ImportDeclaration(node) {
+        // Skip imports without specifiers in `.d.ts` files
+        if (
+          node.specifiers.length === 0 &&
+          context.getFilename().endsWith('.d.ts')
+        )
+          return
+
         verifySource(node.source, () => {
           if (SUBSET_AT_VUE_MODULES.has(node.source.value)) {
             // If the module is a subset of 'vue', we can safely change it to 'vue'.

tests/lib/rules/prefer-import-from-vue.js

@@ -22,7 +22,11 @@ tester.run('prefer-import-from-vue', rule, {
     `export * from 'vue'`,
     `import Foo from 'foo'`,
     `import { createApp } from 'vue'
-    export { createApp }`
+    export { createApp }`,
+    {
+      filename: 'test.d.ts',
+      code: `import '@vue/runtime-dom'`
+    }
   ],
   invalid: [
     {