DRY repository name parsing

lib/get_repo_data.js

@@ -1,4 +1,4 @@
-const repo = require('./get_repo')()
+const repo = require('./repo_name')
 const { grey } = require('chalk')
 
 const getPagesSequentially = require('./get_pages_sequentially')

lib/repo_name.js

@@ -0,0 +1,16 @@
+const { execSync } = require('child_process')
+
+// Support the following remote URL format
+// - git@github.com:owner/repo-name.git
+// - https://github.com/owner/repo-name
+// - https://github.com/owner/repo-name.git
+
+const getName = remoteCmd => {
+  const cmd = `${remoteCmd} | awk '{printf $2}' | sed 's|^.*:||' | sed 's|//github.com/||' | sed 's|\.git$||'`
+  return execSync(cmd).toString().trim()
+}
+
+const githubOriginRemote = getName('git remote -v | grep origin | grep github.com')
+const firstGithubRemote = getName('git remote -v | grep github.com | head -n 1')
+
+module.exports = githubOriginRemote.match(/.+\/.+/) != null ? githubOriginRemote : firstGithubRemote

package.json

@@ -5,7 +5,8 @@
   "bin": {
     "backup-github-repo": "./bin/backup_github_repo",
     "download-github-repo-json": "./scripts/download_json",
-    "download-github-repo-html": "./scripts/download_html"
+    "download-github-repo-html": "./scripts/download_html",
+    "repository-name": "./scripts/repository_name"
   },
   "main": "index.js",
   "directories": {

scripts/download_html

@@ -1,7 +1,9 @@
 #!/usr/bin/env bash
 set -eu
 
-repo=$(git remote -v | grep github.com | head -n 1 | awk '{printf $2}' | sed 's|^.*:||' | sed 's|//github.com/||' | sed 's|\.git$||')
+# repository-name is defined in package.json
+repo=$(repository-name)
+
 echo "repo: $repo"
 last_id=$(jd "./repo-backup/data.json" ._keys._last)
 echo "entries: $last_id"

scripts/repository_name

@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+console.log(require('../lib/repo_name.js'))