[Bug] Search for changed files is limited to the first 30 files in the PR

[alexdupre]

Describe the bug

We noticed that the action wasn't optimizing all images added in the PR. The reason is likely attributable to the missing pagination here: https://github.com/calibreapp/image-actions/blob/main/src/get-changed-images.ts#L23-L29

Octokit documentation: https://octokit.github.io/rest.js/v22/#pagination

How To Reproduce

Create a Pull Request with more than 30 added images

Screenshots

No response

Relevant Log Output

Found 30 images to process: ...

Code of Conduct

• I agree to follow this project’s Code of Conduct

[benschwarz]

Ah, nice @alexdupre, I hadn't noticed this with my test set. Thanks for logging. This is a bug!

[alexdupre]

I haven't tested it, but the fix might be as simple as:

--- a/src/get-changed-images.ts
+++ b/src/get-changed-images.ts
@@ -22,7 +22,7 @@ const getChangedImages = async (): Promise<string[] | null> => {

     core.info(`Fetching changed files for PR #${pullNumber}…`)

-    const { data: files } = await api.rest.pulls.listFiles({
+    const files = await api.paginate(api.rest.pulls.listFiles, {
       owner,
       repo,
       pull_number: pullNumber