@jsenv/github-pull-request-impact 1.6.4 → 1.6.8

package/README.md

@@ -1,17 +1,13 @@
-# GitHub pull request impact
+# GitHub pull request impact [![npm package](https://img.shields.io/npm/v/@jsenv/github-pull-request-impact.svg?logo=npm&label=package)](https://www.npmjs.com/package/@jsenv/github-pull-request-impact)
 
 Report pull request impact on something.
 
-[![npm package](https://img.shields.io/npm/v/@jsenv/github-pull-request-impact.svg?logo=npm&label=package)](https://www.npmjs.com/package/@jsenv/github-pull-request-impact)
-[![github main](https://github.com/jsenv/github-pull-request-impact/workflows/main/badge.svg)](https://github.com/jsenv/github-pull-request-impact/actions?workflow=main)
-[![codecov coverage](https://codecov.io/gh/jsenv/github-pull-request-impact/branch/main/graph/badge.svg)](https://codecov.io/gh/jsenv/github-pull-request-impact)
-
 # Presentation
 
 Under construction.
 
-This repository publishes `@jsenv/github-pull-request-impact` on npm. This package is reused internally by:
+This repository publishes `@jsenv/github-pull-request-impact` on NPM. This package is reused internally by:
 
-- [@jsenv/performance-impact](https://github.com/jsenv/performance-impact)
-- [@jsenv/file-size-impact](https://github.com/jsenv/file-size-impact)
-- [@jsenv/lighthouse-score-impact](https://github.com/jsenv/jsenv-lighthouse-score-impact)
+- [@jsenv/performance-impact](../jsenv-performance-impact)
+- [@jsenv/file-size-impact](../jsenv-file-size-impact)
+- [@jsenv/lighthouse-score-impact](../jsenv-performance-impact)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/github-pull-request-impact",
-  "version": "1.6.4",
+  "version": "1.6.8",
   "description": "Report pull request impact on something",
   "license": "MIT",
   "author": {
@@ -10,13 +10,15 @@
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/jsenv/github-pull-request-impact"
+    "url": "https://github.com/jsenv/workflow",
+    "directory": "packages/jsenv-github-pull-request-impact"
   },
   "engines": {
-    "node": ">=14.17.0"
+    "node": ">=16.13.0"
   },
   "publishConfig": {
-    "access": "public"
+    "access": "public",
+    "registry": "https://registry.npmjs.org"
   },
   "type": "module",
   "exports": {
@@ -25,42 +27,31 @@
     },
     "./*": "./*"
   },
+  "main": "./main.js",
   "files": [
     "/src/",
     "/main.js"
   ],
   "scripts": {
-    "eslint-check": "node ./node_modules/eslint/bin/eslint.js .",
-    "test": "node ./script/test/test.js",
-    "generate-import-map": "node ./script/generate-import-map/generate-import-map.js",
+    "eslint": "npx eslint . --ext=.js,.mjs",
+    "importmap": "node ./script/importmap/importmap.mjs",
+    "test": "node ./script/test/test.mjs",
     "test-with-coverage": "npm run test -- --coverage",
-    "prettier-format": "node ./script/prettier-format/prettier-format.js",
-    "prettier-format-stage": "npm run prettier-format -- --staged",
-    "prettier-check": "npm run prettier-format -- --dry-run",
-    "log-performance-report": "node .github/workflows/performance_impact/generate_performance_report.js --log",
-    "upload-coverage": "node ./script/upload-coverage/upload-coverage.js",
-    "prepublishOnly": "node ./script/transform-package/remove-postinstall.js",
-    "postpublish": "node ./script/transform-package/restore-postinstall.js"
+    "prettier": "prettier --write ."
   },
   "dependencies": {
-    "@jsenv/cancellation": "3.0.0",
+    "@jsenv/filesystem": "2.7.1",
     "@jsenv/logger": "4.0.1",
-    "@jsenv/server": "6.1.0",
-    "@jsenv/util": "4.1.1"
+    "node-fetch": "2.6.7"
   },
   "devDependencies": {
-    "@jsenv/assert": "2.2.5",
-    "@jsenv/codecov-upload": "3.5.0",
-    "@jsenv/core": "19.0.2",
-    "@jsenv/eslint-config": "15.0.1",
-    "@jsenv/github-release-package": "1.2.3",
-    "@jsenv/importmap-eslint-resolver": "5.0.0",
-    "@jsenv/node-module-import-map": "13.6.1",
-    "@jsenv/package-publish": "1.6.2",
-    "@jsenv/performance-impact": "1.2.1",
-    "@jsenv/prettier-check-project": "5.6.1",
-    "eslint": "7.31.0",
-    "eslint-plugin-import": "2.23.4",
-    "prettier": "2.3.2"
+    "@jsenv/assert": "2.4.1",
+    "@jsenv/core": "25.3.0",
+    "@jsenv/eslint-config": "16.0.9",
+    "@jsenv/importmap-eslint-resolver": "5.2.5",
+    "@jsenv/importmap-node-module": "5.1.3",
+    "eslint": "8.7.0",
+    "eslint-plugin-import": "2.25.4",
+    "prettier": "2.5.1"
   }
 }

package/src/commentGitHubPullRequestImpact.js

@@ -4,16 +4,14 @@
  * @jsenv/file-size-impact, @jsenv/performance-impact and @jsenv/lighthouse-score-impact
  */
 
-import { createLogger } from "@jsenv/logger"
 import {
-  createCancellationToken,
-  createCancellationSource,
-  composeCancellationToken,
-} from "@jsenv/cancellation"
-import { assertAndNormalizeDirectoryUrl, urlToFileSystemPath } from "@jsenv/util"
+  assertAndNormalizeDirectoryUrl,
+  urlToFileSystemPath,
+} from "@jsenv/filesystem"
+import { createLogger } from "@jsenv/logger"
 
+import * as githubRESTAPI from "./internal/github_rest_api.js"
 import { exec } from "./internal/exec.js"
-import { GET, POST, PATCH } from "./internal/git_hub_api.js"
 import { renderGeneratedByText } from "./internal/renderGeneratedByText.js"
 import { createGitHubPullRequestCommentText } from "./internal/createGitHubPullRequestCommentText.js"
 
@@ -21,8 +19,6 @@ export const commentGitHubPullRequestImpact = async ({
   logLevel,
   commandLogs = false,
   infoLogs = true,
-  cancellationToken = createCancellationToken(),
-  cancelOnSIGINT = true,
   projectDirectoryUrl,
 
   githubToken,
@@ -47,51 +43,46 @@ export const commentGitHubPullRequestImpact = async ({
 }) => {
   projectDirectoryUrl = assertAndNormalizeDirectoryUrl(projectDirectoryUrl)
   if (typeof collectInfo !== "function") {
-    throw new TypeError(`collectInfo must be a function but received ${collectInfo}`)
+    throw new TypeError(
+      `collectInfo must be a function but received ${collectInfo}`,
+    )
   }
   if (typeof commentIdentifier !== "string") {
-    throw new TypeError(`commentIdentifier must be a string but received ${commentIdentifier}`)
+    throw new TypeError(
+      `commentIdentifier must be a string but received ${commentIdentifier}`,
+    )
   }
   if (typeof githubToken !== "string") {
-    throw new TypeError(`githubToken must be a string but received ${githubToken}`)
+    throw new TypeError(
+      `githubToken must be a string but received ${githubToken}`,
+    )
   }
   if (typeof repositoryOwner !== "string") {
-    throw new TypeError(`repositoryOwner must be a string but received ${repositoryOwner}`)
+    throw new TypeError(
+      `repositoryOwner must be a string but received ${repositoryOwner}`,
+    )
   }
   if (typeof repositoryName !== "string") {
-    throw new TypeError(`repositoryName must be a string but received ${repositoryName}`)
+    throw new TypeError(
+      `repositoryName must be a string but received ${repositoryName}`,
+    )
   }
   pullRequestNumber = String(pullRequestNumber)
   if (typeof pullRequestNumber !== "string") {
-    throw new TypeError(`pullRequestNumber must be a string but received ${pullRequestNumber}`)
+    throw new TypeError(
+      `pullRequestNumber must be a string but received ${pullRequestNumber}`,
+    )
   }
 
   const logger = createLogger({ logLevel })
 
   let cleanup = () => {}
 
-  if (cancelOnSIGINT) {
-    const processCancellationSource = createCancellationSource()
-    const processCancellationToken = processCancellationSource.token
-    cancellationToken = composeCancellationToken(cancellationToken, processCancellationToken)
-    const SIGINTCallback = () => {
-      processCancellationSource.cancel("process SIGINT")
-    }
-    process.once("SIGINT", SIGINTCallback)
-
-    // beware if someday something do things on cleanup
-    // to call the old cleanup function
-    cleanup = () => {
-      process.removeListener("SIGINT", SIGINTCallback)
-    }
-  }
-
   try {
     return await commentPrImpact({
       logger,
       commandLogs,
       infoLogs,
-      cancellationToken,
       projectDirectoryUrl,
 
       githubToken,
@@ -120,7 +111,6 @@ const commentPrImpact = async ({
   logger,
   infoLogs,
   commandLogs,
-  cancellationToken,
   projectDirectoryUrl,
 
   githubToken,
@@ -156,8 +146,8 @@ const commentPrImpact = async ({
   const pullApiUrl = `https://api.github.com/repos/${repositoryOwner}/${repositoryName}/pulls/${pullRequestNumber}`
   // https://developer.github.com/v3/pulls/#get-a-pull-request
   logger.debug(`get pull request ${pullApiUrl}`)
-  const pullRequest = await GET(pullApiUrl, {
-    cancellationToken,
+  const pullRequest = await githubRESTAPI.GET({
+    url: pullApiUrl,
     githubToken,
   })
   const pullRequestHtmlUrl = pullRequest.html_url
@@ -187,11 +177,13 @@ const commentPrImpact = async ({
 
   logger.debug(`searching comment in pull request ${pullRequestHtmlUrl}`)
   const pullRequestCommentApiUrl = pullRequest.comments_url
-  const comments = await GET(pullRequestCommentApiUrl, {
+  const comments = await pullApiUrl.GET({
+    url: pullRequestCommentApiUrl,
     githubToken,
-    cancellationToken,
   })
-  const existingComment = comments.find(({ body }) => body.includes(commentIdentifier))
+  const existingComment = comments.find(({ body }) =>
+    body.includes(commentIdentifier),
+  )
   if (existingComment) {
     logger.debug(`comment found at ${existingComment.html_url}.`)
   } else {
@@ -224,27 +216,29 @@ const commentPrImpact = async ({
         return existingComment
       }
       logger.info(`updating comment at ${existingComment.html_url}`)
-      const comment = await PATCH(
-        `https://api.github.com/repos/${repositoryOwner}/${repositoryName}/issues/comments/${existingComment.id}`,
-        { body: commentBody },
-        { githubToken, cancellationToken },
-      )
+      const comment = await githubRESTAPI.PATCH({
+        url: `https://api.github.com/repos/${repositoryOwner}/${repositoryName}/issues/comments/${existingComment.id}`,
+        githubToken,
+        body: { body: commentBody },
+      })
       logger.info("comment updated")
       return comment
     }
 
     logger.info(`creating comment`)
-    const comment = await POST(
-      `https://api.github.com/repos/${repositoryOwner}/${repositoryName}/issues/${pullRequestNumber}/comments`,
-      { body: commentBody },
-      { githubToken, cancellationToken },
-    )
+    const comment = await githubRESTAPI.POST({
+      url: `https://api.github.com/repos/${repositoryOwner}/${repositoryName}/issues/${pullRequestNumber}/comments`,
+      githubToken,
+      body: { body: commentBody },
+    })
     logger.info(`comment created at ${comment.html_url}`)
     return comment
   }
 
-  const isFork = pullRequest.base.repo.full_name !== pullRequest.head.repo.full_name
-  const isInPullRequestWorkflow = process.env.GITHUB_EVENT_NAME === "pull_request"
+  const isFork =
+    pullRequest.base.repo.full_name !== pullRequest.head.repo.full_name
+  const isInPullRequestWorkflow =
+    process.env.GITHUB_EVENT_NAME === "pull_request"
   if (isFork && isInPullRequestWorkflow) {
     logger.warn(formatGithubTokenNotAllowedToCommentWarning())
   }
@@ -260,10 +254,13 @@ const commentPrImpact = async ({
   let beforeMergeInfo
   try {
     // cannot use depth=1 arg otherwise git merge might have merge conflicts
-    await execCommandInProjectDirectory(`git fetch --no-tags --prune origin ${pullRequestBase}`)
-    await execCommandInProjectDirectory(`git reset --hard origin/${pullRequestBase}`)
+    await execCommandInProjectDirectory(
+      `git fetch --no-tags --prune origin ${pullRequestBase}`,
+    )
+    await execCommandInProjectDirectory(
+      `git reset --hard origin/${pullRequestBase}`,
+    )
     beforeMergeInfo = await collectInfo({
-      cancellationToken,
       beforeMerge: true,
       execCommandInProjectDirectory,
     })
@@ -273,7 +270,10 @@ const commentPrImpact = async ({
   } catch (error) {
     logger.error(error.stack)
     const comment = await patchOrPostComment(
-      createCommentForBeforeMergeError(error, { pullRequestHead, pullRequestBase }),
+      createCommentForBeforeMergeError(error, {
+        pullRequestHead,
+        pullRequestBase,
+      }),
     )
     if (catchError) {
       return { error, comment }
@@ -285,11 +285,15 @@ const commentPrImpact = async ({
   try {
     // buildCommand might generate files that could conflict when doing the merge
     // reset to avoid potential merge conflicts
-    await execCommandInProjectDirectory(`git reset --hard origin/${pullRequestBase}`)
+    await execCommandInProjectDirectory(
+      `git reset --hard origin/${pullRequestBase}`,
+    )
     // Avoid "The following untracked working tree files would be overwritten by merge" error
     await execCommandInProjectDirectory(`git clean -d -f .`)
     // cannot use depth=1 arg otherwise git merge might have merge conflicts
-    await execCommandInProjectDirectory(`git fetch --no-tags --prune origin ${headRef}`)
+    await execCommandInProjectDirectory(
+      `git fetch --no-tags --prune origin ${headRef}`,
+    )
     // ensure there is user.email + user.name required to perform git merge command
     // without them git would complain that it does not know who we are
 
@@ -301,11 +305,12 @@ const commentPrImpact = async ({
       valueIfMissing: "Your Name",
       execCommandInProjectDirectory,
     })
-    await execCommandInProjectDirectory(`git merge FETCH_HEAD --allow-unrelated-histories`)
+    await execCommandInProjectDirectory(
+      `git merge FETCH_HEAD --allow-unrelated-histories`,
+    )
     await restoreGitUserEmail()
     await restoreGitUserName()
     afterMergeInfo = await collectInfo({
-      cancellationToken,
       afterMerge: true,
       execCommandInProjectDirectory,
     })
@@ -315,7 +320,10 @@ const commentPrImpact = async ({
   } catch (error) {
     logger.error(error.stack)
     const gitHubComment = await patchOrPostComment(
-      createCommentForAfterMergeError(error, { pullRequestHead, pullRequestBase }),
+      createCommentForAfterMergeError(error, {
+        pullRequestHead,
+        pullRequestBase,
+      }),
     )
     if (catchError) {
       return { error, gitHubComment }
@@ -336,7 +344,6 @@ const commentPrImpact = async ({
   }
 
   const commentForComparison = await createCommentForComparison({
-    cancellationToken,
     logger,
 
     pullRequestBase,
@@ -350,12 +357,17 @@ const commentPrImpact = async ({
   return { gitHubComment }
 }
 
-const ensureGitConfig = async (name, { execCommandInProjectDirectory, valueIfMissing }) => {
+const ensureGitConfig = async (
+  name,
+  { execCommandInProjectDirectory, valueIfMissing },
+) => {
   try {
     await execCommandInProjectDirectory(`git config ${name}`)
     return () => {}
   } catch (e) {
-    await execCommandInProjectDirectory(`git config ${name} "${valueIfMissing}"`)
+    await execCommandInProjectDirectory(
+      `git config ${name} "${valueIfMissing}"`,
+    )
     return async () => {
       await execCommandInProjectDirectory(`git config --unset ${name}`)
     }
@@ -391,7 +403,10 @@ To fix this, change "pull_request" for "pull_request_target" in your workflow fi
 See https://docs.github.com/en/actions/reference/events-that-trigger-workflows#pull_request_target`
 }
 
-const jsenvCreateBeforeMergeErrorComment = (error, { pullRequestBase, pullRequestHead }) => {
+const jsenvCreateBeforeMergeErrorComment = (
+  error,
+  { pullRequestBase, pullRequestHead },
+) => {
   return `---
 
 **Error:** Error while trying to collect info before merging ${pullRequestHead} into ${pullRequestBase}.
@@ -401,7 +416,10 @@ const jsenvCreateBeforeMergeErrorComment = (error, { pullRequestBase, pullReques
 ---`
 }
 
-const jsenvCreateCommentForAfterMergeError = (error, { pullRequestBase, pullRequestHead }) => {
+const jsenvCreateCommentForAfterMergeError = (
+  error,
+  { pullRequestBase, pullRequestHead },
+) => {
   return `---
 
 **Error:** Error while trying to collect info after merging ${pullRequestHead} into ${pullRequestBase}.
@@ -411,7 +429,10 @@ const jsenvCreateCommentForAfterMergeError = (error, { pullRequestBase, pullRequ
 ---`
 }
 
-const jsenvCreateCommentForVersionMismatch = ({ beforeMergeVersion, afterMergeversion }) => {
+const jsenvCreateCommentForVersionMismatch = ({
+  beforeMergeVersion,
+  afterMergeversion,
+}) => {
   return `---
 
 **Error:** Cannot compare before and after merge data because the version are different.
@@ -423,7 +444,10 @@ After merge data version: ${afterMergeversion}
 ---`
 }
 
-const jsenvCreateCommentForComparison = ({ beforeMergeInfo, afterMergeInfo }) => {
+const jsenvCreateCommentForComparison = ({
+  beforeMergeInfo,
+  afterMergeInfo,
+}) => {
   return `--- before merge ---
 ${JSON.stringify(beforeMergeInfo, null, "  ")}
 --- after merge ---

package/src/internal/exec.js

@@ -1,6 +1,9 @@
-import { exec as nodeExec } from "child_process"
+import { exec as nodeExec } from "node:child_process"
 
-export const exec = (command, { cwd, onLog = () => {}, onErrorLog = () => {} } = {}) => {
+export const exec = (
+  command,
+  { cwd, onLog = () => {}, onErrorLog = () => {} } = {},
+) => {
   return new Promise((resolve, reject) => {
     const commandProcess = nodeExec(
       command,

package/src/internal/{git_hub_api.js → github_rest_api.js}

@@ -1,11 +1,14 @@
-import { fetchUrl } from "@jsenv/server"
+import nodeFetch from "node-fetch"
 
-export const GET = async (url, { githubToken, cancellationToken } = {}) => {
-  return sendHttpRequest(url, {
-    cancellationToken,
+export const GET = async ({ url, githubToken, headers }) => {
+  return sendHttpRequest({
+    url,
     method: "GET",
-    headers: tokenToHeaders(githubToken),
-    responseStatusMap: {
+    headers: {
+      ...tokenToHeaders(githubToken),
+      headers,
+    },
+    responseStatusHandlers: {
       200: async (response) => {
         const json = await response.json()
         return json
@@ -15,13 +18,16 @@ export const GET = async (url, { githubToken, cancellationToken } = {}) => {
   })
 }
 
-export const POST = (url, body, { cancellationToken, githubToken } = {}) => {
-  return sendHttpRequest(url, {
-    cancellationToken,
+export const POST = ({ url, body, githubToken, headers }) => {
+  return sendHttpRequest({
+    url,
     method: "POST",
-    headers: tokenToHeaders(githubToken),
+    headers: {
+      ...tokenToHeaders(githubToken),
+      headers,
+    },
     body: JSON.stringify(body),
-    responseStatusMap: {
+    responseStatusHandlers: {
       201: async (response) => {
         const json = await response.json()
         return json
@@ -30,13 +36,16 @@ export const POST = (url, body, { cancellationToken, githubToken } = {}) => {
   })
 }
 
-export const PATCH = async (url, body, { cancellationToken, githubToken } = {}) => {
-  return sendHttpRequest(url, {
-    cancellationToken,
+export const PATCH = async ({ url, body, githubToken, headers }) => {
+  return sendHttpRequest({
+    url,
     method: "PATCH",
-    headers: tokenToHeaders(githubToken),
+    headers: {
+      ...tokenToHeaders(githubToken),
+      headers,
+    },
     body: JSON.stringify(body),
-    responseStatusMap: {
+    responseStatusHandlers: {
       200: async (response) => {
         const json = await response.json()
         return json
@@ -54,17 +63,21 @@ const tokenToHeaders = (token) => {
   }
 }
 
-const sendHttpRequest = async (
+const sendHttpRequest = async ({
   url,
-  { cancellationToken, method, headers, body, responseStatusMap },
-) => {
+  method,
+  headers,
+  body,
+  responseStatusHandlers = {},
+}) => {
   let response
   try {
-    response = await fetchUrl(url, {
-      cancellationToken,
+    response = await nodeFetch(url, {
       method,
       headers: {
-        ...(typeof body === "undefined" ? {} : { "content-length": Buffer.byteLength(body) }),
+        ...(typeof body === "undefined"
+          ? {}
+          : { "content-length": Buffer.byteLength(body) }),
         ...headers,
       },
       body,
@@ -80,16 +93,16 @@ ${error.stack}`)
   }
 
   const { status } = response
-  if (status in responseStatusMap) {
-    return responseStatusMap[response.status](response)
+  const responseStatusHandler = responseStatusHandlers[status]
+  if (responseStatusHandler) {
+    return responseStatusHandler(response)
   }
-
   const responseBodyAsJson = await response.json()
   const error = new Error(`unexpected response status.
 --- response status ---
 ${response.status}
 --- expected response status ---
-${Object.keys(responseStatusMap).join(", ")}
+${Object.keys(responseStatusHandlers).join(", ")}
 --- request method ---
 ${method}
 --- request url ---

package/src/readGitHubWorkflowEnv.js

@@ -1,28 +1,25 @@
-import { readFileSync } from "fs"
+import { readFileSync } from "node:fs"
 
 export const readGitHubWorkflowEnv = () => {
   const eventName = process.env.GITHUB_EVENT_NAME
   if (!eventName) {
-    throw new Error(`missing process.env.GITHUB_EVENT_NAME, we are not in a github workflow`)
+    throw new Error(
+      `missing process.env.GITHUB_EVENT_NAME, we are not in a github workflow`,
+    )
   }
   if (eventName !== "pull_request" && eventName !== "pull_request_target") {
     throw new Error(`must be called only in a pull request`)
   }
-
   const githubToken = process.env.GITHUB_TOKEN
-
   if (!githubToken) {
     throw new Error(`missing githubToken`)
   }
-
   const githubRepository = process.env.GITHUB_REPOSITORY
   if (!githubRepository) {
     throw new Error(`missing process.env.GITHUB_REPOSITORY`)
   }
-
   const [repositoryOwner, repositoryName] = githubRepository.split("/")
   const pullRequestNumber = readPullRequestNumber()
-
   const runId = process.env.GITHUB_RUN_ID
   const runLink = runId
     ? {
@@ -30,7 +27,6 @@ export const readGitHubWorkflowEnv = () => {
         text: `${process.env.GITHUB_WORKFLOW || "workflow"}#${runId}`,
       }
     : undefined
-
   return {
     projectDirectoryUrl: process.env.GITHUB_WORKSPACE,
     githubToken,
@@ -46,10 +42,8 @@ const readPullRequestNumber = () => {
   if (!githubRef) {
     throw new Error(`missing process.env.GITHUB_REF`)
   }
-
   const pullRequestNumber = githubRefToPullRequestNumber(githubRef)
   if (pullRequestNumber) return pullRequestNumber
-
   // https://github.com/actions/checkout/issues/58#issuecomment-589447479
   const githubEventFilePath = process.env.GITHUB_EVENT_PATH
   if (githubEventFilePath) {
@@ -67,7 +61,6 @@ ${githubEventFilePath}
       return pullRequestNumber
     }
   }
-
   throw new Error(`cannot get pull request number from process.env.GITHUB_REF
 --- process.env.GITHUB_REF ---
 ${githubRef}`)
@@ -77,7 +70,9 @@ const githubRefToPullRequestNumber = (githubRef) => {
   const pullPrefix = "refs/pull/"
   const pullRequestNumberStartIndex = githubRef.indexOf(pullPrefix)
   if (pullRequestNumberStartIndex === -1) return undefined
-  const afterPull = githubRef.slice(pullRequestNumberStartIndex + pullPrefix.length)
+  const afterPull = githubRef.slice(
+    pullRequestNumberStartIndex + pullPrefix.length,
+  )
   const slashAfterPullIndex = afterPull.indexOf("/")
   if (slashAfterPullIndex === -1) return undefined
   const pullRequestNumberString = afterPull.slice(0, slashAfterPullIndex)