Harden telemetryOnOpenFile (#36993)

RyanCavanaugh · web-flow · commit c22cdb446ab0 · 2020-02-25T11:32:32.000-08:00
diff --git a/src/server/editorServices.ts b/src/server/editorServices.ts
@@ -3117,8 +3117,9 @@ namespace ts.server {
                 return;
             }
 
-            const info: OpenFileInfo = { checkJs: !!project.getSourceFile(scriptInfo.path)!.checkJsDirective };
-            this.eventHandler({ eventName: OpenFileInfoTelemetryEvent, data: { info } });
+            const sourceFile = project.getSourceFile(scriptInfo.path);
+            const checkJs = !!sourceFile && !!sourceFile.checkJsDirective;
+            this.eventHandler({ eventName: OpenFileInfoTelemetryEvent, data: { info: { checkJs } } });
         }
 
         /**

Original file line number	Diff line number	Diff line change
`@@ -3117,8 +3117,9 @@ namespace ts.server {`
`3117`	`3117`	`return;`
`3118`	`3118`	`}`
`3119`	`3119`
`3120`		`- const info: OpenFileInfo = { checkJs: !!project.getSourceFile(scriptInfo.path)!.checkJsDirective };`
`3121`		`- this.eventHandler({ eventName: OpenFileInfoTelemetryEvent, data: { info } });`
	`3120`	`+ const sourceFile = project.getSourceFile(scriptInfo.path);`
	`3121`	`+ const checkJs = !!sourceFile && !!sourceFile.checkJsDirective;`
	`3122`	`+ this.eventHandler({ eventName: OpenFileInfoTelemetryEvent, data: { info: { checkJs } } });`
`3122`	`3123`	`}`
`3123`	`3124`
`3124`	`3125`	`/**`