Instead of the Node process exiting instantly on file change, send a signal to .NET that it should restart. This is working towards the connection-draining feature.

SteveSandersonMS · SteveSandersonMS · commit 26e8bd823cd2 · 2016-07-07T12:01:28.000+01:00
diff --git a/src/Microsoft.AspNetCore.NodeServices/Content/Node/entrypoint-http.js b/src/Microsoft.AspNetCore.NodeServices/Content/Node/entrypoint-http.js
@@ -184,7 +184,10 @@
 	        var ext = path.extname(filename);
 	        if (extensions.indexOf(ext) >= 0) {
 	            console.log('Restarting due to file change: ' + filename);
-	            process.exit(0);
+	            // Temporarily, the file-watching logic is in Node, so we signal it's time to restart by
+	            // sending the following message back to .NET. Soon the file-watching logic will move over
+	            // to the .NET side, and this whole file can be removed.
+	            console.log('[Microsoft.AspNetCore.NodeServices:Restart]');
 	        }
 	    });
 	}
diff --git a/src/Microsoft.AspNetCore.NodeServices/Content/Node/entrypoint-socket.js b/src/Microsoft.AspNetCore.NodeServices/Content/Node/entrypoint-socket.js
@@ -95,7 +95,10 @@
 	        var ext = path.extname(filename);
 	        if (extensions.indexOf(ext) >= 0) {
 	            console.log('Restarting due to file change: ' + filename);
-	            process.exit(0);
+	            // Temporarily, the file-watching logic is in Node, so we signal it's time to restart by
+	            // sending the following message back to .NET. Soon the file-watching logic will move over
+	            // to the .NET side, and this whole file can be removed.
+	            console.log('[Microsoft.AspNetCore.NodeServices:Restart]');
 	        }
 	    });
 	}
diff --git a/src/Microsoft.AspNetCore.NodeServices/HostingModels/OutOfProcessNodeInstance.cs b/src/Microsoft.AspNetCore.NodeServices/HostingModels/OutOfProcessNodeInstance.cs
@@ -18,6 +18,7 @@ namespace Microsoft.AspNetCore.NodeServices.HostingModels
     public abstract class OutOfProcessNodeInstance : INodeInstance
     {
         private const string ConnectionEstablishedMessage = "[Microsoft.AspNetCore.NodeServices:Listening]";
+        private const string NeedsRestartMessage = "[Microsoft.AspNetCore.NodeServices:Restart]";
         private readonly TaskCompletionSource<object> _connectionIsReadySource = new TaskCompletionSource<object>();
         private bool _disposed;
         private readonly StringAsTempFile _entryPointScript;
@@ -128,6 +129,13 @@ private void ConnectToInputOutputStreams()
                     _connectionIsReadySource.SetResult(null);
                     initializationIsCompleted = true;
                 }
+                else if (evt.Data == NeedsRestartMessage)
+                {
+                    // Temporarily, the file-watching logic is in Node, so look out for the
+                    // signal that we need to restart. This can be removed once the file-watching
+                    // logic is moved over to the .NET side.
+                    Dispose();
+                }
                 else if (evt.Data != null)
                 {
                     OnOutputDataReceived(evt.Data);
diff --git a/src/Microsoft.AspNetCore.NodeServices/TypeScript/Util/AutoQuit.ts b/src/Microsoft.AspNetCore.NodeServices/TypeScript/Util/AutoQuit.ts
@@ -8,7 +8,11 @@ export function autoQuitOnFileChange(rootDir: string, extensions: string[]) {
         var ext = path.extname(filename);
         if (extensions.indexOf(ext) >= 0) {
             console.log('Restarting due to file change: ' + filename);
-            process.exit(0);
+
+            // Temporarily, the file-watching logic is in Node, so we signal it's time to restart by
+            // sending the following message back to .NET. Soon the file-watching logic will move over
+            // to the .NET side, and this whole file can be removed.
+            console.log('[Microsoft.AspNetCore.NodeServices:Restart]');
         }
     });
 }

Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,10 @@`
`184`	`184`	`var ext = path.extname(filename);`
`185`	`185`	`if (extensions.indexOf(ext) >= 0) {`
`186`	`186`	`console.log('Restarting due to file change: ' + filename);`
`187`		`- process.exit(0);`
	`187`	`+ // Temporarily, the file-watching logic is in Node, so we signal it's time to restart by`
	`188`	`+ // sending the following message back to .NET. Soon the file-watching logic will move over`
	`189`	`+ // to the .NET side, and this whole file can be removed.`
	`190`	`+ console.log('[Microsoft.AspNetCore.NodeServices:Restart]');`
`188`	`191`	`}`
`189`	`192`	`});`
`190`	`193`	`}`
Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,10 @@`
`95`	`95`	`var ext = path.extname(filename);`
`96`	`96`	`if (extensions.indexOf(ext) >= 0) {`
`97`	`97`	`console.log('Restarting due to file change: ' + filename);`
`98`		`- process.exit(0);`
	`98`	`+ // Temporarily, the file-watching logic is in Node, so we signal it's time to restart by`
	`99`	`+ // sending the following message back to .NET. Soon the file-watching logic will move over`
	`100`	`+ // to the .NET side, and this whole file can be removed.`
	`101`	`+ console.log('[Microsoft.AspNetCore.NodeServices:Restart]');`
`99`	`102`	`}`
`100`	`103`	`});`
`101`	`104`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,11 @@ export function autoQuitOnFileChange(rootDir: string, extensions: string[]) {`
`8`	`8`	`var ext = path.extname(filename);`
`9`	`9`	`if (extensions.indexOf(ext) >= 0) {`
`10`	`10`	`console.log('Restarting due to file change: ' + filename);`
`11`		`- process.exit(0);`
	`11`	`+`
	`12`	`+ // Temporarily, the file-watching logic is in Node, so we signal it's time to restart by`
	`13`	`+ // sending the following message back to .NET. Soon the file-watching logic will move over`
	`14`	`+ // to the .NET side, and this whole file can be removed.`
	`15`	`+ console.log('[Microsoft.AspNetCore.NodeServices:Restart]');`
`12`	`16`	`}`
`13`	`17`	`});`
`14`	`18`	`}`