Supress console.log & others inside the TS Server runtime so that language service plugins don't accidentally use them and kill the server - fixes microsoft#31209

orta · orta · commit f04c7ed8334b · 2019-08-01T17:36:09.000-04:00
diff --git a/src/tsserver/server.ts b/src/tsserver/server.ts
@@ -970,4 +970,12 @@ namespace ts.server {
     if (ts.sys.tryEnableSourceMapsForHost && /^development$/i.test(ts.sys.getEnvironmentVariable("NODE_ENV"))) {
         ts.sys.tryEnableSourceMapsForHost();
     }
+
+    // Overwrites the current console messages to instead write to
+    // the log. This is so that language service plugins which use
+    // console.log don't break the message passing between tsserver
+    // and the client
+    console.log = (msg) => logger.msg(msg, Msg.Info);
+    console.warn = (msg) => logger.msg(msg, Msg.Err);
+    console.error = (msg) => logger.msg(msg, Msg.Err);
 }

Original file line number	Diff line number	Diff line change
`@@ -970,4 +970,12 @@ namespace ts.server {`
`970`	`970`	`if (ts.sys.tryEnableSourceMapsForHost && /^development$/i.test(ts.sys.getEnvironmentVariable("NODE_ENV"))) {`
`971`	`971`	`ts.sys.tryEnableSourceMapsForHost();`
`972`	`972`	`}`
	`973`	`+`
	`974`	`+ // Overwrites the current console messages to instead write to`
	`975`	`+ // the log. This is so that language service plugins which use`
	`976`	`+ // console.log don't break the message passing between tsserver`
	`977`	`+ // and the client`
	`978`	`+ console.log = (msg) => logger.msg(msg, Msg.Info);`
	`979`	`+ console.warn = (msg) => logger.msg(msg, Msg.Err);`
	`980`	`+ console.error = (msg) => logger.msg(msg, Msg.Err);`
`973`	`981`	`}`