Clone repo before loading configuration

__tests__/main.test.ts

@@ -9,10 +9,8 @@
 import * as core from '@actions/core'
 import * as main from '../src/main'
 import * as initializer from '../src/initializer'
-import { SizeUp, Score } from 'sizeup-core'
+import { SizeUp } from 'sizeup-core'
 import * as github from '@actions/github'
-import { Context } from 'sizeup-core/dist/context'
-import { CategoryConfiguration } from 'sizeup-core/dist/category-configuration'
 
 function pullRequestEventContext(overrides = {}): object {
   return {
@@ -82,21 +80,17 @@ describe('action', () => {
   // Shallow clone original @actions/github context
   const originalContext = { ...github.context }
 
+  // Mock cloning the repo
+  jest.spyOn(SizeUp, 'clone').mockImplementation(async () => {})
+
   // Mock the diff that we use for evaluation.
-  jest.spyOn(SizeUp, 'evaluate').mockImplementation(async () =>
-    Promise.resolve(
-      new Score(
-        '',
-        1,
-        new Context({
-          categories: new CategoryConfiguration([
-            { name: 'extra small', label: { name: 'xs' }, lte: 10 },
-            { name: 'large', label: { name: 'l' } }
-          ])
-        })
+  jest
+    .spyOn(SizeUp, 'diff')
+    .mockImplementation(async () =>
+      Promise.resolve(
+        '--- README.md	2023-10-16 16:35:38\n+++ README-AGAIN.md	2023-10-16 16:36:07\n@@ -0,0 +1 @@\n+# Hello, World!'
       )
     )
-  )
 
   // Mock core.getInput() such that we verify that we retrieve the auth token
   // from the right input variable.

package.json

@@ -70,7 +70,7 @@
     "@actions/github": "^5.1.1",
     "@octokit/webhooks": "^12.0.3",
     "simple-git": "^3.21.0",
-    "sizeup-core": "^0.5.4",
+    "sizeup-core": "^0.5.6",
     "yaml": "^2.3.2"
   },
   "devDependencies": {

src/main.ts

@@ -34,6 +34,12 @@ export async function run(): Promise<void> {
 
     const pullRequest = github.context.payload.pull_request as PullRequest
 
+    await SizeUp.clone(
+      core.getInput('token'),
+      pullRequest.base.repo.full_name,
+      pullRequest.head.ref
+    )
+
     const config = loadConfiguration()
     const optInStatus = getOptInStatus(pullRequest, config)
     if (optInStatus === OptInStatus.Out) return
@@ -70,27 +76,23 @@ async function evaluatePullRequest(
   const pullRequestNickname = `${pull.base.repo.full_name}#${pull.number}`
   core.info(`Evaluating pull request ${pullRequestNickname}`)
 
-  let sizeupConfigFile = undefined
+  let sizeupConfigPath = undefined
   if (config.sizeup) {
-    sizeupConfigFile = path.resolve(__dirname, './tmp/sizeup.yaml')
-    fs.mkdirSync(path.dirname(sizeupConfigFile))
-    fs.writeFileSync(sizeupConfigFile, YAML.stringify(config.sizeup))
+    sizeupConfigPath = path.resolve(__dirname, './tmp/sizeup.yaml')
+    fs.mkdirSync(path.dirname(sizeupConfigPath))
+    fs.writeFileSync(sizeupConfigPath, YAML.stringify(config.sizeup))
   }
 
-  const score = await SizeUp.evaluate(
-    {
-      repo: pull.base.repo.full_name,
-      headRef: pull.head.ref,
-      baseRef: pull.base.ref,
-      diffOptions: core.getInput('git-diff-options').split(/\s+/),
-      token: core.getInput('token'),
-      cloneDirectory: '.'
-    },
-    sizeupConfigFile
+  const diff = await SizeUp.diff(
+    core.getInput('token'),
+    pull.base.ref,
+    core.getInput('git-diff-options').split(/\s+/)
   )
 
-  if (sizeupConfigFile) {
-    fs.rmSync(sizeupConfigFile, { force: true, recursive: true })
+  const score = await SizeUp.evaluate(diff, sizeupConfigPath)
+
+  if (sizeupConfigPath) {
+    fs.rmSync(sizeupConfigPath, { force: true, recursive: true })
   }
 
   const categoryDescription = score.category ? `(${score.category.name})` : ''