From 03dcdda44342a07ea10701ae1725f174941d2d97 Mon Sep 17 00:00:00 2001
From: zhengbli <zhengli@microsoft.com>
Date: Thu, 18 Aug 2016 17:12:40 -0700
Subject: [PATCH] Treat special property access symbol differently ... when
 retriving documentation

---
 src/services/services.ts                      | 18 +++++++++++++++++
 .../completionEntryDetailAcrossFiles01.ts     | 20 +++++++++++++++++++
 2 files changed, 38 insertions(+)
 create mode 100644 tests/cases/fourslash/server/completionEntryDetailAcrossFiles01.ts

diff --git a/src/services/services.ts b/src/services/services.ts
index ea85caf1332..64b872d14a7 100644
--- a/src/services/services.ts
+++ b/src/services/services.ts
@@ -4913,6 +4913,24 @@ namespace ts {
 
             if (!documentation) {
                 documentation = symbol.getDocumentationComment();
+                if ((!documentation || documentation.length === 0) && symbol.flags & SymbolFlags.Property) {
+                    // For some special property access expressions like `experts.foo = foo` or `module.exports.foo = foo`
+                    // there documentation comments might be attached to the right hand side symbol of their declarations.
+                    // The pattern of such special property access is that the parent symbol is the symbol of the file.
+                    if (symbol.parent && forEach(symbol.parent.declarations, declaration => declaration.kind === SyntaxKind.SourceFile)) {
+                        forEach(symbol.declarations, declaration => {
+                            if (declaration.parent && declaration.parent.kind === SyntaxKind.BinaryExpression) {
+                                const rhsSymbol = program.getTypeChecker().getSymbolAtLocation((<BinaryExpression>declaration.parent).right);
+                                if (rhsSymbol) {
+                                    documentation = rhsSymbol.getDocumentationComment();
+                                    if (documentation && documentation.length > 0) {
+                                        return true;
+                                    }
+                                }
+                            }
+                        });
+                    }
+                }
             }
 
             return { displayParts, documentation, symbolKind };
diff --git a/tests/cases/fourslash/server/completionEntryDetailAcrossFiles01.ts b/tests/cases/fourslash/server/completionEntryDetailAcrossFiles01.ts
new file mode 100644
index 00000000000..243975fde2d
--- /dev/null
+++ b/tests/cases/fourslash/server/completionEntryDetailAcrossFiles01.ts
@@ -0,0 +1,20 @@
+/// <reference path="../fourslash.ts"/>
+
+// @allowNonTsExtensions: true
+// @Filename: a.js
+//// /**
+////  * Modify the parameter
+////  * @param {string} p1
+////  */
+//// var foo = function (p1) { }
+//// exports.foo = foo;
+//// fo/*1*/
+
+// @Filename: b.ts
+//// import a = require("./a");
+//// a.fo/*2*/
+
+goTo.marker('1');
+verify.completionEntryDetailIs("foo", "var foo: (p1: string) => void", "Modify the parameter");
+goTo.marker('2');
+verify.completionEntryDetailIs("foo", "(property) a.foo: (p1: string) => void", "Modify the parameter");