chore(ci): refactor (#315)

Author: DerekRoberts

src/main.ts

@@ -12,47 +12,52 @@ if (!markdown || !token || !limit_to_pr_opened) {
 
 // Main function
 async function action(): Promise<void> {
-  // If limit_to_pr_opened is true, then verify status (opened, reopened)
-  const trigger = JSON.stringify(context.payload.action) || ''
+  // If limit_to_pr_opened is true, verify status (opened, reopened)
+  const trigger = context.payload.action || ''
   const statuses = ['opened', 'reopened']
-  if (limit_to_pr_opened === 'true' && statuses.includes(trigger)) {
-    info('PR not opened or reopened with limit_to_pr_opened=true.  Exiting.')
+  if (limit_to_pr_opened === 'true' && !statuses.includes(trigger)) {
+    info('PR not opened or reopened with limit_to_pr_opened=true. Exiting.')
     return
   }
 
-  // Get PR body from GitHub context (previously using octokit)
-  const body = context.payload.pull_request?.body || ''
+  // Ensure pull request exists
+  if (!context.payload.pull_request) {
+    error('Error: No pull request found in context. Exiting.')
+    return
+  }
 
-  // Note: Any of these checks can work
-  //   body.includes(markdown)
-  //   body.endsWith(markdown)
-  //   body.match(new RegExp(markdown))
-  //   !~body.indexOf(markdown)
-  //   !~body.search(markdown)
+  // Get PR body from GitHub context
+  const body = context.payload.pull_request?.body || ''
 
   // If message is already present, then return/exit
-  if (body.endsWith(markdown)) {
-    info('Markdown message is already present.  Exiting.')
+  if (body.includes(markdown)) {
+    info('Markdown message is already present. Exiting.')
     return
   }
 
-  // If not present, then append
-  const octokit = getOctokit(token)
-  if (!body.endsWith(markdown)) {
-    info('Description is being updated.')
-    await octokit.rest.pulls.update({
-      owner: context.repo.owner,
-      repo: context.repo.repo,
-      pull_number: context.payload.number,
-      // Split out any duplicate messages, has been an issue
-      body: body.split(markdown)[0].concat(`\n\n${markdown}`)
-    })
-    return
-  }
+  // Append markdown after removing existing duplicates
+  const updatedBody = `${body
+    .split('\n')
+    .filter(line => line.trim() !== markdown.trim())
+    .join('\n')
+    .trim()}\n\n${markdown}`
 
-  // If here, kick up an error
-  error('Unexpected result.  Please verify the action has performed correctly.')
+  // Update PR body
+  const octokit = getOctokit(token)
+  info('Description is being updated.')
+  await octokit.rest.pulls.update({
+    owner: context.repo.owner,
+    repo: context.repo.repo,
+    pull_number: context.payload.pull_request.number,
+    body: updatedBody
+  })
 }
 
 // Run main function
-action()
+;(async () => {
+  try {
+    await action()
+  } catch (err) {
+    error(`Unexpected error: ${err instanceof Error ? err.message : err}`)
+  }
+})()