Create helper to setup exit handlers

Author: Kjir

packages/react-dev-utils/README.md

@@ -105,6 +105,19 @@ module.exports = {
 }
 ```
 
+#### `addProcessExitHandlers(): void`
+
+Exits the process when stdin is closed by an external program.<br>
+This is useful when the dev server is started by some other tool (i.e.: Phoenix)
+which then uses the standard Unix convention of closing the standard input to
+signal the end of the program.
+
+```js
+var addProcessExitHandlers = require('react-dev-utils/addProcessExitHandlers');
+
+addProcessExitHandlers();
+```
+
 #### `checkRequiredFiles(files: Array<string>): boolean`
 
 Makes sure that all passed files exist.<br>

packages/react-dev-utils/addProcessExitHandlers.js

@@ -0,0 +1,22 @@
+/**
+ * Copyright (c) 2015-present, Facebook, Inc.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+'use strict';
+
+function addProcessExitHandlers() {
+  // Issue: https://github.com/facebookincubator/create-react-app/issues/1753
+  // The below lines are added to make sure that this process is
+  // exited when stdin is ended. The consequence of not doing this means
+  // that all watch processes will stay running despite the process that spawned
+  // them being closed.
+  process.stdin.on('end', function() {
+    process.exit(0);
+  });
+  process.stdin.resume();
+}
+
+module.exports = addProcessExitHandlers;

packages/react-dev-utils/package.json

@@ -11,6 +11,7 @@
     "node": ">=6"
   },
   "files": [
+    "addProcessExitHandlers.js",
     "checkRequiredFiles.js",
     "clearConsole.js",
     "crashOverlay.js",

packages/react-scripts/scripts/start.js

@@ -38,6 +38,7 @@ const openBrowser = require('react-dev-utils/openBrowser');
 const paths = require('../config/paths');
 const config = require('../config/webpack.config.dev');
 const createDevServerConfig = require('../config/webpackDevServer.config');
+const addProcessExitHandlers = require('react-dev-utils/addProcessExitHandlers');
 
 const useYarn = fs.existsSync(paths.yarnLockFile);
 const isInteractive = process.stdout.isTTY;
@@ -47,15 +48,7 @@ if (!checkRequiredFiles([paths.appHtml, paths.appIndexJs])) {
   process.exit(1);
 }
 
-// Issue: https://github.com/facebookincubator/create-react-app/issues/1753
-// The below lines are added to make sure that this process is
-// exited when stdin is ended. The consequence of not doing this means
-// that all watch processes will stay running despite the process that spawned
-// them being closed.
-process.stdin.on('end', function() {
-  process.exit(0);
-});
-process.stdin.resume();
+addProcessExitHandlers();
 
 // Tools like Cloud9 rely on this.
 const DEFAULT_PORT = parseInt(process.env.PORT, 10) || 3000;

packages/react-scripts/scripts/test.js

@@ -23,6 +23,7 @@ process.on('unhandledRejection', err => {
 // Ensure environment variables are read.
 require('../config/env');
 
+const addProcessExitHandlers = require('react-dev-utils/addProcessExitHandlers');
 const jest = require('jest');
 const argv = process.argv.slice(2);
 
@@ -31,16 +32,7 @@ const argv = process.argv.slice(2);
 if (!process.env.CI && argv.indexOf('--coverage') < 0) {
   argv.push('--watch');
 
-  // Issue: https://github.com/facebookincubator/create-react-app/issues/1753
-  // The below lines are added to make sure that this process is
-  // exited when stdin is ended. The consequence of not doing this means
-  // that all watch processes will stay running despite the process that spawned
-  // them being closed.
-
-  process.stdin.on('end', function() {
-    process.exit(0);
-  });
-  process.stdin.resume();
+  addProcessExitHandlers();
 }
 
 // @remove-on-eject-begin