feat(doctor): Try installing packages with Homebrew (react-native-com…

…munity#671)
Rayberk · Sep 4, 2019 · 722c08c · 722c08c
1 parent a97c5ab
commit 722c08c
Show file tree

Hide file tree

Showing 4 changed files with 52 additions and 7 deletions.
diff --git a/packages/cli/src/commands/doctor/healthchecks/packageManagers.js b/packages/cli/src/commands/doctor/healthchecks/packageManagers.js
@@ -4,8 +4,9 @@ import {
   PACKAGE_MANAGERS,
   doesSoftwareNeedToBeFixed,
 } from '../checkInstallation';
+import {install} from '../../../tools/install';
 
-const identifyPackageManager = () => {
+const packageManager = (() => {
   if (fs.existsSync('yarn.lock')) {
     return PACKAGE_MANAGERS.YARN;
   }
@@ -15,9 +16,7 @@ const identifyPackageManager = () => {
   }
 
   return undefined;
-};
-
-const packageManager = identifyPackageManager();
+})();
 
 const yarn = {
   label: 'yarn',
@@ -32,7 +31,8 @@ const yarn = {
   // or if we can't identify that the user uses yarn or npm
   visible:
     packageManager === PACKAGE_MANAGERS.YARN || packageManager === undefined,
-  runAutomaticFix: () => console.log('should fix node'),
+  runAutomaticFix: async ({loader}) =>
+    await install('yarn', 'https://yarnpkg.com/docs/install', loader),
 };
 
 const npm = {
@@ -47,7 +47,8 @@ const npm = {
   // or if we can't identify that the user uses yarn or npm
   visible:
     packageManager === PACKAGE_MANAGERS.NPM || packageManager === undefined,
-  runAutomaticFix: () => console.log('should fix node'),
+  runAutomaticFix: async ({loader}) =>
+    await install('node', 'https://nodejs.org/', loader),
 };
 
 export {packageManager, yarn, npm};
diff --git a/packages/cli/src/commands/doctor/healthchecks/watchman.js b/packages/cli/src/commands/doctor/healthchecks/watchman.js
@@ -1,5 +1,6 @@
 import versionRanges from '../versionRanges';
 import {doesSoftwareNeedToBeFixed} from '../checkInstallation';
+import {install} from '../../../tools/install';
 
 export default {
   label: 'Watchman',
@@ -9,5 +10,10 @@ export default {
       versionRange: versionRanges.WATCHMAN,
     }),
   }),
-  runAutomaticFix: () => console.log('should fix watchman'),
+  runAutomaticFix: async ({loader}) =>
+    await install(
+      'watchman',
+      'https://facebook.github.io/watchman/docs/install.html',
+      loader,
+    ),
 };
diff --git a/packages/cli/src/tools/brewInstall.js b/packages/cli/src/tools/brewInstall.js
@@ -0,0 +1,19 @@
+// @flow
+import {logger} from '@react-native-community/cli-tools';
+import execa from 'execa';
+import Ora from 'ora';
+
+async function brewInstall(pkg: string, loader: typeof Ora) {
+  loader.start(`Installing ${pkg}`);
+  try {
+    await execa('brew', ['install', pkg]);
+    loader.succeed();
+  } catch (error) {
+    logger.log(error.stderr);
+    loader.fail(
+      `An error occured while trying to install ${pkg}. Please try again manually: brew install ${pkg}`,
+    );
+  }
+}
+
+export {brewInstall};
diff --git a/packages/cli/src/tools/install.js b/packages/cli/src/tools/install.js
@@ -0,0 +1,19 @@
+// @flow
+import Ora from 'ora';
+import {brewInstall} from './brewInstall';
+
+async function install(pkg: string, source: string, loader: typeof Ora) {
+  try {
+    switch (process.platform) {
+      case 'darwin':
+        await brewInstall(pkg, loader);
+        break;
+      default:
+        throw new Error('Not implemented yet');
+    }
+  } catch (_error) {
+    loader.info(`Please download and install '${pkg}' from ${source}.`);
+  }
+}
+
+export {install};