ref: GHA provider to use @actions/github (#152)

Update GHA provider utils to use @actions/github to access the correct commit SHA's.

Author: nsdeschenes

.changeset/warm-moles-float.md

@@ -0,0 +1,12 @@
+---
+"@codecov/bundler-plugin-core": patch
+"@codecov/solidstart-plugin": patch
+"@codecov/remix-vite-plugin": patch
+"@codecov/sveltekit-plugin": patch
+"@codecov/webpack-plugin": patch
+"@codecov/rollup-plugin": patch
+"@codecov/nuxt-plugin": patch
+"@codecov/vite-plugin": patch
+---
+
+Swap over to using @actions/github to grab head and base commit sha's in GHA

packages/bundler-plugin-core/package.json

@@ -39,12 +39,14 @@
     "generate:typedoc": "typedoc --options ./typedoc.json"
   },
   "dependencies": {
+    "@actions/github": "^6.0.0",
     "chalk": "4.1.2",
     "semver": "^7.5.4",
     "unplugin": "^1.10.1",
     "zod": "^3.22.4"
   },
   "devDependencies": {
+    "@octokit/webhooks-definitions": "^3.67.3",
     "@types/node": "^20.11.15",
     "@types/semver": "^7.5.6",
     "@vitest/coverage-v8": "^1.5.0",

packages/bundler-plugin-core/src/utils/providers/GitHubActions.ts

@@ -1,3 +1,4 @@
+import * as GitHub from "@actions/github";
 import { jsonSchema } from "../../schemas.ts";
 import {
   type ProviderEnvs,
@@ -6,7 +7,7 @@ import {
 } from "../../types.ts";
 import { type Output } from "../Output.ts";
 import { debug } from "../logging.ts";
-import { runExternalProgram } from "../runExternalProgram.ts";
+import { type PullRequestEvent } from "@octokit/webhooks-definitions/schema";
 
 export function detect(envs: ProviderEnvs): boolean {
   return Boolean(envs?.GITHUB_ACTIONS);
@@ -136,30 +137,12 @@ function _getSHA(
     return args.sha;
   }
 
-  const pr = _getPR(inputs);
+  const context = GitHub.context;
 
   let commit = envs?.GITHUB_SHA;
-  if (pr) {
-    const mergeCommitRegex = /^[a-z0-9]{40} [a-z0-9]{40}$/;
-    const mergeCommitMessage = runExternalProgram("git", [
-      "show",
-      "--no-patch",
-      "--format=%P",
-    ]);
-
-    debug(`Merge commit message: ${mergeCommitMessage}`, {
-      enabled: output.debug,
-    });
-
-    if (mergeCommitRegex.exec(mergeCommitMessage)) {
-      const splitMergeCommit = mergeCommitMessage.split(" ");
-
-      debug(`Split commit message: ${splitMergeCommit}`, {
-        enabled: output.debug,
-      });
-
-      commit = splitMergeCommit[1];
-    }
+  if (["pull_request", " pull_request_target"].includes(context.eventName)) {
+    const payload = context.payload as PullRequestEvent;
+    commit = payload.pull_request.head.sha;
   }
 
   debug(`Using commit: ${commit ?? null}`, { enabled: output.debug });
@@ -178,28 +161,10 @@ function _getCompareSHA(
   }
 
   let compareSha = null;
-  const pr = _getPR(inputs);
-  if (pr) {
-    const mergeCommitRegex = /^[a-z0-9]{40} [a-z0-9]{40}$/;
-    const mergeCommitMessage = runExternalProgram("git", [
-      "show",
-      "--no-patch",
-      "--format=%P",
-    ]);
-
-    debug(`Merge commit message: ${mergeCommitMessage}`, {
-      enabled: output.debug,
-    });
-
-    if (mergeCommitRegex.exec(mergeCommitMessage)) {
-      const splitMergeCommit = mergeCommitMessage.split(" ");
-
-      debug(`Split commit message: ${splitMergeCommit}`, {
-        enabled: output.debug,
-      });
-
-      compareSha = splitMergeCommit?.[0] ? splitMergeCommit[0] : null;
-    }
+  const context = GitHub.context;
+  if (["pull_request", " pull_request_target"].includes(context.eventName)) {
+    const payload = context.payload as PullRequestEvent;
+    compareSha = payload.pull_request.base.sha;
   }
 
   debug(`Using compareSha: ${compareSha ?? null}`, { enabled: output.debug });