fix: support specifying --pm yarn and --skip-install together

Author: azizbecha

.gitignore

@@ -13,3 +13,4 @@ build/
 coverage
 *.env
 .parcel-cache
+.yarn/

__e2e__/init.test.ts

@@ -1,5 +1,6 @@
 import fs from 'fs';
 import path from 'path';
+import execa from 'execa';
 import {runCLI, getTempDirectory, cleanup, writeFiles} from '../jest/helpers';
 import slash from 'slash';
 
@@ -43,6 +44,15 @@ if (process.platform === 'win32') {
   templatePath = `file://${templatePath}`;
 }
 
+function isYarnAvailable() {
+  try {
+    execa.sync('yarn', ['--version'], {stdio: 'pipe'});
+    return true;
+  } catch (error) {
+    return false;
+  }
+}
+
 test('init fails if the directory already exists and --replace-directory false', () => {
   fs.mkdirSync(path.join(DIR, PROJECT_NAME));
 
@@ -152,6 +162,34 @@ test('init skips installation of dependencies with --skip-install', () => {
   );
 });
 
+test('init supports --pm yarn together with --skip-install', () => {
+  if (!isYarnAvailable()) {
+    return;
+  }
+
+  createCustomTemplateFiles();
+
+  const {stdout, stderr} = runCLI(DIR, [
+    'init',
+    '--template',
+    templatePath,
+    PROJECT_NAME,
+    '--pm',
+    'yarn',
+    '--skip-install',
+  ]);
+
+  expect(stderr).not.toContain(`Couldn't find the "`);
+  expect(stdout).toContain('Run instructions');
+
+  const dirFiles = fs.readdirSync(path.join(DIR, PROJECT_NAME)).sort();
+  const expectedFiles = customTemplateCopiedFiles
+    .filter((file) => !['node_modules', 'package-lock.json'].includes(file))
+    .concat(['.yarn', '.yarnrc.yml'])
+    .sort();
+  expect(dirFiles).toEqual(expectedFiles);
+});
+
 // react-native-macos stopped shipping `template.config.js` for 0.75, so this test is disabled. in future releases we should re-enable once `template.config.js` will be there again.
 test.skip('init --platform-name should work for out of tree platform', () => {
   createCustomTemplateFiles();

package.json

@@ -18,7 +18,7 @@
     "link-packages": "node ./scripts/linkPackages.js",
     "publish": "yarn build-clean-all && yarn install && lerna publish --force-publish",
     "publish:next": "yarn build-clean-all && yarn install && lerna publish --force-publish --dist-tag next",
-    "prepare": "husky install"
+    "prepare": "husky"
   },
   "devDependencies": {
     "@babel/core": "^7.0.0",

packages/cli/src/commands/init/init.ts

@@ -281,12 +281,12 @@ async function createFromTemplate({
 
       if (process.platform === 'darwin') {
         const installPodsValue = String(installCocoaPods);
-        const reactNativePath = path.dirname(
-          require.resolve('react-native', {paths: [projectDirectory]}),
-        );
 
         try {
           if (installPodsValue === 'true') {
+            const reactNativePath = path.dirname(
+              require.resolve('react-native', {paths: [projectDirectory]}),
+            );
             didInstallPods = true;
             await runCodegen({
               root: projectDirectory,
@@ -308,6 +308,9 @@ async function createFromTemplate({
             didInstallPods = installCocoapods;
 
             if (installCocoapods) {
+              const reactNativePath = path.dirname(
+                require.resolve('react-native', {paths: [projectDirectory]}),
+              );
               await runCodegen({
                 root: projectDirectory,
                 platform: 'ios',
@@ -321,7 +324,7 @@ async function createFromTemplate({
           }
         } catch (error) {
           logger.error(
-            `\nInstalling Cocoapods failed. This doesn't affect project initialization and you can safely proceed. However, you will need to install Cocoapods manually when running iOS, follow additional steps in "Run instructions for iOS" section.\n`,
+            '\nInstalling Cocoapods failed. This doesn\'t affect project initialization and you can safely proceed. However, you will need to install Cocoapods manually when running iOS, follow additional steps in "Run instructions for iOS" section.\n',
           );
           logger.error((error as Error).message as string);
         }

packages/cli/src/commands/init/template.ts

@@ -37,22 +37,24 @@ export async function installTemplatePackage(
     };
 
     // React Native doesn't support PnP, so we need to set nodeLinker to node-modules. Read more here: https://github.com/react-native-community/cli/issues/27#issuecomment-1772626767
-    executeCommand(
+    await executeCommand(
       'yarn',
       ['config', 'set', 'nodeLinker', 'node-modules'],
       options,
     );
 
-    executeCommand(
+    await executeCommand(
       'yarn',
       ['config', 'set', 'nmHoistingLimits', 'workspaces'],
       options,
     );
 
-    for (let key in yarnConfigOptions) {
-      if (yarnConfigOptions.hasOwnProperty(key)) {
-        let value = yarnConfigOptions[key];
-        executeCommand('yarn', ['config', 'set', key, value], options);
+    if (yarnConfigOptions) {
+      for (let key in yarnConfigOptions) {
+        if (Object.prototype.hasOwnProperty.call(yarnConfigOptions, key)) {
+          const value = yarnConfigOptions[key];
+          await executeCommand('yarn', ['config', 'set', key, value], options);
+        }
       }
     }
   }