api-diff can take two commits (#2080)

api-diff can take two commits

Author: david-crespo

tools/deno/api-diff.ts

@@ -7,8 +7,8 @@
  *
  * Copyright Oxide Computer Company
  */
-import * as flags from 'https://deno.land/std@0.208.0/flags/mod.ts'
 import { exists } from 'https://deno.land/std@0.208.0/fs/mod.ts'
+import { parseArgs } from 'https://deno.land/std@0.220.1/cli/mod.ts'
 import { $ } from 'https://deno.land/x/dax@0.39.1/mod.ts'
 
 const HELP = `
@@ -21,17 +21,19 @@ Requirements:
 
 Usage:
   ./tools/deno/api-diff.ts [-f] [PR number or commit SHA]
+  ./tools/deno/api-diff.ts [-f] [commit SHA] [commit SHA]
   ./tools/deno/api-diff.ts -h
 
 Flags:
   -f, --force       Download spec and regen client even if dir already exists
   -h, --help        Show this help message
 
 Parameters:
-  PR number or commit SHA: If left out, interactive picker is shown
+  PR number or commit SHA: If left out, interactive picker is shown.
+  If two positional arguments are passed, we assume they are commits.
 `.trim()
 
-function printHelpAndExit() {
+function printHelpAndExit(): never {
   console.log(HELP)
   Deno.exit()
 }
@@ -54,10 +56,16 @@ async function pickPr() {
 }
 
 async function getCommitRange(
-  arg: string | number | undefined
+  args: Array<string | number>
 ): Promise<{ base: string; head: string }> {
-  if (!arg || typeof arg === 'number') {
-    const prNum = arg || (await pickPr())
+  // if there are two or more args, assume two commits
+  if (args.length >= 2) {
+    return { base: args[0].toString(), head: args[1].toString() }
+  }
+
+  // if there are no args or the arg is a number, we're talking about a PR
+  if (args.length === 0 || typeof args[0] === 'number') {
+    const prNum = args[0] || (await pickPr())
     const query = `{
       repository(owner: "oxidecomputer", name: "omicron") {
         pullRequest(number: ${prNum}) {
@@ -72,10 +80,11 @@ async function getCommitRange(
   }
 
   // otherwise assume it's a commit
+  const head = args[0]
   const parents =
-    await $`gh api repos/oxidecomputer/omicron/commits/${arg} --jq '.parents'`.json()
+    await $`gh api repos/oxidecomputer/omicron/commits/${head} --jq '.parents'`.json()
   if (parents.length > 1) throw new Error(`Commit has multiple parents:`)
-  return { base: parents[0].sha, head: arg }
+  return { base: parents[0].sha, head }
 }
 
 async function genForCommit(commit: string, force: boolean) {
@@ -102,14 +111,14 @@ if (!$.commandExistsSync('gh')) throw Error('Need gh (GitHub CLI)')
 // prefer difftastic if it exists. https://difftastic.wilfred.me.uk/
 const diffTool = $.commandExistsSync('difft') ? 'difft' : 'diff'
 
-const args = flags.parse(Deno.args, {
+const args = parseArgs(Deno.args, {
   alias: { force: 'f', help: 'h' },
   boolean: ['force', 'help'],
 })
 
 if (args.help) printHelpAndExit()
 
-const { base, head } = await getCommitRange(args._[0])
+const { base, head } = await getCommitRange(args._)
 
 const tmpDirBase = await genForCommit(base, args.force)
 const tmpDirHead = await genForCommit(head, args.force)