Add AddScriptWithoutTransform method for loading JavaScript file without JSX transformation.

It's becoming more and more common to use external build systems (Gulp, Grunt) or bundlers (Webpack, Browserify) that handle the JSX transformation. The output of these can be loaded directly into ReactJS.NET as it's just vanilla JavaScript. `AddScriptWithoutTransform` makes loading these files more efficient by just loading them directly. Closes #91

Author: Daniel15

build.proj

@@ -10,8 +10,8 @@ of patent rights can be found in the PATENTS file in the same directory.
 <Project ToolsVersion="4.0" DefaultTargets="Build;Test;Package" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
 	<PropertyGroup>
 		<Major>1</Major>
-		<Minor>4</Minor>
-		<Build>1</Build>
+		<Minor>5</Minor>
+		<Build>0</Build>
 		<Revision>0</Revision>
 		<DevNuGetServer>http://reactjs.net/packages/</DevNuGetServer>
 		<MSBuildCommunityTasksPath>$(MSBuildProjectDirectory)\tools\MSBuildTasks</MSBuildCommunityTasksPath>

src/React.AspNet/project.json

@@ -1,5 +1,5 @@
 {
-    "version": "1.4.1-*",
+    "version": "1.5.0-*",
     "configurations": {
         "Debug": {
             "compilationOptions": {

src/React.Core/IReactSiteConfiguration.cs

@@ -19,7 +19,9 @@ public interface IReactSiteConfiguration
 	{
 		/// <summary>
 		/// Adds a script to the list of scripts that are executed. This should be called for all
-		/// React components and their dependencies.
+		/// React components and their dependencies. If the script does not have any JSX in it
+		/// (for example, it's built using Webpack or Gulp), use 
+		/// <see cref="AddScriptWithoutTransform"/> instead.
 		/// </summary>
 		/// <param name="filename">
 		/// Name of the file to execute. Should be a server relative path starting with ~ (eg. 
@@ -29,10 +31,29 @@ public interface IReactSiteConfiguration
 		IReactSiteConfiguration AddScript(string filename);
 
 		/// <summary>
-		/// Gets a list of all the scripts that have been added to this configuration.
+		/// Adds a script to the list of scripts that are executed. This is the same as
+		/// <see cref="AddScript"/> except it does not run JSX transformation on the script and thus is
+		/// more efficient.
+		/// </summary>
+		/// <param name="filename">
+		/// Name of the file to execute. Should be a server relative path starting with ~ (eg. 
+		/// <c>~/Scripts/Awesome.js</c>)
+		/// </param>
+		/// <returns>The configuration, for chaining</returns>
+		IReactSiteConfiguration AddScriptWithoutTransform(string filename);
+
+		/// <summary>
+		/// Gets a list of all the scripts that have been added to this configuration and require JSX
+		/// transformation to be run.
 		/// </summary>
 		IList<string> Scripts { get; }
 
+		/// <summary>
+		/// Gets a list of all the scripts that have been added to this configuration and do not 
+		/// require JSX transformation to be run.
+		/// </summary>
+		IList<string> ScriptsWithoutTransform { get; } 
+
 		/// <summary>
 		/// A value indicating if es6 syntax should be rewritten.
 		/// </summary>

src/React.Core/JavaScriptEngineFactory.cs

@@ -78,7 +78,9 @@ IFileSystem fileSystem
 			{
 				_pool = CreatePool();
 				_resetPoolTimer = new Timer(OnResetPoolTimer);
-				_watchedFiles = new HashSet<string>(_config.Scripts.Select(
+
+				var allFiles = _config.Scripts.Concat(_config.ScriptsWithoutTransform);
+				_watchedFiles = new HashSet<string>(allFiles.Select(
 					fileName => _fileSystem.MapPath(fileName).ToLowerInvariant()
 				));
 				try
@@ -136,6 +138,27 @@ protected virtual void InitialiseEngine(IJsEngine engine)
 			engine.ExecuteResource("React.Resources.react-with-addons.js", thisAssembly);
 			engine.Execute("var React = global.React");
 			engine.ExecuteResource("React.Resources.JSXTransformer.js", thisAssembly);
+
+			// Only scripts that don't need JSX transformation can run immediately here
+			// JSX files are loaded in ReactEnvironment.
+			foreach (var file in _config.ScriptsWithoutTransform)
+			{
+				try
+				{
+					var contents = _fileSystem.ReadAsString(file);
+					engine.Execute(contents);
+				}
+				catch (JsRuntimeException ex)
+				{
+					throw new ReactScriptLoadException(string.Format(
+						"Error while loading \"{0}\": {1}\r\nLine: {2}\r\nColumn: {3}",
+						file,
+						ex.Message,
+						ex.LineNumber,
+						ex.ColumnNumber
+					));
+				}
+			}
 		}
 
 		/// <summary>

src/React.Core/ReactEnvironment.cs

@@ -157,7 +157,8 @@ public virtual string Version
 		}
 
 		/// <summary>
-		/// Ensures any user-provided scripts have been loaded
+		/// Ensures any user-provided scripts have been loaded. This only loads JSX files; files
+		/// that need no transformation are loaded in JavaScriptEngineFactory.
 		/// </summary>
 		protected virtual void EnsureUserScriptsLoaded()
 		{

src/React.Core/ReactSiteConfiguration.cs

@@ -39,13 +39,21 @@ public ReactSiteConfiguration()
 		}
 
 		/// <summary>
-		/// All the scripts that have been added to this configuration
+		/// All the scripts that have been added to this configuration and require JSX 
+		/// transformation to be run.
 		/// </summary>
 		private readonly IList<string> _scriptFiles = new List<string>();
+		/// <summary>
+		/// All the scripts that have been added to this configuration and do not require JSX
+		/// transformation to be run.
+		/// </summary>
+		private readonly IList<string> _scriptFilesWithoutTransform = new List<string>();
 
 		/// <summary>
 		/// Adds a script to the list of scripts that are executed. This should be called for all
-		/// React components and their dependencies.
+		/// React components and their dependencies. If the script does not have any JSX in it
+		/// (for example, it's built using Webpack or Gulp), use 
+		/// <see cref="AddScriptWithoutTransform"/> instead.
 		/// </summary>
 		/// <param name="filename">
 		/// Name of the file to execute. Should be a server relative path starting with ~ (eg. 
@@ -59,13 +67,38 @@ public IReactSiteConfiguration AddScript(string filename)
 		}
 
 		/// <summary>
-		/// Gets a list of all the scripts that have been added to this configuration.
+		/// Adds a script to the list of scripts that are executed. This is the same as
+		/// <see cref="AddScript"/> except it does not run JSX transformation on the script and thus is
+		/// more efficient.
+		/// </summary>
+		/// <param name="filename">
+		/// Name of the file to execute. Should be a server relative path starting with ~ (eg. 
+		/// <c>~/Scripts/Awesome.js</c>)
+		/// </param>
+		/// <returns>The configuration, for chaining</returns>
+		public IReactSiteConfiguration AddScriptWithoutTransform(string filename)
+		{
+			_scriptFilesWithoutTransform.Add(filename);
+			return this;
+		}
+
+		/// <summary>
+		/// Gets a list of all the scripts that have been added to this configuration and require JSX
+		/// transformation to be run.
 		/// </summary>
 		public IList<string> Scripts
 		{
 			get { return new ReadOnlyCollection<string>(_scriptFiles); }
 		}
 
+		/// <summary>
+		/// Gets a list of all the scripts that have been added to this configuration.
+		/// </summary>
+		public IList<string> ScriptsWithoutTransform
+		{
+			get { return new ReadOnlyCollection<string>(_scriptFilesWithoutTransform); }
+		}
+
 		/// <summary>
 		/// A value indicating if es6 syntax should be rewritten.
 		/// </summary>

src/React.Sample.Webpack/App_Start/ReactConfig.cs

@@ -16,7 +16,7 @@ public static class ReactConfig
 		public static void Configure()
 		{
 			ReactSiteConfiguration.Configuration
-				.AddScript("~/build/server.bundle.js");
+				.AddScriptWithoutTransform("~/build/server.bundle.js");
 		}
 	}
 }