fix(eslint-plugin): [no-empty-interface] use suggestion fixer for ambient contexts

packages/eslint-plugin/src/rules/no-empty-interface.ts

@@ -1,4 +1,9 @@
 import * as util from '../util';
+import { AST_NODE_TYPES } from '@typescript-eslint/experimental-utils';
+import {
+  RuleFix,
+  RuleFixer,
+} from '@typescript-eslint/experimental-utils/dist/ts-eslint';
 
 type Options = [
   {
@@ -43,6 +48,7 @@ export default util.createRule<Options, MessageIds>({
     return {
       TSInterfaceDeclaration(node): void {
         const sourceCode = context.getSourceCode();
+        const filename = context.getFilename();
 
         if (node.body.body.length !== 0) {
           // interface contains members --> Nothing to report
@@ -58,21 +64,45 @@ export default util.createRule<Options, MessageIds>({
         } else if (extend.length === 1) {
           // interface extends exactly 1 interface --> Report depending on rule setting
           if (!allowSingleExtends) {
+            const fix = (fixer: RuleFixer): RuleFix => {
+              let typeParam = '';
+              if (node.typeParameters) {
+                typeParam = sourceCode.getText(node.typeParameters);
+              }
+              return fixer.replaceText(
+                node,
+                `type ${sourceCode.getText(
+                  node.id,
+                )}${typeParam} = ${sourceCode.getText(extend[0])}`,
+              );
+            };
+
+            // Check if interface is within ambient declaration
+            let useAutoFix = true;
+            if (util.isDefinitionFile(filename)) {
+              const scope = context.getScope();
+              if (
+                scope.block.parent &&
+                scope.block.parent.type ===
+                  AST_NODE_TYPES.TSModuleDeclaration &&
+                scope.block.parent.declare
+              ) {
+                useAutoFix = false;
+              }
+            }
+
             context.report({
               node: node.id,
               messageId: 'noEmptyWithSuper',
-              fix: fixer => {
-                let typeParam = '';
-                if (node.typeParameters) {
-                  typeParam = sourceCode.getText(node.typeParameters);
-                }
-                return fixer.replaceText(
-                  node,
-                  `type ${sourceCode.getText(
-                    node.id,
-                  )}${typeParam} = ${sourceCode.getText(extend[0])}`,
-                );
-              },
+              fix: useAutoFix ? fix : undefined,
+              suggest: useAutoFix
+                ? undefined
+                : [
+                    {
+                      messageId: 'noEmptyWithSuper',
+                      fix,
+                    },
+                  ],
             });
           }
         }

packages/eslint-plugin/tests/rules/no-empty-interface.test.ts

@@ -148,5 +148,27 @@ type Foo<T> = Bar<T>
         },
       ],
     },
+    {
+      filename: 'test.d.ts',
+      code: `
+declare module FooBar {
+  type Baz = typeof baz;
+  export interface Bar extends Baz {}
+}
+      `,
+      errors: [
+        {
+          messageId: 'noEmptyWithSuper',
+          line: 4,
+        },
+      ],
+      // output matches input because a suggestion was made
+      output: `
+declare module FooBar {
+  type Baz = typeof baz;
+  export interface Bar extends Baz {}
+}
+      `,
+    },
   ],
 });