[cli] Fix vc link not respecting --yes (vercel#8412)

As of 28.0.0, `vc link` stopped respecting `--yes` and would wait for prompt if it found a Git repository to connect. This PR passes `autoConfirm` through to the Git prompt, so that `vc link` continues to respect `--yes`.

Here's what `vc link` looks like after this PR if a user has a local Git repository:

<img width="513" alt="Screen Shot 2022-08-16 at 10 05 56 AM" src="https://user-images.githubusercontent.com/14811170/184899917-c3dc0603-370d-4c86-afb8-19758fbc051a.png">

Not sure if we want to remove the log messages to be consistent with the rest of `vc link`?

### 📋 Checklist

<!--
  Please keep your PR as a Draft until the checklist is complete
-->

#### Tests

- [x] The code changed/added as part of this PR has been covered with tests
- [x] All tests pass locally with `yarn test-unit`

#### Code Review

- [ ] This PR has a concise title and thorough description useful to a reviewer
- [ ] Issue from task tracker has a link to this PR

Author: MatthewStanciu

packages/cli/src/util/link/add-git-connection.ts

@@ -28,6 +28,7 @@ export async function addGitConnection(
   org: Org,
   project: Project,
   remoteUrls: Dictionary<string>,
+  autoConfirm: Boolean,
   settings?: ProjectSettings
 ): Promise<number | void> {
   if (!settings) {
@@ -39,15 +40,17 @@ export async function addGitConnection(
       org,
       project,
       remoteUrls,
-      settings || project
+      settings || project,
+      autoConfirm
     );
   } else if (Object.keys(remoteUrls).length > 1 && !project.link) {
     return addMultipleGitRemotes(
       client,
       org,
       project,
       remoteUrls,
-      settings || project
+      settings || project,
+      autoConfirm
     );
   }
 }
@@ -57,7 +60,8 @@ async function addSingleGitRemote(
   org: Org,
   project: Project,
   remoteUrls: Dictionary<string>,
-  settings: ProjectSettings
+  settings: ProjectSettings,
+  autoConfirm: Boolean
 ) {
   const [remoteName, remoteUrl] = Object.entries(remoteUrls)[0];
   const repoInfo = parseRepoUrl(remoteUrl);
@@ -81,27 +85,43 @@ async function addSingleGitRemote(
     (project.link.org !== parsedOrg ||
       project.link.repo !== repo ||
       project.link.type !== provider);
-  const shouldConnect = await promptGitConnectSingleUrl(
+
+  let shouldConnectOption: string | undefined;
+  if (autoConfirm) {
+    shouldConnectOption = 'yes';
+  } else {
+    shouldConnectOption = await promptGitConnectSingleUrl(
+      client,
+      project,
+      remoteName,
+      remoteUrl,
+      replace
+    );
+  }
+  return handleOptions(
+    shouldConnectOption,
     client,
+    org,
     project,
-    remoteName,
-    remoteUrl,
-    replace
+    settings,
+    repoInfo
   );
-  return handleOptions(shouldConnect, client, org, project, settings, repoInfo);
 }
 
 async function addMultipleGitRemotes(
   client: Client,
   org: Org,
   project: Project,
   remoteUrls: Dictionary<string>,
-  settings: ProjectSettings
+  settings: ProjectSettings,
+  autoConfirm: Boolean
 ) {
-  client.output.log('Found multiple Git remote URLs in Git config.');
-  const remoteUrlOrOptions = await promptGitConnectMultipleUrls(
-    client,
-    remoteUrls
-  );
-  return handleOptions(remoteUrlOrOptions, client, org, project, settings);
+  let remoteUrl: string | undefined;
+  if (autoConfirm) {
+    remoteUrl = remoteUrls['origin'] || Object.values(remoteUrls)[0];
+  } else {
+    client.output.log('Found multiple Git remote URLs in Git config.');
+    remoteUrl = await promptGitConnectMultipleUrls(client, remoteUrls);
+  }
+  return handleOptions(remoteUrl, client, org, project, settings);
 }

packages/cli/src/util/link/setup-and-link.ts

@@ -136,7 +136,8 @@ export default async function setupAndLink(
         client,
         org,
         project,
-        remoteUrls
+        remoteUrls,
+        autoConfirm
       );
       if (typeof connectGit === 'number') {
         return { status: 'error', exitCode: connectGit };
@@ -264,6 +265,7 @@ export default async function setupAndLink(
         org,
         project,
         remoteUrls,
+        autoConfirm,
         settings
       );
       if (typeof connectGit === 'number') {

packages/cli/test/fixtures/unit/link-connect-git/multiple-remotes-prefer-origin/.gitignore

@@ -0,0 +1,2 @@
+!.vercel
+.vercel

packages/cli/test/fixtures/unit/link-connect-git/multiple-remotes-prefer-origin/git/config

@@ -306,5 +306,52 @@ describe('link', () => {
         process.chdir(originalCwd);
       }
     });
+    it('should respect --yes', async () => {
+      const cwd = fixture('single-remote');
+      try {
+        process.chdir(cwd);
+        await fs.rename(join(cwd, 'git'), join(cwd, '.git'));
+
+        useUser();
+        useProject({
+          ...defaultProject,
+          name: 'single-remote',
+          id: 'single-remote',
+        });
+        useTeams('team_dummy');
+        client.setArgv('--yes');
+        const linkPromise = link(client);
+        expect(client.stderr).not.toOutput('Do you want to connect "origin"');
+        await expect(client.stderr).toOutput('Linked to');
+        await expect(linkPromise).resolves.toEqual(0);
+      } finally {
+        await fs.rename(join(cwd, '.git'), join(cwd, 'git'));
+        process.chdir(originalCwd);
+      }
+    });
+    it('should respect --yes for multiple remotes when origin is not the first', async () => {
+      const cwd = fixture('multiple-remotes-prefer-origin');
+      try {
+        process.chdir(cwd);
+        await fs.rename(join(cwd, 'git'), join(cwd, '.git'));
+        useUser();
+        useProject({
+          ...defaultProject,
+          name: 'multiple-remotes-prefer-origin',
+          id: 'multiple-remotes-prefer-origin',
+        });
+        useTeams('team_dummy');
+        client.setArgv('--yes');
+        const linkPromise = link(client);
+        expect(client.stderr).not.toOutput('Found multiple Git remote URLs');
+        await expect(client.stderr).toOutput(
+          'Connected GitHub repository user/repo'
+        );
+        await expect(linkPromise).resolves.toEqual(0);
+      } finally {
+        await fs.rename(join(cwd, '.git'), join(cwd, 'git'));
+        process.chdir(originalCwd);
+      }
+    });
   });
 });