error: body is too long (maximum is 65536 characters) about add-pr-comment HOT 5 OPEN

hey @mshick, have you made any progress on that? Happy to help if you don't have the bandwidth.

from add-pr-comment.

Interesting issue. I intend to address this by checking the message size and breaking the message into parts if we hit the limit.

I'll make this behavior optional, the default will be to truncate the message with ... to avoid the error.

from add-pr-comment.

It seems that this 65536 chars is a hard limit https://github.com/orgs/community/discussions/41331 on the gh api

one very interesting this is that this is actually misleading, if you see the following run in the same repo of mine, i am able to pass a 200 KB txt file inside which is higher than 65536 chars (~65 KB)

Links

• 200 KB comment
• Corresponding run

Screenshot

from add-pr-comment.

We can probably have a function like below to splits the comment into multiple comments based on maxSize which can be 65536(though in actuality it seems to extend till ~200 KB) and sepEnd and sepStart variables can say what must be the separator when the comment gets split into multiple comments

example of sepStart - Continued from previous comment
example of sepEnd - Output length greater than max comment size. Continued in next comment.

The return value of this can be used to make multiple calls to the create comment API. Same can also probably bedone for the message-path and message-paths variants

function SplitComment(comment, maxSize, sepEnd, sepStart) {
  if (comment.length <= maxSize) {
    return [comment];
  }

  const maxWithSep = maxSize - sepEnd.length - sepStart.length;
  const comments = [];
  const numComments = Math.ceil(comment.length / maxWithSep);

  for (let i = 0; i < numComments; i++) {
    const start = i * maxWithSep;
    const end = Math.min(comment.length, (i + 1) * maxWithSep);
    let portion = comment.substring(start, end);

    if (i < numComments - 1) {
      portion += sepEnd;
    }
    if (i > 0) {
      portion = sepStart + portion;
    }

    comments.push(portion);
  }

  return comments;
}

from add-pr-comment.

Oops, this looks misleading due to me merging a PR on a fork I made to check the tests would pass.
The one line above this comment is the PR I submitted to the actual repository..

from add-pr-comment.