Merge pull request #1 from ljetten/feat/mshick#93-trim-message

trim message when exceeds maximum characters

__tests__/add-pr-comment.test.ts

@@ -12,6 +12,7 @@ import apiResponse from './sample-pulls-api-response.json'
 const messagePath1Fixture = path.resolve(__dirname, './message-part-1.txt')
 const messagePath1FixturePayload = await fs.readFile(messagePath1Fixture, 'utf-8')
 const messagePath2Fixture = path.resolve(__dirname, './message-part-2.txt')
+const messagePathTooLongFixture = path.resolve(__dirname, './message-too-long.txt')
 
 const repoToken = '12345'
 const commitSha = 'abc123'
@@ -205,6 +206,19 @@ describe('add-pr-comment action', () => {
     expect(core.setOutput).toHaveBeenCalledWith('comment-id', postIssueCommentsResponse.id)
   })
 
+  it('creates a trimmed comment with a message-path', async () => {
+    inputs.message = undefined
+    inputs['message-path'] = messagePathTooLongFixture
+    inputs['allow-repeats'] = 'true'
+
+    let endOfMessage = "...";
+
+    await expect(run()).resolves.not.toThrow()
+    expect(endOfMessage).toEqual(messagePayload?.body.slice(-3))
+    expect(core.setOutput).toHaveBeenCalledWith('comment-created', 'true')
+    expect(core.setOutput).toHaveBeenCalledWith('comment-id', postIssueCommentsResponse.id)
+  })
+
   it('supports globs in message paths', async () => {
     inputs.message = undefined
     inputs['message-path'] = `${path.resolve(__dirname)}/message-part-*.txt`

src/message.ts

@@ -57,6 +57,7 @@ export async function getMessage({
 
 export async function getMessageFromPath(searchPath: string) {
   let message = ''
+  const maxCharacterLength = 65536
 
   const files = await findFiles(searchPath)
 
@@ -68,7 +69,10 @@ export async function getMessageFromPath(searchPath: string) {
     message += await fs.readFile(path, { encoding: 'utf8' })
   }
 
-  return message
+  // return trimmed message if message is too long (maximum is 65536 characters)
+  return message.length > maxCharacterLength
+    ? message.substring(0, maxCharacterLength - 3) + '...'
+    : message
 }
 
 export function addMessageHeader(messageId: string, message: string) {