Skip to content

feat: use pypdfium2 to split pdf instead of pypdf - #327

Merged
awalker4 merged 6 commits into
mainfrom
feat/use-pypdfium2-to-split-page
Feb 3, 2026
Merged

awalker4 merged 6 commits into
mainfrom
feat/use-pypdfium2-to-split-page

Conversation

@badGarnet

Copy link
Copy Markdown
Collaborator

This PR replaces pypdf with pypdfium2 when splitting a pdf file into chunks. This a more robust and faster library, avoiding occasional RecursionError that can happen with pypdf.

@socket-security

socket-security Bot commented Feb 3, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedpypi/​pypdfium2@​5.3.099100100100100

View full report

@badGarnet
badGarnet requested a review from awalker4 February 3, 2026 01:10

@awalker4 awalker4 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Although heads up that pyproject is autogenerated - you'll want to set the version and requirements list in gen.yaml

@badGarnet
badGarnet enabled auto-merge (squash) February 3, 2026 02:25
@awalker4
awalker4 disabled auto-merge February 3, 2026 16:28
@awalker4
awalker4 merged commit 8e5ae08 into main Feb 3, 2026
24 of 31 checks passed
@awalker4
awalker4 deleted the feat/use-pypdfium2-to-split-page branch February 3, 2026 16:28
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.

2 participants