Visitar URL original
⬆️ Update dependency pdf-to-png-converter to v4 by renovate[bot] · Pull Request #1548 · cigaleapp/cigale · GitHub
Skip to content

⬆️ Update dependency pdf-to-png-converter to v4 - #1548

Merged
gwennlbh merged 7 commits into
mainfrom
renovate/pdf-to-png-converter-4.x
May 4, 2026
Merged

gwennlbh merged 7 commits into
mainfrom
renovate/pdf-to-png-converter-4.x

Conversation

@renovate

@renovate renovate Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
pdf-to-png-converter ^3.18.0 → ^4.0.0 age confidence

Release Notes

dichovsky/pdf-to-png-converter (pdf-to-png-converter)

v4.0.0

Compare Source

Security
  • Pinned GitHub Actions to full commit SHAs to prevent supply-chain attacks via mutable tags
  • Added explicit permissions: contents: read to all workflows
  • Added npm audit --audit-level=high step to CI and publish workflows
Added
  • SECURITY.md with vulnerability disclosure policy
  • CHANGELOG.md (this file)
  • CONTRIBUTING.md with contributor guide
  • .nvmrc pinned to Node 22
  • .editorconfig for consistent editor settings across contributors
  • .github/dependabot.yml for automated weekly npm and GitHub Actions updates
  • docker-compose.yml for local Docker test runs
  • prepublishOnly script to prevent stale local publishes
  • Pre-commit hooks via husky + lint-staged (ESLint + Prettier on staged files)
  • license-checker as a pinned devDependency (replaces npx license-checker)
  • npm run test:license step added to CI
  • Coverage reports uploaded as workflow artifacts per Node version
  • pull_request trigger added to test workflow
Changed
  • Breaking (major): PngPageOutput is now a discriminated union with kind: 'metadata' | 'content' | 'file'; consumers should branch on kind before using mode-specific fields like path or content
  • Docker base image switched from node:22.19.0 to node:22.19.0-slim
  • Dockerfile now runs as non-root node user
  • Dockerfile converted to multi-stage build with dependency layer caching
  • NODE_ENV=test set in Dockerfile
  • dockerfile renamed to Dockerfile (standard casing)
  • CI matrix expanded to Node 20, 22, and 24
  • Coverage thresholds raised: lines: 90, functions: 90, branches: 85
  • MAX_VIEWPORT_SCALE and MAX_CANVAS_PIXELS moved from inline in pdfToPng.ts to const.ts
  • CMap and font paths now resolved at call time in propsToPdfDocInitParams instead of at module-load time
  • pdfjs-dist dynamic import cached at module level for repeated-call performance
  • All optional properties in README options table now marked with ?
Fixed
  • Wrong import path in props.to.pdf.doc.init.params.test.ts (../src/types/... → ../src/interfaces/...)
  • Split import type { Canvas} with missing space in canvas.and.context.ts merged into single import
Refactored
  • Extracted processAndSavePage helper to eliminate duplicated render+save logic in parallel and sequential paths
  • Eliminated redundant pngPagesOutput outer array in pdfToPng
  • Split processPdfPage into getPageMetadata (metadata-only path) and renderPdfPage (render path)
  • Converted pdf.to.file.test.js (JavaScript, importing from ../out) to TypeScript importing from ../src


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • Between 09:00 PM and 11:59 PM, only on Sunday (* 21-23 * * 7)
    • Between 12:00 AM and 04:59 PM, only on Monday (* 0-16 * * 1)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate
renovate Bot temporarily deployed to #1548: App preview May 4, 2026 06:26 Inactive
@renovate
renovate Bot temporarily deployed to #1548: Bundle breakdown May 4, 2026 06:37 Inactive
@github-actions

github-actions Bot commented May 4, 2026 •

Copy link
Copy Markdown
Contributor

Bundle Stats

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Total bundle size % Changed
228 46.66 MB
11.53 MB
0%
Changeset
File Δ Size
internal/client.js 🆕 +94 B 0 B → 94 B
load-utils/index.js 🔥 -608 B (-100%) 608 B → 0 B
View detailed bundle breakdown

Added

No assets were added

Removed

No assets were removed

Bigger

No assets were bigger

Smaller

No assets were smaller

@renovate
renovate Bot temporarily deployed to #1548: E2E tests May 4, 2026 06:46 Inactive
@github-actions

github-actions Bot commented May 4, 2026 •

Copy link
Copy Markdown
Contributor

Performance changes with main

Browser Test Metric Head Base Change
webkit startup @blank Total 7 984 ms 9 121,6 ms 🔻-1 137,6 ms
startup Total 8 925,2 ms 8 945,2 ms 🔻-20 ms
chromium startup DOM 579,32 556,06 🔺+23,26
Load 579,44 556,2 🔺+23,24
Bytes 13 627 517 13 627 517 0
LCP 6 664,8 6 479,2 🔺+185,6
TBT 1 893,6 1 896,6 🔻-3
CLS 0,02 0,02 🔻-0
Total 7 355 ms 7 211,2 ms 🔺+143,8 ms
startup @blank DOM 311,54 354,08 🔻-42,54
Load 311,74 354,3 🔻-42,56
Bytes 3 590 701 3 590 700 🔺+1
LCP 7 179,2 7 215,2 🔻-36
TBT 2 463 2 523,8 🔻-60,8
CLS 0,03 0,03 🔺+0
Total 7 813,4 ms 7 690,6 ms 🔺+122,8 ms

Total: total, DOM: domCompleteTiming, Load: pageloadTiming, Bytes: totalBytes, LCP: largestContentfulPaint, TBT: totalBlockingTime, CLS: cumulativeLayoutShift,

@gwennlbh
gwennlbh force-pushed the renovate/pdf-to-png-converter-4.x branch from 1414d9c to d1bcda8 Compare May 4, 2026 11:14
@renovate

renovate Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@gwennlbh
gwennlbh temporarily deployed to #1548: App preview May 4, 2026 11:17 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: Bundle breakdown May 4, 2026 11:25 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: E2E tests May 4, 2026 11:28 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: App preview May 4, 2026 12:24 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: Bundle breakdown May 4, 2026 12:32 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: E2E tests May 4, 2026 12:37 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: App preview May 4, 2026 13:10 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: Bundle breakdown May 4, 2026 13:18 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: E2E tests May 4, 2026 13:22 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: App preview May 4, 2026 13:39 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: Bundle breakdown May 4, 2026 13:47 — with GitHub Actions Inactive
@gwennlbh
gwennlbh marked this pull request as draft May 4, 2026 14:18
@gwennlbh
gwennlbh marked this pull request as draft May 4, 2026 14:18
@gwennlbh

gwennlbh commented May 4, 2026

Copy link
Copy Markdown
Member

@copilot seems like the google slides script step doesn't write images anymore. in the latest commit (done by protocols.yml workflow), it seems like every species that gets described by the Google Slides step now has an empty images array. Fix this (it may be related to the update to pdf-to-png converter)

Copilot AI commented May 4, 2026

Copy link
Copy Markdown
Contributor

@copilot seems like the google slides script step doesn't write images anymore. in the latest commit (done by protocols.yml workflow), it seems like every species that gets described by the Google Sli...

Fixed in d6eb3d8. The renovate bot introduced a backwards filter condition: kind !== 'content' was filtering out all the actual PNG content results (in v4, kind === 'content' is the type that carries the content buffer). Changed to kind === 'content' so only the content-type results are kept.

Copilot AI requested a review from gwennlbh May 4, 2026 14:24
Copilot AI temporarily deployed to #1548: App preview May 4, 2026 14:30 Inactive
@gwennlbh
gwennlbh force-pushed the renovate/pdf-to-png-converter-4.x branch from d6eb3d8 to b14970b Compare May 4, 2026 14:35
@gwennlbh
gwennlbh temporarily deployed to #1548: App preview May 4, 2026 14:39 — with GitHub Actions Inactive
@gwennlbh
gwennlbh temporarily deployed to #1548: Bundle breakdown May 4, 2026 14:47 — with GitHub Actions Inactive
@gwennlbh
gwennlbh marked this pull request as ready for review May 4, 2026 16:30
@gwennlbh
gwennlbh merged commit 25e7d19 into main May 4, 2026
47 of 51 checks passed
@gwennlbh
gwennlbh deleted the renovate/pdf-to-png-converter-4.x branch May 4, 2026 16:32
@blank

blank commented May 11, 2026 via email

Copy link
Copy Markdown

@gwennlbh

Copy link
Copy Markdown
Member

Please do not @ the "blank" account—thank you. My inbox is flooded with irrelevant emails. Blank github-actions[bot] @.>于2026年5月4日 周一14:47写道:
…
github-actions[bot] left a comment (cigaleapp/cigale#1548) <#1548 (comment)> Performance changes with main https://github.com/cigaleapp/cigale/tree/main Browser Test Metric Head Base Change [image: webkit] https://camo.githubusercontent.com/2928e826e004f0dd2f8dd6eb08046ae0fcb7f7876fe6dc17abfec3b453aab5de/68747470733a2f2f75706c6f61642e77696b696d656469612e6f72672f77696b6970656469612f66722f7468756d622f622f62652f5765624b69745f6c6f676f5f253238323031352532392e7376672f3132383070782d5765624b69745f6c6f676f5f253238323031352532392e7376672e706e67 startup @blank https://github.com/blank Total 8 900 ms 8 928,6 ms 🔻-28,6 ms startup Total 8 910,8 ms 8 855,8 ms 🔺+55 ms [image: chromium] https://camo.githubusercontent.com/b0b0dd79e78541d2869f80819be83c9ebbaba4d0c9f4cab38e1f0fc4df28196e/68747470733a2f2f75706c6f61642e77696b696d656469612e6f72672f77696b6970656469612f636f6d6d6f6e732f7468756d622f652f65312f476f6f676c655f4368726f6d655f69636f6e5f25323846656272756172795f323032322532392e7376672f3338343070782d476f6f676c655f4368726f6d655f69636f6e5f25323846656272756172795f323032322532392e7376672e706e67 startup DOM 563,92 538,04 🔺+25,88 Load 564,34 538,2 🔺+26,14 Bytes 13 627 517 13 627 517 0 LCP 6 420,8 6 609,6 🔻-188,8 TBT 1 808,4 1 884 🔻-75,6 CLS 0,02 0,02 🔺+0 Total 6 835,8 ms 7 239,8 ms 🔻-404 ms startup @blank https://github.com/blank DOM 358,42 329,12 🔺+29,3 Load 358,58 329,3 🔺+29,28 Bytes 3 590 700 3 590 700 0 LCP 7 156,8 7 317,6 🔻-160,8 TBT 2 494,2 2 585 🔻-90,8 CLS 0,03 0,03 🔻-0 Total 7 444,8 ms 7 901,2 ms 🔻-456,4 ms Total: total, DOM: domCompleteTiming, Load: pageloadTiming, Bytes: totalBytes, LCP: largestContentfulPaint, TBT: totalBlockingTime, CLS: cumulativeLayoutShift, — Reply to this email directly, view it on GitHub <#1548 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAXTD6WUX6S6FORL2NC4KD4ZA4GZAVCNFSM6AAAAACYPW7WFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DGNRYHAZDQNRUGA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.Message ID: @.
>

sorry, i didnt know it would send emails like that... it's Playwright's syntax for adding tags to tests lol

renovate Bot added a commit that referenced this pull request Aug 3, 2026
##### [vv7.0.0](https://github.com/actions/setup-node/releases/tag/v7.0.0)

##### What's Changed

##### Enhancements:

- Add cache-primary-key and cache-matched-key as outputs by [@gowridurgad](https://github.com/gowridurgad) in [#1577](actions/setup-node#1577)
- Migrate to ESM and upgrade dependencies by [@gowridurgad](https://github.com/gowridurgad) in [#1574](actions/setup-node#1574)

##### Bug fixes:

- Remove dummy NODE\_AUTH\_TOKEN export by [@gowridurgad](https://github.com/gowridurgad) in [#1558](actions/setup-node#1558)
- Only use `mirrorToken` in `getManifest` if it's provided by [@deiga](https://github.com/deiga) in [#1548](actions/setup-node#1548)

##### Documentation updates:

- Add documentation for publishing to npm with Trusted Publisher (OIDC) by [@chiranjib-swain](https://github.com/chiranjib-swain) in [#1536](actions/setup-node#1536)
- docs: Update restore-only cache documentation by [@priya-kinthali](https://github.com/priya-kinthali) in [#1550](actions/setup-node#1550)
- docs: Update caching recommendations to mitigate cache poisoning risks by [@chiranjib-swain](https://github.com/chiranjib-swain) in [#1567](actions/setup-node#1567)

##### Dependency update:

- Upgrade [@actions/cache](https://github.com/actions/cache) to 5.1.0, log cache write denied by [@jasongin](https://github.com/jasongin) in [#1569](actions/setup-node#1569)

##### New Contributors

- [@chiranjib-swain](https://github.com/chiranjib-swain) made their first contribution in [#1536](actions/setup-node#1536)
- [@deiga](https://github.com/deiga) made their first contribution in [#1548](actions/setup-node#1548)
- [@jasongin](https://github.com/jasongin) made their first contribution in [#1569](actions/setup-node#1569)

**Full Changelog**: <actions/setup-node@v6...v7.0.0>
##### [vv7](actions/setup-node@v6.5.0...v7.0.0)
gwennlbh pushed a commit that referenced this pull request Aug 3, 2026
##### [vv7.0.0](https://github.com/actions/setup-node/releases/tag/v7.0.0)

##### What's Changed

##### Enhancements:

- Add cache-primary-key and cache-matched-key as outputs by [@gowridurgad](https://github.com/gowridurgad) in [#1577](actions/setup-node#1577)
- Migrate to ESM and upgrade dependencies by [@gowridurgad](https://github.com/gowridurgad) in [#1574](actions/setup-node#1574)

##### Bug fixes:

- Remove dummy NODE\_AUTH\_TOKEN export by [@gowridurgad](https://github.com/gowridurgad) in [#1558](actions/setup-node#1558)
- Only use `mirrorToken` in `getManifest` if it's provided by [@deiga](https://github.com/deiga) in [#1548](actions/setup-node#1548)

##### Documentation updates:

- Add documentation for publishing to npm with Trusted Publisher (OIDC) by [@chiranjib-swain](https://github.com/chiranjib-swain) in [#1536](actions/setup-node#1536)
- docs: Update restore-only cache documentation by [@priya-kinthali](https://github.com/priya-kinthali) in [#1550](actions/setup-node#1550)
- docs: Update caching recommendations to mitigate cache poisoning risks by [@chiranjib-swain](https://github.com/chiranjib-swain) in [#1567](actions/setup-node#1567)

##### Dependency update:

- Upgrade [@actions/cache](https://github.com/actions/cache) to 5.1.0, log cache write denied by [@jasongin](https://github.com/jasongin) in [#1569](actions/setup-node#1569)

##### New Contributors

- [@chiranjib-swain](https://github.com/chiranjib-swain) made their first contribution in [#1536](actions/setup-node#1536)
- [@deiga](https://github.com/deiga) made their first contribution in [#1548](actions/setup-node#1548)
- [@jasongin](https://github.com/jasongin) made their first contribution in [#1569](actions/setup-node#1569)

**Full Changelog**: <actions/setup-node@v6...v7.0.0>
##### [vv7](actions/setup-node@v6.5.0...v7.0.0)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

This branch was successfully deployed

3 active (1 outdated) deployments
#1548: Bundle breakdown — 2e0b9e84 Deployed May 4, 2026 by gwennlbh via Bundle size / Breakdown site / Deploy #6870
#1548: App preview — 2e0b9e84 Deployed May 4, 2026 by gwennlbh via PR Preview / Deploy / Deploy #6870
#1548: E2E tests — f732bde5 Deployed May 4, 2026 by gwennlbh via E2E / Upload report / Deploy #6859
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants