feat(plugin-git): make git: protocol default to HEAD (#2557)

* feat(plugin-git): make git: protocol default to HEAD

* test: update snapshot

Author: paul-soporan

.yarn/versions/a278601e.yml

@@ -0,0 +1,24 @@
+releases:
+  "@yarnpkg/cli": minor
+  "@yarnpkg/plugin-git": minor
+  "@yarnpkg/plugin-github": minor
+
+declined:
+  - "@yarnpkg/plugin-compat"
+  - "@yarnpkg/plugin-constraints"
+  - "@yarnpkg/plugin-dlx"
+  - "@yarnpkg/plugin-essentials"
+  - "@yarnpkg/plugin-init"
+  - "@yarnpkg/plugin-interactive-tools"
+  - "@yarnpkg/plugin-node-modules"
+  - "@yarnpkg/plugin-npm-cli"
+  - "@yarnpkg/plugin-pack"
+  - "@yarnpkg/plugin-patch"
+  - "@yarnpkg/plugin-pnp"
+  - "@yarnpkg/plugin-stage"
+  - "@yarnpkg/plugin-typescript"
+  - "@yarnpkg/plugin-version"
+  - "@yarnpkg/plugin-workspace-tools"
+  - "@yarnpkg/builder"
+  - "@yarnpkg/core"
+  - "@yarnpkg/doctor"

packages/plugin-git/sources/__snapshots__/gitUtils.test.js.snap

@@ -94,7 +94,7 @@ Object {
   "repo": "https://github.com/GitHubOrg/foo-bar.js.git",
   "treeish": Object {
     "protocol": "head",
-    "request": "master",
+    "request": "HEAD",
   },
 }
 `;
@@ -164,7 +164,7 @@ Object {
   "repo": "https://github.com/GitHubOrg/foo-bar.js.git",
   "treeish": Object {
     "protocol": "head",
-    "request": "master",
+    "request": "HEAD",
   },
 }
 `;
@@ -175,7 +175,7 @@ Object {
   "repo": "https://github.com/GitHubOrg/foo2bar.js.git",
   "treeish": Object {
     "protocol": "head",
-    "request": "master",
+    "request": "HEAD",
   },
 }
 `;
@@ -241,7 +241,7 @@ Object {
   "repo": "https://github.com/GitHubOrg/foo-bar.js.git",
   "treeish": Object {
     "protocol": "head",
-    "request": "master",
+    "request": "HEAD",
   },
 }
 `;

packages/plugin-git/sources/gitUtils.ts

@@ -60,7 +60,7 @@ export function splitRepoUrl(url: string): RepoUrlParts {
       repo: url,
       treeish: {
         protocol: TreeishProtocols.Head,
-        request: `master`,
+        request: `HEAD`,
       },
       extra: {},
     };
@@ -89,7 +89,7 @@ export function splitRepoUrl(url: string): RepoUrlParts {
       request = extra[requestedProtocol]! as string;
     } else {
       protocol = TreeishProtocols.Head;
-      request = `master`;
+      request = `HEAD`;
     }
 
     for (const key of Object.values(TreeishProtocols))
@@ -176,7 +176,7 @@ export async function lsRemote(repo: string, configuration: Configuration) {
 
   let res: {stdout: string};
   try {
-    res = await execUtils.execvp(`git`, [`ls-remote`, `--refs`, normalizedRepoUrl], {
+    res = await execUtils.execvp(`git`, [`ls-remote`, normalizedRepoUrl], {
       cwd: configuration.startingCwd,
       env: makeGitEnvironment(),
       strict: true,
@@ -188,7 +188,7 @@ export async function lsRemote(repo: string, configuration: Configuration) {
 
   const refs = new Map();
 
-  const matcher = /^([a-f0-9]{40})\t(refs\/[^\n]+)/gm;
+  const matcher = /^([a-f0-9]{40})\t([^\n]+)/gm;
   let match;
 
   while ((match = matcher.exec(res.stdout)) !== null)
@@ -214,7 +214,7 @@ export async function resolveUrl(url: string, configuration: Configuration) {
       }
 
       case TreeishProtocols.Head: {
-        const head = refs.get(`refs/heads/${request}`);
+        const head = refs.get(request === `HEAD` ? request : `refs/heads/${request}`);
         if (typeof head === `undefined`)
           throw new Error(`Unknown head ("${request}")`);