avoid add missing member in declaration file

Author: Kingwl

src/services/codefixes/fixAddMissingMember.ts

@@ -113,6 +113,11 @@ namespace ts.codefix {
     }
     type Info = EnumInfo | ClassOrInterfaceInfo;
 
+    function isInNodeModulesDeclarationFile(node: Node) {
+        const sourceFile = getSourceFileOfNode(node);
+        return sourceFile.isDeclarationFile && startsWith(sourceFile.resolvedPath, "node_modules/") || sourceFile.resolvedPath.indexOf("/node_modules/") !== -1;
+    }
+
     function getInfo(tokenSourceFile: SourceFile, tokenPos: number, checker: TypeChecker): Info | undefined {
         // The identifier of the missing property. eg:
         // this.missing = 1;
@@ -131,15 +136,15 @@ namespace ts.codefix {
 
         // Prefer to change the class instead of the interface if they are merged
         const classOrInterface = find(symbol.declarations, isClassLike) || find(symbol.declarations, isInterfaceDeclaration);
-        if (classOrInterface) {
+        if (classOrInterface && !isInNodeModulesDeclarationFile(classOrInterface)) {
             const makeStatic = ((leftExpressionType as TypeReference).target || leftExpressionType) !== checker.getDeclaredTypeOfSymbol(symbol);
             const declSourceFile = classOrInterface.getSourceFile();
             const inJs = isSourceFileJS(declSourceFile);
             const call = tryCast(parent.parent, isCallExpression);
             return { kind: InfoKind.ClassOrInterface, token, parentDeclaration: classOrInterface, makeStatic, declSourceFile, inJs, call };
         }
         const enumDeclaration = find(symbol.declarations, isEnumDeclaration);
-        if (enumDeclaration) {
+        if (enumDeclaration && !isInNodeModulesDeclarationFile(enumDeclaration)) {
             return { kind: InfoKind.Enum, token, parentDeclaration: enumDeclaration };
         }
         return undefined;

tests/cases/fourslash/addMemberInDeclarationFile.ts

@@ -0,0 +1,17 @@
+/// <reference path="fourslash.ts" />
+
+// @Filename: ./declarations.d.ts
+//// interface Response {}
+
+// @Filename: foo.ts
+//// import './declarations.d.ts'
+//// declare const resp: Response
+//// resp.test()
+
+goTo.file('foo.ts')
+
+verify.codeFixAvailable([
+    { description: "Declare method 'test'" },
+    { description: "Declare property 'test'" },
+    { description: "Add index signature for property 'test'" }
+])

tests/cases/fourslash/addMemberNotInNodeModulesDeclarationFile.ts

@@ -0,0 +1,13 @@
+/// <reference path="fourslash.ts" />
+
+// @Filename: /node_modules/foo/declarations.d.ts
+//// interface Response {}
+
+// @Filename: foo.ts
+//// import '/node_modules/foo/declarations.d.ts'
+//// declare const resp: Response
+//// resp.test()
+
+goTo.file('foo.ts')
+
+verify.not.codeFixAvailable()