Better error reporting when Node.js is not installed when generating TypeScript files.

Author: KirillOsenkov

src/Common/Log.cs

@@ -14,14 +14,15 @@ public class Log
         public static string ErrorLogFilePath = ErrorLogFile;
         public static string MessageLogFilePath = MessageLogFile;
 
-        public static void Exception(Exception e, string message)
+        public static void Exception(Exception e, string message, bool isSevere = true)
         {
-            WriteToFile(message + Environment.NewLine + e.ToString(), ErrorLogFilePath);
+            var text = message + Environment.NewLine + e.ToString();
+            Exception(text, isSevere);
         }
 
-        public static void Exception(string message)
+        public static void Exception(string message, bool isSevere = true)
         {
-            Write(message, ConsoleColor.Red);
+            Write(message, isSevere ? ConsoleColor.Red : ConsoleColor.Yellow);
             WriteToFile(message, ErrorLogFilePath);
         }
 

src/HtmlGenerator/Pass1-Generation/TypeScriptSupport.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.ComponentModel;
 using System.IO;
 using System.Linq;
 using System.Text;
@@ -50,7 +51,15 @@ public void Generate(IEnumerable<string> typeScriptFiles)
                 libFile = Path.Combine(Common.Paths.BaseAppFolder, "TypeScriptSupport", "lib.d.ts");
             }
 
-            GenerateCore(list, libFile);
+            try
+            {
+                GenerateCore(list, libFile);
+            }
+            catch (Exception ex)
+            {
+                Log.Exception(ex, "Error when generating TypeScript files");
+                return;
+            }
 
             ProjectGenerator.GenerateReferencesDataFilesToAssembly(
                 Paths.SolutionDestinationFolder,
@@ -81,7 +90,19 @@ private void GenerateCore(IEnumerable<string> fileNames, string libFile)
             File.WriteAllText(argumentsJson, json);
 
             var analyzerJs = Path.Combine(Common.Paths.BaseAppFolder, @"TypeScriptSupport\analyzer.js");
-            var result = new ProcessLaunchService().RunAndRedirectOutput("node", string.Format("\"{0}\" {1}", analyzerJs, argumentsJson));
+            var arguments = string.Format("\"{0}\" {1}", analyzerJs, argumentsJson);
+
+            ProcessLaunchService.ProcessRunResult result;
+            try
+            {
+                result = new ProcessLaunchService().RunAndRedirectOutput("node", arguments);
+            }
+            catch (Win32Exception)
+            {
+                Log.Write("Warning: Node.js is required to generate TypeScript files. Skipping generation. Download Node.js from https://nodejs.org.", ConsoleColor.Yellow);
+                Log.Exception("Node.js is not installed.");
+                return;
+            }
 
             foreach (var file in Directory.GetFiles(output))
             {

src/HtmlGenerator/Utilities/FirstChanceExceptionHandler.cs

@@ -125,7 +125,7 @@ public static void HandleFirstChanceException(object sender, FirstChanceExceptio
                     }
                 }
 
-                Log.Exception(ex, message);
+                Log.Exception(ex, message, isSevere: false);
             }
             finally
             {