microsoft · jramsay · Oct 28, 2016 · Oct 24, 2016 · Oct 27, 2016 · Oct 28, 2016
diff --git a/src/compiler/program.ts b/src/compiler/program.ts
@@ -725,6 +725,14 @@ namespace ts {
         }
 
         function getSyntacticDiagnosticsForFile(sourceFile: SourceFile): Diagnostic[] {
+            // For JavaScript files, we report semantic errors for using TypeScript-only
+            // constructs from within a JavaScript file as syntactic errors.
+            if (isSourceFileJavaScript(sourceFile)) {
+                if (!sourceFile.additionalSyntacticDiagnostics) {
+                    sourceFile.additionalSyntacticDiagnostics = getJavaScriptSyntacticDiagnosticsForFile(sourceFile);
+                }
+                return concatenate(sourceFile.additionalSyntacticDiagnostics, sourceFile.parseDiagnostics);
+            }
             return sourceFile.parseDiagnostics;
         }
 
@@ -757,20 +765,18 @@ namespace ts {
 
                 Debug.assert(!!sourceFile.bindDiagnostics);
                 const bindDiagnostics = sourceFile.bindDiagnostics;
-                // For JavaScript files, we don't want to report the normal typescript semantic errors.
-                // Instead, we just report errors for using TypeScript-only constructs from within a
-                // JavaScript file.
-                const checkDiagnostics = isSourceFileJavaScript(sourceFile) ?
-                    getJavaScriptSemanticDiagnosticsForFile(sourceFile) :
-                    typeChecker.getDiagnostics(sourceFile, cancellationToken);
+                // For JavaScript files, we don't want to report semantic errors.
+                // Instead, we'll report errors for using TypeScript-only constructs from within a
+                // JavaScript file when we get syntactic diagnostics for the file.
+                const checkDiagnostics = isSourceFileJavaScript(sourceFile) ? [] : typeChecker.getDiagnostics(sourceFile, cancellationToken);
                 const fileProcessingDiagnosticsInFile = fileProcessingDiagnostics.getDiagnostics(sourceFile.fileName);
                 const programDiagnosticsInFile = programDiagnostics.getDiagnostics(sourceFile.fileName);
 
                 return bindDiagnostics.concat(checkDiagnostics, fileProcessingDiagnosticsInFile, programDiagnosticsInFile);
             });
         }
 
-        function getJavaScriptSemanticDiagnosticsForFile(sourceFile: SourceFile): Diagnostic[] {
+        function getJavaScriptSyntacticDiagnosticsForFile(sourceFile: SourceFile): Diagnostic[] {
             return runWithCancellationToken(() => {
                 const diagnostics: Diagnostic[] = [];
                 walk(sourceFile);

diff --git a/src/compiler/types.ts b/src/compiler/types.ts
@@ -2024,6 +2024,9 @@ namespace ts {
         // as well as code diagnostics).
         /* @internal */ parseDiagnostics: Diagnostic[];
 
+        // Stores additional file level diagnostics reported by the program
+        /* @internal */ additionalSyntacticDiagnostics?: Diagnostic[];
+
         // File level diagnostics reported by the binder.
         /* @internal */ bindDiagnostics: Diagnostic[];
 

diff --git a/...lash/getJavaScriptSemanticDiagnostics1.ts → ...ash/getJavaScriptSyntacticDiagnostics1.ts b/...lash/getJavaScriptSemanticDiagnostics1.ts → ...ash/getJavaScriptSyntacticDiagnostics1.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// import a = b;
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'import ... =' can only be used in a .ts file.",
     "start": 0,

diff --git a/...ash/getJavaScriptSemanticDiagnostics10.ts → ...sh/getJavaScriptSyntacticDiagnostics10.ts b/...ash/getJavaScriptSemanticDiagnostics10.ts → ...sh/getJavaScriptSyntacticDiagnostics10.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// function F<T>() { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'type parameter declarations' can only be used in a .ts file.",
     "start": 11,

diff --git a/...ash/getJavaScriptSemanticDiagnostics11.ts → ...sh/getJavaScriptSyntacticDiagnostics11.ts b/...ash/getJavaScriptSemanticDiagnostics11.ts → ...sh/getJavaScriptSyntacticDiagnostics11.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// function F(): number { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'types' can only be used in a .ts file.",
     "start": 14,

diff --git a/...ash/getJavaScriptSemanticDiagnostics12.ts → ...sh/getJavaScriptSyntacticDiagnostics12.ts b/...ash/getJavaScriptSemanticDiagnostics12.ts → ...sh/getJavaScriptSyntacticDiagnostics12.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// declare var v;
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'declare' can only be used in a .ts file.",
     "start": 0,

diff --git a/...ash/getJavaScriptSemanticDiagnostics13.ts → ...sh/getJavaScriptSyntacticDiagnostics13.ts b/...ash/getJavaScriptSemanticDiagnostics13.ts → ...sh/getJavaScriptSyntacticDiagnostics13.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// var v: () => number;
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'types' can only be used in a .ts file.",
     "start": 7,

diff --git a/...ash/getJavaScriptSemanticDiagnostics14.ts → ...sh/getJavaScriptSyntacticDiagnostics14.ts b/...ash/getJavaScriptSemanticDiagnostics14.ts → ...sh/getJavaScriptSyntacticDiagnostics14.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// Foo<number>();
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'type arguments' can only be used in a .ts file.",
     "start": 4,

diff --git a/...ash/getJavaScriptSemanticDiagnostics15.ts → ...sh/getJavaScriptSyntacticDiagnostics15.ts b/...ash/getJavaScriptSemanticDiagnostics15.ts → ...sh/getJavaScriptSyntacticDiagnostics15.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// function F(public p) { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'parameter modifiers' can only be used in a .ts file.",
     "start": 11,

diff --git a/...ash/getJavaScriptSemanticDiagnostics16.ts → ...sh/getJavaScriptSyntacticDiagnostics16.ts b/...ash/getJavaScriptSemanticDiagnostics16.ts → ...sh/getJavaScriptSyntacticDiagnostics16.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// function F(p?) { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'?' can only be used in a .ts file.",
     "start": 12,

diff --git a/...ash/getJavaScriptSemanticDiagnostics17.ts → ...sh/getJavaScriptSyntacticDiagnostics17.ts b/...ash/getJavaScriptSemanticDiagnostics17.ts → ...sh/getJavaScriptSyntacticDiagnostics17.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// function F(a: number) { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'types' can only be used in a .ts file.",
     "start": 14,

diff --git a/...ash/getJavaScriptSemanticDiagnostics18.ts → ...sh/getJavaScriptSyntacticDiagnostics18.ts b/...ash/getJavaScriptSemanticDiagnostics18.ts → ...sh/getJavaScriptSyntacticDiagnostics18.ts
@@ -9,7 +9,7 @@
 ////}
 
 goTo.file("a.js");
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "\'public\' can only be used in a .ts file.",
     "start": 93,
@@ -25,7 +25,7 @@ verify.getSemanticDiagnostics(`[
 ////}
 
 goTo.file("b.js");
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'types' can only be used in a .ts file.",
     "start": 17,

diff --git a/...ash/getJavaScriptSemanticDiagnostics19.ts → ...sh/getJavaScriptSyntacticDiagnostics19.ts b/...ash/getJavaScriptSemanticDiagnostics19.ts → ...sh/getJavaScriptSyntacticDiagnostics19.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// enum E { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'enum declarations' can only be used in a .ts file.",
     "start": 5,

diff --git a/...lash/getJavaScriptSemanticDiagnostics2.ts → ...ash/getJavaScriptSyntacticDiagnostics2.ts b/...lash/getJavaScriptSemanticDiagnostics2.ts → ...ash/getJavaScriptSyntacticDiagnostics2.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// export = b;
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'export=' can only be used in a .ts file.",
     "start": 0,

diff --git a/...ash/getJavaScriptSemanticDiagnostics21.ts → ...sh/getJavaScriptSyntacticDiagnostics21.ts b/...ash/getJavaScriptSemanticDiagnostics21.ts → ...sh/getJavaScriptSyntacticDiagnostics21.ts
diff --git a/...ash/getJavaScriptSemanticDiagnostics22.ts → ...sh/getJavaScriptSyntacticDiagnostics22.ts b/...ash/getJavaScriptSemanticDiagnostics22.ts → ...sh/getJavaScriptSyntacticDiagnostics22.ts
diff --git a/...ash/getJavaScriptSemanticDiagnostics23.ts → ...sh/getJavaScriptSyntacticDiagnostics23.ts b/...ash/getJavaScriptSemanticDiagnostics23.ts → ...sh/getJavaScriptSyntacticDiagnostics23.ts
diff --git a/...ash/getJavaScriptSemanticDiagnostics24.ts → ...sh/getJavaScriptSyntacticDiagnostics24.ts b/...ash/getJavaScriptSemanticDiagnostics24.ts → ...sh/getJavaScriptSyntacticDiagnostics24.ts
diff --git a/...lash/getJavaScriptSemanticDiagnostics3.ts → ...ash/getJavaScriptSyntacticDiagnostics3.ts b/...lash/getJavaScriptSemanticDiagnostics3.ts → ...ash/getJavaScriptSyntacticDiagnostics3.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// class C<T> { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'type parameter declarations' can only be used in a .ts file.",
     "start": 8,

diff --git a/...lash/getJavaScriptSemanticDiagnostics4.ts → ...ash/getJavaScriptSyntacticDiagnostics4.ts b/...lash/getJavaScriptSemanticDiagnostics4.ts → ...ash/getJavaScriptSyntacticDiagnostics4.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// public class C { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'public' can only be used in a .ts file.",
     "start": 0,

diff --git a/...lash/getJavaScriptSemanticDiagnostics5.ts → ...ash/getJavaScriptSyntacticDiagnostics5.ts b/...lash/getJavaScriptSemanticDiagnostics5.ts → ...ash/getJavaScriptSyntacticDiagnostics5.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// class C implements D { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'implements clauses' can only be used in a .ts file.",
     "start": 8,

diff --git a/...lash/getJavaScriptSemanticDiagnostics6.ts → ...ash/getJavaScriptSyntacticDiagnostics6.ts b/...lash/getJavaScriptSemanticDiagnostics6.ts → ...ash/getJavaScriptSyntacticDiagnostics6.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// interface I { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'interface declarations' can only be used in a .ts file.",
     "start": 10,

diff --git a/...lash/getJavaScriptSemanticDiagnostics7.ts → ...ash/getJavaScriptSyntacticDiagnostics7.ts b/...lash/getJavaScriptSemanticDiagnostics7.ts → ...ash/getJavaScriptSyntacticDiagnostics7.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// module M { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'module declarations' can only be used in a .ts file.",
     "start": 7,

diff --git a/...lash/getJavaScriptSemanticDiagnostics8.ts → ...ash/getJavaScriptSyntacticDiagnostics8.ts b/...lash/getJavaScriptSemanticDiagnostics8.ts → ...ash/getJavaScriptSyntacticDiagnostics8.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// type a = b;
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'type aliases' can only be used in a .ts file.",
     "start": 5,

diff --git a/...lash/getJavaScriptSemanticDiagnostics9.ts → ...ash/getJavaScriptSyntacticDiagnostics9.ts b/...lash/getJavaScriptSemanticDiagnostics9.ts → ...ash/getJavaScriptSyntacticDiagnostics9.ts
@@ -4,7 +4,7 @@
 // @Filename: a.js
 //// public function F() { }
 
-verify.getSemanticDiagnostics(`[
+verify.getSyntacticDiagnostics(`[
   {
     "message": "'public' can only be used in a .ts file.",
     "start": 0,

diff --git a/tests/cases/fourslash/server/getJavaScriptSyntacticDiagnostics02.ts b/tests/cases/fourslash/server/getJavaScriptSyntacticDiagnostics02.ts
@@ -0,0 +1,40 @@
+/// <reference path="../fourslash.ts" />
+
+// @allowJs: true
+// @Filename: b.js
+//// var a = "a";
+//// var b: boolean = true;
+//// function foo(): string { }
+//// var var = "c";
+
+verify.getSyntacticDiagnostics(`[
+  {
+    "message": "\'types\' can only be used in a .ts file.",
+    "start": 20,
+    "length": 7,
+    "category": "error",
+    "code": 8010
+  },
+  {
+    "message": "Variable declaration expected.",
+    "start": 67,
+    "length": 3,
+    "category": "error",
+    "code": 1134
+  },
+  {
+    "message": "Variable declaration expected.",
+    "start": 71,
+    "length": 1,
+    "category": "error",
+    "code": 1134
+  },
+  {
+    "message": "Variable declaration expected.",
+    "start": 73,
+    "length": 3,
+    "category": "error",
+    "code": 1134
+  }
+]`);
+verify.getSemanticDiagnostics(`[]`);