From 455982d41c59e4aa35ac7991d38374d9d1de7eb1 Mon Sep 17 00:00:00 2001
From: Jean Lauliac <jeanlauliac@fb.com>
Date: Mon, 9 Jan 2017 10:10:09 -0800
Subject: [PATCH] packager: create the reporter higher in the stack

Reviewed By: davidaurelio

Differential Revision: D4392283

fbshipit-source-id: 4cd470ca0cbddcbb515407b5249272a758849b82
---
 README.md                        |  2 ++
 react-packager/react-packager.js | 18 +++++++++++-------
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/README.md b/README.md
index 0493d955..698b7806 100644
--- a/README.md
+++ b/README.md
@@ -112,6 +112,8 @@ Builds a bundle according to the provided options.
   itself
 * `getTransformOptions` function: A function that acts as a middleware for
   generating options to pass to the transformer based on the bundle being built.
+* `reporter` object (required): An object with a single function `update` that
+  is called when events are happening: build updates, warnings, errors.
 
 #### `bundleOptions`
 
diff --git a/react-packager/react-packager.js b/react-packager/react-packager.js
index 339adfce..9291c540 100644
--- a/react-packager/react-packager.js
+++ b/react-packager/react-packager.js
@@ -14,6 +14,7 @@
 const Logger = require('./src/Logger');
 
 const debug = require('debug');
+const invariant = require('invariant');
 
 import type {Reporter} from './src/lib/reporting';
 
@@ -27,6 +28,13 @@ type Options = {
   watch?: boolean,
 };
 
+type StrictOptions = {
+  nonPersistent: boolean,
+  projectRoots: Array<string>,
+  reporter: Reporter,
+  watch?: boolean,
+};
+
 exports.buildBundle = function(options: Options, bundleOptions: {}) {
   var server = createNonPersistentServer(options);
   return server.buildBundle(bundleOptions)
@@ -59,21 +67,17 @@ function enableDebug() {
   debug.enable(debugPattern);
 }
 
-function createServer(options: Options) {
+function createServer(options: StrictOptions) {
   // the debug module is configured globally, we need to enable debugging
   // *before* requiring any packages that use `debug` for logging
   if (options.verbose) {
     enableDebug();
   }
 
+  // Some callsites may not be Flowified yet.
+  invariant(options.reporter != null, 'createServer() requires reporter');
   const serverOptions = Object.assign({}, options);
   delete serverOptions.verbose;
-  if (serverOptions.reporter == null) {
-    // It's unsound to set-up the reporter here, but this allows backward
-    // compatibility.
-    var TerminalReporter = require('./src/lib/TerminalReporter');
-    serverOptions.reporter = new TerminalReporter();
-  }
   var Server = require('./src/Server');
   return new Server(serverOptions);
 }