Draft analysis

Author: Lere Williams

.eslintrc

@@ -2,5 +2,6 @@
   "extends": [
     "oclif",
     "oclif-typescript"
-  ]
+  ],
+  "indent": ["SwitchCase", 2]
 }

package-lock.json

@@ -21,7 +21,15 @@
     "@oclif/plugin-help": "^5",
     "@oclif/plugin-plugins": "^5.0.18",
     "@oclif/plugin-warn-if-update-available": "^3.0.2",
-    "octokit": "^3.1.1",
+    "@octokit/plugin-paginate-rest": "^13.1.1",
+    "@octokit/plugin-request-log": "^6.0.0",
+    "@octokit/request-error": "^7.0.0",
+    "@octokit/rest": "^22.0.0",
+    "csv-parse": "^5.6.0",
+    "duckdb-async": "^1.1.1",
+    "node-fetch": "^3.3.2",
+    "node-stream-zip": "^1.15.0",
+    "p-limit": "^6.1.0",
     "read": "^3.0.1",
     "simple-git": "^3.20.0",
     "sizeup-core": "^0.5.7"
@@ -47,8 +55,13 @@
     "dirname": "sizeup",
     "default": ".",
     "commands": {
-      "strategy": "single",
-      "target": "./dist/index.js"
+      "strategy": "pattern",
+      "target": "./dist/commands"
+    },
+    "topics": {
+      "stats": {
+        "description": "Analyze stats generated by sizeup-action"
+      }
     },
     "plugins": [
       "@oclif/plugin-help",

package.json

@@ -1,12 +1,14 @@
 /* eslint-disable perfectionist/sort-classes */
 import {Args, Command, Flags, ux} from '@oclif/core'
 import * as fs from 'node:fs'
-import {Octokit} from 'octokit'
+import {Octokit} from '@octokit/rest'
 import {read} from 'read'
 import {simpleGit} from 'simple-git'
-import {Score, SizeUp as SizeUpCore} from 'sizeup-core'
+import {Score, SizeUp} from 'sizeup-core'
+
+export default class Evaluate extends Command {
+  static aliases = ['eval']
 
-export default class SizeUp extends Command {
   static args = {
     diff: Args.string({
       default: '',
@@ -46,6 +48,7 @@ export default class SizeUp extends Command {
     'token-path': Flags.string({
       char: 't',
       description: 'Path to a file containing a GitHub API token.\n'
+       + 'When evaluated a pull request from a private repository, this token must have the full `repo` scope and may also need to be SSO-enabled for the appropriate organization.\n'
        + 'If this flag is omitted and the `diff` argument is a URL, then this tool will prompt for a token instead.',
       required: false,
     }),
@@ -59,7 +62,7 @@ export default class SizeUp extends Command {
   static strict = false
 
   async run(): Promise<void> {
-    const {args, flags} = await this.parse(SizeUp)
+    const {args, flags} = await this.parse(Evaluate)
     let score: Score | undefined
 
     if (args.diff?.startsWith('https://')) {
@@ -103,12 +106,12 @@ export default class SizeUp extends Command {
           const cloneDirectory = `/tmp/${repo}`
 
           // Clear the contents of the clone directory,
-          // otherwise SizeUpCore.evaluate will refuse to overwrite them.
+          // otherwise SizeUp.evaluate will refuse to overwrite them.
           fs.rmSync(cloneDirectory, {force: true, recursive: true})
           fs.mkdirSync(cloneDirectory, {recursive: true})
 
           return {
-            result: SizeUpCore.evaluate(
+            result: SizeUp.evaluate(
               {
                 baseRef: pull.data.base.ref,
                 cloneDirectory,
@@ -145,7 +148,7 @@ export default class SizeUp extends Command {
 
     return this.reportProgress(
       `Evaluating the diff with the ${this.configChoice(flags)}`,
-      async () => ({result: await SizeUpCore.evaluate(diff, flags['config-path'])}),
+      async () => ({result: await SizeUp.evaluate(diff, flags['config-path'])}),
     )
   }
 
@@ -165,7 +168,7 @@ export default class SizeUp extends Command {
 
     return this.reportProgress(
       `Evaluating the diff with the ${this.configChoice(flags)}`,
-      async () => ({result: await SizeUpCore.evaluate(diff, flags['config-path'])}),
+      async () => ({result: await SizeUp.evaluate(diff, flags['config-path'])}),
     )
   }