Revert "Replace projectRoots with projectRoot + watchRoots"

This reverts commit c5ce762.

Author: hramos

local-cli/bundle/buildBundle.js

@@ -75,6 +75,7 @@ async function buildBundle(
       : defaultProvidesModuleNodeModules;
 
   const terminal = new Terminal(process.stdout);
+
   const server = new Server({
     asyncRequireModulePath: config.getAsyncRequireModulePath(),
     assetExts: defaultAssetExts.concat(assetExts),
@@ -95,15 +96,14 @@ async function buildBundle(
     platforms: defaultPlatforms.concat(platforms),
     postMinifyProcess: config.postMinifyProcess,
     postProcessBundleSourcemap: config.postProcessBundleSourcemap,
-    projectRoot: config.getProjectRoot(),
+    projectRoots: config.getProjectRoots(),
     providesModuleNodeModules: providesModuleNodeModules,
     reporter: new TerminalReporter(terminal),
     resetCache: args.resetCache,
     resolveRequest: config.resolveRequest,
     sourceExts: sourceExts.concat(defaultSourceExts),
     transformModulePath: transformModulePath,
     watch: false,
-    watchFolders: config.getWatchFolders(),
     workerPath: config.getWorkerPath && config.getWorkerPath(),
   });
 

local-cli/dependencies/dependencies.js

@@ -34,7 +34,7 @@ function dependencies(argv, config, args, packagerInstance) {
   const packageOpts = {
     assetRegistryPath: ASSET_REGISTRY_PATH,
     cacheStores: [],
-    projectRoot: config.getProjectRoot(),
+    projectRoots: config.getProjectRoots(),
     blacklistRE: config.getBlacklistRE(),
     dynamicDepsInPackages: config.dynamicDepsInPackages,
     getPolyfills: config.getPolyfills,
@@ -44,14 +44,12 @@ function dependencies(argv, config, args, packagerInstance) {
     transformModulePath: transformModulePath,
     extraNodeModules: config.extraNodeModules,
     verbose: config.verbose,
-    watchFolders: config.getWatchFolders(),
     workerPath: config.getWorkerPath(),
   };
 
-  const relativePath = path.relative(
-    packageOpts.projectRoot,
-    rootModuleAbsolutePath,
-  );
+  const relativePath = packageOpts.projectRoots.map(root =>
+    path.relative(root, rootModuleAbsolutePath),
+  )[0];
 
   const options = {
     platform: args.platform,
@@ -77,7 +75,7 @@ function dependencies(argv, config, args, packagerInstance) {
         // (a) JS code to not depend on anything outside this directory, or
         // (b) Come up with a way to declare this dependency in Buck.
         const isInsideProjectRoots =
-          packageOpts.watchFolders.filter(root => modulePath.startsWith(root))
+          packageOpts.projectRoots.filter(root => modulePath.startsWith(root))
             .length > 0;
 
         if (isInsideProjectRoots) {

local-cli/server/middleware/getDevToolsMiddleware.js

@@ -23,11 +23,11 @@ function escapePath(pathname) {
   return '"' + pathname + '"';
 }
 
-function launchDevTools({host, watchFolders}, isChromeConnected) {
+function launchDevTools({host, projectRoots}, isChromeConnected) {
   // Explicit config always wins
   var customDebugger = process.env.REACT_DEBUGGER;
   if (customDebugger) {
-    var projects = watchFolders.map(escapePath).join(' ');
+    var projects = projectRoots.map(escapePath).join(' ');
     var command = customDebugger + ' ' + projects;
     console.log('Starting custom debugger by executing: ' + command);
     exec(command, function(error, stdout, stderr) {

local-cli/server/middleware/openStackFrameInEditorMiddleware.js

@@ -11,11 +11,11 @@
 
 const launchEditor = require('../util/launchEditor');
 
-module.exports = function({watchFolders}) {
+module.exports = function({projectRoots}) {
   return function(req, res, next) {
     if (req.url === '/open-stack-frame') {
       const frame = JSON.parse(req.rawBody);
-      launchEditor(frame.file, frame.lineNumber, watchFolders);
+      launchEditor(frame.file, frame.lineNumber, projectRoots);
       res.end('OK');
     } else {
       next();

local-cli/server/runServer.js

@@ -60,7 +60,6 @@ export type Args = {|
   +resetCache: boolean,
   +sourceExts: $ReadOnlyArray<string>,
   +verbose: boolean,
-  +watchFolders: $ReadOnlyArray<string>,
 |};
 
 function runServer(
@@ -101,7 +100,7 @@ function runServer(
     .use(indexPageMiddleware)
     .use(packagerServer.processRequest.bind(packagerServer));
 
-  args.watchFolders.forEach(root => app.use(serveStatic(root)));
+  args.projectRoots.forEach(root => app.use(serveStatic(root)));
 
   app.use(morgan('combined')).use(errorhandler());
 
@@ -197,7 +196,7 @@ function getPackagerServer(args, config, reporter) {
     polyfillModuleNames: config.getPolyfillModuleNames(),
     postMinifyProcess: config.postMinifyProcess,
     postProcessBundleSourcemap: config.postProcessBundleSourcemap,
-    projectRoot: config.getProjectRoot(),
+    projectRoots: args.projectRoots,
     providesModuleNodeModules: providesModuleNodeModules,
     reporter,
     resetCache: args.resetCache,
@@ -206,7 +205,6 @@ function getPackagerServer(args, config, reporter) {
     transformModulePath: transformModulePath,
     verbose: args.verbose,
     watch: !args.nonPersistent,
-    watchFolders: config.getWatchFolders(),
     workerPath: config.getWorkerPath(),
   });
 }

local-cli/server/server.js

@@ -75,13 +75,10 @@ module.exports = {
       default: [],
     },
     {
-      command: '--watchFolders [list]',
-      description:
-        'Sepcify any additional folders to be added to the watch list',
+      command: '--projectRoots [list]',
+      description: 'override the root(s) to be used by the packager',
       parse: (val: string) => val.split(','),
-      default: (config: ConfigT) => {
-        return config.getProjectRoots ? config.getProjectRoots() : undefined;
-      },
+      default: (config: ConfigT) => config.getProjectRoots(),
     },
     {
       command: '--assetExts [list]',