bug: duplicates (#240)

src/main.ts

@@ -30,24 +30,35 @@ async function action(): Promise<void> {
     pull_number: context.payload.number
   })
 
+  // Note: Any of these checks can work
+  //   body.includes(markdown)
+  //   body.endsWith(markdown)
+  //   body.match(new RegExp(markdown))
+  //   !~body.indexOf(markdown)
+  //   !~body.search(markdown)
+
   // Exit/return if our markdown message is already present
-  let body = pullRequest.body || ''
-  if (body.includes(markdown)) {
+  const body = pullRequest.body || ''
+  if (body.endsWith(markdown)) {
     info('Markdown message is already present.  Exiting.')
     return
   }
 
-  // There have been issues with duplicate messages, so remove those
-  body = body.split(markdown)[0]
-
   // If we're here update the body
-  info('Description is being updated.')
-  await octokit.rest.pulls.update({
-    owner: context.repo.owner,
-    repo: context.repo.repo,
-    pull_number: context.payload.number,
-    body: body.concat(`\n\n${markdown}`)
-  })
+  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
+  }
+
+  // If here, something went wrong ...which seems to happen a lot
+  error('Unexpected result.  Please verify the action has performed correctly.')
 }
 
 // Run main function