fix: fixes bug with repo url handling (#97)

* fix: fixes bug with repo url handling

when repo url was user/name shorthand we failed if it had extra path
chunks. user/name/tacos etc. because users have been allowed to publish
packages this way have have to add support for reading thme out of the
registry this way.

src/lib/packument.ts

@@ -66,9 +66,15 @@ export const repoToGithub = (
       if (url && url.indexOf('https://github.com') === 0) {
         return {url, name: new URL(url).pathname.substr(1)};
       }
-      if (!url && repo.url.match(/^[^/]+\/[^/]+$/)) {
+
+      if (!url && /^[^/]+\/[^/]+/.test(repo.url)) {
         //'xxxx/xxxx' username/repo specifier
-        return {url: 'https://github.com/' + repo.url, name: repo.url};
+        const matches = repo.url.match(/^([^/]+\/[^/]+)(.+)?$/);
+        // grabs the first 2 / delimited chunks.
+        const cleaned = matches ? matches[1] : null;
+        if (cleaned) {
+          return {url: 'https://github.com/' + cleaned, name: cleaned};
+        }
       }
     }
   }

test/lib/packument.ts

@@ -0,0 +1,27 @@
+import {expect} from 'chai';
+import {describe, it} from 'mocha';
+
+import {repoToGithub} from '../../src/lib/packument';
+
+describe('repoToGithub', () => {
+  it('handles github url shorthand with extra path segments', () => {
+    const result = repoToGithub({
+      type: 'git',
+      url:
+        'GoogleCloudPlatform/cloud-for-marketing/tree/master/marketing-analytics/activation/common-libs/nodejs-common',
+    });
+
+    const match = repoToGithub({
+      type: 'git',
+      url: 'GoogleCloudPlatform/cloud-for-marketing',
+    });
+
+    const expected = {
+      url: 'https://github.com/GoogleCloudPlatform/cloud-for-marketing',
+      name: 'GoogleCloudPlatform/cloud-for-marketing',
+    };
+
+    expect(result).to.eql(expected);
+    expect(match).to.eql(expected);
+  });
+});