[sc-53104] Fix PR number parsing for multi-line commit messages (#146)

Author: Blacksmoke16

dist/index.js

@@ -277,6 +277,22 @@ describe('getMessageAuthor', () => {
       })
     })
 
+    describe('when the head commit message is multiple lines', () => {
+      beforeEach(() => {
+        github.context.payload.head_commit.message =
+          'My new Feature (#8987)\n' +
+          '  \n' +
+          '  Co-authored-by: Some Body <[email protected]>'
+      })
+
+      it('does not fallback on the merge queue user', async () => {
+        expect(await getMessageAuthor(octokit, slack)).toStrictEqual({
+          username: 'mdavis',
+          icon_url: 'github.com/mdavis'
+        })
+      })
+    })
+
     describe('when the request for PR info fails/does not include required info', () => {
       beforeEach(() => {
         ;(octokit.rest.pulls.get as unknown as jest.Mock).mockImplementation(

src/__tests__/getMessageAuthor.test.ts

@@ -126,7 +126,7 @@ async function getPullRequestMergerFromPushCommit(
     throw new Error('Unexpected push event payload (undefined head_commit).')
   }
 
-  const matches = commit.message.match(/\(#(\d+)\)$/)
+  const matches = commit.message.match(/\(#(\d+)\)$/m)
 
   if (!matches) {
     throw new Error(