Fix support for private GitHub shortcuts deps. Fix #573

__tests__/package-resolver.js

@@ -12,6 +12,7 @@ import * as fs from '../src/util/fs.js';
 jasmine.DEFAULT_TIMEOUT_INTERVAL = 60000;
 
 const path = require('path');
+const isCI = require('is-ci');
 
 function addTest(pattern, registry = 'npm') {
   // TODO renable these test.concurrent
@@ -35,6 +36,7 @@ function addTest(pattern, registry = 'npm') {
   });
 }
 
+// Public deps
 addTest('github:PolymerElements/paper-elements', 'bower'); // github url
 addTest('https://github.com/PolymerElements/font-roboto.git', 'bower'); // hosted git url
 addTest('https://github.com/npm-ml/re'); // git url with no .git
@@ -53,3 +55,12 @@ addTest('react-native'); // npm
 addTest('ember-cli'); // npm
 addTest('npm:gulp'); // npm
 addTest('@polymer/iron-icon'); // npm scoped package
+
+// Private deps
+
+// Only the yarn CI tools have access to this private deps. If you want to test this locally,
+// remove the if condition and change the urls to match your private deps.
+if (isCI) {
+  addTest('yarnpkg/private-dep#c6cf811'); // private github shortcut
+  addTest('github:yarnpkg/private-dep#c6cf811'); // private github shortcut, with provider
+}

package.json

@@ -19,6 +19,7 @@
     "ini": "^1.3.4",
     "invariant": "^2.2.0",
     "is-builtin-module": "^1.0.0",
+    "is-ci": "^1.0.9",
     "leven": "^2.0.0",
     "loud-rejection": "^1.2.0",
     "minimatch": "^3.0.3",

src/resolvers/exotics/bitbucket-resolver.js

@@ -16,6 +16,10 @@ export default class BitbucketResolver extends HostedGitResolver {
   }
 
   static getGitSSHUrl(parts: ExplodedFragment): string {
+    return `git+ssh://[email protected]/${ parts.user }/${ parts.repo }.git`;
+  }
+
+  static getGitSSH(parts: ExplodedFragment): string {
     return `[email protected]:${parts.user}/${parts.repo}.git`;
   }
 

src/resolvers/exotics/github-resolver.js

@@ -26,6 +26,10 @@ export default class GitHubResolver extends HostedGitResolver {
   }
 
   static getGitSSHUrl(parts: ExplodedFragment): string {
+    return `git+ssh://[email protected]/${ parts.user }/${ parts.repo }.git`;
+  }
+
+  static getGitSSH(parts: ExplodedFragment): string {
     return `[email protected]:${parts.user}/${parts.repo}.git`;
   }
 

src/resolvers/exotics/gitlab-resolver.js

@@ -16,6 +16,10 @@ export default class GitLabResolver extends HostedGitResolver {
   }
 
   static getGitSSHUrl(parts: ExplodedFragment): string {
+    return `git+ssh://[email protected]/${ parts.user }/${ parts.repo }.git`;
+  }
+
+  static getGitSSH(parts: ExplodedFragment): string {
     return `[email protected]:${parts.user}/${parts.repo}.git`;
   }
 

src/resolvers/exotics/hosted-git-resolver.js

@@ -71,6 +71,11 @@ export default class HostedGitResolver extends ExoticResolver {
     throw new Error('Not implemented');
   }
 
+  static getGitSSH(exploded: ExplodedFragment): string {
+    exploded;
+    throw new Error('Not implemented');
+  }
+
   static getHTTPFileUrl(exploded: ExplodedFragment, filename: string, commit: string) {
     exploded;
     filename;
@@ -167,14 +172,15 @@ export default class HostedGitResolver extends ExoticResolver {
   }
 
   async resolve(): Promise<Manifest> {
-    const httpUrl = this.constructor.getGitHTTPUrl(this.exploded);
-    const sshUrl = this.constructor.getGitSSHUrl(this.exploded);
+    const gitHTTPUrl = this.constructor.getGitHTTPUrl(this.exploded);
+    const gitSSHUrl = this.constructor.getGitSSHUrl(this.exploded);
+    const gitSSH = this.constructor.getGitSSH(this.exploded);
 
     // If we can access the files over HTTP then we should as it's MUCH faster than git
     // archive and tarball unarchiving. The HTTP API is only available for public repos
     // though.
-    if (await this.hasHTTPCapability(httpUrl)) {
-      return await this.resolveOverHTTP(httpUrl);
+    if (await this.hasHTTPCapability(gitHTTPUrl)) {
+      return await this.resolveOverHTTP(gitHTTPUrl);
     }
 
     // If the url is accessible over git archive then we should immediately delegate to
@@ -183,13 +189,13 @@ export default class HostedGitResolver extends ExoticResolver {
     // NOTE: Here we use a different url than when we delegate to the git resolver later on.
     // This is because `git archive` requires access over ssh and github only allows that
     // if you have write permissions
-    if (await Git.hasArchiveCapability(sshUrl)) {
-      const archiveClient = new Git(this.config, sshUrl, this.hash);
+    if (await Git.hasArchiveCapability(gitSSH)) {
+      const archiveClient = new Git(this.config, gitSSH, this.hash);
       const commit = await archiveClient.initRemote();
-      return await this.fork(GitResolver, true, `${sshUrl}#${commit}`);
+      return await this.fork(GitResolver, true, `${gitSSH}#${commit}`);
     }
 
-    // fallback to the plain git resolver
-    return await this.fork(GitResolver, true, sshUrl);
+    // fallback to the plain git resolver (and append the hash only if has one)
+    return await this.fork(GitResolver, true, `${gitSSHUrl}${this.hash ? `#${this.hash}` : ''}`);
   }
 }

yarn.lock

@@ -2671,7 +2671,7 @@ is-builtin-module@^1.0.0:
   dependencies:
     builtin-modules "^1.0.0"
 
-is-ci@^1.0.9:
+is-ci, is-ci@^1.0.9:
   version "1.0.9"
   resolved "https://registry.yarnpkg.com/is-ci/-/is-ci-1.0.9.tgz#de2c5ffe49ab3237fda38c47c8a3bbfd55bbcca7"