Remove unused `packagerInstance` option and rename it to `server`

Reviewed By: rafeca

Differential Revision: D6435505

fbshipit-source-id: e1912f20be517be49648b0eeb52c53f66e3fae55

local-cli/bundle/buildBundle.js

@@ -57,7 +57,6 @@ async function buildBundle(
   },
   config: ConfigT,
   output = outputBundle,
-  packagerInstance,
 ) {
   // This is used by a bazillion of npm modules we don't control so we don't
   // have other choice than defining it as an env variable here.
@@ -76,10 +75,6 @@ async function buildBundle(
     platform: args.platform,
   };
 
-  // If a packager instance was not provided, then just create one for this
-  // bundle command and close it down afterwards.
-  var shouldClosePackager = false;
-  if (!packagerInstance) {
   const assetExts = (config.getAssetExts && config.getAssetExts()) || [];
   const sourceExts = (config.getSourceExts && config.getSourceExts()) || [];
   const platforms = (config.getPlatforms && config.getPlatforms()) || [];
@@ -94,7 +89,8 @@ async function buildBundle(
       : defaultProvidesModuleNodeModules;
 
   const terminal = new Terminal(process.stdout);
-    const options = {
+
+  const server = new Server({
     assetExts: defaultAssetExts.concat(assetExts),
     assetRegistryPath: ASSET_REGISTRY_PATH,
     blacklistRE: config.getBlacklistRE(),
@@ -118,18 +114,14 @@ async function buildBundle(
     transformModulePath: transformModulePath,
     watch: false,
     workerPath: config.getWorkerPath && config.getWorkerPath(),
-    };
+  });
 
-    packagerInstance = new Server(options);
-    shouldClosePackager = true;
-  }
-
-  const bundle = await output.build(packagerInstance, requestOpts);
+  const bundle = await output.build(server, requestOpts);
 
   await output.save(bundle, args, log);
 
   // Save the assets of the bundle
-  const outputAssets = await packagerInstance.getAssets({
+  const outputAssets = await server.getAssets({
     ...Server.DEFAULT_BUNDLE_OPTIONS,
     ...requestOpts,
     bundleType: 'todo',
@@ -142,9 +134,7 @@ async function buildBundle(
     args.assetsDest,
   );
 
-  if (shouldClosePackager) {
-    packagerInstance.end();
-  }
+  server.end();
 
   return assets;
 }

local-cli/bundle/bundle.js

@@ -15,21 +15,17 @@ const outputBundle = require('metro/src/shared/output/bundle');
 /**
  * Builds the bundle starting to look for dependencies at the given entry path.
  */
-function bundleWithOutput(argv, config, args, output, packagerInstance) {
+function bundleWithOutput(argv, config, args, output) {
   if (!output) {
     output = outputBundle;
   }
-  return buildBundle(args, config, output, packagerInstance);
-}
-
-function bundle(argv, config, args, packagerInstance) {
-  return bundleWithOutput(argv, config, args, undefined, packagerInstance);
+  return buildBundle(args, config, output);
 }
 
 module.exports = {
   name: 'bundle',
   description: 'builds the javascript bundle for offline use',
-  func: bundle,
+  func: bundleWithOutput,
   options: bundleCommandLineArgs,
 
   // not used by the CLI itself

local-cli/bundle/unbundle.js

@@ -15,8 +15,8 @@ const outputUnbundle = require('metro/src/shared/output/unbundle');
 /**
  * Builds the bundle starting to look for dependencies at the given entry path.
  */
-function unbundle(argv, config, args, packagerInstance) {
-  return bundleWithOutput(argv, config, args, outputUnbundle, packagerInstance);
+function unbundle(argv, config, args) {
+  return bundleWithOutput(argv, config, args, outputUnbundle);
 }
 
 module.exports = {